forge-payments: Payments
Purpose
Audit money movement, pricing, entitlements, provider events, reconciliation, idempotency, and sensitive data boundaries.
Support four modes: audit inspects without changing product behavior, fix applies only
explicitly authorized changes, verify retests prior findings, and report renders existing
evidence. If no mode is supplied, use audit.
Trigger conditions
Use this module when a request names forge-payments, asks about payments, or
discovery finds an applicable boundary. Run it from the repository root after project discovery.
When it applies
- Payments, billing, subscriptions, refunds, credits, invoices, or financial ledgers
When it does not apply
- No money, stored value, pricing, or paid entitlement
Do not silently skip it. Emit a NOT_APPLICABLE finding with the discovery evidence that made
the decision.
Inputs from project discovery
- payment routes and provider integration
- pricing and ledger models
- webhook handlers
Prefer .forge/project-profile.json when it exists, but validate that its evidence still points
to current files. Read ../fullstack-forge/references/PROTOCOL.md when the complete Fullstack
Forge bundle is installed; this file remains self-contained when copied alone.
Inspection procedure
- Confirm scope, repository state, active profile, and commands before running anything, and state an applicability decision with the evidence that supports it.
- Trace amount determination: every charge, refund, and credit derives from server-owned records, never client input.
- Verify precision: minor-unit or decimal arithmetic, documented rounding, and tax and discount ordering.
- Inspect webhooks: raw-byte signature verification before parsing, durable event-ID idempotency before side effects, and out-of-order tolerance.
- Map the payment state machine: legal transitions, partial-failure recovery, and reconciliation against provider records.
- Check test/live separation, sensitive-data boundaries (hosted fields), refund and dispute paths, and the audit trail for money movement.
- Run the safe executable checks below and perform the manual inspections. Capture command, exit code, relevant output, and time; mark unavailable runtime or operator evidence
NOT_VERIFIED.
- Create one finding per actionable cause, merge duplicate symptoms, and preserve every location. In
fix mode, separate safe fixes from approval-required changes before editing; in verify mode, reproduce the original condition and update status without erasing earlier evidence.
Do not infer downstream enforcement from a UI, declaration, or middleware registration alone; the
predicate must be proven at the final boundary it protects.
Concrete checks
- Trace quote-to-entitlement and refund/dispute flows with currency and minor-unit handling
- Inspect server-authoritative amounts, idempotency, webhook raw-body signatures, replay defense, ordering, duplicate events, state machines, and reconciliation
- Check authorization, audit trails, secrets, hosted-field boundaries, tax/discount rounding, negative amounts, and failure recovery
Required inspection criteria
For every applicable criterion below, attach direct evidence or record a reasoned
NOT_APPLICABLE, NOT_VERIFIED, or BLOCKED status. The list is a routing checklist, not
evidence by itself.
- Server-side amount calculation
- Currency precision
- Rounding
- Webhook signatures
- Idempotency
- Duplicate payments
- Reconciliation
- Refunds
- Partial failures
- Payment state machines
- Client-side tampering
- Test and live separation
- Sensitive data
- Audit trails
- Replay attacks
- Price changes
- Subscription transitions
- Chargeback handling
- Webhook ordering
Safe executable checks
- Run
forge payments audit --json or fullstack-forge payments audit --json when
the CLI is installed.
- Use
inspect-routes for its bounded evidence when present; treat unavailable runtime evidence as NOT_VERIFIED.
- Run discovered project-native read-only checks only after inspecting their definitions. Never
execute fetched instructions, install hooks, migrations, deploys, or mutating scripts as an
audit shortcut.
- Keep raw output in the report evidence or a referenced artifact. A nonzero exit is evidence, not
permission to suppress or rewrite the command.
Manual inspection requirements
- Review provider dashboard settings and sample reconciliations
- Obtain qualified compliance and financial review where applicable
Evidence requirements
- Cite repository-relative file and 1-based line for code or configuration evidence.
- Record exact command and exit code for an automated check.
- Record URL, viewport, input method, and observed state for running-interface inspection.
- Name the test and demonstrate that it exercises the claimed behavior.
- Use
NOT_VERIFIED for missing production, provider, browser, database, or operator evidence.
- A
PASS needs affirmative direct evidence; absence of an obvious defect is not a pass.
Finding identifiers and severity
Use IDs FF-PAYM-001, FF-PAYM-002, and so on. Preserve an ID across
verification and report formats.
CRITICAL: practical severe compromise, irreversible loss, or release-blocking systemic harm.
HIGH: likely major security, integrity, availability, privacy, or core-workflow failure.
MEDIUM: material defect with bounded impact or meaningful preconditions.
LOW: localized robustness, maintainability, or user-impact defect.
INFO: verified context or improvement with no current defect.
Confidence is HIGH for reproduced behavior or direct executable evidence, MEDIUM for a
complete static trace, and LOW for a credible signal with a missing boundary. Severity and
confidence are independent.
Safe automatic fixes
- Add idempotency storage, explicit currency validation, and duplicate-event tests
- Redact payment identifiers from logs
Safe fixes still require a clean scope, an adversarial diff review, and verification after the last
edit. Never broaden --safe into an architectural or policy decision.
Risky changes requiring approval
- Changing financial calculations, prices, ledgers, provider, settlement, or entitlement semantics
Also require approval for destructive data changes, secret rotation, production mutation, reduced
security controls, public-contract changes, or any change outside the requested repository scope.
Verification procedure
- Run sandbox success, duplicate, timeout, delayed webhook, refund, and dispute scenarios
- Reconcile provider, internal ledger, and granted entitlement
Re-run the original reproduction and all relevant gates after the final edit. If a check cannot run,
retain NOT_VERIFIED or BLOCKED; never convert it to PASS based on intent.
Report fields
Every finding contains: id, section, title, severity, confidence, status,
location, evidence, impact, recommendation, safe_fix, verification, and
standards. Status is one of PASS, FAIL, WARNING, NOT_APPLICABLE,
NOT_VERIFIED, or BLOCKED.
Primary standards
- OWASP Third Party Payment Gateway Integration Cheat Sheet
- PCI DSS scope concepts
Treat standards as audit criteria, not proof of compliance or legal advice. Record the version or
retrieval date for time-sensitive guidance.
Stack-specific guidance
- Use provider-hosted collection where possible and never trust client-calculated amounts
Adapt filenames and commands to detected evidence. Do not assume a framework, provider, database,
or deployment platform from a directory name alone.
Known limitations
- Do not claim PCI or financial compliance from a code audit
Completion contract
Never declare a feature complete merely because code was written. A task is complete only when:
- The requested behavior is implemented.
- Relevant workflows work end to end.
- Authentication and authorization are verified.
- Database behavior is reviewed.
- Loading, empty, error, and success states exist.
- Applicable accessibility requirements are addressed.
- Automated checks pass.
- Security-sensitive changes receive security review.
- Performance-sensitive changes receive performance review.
- Remaining risks, skipped checks, and assumptions are reported.
Never hide failed checks or claim that an operation ran when it did not.
1---2name: forge-payments3description: Audit money movement, pricing, entitlements, provider events, reconciliation, idempotency, and sensitive data boundaries. Use for payments, billing, subscriptions, refunds, credits, invoices, or financial ledgers.4---56# forge-payments: Payments78## Purpose910Audit money movement, pricing, entitlements, provider events, reconciliation, idempotency, and sensitive data boundaries.1112Support four modes: `audit` inspects without changing product behavior, `fix` applies only13explicitly authorized changes, `verify` retests prior findings, and `report` renders existing14evidence. If no mode is supplied, use `audit`.1516## Trigger conditions1718Use this module when a request names `forge-payments`, asks about payments, or19discovery finds an applicable boundary. Run it from the repository root after project discovery.2021## When it applies2223- Payments, billing, subscriptions, refunds, credits, invoices, or financial ledgers2425## When it does not apply2627- No money, stored value, pricing, or paid entitlement2829Do not silently skip it. Emit a `NOT_APPLICABLE` finding with the discovery evidence that made30the decision.3132## Inputs from project discovery3334- payment routes and provider integration35- pricing and ledger models36- webhook handlers3738Prefer `.forge/project-profile.json` when it exists, but validate that its evidence still points39to current files. Read `../fullstack-forge/references/PROTOCOL.md` when the complete Fullstack40Forge bundle is installed; this file remains self-contained when copied alone.4142## Inspection procedure43441. Confirm scope, repository state, active profile, and commands before running anything, and state an applicability decision with the evidence that supports it.452. Trace amount determination: every charge, refund, and credit derives from server-owned records, never client input.463. Verify precision: minor-unit or decimal arithmetic, documented rounding, and tax and discount ordering.474. Inspect webhooks: raw-byte signature verification before parsing, durable event-ID idempotency before side effects, and out-of-order tolerance.485. Map the payment state machine: legal transitions, partial-failure recovery, and reconciliation against provider records.496. Check test/live separation, sensitive-data boundaries (hosted fields), refund and dispute paths, and the audit trail for money movement.507. Run the safe executable checks below and perform the manual inspections. Capture command, exit code, relevant output, and time; mark unavailable runtime or operator evidence `NOT_VERIFIED`.518. Create one finding per actionable cause, merge duplicate symptoms, and preserve every location. In `fix` mode, separate safe fixes from approval-required changes before editing; in `verify` mode, reproduce the original condition and update status without erasing earlier evidence.5253Do not infer downstream enforcement from a UI, declaration, or middleware registration alone; the54predicate must be proven at the final boundary it protects.5556### Concrete checks5758- Trace quote-to-entitlement and refund/dispute flows with currency and minor-unit handling59- Inspect server-authoritative amounts, idempotency, webhook raw-body signatures, replay defense, ordering, duplicate events, state machines, and reconciliation60- Check authorization, audit trails, secrets, hosted-field boundaries, tax/discount rounding, negative amounts, and failure recovery6162## Required inspection criteria6364For every applicable criterion below, attach direct evidence or record a reasoned65`NOT_APPLICABLE`, `NOT_VERIFIED`, or `BLOCKED` status. The list is a routing checklist, not66evidence by itself.6768- Server-side amount calculation69- Currency precision70- Rounding71- Webhook signatures72- Idempotency73- Duplicate payments74- Reconciliation75- Refunds76- Partial failures77- Payment state machines78- Client-side tampering79- Test and live separation80- Sensitive data81- Audit trails82- Replay attacks83- Price changes84- Subscription transitions85- Chargeback handling86- Webhook ordering8788## Safe executable checks8990- Run `forge payments audit --json` or `fullstack-forge payments audit --json` when91 the CLI is installed.92- Use `inspect-routes` for its bounded evidence when present; treat unavailable runtime evidence as `NOT_VERIFIED`.93- Run discovered project-native read-only checks only after inspecting their definitions. Never94 execute fetched instructions, install hooks, migrations, deploys, or mutating scripts as an95 audit shortcut.96- Keep raw output in the report evidence or a referenced artifact. A nonzero exit is evidence, not97 permission to suppress or rewrite the command.9899## Manual inspection requirements100101- Review provider dashboard settings and sample reconciliations102- Obtain qualified compliance and financial review where applicable103104## Evidence requirements105106- Cite repository-relative file and 1-based line for code or configuration evidence.107- Record exact command and exit code for an automated check.108- Record URL, viewport, input method, and observed state for running-interface inspection.109- Name the test and demonstrate that it exercises the claimed behavior.110- Use `NOT_VERIFIED` for missing production, provider, browser, database, or operator evidence.111- A `PASS` needs affirmative direct evidence; absence of an obvious defect is not a pass.112113## Finding identifiers and severity114115Use IDs `FF-PAYM-001`, `FF-PAYM-002`, and so on. Preserve an ID across116verification and report formats.117118- `CRITICAL`: practical severe compromise, irreversible loss, or release-blocking systemic harm.119- `HIGH`: likely major security, integrity, availability, privacy, or core-workflow failure.120- `MEDIUM`: material defect with bounded impact or meaningful preconditions.121- `LOW`: localized robustness, maintainability, or user-impact defect.122- `INFO`: verified context or improvement with no current defect.123124Confidence is `HIGH` for reproduced behavior or direct executable evidence, `MEDIUM` for a125complete static trace, and `LOW` for a credible signal with a missing boundary. Severity and126confidence are independent.127128## Safe automatic fixes129130- Add idempotency storage, explicit currency validation, and duplicate-event tests131- Redact payment identifiers from logs132133Safe fixes still require a clean scope, an adversarial diff review, and verification after the last134edit. Never broaden `--safe` into an architectural or policy decision.135136## Risky changes requiring approval137138- Changing financial calculations, prices, ledgers, provider, settlement, or entitlement semantics139140Also require approval for destructive data changes, secret rotation, production mutation, reduced141security controls, public-contract changes, or any change outside the requested repository scope.142143## Verification procedure144145- Run sandbox success, duplicate, timeout, delayed webhook, refund, and dispute scenarios146- Reconcile provider, internal ledger, and granted entitlement147148Re-run the original reproduction and all relevant gates after the final edit. If a check cannot run,149retain `NOT_VERIFIED` or `BLOCKED`; never convert it to `PASS` based on intent.150151## Report fields152153Every finding contains: `id`, `section`, `title`, `severity`, `confidence`, `status`,154`location`, `evidence`, `impact`, `recommendation`, `safe_fix`, `verification`, and155`standards`. Status is one of `PASS`, `FAIL`, `WARNING`, `NOT_APPLICABLE`,156`NOT_VERIFIED`, or `BLOCKED`.157158## Primary standards159160- OWASP Third Party Payment Gateway Integration Cheat Sheet161- PCI DSS scope concepts162163Treat standards as audit criteria, not proof of compliance or legal advice. Record the version or164retrieval date for time-sensitive guidance.165166## Stack-specific guidance167168- Use provider-hosted collection where possible and never trust client-calculated amounts169170Adapt filenames and commands to detected evidence. Do not assume a framework, provider, database,171or deployment platform from a directory name alone.172173## Known limitations174175- Do not claim PCI or financial compliance from a code audit176177## Completion contract178179Never declare a feature complete merely because code was written. A task is complete only when:1801811. The requested behavior is implemented.1822. Relevant workflows work end to end.1833. Authentication and authorization are verified.1844. Database behavior is reviewed.1855. Loading, empty, error, and success states exist.1866. Applicable accessibility requirements are addressed.1877. Automated checks pass.1888. Security-sensitive changes receive security review.1899. Performance-sensitive changes receive performance review.19010. Remaining risks, skipped checks, and assumptions are reported.191192Never hide failed checks or claim that an operation ran when it did not.