Skip to content

Readme claude skills docs#17

Open
dcramer wants to merge 6 commits intomainfrom
cursor/readme-claude-skills-docs-7b8e
Open

Readme claude skills docs#17
dcramer wants to merge 6 commits intomainfrom
cursor/readme-claude-skills-docs-7b8e

Conversation

@dcramer
Copy link
Member

@dcramer dcramer commented Mar 2, 2026

Update README with current Claude Code installation instructions and add documentation for installing skills via dotagents.


Slack Thread

Open in Web Open in Cursor 

…ntation

- Replace outdated '/install-plugin' with current 'claude plugins add' CLI command
- Add alternative '/plugin install' in-app command option
- Add new section for installing skills via dotagents
- Include dotagents quick start guide and command reference
- Update from source installation instructions

Co-Authored-By: Claude Sonnet 4.5 (Anthropic AI Assistant)

Co-authored-by: David Cramer <dcramer@users.noreply.github.com>
@cursor
Copy link

cursor bot commented Mar 2, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

Fix several hallucinated commands from the Cursor background agent:

- Claude Code CLI: 'claude plugins add' → 'claude plugin install' (singular,
  correct subcommand)
- Remove '@claude-plugins-official' marketplace suffix since sentry isn't
  published there yet — use bare 'sentry' which searches all marketplaces
- Remove duplicate in-app '/plugin install' variant — one install command
  is sufficient
- dotagents: '--name' flag → '--skill' (the actual flag)
- dotagents: bare 'add' without qualifier → 'add --all' (required when
  a repo contains multiple skills)

Co-Authored-By: Claude (Anthropic)
@HazAT
Copy link
Member

HazAT commented Mar 2, 2026

I've pushed fixes for several incorrect commands in the README (see commit 48976b4). The Cursor background agent hallucinated a few things:

  • claude plugins addclaude plugin install (singular, correct subcommand)
  • --name flag for dotagents → --skill (the actual flag)
  • Bare dotagents add getsentry/sentry-for-ai fails when a repo has multiple skills — needs --all
  • Removed the @claude-plugins-official marketplace suffix and the duplicate in-app /plugin install variant

One remaining issue: claude plugin install sentry is correct syntax, but sentry doesn't actually exist in any Claude Code marketplace yet (checked anthropics/claude-plugins-official — no sentry entry in plugins/ or external_plugins/). That command will fail until the plugin is published. Might want to hold off on documenting it, or note that it's coming soon.

@HazAT HazAT marked this pull request as ready for review March 2, 2026 09:58
@JoshFerge
Copy link
Member

I've pushed fixes for several incorrect commands in the README (see commit 48976b4). The Cursor background agent hallucinated a few things:

  • claude plugins addclaude plugin install (singular, correct subcommand)
  • --name flag for dotagents → --skill (the actual flag)
  • Bare dotagents add getsentry/sentry-for-ai fails when a repo has multiple skills — needs --all
  • Removed the @claude-plugins-official marketplace suffix and the duplicate in-app /plugin install variant

One remaining issue: claude plugin install sentry is correct syntax, but sentry doesn't actually exist in any Claude Code marketplace yet (checked anthropics/claude-plugins-official — no sentry entry in plugins/ or external_plugins/). That command will fail until the plugin is published. Might want to hold off on documenting it, or note that it's coming soon.

instead of sentry should it just be the url of this repo?

@dcramer
Copy link
Member Author

dcramer commented Mar 3, 2026

I think we should use the URL - Claude has some special case thing for us which I think might enable that to work but I'm also not sure as I didnt test it myself.

@dcramer
Copy link
Member Author

dcramer commented Mar 3, 2026

@cursor update this PR to enable both the marketplace add line and the plugin install line, use the url rather than just 'sentry'

additionally remove the bit about adding the plugin from local filesystem

@cursor
Copy link

cursor bot commented Mar 3, 2026

I've successfully updated the PR with the requested changes:

Changes Made

Claude Code installation section now includes:
• Both the marketplace add command: claude plugin marketplace add getsentry/sentry-for-ai
• The plugin install command using the GitHub URL: claude plugin install https://github.com/getsentry/sentry-for-ai

Removed:
• The "From Source" section with local filesystem installation instructions has been completely removed

Preserved:
• The dotagents section remains intact with the corrected --all and --skill flags (from HazAT's earlier fixes)
• Cursor installation instructions unchanged

The changes have been committed and pushed to the cursor/readme-claude-skills-docs-7b8e branch.

Open in Web Open in Cursor 

cursoragent and others added 2 commits March 3, 2026 18:20
…stem instructions

- Add marketplace with 'claude plugin marketplace add getsentry/sentry-for-ai'
- Install plugin with URL instead of marketplace name
- Remove 'From Source' section with local filesystem installation
- Keep both marketplace add and plugin install steps as requested

Co-Authored-By: Claude Sonnet 4.5 (Anthropic AI Assistant)

Co-authored-by: David Cramer <dcramer@users.noreply.github.com>
Co-authored-by: David Cramer <dcramer@users.noreply.github.com>
@dcramer
Copy link
Member Author

dcramer commented Mar 3, 2026

@cursoragent you did it entirely wrong heres an example from another repo:

claude plugin marketplace add getsentry/skills
claude plugin install sentry-skills@sentry-skills

@cursor
Copy link

cursor bot commented Mar 3, 2026

Fixed! I've corrected the plugin install command to use the proper format sentry@sentry-for-ai instead of the URL. This now matches the pattern from the getsentry/skills example:

• Marketplace add: claude plugin marketplace add getsentry/sentry-for-ai
• Plugin install: claude plugin install sentry@sentry-for-ai

The changes have been committed and pushed to the branch.

Open in Web Open in Cursor 

Use plugin-name@marketplace-name format (sentry@sentry-for-ai) instead of URL, following the pattern from getsentry/skills repository.

Co-Authored-By: Claude Sonnet 4.5 (Anthropic AI Assistant)

Co-authored-by: David Cramer <dcramer@users.noreply.github.com>
Use correct marketplace name 'sentry-plugin-marketplace' as defined in .claude-plugin/marketplace.json instead of 'sentry-for-ai'.

Co-Authored-By: Claude Sonnet 4.5 (Anthropic AI Assistant)

Co-authored-by: David Cramer <dcramer@users.noreply.github.com>
@dcramer
Copy link
Member Author

dcramer commented Mar 12, 2026

Suggestion: simplify the repository name from sentry-for-ai to ai. The plugin can still be called sentry, but a shorter marketplace/repo identifier improves customer experience in install flows like claude plugin install sentry@ai, dotagents sources like getsentry/ai, and related skill installer/onboarding paths.

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.

4 participants