Skip to content

Stabilize CI run tests and fix timeout-hang root cause#12

Merged
eve-senara merged 1 commit intomasterfrom
fix/ci-codex-test-determinism-v0.2.21
Feb 23, 2026
Merged

Stabilize CI run tests and fix timeout-hang root cause#12
eve-senara merged 1 commit intomasterfrom
fix/ci-codex-test-determinism-v0.2.21

Conversation

@eve-senara
Copy link
Collaborator

Summary

  • fix timeout/hang in run command tests by removing fragile setup module mock path
  • set deterministic OPENAI_API_KEY in run test harness beforeEach/afterEach
  • keep task-runner test deterministic by stubbing executeTask in failed-status path

Validation

  • npm run lint
  • npm run lint:type-aware
  • npm run typecheck
  • npm run test
  • npm run build
  • npm run validate
  • bun test src/core/task-runner.test.ts
  • bun test src/cli/commands/run.test.ts
  • bun test src/cli/commands/run.postexec-json.integration.test.ts

@github-actions
Copy link

No issues found. This PR stabilizes CI test determinism by properly setting/restoring OPENAI_API_KEY in the run command test harness and adding a setExecuteTaskForTests stub to prevent non-deterministic behavior in the cyclic-task failure test path.

@eve-senara eve-senara merged commit 15024b3 into master Feb 23, 2026
1 check passed
@eve-senara eve-senara deleted the fix/ci-codex-test-determinism-v0.2.21 branch February 23, 2026 13:00
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