Skip to content

update: conditionally initialize Tauri registry and enhance TerminalP…#88

Merged
maximedogawa merged 1 commit into
mainfrom
fix-web-marketing-no-local-network-prompt
Apr 21, 2026
Merged

update: conditionally initialize Tauri registry and enhance TerminalP…#88
maximedogawa merged 1 commit into
mainfrom
fix-web-marketing-no-local-network-prompt

Conversation

@maximedogawa
Copy link
Copy Markdown
Collaborator

@maximedogawa maximedogawa commented Apr 21, 2026

…review for marketing website

Summary by CodeRabbit

  • New Features

    • Terminal preview displays tailored demo content and "demo" status indicator on the marketing website.
  • Improvements

    • Application initialization now optimized to skip unnecessary backend connections in non-Tauri environments.

@maximedogawa maximedogawa merged commit cc7731d into main Apr 21, 2026
1 check was pending
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 21, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ece4da50-fac3-4ffc-b079-a2c858efa785

📥 Commits

Reviewing files that changed from the base of the PR and between 1e09914 and 2915aca.

📒 Files selected for processing (2)
  • src/main.tsx
  • src/modules/bot/components/TerminalPreview.tsx

📝 Walkthrough

Walkthrough

The pull request adds runtime environment detection to control app initialization and feature behavior. The Tauri bridge initialization is now conditionally invoked based on whether the app runs in a Tauri environment, and the terminal preview component adjusts its behavior and display state based on whether it's running on a marketing website.

Changes

Cohort / File(s) Summary
Initialization Control
src/main.tsx
Wrapped initTauriRegistryBridge() invocation in a conditional check using isTauriApp(), preventing bridge initialization in non-Tauri runtimes.
Terminal Preview Behavior
src/modules/bot/components/TerminalPreview.tsx
Added isMarketingWebsite() detection to initialize lines state with marketing-specific demo data, added early return to skip streaming logic on marketing site, and updated status display text from "live" to "demo" in marketing context.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related PRs

Poem

🐰 A rabbit hops through different grounds,
Some paths need bridges, some don't spin around,
On market meadows, we demo with glee,
While Tauri homes get initialization decree! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-web-marketing-no-local-network-prompt

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@maximedogawa maximedogawa deleted the fix-web-marketing-no-local-network-prompt branch April 21, 2026 17:21
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