Skip to content

feat: populate verifyContext from core.verify.resolve attestation#2

Merged
exodus-jee merged 1 commit into
exodus-forkfrom
jee/feat/enable-verify-api
May 12, 2026
Merged

feat: populate verifyContext from core.verify.resolve attestation#2
exodus-jee merged 1 commit into
exodus-forkfrom
jee/feat/enable-verify-api

Conversation

@exodus-jee

@exodus-jee exodus-jee commented May 12, 2026

Copy link
Copy Markdown

Summary

Mirror the getVerifyContext activation from ExodusForks/walletconnect-monorepo in auth-client/src/controllers/engine.ts, so auth_request flows populate verifyContext.verified.{origin, validation, isScam} from core.verify.resolve instead of always returning validation: "UNKNOWN".

Dependency

Requires ExodusForks/walletconnect-monorepo#24 (which re-enables core.verify.resolve) to be published first; otherwise resolve still throws.

Test plan

  • No existing test asserts on verified.validation content
  • Existing assertions of validation === "UNKNOWN" still hold (test attestations 404 -> caught -> UNKNOWN)

@exodus-jee exodus-jee self-assigned this May 12, 2026
@exodus-jee
exodus-jee force-pushed the jee/feat/enable-verify-api branch from 942d4d0 to 51574f6 Compare May 12, 2026 10:07

@guten-exodus guten-exodus left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@exodus-jee
exodus-jee marked this pull request as draft May 12, 2026 14:57
@exodus-jee
exodus-jee force-pushed the jee/feat/enable-verify-api branch 2 times, most recently from 81bd358 to dddbc5e Compare May 12, 2026 21:31
@exodus-jee
exodus-jee force-pushed the jee/feat/enable-verify-api branch from dddbc5e to 5206428 Compare May 12, 2026 21:46
@exodus-jee
exodus-jee marked this pull request as ready for review May 12, 2026 22:21
@exodus-jee
exodus-jee merged commit c4cc96e into exodus-fork May 12, 2026
3 of 4 checks passed
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.

2 participants