Codex review: keeping this open for maintainer follow-up; there is still a little grit to resolve.

Summary
Keep open: this is a valid source-reproducible doctor bug, but #79203 is already open with closing syntax, so this issue should wait for that PR to be reviewed and merged.

Reproducibility: yes. from source inspection: current main can emit a runtime note from the daemon install plan and then separately emit doctor runtime/audit guidance for the same Node migration path. I did not run a live daemon install because this review is read-only.

Next step
The issue is already paired with an open fixing PR, so ClawSweeper should not create a competing repair job.

Review details

Best possible solution:

Land the linked PR or an equivalent focused fix that suppresses only the duplicate fallback warning while preserving real unsupported or missing Node guidance.

Do we have a high-confidence way to reproduce the issue?

Yes, from source inspection: current main can emit a runtime note from the daemon install plan and then separately emit doctor runtime/audit guidance for the same Node migration path. I did not run a live daemon install because this review is read-only.

Is this the best way to solve the issue?

Yes: the linked PR is the narrowest maintainable direction because it keeps the specific install-plan warning and only suppresses the duplicate doctor fallback for an already-selected Node runtime.

Acceptance criteria:

pnpm test src/commands/doctor-gateway-services.test.ts

What I checked:

Linked fix PR is open: The linked PR is open, targets the duplicate gateway runtime warning, and its body says it closes this issue. (5b52527feb9c)
Install plan emits runtime warnings: buildGatewayInstallPlan emits daemon runtime warnings through the supplied warn callback after resolving program arguments. (src/commands/daemon-install-helpers.ts:535, 79e31421225b)
Doctor also emits runtime repair warnings: Current main separately checks needsNodeRuntime and prints gateway runtime notes before rendering service audit issues, which leaves a duplicate-warning path. (src/commands/doctor-gateway-services.ts:425, 79e31421225b)
Runtime audit produces the migration issue: The service audit reports version-manager Node as a recommended gateway runtime migration issue. (src/daemon/service-audit.ts:498, 79e31421225b)
PR patch targets the duplicate path: The linked PR guards the fallback doctor runtime warning with runtimeChoice !== "node" and adds a regression test for the duplicate-warning case. (src/commands/doctor-gateway-services.ts:425, 5b52527feb9c)
Docs define the intended warning behavior: Gateway doctor is documented to warn about Bun/version-manager runtimes and offer migration to system Node, not to repeat contradictory warnings. Public docs: docs/gateway/doctor.md. (docs/gateway/doctor.md:490, 79e31421225b)

Likely related people:

steipete: GitHub commit history shows repeated recent work on gateway doctor/service runtime behavior, and the open linked fix PR is authored by this handle. (role: recent maintainer and fix author; confidence: high; commits: 955b0256976a, eb3e4f20a0af, 67f1266fe8a2; files: src/commands/doctor-gateway-services.ts, src/daemon/service-audit.ts, src/commands/doctor-gateway-services.test.ts)
Daev Mithran: Local blame in this checkout attributes the central gateway doctor service and runtime audit code to commit 9e1e597. (role: introduced current local implementation; confidence: medium; commits: 9e1e59717ffd; files: src/commands/doctor-gateway-services.ts, src/daemon/service-audit.ts, src/commands/daemon-install-runtime-warning.ts)
vincentkoc: Remote history shows recent doctor service-repair work in the same file area before this report. (role: recent adjacent maintainer; confidence: medium; commits: 24fc40b1332e; files: src/commands/doctor-gateway-services.ts)

Codex review notes: model gpt-5.5, reasoning high; reviewed against 79e31421225b.

Uh oh!

Uh oh!

Gateway doctor duplicates Node runtime warnings #79243

Description

Metadata

Metadata

Assignees

Labels

Type

Fields

Projects

Milestone

Relationships

Development

Issue actions