Skip to content

Conversation

@khaong
Copy link
Contributor

@khaong khaong commented Feb 5, 2026

while we're here - let's rename the branch to what it actually contains


Note

Medium Risk
Branch-name changes affect where checkpoint metadata is written/read, so existing repos expecting entire/sessions/v1 may need migration or compatibility handling.

Overview
Renames the metadata orphan branch used to store committed checkpoint metadata from entire/sessions/v1 to entire/checkpoints/v1 by updating paths.MetadataBranchName.

Updates cleanup/shadow-branch logic, CLI tests, and helper scripts (test harness, attribution E2E, and migrate-sessions.sh) to reference the new branch name and ensure it remains excluded from shadow-branch cleanup.

Written by Cursor Bugbot for commit 4f32697. This will update automatically on new commits. Configure here.

@khaong khaong requested a review from a team as a code owner February 5, 2026 09:35
Copilot AI review requested due to automatic review settings February 5, 2026 09:35
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 PR renames the metadata branch from entire/sessions/v1 to entire/checkpoints/v1 to better reflect that the branch stores checkpoint metadata rather than session data. This is a simple constant rename with the goal of improving code clarity.

Changes:

  • Updates the MetadataBranchName constant in paths/paths.go from "entire/sessions/v1" to "entire/checkpoints/v1"

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 3 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@khaong khaong enabled auto-merge February 5, 2026 09:42
@khaong khaong disabled auto-merge February 5, 2026 09:42
@khaong khaong merged commit 01e95ba into main Feb 5, 2026
4 checks passed
@khaong khaong deleted the alex/checkpoints-is-way-better-thanks-daniel branch February 5, 2026 09:47
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.

1 participant