Skip to content

fix(app): avoid prompt tooltip Switch on startup#17857

Merged
adamdotdevin merged 1 commit intoanomalyco:devfrom
Hona:fix/t-crash
Mar 17, 2026
Merged

fix(app): avoid prompt tooltip Switch on startup#17857
adamdotdevin merged 1 commit intoanomalyco:devfrom
Hona:fix/t-crash

Conversation

@Hona
Copy link
Member

@Hona Hona commented Mar 16, 2026

Summary

  • stop passing a live Solid <Switch> tree through the prompt submit tooltip value
  • render the prompt submit tooltip content via a local helper so restored running sessions do not crash desktop startup
  • validated with bun typecheck in packages/app

@Hona Hona requested a review from adamdotdevin as a code owner March 16, 2026 23:21
Copilot AI review requested due to automatic review settings March 16, 2026 23:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts the prompt submit button tooltip rendering to avoid passing a live Solid <Switch> tree as the tooltip value, which can crash desktop startup when restoring running sessions.

Changes:

  • Replaces the inline <Switch>/<Match> tooltip value with a local tip() helper that returns the appropriate JSX.
  • Removes unused Solid imports (Switch, Match) from prompt-input.tsx.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@adamdotdevin adamdotdevin merged commit dbbe931 into anomalyco:dev Mar 17, 2026
17 of 18 checks passed
ConanXu-math pushed a commit to ConanXu-math/opencode that referenced this pull request Mar 17, 2026
AvatarGanymede pushed a commit to AvatarGanymede/opencode-dev that referenced this pull request Mar 19, 2026
demostanis pushed a commit to demostanis/opencode that referenced this pull request Mar 19, 2026
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.

3 participants