Skip to content

fix(native): fix minikit 4.0 parsing and add more debug logs#248

Merged
Takaros999 merged 6 commits into
mainfrom
takis/always-show-idkit-result
May 7, 2026
Merged

fix(native): fix minikit 4.0 parsing and add more debug logs#248
Takaros999 merged 6 commits into
mainfrom
takis/always-show-idkit-result

Conversation

@Takaros999
Copy link
Copy Markdown
Contributor

@Takaros999 Takaros999 commented May 5, 2026

This PR:

  • Fixed parsing in the native transport for 4.0 proofs, we're now reusing a wasm utility to match the bridge behaviour
  • Fixed idempotency bug for handleVerify callback in react dev mode
  • Added more debug logs

@vercel
Copy link
Copy Markdown

vercel Bot commented May 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
idkit-js-example Ready Ready Preview, Comment May 7, 2026 10:53pm

Request Review

SeanROlszewski
SeanROlszewski previously approved these changes May 7, 2026
@Takaros999 Takaros999 changed the title chore(example): always display idkit result fix(native): fix minikit 4.0 parsing and add more debug logs May 7, 2026
@Takaros999
Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Takaros999 Takaros999 merged commit 453c340 into main May 7, 2026
18 checks passed
@Takaros999 Takaros999 deleted the takis/always-show-idkit-result branch May 7, 2026 23:25
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.

3 participants