Skip to content

docs: add skill drift-check line to release workflow - #45

Merged
ManningWorks merged 1 commit into
mainfrom
maintainer/projex-skill-drift-line
Oct 12, 2026
Merged

ManningWorks merged 1 commit into
mainfrom
maintainer/projex-skill-drift-line

Conversation

@ManningWorks

Copy link
Copy Markdown
Owner

What

Adds one release-process line to the existing release workflow, so the new end-user agent skill (skills/projex/SKILL.md, from #40) gets diffed against the current CLI surface on every release.

Where and why

.opencode/agents/release-manager.md is the repo's real release-process location — it encodes the bump → changelog → doc-verification → tests → tag workflow and is the doc AGENTS.md points to for @release-manager duties. The line is added as a checklist bullet in step 3 "Verify Documentation", adapted to the doc's existing bullet convention:

  • Verify agent skill drift: diff skills/projex/SKILL.md against current CLI --help output and flags; update the skill in the release PR if drifted

Out of scope

Skill content, README/docs links, and CLI changes — none touched.

One-line change; no code, no CLI surface, no semver impact.

@vercel

vercel Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
projex-docs Ready Ready Preview Oct 12, 2026 2:13am UTC

@coderabbitai

coderabbitai Bot commented Oct 12, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 598466e0-3316-488e-a2f5-67532239f4c3

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@ManningWorks

Copy link
Copy Markdown
Owner Author

Review verdict: APPROVE — no blocking findings. Reviewed against the four checks:

  1. Placement — pass. Line 104, step 3 "Verify Documentation", top-level bullet alongside "Check if release includes…", before the "Before proceeding:" gate. Correct level — it is the release manager's own check, not nested under @documentation-manager's delegated list — and it reads consistently with the doc's bullet and backtick conventions.
  2. Mechanical-check refinement — ruling: fine as-is (non-blocking LOW). The skill ships a captured drift baseline at skills/projex/references/cli-help.md (landed with docs: add projex agent skill (skills/projex) #40) with a "How to re-capture" recipe, so the check could be made mechanical: re-capture --help, diff against the baseline. Also, if flags drift, BOTH SKILL.md and the baseline need updating, and the line only names SKILL.md. I am not requiring rewording: as written the line is correct and actionable, and a stale baseline self-corrects (it re-flags on the next release). Optional one-line follow-up: "Verify agent skill drift: re-capture CLI --help per skills/projex/references/cli-help.md and diff against the baseline; if drifted, update both SKILL.md and the baseline in the release PR."
  3. Changelog — no entry needed. This is an internal release-process doc (.opencode/agents/), not user-facing; CHANGELOG.md (Keep a Changelog) tracks user-visible changes. Ruling recorded here so the merge tail does not stall on the judgment.
  4. Scope — pass. +1/−0, single file, single hunk, exactly one line, nothing else.

Note: PR #40 (home of the referenced skill) merged to main today (4679bf7), so the skills/projex/SKILL.md path the line references now exists.

Reviewed by Hermes Agent (reviewer profile)

@ManningWorks
ManningWorks merged commit 965a907 into main Oct 12, 2026
7 checks passed

This branch was successfully deployed

1 active deployment
Preview — 661dac1b Deployed Oct 12, 2026 by vercel[bot]
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