Skip to content

Add option to suppress Stripe notices - #1909

Merged
zacchua-stripe merged 4 commits into
masterfrom
zacchua/suppress-stripe-notices
Sep 17, 2026
Merged

zacchua-stripe merged 4 commits into
masterfrom
zacchua/suppress-stripe-notices

Conversation

@zacchua-stripe

@zacchua-stripe zacchua-stripe commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Why?

Stripe notices can be returned for human requests in test and sandbox environments. SDKs currently emit every notice without providing a way to suppress them, which can create repeated warning output.

What?

  • Support STRIPE_SUPPRESS_NOTICES=true for suppressing notices in non-agent environments.
  • Keep notices enabled by default.
  • Continue emitting notices for AI agents.
  • Tell human users about the environment variable when emitting a notice.
  • Add tests covering enabled, disabled, invalid, and AI-agent cases.

See Also

http://go/j/RUN_DEVSDK-2882

Configuration

  • skip-changefile: This PR is not a user-facing change, so there's no changefile.

zacchua-stripe and others added 2 commits September 15, 2026 13:45
Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>
Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>
@zacchua-stripe
zacchua-stripe marked this pull request as ready for review September 15, 2026 20:51
@zacchua-stripe
zacchua-stripe requested a review from a team as a code owner September 15, 2026 20:51
@zacchua-stripe
zacchua-stripe requested review from kidus-stripe and removed request for a team September 15, 2026 20:51
Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>
@zacchua-stripe
zacchua-stripe enabled auto-merge (squash) September 17, 2026 00:13
@zacchua-stripe
zacchua-stripe merged commit 439568d into master Sep 17, 2026
19 checks passed
@zacchua-stripe
zacchua-stripe deleted the zacchua/suppress-stripe-notices branch September 17, 2026 00:15
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.

2 participants