fix: aadhaar verifier abi

[Nesopie]

Summary by CodeRabbit

• New Features

  • Added support for verifying Aadhaar-based attestations in the SDK’s self backend, expanding compatibility to Aadhaar proof formats while maintaining existing flows for other attestations.

• Chores

  • Bumped sdk/core version to 1.1.0-beta.2.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Adds Aadhaar-specific verifier support by branching on attestationId == 3 in SelfBackendVerifier to connect to AadhaarVerifier; introduces AadhaarVerifier ABI; bumps sdk/core package version from 1.1.0-beta.1 to 1.1.0-beta.2. No other behavioral or export changes.

Changes

Cohort / File(s)	Summary of changes
Version bump sdk/core/package.json	Version updated 1.1.0-beta.1 → 1.1.0-beta.2; no other fields changed.
Verifier logic update sdk/core/src/SelfBackendVerifier.ts	Imports AadhaarVerifier and factory; widens contract type to Verifier | AadhaarVerifier; selects AadhaarVerifier when attestationId == 3; otherwise uses existing Verifier path; retains error handling.
New ABI sdk/core/src/abi/AadhaarVerifier.json	Adds ABI with view method verifyProof(uint256[2], uint256[2][2], uint256[2], uint256[19]) returning bool.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Client
  participant SelfBackendVerifier
  participant VerifierFactory as Verifier__factory
  participant AadhaarFactory as AadhaarVerifier__factory
  participant Verifier
  participant AadhaarVerifier

  Client->>SelfBackendVerifier: verify(...)
  alt attestationId == 3
    SelfBackendVerifier->>AadhaarFactory: connect(address, provider)
    AadhaarFactory-->>SelfBackendVerifier: AadhaarVerifier instance
    SelfBackendVerifier->>AadhaarVerifier: verifyProof(a,b,c,pubSignals)
    AadhaarVerifier-->>SelfBackendVerifier: bool
  else other attestations
    SelfBackendVerifier->>VerifierFactory: connect(address, provider)
    VerifierFactory-->>SelfBackendVerifier: Verifier instance
    SelfBackendVerifier->>Verifier: verifyProof(a,b,c,pubSignals)
    Verifier-->>SelfBackendVerifier: bool
  end
  SelfBackendVerifier-->>Client: result

  note over SelfBackendVerifier,AadhaarVerifier: New branch for attestationId == 3

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Feat/aadhaar sdk #1082 — Also edits SelfBackendVerifier for Aadhaar-specific handling (timestamp/public-signal), complementing this verifier-connection change.
• fix: sdk #1085 — Introduces special-case behavior for attestationId == 3 in SelfBackendVerifier, overlapping with this branching logic.
• feat: add better error handling #691 — Modifies SelfBackendVerifier message and age fields; touches the same verification flow.

Suggested reviewers

• remicolin
• transphorm

Poem

A beta bumps, a branch appears,
Aadhaar’s gate now gently steers.
Proofs align in tidy rows,
Verifiers choose the path that flows.
One truth returned, simple and clear—
Version sings: a new frontier. 🚀

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/aadhaar-verifier

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 779fea5 and d28d8ff.

📒 Files selected for processing (3)

• sdk/core/package.json (1 hunks)
• sdk/core/src/SelfBackendVerifier.ts (2 hunks)
• sdk/core/src/abi/AadhaarVerifier.json (1 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}