fix(gateway): surface unreachable status diagnostics#74691
fix(gateway): surface unreachable status diagnostics#74691clawsweeper[bot] merged 2 commits intomainfrom
Conversation
|
Codex review: passed. Summary Reproducibility: yes. for the missing diagnostic: current main can reach an all-unreachable, zero-discovery gateway probe state but has no warning branch or discovery-count input. No for the underlying #49012 freeze itself; that remains a separate root-cause investigation. Next step before merge Security Review detailsBest possible solution: Land this diagnostic patch after exact-head checks pass, while keeping #49012 open for the separate frozen gateway or port-conflict root cause. Do we have a high-confidence way to reproduce the issue? Yes for the missing diagnostic: current main can reach an all-unreachable, zero-discovery gateway probe state but has no warning branch or discovery-count input. No for the underlying #49012 freeze itself; that remains a separate root-cause investigation. Is this the best way to solve the issue? Yes. Deriving a warning from existing reachability and Bonjour discovery data is a narrow maintainable fix for collecting better diagnostics without pretending to fix the gateway freeze. What I checked:
Likely related people:
Codex review notes: model gpt-5.5, reasoning high; reviewed against b9eb31b54cfa. |
|
/clawsweeper automerge |
clawsweeper
Bot
added
the
clawsweeper:automerge
|
🦞🦞 Source: What merged:
Fixups included:
The automerge loop is complete. Automerge progress:
|
clawsweeper
Bot
added
the
clawsweeper:human-review
|
🦞🦞 Source: I added |
clawsweeper
Bot
removed
the
clawsweeper:human-review
clawsweeper
Bot
force-pushed
the
fix/gateway-status-unreachable-49012
branch
from
ab17840 to
50fb29c
Compare
Summary: - The PR adds a `no_gateway_reachable` gateway probe warning, passes discovery count into warning construction, adds focused coverage, and updates the changelog. - Reproducibility: yes. for the missing diagnostic: current main can reach an all-unreachable, zero-discovery ... -count input. No for the underlying openclaw#49012 freeze itself; that remains a separate root-cause investigation. ClawSweeper fixups: - Included follow-up commit: fix(gateway): surface unreachable status diagnostics Validation: - ClawSweeper review passed for head 50fb29c. - Required merge gates passed before the squash merge. Prepared head SHA: 50fb29c Review: openclaw#74691 (comment) Co-authored-by: Vincent Koc <vincentkoc@ieee.org> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
1 participant
Summary
no_gateway_reachableGateway status warning when every probe is unreachable and Bonjour discovers no local gatewaysRefs #49012. This is a diagnostic/remediation-data fix, not a root-cause closure.
Validation
OPENCLAW_TESTBOX=1 OPENCLAW_TESTBOX_ID=tbx_01kqdttjtb1nnj92za0mka8032 pnpm testbox:run --id tbx_01kqdttjtb1nnj92za0mka8032 -- pnpm test:serial src/commands/gateway-status/output.test.ts src/commands/gateway-status.test.ts— passed, 27 testsOPENCLAW_TESTBOX=1 OPENCLAW_TESTBOX_ID=tbx_01kqdttjtb1nnj92za0mka8032 pnpm testbox:run --id tbx_01kqdttjtb1nnj92za0mka8032 -- node scripts/run-oxlint.mjs --tsconfig tsconfig.oxlint.core.json src/commands/gateway-status.ts src/commands/gateway-status/output.ts src/commands/gateway-status/output.test.ts src/commands/gateway-status.test.ts— passedOPENCLAW_TESTBOX=1 OPENCLAW_TESTBOX_ID=tbx_01kqdttjtb1nnj92za0mka8032 pnpm testbox:run --id tbx_01kqdttjtb1nnj92za0mka8032 -- pnpm check:changed -- CHANGELOG.md src/commands/gateway-status.ts src/commands/gateway-status/output.ts src/commands/gateway-status/output.test.ts src/commands/gateway-status.test.ts— blocked by unrelatedpackages/sdk/src/index.test.tsoxlinttypescript-eslint(no-redundant-type-constituents)errors after conflict-marker, changelog-attribution, typecheck core, and typecheck core test steps passedAI-assisted: yes; diff reviewed before opening.