Skip to content

chore: revert PR #180 (pin plugin to CLI v3.0) — redo after CLI GA#181

Merged
kengio merged 1 commit into
mainfrom
chore/revert-pin-180
May 20, 2026
Merged

chore: revert PR #180 (pin plugin to CLI v3.0) — redo after CLI GA#181
kengio merged 1 commit into
mainfrom
chore/revert-pin-180

Conversation

@kengio
Copy link
Copy Markdown
Collaborator

@kengio kengio commented May 20, 2026

Reverts #180. Will redo after CLI v3.0.0 GA.

Why now

PR #180 declared a hard `requires.cli: '>=3.0.0-alpha.1'` and bumped the plugin version 2.4.13 → 3.0.0-alpha.1 ahead of the CLI's actual GA. Three issues surfaced in practice:

  1. The v3 CLI is still pre-GA (v3.0.0-alpha.5 in flight as of today). Anyone on a clean install picks up the plugin via `vault-sync` and then trips the version check until they manually update the CLI.
  2. The plugin itself still works against both Bun v2.3.3 and the Rust alpha series — there's no skill that requires v3-only behaviour today.
  3. Coupling plugin 3.0.0-alpha.1 to CLI 3.0.0-alpha.1 confuses the versioning story: plugin alpha + CLI alpha both at .alpha.1, drifting separately as each repo bumps.

Plan

  • This PR restores the pre-feat: pin plugin to CLI v3.0 (Rust) #180 plugin.json shape (no `requires.cli` field, no `version: 3.0.0-alpha.1`) and bumps the plugin to v2.4.14 to record the revert.
  • The CLI-v3 pin will be re-applied as a follow-up — likely `requires.cli: '>=3.0.0'` — once the CLI's v3.0.0 GA tag ships.

Diff

  • `.claude/plugins/onebrain/.claude-plugin/plugin.json`: `version` back to `2.4.14`, `requires.cli` field removed.
  • `PLUGIN-CHANGELOG.md`: drop the `v3.0.0-alpha.1` entry, add a v2.4.14 entry documenting the revert.

Test plan

  • `plugin.json` parses as valid JSON
  • `PLUGIN-CHANGELOG.md` frontmatter `latest_version` matches `plugin.json` `version`
  • No skill or instruction in the plugin actually requires CLI v3-only behaviour today (verified by audit summarised in PR feat: pin plugin to CLI v3.0 (Rust) #180 itself — all 13 subcommand argv shapes are still backwards-compatible with the Bun CLI)

Reverts commit 7d368f9.

PR #180 declared a hard `requires.cli: '>=3.0.0-alpha.1'` and bumped
the plugin version 2.4.13 -> 3.0.0-alpha.1 ahead of the CLI's actual
GA. Reasoning at the time: signal that downgrading the CLI to the v2.x
Bun binary is unsupported under the new plugin.

In practice this lands too early:

- The v3 CLI is still pre-GA (v3.0.0-alpha.5 at time of this commit).
  Anyone on a clean install picks up the plugin via vault-sync and
  then trips the version check until they manually update the CLI.
- The plugin itself still works against both Bun v2.3.3 and the Rust
  alpha series — there's no skill that actually requires v3-only
  behaviour today.
- Coupling plugin version 3.0.0-alpha.1 to the CLI alpha confuses the
  versioning story: plugin alpha + CLI alpha both at .alpha.1, drifting
  separately as each repo bumps.

This commit restores the pre-PR-#180 plugin.json shape (no
`requires.cli` field, no `version: 3.0.0-alpha.1`) and bumps the
plugin to v2.4.14 to record the revert. The CLI-v3 pin will be
re-applied — likely as `requires.cli: '>=3.0.0'` — once the CLI's
v3.0.0 GA tag ships.
@kengio kengio merged commit 370de99 into main May 20, 2026
1 check passed
@kengio kengio deleted the chore/revert-pin-180 branch May 20, 2026 06:51
kengio added a commit that referenced this pull request May 20, 2026
PR #181 reverted PR #180 and added a PLUGIN-CHANGELOG v2.4.14 entry,
but the plugin.json bump from 2.4.13 → 2.4.14 was missed during the
amend pass (Edit tool blocked on a Read-required precondition that
got dismissed in the noise). Result: PLUGIN-CHANGELOG advertises
v2.4.14 while plugin.json still reports v2.4.13.

This was caught when running /update against a vault on plugin
v2.4.13 — the version-check step (read upstream plugin.json + compare)
returned 'Already up to date' instead of triggering the v2.4.14 sync.

One-line bump to restore consistency. No code or config change beyond
the version field.
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