Skip to content

Add notification-backed voice queue and related UI fixes#97

Open
kcosr wants to merge 32 commits intomainfrom
feat/voice-notification-queue
Open

Add notification-backed voice queue and related UI fixes#97
kcosr wants to merge 32 commits intomainfrom
feat/voice-notification-queue

Conversation

@kcosr
Copy link
Copy Markdown
Owner

@kcosr kcosr commented Apr 9, 2026

Summary

  • add notification-backed Android voice queueing, durable notifications, and native voice/runtime fixes
  • improve notifications and chat UI flows, including session switching, HTML export, and mobile/back-button handling
  • add notes editor compact mode and preserve chat scroll position while resizing split panes
  • add Pi SDK context-window override support and related config/docs updates

Verification

  • npm run build
  • focused vitest suites for notes, notifications, voice FAB/session selection, and split resize behavior
  • focused Android unit tests for AssistantVoiceRuntimeService and related queue/notification classes
  • deploy-side full rebuild and Android flavor deploys during validation

Kevin and others added 30 commits April 6, 2026 21:04
… to session

Android durable notifications for session_attention now use the session title as
the content title instead of the generic "Latest assistant reply" label. Tapping a
notification opens or activates the linked session's chat panel via a new
openSession bridge event, matching the behavior of the in-app notification panel.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use bitwise AND (hashCode & 0x7FFFFFFF) instead of Math.abs to avoid
Integer.MIN_VALUE returning a negative value. Compute request codes with
addition instead of multiplication to prevent int overflow. Fix 2-space
indentation in buildAdapterTtsRequestBody/StopRequestBody to match the
4-space convention used everywhere else.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Record session activity before publishing the notification so the
notification carries the post-bump revision. Wrap recordSessionActivity
in try/catch so a session-index write failure does not poison the
completed turn — falls back to the pre-bump summary when needed.

Adds a test that asserts the published notification uses the post-activity
revision, not the pre-activity one.

Co-Authored-By: Claude Opus 4.6 (1M context) <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