Skip to content

Refresh tool category docs#96

Open
abhinavgautam01 wants to merge 2 commits into
git-pkgs:mainfrom
abhinavgautam01:issue-44-library-category-cleanup
Open

Refresh tool category docs#96
abhinavgautam01 wants to merge 2 commits into
git-pkgs:mainfrom
abhinavgautam01:issue-44-library-category-cleanup

Conversation

@abhinavgautam01

Copy link
Copy Markdown
Contributor

Closes #44.

Summary

Refreshes the generated README tool list so detection-only libraries are shown under a separate Library section instead of being mixed into Build.

Also updates CONTRIBUTING.md to document the current category set, including library, codegen, database, infrastructure, monorepo, environment, i18n, release and coverage.

Validation

go test ./...
go run ./cmd/brief list -readme tools | rg '^\*\*(Build|Library):'

@andrew andrew left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

@abhinavgautam01 abhinavgautam01 force-pushed the issue-44-library-category-cleanup branch from 5aa85fc to 8af12eb Compare June 15, 2026 08:59
@abhinavgautam01

Copy link
Copy Markdown
Contributor Author

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.

Thanks, fixed.

Rebased this branch onto current main and refreshed the generated README block with go run ./cmd/brief list -readme tools.

The README now reports 534 tool definitions and the Library section includes the ten definitions merged in #94: Busboy, Celery, Fabric, JDOM, Laravel Sanctum, Spring Security, Tera, Velocity, cross-spawn and dom4j.

Validation: go test ./...

@andrew andrew left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

@abhinavgautam01 abhinavgautam01 force-pushed the issue-44-library-category-cleanup branch from 59ea0ab to 8af12eb Compare June 29, 2026 01:10
@abhinavgautam01

Copy link
Copy Markdown
Contributor Author

thanks for the review, @andrew !
I have gone ahead and dropped the ci.yml change from this PR as requested, so it now strictly focuses on the tool category docs refresh. I'll open a separate PR for the Windows Defender CI change shortly...

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.

Build category in tool list is a dumping ground

2 participants