Skip to content

fix : play back icon for the downlaoded tts - #15

Merged
rex-9 merged 1 commit into
devfrom
feat/ps-qa-fix
Sep 30, 2026
Merged

rex-9 merged 1 commit into
devfrom
feat/ps-qa-fix

Conversation

@novenaurion

Copy link
Copy Markdown
Collaborator

Summary

Chat TTS clips already attached to a message now show a play icon instead of the speaker, so downloaded audio can be replayed without queueing TTS again.

Why

This change is needed because Core stores message audio as type: "tts" and format: "audio". The UI looked for type === "audio", so getMessageAudioUrl always returned null. The problem it solves is the TTS control staying on speaker even when assets[] already had a ready MP3 URL.

Changes

  • Changed src/modules/ai/types.ts findMessageAudioAsset to match item.format === "audio" instead of item.type === "audio"

Testing

This change was verified by opening an assistant message that already has a TTS asset (format: "audio", url present). The control should be play (or stop while playing), not speaker. Tapping play should use the existing URL, not queue another TTS job.

  • Unit tests
  • Integration tests
  • Manual testing
  • Existing test suite passes

Related Issues

N/A

Screenshots / Videos=

Notes

No API or migration change. metadata.tts_status is not used for the icon. Existing messages with type: "tts" and format: "audio" start matching immediately.
Unit tests for this helper were not added in this commit (one-line match fix only).

Checklist

  • Code follows the project's conventions
  • Tests were added/updated where appropriate
  • Documentation was updated where appropriate
  • No unrelated changes are included

@vercel

vercel Bot commented Sep 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
rexone Ready Ready Preview Sep 30, 2026 5:23am UTC

@rex-9
rex-9 merged commit e339b80 into dev Sep 30, 2026
3 checks passed

This branch was successfully deployed

1 active deployment
Preview — f6a7a70f Deployed Sep 30, 2026 by vercel[bot]
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