Commit Graph

44 Commits

Author SHA1 Message Date
Daniel Szoke 343f13111d ref(create-pr): Extract PR template retrieval to script
Move the conditional PR template logic from SKILL.md instructions into
a deterministic bash script. This makes the template retrieval reliable
and testable rather than depending on the LLM to interpret empty/null
output from the gh command.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-13 13:51:01 +01:00
Daniel Szoke 08057261de feat(create-pr): Add PR template support
Fetch the repository's PR template using `gh repo view --json
pullRequestTemplates` before writing the PR description. This ensures
PRs follow the repo's expected format and fill in required sections.

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-12 17:23:50 +01:00
David Cramer 0189f2ed17 Merge pull request #17 from getsentry/docs/contributing-guidelines
docs: Add contributor guidelines
2026-01-10 11:06:54 -08:00
David Cramer fc5a75b467 Merge pull request #16 from getsentry/feat/agents-md-skill
feat: Add agents-md skill for maintaining AGENTS.md files
2026-01-10 11:03:41 -08:00
David Cramer 99623274f6 docs: Add contributor guidelines
Establish contribution philosophy emphasizing incremental change and
self-review as the default workflow. Include instructions for testing
skills locally and a checklist for adding new skills.

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-10 11:02:46 -08:00
David Cramer 65aa458f2c feat: Add agents-md skill for maintaining AGENTS.md files
New skill that guides agents to create and maintain concise AGENTS.md
documentation with CLAUDE.md as a symlink. Enforces minimal style with
headers, bullets, and skill references instead of verbose prose.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-10 10:45:27 -08:00
David Cramer ff7dcae620 Merge pull request #14 from getsentry/armcknight/fix/local-clone-installation
fix(README): local clone plugin installation
2026-01-10 10:32:16 -08:00
David Cramer 5e453ffb20 Merge pull request #15 from getsentry/fix/deslop-base-branch
fix: Use dynamic base branch detection across skills
2026-01-10 10:32:12 -08:00
Josh Ferge f2fa7afe84 fix(create-pr, find-bugs): Use dynamic base branch detection
Apply the same gh repo view pattern for consistency across all skills
that diff against the default branch.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-09 18:03:29 -05:00
Josh Ferge be09e6dc1b refactor: Use gh repo view for simpler default branch detection
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-09 17:59:28 -05:00
Josh Ferge 818ff8731e fix(deslop): Support repos using master instead of main
Dynamically detect the default branch using git symbolic-ref instead of
hardcoding "main". Falls back to "main" if detection fails.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-09 17:54:02 -05:00
Andrew McKnight 4e84ef32bd fix(README): local clone plugin installation 2026-01-08 14:48:20 -09:00
David Cramer caaee2ad87 Merge pull request #13 from getsentry/docs/commit-skill-ai-attribution
docs(commit): Add Co-Authored-By guidance for AI-generated changes
2026-01-07 17:01:29 -08:00
David Cramer e311628a98 docs(commit): Add Co-Authored-By guidance for AI-generated changes
Document that commits with changes primarily generated by a coding agent
should include the Co-Authored-By attribution in the footer. Clarify that
this is the only indicator of AI involvement that should appear in commits.

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-07 16:50:55 -08:00
David Cramer 91a181bb35 Merge pull request #12 from getsentry/fix/sentry-mcp-config
fix(claude-settings-audit): Use correct Sentry MCP HTTP configuration
2026-01-06 19:02:50 -08:00
David Cramer d76d61e30e fix(claude-settings-audit): Use correct Sentry MCP HTTP configuration
The skill was recommending a command-based MCP server using uvx, but
Sentry MCP is a hosted service using streaming HTTP. Update to use
the correct URL-based configuration with type "http".

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-06 18:57:22 -08:00
David Cramer 3501a31fdd Merge pull request #9 from getsentry/feat/improve-skill-workflows
feat(skills): Add prerequisite checks to commit and create-pr skills
2026-01-06 11:27:55 -08:00
David Cramer 6ef251d038 feat(skills): Add prerequisite checks to commit and create-pr skills
Add workflow prerequisites to ensure proper git hygiene:
- commit skill: Check if on main/master and prompt to create feature branch
- create-pr skill: Check for uncommitted changes and invoke commit skill first
- AGENTS.md: Add step to include new skills in .claude/settings.json allowlist

These changes help enforce the expected workflow where changes are made on
feature branches and properly committed before creating PRs.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-06 10:59:23 -08:00
David Cramer 92829e28c2 Merge pull request #8 from getsentry/update-repo
docs: Update GitHub URLs for repo rename
2026-01-06 10:58:59 -08:00
David Cramer 4fbd6556ed docs: Update GitHub URLs for repo rename to getsentry/skills
Update README installation instructions to reference the new repository
name after renaming from getsentry/sentry-skills to getsentry/skills.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-06 10:54:07 -08:00
David Cramer 6a77037386 Merge pull request #7 from getsentry/add-skills
chore: Add sentry-skills to project settings allowlist
2026-01-06 10:51:09 -08:00
David Cramer 0912e0a2cd chore: Add sentry-skills to project settings allowlist
Pre-approve all sentry-skills for use in this repository to streamline
the development workflow when working on the plugin itself.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-06 10:49:28 -08:00
David Cramer af5d60af6b Merge pull request #6 from getsentry/fix-branch-names
fix(iterate-pr): Use explicit branch name in git push command
2026-01-06 10:30:08 -08:00
David Cramer 5555fb4b69 fix(iterate-pr): Use explicit branch name in git push command
Bare `git push` may push all tracked branches depending on git config.
Specify origin and current branch explicitly to ensure only the
working branch is pushed.
2026-01-06 10:26:06 -08:00
David Cramer 9c8168f5a7 Merge pull request #5 from getsentry/improve-settings-audit-skill
fix(claude-settings-audit): Improve recommendation accuracy
2026-01-06 10:21:57 -08:00
David Cramer 79adde273c fix(claude-settings-audit): Add skills allowlist for Sentry projects
- Add Skill() permissions for all sentry-skills to the recommendation
- Update AGENTS.md to remind contributors to update the skills list
  when adding new skills

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-06 10:19:18 -08:00
David Cramer f892d4142d fix(claude-settings-audit): Improve recommendation accuracy
- Never suggest commands with user-specific absolute paths
- Never suggest custom scripts that may have side effects
- Only include the package manager actually used by the project
  (e.g., if pnpm is detected, don't suggest npm/yarn commands)
- Reorganize stack-specific commands by detection criteria
- Add explicit "Package Manager Rules" table for clarity

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-06 10:11:07 -08:00
David Cramer 2f63686e4b Merge pull request #4 from getsentry/claude-settings-skill
feat: Add claude-settings-audit skill
2026-01-06 09:58:34 -08:00
David Cramer ea591e9b22 feat: Add claude-settings-audit skill
Analyzes repositories to generate recommended Claude Code settings.json
permissions based on detected tech stack:
- Detects languages (Python, Node, Go, Rust, Ruby, Java)
- Detects build tools (Docker, Make, Terraform)
- Suggests baseline commands (git, gh, file exploration)
- Suggests framework-specific WebFetch domains
- Suggests MCP servers (Sentry, Linear) when applicable

Also updates this repo's settings.json with recommended allow list.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-06 09:52:57 -08:00
David Cramer 8bba5745db Rely on bash commands 2026-01-05 19:26:24 -08:00
David Cramer 1b0a0e00fa Merge pull request #3 from getsentry/add-create-pr-skill
feat: Add create-pr skill for Sentry PR conventions
2026-01-05 16:53:13 -08:00
David Cramer c32f22bd94 feat: Add create-pr skill for Sentry PR conventions
New skill that guides PR creation following Sentry's engineering practices:
- PR description structure (what, why, alternatives, context)
- Title format following commit conventions
- Issue reference syntax (Fixes/Refs)
- Guidelines on PR scope and reviewer assignment

Notably excludes "Test plan" sections per Sentry conventions.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-05 16:52:04 -08:00
David Cramer cba22231ed Merge pull request #2 from getsentry/add-iterate-pr-skill
Add iterate-pr skill for automated PR iteration
2026-01-05 16:48:37 -08:00
David Cramer c17f830b14 Add iterate-pr skill for automated PR iteration
New skill that automates the CI feedback loop:
- Checks CI status first, waits for bot checks (sentry/codecov/cursor/bugbot/seer)
- Gathers review feedback from humans and bots
- Investigates failures via gh run view --log-failed
- Validates feedback before acting on it
- Commits fixes and waits for CI to complete
- Repeats until all checks pass

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-05 16:42:17 -08:00
David Cramer 762c8a3a42 Update README and AGENTS.md documentation
- Clarify repo is primarily for Claude Code but follows open Agent Skills format
- Add instruction to update README when creating new skills
- Remove duplicated skills list from AGENTS.md
- Add missing deslop skill to README

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-05 16:16:08 -08:00
David Cramer 00739f740b Supposedly version is optional, lets test it 2026-01-05 16:07:21 -08:00
David Cramer 6358fc4701 Fix information on updates 2026-01-05 16:01:20 -08:00
Josh Ferge 3f6c5851d9 Merge pull request #1 from getsentry/joshferge/desloppify
feat: Add deslop skill for removing AI-generated code slop
2026-01-05 19:00:32 -05:00
David Cramer b8cab86eac add find-bugs and update docs 2026-01-05 15:55:29 -08:00
Josh Ferge e5fc9c5178 feat: Add deslop skill for removing AI-generated code slop
Adds a skill to identify and remove common AI code patterns like
unnecessary comments, defensive checks, any casts, and inline imports.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-05 18:54:35 -05:00
David Cramer 118ea92ac4 oops 2026-01-05 14:49:03 -08:00
David Cramer 622bafc03b correct source path 2026-01-05 14:46:49 -08:00
David Cramer 5ae6cbb122 reorganize 2026-01-05 14:44:41 -08:00
David Cramer da8f1cd882 first commit 2026-01-05 14:43:16 -08:00