Skip to content

fix(app): prevent command palette first-open flash - #35858

Merged
Brendonovich merged 1 commit into
devfrom
cmd-p-modal-flash
Jul 8, 2026
Merged

Brendonovich merged 1 commit into
devfrom
cmd-p-modal-flash

Conversation

@opencode-agent

@opencode-agent opencode-agent Bot commented Jul 8, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • eagerly load the V2 command palette with its already code-split parent selector
  • avoid the first-open Suspense fallback that blanks the session before the modal paints

Context

This follows the established fix from #35349: preserve the transition boundary and eagerly load the nested visible controls. The shared dialog API already wraps show() in startTransition; this removes the nested lazy suspension inside that transition.

Test plan

  • bun typecheck in packages/app
  • repository pre-push typecheck
  • post-fix first-open video captured at 1280×720 with the real app UI and seeded mock OpenCode backend

Evidence

Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com>
@opencode-agent
opencode-agent Bot force-pushed the cmd-p-modal-flash branch from a500ccf to 77e7309 Compare July 8, 2026 04:39
@opencode-agent
opencode-agent Bot marked this pull request as ready for review July 8, 2026 18:35
@Brendonovich
Brendonovich merged commit c33a201 into dev Jul 8, 2026
11 checks passed
@Brendonovich
Brendonovich deleted the cmd-p-modal-flash branch July 8, 2026 22:51
rldona pushed a commit to rldona/FlupCode that referenced this pull request Sep 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants