Skip to content

fix(ci): regenerate authorization_selector_collision snapshot after noir bump#22773

Closed
AztecBot wants to merge 2 commits intomerge-train/fairiesfrom
claudebox/fix-auth-selector-snapshot
Closed

fix(ci): regenerate authorization_selector_collision snapshot after noir bump#22773
AztecBot wants to merge 2 commits intomerge-train/fairiesfrom
claudebox/fix-auth-selector-snapshot

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

The contracts/authorization_selector_collision/ snapshot in
noir-projects/noir-contracts-comp-failures/ was authored against
the noir at commit ad02a20cd8 (pre-bump). Nightly-2026-04-23
(#22653, f39ac4f574) removed the check_parent_traits_are_implemented
function and folded its super-trait bound check into
check_trait_impl_where_clause_matches_trait_where_clause.

Both functions shared an early-return guard when trait_impl.trait_id
is None (the case this test hits, because the cascading resolution
failures above it never bind a trait ID). In the old compiler the
guard fired twice — once per function — emitting two missing trait ID
errors. In the new compiler it fires once.

Regenerated the snapshot to drop the now-absent
check_parent_traits_are_implemented: missing trait ID line.

Verified locally: ./bootstrap.sh test authorization_selector_collision
reports ✓ Compilation failed as expected with correct error(s).

See http://ci.aztec-labs.com/b6fb83475dc5b0b0 for the failing run on
PR #22765.

ClaudeBox log: https://claudebox.work/s/d72e1449fb744a16?run=79

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Apr 24, 2026
@vezenovm
Copy link
Copy Markdown
Contributor

Succeeded by #22771

@vezenovm vezenovm closed this Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants