Skip to content

Refresh tool category docs#96

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

Refresh tool category docs#96
abhinavgautam01 wants to merge 3 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 ./...

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