Refresh tool category docs#96
Conversation
andrew
left a comment
There was a problem hiding this comment.
The CONTRIBUTING.md table looks right and matches report.CategoryOrder.
The README block is generated though, and it's gone stale since #94 merged ten more library defs. Main now produces 534 tool definitions (this PR has 524) and the Library line is missing Busboy, Celery, Fabric, JDOM, Laravel Sanctum, Spring Security, Tera, Velocity, cross-spawn and dom4j.
Could you rebase on main and re-run go run ./cmd/brief list -readme tools to refresh the block between the brief:tools:start/end markers? After that this is good to go.
5aa85fc to
8af12eb
Compare
Thanks, fixed. Rebased this branch onto current The README now reports Validation: |
andrew
left a comment
There was a problem hiding this comment.
The docs refresh itself is good now: the README block regenerated to 534 definitions and all ten previously-missing libraries are listed, and the CONTRIBUTING table still matches report.CategoryOrder.
The one thing holding this up is the .github/workflows/ci.yml change wrapping the Windows Defender step in try/catch. It's a reasonable change on its own, but it's unrelated to refreshing the tool category docs and shouldn't ride along in this PR. Could you drop it from here and open it as its own PR? Once ci.yml is out, this is good to merge.
59ea0ab to
8af12eb
Compare
|
thanks for the review, @andrew ! |
Closes #44.
Summary
Refreshes the generated README tool list so detection-only libraries are shown under a separate
Librarysection instead of being mixed intoBuild.Also updates
CONTRIBUTING.mdto document the current category set, includinglibrary,codegen,database,infrastructure,monorepo,environment,i18n,releaseandcoverage.Validation