Skip to content

Add shared PLUME signature test vectors - #145

Open
JustABiologist wants to merge 1 commit into
plume-sig:mainfrom
JustABiologist:shared-plume-vectors-24
Open

JustABiologist wants to merge 1 commit into
plume-sig:mainfrom
JustABiologist:shared-plume-vectors-24

Conversation

@JustABiologist

@JustABiologist JustABiologist commented May 12, 2026

Copy link
Copy Markdown

Closes #24

Summary

  • add a shared test-vectors/plume-signatures.json fixture for a deterministic low-entropy V1/V2 PLUME signing case
  • have RustCrypto, Arkworks, and Circom JS tests load the same vector instead of repeating constants
  • keep the existing JS signal expectations while deriving them from the shared fixture

Verification

  • corepack pnpm@8.15.9 run types in circuits/circom/test/javascript
  • corepack pnpm@8.15.9 test -- signals.test.ts in circuits/circom/test/javascript
  • corepack pnpm@8.15.9 exec prettier -c test/consts.ts test/signals.test.ts --config ../../../../.prettierrc --ignore-path ../../../../.prettierignore
  • git diff --check

Not run locally: Rust tests because this environment does not have cargo; full top-level Circom install because frozen install is blocked by the current lockfile metadata and CI already uses --no-frozen-lockfile.

@gitguardian

gitguardian Bot commented May 12, 2026

Copy link
Copy Markdown

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
While these secrets were previously flagged, we no longer have a reference to the
specific commits where they were detected. Once a secret has been leaked into a git
repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@JustABiologist
JustABiologist force-pushed the shared-plume-vectors-24 branch from f10411b to b8fb675 Compare May 12, 2026 17:53
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.

Differential test to verify all implementations

1 participant