feat(bmad): 6.10 compatibility hardening + quick-dev refresh#8
Merged
Conversation
Verified against the real bmad-method@6.10.0 package; all six invoked skills present, invoked surface byte-stable. Compatibility hardening: - config.toml tolerance: new _Sprintpilot/lib/runtime/bmad-output.js (dedupes 3 copy-pasted readers) prefers config.yaml, falls back to the v6.4+ _bmad/config.toml / config.user.toml; installer mirror in lib/core/bmad-config.js. Confirmed necessary — 6.10 writes config.toml as the primary config. - soft version guard: checkBmadCompat() warns (never blocks) at install when BMad is below floor or above tested-through. - sprint-status resilience: readers verified robust to v6.9 retrospective action-items (top-level + nested); helpers exposed for tests. - code-review heading tolerance: verifyCodeReview accepts v6.10 heading variants (heading confirmed still "### Review Findings"). - memlog non-conflict: docs note + static guard test (Sprintpilot never writes under _bmad/, does not adopt memlog). - manifest: tested-through -> v6.10.0 (floor stays >=6.2.1) + parseable bmad_tested_through field. Quick-dev (nano) refresh: - lock-in test that verifyNanoQuickDev requires terminal 'done' even as 6.10 quick-dev self-syncs sprint-status through in-progress/review/done. Docs (design): - docs/bmad-dev-auto-evaluation.md — dev-auto/bmad-loop spike: stay differentiated, compose later; never run bmad-loop concurrently. - docs/quick-dev-fast-lane-plan.md — approved design for a per-story fast lane (build tracked separately). Tests: 2267 pass (+33), biome clean. Co-Authored-By: Claude Opus 4.8 (1M context) <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 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.
Makes Sprintpilot correct against BMAD Method v6.10 (was tested-through v6.8), verified against the real
bmad-method@6.10.0package. All six invoked skills present; invoked surface byte-stable.Compatibility hardening
_Sprintpilot/lib/runtime/bmad-output.js(dedupes 3 copy-pastedoutput_folderreaders) prefersconfig.yaml, falls back to the v6.4+_bmad/config.toml/config.user.toml; installer mirror inlib/core/bmad-config.js. Confirmed necessary — the 6.10 installer writesconfig.tomlas the primary config.checkBmadCompat()warns (never blocks) at install when BMad is below floor or above tested-through.verifyCodeReviewaccepts v6.10 heading variants (heading confirmed still### Review Findings; kept strictfindings[]contract).>=6.2.1) + parseablebmad_tested_through.Quick-dev (nano) refresh
verifyNanoQuickDevrequires terminaldoneeven as 6.10 quick-dev self-syncs sprint-status throughin-progress/review/done.Docs (design)
docs/bmad-dev-auto-evaluation.md— dev-auto/bmad-loop spike (stay differentiated; never run bmad-loop concurrently).docs/quick-dev-fast-lane-plan.md— approved design for a per-story fast lane (build tracked separately).Verification
cd tests && npm test→ 2267 passed (+33), 0 failuresnode --checkcleanVersion bump/publish intentionally deferred to
/release.🤖 Generated with Claude Code