Skip to content

Agent Skill: Small commits / good git history#153

Draft
pmtk wants to merge 1 commit into
openshift-eng:mainfrom
pmtk:smol-commits
Draft

Agent Skill: Small commits / good git history#153
pmtk wants to merge 1 commit into
openshift-eng:mainfrom
pmtk:smol-commits

Conversation

@pmtk
Copy link
Copy Markdown
Contributor

@pmtk pmtk commented May 20, 2026

Summary by CodeRabbit

  • New Features

    • Added git-commits plugin enabling automated analysis and staging of changes into logical, coherent commits.
    • Introduced small-commits workflow skill for organizing pending changes with optional fixup commit and autosquash support.
  • Chores

    • Updated plugin registry with new plugin entries and reorganized existing plugins.
    • Bumped microshift-release version to 1.1.0.

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 20, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pmtk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 20, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 20, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Excluded labels (none allowed) (1)
  • do-not-merge/work-in-progress

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 2bbccde0-5b82-4a48-8e56-0da929abc47b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

This PR introduces a new git-commits plugin for the edge-tooling project, enabling developers to structure pending changes into coherent small commits. The change updates the plugin registry, defines the plugin manifest and skill reference, and provides comprehensive documentation covering installation, usage modes, workflow steps, edge cases, and examples.

Changes

Git-commits plugin addition

Layer / File(s) Summary
Plugin registry updates
.claude-plugin/marketplace.json
Registers the new git-commits plugin entry, relocates edge-ic and skills-review entries to new positions, and updates microshift-release version from 1.0.0 to 1.1.0.
Plugin manifest and skill reference
plugins/git-commits/.claude-plugin/plugin.json, .claude/skills/git-commits-small-commits
Defines the git-commits plugin metadata (name, description, version 1.0.0, author, homepage, Apache-2.0 license) and creates a skill reference alias linking to the plugin's small-commits skill implementation.
Plugin README and user guide
plugins/git-commits/README.md
Documents plugin installation via marketplace, two invocation modes (explicit and automatic), behavioral features (hunk-based staging, fixup commits, autosquash), requirements, and author attribution.
Skill workflow specification
plugins/git-commits/skills/small-commits/SKILL.md
Defines the small-commits skill's complete workflow: metadata and invocation constraints, goal statement with prerequisite checks, ordered execution steps (survey changes, plan commits, stage via patch-hunks, optional fixup/autosquash rebase), operational constraints and edge-case handling, usage examples, and side-effect notes.

🎯 2 (Simple) | ⏱️ ~12 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@pmtk pmtk marked this pull request as draft May 20, 2026 08:22
@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 20, 2026
Copy link
Copy Markdown
Contributor

@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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.claude/skills/git-commits-small-commits:
- Line 1: The entry in .claude/skills (the thin-wrapper
"git-commits-small-commits" pointing to
../../plugins/git-commits/skills/small-commits) must be removed and replaced by
the plugin-native skill under plugins/git-commits/skills; delete the
thin-wrapper file in .claude/skills, ensure the actual skill lives and is
authoritative in plugins/git-commits/skills/small-commits, and update any
registrations or references to point directly at that plugin path so there's a
single source of truth and no wrapper drift.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 37598f11-4139-4741-91f8-cd0f1d36edf7

📥 Commits

Reviewing files that changed from the base of the PR and between 40ac5ea and 2f379b9.

📒 Files selected for processing (5)
  • .claude-plugin/marketplace.json
  • .claude/skills/git-commits-small-commits
  • plugins/git-commits/.claude-plugin/plugin.json
  • plugins/git-commits/README.md
  • plugins/git-commits/skills/small-commits/SKILL.md

Comment thread .claude/skills/git-commits-small-commits Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant