Skip to content

perf(app): defer the closed prompt menu - #24

Merged
Itsnotaka merged 1 commit into
mainfrom
daniel/frontend-defer-prompt-menu
Aug 4, 2026
Merged

Itsnotaka merged 1 commit into
mainfrom
daniel/frontend-defer-prompt-menu

Conversation

@Itsnotaka

@Itsnotaka Itsnotaka commented Aug 4, 2026 •

Copy link
Copy Markdown
Contributor

Summary:

  • split the closed composer prompt menu behind one cached import
  • start loading only when a live slash or mention trigger exists
  • correct the startup graph check to ignore compiler-erased type-only edges

Performance:

  • eager Home source: 909,275 to 890,793 bytes (-2.03%)
  • initial JS: 1,582,460 to 1,571,168 bytes (-0.71%)
  • pooled Electron window-ready: 4,413 to 4,383 ms
  • React initial render: 2.2 ms on both sides; commit-to-frame 6.6 to 6.7 ms

Verification:

  • startup review guard passes and Vite reaches ready
  • focused resource test passes
  • production app build passes
  • typecheck, architecture, design, and React review retain only documented main-branch findings

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@cursor

cursor Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@vercel

vercel Bot commented Aug 4, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
honk-marketing Ready Ready Preview Aug 4, 2026 7:58pm

Request Review

@Itsnotaka
Itsnotaka merged commit ba66e6f into main Aug 4, 2026
4 of 5 checks passed
Itsnotaka added a commit that referenced this pull request Aug 6, 2026
@Itsnotaka
Itsnotaka deleted the daniel/frontend-defer-prompt-menu branch August 6, 2026 05:46

This branch was successfully deployed

1 active deployment
Preview — b72e36e4 Deployed Aug 4, 2026 by vercel[bot]
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