Skip to content

docs: move native codex plugins into capabilities#84341

Merged
kevinlin-openai merged 1 commit into
openclaw:mainfrom
kevinslin:dev/kevinlin/codex-native-plugins-capabilities-tab
May 19, 2026
Merged

docs: move native codex plugins into capabilities#84341
kevinlin-openai merged 1 commit into
openclaw:mainfrom
kevinslin:dev/kevinlin/codex-native-plugins-capabilities-tab

Conversation

@kevinslin

Copy link
Copy Markdown
Contributor

Summary

  • move plugins/codex-native-plugins from the References tab to the Capabilities tab
  • place it in Bundled plugin guides directly under plugins/codex-harness

Verification

  • python - <<"PY"\nimport json\nfrom pathlib import Path\njson.loads(Path("docs/docs.json").read_text())\nprint("docs/docs.json parses")\nPY
  • git diff --check

@openclaw-barnacle openclaw-barnacle Bot added docs Improvements or additions to documentation size: XS maintainer Maintainer-authored PR labels May 19, 2026
@clawsweeper

clawsweeper Bot commented May 19, 2026

Copy link
Copy Markdown
Contributor

Codex review: needs maintainer review before merge.

Workflow note: Future ClawSweeper reviews update this same comment in place.

How this review workflow works
  • ClawSweeper keeps one durable marker-backed review comment per issue or PR.
  • Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
  • A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
  • PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
  • Maintainers can also comment @clawsweeper review to request a fresh review only.
  • Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
  • Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
  • Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

Summary
The PR moves plugins/codex-native-plugins in docs/docs.json from Reference > Codex harness to Capabilities > Bundled plugin guides directly after plugins/codex-harness.

Reproducibility: not applicable. this is a docs navigation PR, not a runtime bug report. Source inspection and the PR diff are enough to review the requested placement change.

PR rating
Overall: 🐚 platinum hermit
Proof: 🌊 off-meta tidepool
Patch quality: 🐚 platinum hermit
Summary: Small docs-only navigation change with a coherent placement rationale and no blocking findings; merge remains maintainer-gated.

Rank-up moves:

  • none
What the crustacean ranks mean
  • 🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
  • 🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
  • 🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
  • 🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
  • 🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
  • 🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
  • 🌊 off-meta tidepool: rating does not apply to this item.

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.

PR egg
✨ Hatched: ✨ glimmer Brave Test Hopper

        /\     /\            
      _/  \___/  \_          
     /  ( o   o )  \         
    |      \_/      |        
    |   /\  ===  /\ |        
     \_/  \_____/  \_/       
        _/|_| |_|\_          
       /__| | | |__\         
          ' ' ' '            
         /_/     \_\         
       `-----------'         
.*~~~~~~~~~~~~~~~~~~~*.      

Rarity: ✨ glimmer.
Trait: stacks clean commits.
Image traits: location proof lagoon; accessory CI status badge; palette cobalt, lime, and pearl; mood mischievous; pose leaning over a miniature review desk; shell smooth pearl shell; lighting subtle sparkle highlights; background delicate sparkle particles.
How to hatch it: once this PR reaches status: 👀 ready for maintainer look or status: 🚀 automerge armed, the PR author or a maintainer can comment @clawsweeper hatch to turn this ASCII egg into its generated creature image.
Share on X: post this hatch
Copy: My PR egg hatched a ✨ glimmer Brave Test Hopper in ClawSweeper.

What is this egg doing here?
  • Eggs appear after the PR passes real-behavior proof. It is here for vibes, not verdicts: it does not change labels, ratings, merge decisions, or automation.
  • The shell reacts to review momentum: open follow-up work warms it up, re-review makes it wobble, and a clean final review lets it hatch.
  • Hatchable usually means sufficient real-behavior proof, no blocking P0/P1/P2 findings, no security attention needed, and clean correctness.
  • The hatch is seeded from this repository and PR number, so the same PR keeps the same creature; the reviewed head SHA can only change safe visual details.
  • Rarity is just collectible sparkle: 🥚 common, 🌱 uncommon, 💎 rare, ✨ glimmer, and 🌈 legendary.

Real behavior proof
Not applicable: Real behavior proof is not required because this PR only changes files under docs/.

Risk before merge
Why this matters: - The remaining uncertainty is docs information architecture: a maintainer should confirm that Native Codex plugins belongs under Capabilities rather than the reference group before merge.

Maintainer options:

  1. Decide the mitigation before merge
    Merge the nav-only docs change if a maintainer agrees the Native Codex plugins guide should be discoverable beside the bundled Codex harness guide, with no runtime changes.
  2. Pause or close
    Do not merge this PR until maintainers decide whether the risk is worth taking.

Next step before merge
The protected maintainer label means cleanup automation should leave the PR open for explicit maintainer merge or close judgment; there is no narrow repair to queue.

Security
Cleared: Cleared: the diff only edits Mintlify docs navigation JSON and does not touch scripts, dependencies, workflows, secrets, or runtime code.

Review details

Best possible solution:

Merge the nav-only docs change if a maintainer agrees the Native Codex plugins guide should be discoverable beside the bundled Codex harness guide, with no runtime changes.

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

Not applicable: this is a docs navigation PR, not a runtime bug report. Source inspection and the PR diff are enough to review the requested placement change.

Is this the best way to solve the issue?

Yes, conditionally: moving the guide-style page next to the base Codex harness guide is a narrow maintainable docs change. Because the maintainer label is present, the final placement decision should come from a maintainer.

Label justifications:

  • P3: This is a low-risk documentation navigation cleanup with no runtime behavior change.

Acceptance criteria:

  • python -c 'import json,pathlib; json.loads(pathlib.Path("docs/docs.json").read_text())'
  • git diff --check

What I checked:

Likely related people:

  • kevinslin: Commit 9b97e1e added recent Codex plugin management docs and touched both the Codex harness and Native Codex plugins pages, so this person is the strongest routing candidate for the page placement. (role: recent Codex native plugin contributor; confidence: high; commits: 9b97e1ef2fd2; files: docs/plugins/codex-native-plugins.md, docs/plugins/codex-harness.md, extensions/codex/src/command-plugins-management.ts)
  • Ayaan Zaidi: Current-line blame for the docs navigation groups involved in this move points to commit 1f794d2, which carries the current docs/docs.json structure in this checkout. (role: docs navigation contributor; confidence: medium; commits: 1f794d2816d1; files: docs/docs.json)

Codex review notes: model gpt-5.5, reasoning high; reviewed against 5c9a8f33b312.

@clawsweeper clawsweeper Bot added rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. labels May 19, 2026
@kevinlin-openai kevinlin-openai merged commit ecb6da9 into openclaw:main May 19, 2026
59 of 66 checks passed
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 24, 2026
galiniliev pushed a commit to galiniliev/openclaw that referenced this pull request May 25, 2026
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation maintainer Maintainer-authored PR P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. size: XS status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants