Skip to content

feat(live-voice): default work to the current session - #1265

Merged
vastsa merged 4 commits into
mainfrom
feat/live-voice-all-sessions
Sep 30, 2026
Merged

vastsa merged 4 commits into
mainfrom
feat/live-voice-all-sessions

Conversation

@vastsa

@vastsa vastsa commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Start Live Voice work on the active Composer session by default; allow voice switching among Desktop, Native Pi, and Remote sessions.
  • Revalidate targets in Main against a fresh catalog, route stop/queue cancellation by each operation's original session, and preserve each backend's permission/approval boundary.
  • Keep bounded history sharing as a separate, default-off call consent.

Verification

  • pnpm test: workspace packages and Rust host-core passed.
  • Desktop typecheck; host-runtime typecheck; pnpm lint; pnpm docs:check (83 English/Chinese spec pairs, 536 pages).
  • Desktop Live Voice/Work tests: 34 passed; Host Live Work Coordinator tests: 29 passed; i18n tests: 28 passed.
  • pnpm test:e2e: 23 passed, 2 live-model cases skipped (no API key).
  • Local Live Voice fixture E2E: 5/5 journeys; mounted interaction E2E: 46/46 checks.
  • GitHub PR checks passed: JS build/typecheck/lint/architecture/tests, Docs, Rust host-core, and latest-base gate.

Real-provider, paid API, hardware, and user-Desktop tests were not run. Fixture E2Es used an isolated profile, local TLS fixture, and synthetic microphone.

Use the active Composer target so voice work can start without preselection; resolve it in Main against a fresh multi-backend catalog and keep voice-selected switches opaque.

Keep execution permissions backend-owned, context-history consent independent, and stop/cancel controls bound to each operation's original session.
Copilot AI balanced review requested due to automatic review settings September 30, 2026 18:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Provide the playback monitor's AudioContext in the controller fixture and run the packaged-owner path check as ESM so import.meta.url matches production.
vastsa added 2 commits October 1, 2026 02:41
Update the shared feature branch to the current remote main before final integration checks. Preserve the approved Composer-default target flow while incorporating concurrent upstream changes.
Keep the playback-failure interaction scenario aligned with the production call bar, which reports one-shot action errors through the global ToastHost.
@vastsa
vastsa merged commit 9b3f9ba into main Sep 30, 2026
4 checks passed
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.

2 participants