Skip to content

feat(stablecoin): expose position account - #334

Open
3esmit wants to merge 1 commit into
logos-blockchain:mainfrom
3esmit:feat/stablecoin-module-position-account
Open

feat(stablecoin): expose position account#334
3esmit wants to merge 1 commit into
logos-blockchain:mainfrom
3esmit:feat/stablecoin-module-position-account

Conversation

@3esmit

@3esmit 3esmit commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add pure stablecoin_ffi operations for deterministic position/vault PDA resolution and strict Position decoding
  • expose positionAccount with exact owner/nonce lookup, one direct account read, and stable not_found responses carrying derived IDs
  • preserve full-range integer values as decimal strings and validate program ownership plus all stored position identity fields
  • add fixed PDA vectors, malformed/invariant rejection coverage, C boundary tests, and universal module tests

Validation

  • cargo +nightly fmt --all -- --check
  • taplo fmt --check .
  • make clippy
  • make clippy-guest
  • RISC0_DEV_MODE=1 cargo +1.94.0 test --workspace --exclude integration_tests
  • RISC0_DEV_MODE=1 cargo +1.94.0 test -p integration_tests
  • IDL drift check
  • make build-programs
  • nix build path:.#stablecoin_ffi path:.#stablecoin-module path:.#stablecoin-module-tests

Closes #325

Copilot AI lite review requested due to automatic review settings August 27, 2026 05:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

Expose stablecoin position resolution and decoding

2 participants