feat(quick-dev): experimental combined execution mode (WIP)#30
Draft
alexeyv wants to merge 1 commit into
Draft
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Introduces `quick-dev` as an alternate child step alongside the existing
split create/dev/auto/review flow. In quick-dev mode a single Quick Dev
child session handles planning, coding, test generation, and review for
a story, while the orchestrator still verifies completion against the
story/sprint-status done gate before commit.
- Add `quick-dev` step to orchestration-policy.json with the
review_completion verifier, plus prompt template and parse schema.
- Thread `executionMode` ("split" | "quick-dev") through preflight,
state document, and Story Progress table (new quick-dev column).
- Extend agent resolution and runtime policy to recognise quick-dev as
a valid task/step name.
- Add Quick Dev branch to step-03 and execution-mode-aware row updates
in step-03a/03b.
- Update README, cli-reference, scripts-reference, workflow-commands,
smoke-test, and unit tests to cover the new mode.
Does not replace the main automator loop; quick-dev is opt-in via the
executionMode override.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
9d7a67d to
cdbb65d
Compare
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.
Summary
quick-devas an experimental alternate child step alongside the existing split create/dev/auto/review flow. A single Quick Dev child session handles planning, coding, test generation, and review; Automator still verifies the story/sprint-status done gate before commit.overrides.executionMode(split|quick-dev) through preflight, the state document template, and the Story Progress table (adds aquick-devcolumn).orchestrator.py,orchestrator_epic_agents.py,agent_config.py), runtime policy validation, and orchestration policy (data/orchestration-policy.json) withreview_completionas the success verifier.data/prompts/quick-dev.mdprompt template anddata/parse/quick-dev.jsonparse schema.step-03-execute.mdwith a Quick Dev branch that bypasses sections A/B when the mode is active, and makesstep-03a/step-03brow updates execution-mode aware.test_agent_config_model,test_runtime_policy,test_success_verifiers) to cover the new mode.This does not replace the main automator loop — quick-dev is strictly opt-in via the
executionModeoverride and abmad-quick-devskill must be installed.Motivation: in local trials the combined Quick Dev session has been roughly 2x faster and used about half the tokens of the split flow. Once it has soaked in, it may become the default, but for now it ships side-by-side.
Status / known gaps
bmad-quick-devskill itself is upstream of this repo; this PR only adds the Automator-side integration.Test plan
PYTHONPATH=skills/bmad-story-automator/src python -m unittest tests.test_agent_config_model tests.test_runtime_policy tests.test_success_verifiers(currently 122 tests OK locally)scripts/smoke-test.shrun with abmad-quick-devskill installedoverrides.executionMode: quick-devsplitmode to confirm no regression in the existing flowquick-devcolumn)