Skip to content

fix: align configured rules with selected analysis - #752

Merged
kvlonge merged 1 commit into
mainfrom
fix/configured-rules-complete-analysis
Sep 20, 2026
Merged

kvlonge merged 1 commit into
mainfrom
fix/configured-rules-complete-analysis

Conversation

@kvlonge

@kvlonge kvlonge commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

Why

Selected planning and build commands restrict deep SQL analysis to their model scope, but configured rules were evaluated against the entire project. Unselected models could therefore produce false semantic or contract findings because their deep-analysis facts were intentionally absent.

Changes

  • resolve configured-rule subjects from the same selection policy used for deep SQL analysis;
  • preserve full-project rule evaluation when no selector is provided or Python-node resolution requires complete analysis;
  • add real CLI planning coverage for a typed unselected model;
  • retain focused/full rule-cache and project-rule boundary coverage.

Verification

  • 51 focused rule and compile-pipeline tests;
  • make check-ci;
  • make test;
  • selected real-project planning reproduction with configured rules;
  • independent read-only review;
  • public-repository hygiene scan.

@kvlonge
kvlonge enabled auto-merge (squash) September 20, 2026 09:15
@kvlonge
kvlonge merged commit 956e993 into main Sep 20, 2026
17 checks passed
@kvlonge
kvlonge deleted the fix/configured-rules-complete-analysis branch September 20, 2026 09:26
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.

1 participant