Skip to content

fix(cli): avoid render test hook timeout#2780

Merged
miguel-heygen merged 1 commit into
mainfrom
fix/main-cli-render-hook-timeout
Jul 25, 2026
Merged

fix(cli): avoid render test hook timeout#2780
miguel-heygen merged 1 commit into
mainfrom
fix/main-cli-render-hook-timeout

Conversation

@miguel-heygen

Copy link
Copy Markdown
Collaborator

Summary

  • collect the heavy render command module at file scope after Vitest hoists mocks
  • remove three deadline-limited beforeAll imports
  • keep the existing 30s hook timeout unchanged

Root cause

Required Test failed on main at https://github.com/heygen-com/hyperframes/actions/runs/30159241131/job/89681765017 because packages/cli/src/commands/render.test.ts cold-loaded the render module graph inside beforeAll. Under parallel monorepo CI contention, module collection exceeded the 30s hook deadline. The triggering parser merge did not touch the CLI test; it exposed this existing timing dependency.

Verification

  • RED before fix: bunx vitest run src/commands/render.test.ts --hookTimeout=100 timed out in all three import hooks
  • GREEN after fix: same command passes 69/69 tests
  • bun run typecheck in packages/cli
  • bun run test in packages/cli: 164 files passed, 1 skipped; 2,139 tests passed, 2 skipped
  • bunx oxfmt --check src/commands/render.test.ts
  • pre-commit lint, format, fallow, tracked-artifacts, and typecheck hooks

@miguel-heygen
miguel-heygen merged commit 3b3d4f5 into main Jul 25, 2026
45 checks passed
@miguel-heygen
miguel-heygen deleted the fix/main-cli-render-hook-timeout branch July 25, 2026 14:13
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