feat(home): early-bird banner + assistant→agent terminology

[senamakel]

Summary

• Add EarlyBirdyBanner to Home promoting the subscription discount
• Rename user-facing "assistant" → "agent" across Conversations, Skills, Home, and setup modals for consistent terminology

Test plan

• pnpm typecheck clean
• pnpm lint clean
• Open Home — Early Bird banner renders and dismisses correctly
• Conversations / Skills / setup modals show "agent" wording everywhere

Summary by CodeRabbit

• New Features

  • EarlyBirdy promotional banner: 60% discount, CTA to billing, optional dismiss control.

• Improvements

  • Replaced “AI assistant” wording with “agent” across UI and previews.
  • Clarified Channels/Integrations descriptions and refined setup, error, and connection copy.

• Tests

  • Added unit and integration tests covering the Home banners and banner dismissal behavior.

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds a new EarlyBirdyBanner component and integrates it into the Home page; updates user-facing copy to replace “AI assistant” with “agent” across several modals, a timeout message, and a config placeholder; updates and extends unit tests for banners and Home behavior.

Changes

Early-bird Banner + Rebranding

Layer / File(s)	Summary
New Component app/src/components/home/HomeBanners.tsx	Adds exported EarlyBirdyBanner({ onDismiss? }) that renders a discount headline, EARLYBIRDY code badge, CTA opening BILLING_DASHBOARD_URL via openUrl, and an optional dismiss (✕) button calling onDismiss.
Home Page Wiring app/src/pages/Home.tsx	Imports EarlyBirdyBanner, adds local showEarlyBirdy state from shouldShowBanner('home-earlybirdy', Number.MAX_SAFE_INTEGER), renders the banner before DiscordBanner, and calls dismissBanner('home-earlybirdy') on dismiss; also updates socket status copy to “Message your agent…”.
Copy Updates (Skills / Modals / Pages / Config) app/src/components/skills/ScreenIntelligenceSetupModal.tsx, app/src/components/skills/VoiceSetupModal.tsx, app/src/pages/Conversations.tsx, app/src/pages/Skills.tsx, app/src/utils/tauriCommands/config.ts	Replaces occurrences of “AI assistant” with “agent” (Screen Intelligence/Voice modal messages, conversations silence-timeout error string, config AIPreview.soul.description) and updates Skills page copy in Channels/Integrations; adds text-center styling to Voice success paragraph.
Tests app/src/components/home/__tests__/HomeBanners.test.tsx, app/src/pages/__tests__/Home.test.tsx	Expands HomeBanners tests to cover EarlyBirdyBanner rendering, CTA openUrl(BILLING_DASHBOARD_URL), and conditional dismiss behavior; updates Home tests with mocks and adds integration tests for banner visibility and dismissal.

Estimated Code Review Effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly Related PRs

• feat(home): banners, welcome typewriter, conversation gating #936 — Related: also modified app/src/components/home/HomeBanners.tsx and added/changed banner components and tests.
• feat(upsell): usage-aware banners, rate-limit modal, and shared usage hook #440 — Related: earlier work on upsell/banner billing CTA behavior and use of upsell dismissal utilities (shouldShowBanner/dismissBanner).
• fix(chat): increase agent silence timeout from 2 to 10 minutes #719 — Related: previously modified the same safety_timeout/silence-timeout messaging in Conversations.

Poem

🐰 I hopped in quick with a cheerful cheer,
“EARLYBIRDY” offered to those who hear.
I nudged each modal to call you an agent,
Banners and tests—my ears are adjacent.
Click the CTA, but mind the dismiss—tiny rabbit bliss!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 28.57% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the two main changes: adding an early-bird banner component to the home page and updating terminology from 'assistant' to 'agent' throughout the codebase.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Review rate limit: 4/5 reviews remaining, refill in 12 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@app/src/components/home/HomeBanners.tsx`:
- Around line 102-107: In the HomeBanners component fix the promo sentence typo
by replacing the fragment "to on your" in the JSX (the string surrounding the
discount span/button in HomeBanners.tsx) with the correct phrasing — e.g., "on
your" (or "to use on your" if you prefer) so the sentence reads properly around
the EARLYBIRDY span and the following button. Ensure only the text node is
changed and JSX spacing remains intact.
- Around line 90-120: EarlyBirdyBanner currently has no way to be dismissed; add
a dismissal prop and UI and wire it from the Home page: update EarlyBirdyBanner
to accept an optional onDismiss: () => void prop, render a small accessible
dismiss control (e.g., an X button in the banner header) that calls onDismiss()
when clicked, and export/use that prop in app/src/pages/Home.tsx by wrapping the
banner with the existing helpers: only render EarlyBirdyBanner when
shouldShowBanner('home-earlybirdy') is true and pass onDismiss={() =>
dismissBanner('home-earlybirdy')} so the banner state is cleared when the close
control is pressed.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 454f18d1-b2c4-4874-9276-41333171472d

📥 Commits

Reviewing files that changed from the base of the PR and between 67410d5 and 0ff46e1.

⛔ Files ignored due to path filters (1)

• app/src-tauri/Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (7)

• app/src/components/home/HomeBanners.tsx
• app/src/components/skills/ScreenIntelligenceSetupModal.tsx
• app/src/components/skills/VoiceSetupModal.tsx
• app/src/pages/Conversations.tsx
• app/src/pages/Home.tsx
• app/src/pages/Skills.tsx
• app/src/utils/tauriCommands/config.ts

[coderabbitai]

🧹 Nitpick comments (1)

app/src/components/home/HomeBanners.tsx (1)

90-90: ⚡ Quick win

Use a props interface for EarlyBirdyBanner instead of an inline object type.

This keeps the new component aligned with the TypeScript object-shape convention and makes future prop growth cleaner.

♻️ Proposed refactor

+interface EarlyBirdyBannerProps {
+  onDismiss?: () => void;
+}
+
-export function EarlyBirdyBanner({ onDismiss }: { onDismiss?: () => void }) {
+export function EarlyBirdyBanner({ onDismiss }: EarlyBirdyBannerProps) {

As per coding guidelines, "**/*.{ts,tsx}: Prefer interface for defining object shapes in TypeScript".

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@app/src/components/home/HomeBanners.tsx` at line 90, The EarlyBirdyBanner
component currently uses an inline prop type ({ onDismiss?: () => void });
change this to a named interface (e.g., EarlyBirdyBannerProps) and update the
component signature to use that interface (export function
EarlyBirdyBanner(props: EarlyBirdyBannerProps) or destructure as ({ onDismiss }:
EarlyBirdyBannerProps)); ensure the interface is exported if needed and update
any usages or tests referencing the component signature to use the new type
name.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@app/src/components/home/HomeBanners.tsx`:
- Line 90: The EarlyBirdyBanner component currently uses an inline prop type ({
onDismiss?: () => void }); change this to a named interface (e.g.,
EarlyBirdyBannerProps) and update the component signature to use that interface
(export function EarlyBirdyBanner(props: EarlyBirdyBannerProps) or destructure
as ({ onDismiss }: EarlyBirdyBannerProps)); ensure the interface is exported if
needed and update any usages or tests referencing the component signature to use
the new type name.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8f7c46c3-871d-4923-ab0f-e31d9bfbfa28

📥 Commits

Reviewing files that changed from the base of the PR and between 0ff46e1 and 477270e.

📒 Files selected for processing (4)

• app/src/components/home/HomeBanners.tsx
• app/src/components/home/__tests__/HomeBanners.test.tsx
• app/src/pages/Home.tsx
• app/src/pages/__tests__/Home.test.tsx