🚀 release: promote standalone clean-commit skill - #14
Conversation
* 📦 new: add installable clean-commit skill * 🧪 test: record standalone skill discovery without clean workflow
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The package, documentation, and verification guidance are consistent with the specification and stated promotion scope.
Review effort: Balanced
Findings: None
What changed in this PR
Promotes the standalone Clean Commit skill and its installation documentation to main.
Changes:
- Adds the self-contained skill and Codex plugin marketplace package.
- Documents installation, usage, maintenance, and verification.
- Clarifies breaking-change eligibility and the 72-character subject limit.
| File | Description |
|---|---|
skills/clean-commit/SKILL.md |
Defines standalone commit guidance. |
.codex-plugin/plugin.json |
Declares plugin metadata and skill path. |
.agents/plugins/marketplace.json |
Publishes the plugin catalog entry. |
README.md |
Documents installation and usage. |
tests/skill-scenarios.md |
Records manual verification scenarios and results. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
warengonzaga
left a comment
There was a problem hiding this comment.
Reviewed 6a0f59e9097e6c6fccc2eee0f42a637d65245882 against main using Clean Workflow / Clean Code Review, including the full promotion diff, package manifests, skill instructions, README, verification guidance, and existing paginated reviews/comments/threads.
No actionable findings in the reviewed scope. Copilot reported no findings. The skill matches the nine types, staged-diff boundaries, eligible breaking markers, and full-subject length limit. No Commit changes were needed.
Checks run in this pass: Skill Creator validation, manifest names and package paths, changed-document relative links, and diff whitespace checks all passed. An expanded link scan found examples/EXAMPLES.md in the unchanged quick reference is missing; verified that this link is identical on main, so it is a preexisting issue outside this promotion diff.
Installation/discovery and remote dev install/update results were inspected from the earlier verification record, not rerun in this pass. No model invocation, other vendor host, or live GitHub mutation through the skills was tested. Stable main installation remains pending this promotion. Comment-only review; no formal approval or merge of this promotion was submitted.
warengonzaga
left a comment
There was a problem hiding this comment.
No actionable findings in the reviewed scope at 3b06934daabfdfea40cb03b7b44fbc2c0eb3ebd3.
Rechecked the changes since the previous review: the release workflow and README automation documentation. The workflow matches the Clean Coding and Clean Code Review reference configuration, and its pinned action inputs and outputs support the plan, plugin-version update, and publish sequence.
Validation: reference-workflow comparison, YAML/action-contract checks, shell syntax, isolated execution of the manifest update and Git staging, README relative links, and whitespace checks passed.
GitHub currently reports this PR mergeable with a clean merge state. No CI checks are reported for this PR. The release has not been executed on a GitHub runner; it runs after the promotion reaches main. No new model-driven skill session was exercised in this follow-up. This is a comment review; no approval or merge was submitted.
Make the standalone
clean-commitskill available from the defaultmainbranch, including its plugin package, marketplace catalog, verification documentation, and updated README.Adds standalone commit drafting and validation guidance based on the staged diff. The README now explains installation verification, prerequisites, practical prompts, stable and development sources, updates/removal, eligible breaking-change types, and the complete 72-character subject limit.
Includes implementation #12 and README update #13. Closes #11.
Validation
devinstallation, listing, marketplace upgrade, reinstall, and removal commands passed using Codex CLI 0.158.0-alpha.2.1.devmatches the checked content.tests/skill-scenarios.md.Verification limits and merge strategy
No new model-invocation test or other vendor's host test was run, and no live GitHub label mutations or workflow merges were exercised through the skills. Stable
maininstallation will become available when this promotion merges. The added release workflow will publish tags and GitHub Releases after qualifying pushes to main, including this promotion. No release has been published yet, and downstream Clean Workflow bundle integration remains separate.Use Create a merge commit to preserve the
dev→mainpromotion boundary. Keepdevfor continuing integration.Release automation
Includes setup #15:
.github/workflows/release.ymlexactly matches the configuration in Clean Coding and Clean Code Review, usingwgtechlabs/release-build-flow-action@6df9cb42c24c296d902150d051a0b6be4422cccc. It plans the release, updates the plugin version, and publishes the planned changelog/tag/release. Automatic synchronization of other manifests is disabled. The README documents the behavior.YAML, pinned action input/output compatibility, shell syntax, changed-document links, and whitespace checks passed. The manifest update step passed in disposable Git repositories and staged only the intended version change. The complete GitHub-hosted release has not been run; it activates on main.
Clean Commit already has v1.0.0, so the action continues that release history; the initial-version setting applies only when there are no tags.