Skip to content

Fix DEV-57 install gap: bump plugin.json, gate version parity in CI - #2

Merged
TAJD merged 1 commit into
mainfrom
dev57-manifest-fix
Sep 6, 2026
Merged

TAJD merged 1 commit into
mainfrom
dev57-manifest-fix

Conversation

@TAJD

@TAJD TAJD commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • The DEV-57 fix (0.3.2: fix comment-guard false positives (DEV-28/41/43/49, DEV-57) #1) merged with package.json and CHANGELOG.md bumped to 0.3.2 but stop-writing-comments/.claude-plugin/plugin.json left at 0.3.0 — the file claude plugin update actually reads. Installed sessions saw "already at the latest version (0.3.0)" and never picked up the fix.
  • Bumps plugin.json to 0.3.2.
  • Adds a check to the validate CI job: fails the build if package.json's version and plugin.json's version ever diverge again, so this class of "tests pass, release can't ship" gap can't recur silently.

Test plan

  • npm test — 153/153 passing
  • Verified the new CI check fails when the two version fields are set to mismatched values, and passes at 0.3.2/0.3.2

HUMAN CHECK

  1. After merge, run claude plugin update stop-writing-comments (or restart a session) and confirm it now reports 0.3.2, not a no-op at 0.3.0.
  2. This does not retroactively fix sessions already running the cached 0.3.0 hook — they need claude plugin update + restart to pick up either this or the underlying DEV-57 fix.

🤖 Generated with Claude Code

https://claude.ai/code/session_01EAvmyrEwiqgQhbaA3w4TsM

…w-up)

0.3.1 and 0.3.2 bumped package.json/CHANGELOG but never
stop-writing-comments/.claude-plugin/plugin.json — the file the plugin
installer actually reads. claude plugin update reported "already at the
latest version (0.3.0)" and installed nothing even after the DEV-57 merge.

Bump plugin.json to 0.3.2, and add a CI check in the validate job that
fails the build if package.json's version and plugin.json's version ever
diverge again.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EAvmyrEwiqgQhbaA3w4TsM
@TAJD
TAJD merged commit 0a32574 into main Sep 6, 2026
7 checks passed
@TAJD
TAJD deleted the dev57-manifest-fix branch September 6, 2026 10:06
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