Skip to content

Fix Rift workspace path resolution and rollback - #113

Merged
dbpolito merged 1 commit into
mainfrom
fix/rift-workspace-resolution
Jul 28, 2026
Merged

Fix Rift workspace path resolution and rollback#113
dbpolito merged 1 commit into
mainfrom
fix/rift-workspace-resolution

Conversation

@dbpolito

Copy link
Copy Markdown
Contributor

Ticket

SKIPPED

Description

Resolve Rift workspaces from the stable OpenCode project checkout so named workspaces use Rift's expected sibling location instead of nesting beneath another managed workspace. Make workspace setup transactional by removing partial resources on creation, session, or ownership failures, and collapse duplicate managed-workspace listings.

Checklist

Workspace Resolution

  • Resolve Rift registration from the stable project checkout
  • Verify Rift creation matches the directory registered by OpenCode
  • Preserve Rift's native destination selection

Failure Recovery

  • Roll back partial Rift and native worktree creation
  • Reject foreign project ownership before creating a session
  • Report expected and actual ownership when validation fails

Workspace Discovery

  • Deduplicate managed workspace entries by normalized directory
  • Return the Rift workspace ID after successful creation

Validation

  • Verify that bun run compile succeeds
  • Confirm that bun run typecheck reports no errors
  • Check that all 80 tests pass with bun run test

- resolve Rift paths from the stable project checkout
- roll back partial workspace and session setup failures
- validate project ownership and deduplicate workspace listings
- cover failure recovery with Navigator regression tests
@dbpolito dbpolito self-assigned this Jul 28, 2026
@dbpolito
dbpolito merged commit f3799f5 into main Jul 28, 2026
1 check passed
@dbpolito
dbpolito deleted the fix/rift-workspace-resolution branch July 28, 2026 19:05
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.

1 participant