Uh oh!
There was an error while loading. Please reload this page.
docs(skill): teach the automation skill the ADR-0044 send-back-for-revision shape - #2294
Merged
Merged
Conversation
…vision shape The objectstack-automation skill documented approve/reject approvals (ADR-0019) but not the ADR-0044 revise loop, so AI-authored approval flows omit it (and a revise loop hand-built without the back-edge gets rejected by registerFlow). Add a "Send-back for revision" section to the Approvals area: the `revise` out-edge → a signal `wait` node, the resubmit edge typed `type: 'back'` (the only thing that legalizes the cycle — graph-minus-back-edges must be a DAG, authors opt in edge by edge), and the `maxRevisions` budget. Calls out the two shapes the compile-time flow lint flags (dead-end revise; unmarked back-edge) and points at the canonical showcase_budget_approval. Completes the agent-teaching slice of #2274 (ADR-0044), alongside the compile-time lint (#2279), the flow-builder `back` edge style (#2291), and the engine's already-specific cycle-rejection hint. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Uh oh!
There was an error while loading. Please reload this page.
This was referenced Jun 24, 2026
os-zhuang added a commit
that referenced
this pull request
Jun 24, 2026
) Adds skills/objectstack-automation/evals/approvals/test-revise-loop.md, the first concrete eval under the skills' (previously placeholder) evals/ format — Scenario / Expected Output / Common Mistakes / Validation Criteria. It pins the expected send-back-for-revision shape (revise branch + signal wait + type:'back' resubmit + maxRevisions), the three footguns the compile-time lint flags, and registerFlow- based pass criteria. This is the authorable (a) half of #2274's eval slice. The executable eval RUNNER (b) remains a separate platform project — the skills' evals/ harness is still "not yet implemented" per each dir's README. Refs #2274, #1770 (ADR-0044). Builds on the lint (#2279) and skill (#2294). Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Completes the agent-teaching slice of #2274 (ADR-0044). The
objectstack-automationskill — how the AI agent learns to author flows in this repo — documented approve/reject approvals (ADR-0019) but not the revise loop, so AI-authored approval flows omit send-back (and a revise loop built without the back-edge gets rejected byregisterFlow).Adds a Send-back for revision (ADR-0044) section to the Approvals area:
reviseout-edge → a signalwaitnode,type: 'back'(the only thing that legalizes the cycle — graph-minus-back-edges must be a DAG; authors opt in edge by edge, per the ADR),maxRevisionsbudget (default 3;0disables),showcase_budget_approval.ADR-aligned (teaches correct opt-in — no auto-mutation, which would contradict ADR-0044 D5). Markdown-only;
check:doc-authoring✓ andcheck:skill-docsin-sync ✓ (frontmatter unchanged, so no generated-doc regen).This closes the practical scope of #2274: lint #2279 + spec #2291 + this skill, with the engine's cycle-rejection error already carrying the specific hint. Remaining (optional): a dedicated agent eval. Refs #2274, #1770. 🤖 Generated with Claude Code