Skip to content

feat: validate complete raw SLH-DSA private keys off thread - #1093

Draft
sidhujag wants to merge 1 commit into
margelo:mainfrom
sidhujag:feat/slh-dsa-raw-key-validation
Draft

sidhujag wants to merge 1 commit into
margelo:mainfrom
sidhujag:feat/slh-dsa-raw-key-validation

Conversation

@sidhujag

@sidhujag sidhujag commented Nov 8, 2026 •

Copy link
Copy Markdown

Adds SlhDsa.validateRawPrivateKey() to validate a complete raw SLH-DSA private key off the JavaScript thread and return its public key. It uses the existing OpenSSL provider path and owned worker input, retaining variant and in-flight admission checks. The public SlhDsa export makes the existing native implementation available to consumers.

The example runner registers fixed NIST key-generation vectors for all twelve variants, corrupted roots, malformed input, copied input after caller erasure, worker-drain behavior and admission reuse. Documentation describes the API and its scope.

Validation already completed: TypeScript/package preparation, lint/formatting, C++20 syntax checks, unchanged generated Nitro bindings, source-vector comparison and independent source reviews. Native example execution has not run. This draft is for source review and must remain gated on the repository's exact-source iOS/Android qualification and maintainer attestation. No status or attestation has been fabricated.

Pali's draft native integration is syscoin/pali-wallet#1410. It temporarily adopts exactly the five production files at this commit and binds their hashes into its qualification identity. Its CI installation repair does not establish validator correctness or device readiness.

@vercel

vercel Bot commented Nov 8, 2026

Copy link
Copy Markdown

@sidhujag is attempting to deploy a commit to the Margelo Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
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.

1 participant