Skip to content

feat(skills): 拆分 git 提交规范为独立 skill#32

Merged
DCjanus merged 2 commits intomasterfrom
feat/git-commit-skill
Mar 12, 2026
Merged

feat(skills): 拆分 git 提交规范为独立 skill#32
DCjanus merged 2 commits intomasterfrom
feat/git-commit-skill

Conversation

@DCjanus
Copy link
Copy Markdown
Owner

@DCjanus DCjanus commented Mar 12, 2026

Why

  • 将 git 提交相关规范从 AGENTS.md 中拆出,减少全局规则体积
  • 把提交/推送授权边界、提交信息与分支命名约定集中到单一 skill,便于复用

What

  • 新增 git-commit skill,收拢 commit、push、branch、message、trailer 等约定
  • 精简 AGENTS.md 中分散的 git 规则,改为在相关场景下引用该 skill
  • 调整章节编号,保留原有依赖与其他通用规则

Testing

  • 已检查 AGENTS.mdskills/git-commit/SKILL.md 的 diff
  • 已确认新 skill 覆盖原先 git 相关规则,且 AGENTS.md 仅保留入口说明

Co-authored-by: OpenAI Codex <codex@openai.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 12, 2026

📝 Walkthrough

Summary by CodeRabbit

Release Notes

  • Documentation
    • Reorganized and restructured agent documentation for improved navigation and clarity.
    • Updated bug-fixing test strategy guidelines to align with project standards.
    • Added comprehensive documentation for git-commit functionality, including usage conventions, commit message formats, and common scenarios.

Walkthrough

Documentation restructuring of AGENTS.md reorganizes sections related to Git commits, dependency management, testing strategies, and documentation standards. A new git-commit SKILL.md file documents operations covering git commits, pushes, and branch naming conventions with safeguards and authorization checks.

Changes

Cohort / File(s) Summary
AGENTS.md Restructuring
AGENTS.md
Comprehensive section reordering: sections 2.3–2.4 combined (Git commits merged with dependency management), sections 3.x renumbered and reordered (default policies, documentation style, Git conventions, version strategy), and sections 4.x renumbered (GitHub operations restructured to emphasize git-commit tool usage). Testing strategy descriptions refined to shift from mandatory test-first approach to flexible testing aligned with project conventions. Content largely preserved with updated numbering and hierarchies.
New Git Commit Skill Documentation
skills/git-commit/SKILL.md
New documentation file introducing git-commit skill covering scope (commit, push, branch naming), non-interactive safeguards, authorization checks, and convention adherence. Includes quick start, usage conventions (when/how to commit, trailer usage, branch naming), common scenarios with examples, commit message conventions (type/scope/summary), and reference to help commands.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: extracting git commit conventions from AGENTS.md into a new independent skill.
Description check ✅ Passed The description is directly related to the changeset, explaining the motivation, what was changed, and testing performed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/git-commit-skill

Comment @coderabbitai help to get the list of available commands and usage tips.

Co-authored-by: OpenAI Codex <codex@openai.com>
@DCjanus DCjanus marked this pull request as ready for review March 12, 2026 01:55
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
skills/git-commit/SKILL.md (1)

89-91: Remove or replace the self-referencing link.

The resources section contains [SKILL.md](SKILL.md), which links to the current file itself. This provides no value to readers who are already viewing this document. Consider either removing this section entirely or adding links to genuinely useful external resources (e.g., official Conventional Commits or Conventional Branch documentation).

♻️ Suggested fix
 ## 资源
 
-- [SKILL.md](SKILL.md)
+- [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/)
+- [Conventional Branch](https://conventional-branch.github.io/)

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 98ce6abf-e818-471e-8c51-147ee3990c3b

📥 Commits

Reviewing files that changed from the base of the PR and between 5d192be and 6e94116.

📒 Files selected for processing (2)
  • AGENTS.md
  • skills/git-commit/SKILL.md

@DCjanus DCjanus merged commit 3396ecd into master Mar 12, 2026
2 of 3 checks passed
@DCjanus DCjanus deleted the feat/git-commit-skill branch March 12, 2026 08:11
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.

1 participant