Skip to content

[stripe agent setup] Harden skills checks for plugin-only paths#1758

Open
gusnguyen-stripe wants to merge 4 commits into
masterfrom
gusnguyen/agent-setup-skills-behavior
Open

[stripe agent setup] Harden skills checks for plugin-only paths#1758
gusnguyen-stripe wants to merge 4 commits into
masterfrom
gusnguyen/agent-setup-skills-behavior

Conversation

@gusnguyen-stripe

@gusnguyen-stripe gusnguyen-stripe commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Reviewers

r? @jleong-stripe
cc @stripe/developer-products

Summary

Refines when and how stripe agent setup fetches skills status so plugin-only non-interactive paths stay fast, errors fail open, and plugin status takes priority in --status output.

What it does

Plugin-only paths — keeps non-interactive plugin setup independent from skills checks:

  • stripe agent setup --client <id> does not fetch skills status
  • Calling-agent plugin setup does not fetch skills status (optimize for latency)
  • --status, --json, and interactive mode still fetch skills status because they display it

Fail open — skills check errors do not block status or TUI flows; errors are surfaced in output instead of aborting setup.

Status ordering — when agents are detected, --status shows plugin status before skills status.

Stack

PR 3 of 4. Depends on #1757.


Stack created with GitHub Stacks CLIGive Feedback 💬

@gusnguyen-stripe gusnguyen-stripe changed the title gusnguyen/agent setup skills behavior [stripe agent setup] Harden skills checks for plugin-only paths Jul 8, 2026
@gusnguyen-stripe gusnguyen-stripe marked this pull request as ready for review July 8, 2026 22:39
@gusnguyen-stripe gusnguyen-stripe requested a review from a team as a code owner July 8, 2026 22:39
@gusnguyen-stripe gusnguyen-stripe force-pushed the gusnguyen/agent-setup-skills-behavior branch from 6ec5a71 to c057506 Compare July 8, 2026 22:39
@gusnguyen-stripe gusnguyen-stripe force-pushed the gusnguyen/agent-setup-skills-status branch 2 times, most recently from ac5e3aa to 6844551 Compare July 8, 2026 23:02
@gusnguyen-stripe gusnguyen-stripe force-pushed the gusnguyen/agent-setup-skills-behavior branch from c057506 to 26f6e0f Compare July 8, 2026 23:02
@gusnguyen-stripe gusnguyen-stripe force-pushed the gusnguyen/agent-setup-skills-behavior branch from 26f6e0f to 32a33f0 Compare July 9, 2026 17:37
@gusnguyen-stripe gusnguyen-stripe force-pushed the gusnguyen/agent-setup-skills-behavior branch from 32a33f0 to 19f7fc6 Compare July 9, 2026 18:16
@gusnguyen-stripe gusnguyen-stripe force-pushed the gusnguyen/agent-setup-skills-status branch 2 times, most recently from 07f0f0d to 4e2215b Compare July 9, 2026 18:21
@gusnguyen-stripe gusnguyen-stripe force-pushed the gusnguyen/agent-setup-skills-behavior branch from 19f7fc6 to 27296d9 Compare July 9, 2026 18:21
Base automatically changed from gusnguyen/agent-setup-skills-status to master July 9, 2026 19:11
gusnguyen-stripe and others added 4 commits July 9, 2026 12:13
Co-authored-by: Cursor <cursoragent@cursor.com>
Committed-By-Agent: cursor
When the remote skills index is unreachable, surface per-scope StatusError in --status and --json instead of aborting the command, while keeping install fail-closed.

Co-authored-by: Cursor <cursoragent@cursor.com>
Committed-By-Agent: cursor
Show skills only when they already exist on disk, skip the remote check when agents are detected with no skill dirs, and omit install_skills from JSON in that case.

Co-authored-by: Cursor <cursoragent@cursor.com>
Committed-By-Agent: cursor
Partial no longer exists; a scope is installed when current or out of date.

Co-authored-by: Cursor <cursoragent@cursor.com>
Committed-By-Agent: cursor
@gusnguyen-stripe gusnguyen-stripe force-pushed the gusnguyen/agent-setup-skills-behavior branch from 27296d9 to 72e5d78 Compare July 9, 2026 19:14
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.

3 participants