Skip to content

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

Merged
pelikhan merged 1 commit into
mainfrom
rig-claude-compat/2026-08-21-7ba4a8fccd14a5cc
Aug 22, 2026
Merged

[rig-claude] Improve Claude dynamic-workflow compatibility for rig#467
pelikhan merged 1 commit into
mainfrom
rig-claude-compat/2026-08-21-7ba4a8fccd14a5cc

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Compatibility gap addressed

skills/rig/references/dynamic-workflows.md had a thin "Porting from Claude dynamic workflows" section that only pointed to claude-workflow-conversion.md with no sample table. A user landing on this reference first had no direct path to the concrete port examples — they would need to navigate to the conversion reference to discover them.

claude-workflow-conversion.md already contains a well-structured ## Example programs table with 7 samples covering every major Claude dynamic-workflow primitive. This PR adds the identical table to the end of dynamic-workflows.md.

Why this improves transfer from Claude dynamic workflows to Rig

Claude dynamic-workflow users searching for parallel, pipeline, phase, log, budget, call.workflow, or anthropicEngine land on dynamic-workflows.md via the SKILL.md routing table. Without the sample table they had no direct pointer to the canonical port examples. Adding the table makes the samples discoverable from both reference documents.

Files changed

  • skills/rig/references/dynamic-workflows.md — added ## Example programs table (15 lines)

Validation

Docs-only change. No build, typecheck, or test run required per the workflow validation rules.

Remaining intentional differences

None introduced. All behavior differences between rig and Claude dynamic workflows are already documented in claude-workflow-conversion.md under "Behavior differences to keep in mind".

Generated by Daily Rig Claude Dynamic Workflow Compatibility · sonnet46 57.8 AIC · ⌖ 8.01 AIC · ⊞ 5.4K ·

Adds the same sample cross-reference table that claude-workflow-conversion.md
already has, so users who land on dynamic-workflows.md first can immediately
find the direct-port samples without navigating to the conversion reference.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@pelikhan
pelikhan marked this pull request as ready for review August 22, 2026 16:08
@pelikhan
pelikhan merged commit 81cce24 into main Aug 22, 2026
1 check passed
@github-actions

github-actions Bot commented Aug 22, 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 /codebase-design and /grill-with-docs — changes are solid, approving.

📋 Key Themes & Highlights

Positive Highlights

  • ✅ All 7 referenced sample files exist and are correctly linked
  • ✅ Content is an accurate duplicate of the already-validated table in claude-workflow-conversion.md
  • ✅ Improves discoverability — users landing on dynamic-workflows.md via SKILL.md routing now get direct sample pointers without an extra navigation hop
  • ✅ No new vocabulary introduced; terminology stays consistent with existing references

Assessment

The change is minimal, correct, and directly addresses the discoverability gap described in the PR. No architectural concerns — duplicating a navigation aid across two reference files is an intentional and appropriate documentation pattern here.

🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · sonnet46 18.2 AIC · ⌖ 6.89 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