[rig-claude] Improve Claude dynamic-workflow compatibility for rig

[github-actions]

Compatibility gap addressed

Sample 330-nested-workflow-composition.md used Promise.all directly to fan out call.workflow invocations in parallel, but the canonical Claude dynamic-workflow idiom — and the idiomatic rig equivalent — is parallel(thunks).

A user porting await parallel(thunks) from a Claude dynamic workflow would:

1. Not recognise the Promise.all pattern as the rig translation.
2. Lose the error-to-null-hole semantics that parallel() provides (rejected thunks become null rather than propagating, except WorkflowLimitError which still propagates).

Why this improves transfer

The mapping in claude-workflow-conversion.md documents parallel(thunks) → parallel(thunks) as a direct equivalence. The sample now uses exactly that form, reinforcing the mapping and making it immediately recognisable to someone porting from Claude dynamic workflows.

Files changed

• skills/rig/samples/330-nested-workflow-composition.md — replaced Promise.all(dirs.map(...)) with parallel(dirs.map((dir) => () => ...)) and added parallel to the destructured workflow context.

Validation run

npm run sample -- --testNamePattern="skill markdown samples typecheck"

Result: 1 passed (402 skipped) ✓

Remaining intentional differences

None introduced. The existing documented differences in claude-workflow-conversion.md (failure holes, budget units, no sandbox restrictions, no resume journal) remain accurate and unchanged.

Generated by Daily Rig Claude Dynamic Workflow Compatibility · sonnet46 41.2 AIC · ⌖ 7.92 AIC · ⊞ 5.4K · ◷

[pelikhan]

@copilot review and replace Promise.all in other samples

[Copilot]

@copilot review and replace Promise.all in other samples

Updated additional samples to replace Promise-based fan-out with parallel(thunks) where applicable. Included executable sample src/samples/54-large-scale-summarization-rigs.ts and aligned wording in skills/rig/samples/54-large-scale-summarization.md. Addressed in commit a420e9f.

[github-actions]

🧠 Matt Pocock Skills Reviewer has completed the skills-based review. ✅

[github-actions]

Skills-Based Review 🧠

Applied /codebase-design — changes align with rig vocabulary and patterns. Approving.

Key themes:

• parallel(thunks) alignment: All changed files now consistently use parallel(thunks) instead of Promise.all, reinforcing the canonical rig idiom and the Claude-workflow mapping.
• Null-hole semantics handled: 54-large-scale-summarization-rigs.ts filters null results after parallel with a correctly typed NonNullable predicate, preventing downstream type errors.
• searchPlanPromise sequencing: Moving the await outside parallel([...]) is safe — concurrency is preserved since the promise is already launched. The sequenced await also makes the data dependency on searchPlan explicit.

✅ Minimal, surgical diff. ✅ Sample typechecks cleanly.

🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · sonnet46 20.8 AIC · ⌖ 4.07 AIC · ⊞ 6.3K
_{Comment /matt to run again}