Skip to content

fix: map final CTE contract outputs by name - #703

Merged
kvlonge merged 1 commit into
mainfrom
fix/map-final-cte-contract-outputs
Sep 17, 2026
Merged

kvlonge merged 1 commit into
mainfrom
fix/map-final-cte-contract-outputs

Conversation

@kvlonge

@kvlonge kvlonge commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Why

SQBRCONTRACT105 fell back to positional matching when a ceremonial terminal projection selected only part of a wider final CTE. That could check extra intermediary outputs against unrelated contract columns while skipping selected outputs.

Changes

  • Align final-CTE contract metadata through terminal source names.
  • Preserve placeholders for unselected and unnamed intermediary outputs.
  • Include right-hand-only BY NAME outputs.
  • Resolve explicit CTE column aliases before evaluating body projections.
  • Cover subset, alias, positional, wildcard, and BY NAME paths through native and CLI integration tests.

Verification

  • Focused native Rule tests
  • Focused CLI compile integration tests
  • Clippy with warnings denied
  • uv sync --all-extras
  • make check-ci
  • Independent review and finding-focused follow-up
  • Public repository hygiene scan

@kvlonge
kvlonge enabled auto-merge (squash) September 17, 2026 17:31
@kvlonge
kvlonge merged commit 83a3f53 into main Sep 17, 2026
18 checks passed
@kvlonge
kvlonge deleted the fix/map-final-cte-contract-outputs branch September 17, 2026 17:42
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