Skip to content

fix(config): Honor resolved dependency filters#82

Open
reneleonhardt wants to merge 1 commit into
JordanCoin:mainfrom
reneleonhardt:feat/config-only-fallback-followup
Open

fix(config): Honor resolved dependency filters#82
reneleonhardt wants to merge 1 commit into
JordanCoin:mainfrom
reneleonhardt:feat/config-only-fallback-followup

Conversation

@reneleonhardt

@reneleonhardt reneleonhardt commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Honor resolved --only and --exclude filters in dependency, importer, and stdin scans.
  • Reuse those filters in blast-radius graph construction without redundant config loads or filtering.
  • Replace indirect filtered-match hints with direct, JSON-safe .codemap/config.json actions.

Why

This addresses the three follow-up observations from #77: CLI filter precedence did not reach every dependency surface, the guidance spoke about the agent instead of to it, and graph construction repeated configured filtering.

Verification

  • macOS arm64, Go 1.26.5: go vet ./..., staticcheck ./..., and go test -race -coverprofile=coverage.out ./...
  • Linux arm64, Go 1.24.13: go vet ./... and go test -race -coverprofile=coverage.out ./...
  • ast-grep 0.44.1 exercised on both platforms
  • Final whole-diff adversarial review: no Critical, Important, or Minor findings

Developed with carefully directed, manually reviewed AI assistance.

Co-Authored-By: GPT-5.6 Sol codex@openai.com

Thread effective CLI and project filters through dependency, importer, stdin,
and blast-radius scans without redundant config loads. Give filtered-match
guidance direct JSON-safe config actions.

Co-Authored-By: GPT-5.6 Sol <codex@openai.com>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

2 participants