Skip to content

fix: use consistent Checkpoint: xxx subject for all entire/sessions commits#74

Merged
khaong merged 1 commit intomainfrom
alex/fix-checkpoint-alignment
Jan 21, 2026
Merged

fix: use consistent Checkpoint: xxx subject for all entire/sessions commits#74
khaong merged 1 commit intomainfrom
alex/fix-checkpoint-alignment

Conversation

@khaong
Copy link
Contributor

@khaong khaong commented Jan 21, 2026

Summary

  • Fixed inconsistent commit message subjects on the entire/sessions branch
  • Task checkpoints were using custom subjects (e.g., "Subagent: dev completed...") while regular checkpoints used "Checkpoint: xxx"
  • Now all commits consistently use "Checkpoint: " as the subject line
  • Task checkpoint descriptions are preserved in the commit body

Test plan

  • All existing tests pass (mise run test)
  • Lint passes (mise run lint)
  • Verify new checkpoints on entire/sessions all have Checkpoint: xxx subjects

🤖 Generated with Claude Code

…ommits

Previously, task checkpoints set CommitSubject which became the commit
subject line, while regular checkpoints used "Checkpoint: xxx". This
caused inconsistent commit messages on the entire/sessions branch.

Now all commits use "Checkpoint: <id>" as the subject for consistency.
Task checkpoint descriptions are preserved in the commit body.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Entire-Checkpoint: f75ef0f21af3
@khaong khaong requested a review from a team as a code owner January 21, 2026 00:08
Copilot AI review requested due to automatic review settings January 21, 2026 00:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request standardizes commit message subjects on the entire/sessions branch to always use "Checkpoint: " format, while preserving custom task checkpoint descriptions in the commit body.

Changes:

  • Modified buildCommitMessage to always use "Checkpoint: " as the subject line
  • Moved custom task checkpoint descriptions (e.g., "Subagent: dev completed...") from subject to body
  • Updated comments to document the new commit message structure

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@khaong khaong merged commit 6258244 into main Jan 21, 2026
9 checks passed
@khaong khaong deleted the alex/fix-checkpoint-alignment branch January 21, 2026 00:12
@khaong khaong restored the alex/fix-checkpoint-alignment branch January 21, 2026 00:12
@khaong khaong deleted the alex/fix-checkpoint-alignment branch January 21, 2026 00:12
@khaong khaong restored the alex/fix-checkpoint-alignment branch January 21, 2026 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants