feat(agent-skills): Add --skills command to install Sentry agent skills by codyde · Pull Request #1187 · getsentry/sentry-wizard

codyde · 2026-01-13T08:18:20Z

Summary

This PR adds a new wizard feature that installs Sentry agent skills for various AI coding assistants.

Supported Editors

Claude Code (.claude/skills/)
OpenAI Codex (.codex/skills/)
OpenCode (.opencode/skill/)
Cursor (.cursor/skills/)
GitHub Copilot (.github/skills/)
Factory Droid (.factory/skills/)

Features

Interactive mode with scope and editor selection prompts
Headless mode: --skills claude-code opencode
Project-scoped (default) or user-scoped installation: --scope user
Smart update logic: updates Sentry skills, preserves user-created skills
Downloads skills from getsentry/sentry-agent-skills GitHub releases

Usage

# Interactive mode
npx @sentry/wizard --skills

# Headless mode - specify editors
npx @sentry/wizard --skills claude-code cursor

# Install to user profile instead of project
npx @sentry/wizard --skills claude-code --scope user

Changes

bin.ts - Added --skills and --scope CLI arguments
src/agent-skills/agent-skills-wizard.ts - Main wizard implementation
src/agent-skills/editor-configs.ts - Editor configuration and path helpers
src/run.ts - Route to agent skills wizard when --skills flag is present

Testing

Manual testing with each supported editor
Unit tests (TODO)
E2E tests (TODO)

This adds a new wizard feature that installs Sentry agent skills for various AI coding assistants: - Claude Code (.claude/skills/) - OpenAI Codex (.codex/skills/) - OpenCode (.opencode/skill/) - Cursor (.cursor/skills/) - GitHub Copilot (.github/skills/) - Factory Droid (.factory/skills/) Features: - Interactive mode with scope and editor selection prompts - Headless mode: --skills claude-code opencode - Project-scoped (default) or user-scoped installation: --scope user - Smart update logic: updates Sentry skills, preserves user-created skills - Downloads skills from getsentry/sentry-agent-skills GitHub releases Usage: npx @sentry/wizard --skills # Interactive npx @sentry/wizard --skills claude-code cursor # Headless npx @sentry/wizard --skills claude-code --scope user # User profile

github-actions · 2026-01-13T08:18:32Z

Semver Impact of This PR

⚪ None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

Features

(agent-skills) Add --skills command to install Sentry agent skills by codyde in #1187

_{🤖 This preview updates automatically when you update the PR.}

Lms24

Nice! I think the new flow is a good starting point. We could even potentially call it from SDK setup flows (if we want to) but this makes sense standalone as well (as discussed on Slack)

A general question: IIUC for now, this downloads the latest skill version at the time. Is there some way users can easily update skills without having to periodically rerun the wizard? This is more a general "skill ecosystem" question. My only concern is these skills becoming outdated and users not being aware that there are improvements available.

Lms24 · 2026-01-16T16:26:55Z

src/agent-skills/agent-skills-wizard.ts

+      chalk.red(
+        'Failed to download Sentry agent skills from GitHub.\n' +
+          'Please check your internet connection and try again.\n\n' +
+          'You can also install skills manually from:\n' +
+          chalk.cyan('https://github.com/getsentry/sentry-agent-skills'),
+      ),


nit: We usually don't color entire error messages. clack.log.error already marks the output visually as errored. Feel free to disregard though, no strong feelings :) IF you change it, let's also use template strings over concatinating line-by-line. Makes it a bit more readable IMHO.

Lms24 · 2026-01-16T16:40:48Z

src/agent-skills/agent-skills-wizard.ts

+  const tempDir = path.join(os.tmpdir(), `sentry-skills-${Date.now()}`);
+  await fs.promises.mkdir(tempDir, { recursive: true });


l: let's use

Suggested change

const tempDir = path.join(os.tmpdir(), `sentry-skills-${Date.now()}`);

await fs.promises.mkdir(tempDir, { recursive: true });

const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), 'sentry-skills-'));

instead for a truly random temp dir (fs.mkdtemp will automatically append a random sequence to the last - specified in the name).

Lms24 · 2026-01-16T16:41:15Z

src/agent-skills/agent-skills-wizard.ts

+  } catch {
+    // Network error on HEAD, try archive
+    tarballUrl = SKILLS_REPO_ARCHIVE_URL;
+    isArchive = true;
+  }


q: if we land here, something went wrong, correct? Should we emit a log/warning?

Lms24 · 2026-01-16T16:43:24Z

src/agent-skills/agent-skills-wizard.ts

+  const skillsExist = await fs.promises
+    .access(extractPath)
+    .then(() => true)
+    .catch(() => false);


l: can we use fs.existsSync(extractPath) here instead?

Suggested change

const skillsExist = await fs.promises

.access(extractPath)

.then(() => true)

.catch(() => false);

const skillsExist = fs.existsSync(extractPath);

No strong feelings but I think fs.access is nice for high throughput envs but in case of the wizard the sync operation is totally fine

Lms24 · 2026-01-16T16:45:36Z

src/agent-skills/agent-skills-wizard.ts

+    const targetExists = await fs.promises
+      .access(targetPath)
+      .then(() => true)
+      .catch(() => false);


same q about existsSync here

Lms24 · 2026-01-16T17:22:58Z

One more thing: Let's make sure to add an e2e test for this flow, to have it covered. Thanks!

cursor

Cursor Bugbot has reviewed your changes and found 2 potential issues.

^{Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.}

cursor · 2026-02-03T04:58:45Z

src/agent-skills/agent-skills-wizard.ts

+    await fs.promises.rm(tempDir, { recursive: true, force: true });
+  } catch {
+    // Ignore cleanup errors
+  }


Temp directory not fully cleaned up after archive download

Medium Severity

The downloadSkills() function returns extractPath, which is often a subdirectory of the actual temp directory (e.g., <tempDir>/sentry-agent-skills-main/skills when isArchive is true). The cleanup code at line 177 removes only this subdirectory, leaving the parent temp directory orphaned. Each wizard run accumulates leftover temp directories that are never cleaned up.

Additional Locations (1)

src/agent-skills/agent-skills-wizard.ts#L270-L294

cursor · 2026-02-03T04:58:45Z

src/agent-skills/agent-skills-wizard.ts

+    async () => {
+      if (options.scope) {
+        return options.scope;
+      }


Scope selection prompt unreachable in interactive mode

Medium Severity

The interactive scope selection prompt is dead code that can never be reached. The yargs definition sets default: 'project' for the scope option, so finalArgs.scope is always 'project' even when the user doesn't specify it. The wizard then checks if (options.scope) which is always truthy, causing an immediate return before the prompt. Users in interactive mode cannot select between project and user scope.

Additional Locations (2)

bin.ts#L175-L182

src/run.ts#L123-L124

codyde requested a review from Lms24 January 13, 2026 08:19

Lms24 reviewed Jan 16, 2026

View reviewed changes

Lms24 mentioned this pull request Jan 16, 2026

Use sub commands instead of options #1200

Closed

codyde added 5 commits January 22, 2026 09:45

ref(agent-skills): remove chalk coloring from error messages

5e0b7af

ref(agent-skills): use fs.mkdtemp for temp directory

7a12755

ref(agent-skills): add debug logging for archive URL fallback

da7e8f5

ref(agent-skills): use fs.existsSync instead of async access

8b244ea

test(e2e): add agent skills wizard tests

4b0b346

codyde marked this pull request as ready for review February 3, 2026 04:50

cursor bot reviewed Feb 3, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(agent-skills): Add --skills command to install Sentry agent skills#1187

feat(agent-skills): Add --skills command to install Sentry agent skills#1187
codyde wants to merge 6 commits intomasterfrom
feat/agent-skills-wizard

codyde commented Jan 13, 2026

Uh oh!

github-actions bot commented Jan 13, 2026

Uh oh!

Lms24 left a comment

Uh oh!

Lms24 Jan 16, 2026

Uh oh!

Lms24 Jan 16, 2026

Uh oh!

Lms24 Jan 16, 2026

Uh oh!

Lms24 Jan 16, 2026

Uh oh!

Lms24 Jan 16, 2026

Uh oh!

Lms24 commented Jan 16, 2026 •

edited

Loading

Uh oh!

cursor bot left a comment

Uh oh!

cursor bot Feb 3, 2026

Uh oh!

cursor bot Feb 3, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		const tempDir = path.join(os.tmpdir(), `sentry-skills-${Date.now()}`);
		await fs.promises.mkdir(tempDir, { recursive: true });

	const tempDir = path.join(os.tmpdir(), `sentry-skills-${Date.now()}`);
	await fs.promises.mkdir(tempDir, { recursive: true });
	const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), 'sentry-skills-'));

Uh oh!

Conversation

codyde commented Jan 13, 2026

Summary

Supported Editors

Features

Usage

Changes

Testing

Uh oh!

github-actions bot commented Jan 13, 2026

Semver Impact of This PR

📋 Changelog Preview

Features

Uh oh!

Lms24 left a comment

Choose a reason for hiding this comment

Uh oh!

Lms24 Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Lms24 Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Lms24 Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Lms24 Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Lms24 Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Lms24 commented Jan 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor bot Feb 3, 2026

Choose a reason for hiding this comment

Temp directory not fully cleaned up after archive download

Uh oh!

cursor bot Feb 3, 2026

Choose a reason for hiding this comment

Scope selection prompt unreachable in interactive mode

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Lms24 commented Jan 16, 2026 •

edited

Loading