Skip to content

chore: add taxonomy file#91512

Merged
kevinslin merged 3 commits into
openclaw:mainfrom
kevinslin:dev/add-taxonomy
Jun 8, 2026
Merged

chore: add taxonomy file#91512
kevinslin merged 3 commits into
openclaw:mainfrom
kevinslin:dev/add-taxonomy

Conversation

@kevinslin

Copy link
Copy Markdown
Contributor

Summary

  • add docs/taxonomy.yaml with the maturity scorecard taxonomy definition
  • include the 2026-05-26 snapshot metadata, maturity levels, and documented surfaces/categories/features used by the docs taxonomy model

Verification

  • git diff --check origin/main...HEAD

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

clawsweeper Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Codex review: found issues before merge. Reviewed June 8, 2026, 5:49 PM ET / 21:49 UTC.

Summary
This PR adds docs/taxonomy.yaml and docs/maturity-scores.yaml as maturity scorecard taxonomy and score snapshots.

PR surface: Docs +13829. Total +13829 across 2 files.

Reproducibility: not applicable. This PR adds docs data rather than reporting a runtime bug. The review issues are source-reproducible by following the added YAML references against current main.

Review metrics: 3 noteworthy metrics.

  • Dangling scorecard references: 50 completeness references and 281 category-note references. These fields point to support files omitted from the PR, so consumers would not be able to inspect the scorecard evidence.
  • Current-main consumers: 0 consumers found. No current-main path was found that reads the new YAML files, so placement and ownership need maintainer confirmation.
  • Prior mirror history: 1 merged mirror reverted same day. The same scorecard material was recently added broadly and reverted, raising the bar for the narrower replacement shape.

Merge readiness
Overall: 🧂 unranked krab
Proof: 🌊 off-meta tidepool
Patch quality: 🧂 unranked krab
Result: blocked by patch quality or review findings.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

  • Resolve or remove the omitted scorecard support references.
  • [P2] Fix the missing ClawHub docs paths at docs/taxonomy.yaml:2938 and docs/taxonomy.yaml:3006.
  • Get maintainer confirmation for the intended consumer and public docs-data location.

Risk before merge

  • [P1] If merged as-is, taxonomy consumers that follow completeness_instructions or category_note cannot reach the evidence files from the reverted scorecard mirror.
  • [P1] The public docs-data placement is still a maintainer IA decision because current main has no direct consumer for the two YAML snapshots and the broader mirror was reverted the same day.

Maintainer options:

  1. Fix references before merge (recommended)
    Require the snapshot to resolve or remove all embedded docs, completeness, and category-note references before it becomes public docs data.
  2. Approve the public docs-data placement
    If maintainers want this exact public location, record the intended consumer and owner in the PR so the unresolved IA question is closed before landing.
  3. Pause after the reverted mirror
    Close or pause this PR if the maturity scorecard artifacts belong in private/generated docs rather than the OpenClaw public docs source tree.

Next step before merge

  • [P2] The remaining blocker is maintainer docs-IA ownership plus contributor fixes for dangling references, not a safe standalone automated repair lane.

Security
Cleared: The diff adds docs YAML data only; no executable code, workflows, dependencies, package metadata, or concrete secret values were found.

Review findings

  • [P2] Restore or remove the omitted scorecard support refs — docs/taxonomy.yaml:46
  • [P2] Point skill-format at an existing docs page — docs/taxonomy.yaml:2938
  • [P2] Remove the missing security-audits docs reference — docs/taxonomy.yaml:3006
Review details

Best possible solution:

Land only after maintainers confirm the intended owner/consumer and all embedded scorecard references either resolve in the approved location or are removed from the public snapshot.

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

Not applicable; this PR adds docs data rather than reporting a runtime bug. The review issues are source-reproducible by following the added YAML references against current main.

Is this the best way to solve the issue?

No; adding the two public YAML snapshots is not yet the best solution while their support references are dangling and the intended consumer/location is unconfirmed. The safer path is a maintainer-approved location with self-consistent references.

Full review comments:

  • [P2] Restore or remove the omitted scorecard support refs — docs/taxonomy.yaml:46
    The new snapshot keeps completeness_instructions and category_note pointers into the scorecard support tree, but this PR only adds the two YAML files and current main no longer has the referenced maturity-scorecard support files after the revert. Any consumer that follows those fields cannot inspect the evidence, so include the support files in the approved location or remove/normalize these pointers.
    Confidence: 0.9
  • [P2] Point skill-format at an existing docs page — docs/taxonomy.yaml:2938
    This entry points ClawHub publishing evidence at docs/clawhub/skill-format.md, but that page is not present in current main; the skill authoring docs live under docs/tools. Consumers following the taxonomy docs list will hit a dead path unless this is changed to an existing page or the missing page is added.
    Confidence: 0.93
  • [P2] Remove the missing security-audits docs reference — docs/taxonomy.yaml:3006
    The Compatibility and Trust section links to docs/clawhub/security-audits.md, but no such file exists under docs/. Because these docs entries are evidence links for the taxonomy, this should point at the actual trust/audit page or be removed until that page exists.
    Confidence: 0.93

Overall correctness: patch is incorrect
Overall confidence: 0.89

AGENTS.md: found and applied where relevant.

Codex review notes: model gpt-5.5, reasoning high; reviewed against 6c4fb997e52e.

Label changes

Label changes:

  • add P2: This is a normal-priority docs/data PR with merge-blocking data-quality and ownership questions but no runtime outage or security incident.
  • add merge-risk: 🚨 other: Merging would introduce public docs data with unresolved embedded evidence paths and an unsettled placement decision that CI alone does not settle.
  • add rating: 🧂 unranked krab: Overall readiness is 🧂 unranked krab; proof is 🌊 off-meta tidepool and patch quality is 🧂 unranked krab.
  • add status: ⏳ waiting on author: ClawSweeper has contributor-facing work open and is waiting for author action. Not applicable: Real behavior proof is not required because this PR only changes files under docs/.
  • remove rating: 🦐 gold shrimp: Current PR rating is rating: 🧂 unranked krab, so this older rating label is no longer current.
  • remove status: 👀 ready for maintainer look: Current PR status label is status: ⏳ waiting on author.

Label justifications:

  • P2: This is a normal-priority docs/data PR with merge-blocking data-quality and ownership questions but no runtime outage or security incident.
  • merge-risk: 🚨 other: Merging would introduce public docs data with unresolved embedded evidence paths and an unsettled placement decision that CI alone does not settle.
  • rating: 🧂 unranked krab: Overall readiness is 🧂 unranked krab; proof is 🌊 off-meta tidepool and patch quality is 🧂 unranked krab.
  • status: ⏳ waiting on author: ClawSweeper has contributor-facing work open and is waiting for author action. Not applicable: Real behavior proof is not required because this PR only changes files under docs/.
Evidence reviewed

PR surface:

Docs +13829. Total +13829 across 2 files.

View PR surface stats
Area Files Added Removed Net
Source 0 0 0 0
Tests 0 0 0 0
Docs 2 13829 0 +13829
Config 0 0 0 0
Generated 0 0 0 0
Other 0 0 0 0
Total 2 13829 0 +13829

What I checked:

Likely related people:

  • kevinslin: Authored the merged maturity scorecard mirror and the same-day revert, so he is strongly connected to the scorecard docs/data surface beyond opening this PR. (role: recent area contributor; confidence: high; commits: 6cc6f5e210f7, 4c5d8afa38c0; files: docs/maturity-scorecard/**, scripts/docs-sync-publish.mjs, docs/.i18n/glossary.zh-CN.json)
  • Vincent Koc: Recent current-main work added the maturity-scorecard docs-root segment in Markdown link handling, which is adjacent to where this scorecard data would be consumed or rendered. (role: adjacent docs owner; confidence: medium; commits: 105d77d486bf; files: ui/src/ui/markdown.ts)
  • Shakker: Blame on docs/AGENTS.md shows the current internal-docs and public-docs placement policy used to evaluate this PR. (role: docs policy author; confidence: medium; commits: da401341b685; files: docs/AGENTS.md, docs/docs.json)
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.

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.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e487783e18

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread taxonomy.yaml
Comment thread taxonomy.yaml
@clawsweeper clawsweeper Bot added rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Jun 8, 2026
@clawsweeper clawsweeper Bot added rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. P2 Normal backlog priority with limited blast radius. and removed rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Jun 8, 2026
@clawsweeper clawsweeper Bot added the merge-risk: 🚨 other 🚨 Merging this PR has meaningful risk outside the owned taxonomy. label Jun 8, 2026
@kevinslin kevinslin merged commit a54f50a into openclaw:main Jun 8, 2026
170 of 177 checks passed
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jun 9, 2026
* chore: add taxonomy file

* add maturity scores

* move taxonomy doc
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 merge-risk: 🚨 other 🚨 Merging this PR has meaningful risk outside the owned taxonomy. P2 Normal backlog priority with limited blast radius. rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. size: XL status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant