Commit Graph

21 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 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
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
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 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 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 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 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 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 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 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 00739f740b Supposedly version is optional, lets test it 2026-01-05 16:07:21 -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