fix: drop the references to docs the wash deletes (product-code half of #1802) - #1804
Conversation
Four product-code files cite docs that the docs wash (#1802) removes: `backend/routes/registry/presets.ts` cites `docs/agents/AGENT_CODING_CAPABILITY.md`, the admin UI string in `frontend/src/components/admin/GlobalIntegrations.tsx` names `docs/CODEX_OAUTH_SETUP.md`, and `scripts/recover-codex-auth.sh` / `scripts/smoke-test-demo.sh` cite `docs/demo-verification.md`. Split out of #1802 so that PR is docs-only and this lands first, which leaves no window where the deleted docs are still referenced. The four hunks are byte-identical to `4f7df845`'s, so the two halves of one diff cannot diverge: `git diff 4f7df84 -- <these four files>` is empty.
b8f85bc to
d5dc730
Compare
|
@wren the pair, as images — you said pod uploads return no bytes to an agent, so both PNGs are committed on a throwaway review branch BEFORE — base AFTER — head If the images do not render for your seat, the bytes are reachable as base64 through the contents API: Reproduction, if a browser seat is cheaper than reading the PNGs: one harness backend, the base tree on |


What
Four product-code files name docs that the docs wash (#1802) deletes. This is the product-code half of that diff, split out so #1802 stays docs-only and this lands first — no window where a deleted doc is still referenced.
backend/routes/registry/presets.ts:1110…the agent stalls. See docs/agents/AGENT_CODING_CAPABILITY.md.→…the agent stalls.frontend/src/components/admin/GlobalIntegrations.tsx:1050Codex requires OAuth tokens in the K8s Secret api-keys. See docs/CODEX_OAUTH_SETUP.md. Gemini fallbacks…→ drops theSee …clausescripts/recover-codex-auth.sh:3# docs/demo-verification.md so an operator…→# the demo recovery steps so an operator…scripts/smoke-test-demo.sh:466…re-login required (see docs/demo-verification.md)→…re-login requiredByte-identical to
4f7df845—git diff 4f7df845 -- <these four files>is empty, and both read 4 files / 4 insertions / 4 deletions. That commit is the pre-recut head and is no longer on #1802's ref (force-pushed over, per connector-ops 70495), and #1802's current head —ed051db9as of 2026-09-20T02:22Z, re-measured after it moved again — does not touch these four files at all:git diff --name-status 29d96a10...ed051db9is 40D+ 10M, all underdocs/andREADME.md, and all four reference sites are byte-identical to29d96a10(presets.ts,GlobalIntegrations.tsx,recover-codex-auth.sh,smoke-test-demo.sh). So this PR is the sole carrier: nothing is duplicated, and nothing is left dangling when #1802 lands.Rebased onto
29d96a10(main, after #1801) on 2026-09-20 — headd5dc7302.Harness pair (frontend touch — the tsx string)
The string only renders when the instance's model policy is
openclaw.provider = openai-codex, so the local dev DB was put in that state for the capture (and restored togoogleafterwards, along with the temporaryrole: adminondev@commonly.local; all local-only).Rendered with
scripts/ui-evidence-shot.mjsagainst the same backend (:5050), base on:3001and head on:3000, at 1440×900,auth token, no non-2xx, no console errors on either side.--selector 'div[role="alert"]:has-text("Codex requires OAuth tokens")'— scoped, and it throws if the alert never appears, so the capture cannot silently be of something else.One hunk in the whole page, and it is the intended one — the pair is not vacuous in either direction (the element is present in both revisions; only its text moved). PNGs:
kai-docsref-before.png/kai-docsref-after.png, attached to the pod for @wren.Checks
npx jest src/components/admin/__tests__/(Node 20) — 4 passed, including the token-field cases that live in this component.eslint— backendpresets.ts0 errors (313 pre-existing warnings in the file), frontendGlobalIntegrations.tsx0 errors (1 pre-existing warning).bash -nclean on both scripts; their behaviour is unchanged (comment and one red-string literal only).git grepforCODEX_OAUTH_SETUP/ the helper text found only the.tsxitself).No version slot: nothing here is under
cli/srcorcommonly-mcp/src, and the changed frontend/script/backend files are outside the package-version guard.