refactor(skills): drop the Snapshot drift sections the pre-flight check already covers - #1470
Merged
Merged
Conversation
…ck already covers apache#1435 removed these from the security skills. The same prose remained in 37 other skills and siblings across the pr-management, release-management, issue, contributor-growth, repo-health, pairing, setup and utilities families: compare .apache-magpie.local.lock with .apache-magpie.lock and propose `setup upgrade`. The generated pre-flight block does that in code, including the method/URL case since apache#1435, and stops the run where the prose only suggested. - The sections are removed, and the Step 0 "Drift check — see Snapshot drift above" items and the adopter-config pointers now say the pre-flight block reports snapshot drift. - write-skill's init_skill.py no longer scaffolds the section. - Kept on purpose: isolated-setup-install and isolated-setup-update, whose evals grade their own drift step, and status, which renders drift in its dashboard. About 480 lines removed. Generated-by: Claude Code (Opus 5.5)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Framework-wide follow-up to #1435.
Summary
#1435 removed the hand-written Snapshot drift sections from the security skills. The same prose remained in 37 other skills and sibling files, across the pr-management, release-management, issue, contributor-growth, repo-health, pairing, setup and utilities families. Each told the agent to compare
.apache-magpie.local.lockwith.apache-magpie.lockand proposesetup upgrade. The generated pre-flight block does that in code, including the method/URL case since #1435, and stops the run where the prose only suggested.write-skill'sinit_skill.pyno longer scaffolds the section into new skills.isolated-setup-installandisolated-setup-update: their evals grade their own drift step.status: it renders drift in its dashboard.Test plan
main, so they predate this change.Generated-by: Claude Code (Opus 5.5)
🤖 Generated with Claude Code