Skip to content

Conversation

@1egoman
Copy link
Contributor

@1egoman 1egoman commented Feb 2, 2026

More info / context can be found here: #1252 (comment)

Summary by CodeRabbit

  • Bug Fixes

    • Restored the attributes property in chat messages over data streams so messages now include id, timestamp, and attributes.
  • Documentation

    • Clarified hook documentation to reference the current trackRef parameter and removed an outdated example.
  • Chores

    • Added a changeset entry for a patch release.

…sport)

In the conversion from legacy -> data streams these seem to have been
lost, and it is unclear why.

More info: #1252 (comment)
@changeset-bot
Copy link

changeset-bot bot commented Feb 2, 2026

🦋 Changeset detected

Latest commit: aff6c14

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages
Name Type
@livekit/components-core Patch
@livekit/components-react Patch
@agents-ui Patch
@livekit/component-example-next Patch
@livekit/components-js-docs Patch
@livekit/component-docs-storybook Patch
@livekit/components-docs-gen Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai
Copy link

coderabbitai bot commented Feb 2, 2026

📝 Walkthrough

Walkthrough

Restores the attributes property on chat message objects by propagating it from reader.info into emitted ReceivedChatMessage instances; adds a changeset documenting a patch release. Also updates JSDoc in a React hook (documentation-only).

Changes

Cohort / File(s) Summary
Changeset
\.changeset/beige-radios-shop.md
Adds a new changeset documenting a patch release for @livekit/components-core that notes restoring chat message attributes when using data streams transport.
Chat implementation
packages/core/src/components/chat.ts
Propagates attributes (plus existing id and timestamp) from reader.info into emitted ReceivedChatMessage objects; clarifies that attachedFiles/editTimestamp are not included.
Docs (React hook)
packages/react/src/hooks/useIsMuted.ts
JSDoc update: replaces sourceOrTrackRef/example block with trackRef description; no runtime/signature changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐇 I hopped through streams to find,

attributes left far behind.
Now each chat, with time and id,
wears its tags — tidy and rid.
Hooray for messages, whole and kind!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: restoring attributes to the chat message API, which is reflected in the primary code modification in chat.ts that propagates the attributes property.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch add-attachments-back-to-chat-message

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aff6c14 and 306c898.

📒 Files selected for processing (1)
  • packages/react/src/hooks/useIsMuted.ts
🔇 Additional comments (1)
packages/react/src/hooks/useIsMuted.ts (1)

15-25: Doc update is clear and consistent.
The parameter description now matches the overload naming and intent.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2026

size-limit report 📦

Path Size
LiveKitRoom only 6 KB (0%)
LiveKitRoom with VideoConference 32.41 KB (-0.16% 🔽)
All exports 43.36 KB (-0.02% 🔽)

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