Repository navigation
Conversation
An engine command that outlives its deadline failed with a bare "Operation timed out after '30s'", hiding which subcommand stalled and what the engine last reported. The timeout now names the subcommand and carries the stderr tail, keeping the engine-timeout and image-pull kinds. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MBEk1LaE2jUMLrpQ1WCiVy
|
/ai-review Generated by Claude Code |
CLI benchmark: no changes foundCommit
API calls: unchanged for all 7 commands. Binary size: 134.0 MB, unchanged. Details: timing ranges, API calls per command, slowest phasesTiming ranges
API calls per command
Slowest phasesNo command moved beyond noise, so there are no phase deltas. Comment |
There was a problem hiding this comment.
Superseded by a newer AI review
🤖 AI Review
Both independent reviews completed and reported no findings. Inspection of the changed code and surrounding runtime and test logic found no additional actionable issues. Tests and type checks were not run because dependencies are absent.
Findings
No issues found.
Stats
Claude findings: 0 · Codex findings: 0 · Confirmed: 0 · Refuted: 0 · Uncertain: 0
Models: claude-opus-5-5 + gpt-6.1-sol · Trigger: auto · Workflow run
This review runs once per PR. A maintainer can request another with a /ai-review comment.
There was a problem hiding this comment.
🤖 AI Review
Both independent reviews completed and reported no findings. Reading the changed code and surrounding launch/error handling revealed no actionable issues. Tests were not run because dependencies are not installed.
Findings
No issues found.
Stats
Claude findings: 0 · Codex findings: 0 · Confirmed: 0 · Refuted: 0 · Uncertain: 0
Models: claude-opus-5-5 + gpt-6.1-sol · Trigger: manual · Workflow run
This review runs once per PR. A maintainer can request another with a /ai-review comment.
Flaky test check:
|
| Run | Problem |
|---|---|
flaky-focused-run1 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-focused-run10 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-focused-run2 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-focused-run3 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-focused-run4 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-focused-run5 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-focused-run6 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-focused-run7 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-focused-run8 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-focused-run9 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-unit-run1 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-unit-run10 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-unit-run2 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-unit-run3 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-unit-run4 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-unit-run5 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-unit-run6 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-unit-run7 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-unit-run8 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
flaky-unit-run9 |
test step did not run or finish (setup failure, flaky-check error, timeout, or cancel); see the job log |
Each execution is a separate Vitest process; Where lists failing runs, as run.iteration for suites executed more than once per run. The full data is in the flaky-check-report artifact.
|
The red Generated by Claude Code |
TL;DR
Engine command timeouts now name the stalled subcommand and the engine's latest stderr, so the next occurrence of this flake shows which call hung.
Owner.logs.integration.test.ts › persists container output:startServicefor Mailpit failed withengine-timeoutand onlyOperation timed out after '30s'. The error didn't say which engine subcommand ran past the 30s default, so the cause couldn't be found from the logs.Container.commandnow fails withEngine command '<subcommand>' did not finish within <duration>; stderr: <tail>. It keeps theengine-timeout/image-pullkinds. A regression test drives a stalledstartwith a controlled clock.ref
spotted in: https://github.com/supabase/cli/actions/runs/40890624066/job/123506114202 (green rerun: https://github.com/supabase/cli/actions/runs/40890624066/job/123509245894)
Generated by Claude Code