PR Review Summary

Prek Checks

✅ All checks pass — ruff check and ruff format reported no issues.

Mypy

✅ No type errors found in changed Python files (codeflash/version.py).

Code Review

✅ No critical issues found.

Changes reviewed:

• packages/codeflash/runtime/capture.js: Adds !hasExternalLoopRunner guards to 5 early-exit conditions in the capturePerf loop. When an external loop runner (Jest) manages batching, batchSize is already set to 1, so the loop body executes exactly once. The guards prevent the early-exit checks (shared time limit, stability, max loops, error) from incorrectly breaking out before the single execution completes. Logic is sound — the async path (_capturePerfAsync) doesn't need the same guards since it's only entered when batchSize > 1, which can't happen with an external runner.
• codeflash/version.py: Auto-generated version bump (no logic change).

Test Coverage

Overall Python coverage: 79% (2374 passed, 8 failed, 57 skipped)

Notes:

• The primary change is in capture.js, a JavaScript runtime file not covered by Python pytest/coverage tooling.
• The 8 test failures (test_tracer.py) are pre-existing on main and unrelated to this PR.
• capturePerf has no dedicated JS unit tests (packages/codeflash/runtime/__tests__/ only contains dynamic-env-vars.test.js). Consider adding JS tests for this critical benchmarking function in a follow-up.

Last updated: 2026-02-17