Skip to content

fix(phase-complete): improve REQUIREMENTS.md traceability updates#917

Open
ashanuoc wants to merge 1 commit intogsd-build:mainfrom
ashanuoc:fix/848-requirements-traceability-improvements
Open

fix(phase-complete): improve REQUIREMENTS.md traceability updates#917
ashanuoc wants to merge 1 commit intogsd-build:mainfrom
ashanuoc:fix/848-requirements-traceability-improvements

Conversation

@ashanuoc
Copy link

@ashanuoc ashanuoc commented Mar 3, 2026

Summary

Fixes #848 — improves REQUIREMENTS.md traceability updates:

  • Scoped regex: Phase section extraction now uses a bounded regex that stops at the next phase header, preventing cross-phase boundary matching (e.g., Phase 1 without Requirements won't accidentally pick up Phase 2's requirements)
  • In Progress status: Traceability table now handles both and statuses (previously only was converted to )
  • Result field: Added to the result object, as noted missing in the original issue

Test plan

  • Added test: — verifies the new field
  • Added test: — verifies both Pending and In Progress are updated
  • Added test: — verifies Phase 1 without Requirements doesn't affect Phase 2's requirements
  • All 538 tests pass (535 existing + 3 new)

🤖 Generated with Claude Code

…d-build#848)

- Scope phase section regex to prevent cross-phase boundary matching
- Handle 'In Progress' status in traceability table (not just 'Pending')
- Add requirements_updated field to phase complete result object
- Add 3 new tests: result field, In Progress status, cross-boundary safety

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ashanuoc ashanuoc requested a review from glittercowboy as a code owner March 3, 2026 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

phase complete doesn't update REQUIREMENTS.md traceability

1 participant