Repository navigation
Pin the offload skill's CLI to an exact version (Claude directory review) - #38
Merged
Merged
Conversation
…irectory review A range lets npx run a version published after review. CI now fails when the pin in claude/ differs from cli/package.json. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
011235813
marked this pull request as draft
November 9, 2026 00:05
Co-Authored-By: Claude Opus 5.5 <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 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.
Motivation
box-offloadskill runs the CLI asnpx -y @simular-ai/box@^0.2.0. A range lets npx pull a version published after review, so the directory review flags it.@simular-ai/box@0.2.0, and have CI keep the pin equal tocli/package.json.Changes
claude/skills/box-offload/SKILL.md:^0.2.0→0.2.0.claude/.claude-plugin/plugin.json: plugin version 0.2.0 → 0.2.1 so users pick up the change.publish-cli.yml: new step in the plugin job that fails when any@simular-ai/box@…inclaude/differs from the version incli/package.json.README.md: release steps say to bump the CLI version, the skill pin and the plugin version in the same PR, then tag right after the merge.Wiring block diagram
Interactions
npx @simular-ai/box ...incli/src) stay unpinned. They are CLI output, not plugin files, so the review doesn't cover them.Tests
^0.2.0, run the step's three lines from the repo root, and check that it prints the offending line and exits 1./plugin marketplace update simular-box,/plugin update box@simular-box, then/box:offloadin a test repo and check that it runsnpx -y @simular-ai/box@0.2.0.🤖 Generated with Claude Code