Skip to content

docs: register web-pentest skill in optional-skills catalog + sidebar - #32278

Closed
r266-tech wants to merge 1 commit into
NousResearch:mainfrom
r266-tech:docs/catalog-web-pentest-skill
Closed

docs: register web-pentest skill in optional-skills catalog + sidebar#32278
r266-tech wants to merge 1 commit into
NousResearch:mainfrom
r266-tech:docs/catalog-web-pentest-skill

Conversation

@r266-tech

Copy link
Copy Markdown
Contributor

Summary

The web-pentest optional skill (#32265) was merged without running website/scripts/generate-skill-docs.py, so it is missing from:

  • website/docs/reference/optional-skills-catalog.md (no catalog row)
  • website/sidebars.ts (no sidebar entry)
  • website/docs/user-guide/skills/optional/security/ (no generated per-skill page)

Its two same-batch siblings — code-wiki (#32240) and openhands (#32261) — both included these generated docs, so this is a drift left by the third skill in the batch. The other optional-skills/security/ entries (1password, oss-forensics, sherlock) are all cataloged; web-pentest was the only one missing.

Changes

Generated via generate-skill-docs.py, scoped to web-pentest only:

  • Catalog row in the security section (link prefix matched to the existing committed rows)
  • Sidebar entry after security-sherlock
  • New per-skill page security-web-pentest.md (auto-generated from the skill's SKILL.md)

Pure docs/config; no code or skill behavior changes. If this has already been regenerated via a separate run, feel free to close and mark accordingly.

… and user-guide

The web-pentest optional skill (NousResearch#32265) shipped without running
website/scripts/generate-skill-docs.py, so it is missing from the
optional-skills catalog, the docs sidebar, and has no generated
user-guide page — unlike its same-batch siblings code-wiki (NousResearch#32240)
and openhands (NousResearch#32261), which both included the generated docs.

Adds the generated catalog row, sidebar entry, and per-skill page so
the skill is discoverable on the docs site, consistent with the other
optional-skills/security entries (1password, oss-forensics, sherlock).
@alt-glitch alt-glitch added type/docs Documentation improvements P3 Low — cosmetic, nice to have labels May 25, 2026
@teknium1

Copy link
Copy Markdown
Contributor

Thanks for identifying the generated-docs drift.

  • Current main already contains the catalog row at website/docs/reference/optional-skills-catalog.md:209, the generated page at website/docs/user-guide/skills/optional/security/security-web-pentest.md:1, and the sidebar entry at website/sidebars.ts:573.
  • These artifacts were added by 8b6beaab5f8c825ad156ebcc0acd566e1673b7a2 (docs: 30-day overhaul — correctness audit, PR coverage, Nous Portal weave, sidebar reorg (#33782)).
  • Current generator output uses the /docs/user-guide/... catalog URL form in website/scripts/generate-skill-docs.py:559, which is what current main contains.

This is an automated hermes-sweeper review; closing as implemented on main.

@teknium1 teknium1 closed this Jul 13, 2026
@teknium1 teknium1 added the sweeper:implemented-on-main Sweeper: behavior already present on current main label Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have sweeper:implemented-on-main Sweeper: behavior already present on current main type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants