Skip to content

fix(worldremit): validate payout inputs - #878

Open
aiirvizionz wants to merge 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/worldremit-payout-validation
Open

fix(worldremit): validate payout inputs#878
aiirvizionz wants to merge 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/worldremit-payout-validation

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • Validate WorldRemit payout recipient, amount, and currency before returning a transfer id.
  • Reject blank recipients, non-positive/non-finite amounts, and malformed currency codes early.
  • Add regression coverage so invalid payout requests cannot look like successful transfers.

Validation

  • git diff --check
  • node node_modules\vitest\vitest.mjs run packages/payments/worldremit/src/index.test.ts (5 tests passed)

This checkout's standard pnpm flow is currently blocked during dependency preparation by the committed lockfile policy issue around @profullstack/autoblog@0.4.0 lacking a tarball integrity field, so I used the already-present Vitest runtime directly and did not modify the lockfile.

Copilot AI review requested due to automatic review settings July 29, 2026 04:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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.

2 participants