Skip to content

Delay workspace restore until settings load#11

Merged
jacobjmc merged 1 commit intomainfrom
fix/startup-restore-settings-gate
Mar 8, 2026
Merged

Delay workspace restore until settings load#11
jacobjmc merged 1 commit intomainfrom
fix/startup-restore-settings-gate

Conversation

@jacobjmc
Copy link
Owner

@jacobjmc jacobjmc commented Mar 8, 2026

Summary

  • delay startup workspace restore until app settings have finished loading
  • add an explicit restore enable gate to the workspace restore hook
  • cover the new startup sequencing with a regression test

Why

Startup restore could run before saved settings were loaded, causing early workspace/connect error debug entries when the OpenCode binary path had not been resolved yet. Later retries would succeed, so the app worked, but the launch logs looked broken.

Validation

  • npm run typecheck
  • npm run test -- src/features/workspaces/hooks/useWorkspaceRestore.test.tsx
  • npm run test

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 3 files

@jacobjmc jacobjmc force-pushed the fix/startup-restore-settings-gate branch from 5fb24fa to df8836e Compare March 8, 2026 10:01
@jacobjmc jacobjmc merged commit 37cd863 into main Mar 8, 2026
10 checks passed
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