Skip to content

Conversation

@andyrewlee
Copy link
Owner

@andyrewlee andyrewlee commented Jan 28, 2026

andyrewlee and others added 2 commits January 28, 2026 11:52
Replace separate MetadataStore with unified WorkspaceStore that persists
all workspace data in a single workspace.json file per workspace.

Key changes:
- Remove metadata.go in favor of workspace_store.go
- Workspace struct now contains all metadata fields directly
- LoadMetadataFor merges stored metadata into discovered workspaces
- Skip creating primary workspace when branch lookup fails
- Surface errors in ListByRepo when all workspaces fail to load

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove inner quotes from default values in bash parameter expansion
to allow proper word splitting into array elements.

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 5 additional flags.

Open in Devin Review

Previously, workspaces discovered via git worktree without existing
metadata would have Created set to zero (epoch). Now they get
Created = time.Now() so they display correctly in time-sorted UI.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@andyrewlee andyrewlee changed the title Workspaces Improve workspaces Jan 28, 2026
Previously the primary checkout was created fresh on each load, losing
UI state like OpenTabs and ActiveTabIndex. Now we load any existing
metadata and save new primary checkouts so UI state persists across
restarts, matching the behavior of worktrees.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@andyrewlee andyrewlee merged commit 14f32b5 into main Jan 28, 2026
1 check passed
@andyrewlee andyrewlee deleted the workspaces branch January 28, 2026 20:39
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