[stripe agent setup] Polish skills status display#1759
Open
gusnguyen-stripe wants to merge 1 commit into
Open
Conversation
gusnguyen-stripe
changed the title
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-behavior
branch
from
6ec5a71 to
c057506
Compare
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-ui
branch
2 times, most recently
from
a5b9907 to
fc10186
Compare
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-behavior
branch
from
c057506 to
26f6e0f
Compare
jleong-stripe
approved these changes
Jul 9, 2026
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-ui
branch
from
fc10186 to
a444804
Compare
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-behavior
branch
from
26f6e0f to
32a33f0
Compare
vcheung-stripe
approved these changes
Jul 9, 2026
| return color.Green("✔").String(), "installed" | ||
| case agentskills.StatusOutOfDate: | ||
| return color.Yellow("⚠").String(), fmt.Sprintf("%d of %d skills out of date", d.OutOfDateCount, total) | ||
| return color.Yellow("⚠").String(), "outdated" |
Collaborator
There was a problem hiding this comment.
Similar to the last one: "new version available" feels a little nicer to me
Contributor
Author
There was a problem hiding this comment.
I'm going to keep "outdated" because it's a stronger word for agents than "new version available"
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-ui
branch
from
a444804 to
feab23a
Compare
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-behavior
branch
from
32a33f0 to
19f7fc6
Compare
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-ui
branch
from
feab23a to
6422b7a
Compare
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-behavior
branch
from
19f7fc6 to
27296d9
Compare
Simplify the --status skill lines to installed/outdated, tidy the setup TUI skills row, hide the not-installed scope when the other has skills, and share that visibility rule between the text and JSON output paths. Co-authored-by: Cursor <cursoragent@cursor.com> Committed-By-Agent: cursor
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-behavior
branch
from
27296d9 to
72e5d78
Compare
gusnguyen-stripe
force-pushed
the
gusnguyen/agent-setup-skills-ui
branch
from
6422b7a to
3db641e
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
3 participants
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? @jleong-stripe
cc @stripe/developer-products
Summary
Polishes skills status presentation in
stripe agent setupafter the core integration lands: simpler status lines, scope-aware output, and smarter install suggestions.What it does
Simpler status lines — the
--statusskill rows show a plaininstalled/outdatedstate (dropping the per-count text), and the setup TUI skills row is tidied up.Scope-aware display — when one scope is installed and the other is not, show only the installed scope instead of redundant "not installed" lines. This rule is shared between the
--statustext and--jsonoutput so they stay consistent.Install suggestions — do not suggest a skills install command when the skill is already installed in one of the scopes.
Stack
PR 4 of 4. Depends on #1758.
Demo
With no supported agents
Screenshare.-.2026-07-08.3_34_37.PM.mp4
With a supported agent
2nd-demo-with-supported-agents.mp4
Stack created with GitHub Stacks CLI • Give Feedback 💬