Skip to content

feature/INT-1737 - Removed Beta tags from FlowClient, not beta anymore - #223

Merged
david-ruiz-cko merged 1 commit into
masterfrom
feature/INT-1737
Oct 20, 2026
Merged

david-ruiz-cko merged 1 commit into
masterfrom
feature/INT-1737

Conversation

@david-ruiz-cko

Copy link
Copy Markdown
Contributor

This pull request makes minor documentation updates to the FlowClient in the lib/checkout_sdk/payments/flow/flow_client.rb file. The main change is the removal of [Beta] tags from the class and method comments, indicating that the related Payment Flow API operations are no longer considered beta features.

Most important changes:

Documentation updates:

  • Removed [Beta] tag from the FlowClient class comment.
  • Removed [Beta] tag from the request_payment_session method comment.
  • Removed [Beta] tag from the submit_payment_session method comment.
  • Removed [Beta] tag from the request_payment_session_with_payment method comment.

@david-ruiz-cko
david-ruiz-cko requested a review from a team October 20, 2026 12:06
@agent-wall-e

agent-wall-e Bot commented Oct 20, 2026

Copy link
Copy Markdown

🟢 Risk Classification: LOW

Approval route: AI Auto-Approval
Rollback controls: Automated Instant Rollback + feature flags

Classification reasons

  • no_low_class_matched
  • prod_source_modified
  • 2.2.6_logical_extension:The diff removes only comment annotations ([Beta] tags) from existing code without changing any logic, signatures, behavior, or configuration.

Operational gates

  • ✅ jira_ticket (INT-1737)
  • ✅ independent_review

Files analysed: 1


wall-e 2026.06.19-02 · policy 6b4ce2b3b45a…

@agent-wall-e

agent-wall-e Bot commented Oct 20, 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.
2.2.6_logical_extension — The diff removes only comment annotations ([Beta] tags) from existing code without changing any logic, signatures, behavior, or configuration. classifying §2.2.6 Sonnet 4.6 evaluator promoted minor → low: the change reuses existing code paths and does not cross a trust boundary.

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

@agent-wall-e agent-wall-e Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Auto-approved — this PR meets all Low-risk criteria.

All checks passed, no unresolved comments, and the change classification is:

  • no_low_class_matched
  • prod_source_modified
  • 2.2.6_logical_extension:The diff contains only removal of comment tags ([Beta]) from documentation strings with no functional, logic, or configuration changes.

wall-e 2026.06.19-02 · policy 6b4ce2b3b45a…

@david-ruiz-cko
david-ruiz-cko merged commit faf54ea into master Oct 20, 2026
5 checks passed
@david-ruiz-cko
david-ruiz-cko deleted the feature/INT-1737 branch October 20, 2026 12:43
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