Skip to content

🚀 release: promote standalone clean-commit skill - #14

Merged
warengonzaga merged 3 commits into
mainfrom
dev
Sep 30, 2026
Merged

warengonzaga merged 3 commits into
mainfrom
dev

Conversation

@warengonzaga

@warengonzaga warengonzaga commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

Make the standalone clean-commit skill available from the default main branch, 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

  • Recorded installation and discovery without Clean Workflow in a fresh temporary Codex data directory; installed skill bytes matched the package.
  • For this promotion, the README's remote dev installation, listing, marketplace upgrade, reinstall, and removal commands passed using Codex CLI 0.158.0-alpha.2.1.
  • README relative links, code fences, package-version consistency, and diff whitespace checks passed. The README in remote dev matches the checked content.
  • Earlier skill, manifest, and specification checks are recorded in the implementation PR and 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 main installation 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 → main promotion boundary. Keep dev for continuing integration.

Release automation

Includes setup #15: .github/workflows/release.yml exactly matches the configuration in Clean Coding and Clean Code Review, using wgtechlabs/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.

* 📦 new: add installable clean-commit skill

* 🧪 test: record standalone skill discovery without clean workflow
Copilot AI balanced review requested due to automatic review settings September 30, 2026 04:13
@warengonzaga warengonzaga added the convention Convention rules and implementation (Issues/PRs) label Sep 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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 warengonzaga left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

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 warengonzaga left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

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.

@warengonzaga
warengonzaga merged commit fba4c29 into main Sep 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

convention Convention rules and implementation (Issues/PRs)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: add an installable Clean Commit skill and standalone plugin

2 participants