engineering/opencompletion.com
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(tts): rewrap sentence glow when innerHTML overwrites destroyed spans

Browse source 
wrapSentencesForGlow gated only on dataset.glowWrapped='1', but that flag
lives on the container element and survives innerHTML replacement. The
streaming chunk handler does targetMessageElement.innerHTML = sanitized
on every delta, and message_updated / edit-save both replace innerHTML
too. Spans got blown away while the flag persisted, so subsequent glow
attaches found the flag, skipped re-wrapping, and the highlight never
appeared.

Now also require an actual .tts-sentence span to exist before short-
circuiting. If the wrap was destroyed, re-wrap.
This commit is contained in:
russell@unturf.com 2026-06-03 16:07:21 -04:00
parent ad080a7e2a
commit a829c97136
No known key found for this signature in database
1 changed files with 7 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

8
templates/chat.html
View file

@ -975,7 +975,13 @@ async function fetchTTSStreaming(cleanText, model, voice) {
// comparing audio.currentTime to those boundaries. No Web Audio / RMS heuristics.
function wrapSentencesForGlow(container) {
if (!container || container.dataset.glowWrapped === '1') return;
if (!container) return;
// Skip only if the previous wrap survives. innerHTML overwrites (streaming
// chunks, message_updated, edit/save) wipe the spans but the dataset flag
// lives on the container itself — without this spans-present check, manual
// replay after a re-render finds the flag set, skips wrapping, and the
// glow no-ops because querySelectorAll('.tts-sentence') comes back empty.
if (container.dataset.glowWrapped === '1' && container.querySelector('.tts-sentence')) return;
const walker = document.createTreeWalker(container, NodeFilter.SHOW_TEXT, null);
const nodes = [];
let t;