Skip to content

docs(audit): inventory docs/** refs to gutted features (#2530)#2551

Merged
alexey-pelykh merged 2 commits intomainfrom
docs/2530-audit-gutted-feature-refs
Apr 24, 2026
Merged

docs(audit): inventory docs/** refs to gutted features (#2530)#2551
alexey-pelykh merged 2 commits intomainfrom
docs/2530-audit-gutted-feature-refs

Conversation

@alexey-pelykh
Copy link
Copy Markdown

Summary

  • Classifies every docs/**/*.md reference to features removed during the OpenClaw → RemoteClaw fork transition (skills marketplace, remoteclaw models CLI, ClawHub plugin/skill install path, consumer onboarding UX).
  • Grounds each classification in the authoritative removal contract (docs/install/breaking-changes-from-openclaw.md) AND the actual CLI surface registered in src/cli/program/register.*.ts.
  • Deliverable lives at docs/refactor/docs-gutted-feature-audit.md with a follow-up plan grouped into 6 rewrite themes (one already tracked by gut(backend+docs): remove autoAllowSkills from config schema and documentation #2538).

Closes #2530 (SPIKE — rewrites spawn as follow-up items).

Method

  1. Read the authoritative removal contract.
  2. Verify each referenced CLI command actually exists via grep of src/cli/program/register.*.ts and src/cli/plugins-cli.ts.
  3. Grep docs/ for the confirmed-removed invocations:
    • rg -l "remoteclaw models " → 9 files, 46 refs (no models registrar in src/)
    • rg -l "remoteclaw skills " → 2 files, 7 refs (no skills registrar in src/)
    • rg -l "clawhub:" → 4+ files (no clawhub: locator in src/)
    • rg -l "clawhub|ClawHub" → 13 files (mostly marketplace references)
    • rg -l "autoAllowSkills" → 3 files (already tracked in gut(backend+docs): remove autoAllowSkills from config schema and documentation #2538)
  4. Classify each hit: STALE / ACCURATE / CROSS-REF / HISTORICAL (taxonomy from issue body).

Counts

Classification Files Refs
STALE — remoteclaw models CLI 10 46
STALE — remoteclaw skills CLI 2 7
STALE — ClawHub marketplace 7 ~20
STALE — ClawHub threat model 2 ~15
STALE — autoAllowSkills (tracked in #2538) 3 3
STALE — Dead /tools/skills, /tools/clawhub, /tools/skills-config links 8 9
HISTORICAL (preserved) 9
ACCURATE (explicitly verified) 10

AC (per #2530)

  • Classified inventory (table of file : section/line : classification) — docs/refactor/docs-gutted-feature-audit.md
  • Follow-up plan for STALE sections (6 themed rewrite groups documented in the audit file's "Follow-up plan" section)

SPIKE convention: rewrites spawn as follow-up items. Per the issue body's deliverable description ("open a docs(audit): ... follow-up issue or PR batch"), follow-up issues can be filed from the audit's follow-up plan when ready to act. The autoAllowSkills row is already covered by #2538, so not duplicated.

Test plan

  • Docs CI (docs job) renders the new audit file
  • Lint CI passes (oxfmt + oxlint + CSS class drift — ran locally via pre-commit hooks)
  • Dead-link check on /install/breaking-changes-from-openclaw anchor in audit file
  • Manual: preview the audit tables in Astro/Starlight docs build to confirm the CLI-surface evidence table (which had a pipe-in-regex rendering bug on first commit) renders cleanly

Re-verification for future maintainers

Included in the audit file as a standalone script block — the three CLI surfaces this audit depends on can be re-verified with 6 rg commands after any fork-sync.

🤖 Generated with Claude Code

alexey-pelykh and others added 2 commits April 24, 2026 16:49
Classifies every `docs/**/*.md` reference to features removed during the
OpenClaw→RemoteClaw fork transition. Grounds each classification in the
authoritative removal contract (`docs/install/breaking-changes-from-openclaw.md`)
and the actual CLI surface registered in `src/cli/program/register.*.ts`.

Method: rg for command invocations (`remoteclaw models *`, `remoteclaw skills *`,
`clawhub:`, `autoAllowSkills`) and marketplace brand names. Cross-check against
`src/cli/program/` registrars to confirm what does and does not exist.

Summary (per audit file):
- STALE — `remoteclaw models` CLI: 10 files, 46 refs (command not registered)
- STALE — `remoteclaw skills` CLI: 2 files, 7 refs (command not registered)
- STALE — ClawHub marketplace: 7 files, ~20 refs (`clawhub:<pkg>` locator absent in src/)
- STALE — ClawHub threat model: 2 files (attack surface no longer exists)
- STALE — `autoAllowSkills`: 3 files (already tracked in #2538)
- STALE — Dead links to `/tools/skills`, `/tools/clawhub`, `/tools/skills-config`: 8 files
- HISTORICAL (preserved): 9 files
- ACCURATE (verified): 10 files

Deliverable: inventory at `docs/refactor/docs-gutted-feature-audit.md` with
follow-up plan grouped into 6 rewrite themes (auto-deduplicated against #2538).
Rewrites spawn as follow-up items per SPIKE convention.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…dence row

The `clawhub:<package>` row's evidence cell contained an rg pattern with
alternation pipes (`clawhub:|clawHub|...`), which markdown parsers interpret
as column delimiters. oxfmt auto-reflowed the table on first commit, splitting
the cell across phantom columns. Rewrite the evidence in prose to avoid `|`
in inline code inside a table cell.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@alexey-pelykh alexey-pelykh merged commit 39a6643 into main Apr 24, 2026
16 checks passed
@alexey-pelykh alexey-pelykh deleted the docs/2530-audit-gutted-feature-refs branch April 24, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

spike(docs): audit docs/** for references to gutted features

1 participant