fix(web): clear prompt when removing Home example chip (#2989)

[leno23]

/claim #2989

Summary

Removing a selected Home page example prompt chip now also clears the composer input, so the chip and prompt text stay in sync.

Surface area

• apps/web/src/components/HomeHero.tsx — clearSelectedPromptExample() clears both local chip state and the parent prompt via onPromptChange('')
• apps/web/tests/components/HomeHero.rail.test.tsx — regression test for chip removal

Validation

• cd apps/web && pnpm test tests/components/HomeHero.rail.test.tsx

Made with Cursor

[lefarcen]

Hi @leno23! Thanks for picking up the Home example-chip cleanup — the summary makes the chip/input sync issue easy to follow.

A quick coordination note before this goes further: PR #2996 by @xxiaoxiong is already open for #2989, and both PRs touch apps/web/src/components/HomeHero.tsx to clear the prompt when the active example chip is removed. You two may want to compare approaches; the maintainer team will decide which path lands, but I wanted to surface the overlap so neither effort gets wasted.

One PR-body tweak too: could you add the template Surface area checklist and tick UI? Your Summary and Validation already cover the user-facing behavior and test path well; the checklist just helps release review see this touches the Home page interaction.

[github-actions]

Visual regression review

Head: d138dd5 · Base: 1d72914

0 changed · 17 unchanged · 0 missing baseline · 0 failed

Unchanged cases

Case	Main	PR	Diff
visual-avatar-menu
visual-design-systems
visual-home
visual-home-catalog
visual-home-context-picker
visual-home-plugin-filter
visual-integrations
visual-integrations-use-everywhere
visual-new-project-modal
visual-plugin-details
visual-plugins
visual-projects
visual-projects-kanban
visual-settings-byok
visual-settings-execution
visual-tasks
visual-topbar-execution-switcher

Visual diff is advisory only and does not block merging.

[PerishCode]

@leno23 thanks for the focused fix and the regression test.

Reviewed apps/web/src/components/HomeHero.tsx and apps/web/tests/components/HomeHero.rail.test.tsx against the changed ranges:

• clearSelectedPromptExample() clears the composer via onPromptChange('') only when an example chip is active, then drops the chip. That pairs cleanly with the existing onChange guard at lines 827-828 that already nulls the chip when the user edits the prompt away from selectedPromptExample.promptText, so the chip's presence implies the prompt equals the inserted example and wiping it is safe — there is no user-edit text in play to lose.
• The StatefulHero wrapper in the new test gives the component real state, drives the full pick-then-clear flow through home-hero-prompt-example and .home-hero__active-clear, and asserts both the input value and the chip removal — that is a good regression shape for this bug.

Nice, tight bugfix. Heads up that maintainer @lefarcen flagged overlap with #2996 on the same issue and asked for the Surface area UI checkbox in the PR body; worth syncing with them before merge, but neither is a code concern.

_{🔁 Powered by Looper · runner=reviewer · agent=claude-code · An autonomous AI dev team for your GitHub repos.}

[lefarcen]

All gates are green here — flagged this for the maintainer team. Thanks @leno23 for the focused fix and @PerishCode for the review.