Skip to content

fix: update schnorr lib to v0.2.0 and remove remaining is_infinite field usages#22414

Merged
TomAFrench merged 1 commit intobump-noirfrom
claudebox/0ef46c61b9801d30-2
Apr 8, 2026
Merged

fix: update schnorr lib to v0.2.0 and remove remaining is_infinite field usages#22414
TomAFrench merged 1 commit intobump-noirfrom
claudebox/0ef46c61b9801d30-2

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

@AztecBot AztecBot commented Apr 8, 2026

Summary

Fixes remaining noir-contracts compilation failures on PR #22393 (Noir nightly 2026-04-08).

The Noir nightly removed the is_infinite field from EmbeddedCurvePoint, making it a 2-field struct { x, y } with is_infinite() as a computed method.

Changes:

  • Bump schnorr library from v0.1.3 to v0.2.0 in both schnorr account contracts. v0.1.3 constructs EmbeddedCurvePoint { x, y, is_infinite } which no longer compiles. v0.2.0 (released today) uses the new 2-field struct.
  • Remove is_infinite: false from Point struct literal in schnorr_hardcoded_account_contract
  • Remove is_infinite: false from Point struct literals in avm_test_contract

ClaudeBox log: https://claudebox.work/s/0ef46c61b9801d30?run=2

- Bump schnorr library from v0.1.3 to v0.2.0 (v0.1.3 uses the removed
  EmbeddedCurvePoint.is_infinite field)
- Remove is_infinite field from Point struct literal in
  schnorr_hardcoded_account_contract
- Remove is_infinite field from Point struct literals in
  avm_test_contract
@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Apr 8, 2026
@TomAFrench TomAFrench changed the base branch from next to bump-noir April 8, 2026 14:04
@TomAFrench TomAFrench marked this pull request as ready for review April 8, 2026 14:04
@TomAFrench TomAFrench merged commit 90830f6 into bump-noir Apr 8, 2026
30 of 40 checks passed
@TomAFrench TomAFrench deleted the claudebox/0ef46c61b9801d30-2 branch April 8, 2026 14:04
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