Skip to content

Add the update-protocol skill - #51

Merged
lwaldron merged 4 commits into
mainfrom
update-protocol-skill
Oct 23, 2026
Merged

lwaldron merged 4 commits into
mainfrom
update-protocol-skill

Conversation

@lwaldron

@lwaldron lwaldron commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Closes #36, together with #48.

Adds skills/update-protocol: bump version and date, insert the new entry at the top of ## History & Reviews with the user's #### Changes bullets and *No reviews yet.*, and update the pin in every composite in the same repository that lists the protocol in protocols_used, patch-bumping each with a Changes bullet of the form - Pin <name> at 1.0.1 (was 1.0.0).. Every check happens before any file is edited: semver, target version greater than current, date well formed and not earlier than the previous entry, top entry not already at the target, a clean baseline validator run, and the full dependent list with cycle detection. A failed check changes nothing.

Revised after ADR 0018 (#52): the skill no longer looks for dependents in other repositories. A composite elsewhere resolves its pin from its own source and its authors move it when they choose. Same-repository dependents are found by the sibling file existing under protocols/, and the patch bump of a sibling composite is the rule within a repository, where the head must be self-consistent.

Dogfood. On a scratch worktree of waldronlab/agent-protocols, a patch bump of humann4-sgb-aggregation tripped the stale-pin error from #48 on humann4-database-build, and the skill's pin update plus patch bump cleared it. The scratch branch was discarded.

🤖 Drafted with Claude Code

Bumps version and date, inserts the History entry, and updates the pin in
every same-repository composite, checking everything before editing
anything. Closes #36 together with the validator pin check.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

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

⚠️ Code review skipped — your organization has no extra usage available to pay for this review.

If your organization's extra usage balance is empty, an organization admin can add extra usage credits at claude.ai/admin-settings/usage. If its monthly spend limit was reached, an admin can raise it on the same page. If neither applies, contact Anthropic support.

Once extra usage is available, someone with write access to this repository can comment @claude review on this pull request to trigger a review.

lwaldron and others added 2 commits October 23, 2026 14:58
A composite in another repository resolves its pin from its own source
and is unaffected by a bump here. Same-repository dependents are found by
file presence, not by the detected repository name.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@lwaldron
lwaldron force-pushed the update-protocol-skill branch from 0de58b6 to 8db8f18 Compare October 23, 2026 19:07
@lwaldron
lwaldron merged commit 38e0b3e into main Oct 23, 2026
2 checks passed
@lwaldron
lwaldron deleted the update-protocol-skill branch October 23, 2026 19: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.

Add an update-protocol skill

1 participant