Repository navigation
Conversation
Keep one routing contract while reducing the content read for setup, run, and review. Preserve the existing action order and trigger aliases. Refs ai-driven-dev#420
Make the route action follow explicit conflict outcomes and record baseline parity scenarios so reviewers can trace selection rules without a duplicate resolver. Refs ai-driven-dev#420
Keep the review rounds and outcome challenge with the ai-driven-dev#420 delivery evidence, including the runtime verification limit. Refs ai-driven-dev#420
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
🎯 What & why
Make
00-async-deva small router that loads only the selected setup, run, or review sequence. This addresses #420's always-loaded routing detail while preserving the documented trigger and action contracts.🛠️ How it works
SKILL.mdnow contains the flow, action table, and transversal rules.actions/route.mdreads the single detailed routing reference, then opens one mode sequence.review; blocking conflicts stop.🧪 How to verify
node --test scripts/__tests__/async-dev-router.test.jsandnode --test scripts/__tests__/skill-argument-hints.test.js. Expected: 6/6 and 5/5 passing. Both passed locally.node scripts/check-architecture-rules.jsandnode scripts/check-markdown-links.js. Expected: no violations or broken links. Locally: 358 governed files and 0 broken links in 985 files.evidence.md. Expected: review 424 → 364 and setup 796 → 750.The full
scripts-testspre-commit suite was interrupted after more than eight minutes in unrelated telemetry-hook tests. Thecli-testpre-push suite was interrupted after more than ten minutes; this PR changes nocli/files. Both gates remain for CI. Other pre-commit gates andcli-knippassed locally.🔗 Linked issue
Closes #420
✅ I certify