Skip to content

fix: updated the skills plugin details#111

Merged
BYK merged 1 commit into
mainfrom
fix/skills
Jan 29, 2026
Merged

fix: updated the skills plugin details#111
BYK merged 1 commit into
mainfrom
fix/skills

Conversation

@MathurAditya724

Copy link
Copy Markdown
Member

No description provided.

@MathurAditya724 MathurAditya724 requested a review from BYK January 29, 2026 19:59
@github-actions

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

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


Bug Fixes 🐛

  • Updated the skills plugin details by MathurAditya724 in #111

🤖 This preview updates automatically when you update the PR.

@MathurAditya724 MathurAditya724 enabled auto-merge (squash) January 29, 2026 20:00
@BYK BYK disabled auto-merge January 29, 2026 20:01
@BYK BYK merged commit e152944 into main Jan 29, 2026
8 checks passed
@BYK BYK deleted the fix/skills branch January 29, 2026 20:01
{
"name": "sentry/cli",
"path": "plugins/sentry-cli"
"name": "sentry-cli",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: The plugin name was changed to sentry-cli, but the installation instructions in plugins/README.md still reference the old name, sentry/cli, which will cause installation to fail.
Severity: HIGH

Suggested Fix

Update the two instances of claude plugin install sentry/cli in plugins/README.md to claude plugin install sentry-cli to match the new plugin name in the marketplace manifest.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: .claude-plugin/marketplace.json#L8

Potential issue: The plugin's name was changed in `.claude-plugin/marketplace.json` from
`sentry/cli` to `sentry-cli`. However, the installation instructions in
`plugins/README.md` were not updated to reflect this change. As a result, users
following the documentation will run `claude plugin install sentry/cli`, which will fail
because the plugin is now registered as `sentry-cli`. This will prevent users from
installing the plugin.

Did we get this right? 👍 / 👎 to inform future reviews.

sentry-junior Bot added a commit that referenced this pull request May 21, 2026
Code blocks:
- Bumped @sentry/starlight-theme from ^0.2.0 to ^0.3.0 — v0.2.0 had
  --ve-code-bg: #111 which made code blocks darker than intended.
  v0.3.0 uses #141419, matching the theme playground reference.

Homepage header:
- Removed the 'Sentry CLI' text from the homepage header — now shows
  just the wordmark SVG logo like production does. The text next to
  the logo icon looked redundant.
betegon pushed a commit that referenced this pull request May 22, 2026
Code blocks:
- Bumped @sentry/starlight-theme from ^0.2.0 to ^0.3.0 — v0.2.0 had
  --ve-code-bg: #111 which made code blocks darker than intended.
  v0.3.0 uses #141419, matching the theme playground reference.

Homepage header:
- Removed the 'Sentry CLI' text from the homepage header — now shows
  just the wordmark SVG logo like production does. The text next to
  the logo icon looked redundant.
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.

2 participants