Skip to content

perf(assessment): show pronunciation immediately, grammar follows - #9

Merged
Zymonody7 merged 2 commits into
mainfrom
fix/assessment-fast-pron
Jun 7, 2026
Merged

Zymonody7 merged 2 commits into
mainfrom
fix/assessment-fast-pron

Conversation

@Zymonody7

Copy link
Copy Markdown
Owner

Why

Assessment waited for both SOE pronunciation (~3s) and the grammar LLM (~15s) before publishing one message. With the recording upload on top, total often exceeded the client's 30s timeout → "Assessment is taking too long" → nothing shown, even for clean English.

Change

  • Publish pronunciation the moment SOE lands; grammar is sent in a second assessment message for the same turn_id.
  • Client keeps the pending entry so the late grammar message merges into the same turn in place (no duplicate, updates the bubble).
  • Client timeout 30s → 45s as a safety net.

Result: the learner sees pron score + phonemes in ~3–5s; grammar fills in after.

🤖 Generated with Claude Code

Assessment waited for both SOE (~3s) and the grammar LLM (~15s) before publishing
one message, so the UI hit its timeout ('Assessment is taking too long') and showed
nothing. Now pronunciation is published the moment it lands and grammar is sent in a
second 'assessment' message that updates the same turn in place (client keeps the
pending entry so the late grammar still merges). Bumped the client timeout to 45s.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 7, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
uiniq 5a1696a Jun 07 2026, 10:45 AM

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Zymonody7
Zymonody7 merged commit 9342cfd into main Jun 7, 2026
4 of 5 checks passed
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.

1 participant