Skip to content

[test] ci: harden voice-library cold reopen waits - #255

Merged
Probiotics5333 merged 2 commits into
ThinkFlowLab:mainfrom
wangtiance:codex/issue-250-voice-library-waits
Nov 10, 2026
Merged

Probiotics5333 merged 2 commits into
ThinkFlowLab:mainfrom
wangtiance:codex/issue-250-voice-library-waits

Conversation

@wangtiance

Copy link
Copy Markdown
Collaborator

Summary

Voice-library Release checks used the ordinary 15-second interaction budget for cold project recovery and reopened audio downloads. Give those waits explicit 45-second limits, navigate to DOMContentLoaded before checking application readiness, and report unsuccessful media GETs directly instead of hiding them behind a response timeout. Ordinary interactions retain their 15-second limit; existing persisted-job, adoption, asset identity, MIME and playback assertions remain.

Add an optional VOICE_REOPEN_DELAY_MS probe and bilingual instructions. No generation retry or product behavior changes.

Closes #250.

Validation

  • Windows 11 / Edge: unmodified baseline passed en-US and zh-CN (original CI flake not reproduced).
  • Final source: full en-US and zh-CN workflow passed with VOICE_REOPEN_DELAY_MS=17000.
  • Negative control: same delayed workflow with the reopen budget restored to 15000 failed at waitForResponse with Timeout 15000ms exceeded.
  • node --check, browser manifest, npm run check:i18n (2941 keys, zero problems), git diff --check passed.
  • Real isolated project storage, media probing and WAV generation; provider HTTP mocked. No paid model calls. Linux GitHub Release checks remain for CI to verify.

中文:冷启动和重开等待单独设为45秒,普通交互仍为15秒;错误媒体响应直接报告HTTP状态。最终双语17秒延迟回归通过,15秒旧预算负例按预期超时。未复现原始CI偶发失败,仍需GitHub CI验证。

Use explicit 45-second cold-load and reopen waits while retaining the 15-second interaction timeout. Surface failed media HTTP responses directly and provide a deterministic slow-response probe. Refs ThinkFlowLab#250.
…ce-library-waits

# Conflicts:
#	tests/browser/voice_library.mjs
@Probiotics5333

Copy link
Copy Markdown
Collaborator

/lgtm

@Probiotics5333
Probiotics5333 merged commit 090c981 into ThinkFlowLab:main Nov 10, 2026
1 check passed
@FayeSpica FayeSpica added area:ci Continuous integration and checks type:ci Build, test and release automation labels Nov 10, 2026 — with ChatGPT Codex Connector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ci Continuous integration and checks type:ci Build, test and release automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[fix] ci: Voice library Playwright TimeoutError flakes Release checks

3 participants