Skip to content

fix: default manual payments to success in MCP#19

Merged
chirag-zenskar merged 3 commits into
mainfrom
fix/manual-payment-success-status
Jun 7, 2026
Merged

fix: default manual payments to success in MCP#19
chirag-zenskar merged 3 commits into
mainfrom
fix/manual-payment-success-status

Conversation

@sudip-zenskar
Copy link
Copy Markdown
Contributor

Summary

  • default MCP createPayment status metadata to success
  • restrict createPayment status choices to success so manual/offline recorded payments are realized
  • update README payment wording to match the MCP tool contract

Validation

  • node -e JSON.parse(...) for src/mcp-config.json: passed
  • pnpm run test: passed, 23/23
  • git diff --check: passed
  • pnpm run build: blocked by existing tsdown dependency issue: Failed to import module "unrun"

Copy link
Copy Markdown
Contributor

@chirag-zenskar chirag-zenskar left a comment

Choose a reason for hiding this comment

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

Approved. The tool metadata now scopes createPayment to the frontend Add Payment-style successful manual/tax-withheld flow, and timestamp control has been removed so the backend assigns the recorded time.

@chirag-zenskar chirag-zenskar merged commit 4a61504 into main Jun 7, 2026
@chirag-zenskar chirag-zenskar deleted the fix/manual-payment-success-status branch June 7, 2026 13:12
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.

3 participants