fix: suppress alerts for expected billing errors in increment-usage task #27535
+29
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Suppresses trigger.dev alerts for expected billing errors in the
increment-usagetask. These errors are expected in certain scenarios and should not trigger alerts:No such subscription: 'sub_temp_scale_but_should_be_per_user'where the subscription ID contains "temp" or "sandbox"Stripe subscription (sub_xxx) is not usage based- this task only handles usage-based subscriptionsWhen these expected errors occur, the task now logs at
infolevel and returns successfully instead of throwing and triggering alerts.Mandatory Tasks (DO NOT REMOVE)
How should this be tested?
increment-usagetask with a subscription ID containing "temp" or "sandbox"Human Review Checklist
"no such subscription","is not usage based") match actual Stripe/service error messagesisExpectedBillingErrorfunction