Greptile Summary

Fixed flaky test in src/cli/doctor/runner.test.ts by increasing sleep duration from 10ms to 50ms while keeping the assertion at >= 10ms.

Key Changes:

• Changed setTimeout(r, 10) to setTimeout(r, 50) in the duration measurement test (line 33)
• Provides 5x buffer to prevent timing-related test failures
• Maintains the same validation logic for duration measurement

Technical Context:
The original 10ms sleep with a >= 10ms assertion was too tight due to timer precision limitations, measurement overhead, and non-deterministic setTimeout behavior. The fix follows TDD best practices from AGENTS.md by making a flaky test reliably pass without changing its purpose.

Test Results:

• Specific test now passes consistently
• Full test suite: 607 pass, 0 fail across 53 files

Confidence Score: 5/5

• This PR is safe to merge with no risk
• The fix correctly addresses a flaky test by increasing the sleep duration from 10ms to 50ms, providing a 5x buffer while maintaining the same >= 10ms assertion. This is a minimal, focused change that follows TDD best practices by making a failing test reliably pass without altering the test's purpose of validating duration measurement functionality
• No files require special attention

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Test as Test Suite
    participant runCheck as runCheck()
    participant perf as performance.now()
    participant check as check.check()
    participant timer as setTimeout

    Test->>runCheck: Execute test with CheckDefinition
    runCheck->>perf: start = performance.now()
    Note over perf: Record start time
    runCheck->>check: await check.check()
    check->>timer: setTimeout(resolve, 50ms)
    Note over timer: Sleep for 50ms<br/>(changed from 10ms)
    timer-->>check: resolve after 50ms
    check-->>runCheck: return { status: "pass", ... }
    runCheck->>perf: performance.now() - start
    Note over perf: Calculate elapsed time<br/>(~50ms measured)
    runCheck->>runCheck: Math.round(duration)
    Note over runCheck: Round to integer
    runCheck-->>Test: return result with duration
    Test->>Test: expect(duration).toBeGreaterThanOrEqual(10)
    Note over Test: Assert >= 10ms<br/>(passes with 50ms sleep)