Repository navigation
fix(ai-components): replace SpokenMessageText with a standalone useHeardText hook - #3344
Merged
Merged
Conversation
…useHeardText hook ai-components must work without the core SDK, and SpokenMessageText rendered through the core Message components. Remove SpokenMessageText, SpokenTextProvider, SpokenTextContext and useSpokenTextContext, and keep only useHeardText, which now depends on React alone: it takes the text, a heard(now) progress function (Unicode code points) and an optional subscribe, generating and live, and returns the heard text, whether it is paced, and skip(). A drop-in for the StreamedMessageText slot is a recipe in the docs.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
oliverlaz
changed the title
Contributor
|
Size Change: -4.39 kB (-0.37%) Total Size: 1.18 MB 📦 View Changed
ℹ️ View Unchanged
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release-v14 #3344 +/- ##
===============================================
- Coverage 86.84% 86.47% -0.38%
===============================================
Files 526 541 +15
Lines 16699 17590 +891
Branches 5251 5421 +170
===============================================
+ Hits 14503 15211 +708
- Misses 2196 2379 +183 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
github-actions Bot
pushed a commit
that referenced
this pull request
Oct 22, 2026
## [14.14.1](v14.14.0...v14.14.1) (2026-10-22) ### Bug Fixes * **ai-components:** replace SpokenMessageText with a standalone useHeardText hook ([#3344](#3344)) ([ad16506](ad16506)), closes [#3343](#3343) [GetStream/getstream.io#761](https://github.com/GetStream/getstream.io/issues/761) [#3343](#3343) [#3339](#3339)
|
🎉 This PR is included in version 14.14.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎯 Goal
stream-chat-react/ai-componentsmust work standalone, without the core SDK.SpokenMessageText(added in #3343) rendered through the coreMessagecomponents (MessageText, theStreamedMessageTextfallback, and the message and channel contexts), so it doesn't belong in the plugin. This keeps only the utility hook. The drop-in component becomes a recipe in the docs (GetStream/getstream.io#761).This is the last commit of #3343 (
57e84ca2e), which was pushed just before the PR was merged and didn't make it in.🛠 Implementation details
SpokenMessageText,SpokenTextProvider,SpokenTextContext,useSpokenTextContext, their types and their tests.useHeardTextnow lives insrc/plugins/AIComponents/hooks/useHeardText.ts. Its only import isreact; I checked this in the source, the built ESM and the CJS bundle.useHeardText({ text, heard, subscribe?, generating?, live? }) => { heardText, isPaced, skip }.heard(now)returns{ chars, done } | undefined, wherecharscounts Unicode code points.heardandsubscribeare read through refs, so new function identities on each render don't restart pacing.HeardTextProgress,HeardTextFunction,HeardTextSubscribe,UseHeardTextParams,UseHeardTextResult.floor(chars)code points and never goes backwards; surrogate pairs are never split.requestAnimationFrameonly while paced and not done.done && !generating, whenliveis false, or afterskip().renderHooktests. Full suite 3175/3175 passes, as doyarn types,yarn lint,yarn buildandvalidate-cjs/validate-esm/validate-entry-isolation. In the built output,ai-componentsexportsuseHeardTextand nothingSpoken*, and the main entry exports neither.SpokenMessageTextrecipe on this hook: each voice reply is revealed as its audio plays.🎨 UI Changes
None.