Skip to content

Conversation

@andyrewlee
Copy link
Owner

@andyrewlee andyrewlee commented Jan 28, 2026

andyrewlee and others added 2 commits January 28, 2026 08:53
The old naming migration from worktrees → workspaces has served its
purpose. This removes all backward-compatibility code: the migration
runner, legacy env vars, legacy config/metadata file fallbacks, and
associated tests.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional flags.

Open in Devin Review

andyrewlee and others added 3 commits January 28, 2026 09:07
The git layer still uses `git worktree` commands under the hood, but the
exported API and all naming now speaks in terms of workspaces. Key changes:

- Rename worktree.go → workspace.go with unexported internal types
- Rename exported functions: DiscoverWorktrees → DiscoverWorkspaces,
  CreateWorktree → CreateWorkspace, RemoveWorktree → RemoveWorkspace
- Collapse RuntimeLocalWorktree/RuntimeLocalCheckout into RuntimeLocal
  with backward-compatible NormalizeRuntime mapping
- Rename findWorktreeRoot → findRoot in watcher
- Update all comments, test names, test paths, and variable names

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Adopt sandbox's runtime constant naming (RuntimeLocalWorktree,
RuntimeLocalCheckout) instead of RuntimeLocal, add Runtime field to
Workspace struct, and call NormalizeRuntime in metadata Load() to
fix raw legacy values passing through unnormalized.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@andyrewlee andyrewlee merged commit be066da into main Jan 28, 2026
1 check passed
@andyrewlee andyrewlee deleted the remove-migration branch January 28, 2026 17:42
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.

2 participants