Skip to content

Release 6.8.0 - #395

Merged
rodrigojara-cko merged 1 commit into
masterfrom
release/6.8.0
Oct 22, 2026
Merged

rodrigojara-cko merged 1 commit into
masterfrom
release/6.8.0

Conversation

@rodrigojara-cko

@rodrigojara-cko rodrigojara-cko commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor
  • Payment setups: Pay by Bank is sent under the paybybank key; the previous key was ignored by the API. Adds the Pay by Bank risk object.
  • Payment setups: add the card, Apple Pay, Google Pay and instrument payment methods, with card token and details, wallet token data and token, and Google Pay preferred scheme (visa and mastercard). Add payment_methods auto_confirm, settings capture and excluded_payment_methods, order sub-merchant number_of_sales and metadata on the request.
  • Payment setups: add the PayPal payment method with user_action, brand_name, shipping_preference and its read-only action (order_id). Klarna, Tabby and stc pay get the fields the API defines on each method: Klarna action (client_token, session_id) and iban, stc pay action and Tabby payment_types.

Deprecated:

  • OrderSubMerchant $number_of_trades, which the API does not read. Use $number_of_sales.
  • PaymentMethods $pay_by_bank, which the API ignores. Use $paybybank.
  • Klarna, Stcpay and Tabby $payment_method_options, which the API no longer sends. Use the method's $action (Tabby has no replacement).

@agent-wall-e

agent-wall-e Bot commented Oct 22, 2026

Copy link
Copy Markdown

🟡 Risk Classification: MINOR

Approval route: AI Review + Human Approval
Rollback controls: Staged rollout + rollback

Classification reasons

  • no_low_class_matched
  • prod_source_modified

Operational gates

  • ✅ jira_ticket
  • ✅ independent_review

Files analysed: 2


wall-e 2026.06.19-02 · policy 6b4ce2b3b45a…

@agent-wall-e

agent-wall-e Bot commented Oct 22, 2026

Copy link
Copy Markdown
🔬 Debug — why this classification?

Each reason code emitted by the classifier, its source clause in the AI in SDLC Control Framework, and what it means.

Reason code Kind Clause Meaning
no_low_class_matched informational §2.2 (fall-through) None of the deterministic Low classes (§2.2.3, §2.2.4, §2.2.7, docs-only) applied; classifier fell through to LLM evaluation.
prod_source_modified informational §2.1 M7 (informational) At least one file is non-doc, non-test, non-IaC — i.e. application source code was modified.

Kinds:

  • classifying — this rule contributed to the chosen tier.
  • informational — context only; did not by itself decide the tier.

See issue #3 for the proposal to formalise this map as Appendix A of the standards doc.

wall-e 2026.06.19-02 · debug

@agent-wall-e

agent-wall-e Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

🔵 Advisory review: Sound, but needs your judgement

This PR needs a human approval. The code itself reads as correct; whether it should land depends on context I don't have.

This diff only bumps version strings from 6.7.0 to 6.8.0 in two places; the PR description claims significant API changes (new payment methods, new fields, deprecations) but none of that code appears in the diff.

For you to decide

  • The diff contains only version bumps in CheckoutUtils.php and version.json — none of the described feature changes (Pay by Bank key fix, card/Apple Pay/Google Pay/PayPal/Klarna/Tabby/stc pay additions, deprecations) are visible in this diff.
  • A human reviewer needs to confirm whether this is a truncated/partial diff and that the substantive changes are present in the actual PR, or whether the version was bumped without the accompanying code changes.
  • If this is the complete diff, the version bump is inconsistent with the claimed scope of changes and would be misleading to consumers of the SDK.

This is not an approval. wall-e cannot auto-approve this PR — it is an opinion to help whoever does. Advisory review · us.anthropic.claude-sonnet-4-6 · wall-e 2026.06.19-02

@rodrigojara-cko
rodrigojara-cko requested a review from a team October 22, 2026 11:27
@rodrigojara-cko rodrigojara-cko changed the title Release 6.8.0 - Payment setup alignment Release 6.8.0 Oct 22, 2026
@agent-wall-e

agent-wall-e Bot commented Oct 22, 2026

Copy link
Copy Markdown

🟡 Risk Classification: MINOR

Approval route: AI Review + Human Approval
Rollback controls: Staged rollout + rollback

Classification reasons

  • no_low_class_matched
  • prod_source_modified

Operational gates

  • ✅ jira_ticket
  • ✅ independent_review

Files analysed: 2


wall-e 2026.06.19-02 · policy 6b4ce2b3b45a…

@agent-wall-e

agent-wall-e Bot commented Oct 22, 2026

Copy link
Copy Markdown
🔬 Debug — why this classification?

Each reason code emitted by the classifier, its source clause in the AI in SDLC Control Framework, and what it means.

Reason code Kind Clause Meaning
no_low_class_matched informational §2.2 (fall-through) None of the deterministic Low classes (§2.2.3, §2.2.4, §2.2.7, docs-only) applied; classifier fell through to LLM evaluation.
prod_source_modified informational §2.1 M7 (informational) At least one file is non-doc, non-test, non-IaC — i.e. application source code was modified.

Kinds:

  • classifying — this rule contributed to the chosen tier.
  • informational — context only; did not by itself decide the tier.

See issue #3 for the proposal to formalise this map as Appendix A of the standards doc.

wall-e 2026.06.19-02 · debug

- Payment setups: Pay by Bank is sent under the paybybank key; the previous key was ignored by the API. Adds the Pay by Bank risk object.
- Payment setups: add the card, Apple Pay, Google Pay and instrument payment methods, with card token and details, wallet token data and token, and Google Pay preferred scheme (visa and mastercard). Add payment_methods auto_confirm, settings capture and excluded_payment_methods, order sub-merchant number_of_sales and metadata on the request.
- Payment setups: add the PayPal payment method with user_action, brand_name, shipping_preference and its read-only action (order_id). Klarna, Tabby and stc pay get the fields the API defines on each method: Klarna action (client_token, session_id) and iban, stc pay action and Tabby payment_types.

Deprecated:
- OrderSubMerchant $number_of_trades, which the API does not read. Use $number_of_sales.
- PaymentMethods $pay_by_bank, which the API ignores. Use $paybybank.
- Klarna, Stcpay and Tabby $payment_method_options, which the API no longer sends. Use the method's $action (Tabby has no replacement).
@agent-wall-e

agent-wall-e Bot commented Oct 22, 2026

Copy link
Copy Markdown

🟡 Risk Classification: MINOR

Approval route: AI Review + Human Approval
Rollback controls: Staged rollout + rollback

Classification reasons

  • no_low_class_matched
  • prod_source_modified

Operational gates

  • ✅ jira_ticket
  • ✅ independent_review

Files analysed: 2


wall-e 2026.06.19-02 · policy 6b4ce2b3b45a…

@agent-wall-e

agent-wall-e Bot commented Oct 22, 2026

Copy link
Copy Markdown
🔬 Debug — why this classification?

Each reason code emitted by the classifier, its source clause in the AI in SDLC Control Framework, and what it means.

Reason code Kind Clause Meaning
no_low_class_matched informational §2.2 (fall-through) None of the deterministic Low classes (§2.2.3, §2.2.4, §2.2.7, docs-only) applied; classifier fell through to LLM evaluation.
prod_source_modified informational §2.1 M7 (informational) At least one file is non-doc, non-test, non-IaC — i.e. application source code was modified.

Kinds:

  • classifying — this rule contributed to the chosen tier.
  • informational — context only; did not by itself decide the tier.

See issue #3 for the proposal to formalise this map as Appendix A of the standards doc.

wall-e 2026.06.19-02 · debug

@sonarqubecloud

Copy link
Copy Markdown

@rodrigojara-cko
rodrigojara-cko merged commit c6b5b68 into master Oct 22, 2026
6 checks passed
@rodrigojara-cko
rodrigojara-cko deleted the release/6.8.0 branch October 22, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants