Merged
Conversation
Add Phase 1.5 "Investigation" to the phase registry between Re-anchor and TDD/Implement. Add --investigation flag to `task spec` command for patching `## Investigation targets` sections. Update default task spec template to include empty investigation targets placeholder. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ets template Add Phase 1.5 to worker.md between Re-anchor and TDD/Implement phases. Worker reads Required investigation target files before coding, greps for similar functionality, then proceeds. Skips gracefully when no targets. Update plan skill task spec template with ## Investigation targets section (Required/Optional structure). Add good/bad examples to examples.md. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Teach plan-sync to detect stale file paths in ## Investigation targets sections of downstream task specs. When completed tasks rename/move files, paths in Required/Optional targets are updated along with other stale references. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
z23cc
added a commit
that referenced
this pull request
Apr 9, 2026
P0 fixes (state loss — root cause of 5 issues): - get_flow_dir() now walks up directory tree (FLOW_STATE_DIR env → walk-up → CWD) Fixes: #1 state loss, #3 state not persistent, #5 worker parallel fail, #9 .flow symlink issues. Same pattern as git finding .git. - flowctl recover --epic <id> [--dry-run]: rebuilds task completion status from git log. Fixes #11 no recovery after state loss. P1 fixes (guard + review): - Guard graceful fallback: missing tools → "skipped" (not "failed"). Only actual failures block pipeline. Fixes #8. - Review-backend availability check: if rp-cli/codex not in PATH, auto-fallback to "none" with warning. Fixes #7. P2 fixes (UX): - Slug max length 40→20 chars. "Django+React platform with account management" → "fn-3-django-react-plat" not 40-char monster. Fixes #2 #12. - Brainstorm auto-skip: trivial tasks (≤10 words, contains "fix"/"typo"/etc) skip brainstorm entirely. Fixes #6. - --interactive flag: pause at key decisions. Fixes #10. 370 tests pass. Zero new dependencies. Co-Authored-By: Claude Opus 4.6 (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
1 participant
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
--investigationCLI flag toflowctl task specfor patching## Investigation targetssectionsTest plan
cargo test --all— 293 passedflowctl validate --epic— clean--investigationflag and verify section patching🤖 Generated with Claude Code