Skip to content

Commit 5378b9b

Browse files
rickcrawfordclaude
andcommitted
fix: search filters and add pre-commit build/test requirement
Search --protocol, --type, --tag filters were returning all results because fallback returned true for unknown tools. Now correctly excludes non-matching tools. Protocol filter also checks the SearchResult.Protocol field directly before falling back to index. Updated CLAUDE.md to require full build + test suite before every commit. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 88db428 commit 5378b9b

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

‎CLAUDE.md‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,13 @@
4545
- Test: `go test ./...`
4646
- Lint: `go vet ./...`
4747

48+
### Pre-Commit Requirements (MANDATORY)
49+
Before every commit, you MUST run both of these and verify they pass:
50+
1. **Full build:** `go build ./cmd/clictl/...`
51+
2. **Full test suite:** `go test ./...`
52+
53+
Do NOT commit if either fails. Fix the issue first. No exceptions.
54+
4855
### Testing Requirements
4956
- **Run `go test ./...` after every code change** to catch regressions
5057
- **Every new feature must include unit tests** before it is considered complete

0 commit comments

Comments
 (0)