From 88dcfd1a7650ded809e2269c9b6975f4991540e2 Mon Sep 17 00:00:00 2001 From: wenjiefan Date: Thu, 30 Jul 2026 11:13:33 +0200 Subject: [PATCH] Fix bridge skill manifest references after #122 root move #122 moved the plugin manifest to the root plugin.json (and moved the bridge skill to skills/bcquality-al-review/), but the bridge SKILL.md prose still pointed at the now-deleted .claude-plugin/plugin.json: - ## Plugin root told the host to resolve PLUGIN_ROOT by anchoring on .claude-plugin/plugin.json, a marker that no longer exists, so the location-based fallback could never find it. Anchor on root plugin.json. - ## Notes described .claude-plugin/plugin.json as the manifest the plugin uses and root plugin.json as a future form -- the reverse of reality after #122. Describe root plugin.json as canonical and .claude-plugin/ marketplace.json as the marketplace entry. Doc-only; no behavior change. --- skills/bcquality-al-review/SKILL.md | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/skills/bcquality-al-review/SKILL.md b/skills/bcquality-al-review/SKILL.md index 20b2417..32c8207 100644 --- a/skills/bcquality-al-review/SKILL.md +++ b/skills/bcquality-al-review/SKILL.md @@ -24,8 +24,8 @@ Do **not** use this skill to *generate* AL code — it only reviews. ## Plugin root -Resolve `PLUGIN_ROOT` to the directory that contains this plugin's -`.claude-plugin/plugin.json`. This skill lives at +Resolve `PLUGIN_ROOT` to the directory that contains this plugin's root +`plugin.json`. This skill lives at `PLUGIN_ROOT/skills/bcquality-al-review/SKILL.md`, so `PLUGIN_ROOT` is two levels up from this file. All paths below are relative to `PLUGIN_ROOT`. If the host exposes a plugin-root environment variable, prefer it. @@ -93,7 +93,8 @@ caller can log the reason. `enabled-layers` (`BCQUALITY_ENABLED_LAYERS`) — the denied layers' files still exist on disk. Treat `enabled-layers` as a selection filter, not a hard security boundary. A future revision could add a genuine deny mechanism (e.g. pruning the installed tree). -- **Manifest location.** This plugin uses `.claude-plugin/plugin.json`, which both +- **Manifest location.** This plugin's manifest is the root `plugin.json`, which both Claude Code and Copilot CLI accept (verified with Copilot CLI: `plugin install` - reports the bridge skill loaded). Copilot CLI also accepts a root `plugin.json`; if a - future host only reads the root form, dual-home the manifest. + reports the bridge skill loaded). A `.claude-plugin/marketplace.json` alongside it + carries the marketplace entry. Claude Code also reads `.claude-plugin/plugin.json`; if + a future host only reads that form, dual-home the manifest there.