fix(skill-pin): bump to v6 for Inv #13 multi-step BTC flows (closes #463) - #465
Merged
Conversation
Coordinated with vaultpilot-security-skill#17 which adds Invariant #13 covering multi-step BTC flows (combine_btc_psbts, prepare_btc_rbf_bump). Bumps three constants in src/diagnostics/skill-pin-drift.ts: - EXPECTED_SKILL_SHA256: e48d5c0c…007a → 83195093…72df - EXPECTED_SKILL_SENTINEL_B: _v5_ → _v6_ - EXPECTED_SKILL_SENTINEL_C: 9c4a2e7f3d816b50 → 8682084ac4984982 MUST land AFTER the skill repo PR merges. If this PR lands first, the runtime diagnostic in skill-pin-drift.ts fetches SKILL.md from master and finds v5 instead of the now-expected v6, causing every signing flow to halt with `vaultpilot-preflight skill integrity check FAILED` until the skill repo merges. Closes #463. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
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.
Summary
Coordinated MCP-side pin bump for vaultpilot-security-skill#17 which adds Invariant #13: Multi-step BTC flows.
Bumps three constants in
src/diagnostics/skill-pin-drift.ts:EXPECTED_SKILL_SHA256e48d5c0c…007a83195093…72dfEXPECTED_SKILL_SENTINEL_B_v5__v6_EXPECTED_SKILL_SENTINEL_C9c4a2e7f3d816b508682084ac4984982This PR is DRAFT until the skill PR merges, because:
skill-pin-drift.tsfetchesSKILL.mdfrom the skill repo'smasterand compares the hash toEXPECTED_SKILL_SHA256.vaultpilot-preflight skill integrity check FAILED.Test plan
tsc --noEmit)skill-pin-drift.test.ts17/17 pass against new constants🤖 Generated with Claude Code