-
Notifications
You must be signed in to change notification settings - Fork 127
fix(conductor): ensure track completion and doc sync are committed automatically #48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…tomatically This adds explicit commit steps to the implement command for: - Track completion status updates in tracks.md - Project documentation synchronization - Track cleanup (archiving or deleting) BUG=16
commands/conductor/implement.toml
Outdated
| 5. **Finalize Track:** | ||
| - After all tasks in the track's local `plan.md` are completed, you MUST update the track's status in the tracks file. | ||
| - This requires finding the specific heading for the track (e.g., `## [~] Track: <Description>`) and replacing it with the completed status (e.g., `## [x] Track: <Description>`). | ||
| - **Commit Changes:** Stage `conductor/tracks.md` and commit with the message `conductor(track): Mark track '<track_description>' as complete`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's ensure we adhere to the Conventional Commits Specs for commit message formatting. I think in this case that'd be something like chore(conductor): Mark track '<track_description>' as complete
commands/conductor/implement.toml
Outdated
| - **Construct the Message:** Based on the records of which files were changed, construct a summary message. | ||
| - **Commit Changes:** | ||
| - If any files were changed (`product.md`, `tech-stack.md`, or `product-guidelines.md`), you MUST stage them and commit them. | ||
| - **Commit Message:** `conductor(docs): Synchronize docs for track '<track_description>'` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here
commands/conductor/implement.toml
Outdated
| ii. **Archive Track Folder:** Move the track's folder from `conductor/tracks/<track_id>` to `conductor/archive/<track_id>`. | ||
| iii. **Remove from Tracks File:** Read the content of `conductor/tracks.md`, remove the entire section for the completed track (the part that starts with `---` and contains the track description), and write the modified content back to the file. | ||
| iv. **Announce Success:** Announce: "Track '<track_description>' has been successfully archived." | ||
| iv. **Commit Changes:** Stage `conductor/tracks.md` and `conductor/archive/`. Commit with the message `conductor(track): Archive track '<track_description>'`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here
This adds explicit commit steps to the implement command for:
Fixes #16
Testing: test_scenarios.md