Skip to content

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

Merged
pelikhan merged 1 commit into
mainfrom
rig-claude-compat/2026-07-30-3dfbea1bbc975198
Jul 30, 2026
Merged

[rig-claude] Improve Claude dynamic-workflow compatibility for rig#300
pelikhan merged 1 commit into
mainfrom
rig-claude-compat/2026-07-30-3dfbea1bbc975198

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Compatibility gap addressed

The primitive mapping table in skills/rig/references/claude-workflow-conversion.md was missing the until() convergence primitive. A developer porting a Claude dynamic workflow that uses an open-ended while loop for bounded retries or convergence (e.g. "keep fixing until tests pass, up to 8 rounds") would not discover until() and might write an unbounded loop instead.

Why this improves transfer

until is the exact rig idiom for the "bounded convergence loop" pattern that is common in Claude dynamic workflows. The table already covered every other major primitive (parallel, pipeline, phase, log, budget, nested workflow, schema-constrained calls, per-call options). Adding until closes the last visible gap in the conversion surface.

Files changed

  • skills/rig/references/claude-workflow-conversion.md — added one row to the primitive mapping table:

    | open-ended while convergence loop | until({ max, noProgressRounds }, step) | Bounded; stops on done, after max rounds, or after noProgressRounds equal progress keys — prefer over unbounded loops |

Validation

Only documentation changed. Verified:

  • No broken relative links (existing links to dynamic-workflows.md, agent-api.md, and runtime.md unchanged).
  • wc -l went from 155 → 156 lines (one added row).

Remaining intentional differences

All previously documented differences are unchanged:

  • { effort: "high" } has no rig equivalent (encode structurally or in model id)
  • { agentType: "Explore" } has no built-in read-only agent type
  • No sandbox restrictions, resume journal, worktree isolation, or human checkpoints

Generated by Daily Rig Claude Dynamic Workflow Compatibility · sonnet46 35.6 AIC · ⌖ 5.04 AIC · ⊞ 5.4K ·

…n table

The primitive mapping table in claude-workflow-conversion.md was missing
the until() convergence primitive, which is the idiomatic rig replacement
for open-ended while loops in Claude dynamic workflows. Without this entry,
developers porting a workflow that uses a bounded retry/convergence pattern
would not discover until() and might write an unbounded loop instead.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@pelikhan
pelikhan marked this pull request as ready for review July 30, 2026 10:40
@pelikhan
pelikhan merged commit b1ae011 into main Jul 30, 2026
3 checks passed
@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

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

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Skills-Based Review 🧠

Applied /grill-with-docs — this is a clean, accurate documentation addition.

📋 Key Themes & Highlights

Positive Highlights

  • ✅ The new row correctly identifies until({ max, noProgressRounds }, step) as the rig idiom for bounded convergence loops
  • ✅ Placement is consistent — inserted between the workflow() row and top-level return, which is the logical location for a loop primitive
  • ✅ The notes column accurately summarises the three stopping conditions (done, max, noProgressRounds) matching dynamic-workflows.md
  • ✅ The "prefer over unbounded loops" guidance aligns with how until is presented in the reference docs

No issues found. The addition closes the last visible gap in the primitive mapping table.

🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · sonnet46 21.9 AIC · ⌖ 4.72 AIC · ⊞ 6.3K
Comment /matt to run again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant