Skip to content

Remove sandbox code from agent-starter-react - #365

Open
thomasyuill-livekit wants to merge 6 commits into
mainfrom
thomyuill/clt-3229-remove-sandbox-code-from-agent-starter-react
Open

Remove sandbox code from agent-starter-react#365
thomasyuill-livekit wants to merge 6 commits into
mainfrom
thomyuill/clt-3229-remove-sandbox-code-from-agent-starter-react

Conversation

@thomasyuill-livekit

@thomasyuill-livekit thomasyuill-livekit commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removes the LiveKit Cloud sandbox token source and its NEXT_PUBLIC_CONN_DETAILS_ENDPOINT/X-Sandbox-Id handshake, so the app always connects via the self-hosted /api/token route.
  • Removes the remote app-config-injection mechanism (NEXT_PUBLIC_APP_CONFIG_ENDPOINT, SANDBOX_ID, X-Sandbox-ID header) that let LiveKit Cloud override branding for a hosted sandbox.
  • Deletes app-config.ts and inlines its default values directly where they're used (branding/title in app/layout.tsx, feature flags and visualizer settings in components/app/view-controller.tsx, agent dispatch name in app/page.tsx/components/app/app.tsx).
  • Moves the brand --primary/--primary-hover color overrides from a runtime inline <style> tag in layout.tsx into static declarations in styles/globals.css.
  • Removes the hardcoded "SANDBOX" badge (and its now-unused font) from the generated Open Graph image, then deletes the opengraph-image.tsx route entirely.
  • Deletes TEMPLATE.md, whose sole content was the sandbox hosting pitch.
  • Deletes .github/workflows/sync-to-production.yaml, which synced main to a sandbox-production branch via a sandbox-only deploy action.
  • Removes sandbox-only env vars (NEXT_PUBLIC_APP_CONFIG_ENDPOINT, SANDBOX_ID) from .env.example.
  • Updates README.md to drop all sandbox and app-config.ts references, pointing configuration guidance at the components that now hold those literal values.

Fixes CLT-3229.

Test plan

  • pnpm build succeeds
  • pnpm lint shows only pre-existing, unrelated warnings
  • grep -ri sandbox across the repo returns no app-code/docs/CI matches
  • Manually verify pnpm dev connects via /api/token using local .env.local credentials

🤖 Generated with Claude Code

Strips the LiveKit Cloud sandbox infrastructure (remote config injection,
sandbox token source, sandbox env vars/docs/CI) now that sandboxes are
being deprecated, leaving a plain front-end starter backed by the
self-hosted /api/token route.

Fixes CLT-3229

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Aug 7, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
sandbox-voice-assistant Ready Ready Preview Aug 10, 2026 2:41pm

Request Review

thomasyuill-livekit and others added 2 commits August 7, 2026 14:07
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
APP_CONFIG_DEFAULTS is now a static export with no per-request
resolution, so page.tsx and layout.tsx can use it directly.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Deletes the AppConfig abstraction entirely and inlines its default
values directly where they're used (layout branding/styles, agent
dispatch name, view-controller feature flags). Customization now
happens by editing those components directly.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Folds the inline <style> block from layout.tsx into the existing
--primary/--primary-foreground declarations in globals.css instead of
injecting a second rule at runtime.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The audio visualizer needs to know the active color scheme; this was
dropped when the appConfig-driven audioVisualizerColor logic was
removed.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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