Skip to content

Stabilize plan sidebar and file navigation#505

Merged
BunsDev merged 1 commit intomainfrom
okcode/plan-editor-crash
May 4, 2026
Merged

Stabilize plan sidebar and file navigation#505
BunsDev merged 1 commit intomainfrom
okcode/plan-editor-crash

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented May 4, 2026

Summary

  • Stabilized plan flow handling so the plan sidebar opens and stays keyed to the correct turn without triggering render loops.
  • Added ordered-activity derivations in session logic to avoid repeated sorting and make plan, approval, user-input, and work-log calculations deterministic.
  • Fixed file-view navigation to read the latest thread list lazily, keeping the callback stable across thread updates.
  • Added regression coverage for same-thread plan implementation behavior and navigation stability under snapshot churn.

Testing

  • bun run test apps/web/src/hooks/useFileViewNavigation.test.tsx
  • Not run: bun fmt
  • Not run: bun lint
  • Not run: bun typecheck

- Sort thread activities once before deriving plan and work log state
- Keep plan sidebar auto-open keyed to the active turn without rerender loops
- Read latest threads lazily when navigating file views and add regression coverage
@vercel
Copy link
Copy Markdown

vercel Bot commented May 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview May 4, 2026 10:03pm
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 May 4, 2026 10:03pm

@github-actions github-actions Bot added the size:L label May 4, 2026
@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label May 4, 2026
@BunsDev BunsDev merged commit e02cf89 into main May 4, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant