Clean up stripe agent setup --json output #1762
Open
jleong-stripe wants to merge 1 commit into
Open
Conversation
gusnguyen-stripe
approved these changes
Jul 9, 2026
feab23a to
6422b7a
Compare
4531eba to
f24e7ac
Compare
6422b7a to
3db641e
Compare
f24e7ac to
4982f28
Compare
3db641e to
cc04bd1
Compare
4982f28 to
95104e8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Reviewers
r? @gusnguyen-stripe
cc @stripe/developer-products
Summary
Two improvements to the machine-readable output:
Remove top-level
statusfield. The aggregate status was derived by collapsing all per-client statuses into a single value, but it wasn't reliable — a single unresolved client would poison it regardless of whether anything actionable remained. Consumers can derive their own aggregate from theclientsarray if needed.Report Cursor as
status: unknowninstead ofmissing. Cursor's plugin can't be verified from the CLI, somissingwas misleading — it implies we checked and found nothing, when we actually can't check at all. Useunknowninstead until we build out a proper detection method.