fix(connector): [COINBASE] stop declaring manual capture and refunds webhook flow

[errmakov]

Type of Change

• Bugfix
• New feature
• Enhancement
• Refactoring
• Dependency updates
• Documentation
• CI/CD

Description

The Coinbase connector specification declared capabilities that the connector does not implement, so GET /feature_matrix (and the connector docs generated from it) advertised them:

Declaration	Before	After	Why
supported_capture_methods	Automatic, Manual, SequentialAutomatic	Automatic, SequentialAutomatic	ConnectorIntegration<Capture> returns FlowNotSupported, so a manual capture can never be completed
COINBASE_SUPPORTED_WEBHOOK_FLOWS	Payments, Refunds	Payments	Refunds are declared NotSupported, Execute returns FlowNotSupported, and get_webhook_event_type maps no refund event

Because the default ConnectorValidation::validate_connector_against_payment_request checks the requested capture method against supported_capture_methods, a Coinbase payment with capture_method: manual is now rejected up front with NotSupported, instead of being accepted and failing at capture time.

Why SequentialAutomatic is kept (the issue suggests removing it): should_initiate_capture_flow only chains a capture call when the attempt is Authorized, and only for Paybox. Coinbase's status mapping never produces Authorized (New → AuthenticationPending, Completed/Resolved → Charged), so for Coinbase SequentialAutomatic behaves exactly like Automatic and works today. Removing it would start rejecting requests that currently succeed. This matches cryptopay, which declares the same pair.

Additional Changes

• This PR modifies the API contract
• This PR modifies the database schema
• This PR modifies application configuration/environment variables

Motivation and Context

Fixes #14277.

How did you test it?

Added unit tests in crates/hyperswitch_connectors/src/connectors/coinbase.rs that call validate_connector_against_payment_request for each capture method and check get_supported_webhook_flows.

cargo test -p hyperswitch_connectors --features v1,frm,payouts,revenue_recovery,dummy_connector --lib connectors::coinbase::tests

Tests against the old declarations (tests only):

test connectors::coinbase::tests::accepts_automatic_and_sequential_automatic_capture ... ok
test connectors::coinbase::tests::declares_only_payments_webhook_flow ... FAILED
test connectors::coinbase::tests::rejects_manual_capture_because_capture_flow_is_not_supported ... FAILED
  left: Some([Payments, Refunds])
 right: Some([Payments])
assertion failed: !validate_capture_method(CaptureMethod::Manual)
test result: FAILED. 1 passed; 2 failed; 0 ignored; 0 measured; 10 filtered out

With the fix:

test connectors::coinbase::tests::accepts_automatic_and_sequential_automatic_capture ... ok
test connectors::coinbase::tests::declares_only_payments_webhook_flow ... ok
test connectors::coinbase::tests::rejects_manual_capture_because_capture_flow_is_not_supported ... ok
test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 10 filtered out

Not tested against a live Coinbase account: the change only affects declarations and request validation, and no Coinbase API call changes.

Checklist

• I formatted the code cargo +nightly fmt --all
• I addressed lints thrown by cargo clippy
• I reviewed the submitted code
• I added unit tests for my changes where possible

🤖 Generated with Claude Code

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
crates/hyperswitch_connectors/src/connectors/coinbase.rs	2% smaller