Skip to content

refactor(composer): remove transcribe and speak icons - #555

Merged
vastsa merged 2 commits into
mainfrom
refactor/composer-remove-speech-icons
Sep 17, 2026
Merged

vastsa merged 2 commits into
mainfrom
refactor/composer-remove-speech-icons

Conversation

@vastsa

@vastsa vastsa commented Sep 17, 2026

Copy link
Copy Markdown
Owner

Removes the composer mic (transcribe) and speak (readaloud) toolbar buttons and the useComposerSpeech hook, keeping the host speech capability and plugin-facing speech API intact for plugins.

Fixes the request to drop the inline icons while preserving the underlying capability.

The composer toolbar exposed mic (transcribe) and speak (readaloud)
buttons. These were the only UI entry points for the host speech
capability added in 9948d07. Drop the renderer wiring — the Composer
toolbar buttons, their props, and the useComposerSpeech hook — while
keeping the underlying host transcribe/synthesize services and the
plugin-facing speech API intact, since that capability is provided for
plugins rather than inline composer UI.

The generic audio file-attachment handling in the composer editor is
unaffected and remains.
Copilot AI lite review requested due to automatic review settings September 17, 2026 16:52

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.

The composer no longer exposes transcribe/speak buttons (see parent
commit), so the contract test asserting those UI hooks (readComposerSource
checks on status?.transcribe.available etc.) is obsolete. Remove it and
its now-unused import; the host speech-service and IPC-contract tests
remain.
@vastsa
vastsa merged commit 0b4a934 into main Sep 17, 2026
3 checks passed
@vastsa
vastsa deleted the refactor/composer-remove-speech-icons branch September 17, 2026 17:01
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