Skip to content

feat: IDKit docs improvements — signatures, nullifiers, SDK refs#92

Open
Takaros999 wants to merge 5 commits intomainfrom
takis/idkit-improvements
Open

feat: IDKit docs improvements — signatures, nullifiers, SDK refs#92
Takaros999 wants to merge 5 commits intomainfrom
takis/idkit-improvements

Conversation

@Takaros999
Copy link
Copy Markdown
Contributor

Summary

  • New RP Signatures page (/world-id/idkit/signatures) — pseudocode-first algorithm spec with JS/Go SDK examples and cross-language test vectors (verified against both JS and Go test suites)
  • Nullifier guidance in integration guide — Step 6 explains what nullifiers are, why to store them, and recommends NUMERIC(78, 0) in PostgreSQL
  • Fixed signRequest API — updated to v4 options object syntax across integration guide and JS reference; Go example now includes WithAction
  • Added return_to to SDK code examples (JS, Swift, Kotlin)
  • Expanded Go SDK reference — documents NewSigner, WithAction/WithTTL options pattern, RpSignature struct

Test plan

  • All test vectors verified against JS (@noble/secp256k1) and Go (dcrd/secp256k1) implementations
  • Spellcheck passes (cspell)
  • Visual review on Mintlify preview

🤖 Generated with Claude Code

…pdates

- New `signatures.mdx` page with pseudocode algorithm, JS/Go SDK examples, and cross-language test vectors
- Add Step 6 to integration guide: nullifier explanation and PostgreSQL storage guidance
- Fix `signRequest` API calls to use v4 options object across integrate and JS reference
- Add `return_to` parameter to SDK code examples (JS, Swift, Kotlin)
- Expand Go SDK reference with `NewSigner`, `WithAction`/`WithTTL` options
- Add signatures page to docs.json navigation

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mintlify
Copy link
Copy Markdown
Contributor

mintlify bot commented Apr 2, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
world 🟢 Ready View Preview Apr 2, 2026, 9:50 AM

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e723b8c350

ℹ️ 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".

…onstraint

- Fix pseudocode to conditionally allocate 49 or 81 bytes based on action
- Remove trailing comma in PostgreSQL CREATE TABLE example
- Add UNIQUE(nullifier, action) constraint to enforce replay protection at DB level

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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