chore: add commit skill for Claude, Cursor, and Codex#16804
Conversation
Co-Authored-By: Claude <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Addresses PR #16804 review comment about missing create-branch skill dependency. Also cleans up extra agent directories from initial install. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Wouldn't it be better to use dotagents here? Tbh I haven't tried it out yet, but based on that docs page, this use case seems like exactly what dotagents is designed to handle -- the ability to reuse a skill defined elsewhere without needing to copy paste it. |
55fbd44 to
b5a097e
Compare
83c43e1 to
d78326f
Compare
agents.toml
Outdated
| version = 1 | ||
| # Skills are checked into git so collaborators don't need to run 'dotagents install'. | ||
| gitignore = false |
There was a problem hiding this comment.
Bug: The agents.toml configuration lists cursor and codex as agents, but the required symlinks (.cursor/skills, .codex/skills) are not checked into the repository, unlike .claude/skills.
Severity: MEDIUM
Suggested Fix
To ensure the configuration works as intended for all declared agents without requiring an install step, check in the missing symlinks for .cursor/skills and .codex/skills. Alternatively, if support for these agents is not ready, remove cursor and codex from the agents list in agents.toml.
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: agents.toml#L1-L3
Potential issue: The `agents.toml` file sets `gitignore = false` and lists `cursor` and
`codex` in the `agents` field. The intent, as stated in a comment, is that collaborators
do not need to run `dotagents install`. However, while the skills and the
`.claude/skills` symlink are checked in, the corresponding symlinks for `.cursor/skills`
and `.codex/skills` are missing from the repository. Without these symlinks, Cursor and
Codex agents will fail to discover the skills unless a collaborator manually runs
`dotagents install`, which contradicts the intended setup.
There was a problem hiding this comment.
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.
|
|
||
| ## References | ||
|
|
||
| - [Sentry Commit Messages](https://develop.sentry.dev/engineering-practices/commit-messages/) |
There was a problem hiding this comment.
Cursor contribution guidelines deleted without equivalent replacement
Medium Severity
The 310-line .cursor/rules/contribution-guidelines.mdc is deleted but its content — covering MDX file structure, frontmatter fields, component usage (PlatformContent, OnboardingOptionButtons, alerts, expandable sections), content resolution priority, platform-specific guidelines, file naming conventions, navigation/sidebar ordering, and common anti-patterns — is not replaced by the new commit and create-branch skills. The existing .agents/skills/technical-docs/ and .agents/skills/docs-review/ only partially overlap. Cursor loses substantial project-specific documentation guidance.
agents.toml
Outdated
| version = 1 | ||
| # Skills are checked into git so collaborators don't need to run 'dotagents install'. | ||
| gitignore = false | ||
| agents = ["claude", "cursor", "codex"] |
There was a problem hiding this comment.
Cursor lacks skills symlink unlike Claude setup
Medium Severity
agents.toml declares agents = ["claude", "cursor", "codex"] and the PR creates .claude/skills as a symlink to ../.agents/skills, but no corresponding .cursor/skills symlink or Cursor rule file is created. Since gitignore = false means skills are checked into git (no dotagents install needed), Cursor has no mechanism to discover the new skills. The PR title promises skills for Cursor but Cursor gets nothing while losing its existing rules.
Additional Locations (1)
i tried to use dotagents but i am not sure its properly equipped for this usecase. just doing the |
d78326f to
042881c
Compare


DESCRIBE YOUR PR
Add the
commitskill from getsentry/skills for Claude Code, Cursor, and Codex agents..agents/skills/commit/— universal skill (Cursor, Codex, Cline, Gemini CLI, GitHub Copilot, etc.).claude/skills/commit— symlink for Claude Codeskills-lock.json— lock file for installed skillsIS YOUR CHANGE URGENT?
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Co-Authored-By: Claude noreply@anthropic.com