docs: fix docs metadata spellcheck - #93502
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 21, 2026, 9:39 AM ET / 13:39 UTC. Summary PR surface: Docs 0, Other +2. Total +2 across 2 files. Reproducibility: yes. Source inspection shows current main still uses stale frontmatter keys on the page while Review metrics: none identified. Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Next step before merge
Security Review detailsBest possible solution: Merge the narrow docs metadata and ignore-list update after ordinary maintainer review; avoid adding parser aliases for one stale page. Do we have a high-confidence way to reproduce the issue? Yes. Source inspection shows current main still uses stale frontmatter keys on the page while Is this the best way to solve the issue? Yes. Updating the one stale page and adding ignores for two documented camelCase identifiers is narrower than adding parser aliases or broader spellcheck suppression. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 11a2e03bd4de. Label changesLabel justifications:
Evidence reviewedPR surface: Docs 0, Other +2. Total +2 across 2 files. View PR surface stats
What I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
80e031c
into
openclaw:main
Summary
docs/specs/claw-supervisor.mdfrontmatter from the olddescription/readWhenkeys to the canonicalsummary/read_whenkeys parsed bypnpm docs:list.allowInandplanTextto the docs spellcheck ignore list because they are documented camelCase API/config identifiers, not typos.Real behavior proof
Behavior or issue addressed:
pnpm docs:listcould not surface the Claw Supervisor doc summary/read-when metadata because that page used stale frontmatter keys, and docs spellcheck needed ignores for real camelCase docs tokens.Real environment tested: Local OpenClaw checkout on macOS, on branch
codex/docs-metadata-spellcheckat111fb71783cf5c5572becd8df148122cb2efa814.Exact steps or command run after this patch:
pnpm docs:listandpnpm docs:spellcheck.Evidence after fix (screenshot, recording, terminal capture, console output, redacted runtime log, linked artifact, or copied live output): Copied terminal output from the local checkout after this patch:
Observed result after fix:
specs/claw-supervisor.mdappears in the docs listing with the intended summary and read-when hints, and the docs spellcheck command exits with no diagnostics forallowInorplanText.What was not tested: Runtime behavior, packaging, or the full product suite; this is a docs metadata and spellcheck configuration patch only.
Proof limitations or environment constraints:
pnpm docs:spellcheckis quiet on success, so the terminal capture shows the invoked script with no diagnostics after it.Tests and validation
pnpm docs:listpnpm docs:spellcheckpnpm docs:check-mdxpnpm docs:check-linkspnpm docs:check-i18n-glossarypnpm format:docs:checkpnpm lint:docspnpm exec oxfmt --check --threads=1 docs/specs/claw-supervisor.md scripts/codespell-ignore.txtgit diff --check -- docs/specs/claw-supervisor.md scripts/codespell-ignore.txtReview
Claude autoreview reported the patch as correct and landable. It raised one P3 question about whether
allowInwas needed in the ignore list; I verified that removingallowInmakes codespell fail on the bareallowIntokens indocs/channels/groups.md, so the ignore entry is intentional.Risk checklist
Did user-visible behavior change?
Yes, docs tooling now reads this page's summary/read-when metadata correctly.Did config, environment, or migration behavior change?
No.Did security, auth, secrets, network, or tool execution behavior change?
No.What is the highest-risk area? Low docs-tooling risk: keeping the codespell ignore list scoped to real docs identifiers.
How is that risk mitigated? Both ignored tokens were verified against documented identifiers in
docs/**, and docs spellcheck passes after the patch.Current review state
Ready for review and merge once GitHub checks are green.