docs(roadmap): add Phase 2 gap analysis stories P/Q/R/S/T#337
Merged
MScottAdams merged 1 commit intomasterfrom Apr 13, 2026
Merged
docs(roadmap): add Phase 2 gap analysis stories P/Q/R/S/T#337MScottAdams merged 1 commit intomasterfrom
MScottAdams merged 1 commit intomasterfrom
Conversation
- Story P (#331): Ancillary doc updates for vBRIEF-centric model (Tier 2) - Story Q (#332): Go installer rewrite for deft-directive-* namespace (Tier 3) - Story R (#333): Validation tooling for vBRIEF-centric model (Tier 1) - Story S (#334): Pre-cutover detection and backward compatibility guard (Tier 4) - Story T (#335): Release and version boundary items v0.20.0 (Tier 5) - Updated Story F (#315) description: installer scope moved to Story Q - Updated Story G (#316) description: added dependency note on Stories B + O - Updated Story C (#312) description: expanded migration scope - Added all 5 new stories to Open Issues Index
Contributor
Greptile SummaryThis PR adds 5 gap-analysis stories (P–T, issues #331–#335) to the Phase 2 vBRIEF Architecture Cutover roadmap, updates descriptions for Stories C, F, and G, and adds 5 entries to the Open Issues Index. All changes are documentation-only in Confidence Score: 5/5Safe to merge — documentation-only change with no code impact. All five new story entries are internally consistent, the Open Issues Index rows are correctly numbered and formatted, and the existing story description updates accurately reflect scope movement. Both inline comments are P2 suggestions about sequencing nuances, not correctness blockers. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
T1["Tier 1 — Foundation (parallelizable)\n#310 Story A · #311 Story B · #312 Story C\n#323 Story N · #324 Story O\n#333 Story R: Validation tooling (new)"]
T2["Tier 2 — Framework Docs (depends on A)\n#313 Story D\n#331 Story P: Ancillary doc updates (new)"]
T3["Tier 3 — Skills (depends on A + D; parallelizable)\n#314 Story E · #317 Story H · #318 Story I · #319 Story J\n#315 Story F · #316 Story G\n#332 Story Q: Go installer rewrite (new)"]
T4["Tier 4 — CLI & Tests (depends on all above)\n#320 Story K · #321 Story L\n#334 Story S: Pre-cutover detection (new)"]
T5["Tier 5 — Post-Cutover\n#322 Story M\n#335 Story T: Release & version boundary (new)"]
T1 --> T2
T2 --> T3
T3 --> T4
T4 --> T5
Prompt To Fix All With AIThis is a comment left during a code review.
Path: ROADMAP.md
Line: 31
Comment:
**Story R tier placement may warrant a sequencing note**
Story R's scope (scope validator, PROJECT-DEFINITION validator, folder/status consistency, epic-story links) is functionally defined by Story A's output — the validation schemas and folder conventions can't be fully spec'd until Story A lands. Placing R in the same parallelizable Tier 1 is workable but may lead to iterative rework if A's model evolves mid-implementation. Consider either noting an implicit "design-after-A" sequencing expectation in the story description, or bumping R to Tier 2 where Story D already sits for model-dependent work.
How can I resolve this? If you propose a fix, please make it concise.
---
This is a comment left during a code review.
Path: ROADMAP.md
Line: 42
Comment:
**Story Q placed in same tier as the skills it enumerates**
Story Q lists `allSkillNames` as a deliverable, but the canonical names for the `deft-directive-*` skills are being finalized in the same Tier 3 (Stories E, F, G, H, I, J). Since all Tier 3 items are marked "parallelizable via swarm," Story Q could lock in skill names before any of the rename/rewrite stories converge. Worth adding a note on Story Q (or this entry) that the `allSkillNames` list should be treated as draft until all sibling Tier 3 stories confirm their final identifiers.
How can I resolve this? If you propose a fix, please make it concise.Reviews (1): Last reviewed commit: "docs(roadmap): add Phase 2 gap analysis ..." | Re-trigger Greptile |
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
Add 5 new stories to Phase 2 (vBRIEF Architecture Cutover) identified through gap analysis of the existing story set. Updates tier placements and Open Issues Index.
Changes
New stories added to ROADMAP.md Phase 2 tiers:
Updated existing story descriptions:
Also filed (not in this PR):
GitHub issue updates (outside this PR):
Checklist
Conversation