Skip to content

test: add example parity and cross-SDK challenge vectors#6

Merged
brendanjryan merged 2 commits intomainfrom
feat/example-parity-and-vectors
Apr 15, 2026
Merged

test: add example parity and cross-SDK challenge vectors#6
brendanjryan merged 2 commits intomainfrom
feat/example-parity-and-vectors

Conversation

@brendanjryan
Copy link
Copy Markdown
Contributor

@brendanjryan brendanjryan commented Apr 15, 2026

Summary

  • add separate-process client/server examples for basic, charge-basic, charge-hash, and charge-fee-payer to mirror the mpp-rs sample layout while keeping the existing one-command demos
  • make the basic server generate and fund a fresh merchant address on startup, closer to the Rust sample behavior
  • import cross-SDK challenge ID golden vectors from the Rust/Python SDK suites and add targeted regression coverage for untested helpers
  • fix opaque={} handling so empty opaque metadata is preserved in HMAC and wire-format roundtrips like the other SDKs

Testing

  • GOTOOLCHAIN=auto go test ./...

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: 44963ae199

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

Comment thread pkg/mpp/hmac.go
@brendanjryan brendanjryan force-pushed the feat/example-parity-and-vectors branch from 45a3506 to 902cdcf Compare April 15, 2026 23:00
@brendanjryan brendanjryan merged commit 22bb816 into main Apr 15, 2026
5 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.

1 participant