@kolisachint/hoocode-agent 0.5.2 → 0.5.4

package/dist/core/extensions/plugins/formats/platform-targets.d.ts

@@ -1,49 +1,73 @@
 /**
- * Session-wide artifact platform targeting (`--support-platform`).
+ * Session-wide artifact platform targeting (`--platform`).
  *
  * hoocode reads resources from every vendor convention, but when it *writes*
- * artifacts — authored plugins via ProposePlugin, workspace scaffolds via
- * /new-skill //new-agent //new-command — it needs a target layout. This module
- * owns that choice for the whole process:
+ * artifacts it needs a target layout. The two kinds of write have **different
+ * rules**, which is why this module exposes two resolvers rather than one:
+ *
+ *   Plugins            distribution units. A plugin only means anything as a
+ *                      member of a platform ecosystem (there is no `.agents`
+ *                      marketplace), so production is platform-specific and
+ *                      `agents` is never a valid target.
+ *                      → {@link resolvePluginPlatforms}
+ *   Workspace scaffolds  `/new-skill`, `/new-agent`, `/new-command`. These are
+ *                      consumed in place, never shipped, and `.agents/` is a
+ *                      real cross-vendor convention for them.
+ *                      → {@link getWorkspacePlatforms}
  *
  *   token vocabulary   `agents` (alias `native`), `claude`,
  *                      `github` (aliases `copilot`, `gh`)
- *   session state      set once at startup from the `--support-platform` flag
- *                      or the `supportPlatform` setting (main.ts)
- *   resolution         explicit per-call platforms → session targets →
- *                      {@link DEFAULT_AUTHORING_PLATFORMS}
+ *   session state      set once at startup from the `--platform` flag or the
+ *                      `platform` setting (main.ts)
+ *
+ * See docs/plugin-system-architecture.md §1 for the reasoning.
  *
  * Kept beside the format registry (and importing only `types.ts`) so the
  * platform vocabulary and the adapters stay a one-directory concern: adding a
  * vendor means a new adapter file plus a token here, nothing else.
  */
 import type { MarketplacePlatform } from "./types.js";
+/** The platforms a *plugin* can be produced for. `agents` is excluded by construction. */
+export type PluginPlatform = Exclude<MarketplacePlatform, "agents">;
+/** True for the platform tokens that own a plugin ecosystem. */
+export declare function isPluginPlatform(platform: MarketplacePlatform): platform is PluginPlatform;
 /**
- * Default authoring target when neither the call nor the session picked any: the
- * portable native format. Authored artifacts are meant to be reusable, so they
- * default to one vendor-neutral layout (a strict superset hoocode reads directly)
- * rather than forking into vendor copies. Vendor layouts are an opt-in interop
- * concern, requested only via the `--support-platform` session flag.
+ * Default plugin production target when neither the call nor the session picked
+ * one. Claude is the default because it is the only platform whose local loop
+ * closes: an authored plugin lands in `~/.claude/skills/<id>/` and is live on the
+ * next session, whereas Copilot has no in-place drop-in at all.
  */
-export declare const DEFAULT_AUTHORING_PLATFORMS: readonly MarketplacePlatform[];
+export declare const DEFAULT_PLUGIN_PLATFORMS: readonly PluginPlatform[];
 /** Canonicalize one platform token, folding the user-facing aliases. */
 export declare function normalizePlatformToken(token: string): MarketplacePlatform | undefined;
-export interface SupportPlatformParse {
+export interface PlatformParse {
     /** Canonical platforms, deduped, in the order first mentioned. */
     platforms: MarketplacePlatform[];
     /** Tokens that matched no known platform (surfaced as diagnostics, never fatal). */
     invalid: string[];
 }
-/** Parse raw `--support-platform` / `supportPlatform` tokens into canonical platforms. */
-export declare function parseSupportPlatforms(tokens: readonly string[]): SupportPlatformParse;
+/** Parse raw `--platform` / `platform` tokens into canonical platforms. */
+export declare function parsePlatforms(tokens: readonly string[]): PlatformParse;
 /** Set (or clear, with undefined/empty) the session's artifact platform targets. */
-export declare function setSupportPlatforms(platforms: readonly MarketplacePlatform[] | undefined): void;
-/** The session's configured targets, or undefined when `--support-platform` was not given. */
-export declare function getSupportPlatforms(): MarketplacePlatform[] | undefined;
+export declare function setPlatforms(platforms: readonly MarketplacePlatform[] | undefined): void;
 /**
- * Resolve the platforms an authoring write should target: an explicit per-call
- * selection wins, then the session's `--support-platform` targets, then
- * {@link DEFAULT_AUTHORING_PLATFORMS}.
+ * The session's configured targets for **workspace scaffolds**, or undefined when
+ * `--platform` was not given — callers treat undefined as "use the hoocode-native
+ * location" rather than substituting a default. `agents` is a legitimate value
+ * here: a scaffolded skill or command is consumed in place, not distributed.
+ */
+export declare function getWorkspacePlatforms(): MarketplacePlatform[] | undefined;
+/**
+ * Resolve the platforms a **plugin** should be produced for: an explicit per-call
+ * selection wins, then the session's `--platform` targets, then
+ * {@link DEFAULT_PLUGIN_PLATFORMS}.
+ *
+ * `agents` can never be the answer. An explicit `agents` throws — a caller asking
+ * for a plugin in a format no marketplace accepts has made a mistake and should
+ * hear about it. A session-level `agents` is *filtered* instead, because
+ * `--platform` also drives workspace scaffolds, where `agents` is a perfectly
+ * good choice; a user who set it for that reason should not have plugin authoring
+ * blow up in their face. If filtering leaves nothing, the default applies.
  */
-export declare function resolveAuthoringPlatforms(explicit?: readonly MarketplacePlatform[]): MarketplacePlatform[];
+export declare function resolvePluginPlatforms(explicit?: readonly MarketplacePlatform[]): PluginPlatform[];
 //# sourceMappingURL=platform-targets.d.ts.map

package/dist/core/extensions/plugins/formats/platform-targets.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"platform-targets.d.ts","sourceRoot":"","sources":["../../../../../src/core/extensions/plugins/formats/platform-targets.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAEH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,YAAY,CAAC;AAEtD;;;;;;GAMG;AACH,eAAO,MAAM,2BAA2B,EAAE,SAAS,mBAAmB,EAAe,CAAC;AAEtF,wEAAwE;AACxE,wBAAgB,sBAAsB,CAAC,KAAK,EAAE,MAAM,GAAG,mBAAmB,GAAG,SAAS,CAcrF;AAED,MAAM,WAAW,oBAAoB;IACpC,kEAAkE;IAClE,SAAS,EAAE,mBAAmB,EAAE,CAAC;IACjC,oFAAoF;IACpF,OAAO,EAAE,MAAM,EAAE,CAAC;CAClB;AAED,0FAA0F;AAC1F,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,SAAS,MAAM,EAAE,GAAG,oBAAoB,CAWrF;AAKD,oFAAoF;AACpF,wBAAgB,mBAAmB,CAAC,SAAS,EAAE,SAAS,mBAAmB,EAAE,GAAG,SAAS,GAAG,IAAI,CAE/F;AAED,8FAA8F;AAC9F,wBAAgB,mBAAmB,IAAI,mBAAmB,EAAE,GAAG,SAAS,CAEvE;AAED;;;;GAIG;AACH,wBAAgB,yBAAyB,CAAC,QAAQ,CAAC,EAAE,SAAS,mBAAmB,EAAE,GAAG,mBAAmB,EAAE,CAG1G","sourcesContent":["/**\n * Session-wide artifact platform targeting (`--support-platform`).\n *\n * hoocode reads resources from every vendor convention, but when it *writes*\n * artifacts — authored plugins via ProposePlugin, workspace scaffolds via\n * /new-skill //new-agent //new-command — it needs a target layout. This module\n * owns that choice for the whole process:\n *\n *   token vocabulary   `agents` (alias `native`), `claude`,\n *                      `github` (aliases `copilot`, `gh`)\n *   session state      set once at startup from the `--support-platform` flag\n *                      or the `supportPlatform` setting (main.ts)\n *   resolution         explicit per-call platforms → session targets →\n *                      {@link DEFAULT_AUTHORING_PLATFORMS}\n *\n * Kept beside the format registry (and importing only `types.ts`) so the\n * platform vocabulary and the adapters stay a one-directory concern: adding a\n * vendor means a new adapter file plus a token here, nothing else.\n */\n\nimport type { MarketplacePlatform } from \"./types.js\";\n\n/**\n * Default authoring target when neither the call nor the session picked any: the\n * portable native format. Authored artifacts are meant to be reusable, so they\n * default to one vendor-neutral layout (a strict superset hoocode reads directly)\n * rather than forking into vendor copies. Vendor layouts are an opt-in interop\n * concern, requested only via the `--support-platform` session flag.\n */\nexport const DEFAULT_AUTHORING_PLATFORMS: readonly MarketplacePlatform[] = [\"agents\"];\n\n/** Canonicalize one platform token, folding the user-facing aliases. */\nexport function normalizePlatformToken(token: string): MarketplacePlatform | undefined {\n\tswitch (token.trim().toLowerCase()) {\n\t\tcase \"agents\":\n\t\tcase \"native\":\n\t\t\treturn \"agents\";\n\t\tcase \"claude\":\n\t\t\treturn \"claude\";\n\t\tcase \"github\":\n\t\tcase \"copilot\":\n\t\tcase \"gh\":\n\t\t\treturn \"github\";\n\t\tdefault:\n\t\t\treturn undefined;\n\t}\n}\n\nexport interface SupportPlatformParse {\n\t/** Canonical platforms, deduped, in the order first mentioned. */\n\tplatforms: MarketplacePlatform[];\n\t/** Tokens that matched no known platform (surfaced as diagnostics, never fatal). */\n\tinvalid: string[];\n}\n\n/** Parse raw `--support-platform` / `supportPlatform` tokens into canonical platforms. */\nexport function parseSupportPlatforms(tokens: readonly string[]): SupportPlatformParse {\n\tconst platforms: MarketplacePlatform[] = [];\n\tconst invalid: string[] = [];\n\tfor (const token of tokens) {\n\t\tconst trimmed = token.trim();\n\t\tif (!trimmed) continue;\n\t\tconst canonical = normalizePlatformToken(trimmed);\n\t\tif (!canonical) invalid.push(trimmed);\n\t\telse if (!platforms.includes(canonical)) platforms.push(canonical);\n\t}\n\treturn { platforms, invalid };\n}\n\n/** Session-wide targets. Undefined = not configured (fall back to defaults). */\nlet sessionSupportPlatforms: MarketplacePlatform[] | undefined;\n\n/** Set (or clear, with undefined/empty) the session's artifact platform targets. */\nexport function setSupportPlatforms(platforms: readonly MarketplacePlatform[] | undefined): void {\n\tsessionSupportPlatforms = platforms && platforms.length > 0 ? [...platforms] : undefined;\n}\n\n/** The session's configured targets, or undefined when `--support-platform` was not given. */\nexport function getSupportPlatforms(): MarketplacePlatform[] | undefined {\n\treturn sessionSupportPlatforms ? [...sessionSupportPlatforms] : undefined;\n}\n\n/**\n * Resolve the platforms an authoring write should target: an explicit per-call\n * selection wins, then the session's `--support-platform` targets, then\n * {@link DEFAULT_AUTHORING_PLATFORMS}.\n */\nexport function resolveAuthoringPlatforms(explicit?: readonly MarketplacePlatform[]): MarketplacePlatform[] {\n\tif (explicit && explicit.length > 0) return [...explicit];\n\treturn [...(sessionSupportPlatforms ?? DEFAULT_AUTHORING_PLATFORMS)];\n}\n"]}
+{"version":3,"file":"platform-targets.d.ts","sourceRoot":"","sources":["../../../../../src/core/extensions/plugins/formats/platform-targets.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AAEH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,YAAY,CAAC;AAEtD,0FAA0F;AAC1F,MAAM,MAAM,cAAc,GAAG,OAAO,CAAC,mBAAmB,EAAE,QAAQ,CAAC,CAAC;AAEpE,gEAAgE;AAChE,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,mBAAmB,GAAG,QAAQ,IAAI,cAAc,CAE1F;AAED;;;;;GAKG;AACH,eAAO,MAAM,wBAAwB,EAAE,SAAS,cAAc,EAAe,CAAC;AAE9E,wEAAwE;AACxE,wBAAgB,sBAAsB,CAAC,KAAK,EAAE,MAAM,GAAG,mBAAmB,GAAG,SAAS,CAcrF;AAED,MAAM,WAAW,aAAa;IAC7B,kEAAkE;IAClE,SAAS,EAAE,mBAAmB,EAAE,CAAC;IACjC,oFAAoF;IACpF,OAAO,EAAE,MAAM,EAAE,CAAC;CAClB;AAED,2EAA2E;AAC3E,wBAAgB,cAAc,CAAC,MAAM,EAAE,SAAS,MAAM,EAAE,GAAG,aAAa,CAWvE;AAKD,oFAAoF;AACpF,wBAAgB,YAAY,CAAC,SAAS,EAAE,SAAS,mBAAmB,EAAE,GAAG,SAAS,GAAG,IAAI,CAExF;AAED;;;;;GAKG;AACH,wBAAgB,qBAAqB,IAAI,mBAAmB,EAAE,GAAG,SAAS,CAEzE;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,CAAC,EAAE,SAAS,mBAAmB,EAAE,GAAG,cAAc,EAAE,CAclG","sourcesContent":["/**\n * Session-wide artifact platform targeting (`--platform`).\n *\n * hoocode reads resources from every vendor convention, but when it *writes*\n * artifacts it needs a target layout. The two kinds of write have **different\n * rules**, which is why this module exposes two resolvers rather than one:\n *\n *   Plugins            distribution units. A plugin only means anything as a\n *                      member of a platform ecosystem (there is no `.agents`\n *                      marketplace), so production is platform-specific and\n *                      `agents` is never a valid target.\n *                      → {@link resolvePluginPlatforms}\n *   Workspace scaffolds  `/new-skill`, `/new-agent`, `/new-command`. These are\n *                      consumed in place, never shipped, and `.agents/` is a\n *                      real cross-vendor convention for them.\n *                      → {@link getWorkspacePlatforms}\n *\n *   token vocabulary   `agents` (alias `native`), `claude`,\n *                      `github` (aliases `copilot`, `gh`)\n *   session state      set once at startup from the `--platform` flag or the\n *                      `platform` setting (main.ts)\n *\n * See docs/plugin-system-architecture.md §1 for the reasoning.\n *\n * Kept beside the format registry (and importing only `types.ts`) so the\n * platform vocabulary and the adapters stay a one-directory concern: adding a\n * vendor means a new adapter file plus a token here, nothing else.\n */\n\nimport type { MarketplacePlatform } from \"./types.js\";\n\n/** The platforms a *plugin* can be produced for. `agents` is excluded by construction. */\nexport type PluginPlatform = Exclude<MarketplacePlatform, \"agents\">;\n\n/** True for the platform tokens that own a plugin ecosystem. */\nexport function isPluginPlatform(platform: MarketplacePlatform): platform is PluginPlatform {\n\treturn platform !== \"agents\";\n}\n\n/**\n * Default plugin production target when neither the call nor the session picked\n * one. Claude is the default because it is the only platform whose local loop\n * closes: an authored plugin lands in `~/.claude/skills/<id>/` and is live on the\n * next session, whereas Copilot has no in-place drop-in at all.\n */\nexport const DEFAULT_PLUGIN_PLATFORMS: readonly PluginPlatform[] = [\"claude\"];\n\n/** Canonicalize one platform token, folding the user-facing aliases. */\nexport function normalizePlatformToken(token: string): MarketplacePlatform | undefined {\n\tswitch (token.trim().toLowerCase()) {\n\t\tcase \"agents\":\n\t\tcase \"native\":\n\t\t\treturn \"agents\";\n\t\tcase \"claude\":\n\t\t\treturn \"claude\";\n\t\tcase \"github\":\n\t\tcase \"copilot\":\n\t\tcase \"gh\":\n\t\t\treturn \"github\";\n\t\tdefault:\n\t\t\treturn undefined;\n\t}\n}\n\nexport interface PlatformParse {\n\t/** Canonical platforms, deduped, in the order first mentioned. */\n\tplatforms: MarketplacePlatform[];\n\t/** Tokens that matched no known platform (surfaced as diagnostics, never fatal). */\n\tinvalid: string[];\n}\n\n/** Parse raw `--platform` / `platform` tokens into canonical platforms. */\nexport function parsePlatforms(tokens: readonly string[]): PlatformParse {\n\tconst platforms: MarketplacePlatform[] = [];\n\tconst invalid: string[] = [];\n\tfor (const token of tokens) {\n\t\tconst trimmed = token.trim();\n\t\tif (!trimmed) continue;\n\t\tconst canonical = normalizePlatformToken(trimmed);\n\t\tif (!canonical) invalid.push(trimmed);\n\t\telse if (!platforms.includes(canonical)) platforms.push(canonical);\n\t}\n\treturn { platforms, invalid };\n}\n\n/** Session-wide targets. Undefined = not configured (fall back to per-consumer defaults). */\nlet sessionPlatforms: MarketplacePlatform[] | undefined;\n\n/** Set (or clear, with undefined/empty) the session's artifact platform targets. */\nexport function setPlatforms(platforms: readonly MarketplacePlatform[] | undefined): void {\n\tsessionPlatforms = platforms && platforms.length > 0 ? [...platforms] : undefined;\n}\n\n/**\n * The session's configured targets for **workspace scaffolds**, or undefined when\n * `--platform` was not given — callers treat undefined as \"use the hoocode-native\n * location\" rather than substituting a default. `agents` is a legitimate value\n * here: a scaffolded skill or command is consumed in place, not distributed.\n */\nexport function getWorkspacePlatforms(): MarketplacePlatform[] | undefined {\n\treturn sessionPlatforms ? [...sessionPlatforms] : undefined;\n}\n\n/**\n * Resolve the platforms a **plugin** should be produced for: an explicit per-call\n * selection wins, then the session's `--platform` targets, then\n * {@link DEFAULT_PLUGIN_PLATFORMS}.\n *\n * `agents` can never be the answer. An explicit `agents` throws — a caller asking\n * for a plugin in a format no marketplace accepts has made a mistake and should\n * hear about it. A session-level `agents` is *filtered* instead, because\n * `--platform` also drives workspace scaffolds, where `agents` is a perfectly\n * good choice; a user who set it for that reason should not have plugin authoring\n * blow up in their face. If filtering leaves nothing, the default applies.\n */\nexport function resolvePluginPlatforms(explicit?: readonly MarketplacePlatform[]): PluginPlatform[] {\n\tif (explicit && explicit.length > 0) {\n\t\tconst rejected = explicit.filter((p) => !isPluginPlatform(p));\n\t\tif (rejected.length > 0) {\n\t\t\tthrow new Error(\n\t\t\t\t`Cannot author a plugin for platform \"${rejected.join(\", \")}\": plugins are distribution units and ` +\n\t\t\t\t\t\"the native `agents` layout belongs to no marketplace, so it can never be published or installed. \" +\n\t\t\t\t\t\"Target `claude` or `github`, or author a workspace skill instead.\",\n\t\t\t);\n\t\t}\n\t\treturn explicit.filter(isPluginPlatform);\n\t}\n\tconst fromSession = (sessionPlatforms ?? []).filter(isPluginPlatform);\n\treturn fromSession.length > 0 ? fromSession : [...DEFAULT_PLUGIN_PLATFORMS];\n}\n"]}

package/dist/core/extensions/plugins/formats/platform-targets.js

@@ -1,30 +1,42 @@
 /**
- * Session-wide artifact platform targeting (`--support-platform`).
+ * Session-wide artifact platform targeting (`--platform`).
  *
  * hoocode reads resources from every vendor convention, but when it *writes*
- * artifacts — authored plugins via ProposePlugin, workspace scaffolds via
- * /new-skill //new-agent //new-command — it needs a target layout. This module
- * owns that choice for the whole process:
+ * artifacts it needs a target layout. The two kinds of write have **different
+ * rules**, which is why this module exposes two resolvers rather than one:
+ *
+ *   Plugins            distribution units. A plugin only means anything as a
+ *                      member of a platform ecosystem (there is no `.agents`
+ *                      marketplace), so production is platform-specific and
+ *                      `agents` is never a valid target.
+ *                      → {@link resolvePluginPlatforms}
+ *   Workspace scaffolds  `/new-skill`, `/new-agent`, `/new-command`. These are
+ *                      consumed in place, never shipped, and `.agents/` is a
+ *                      real cross-vendor convention for them.
+ *                      → {@link getWorkspacePlatforms}
  *
  *   token vocabulary   `agents` (alias `native`), `claude`,
  *                      `github` (aliases `copilot`, `gh`)
- *   session state      set once at startup from the `--support-platform` flag
- *                      or the `supportPlatform` setting (main.ts)
- *   resolution         explicit per-call platforms → session targets →
- *                      {@link DEFAULT_AUTHORING_PLATFORMS}
+ *   session state      set once at startup from the `--platform` flag or the
+ *                      `platform` setting (main.ts)
+ *
+ * See docs/plugin-system-architecture.md §1 for the reasoning.
  *
  * Kept beside the format registry (and importing only `types.ts`) so the
  * platform vocabulary and the adapters stay a one-directory concern: adding a
  * vendor means a new adapter file plus a token here, nothing else.
  */
+/** True for the platform tokens that own a plugin ecosystem. */
+export function isPluginPlatform(platform) {
+    return platform !== "agents";
+}
 /**
- * Default authoring target when neither the call nor the session picked any: the
- * portable native format. Authored artifacts are meant to be reusable, so they
- * default to one vendor-neutral layout (a strict superset hoocode reads directly)
- * rather than forking into vendor copies. Vendor layouts are an opt-in interop
- * concern, requested only via the `--support-platform` session flag.
+ * Default plugin production target when neither the call nor the session picked
+ * one. Claude is the default because it is the only platform whose local loop
+ * closes: an authored plugin lands in `~/.claude/skills/<id>/` and is live on the
+ * next session, whereas Copilot has no in-place drop-in at all.
  */
-export const DEFAULT_AUTHORING_PLATFORMS = ["agents"];
+export const DEFAULT_PLUGIN_PLATFORMS = ["claude"];
 /** Canonicalize one platform token, folding the user-facing aliases. */
 export function normalizePlatformToken(token) {
     switch (token.trim().toLowerCase()) {
@@ -41,8 +53,8 @@ export function normalizePlatformToken(token) {
             return undefined;
     }
 }
-/** Parse raw `--support-platform` / `supportPlatform` tokens into canonical platforms. */
-export function parseSupportPlatforms(tokens) {
+/** Parse raw `--platform` / `platform` tokens into canonical platforms. */
+export function parsePlatforms(tokens) {
     const platforms = [];
     const invalid = [];
     for (const token of tokens) {
@@ -57,24 +69,44 @@ export function parseSupportPlatforms(tokens) {
     }
     return { platforms, invalid };
 }
-/** Session-wide targets. Undefined = not configured (fall back to defaults). */
-let sessionSupportPlatforms;
+/** Session-wide targets. Undefined = not configured (fall back to per-consumer defaults). */
+let sessionPlatforms;
 /** Set (or clear, with undefined/empty) the session's artifact platform targets. */
-export function setSupportPlatforms(platforms) {
-    sessionSupportPlatforms = platforms && platforms.length > 0 ? [...platforms] : undefined;
+export function setPlatforms(platforms) {
+    sessionPlatforms = platforms && platforms.length > 0 ? [...platforms] : undefined;
 }
-/** The session's configured targets, or undefined when `--support-platform` was not given. */
-export function getSupportPlatforms() {
-    return sessionSupportPlatforms ? [...sessionSupportPlatforms] : undefined;
+/**
+ * The session's configured targets for **workspace scaffolds**, or undefined when
+ * `--platform` was not given — callers treat undefined as "use the hoocode-native
+ * location" rather than substituting a default. `agents` is a legitimate value
+ * here: a scaffolded skill or command is consumed in place, not distributed.
+ */
+export function getWorkspacePlatforms() {
+    return sessionPlatforms ? [...sessionPlatforms] : undefined;
 }
 /**
- * Resolve the platforms an authoring write should target: an explicit per-call
- * selection wins, then the session's `--support-platform` targets, then
- * {@link DEFAULT_AUTHORING_PLATFORMS}.
+ * Resolve the platforms a **plugin** should be produced for: an explicit per-call
+ * selection wins, then the session's `--platform` targets, then
+ * {@link DEFAULT_PLUGIN_PLATFORMS}.
+ *
+ * `agents` can never be the answer. An explicit `agents` throws — a caller asking
+ * for a plugin in a format no marketplace accepts has made a mistake and should
+ * hear about it. A session-level `agents` is *filtered* instead, because
+ * `--platform` also drives workspace scaffolds, where `agents` is a perfectly
+ * good choice; a user who set it for that reason should not have plugin authoring
+ * blow up in their face. If filtering leaves nothing, the default applies.
  */
-export function resolveAuthoringPlatforms(explicit) {
-    if (explicit && explicit.length > 0)
-        return [...explicit];
-    return [...(sessionSupportPlatforms ?? DEFAULT_AUTHORING_PLATFORMS)];
+export function resolvePluginPlatforms(explicit) {
+    if (explicit && explicit.length > 0) {
+        const rejected = explicit.filter((p) => !isPluginPlatform(p));
+        if (rejected.length > 0) {
+            throw new Error(`Cannot author a plugin for platform "${rejected.join(", ")}": plugins are distribution units and ` +
+                "the native `agents` layout belongs to no marketplace, so it can never be published or installed. " +
+                "Target `claude` or `github`, or author a workspace skill instead.");
+        }
+        return explicit.filter(isPluginPlatform);
+    }
+    const fromSession = (sessionPlatforms ?? []).filter(isPluginPlatform);
+    return fromSession.length > 0 ? fromSession : [...DEFAULT_PLUGIN_PLATFORMS];
 }
 //# sourceMappingURL=platform-targets.js.map

package/dist/core/extensions/plugins/formats/platform-targets.js.map

@@ -1 +1 @@
-{"version":3,"file":"platform-targets.js","sourceRoot":"","sources":["../../../../../src/core/extensions/plugins/formats/platform-targets.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAIH;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,2BAA2B,GAAmC,CAAC,QAAQ,CAAC,CAAC;AAEtF,wEAAwE;AACxE,MAAM,UAAU,sBAAsB,CAAC,KAAa,EAAmC;IACtF,QAAQ,KAAK,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,EAAE,CAAC;QACpC,KAAK,QAAQ,CAAC;QACd,KAAK,QAAQ;YACZ,OAAO,QAAQ,CAAC;QACjB,KAAK,QAAQ;YACZ,OAAO,QAAQ,CAAC;QACjB,KAAK,QAAQ,CAAC;QACd,KAAK,SAAS,CAAC;QACf,KAAK,IAAI;YACR,OAAO,QAAQ,CAAC;QACjB;YACC,OAAO,SAAS,CAAC;IACnB,CAAC;AAAA,CACD;AASD,0FAA0F;AAC1F,MAAM,UAAU,qBAAqB,CAAC,MAAyB,EAAwB;IACtF,MAAM,SAAS,GAA0B,EAAE,CAAC;IAC5C,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC5B,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,EAAE,CAAC;QAC7B,IAAI,CAAC,OAAO;YAAE,SAAS;QACvB,MAAM,SAAS,GAAG,sBAAsB,CAAC,OAAO,CAAC,CAAC;QAClD,IAAI,CAAC,SAAS;YAAE,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;aACjC,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,SAAS,CAAC;YAAE,SAAS,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IACpE,CAAC;IACD,OAAO,EAAE,SAAS,EAAE,OAAO,EAAE,CAAC;AAAA,CAC9B;AAED,gFAAgF;AAChF,IAAI,uBAA0D,CAAC;AAE/D,oFAAoF;AACpF,MAAM,UAAU,mBAAmB,CAAC,SAAqD,EAAQ;IAChG,uBAAuB,GAAG,SAAS,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;AAAA,CACzF;AAED,8FAA8F;AAC9F,MAAM,UAAU,mBAAmB,GAAsC;IACxE,OAAO,uBAAuB,CAAC,CAAC,CAAC,CAAC,GAAG,uBAAuB,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;AAAA,CAC1E;AAED;;;;GAIG;AACH,MAAM,UAAU,yBAAyB,CAAC,QAAyC,EAAyB;IAC3G,IAAI,QAAQ,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC;QAAE,OAAO,CAAC,GAAG,QAAQ,CAAC,CAAC;IAC1D,OAAO,CAAC,GAAG,CAAC,uBAAuB,IAAI,2BAA2B,CAAC,CAAC,CAAC;AAAA,CACrE","sourcesContent":["/**\n * Session-wide artifact platform targeting (`--support-platform`).\n *\n * hoocode reads resources from every vendor convention, but when it *writes*\n * artifacts — authored plugins via ProposePlugin, workspace scaffolds via\n * /new-skill //new-agent //new-command — it needs a target layout. This module\n * owns that choice for the whole process:\n *\n *   token vocabulary   `agents` (alias `native`), `claude`,\n *                      `github` (aliases `copilot`, `gh`)\n *   session state      set once at startup from the `--support-platform` flag\n *                      or the `supportPlatform` setting (main.ts)\n *   resolution         explicit per-call platforms → session targets →\n *                      {@link DEFAULT_AUTHORING_PLATFORMS}\n *\n * Kept beside the format registry (and importing only `types.ts`) so the\n * platform vocabulary and the adapters stay a one-directory concern: adding a\n * vendor means a new adapter file plus a token here, nothing else.\n */\n\nimport type { MarketplacePlatform } from \"./types.js\";\n\n/**\n * Default authoring target when neither the call nor the session picked any: the\n * portable native format. Authored artifacts are meant to be reusable, so they\n * default to one vendor-neutral layout (a strict superset hoocode reads directly)\n * rather than forking into vendor copies. Vendor layouts are an opt-in interop\n * concern, requested only via the `--support-platform` session flag.\n */\nexport const DEFAULT_AUTHORING_PLATFORMS: readonly MarketplacePlatform[] = [\"agents\"];\n\n/** Canonicalize one platform token, folding the user-facing aliases. */\nexport function normalizePlatformToken(token: string): MarketplacePlatform | undefined {\n\tswitch (token.trim().toLowerCase()) {\n\t\tcase \"agents\":\n\t\tcase \"native\":\n\t\t\treturn \"agents\";\n\t\tcase \"claude\":\n\t\t\treturn \"claude\";\n\t\tcase \"github\":\n\t\tcase \"copilot\":\n\t\tcase \"gh\":\n\t\t\treturn \"github\";\n\t\tdefault:\n\t\t\treturn undefined;\n\t}\n}\n\nexport interface SupportPlatformParse {\n\t/** Canonical platforms, deduped, in the order first mentioned. */\n\tplatforms: MarketplacePlatform[];\n\t/** Tokens that matched no known platform (surfaced as diagnostics, never fatal). */\n\tinvalid: string[];\n}\n\n/** Parse raw `--support-platform` / `supportPlatform` tokens into canonical platforms. */\nexport function parseSupportPlatforms(tokens: readonly string[]): SupportPlatformParse {\n\tconst platforms: MarketplacePlatform[] = [];\n\tconst invalid: string[] = [];\n\tfor (const token of tokens) {\n\t\tconst trimmed = token.trim();\n\t\tif (!trimmed) continue;\n\t\tconst canonical = normalizePlatformToken(trimmed);\n\t\tif (!canonical) invalid.push(trimmed);\n\t\telse if (!platforms.includes(canonical)) platforms.push(canonical);\n\t}\n\treturn { platforms, invalid };\n}\n\n/** Session-wide targets. Undefined = not configured (fall back to defaults). */\nlet sessionSupportPlatforms: MarketplacePlatform[] | undefined;\n\n/** Set (or clear, with undefined/empty) the session's artifact platform targets. */\nexport function setSupportPlatforms(platforms: readonly MarketplacePlatform[] | undefined): void {\n\tsessionSupportPlatforms = platforms && platforms.length > 0 ? [...platforms] : undefined;\n}\n\n/** The session's configured targets, or undefined when `--support-platform` was not given. */\nexport function getSupportPlatforms(): MarketplacePlatform[] | undefined {\n\treturn sessionSupportPlatforms ? [...sessionSupportPlatforms] : undefined;\n}\n\n/**\n * Resolve the platforms an authoring write should target: an explicit per-call\n * selection wins, then the session's `--support-platform` targets, then\n * {@link DEFAULT_AUTHORING_PLATFORMS}.\n */\nexport function resolveAuthoringPlatforms(explicit?: readonly MarketplacePlatform[]): MarketplacePlatform[] {\n\tif (explicit && explicit.length > 0) return [...explicit];\n\treturn [...(sessionSupportPlatforms ?? DEFAULT_AUTHORING_PLATFORMS)];\n}\n"]}
+{"version":3,"file":"platform-targets.js","sourceRoot":"","sources":["../../../../../src/core/extensions/plugins/formats/platform-targets.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AAOH,gEAAgE;AAChE,MAAM,UAAU,gBAAgB,CAAC,QAA6B,EAA8B;IAC3F,OAAO,QAAQ,KAAK,QAAQ,CAAC;AAAA,CAC7B;AAED;;;;;GAKG;AACH,MAAM,CAAC,MAAM,wBAAwB,GAA8B,CAAC,QAAQ,CAAC,CAAC;AAE9E,wEAAwE;AACxE,MAAM,UAAU,sBAAsB,CAAC,KAAa,EAAmC;IACtF,QAAQ,KAAK,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,EAAE,CAAC;QACpC,KAAK,QAAQ,CAAC;QACd,KAAK,QAAQ;YACZ,OAAO,QAAQ,CAAC;QACjB,KAAK,QAAQ;YACZ,OAAO,QAAQ,CAAC;QACjB,KAAK,QAAQ,CAAC;QACd,KAAK,SAAS,CAAC;QACf,KAAK,IAAI;YACR,OAAO,QAAQ,CAAC;QACjB;YACC,OAAO,SAAS,CAAC;IACnB,CAAC;AAAA,CACD;AASD,2EAA2E;AAC3E,MAAM,UAAU,cAAc,CAAC,MAAyB,EAAiB;IACxE,MAAM,SAAS,GAA0B,EAAE,CAAC;IAC5C,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC5B,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,EAAE,CAAC;QAC7B,IAAI,CAAC,OAAO;YAAE,SAAS;QACvB,MAAM,SAAS,GAAG,sBAAsB,CAAC,OAAO,CAAC,CAAC;QAClD,IAAI,CAAC,SAAS;YAAE,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;aACjC,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,SAAS,CAAC;YAAE,SAAS,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IACpE,CAAC;IACD,OAAO,EAAE,SAAS,EAAE,OAAO,EAAE,CAAC;AAAA,CAC9B;AAED,6FAA6F;AAC7F,IAAI,gBAAmD,CAAC;AAExD,oFAAoF;AACpF,MAAM,UAAU,YAAY,CAAC,SAAqD,EAAQ;IACzF,gBAAgB,GAAG,SAAS,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;AAAA,CAClF;AAED;;;;;GAKG;AACH,MAAM,UAAU,qBAAqB,GAAsC;IAC1E,OAAO,gBAAgB,CAAC,CAAC,CAAC,CAAC,GAAG,gBAAgB,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;AAAA,CAC5D;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,sBAAsB,CAAC,QAAyC,EAAoB;IACnG,IAAI,QAAQ,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACrC,MAAM,QAAQ,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,gBAAgB,CAAC,CAAC,CAAC,CAAC,CAAC;QAC9D,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACzB,MAAM,IAAI,KAAK,CACd,wCAAwC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,wCAAwC;gBAClG,mGAAmG;gBACnG,mEAAmE,CACpE,CAAC;QACH,CAAC;QACD,OAAO,QAAQ,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC;IAC1C,CAAC;IACD,MAAM,WAAW,GAAG,CAAC,gBAAgB,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC;IACtE,OAAO,WAAW,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,GAAG,wBAAwB,CAAC,CAAC;AAAA,CAC5E","sourcesContent":["/**\n * Session-wide artifact platform targeting (`--platform`).\n *\n * hoocode reads resources from every vendor convention, but when it *writes*\n * artifacts it needs a target layout. The two kinds of write have **different\n * rules**, which is why this module exposes two resolvers rather than one:\n *\n *   Plugins            distribution units. A plugin only means anything as a\n *                      member of a platform ecosystem (there is no `.agents`\n *                      marketplace), so production is platform-specific and\n *                      `agents` is never a valid target.\n *                      → {@link resolvePluginPlatforms}\n *   Workspace scaffolds  `/new-skill`, `/new-agent`, `/new-command`. These are\n *                      consumed in place, never shipped, and `.agents/` is a\n *                      real cross-vendor convention for them.\n *                      → {@link getWorkspacePlatforms}\n *\n *   token vocabulary   `agents` (alias `native`), `claude`,\n *                      `github` (aliases `copilot`, `gh`)\n *   session state      set once at startup from the `--platform` flag or the\n *                      `platform` setting (main.ts)\n *\n * See docs/plugin-system-architecture.md §1 for the reasoning.\n *\n * Kept beside the format registry (and importing only `types.ts`) so the\n * platform vocabulary and the adapters stay a one-directory concern: adding a\n * vendor means a new adapter file plus a token here, nothing else.\n */\n\nimport type { MarketplacePlatform } from \"./types.js\";\n\n/** The platforms a *plugin* can be produced for. `agents` is excluded by construction. */\nexport type PluginPlatform = Exclude<MarketplacePlatform, \"agents\">;\n\n/** True for the platform tokens that own a plugin ecosystem. */\nexport function isPluginPlatform(platform: MarketplacePlatform): platform is PluginPlatform {\n\treturn platform !== \"agents\";\n}\n\n/**\n * Default plugin production target when neither the call nor the session picked\n * one. Claude is the default because it is the only platform whose local loop\n * closes: an authored plugin lands in `~/.claude/skills/<id>/` and is live on the\n * next session, whereas Copilot has no in-place drop-in at all.\n */\nexport const DEFAULT_PLUGIN_PLATFORMS: readonly PluginPlatform[] = [\"claude\"];\n\n/** Canonicalize one platform token, folding the user-facing aliases. */\nexport function normalizePlatformToken(token: string): MarketplacePlatform | undefined {\n\tswitch (token.trim().toLowerCase()) {\n\t\tcase \"agents\":\n\t\tcase \"native\":\n\t\t\treturn \"agents\";\n\t\tcase \"claude\":\n\t\t\treturn \"claude\";\n\t\tcase \"github\":\n\t\tcase \"copilot\":\n\t\tcase \"gh\":\n\t\t\treturn \"github\";\n\t\tdefault:\n\t\t\treturn undefined;\n\t}\n}\n\nexport interface PlatformParse {\n\t/** Canonical platforms, deduped, in the order first mentioned. */\n\tplatforms: MarketplacePlatform[];\n\t/** Tokens that matched no known platform (surfaced as diagnostics, never fatal). */\n\tinvalid: string[];\n}\n\n/** Parse raw `--platform` / `platform` tokens into canonical platforms. */\nexport function parsePlatforms(tokens: readonly string[]): PlatformParse {\n\tconst platforms: MarketplacePlatform[] = [];\n\tconst invalid: string[] = [];\n\tfor (const token of tokens) {\n\t\tconst trimmed = token.trim();\n\t\tif (!trimmed) continue;\n\t\tconst canonical = normalizePlatformToken(trimmed);\n\t\tif (!canonical) invalid.push(trimmed);\n\t\telse if (!platforms.includes(canonical)) platforms.push(canonical);\n\t}\n\treturn { platforms, invalid };\n}\n\n/** Session-wide targets. Undefined = not configured (fall back to per-consumer defaults). */\nlet sessionPlatforms: MarketplacePlatform[] | undefined;\n\n/** Set (or clear, with undefined/empty) the session's artifact platform targets. */\nexport function setPlatforms(platforms: readonly MarketplacePlatform[] | undefined): void {\n\tsessionPlatforms = platforms && platforms.length > 0 ? [...platforms] : undefined;\n}\n\n/**\n * The session's configured targets for **workspace scaffolds**, or undefined when\n * `--platform` was not given — callers treat undefined as \"use the hoocode-native\n * location\" rather than substituting a default. `agents` is a legitimate value\n * here: a scaffolded skill or command is consumed in place, not distributed.\n */\nexport function getWorkspacePlatforms(): MarketplacePlatform[] | undefined {\n\treturn sessionPlatforms ? [...sessionPlatforms] : undefined;\n}\n\n/**\n * Resolve the platforms a **plugin** should be produced for: an explicit per-call\n * selection wins, then the session's `--platform` targets, then\n * {@link DEFAULT_PLUGIN_PLATFORMS}.\n *\n * `agents` can never be the answer. An explicit `agents` throws — a caller asking\n * for a plugin in a format no marketplace accepts has made a mistake and should\n * hear about it. A session-level `agents` is *filtered* instead, because\n * `--platform` also drives workspace scaffolds, where `agents` is a perfectly\n * good choice; a user who set it for that reason should not have plugin authoring\n * blow up in their face. If filtering leaves nothing, the default applies.\n */\nexport function resolvePluginPlatforms(explicit?: readonly MarketplacePlatform[]): PluginPlatform[] {\n\tif (explicit && explicit.length > 0) {\n\t\tconst rejected = explicit.filter((p) => !isPluginPlatform(p));\n\t\tif (rejected.length > 0) {\n\t\t\tthrow new Error(\n\t\t\t\t`Cannot author a plugin for platform \"${rejected.join(\", \")}\": plugins are distribution units and ` +\n\t\t\t\t\t\"the native `agents` layout belongs to no marketplace, so it can never be published or installed. \" +\n\t\t\t\t\t\"Target `claude` or `github`, or author a workspace skill instead.\",\n\t\t\t);\n\t\t}\n\t\treturn explicit.filter(isPluginPlatform);\n\t}\n\tconst fromSession = (sessionPlatforms ?? []).filter(isPluginPlatform);\n\treturn fromSession.length > 0 ? fromSession : [...DEFAULT_PLUGIN_PLATFORMS];\n}\n"]}

package/dist/core/extensions/plugins/formats/shared.d.ts

@@ -28,7 +28,17 @@ export interface RawManifest {
     mcpServers?: Record<string, unknown> | string;
     /** Native-only. */
     providers?: unknown;
+    /** Copilot: extension directories, `{ paths, exclusive }`. Parsed for preservation only. */
+    extensions?: unknown;
+    /** Copilot: LSP config path or inline definitions. Parsed for preservation only. */
+    lspServers?: unknown;
+    /** Any key the adapter does not model. */
+    [key: string]: unknown;
 }
+/** Manifest keys not modelled here, preserved verbatim so a re-emit cannot drop them. */
+export declare function unknownManifestFields(raw: RawManifest): Record<string, unknown> | undefined;
+/** Which unsupported surfaces are present under `root`. */
+export declare function detectUnsupportedSurfaces(root: string): string[] | undefined;
 export declare function readJson<T>(file: string): T | null;
 export declare function dirIfExists(root: string, name: string): string | undefined;
 /**
@@ -75,4 +85,16 @@ export declare function toolsYamlList(tools: string | undefined): string | undef
  * `<root>/commands/<name>.md`. Copilot's differs and lives in its adapter.
  */
 export declare function claudeStyleWorkspace(root: string): WorkspaceLayout;
+/** The manifest keys the adapters model, as a sorted list. Input to the drift check (§2.2). */
+export declare function modelledManifestKeys(): string[];
+/**
+ * Surfaces we know about and deliberately do not load, as the **relative paths**
+ * they actually occupy rather than their display labels.
+ *
+ * The distinction bit once already: the labels are written for humans
+ * (`"monitors/"`), the vendor reference writes the real path
+ * (`monitors/monitors.json`), and a drift check comparing the two reported a
+ * known surface as a new finding.
+ */
+export declare function knownUnsupportedSurfaces(): string[];
 //# sourceMappingURL=shared.d.ts.map

package/dist/core/extensions/plugins/formats/shared.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"shared.d.ts","sourceRoot":"","sources":["../../../../../src/core/extensions/plugins/formats/shared.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AACxD,OAAO,KAAK,EAAE,YAAY,EAAE,eAAe,EAAE,MAAM,YAAY,CAAC;AAEhE,8FAA8F;AAC9F,MAAM,WAAW,WAAW;IAC3B,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,MAAM,CAAC,EAAE,MAAM,GAAG;QAAE,IAAI,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;IACpC,gIAAgI;IAChI,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,KAAK,CAAC,EAAE,iBAAiB,GAAG;QAAE,KAAK,CAAC,EAAE,iBAAiB,CAAA;KAAE,CAAC;IAC1D,yGAAyG;IACzG,UAAU,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,MAAM,CAAC;IAC9C,mBAAmB;IACnB,SAAS,CAAC,EAAE,OAAO,CAAC;CACpB;AAED,wBAAgB,QAAQ,CAAC,CAAC,EAAE,IAAI,EAAE,MAAM,GAAG,CAAC,GAAG,IAAI,CAMlD;AAED,wBAAgB,WAAW,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAG1E;AAED;;;;GAIG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,YAAY,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAQ9G;AAED,uEAAuE;AACvE,wBAAgB,WAAW,CAAC,MAAM,EAAE,WAAW,CAAC,QAAQ,CAAC,GAAG,MAAM,GAAG,SAAS,CAE7E;AAED,gGAAgG;AAChG,wBAAgB,cAAc,CAAC,GAAG,EAAE,WAAW,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,MAAM,GAAG,iBAAiB,GAAG,SAAS,CAmBrG;AAED;;;;;;;;;;;;GAYG;AACH,wBAAgB,YAAY,CAC3B,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,MAAM,GAAG,SAAS,EACjD,IAAI,EAAE,MAAM,EACZ,WAAW,SAAc,GACvB,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,SAAS,CAOrC;AA2BD,4DAA4D;AAC5D,wBAAgB,YAAY,CAAC,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,CAE7F;AAED,sFAAsF;AACtF,wBAAgB,QAAQ,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,CAE/C;AAED,kGAAkG;AAClG,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,YAAY,EAAE,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAWpF;AAED,+EAA+E;AAC/E,wBAAgB,IAAI,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAMzC;AAED;;;;GAIG;AACH,wBAAgB,aAAa,CAAC,KAAK,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,GAAG,SAAS,CAO3E;AAED;;;;GAIG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,GAAG,eAAe,CAgBlE","sourcesContent":["/**\n * Shared, format-agnostic helpers used by the plugin format adapters.\n *\n * Kept deliberately free of any runtime dependency on `../manifest.js` (it only\n * imports *types* from there) so the adapter registry can be wired without a\n * module cycle: `manifest.ts → formats/index.ts → formats/<adapter>.ts →\n * shared.ts`, with the back-reference to manifest types erased at compile time.\n */\n\nimport * as fs from \"node:fs\";\nimport * as path from \"node:path\";\nimport type { PluginHooksConfig } from \"../manifest.js\";\nimport type { AuthoredHook, WorkspaceLayout } from \"./types.js\";\n\n/** Raw manifest shape accepted by the `plugin.json`-style formats (agents/claude/copilot). */\nexport interface RawManifest {\n\tname?: string;\n\tversion?: string;\n\tdescription?: string;\n\tauthor?: string | { name?: string };\n\t/** Optional capability-dir override(s): a path (or list of paths) relative to the plugin root, e.g. `\"skills\": \"./skills/\"`. */\n\tskills?: unknown;\n\tcommands?: unknown;\n\tagents?: unknown;\n\tthemes?: unknown;\n\thooks?: PluginHooksConfig | { hooks?: PluginHooksConfig };\n\t/** Inline server map, or a path (Claude Code form, e.g. `\"./.mcp.json\"`) relative to the plugin root. */\n\tmcpServers?: Record<string, unknown> | string;\n\t/** Native-only. */\n\tproviders?: unknown;\n}\n\nexport function readJson<T>(file: string): T | null {\n\ttry {\n\t\treturn JSON.parse(fs.readFileSync(file, \"utf8\")) as T;\n\t} catch {\n\t\treturn null;\n\t}\n}\n\nexport function dirIfExists(root: string, name: string): string | undefined {\n\tconst p = path.join(root, name);\n\treturn fs.existsSync(p) && fs.statSync(p).isDirectory() ? p : undefined;\n}\n\n/**\n * Resolve a capability directory: an explicit manifest override — a path (or\n * list of paths, first existing wins) relative to the plugin root, e.g.\n * `\"skills\": \"./skills/\"` — beats the conventional directory.\n */\nexport function resolveCapabilityDir(root: string, override: unknown, conventional: string): string | undefined {\n\tconst candidates = Array.isArray(override) ? override : [override];\n\tfor (const c of candidates) {\n\t\tif (typeof c !== \"string\" || !c.trim()) continue;\n\t\tconst p = path.resolve(root, c.trim());\n\t\tif (fs.existsSync(p) && fs.statSync(p).isDirectory()) return p;\n\t}\n\treturn dirIfExists(root, conventional);\n}\n\n/** Coerce an author field (string or `{ name }`) to a plain string. */\nexport function parseAuthor(author: RawManifest[\"author\"]): string | undefined {\n\treturn typeof author === \"string\" ? author : author?.name;\n}\n\n/** Accept either `{ hooks: {...} }` or a bare event map, falling back to `hooks/hooks.json`. */\nexport function normalizeHooks(raw: RawManifest[\"hooks\"], root: string): PluginHooksConfig | undefined {\n\tlet config: PluginHooksConfig | undefined;\n\tif (raw && typeof raw === \"object\") {\n\t\tconfig = \"hooks\" in raw && raw.hooks ? (raw.hooks as PluginHooksConfig) : (raw as PluginHooksConfig);\n\t}\n\t// Fall back to the conventional hooks/hooks.json file.\n\tif (!config) {\n\t\tconst hooksFile = path.join(root, \"hooks\", \"hooks.json\");\n\t\tif (fs.existsSync(hooksFile)) {\n\t\t\tconst fileRaw = readJson<RawManifest[\"hooks\"]>(hooksFile);\n\t\t\tif (fileRaw && typeof fileRaw === \"object\") {\n\t\t\t\tconfig =\n\t\t\t\t\t\"hooks\" in fileRaw && fileRaw.hooks\n\t\t\t\t\t\t? (fileRaw.hooks as PluginHooksConfig)\n\t\t\t\t\t\t: (fileRaw as PluginHooksConfig);\n\t\t\t}\n\t\t}\n\t}\n\treturn config && Object.keys(config).length > 0 ? config : undefined;\n}\n\n/**\n * Resolve inline `mcpServers`, a manifest-supplied file path, or a `.mcp.json`\n * file at `root`.\n *\n * `raw` may be:\n *  - a string path (Claude Code form: `\"mcpServers\": \"./.mcp.json\"`), resolved\n *    relative to the plugin root and read as a server file;\n *  - an inline server map (returned as-is);\n *  - undefined, falling back to `<root>/<mcpFileName>`.\n *\n * Server files use `{ mcpServers }` (Claude/native) or `{ servers }` (Code/Copilot);\n * both keys are accepted.\n */\nexport function normalizeMcp(\n\traw: Record<string, unknown> | string | undefined,\n\troot: string,\n\tmcpFileName = \".mcp.json\",\n): Record<string, unknown> | undefined {\n\t// Claude Code form: a path to a server file, relative to the plugin root.\n\tif (typeof raw === \"string\") {\n\t\treturn raw.trim() ? readMcpFile(path.resolve(root, raw.trim())) : undefined;\n\t}\n\tif (raw && Object.keys(raw).length > 0) return raw;\n\treturn readMcpFile(path.join(root, mcpFileName));\n}\n\n/** Read a server file, accepting both `{ mcpServers }` and `{ servers }` keys. */\nfunction readMcpFile(mcpFile: string): Record<string, unknown> | undefined {\n\tif (!fs.existsSync(mcpFile)) return undefined;\n\tconst fileRaw = readJson<{ mcpServers?: Record<string, unknown>; servers?: Record<string, unknown> }>(mcpFile);\n\tconst servers = fileRaw?.mcpServers ?? fileRaw?.servers;\n\treturn servers && Object.keys(servers).length > 0 ? servers : undefined;\n}\n\n// ============================================================================\n// Emit helpers (used by the writer / scaffolding half of each adapter)\n// ============================================================================\n\n/** Serialize a small set of frontmatter fields to YAML. Values are strings only. */\nfunction emitFrontmatter(fields: Record<string, string | undefined>): string {\n\tconst lines: string[] = [\"---\"];\n\tfor (const [key, value] of Object.entries(fields)) {\n\t\tif (value === undefined || value === \"\") continue;\n\t\t// Quote values that could be misparsed as YAML (contain a colon-space or start punctuation).\n\t\tconst needsQuote = /[:#]|^[\\s>|@`&*!%]/.test(value) || value.includes(\"\\n\");\n\t\tlines.push(`${key}: ${needsQuote ? JSON.stringify(value) : value}`);\n\t}\n\tlines.push(\"---\");\n\treturn lines.join(\"\\n\");\n}\n\n/** A markdown capability file: frontmatter block + body. */\nexport function emitMarkdown(fields: Record<string, string | undefined>, body: string): string {\n\treturn `${emitFrontmatter(fields)}\\n\\n${body.trimEnd()}\\n`;\n}\n\n/** Pretty-print a JSON manifest with a trailing newline (matches repo convention). */\nexport function emitJson(value: unknown): string {\n\treturn `${JSON.stringify(value, null, 2)}\\n`;\n}\n\n/** Convert authored hooks into the on-disk `hooks.json` event-map shape shared by all formats. */\nexport function authoredHooksToConfig(hooks: AuthoredHook[]): Record<string, unknown> {\n\tconst byEvent: Record<string, Array<{ matcher?: string; hooks: unknown[] }>> = {};\n\tfor (const h of hooks) {\n\t\tconst group = byEvent[h.event] ?? [];\n\t\tbyEvent[h.event] = group;\n\t\tgroup.push({\n\t\t\t...(h.matcher ? { matcher: h.matcher } : {}),\n\t\t\thooks: [{ type: \"command\", command: h.command, ...(h.timeout ? { timeout: h.timeout } : {}) }],\n\t\t});\n\t}\n\treturn byEvent;\n}\n\n/** Slug-safe filename component (no path separators, conservative charset). */\nexport function slug(name: string): string {\n\treturn name\n\t\t.trim()\n\t\t.replace(/[^a-zA-Z0-9._-]+/g, \"-\")\n\t\t.replace(/^-+|-+$/g, \"\")\n\t\t.slice(0, 80);\n}\n\n/**\n * Render a comma-separated tool allowlist (Claude Code convention, e.g.\n * \"read, grep\") as a YAML flow sequence (`['read', 'grep']`) for formats whose\n * frontmatter takes tools as a list (Copilot custom agents / prompt files).\n */\nexport function toolsYamlList(tools: string | undefined): string | undefined {\n\tconst items = (tools ?? \"\")\n\t\t.split(/[,\\s]+/)\n\t\t.map((t) => t.trim())\n\t\t.filter(Boolean);\n\tif (items.length === 0) return undefined;\n\treturn `[${items.map((t) => `'${t.replace(/'/g, \"''\")}'`).join(\", \")}]`;\n}\n\n/**\n * The workspace layout shared by the `.agents/` and `.claude/` conventions:\n * `<root>/skills/<name>/SKILL.md`, `<root>/agents/<name>.md`,\n * `<root>/commands/<name>.md`. Copilot's differs and lives in its adapter.\n */\nexport function claudeStyleWorkspace(root: string): WorkspaceLayout {\n\treturn {\n\t\troot,\n\t\temitSkill: (s) => ({\n\t\t\tpath: path.join(root, \"skills\", slug(s.name), \"SKILL.md\"),\n\t\t\tcontent: emitMarkdown({ name: s.name, description: s.description }, s.body),\n\t\t}),\n\t\temitAgent: (a) => ({\n\t\t\tpath: path.join(root, \"agents\", `${slug(a.name)}.md`),\n\t\t\tcontent: emitMarkdown({ name: a.name, description: a.description, tools: a.tools, model: a.model }, a.body),\n\t\t}),\n\t\temitCommand: (c) => ({\n\t\t\tpath: path.join(root, \"commands\", `${slug(c.name)}.md`),\n\t\t\tcontent: emitMarkdown({ name: c.name, description: c.description }, c.body),\n\t\t}),\n\t};\n}\n"]}
+{"version":3,"file":"shared.d.ts","sourceRoot":"","sources":["../../../../../src/core/extensions/plugins/formats/shared.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AACxD,OAAO,KAAK,EAAE,YAAY,EAAE,eAAe,EAAE,MAAM,YAAY,CAAC;AAEhE,8FAA8F;AAC9F,MAAM,WAAW,WAAW;IAC3B,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,MAAM,CAAC,EAAE,MAAM,GAAG;QAAE,IAAI,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;IACpC,gIAAgI;IAChI,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,KAAK,CAAC,EAAE,iBAAiB,GAAG;QAAE,KAAK,CAAC,EAAE,iBAAiB,CAAA;KAAE,CAAC;IAC1D,yGAAyG;IACzG,UAAU,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,MAAM,CAAC;IAC9C,mBAAmB;IACnB,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,4FAA4F;IAC5F,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,oFAAoF;IACpF,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,0CAA0C;IAC1C,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC;CACvB;AA0BD,yFAAyF;AACzF,wBAAgB,qBAAqB,CAAC,GAAG,EAAE,WAAW,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,SAAS,CAM3F;AAkBD,2DAA2D;AAC3D,wBAAgB,yBAAyB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,EAAE,GAAG,SAAS,CAG5E;AAED,wBAAgB,QAAQ,CAAC,CAAC,EAAE,IAAI,EAAE,MAAM,GAAG,CAAC,GAAG,IAAI,CAMlD;AAED,wBAAgB,WAAW,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAG1E;AAED;;;;GAIG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,YAAY,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAQ9G;AAED,uEAAuE;AACvE,wBAAgB,WAAW,CAAC,MAAM,EAAE,WAAW,CAAC,QAAQ,CAAC,GAAG,MAAM,GAAG,SAAS,CAE7E;AAED,gGAAgG;AAChG,wBAAgB,cAAc,CAAC,GAAG,EAAE,WAAW,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,MAAM,GAAG,iBAAiB,GAAG,SAAS,CAmBrG;AAED;;;;;;;;;;;;GAYG;AACH,wBAAgB,YAAY,CAC3B,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,MAAM,GAAG,SAAS,EACjD,IAAI,EAAE,MAAM,EACZ,WAAW,SAAc,GACvB,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,SAAS,CAOrC;AAmCD,4DAA4D;AAC5D,wBAAgB,YAAY,CAAC,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,CAG7F;AAED,sFAAsF;AACtF,wBAAgB,QAAQ,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,CAE/C;AAED,kGAAkG;AAClG,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,YAAY,EAAE,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAWpF;AAED,+EAA+E;AAC/E,wBAAgB,IAAI,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAMzC;AAED;;;;GAIG;AACH,wBAAgB,aAAa,CAAC,KAAK,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,GAAG,SAAS,CAO3E;AAED;;;;GAIG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,GAAG,eAAe,CAgBlE;AAED,gGAA+F;AAC/F,wBAAgB,oBAAoB,IAAI,MAAM,EAAE,CAE/C;AAED;;;;;;;;GAQG;AACH,wBAAgB,wBAAwB,IAAI,MAAM,EAAE,CAEnD","sourcesContent":["/**\n * Shared, format-agnostic helpers used by the plugin format adapters.\n *\n * Kept deliberately free of any runtime dependency on `../manifest.js` (it only\n * imports *types* from there) so the adapter registry can be wired without a\n * module cycle: `manifest.ts → formats/index.ts → formats/<adapter>.ts →\n * shared.ts`, with the back-reference to manifest types erased at compile time.\n */\n\nimport * as fs from \"node:fs\";\nimport * as path from \"node:path\";\nimport type { PluginHooksConfig } from \"../manifest.js\";\nimport type { AuthoredHook, WorkspaceLayout } from \"./types.js\";\n\n/** Raw manifest shape accepted by the `plugin.json`-style formats (agents/claude/copilot). */\nexport interface RawManifest {\n\tname?: string;\n\tversion?: string;\n\tdescription?: string;\n\tauthor?: string | { name?: string };\n\t/** Optional capability-dir override(s): a path (or list of paths) relative to the plugin root, e.g. `\"skills\": \"./skills/\"`. */\n\tskills?: unknown;\n\tcommands?: unknown;\n\tagents?: unknown;\n\tthemes?: unknown;\n\thooks?: PluginHooksConfig | { hooks?: PluginHooksConfig };\n\t/** Inline server map, or a path (Claude Code form, e.g. `\"./.mcp.json\"`) relative to the plugin root. */\n\tmcpServers?: Record<string, unknown> | string;\n\t/** Native-only. */\n\tproviders?: unknown;\n\t/** Copilot: extension directories, `{ paths, exclusive }`. Parsed for preservation only. */\n\textensions?: unknown;\n\t/** Copilot: LSP config path or inline definitions. Parsed for preservation only. */\n\tlspServers?: unknown;\n\t/** Any key the adapter does not model. */\n\t[key: string]: unknown;\n}\n\n/**\n * Manifest keys every adapter models; anything else is preserved via\n * `unknownFields`.\n *\n * Surfaced through {@link declaredVocabulary} so the Tier 2 drift check has\n * something to diff the vendor references against. A checker that inferred our\n * coverage by reading the parser would be checking its own inference; a declared\n * set is a claim the codebase makes and the check can falsify.\n */\nconst MODELLED_MANIFEST_KEYS = new Set([\n\t\"name\",\n\t\"version\",\n\t\"description\",\n\t\"author\",\n\t\"skills\",\n\t\"commands\",\n\t\"agents\",\n\t\"themes\",\n\t\"hooks\",\n\t\"mcpServers\",\n\t\"providers\",\n\t\"$schema\",\n]);\n\n/** Manifest keys not modelled here, preserved verbatim so a re-emit cannot drop them. */\nexport function unknownManifestFields(raw: RawManifest): Record<string, unknown> | undefined {\n\tconst out: Record<string, unknown> = {};\n\tfor (const [key, value] of Object.entries(raw)) {\n\t\tif (!MODELLED_MANIFEST_KEYS.has(key)) out[key] = value;\n\t}\n\treturn Object.keys(out).length > 0 ? out : undefined;\n}\n\n/**\n * On-disk component surfaces hoocode does not load, checked against both vendors'\n * file-location tables. The files are left alone; naming them is what makes the\n * gap visible (see NormalizedPlugin.unsupportedSurfaces).\n */\nconst UNSUPPORTED_SURFACES: ReadonlyArray<{ rel: string; label: string }> = [\n\t{ rel: \"workflows\", label: \"workflows/\" },\n\t{ rel: \"output-styles\", label: \"output-styles/\" },\n\t{ rel: \"monitors/monitors.json\", label: \"monitors/\" },\n\t{ rel: \"bin\", label: \"bin/\" },\n\t{ rel: \"settings.json\", label: \"settings.json\" },\n\t{ rel: \".lsp.json\", label: \".lsp.json\" },\n\t{ rel: \"lsp.json\", label: \"lsp.json\" },\n\t{ rel: path.join(\".github\", \"lsp.json\"), label: \".github/lsp.json\" },\n];\n\n/** Which unsupported surfaces are present under `root`. */\nexport function detectUnsupportedSurfaces(root: string): string[] | undefined {\n\tconst found = UNSUPPORTED_SURFACES.filter((s) => fs.existsSync(path.join(root, s.rel))).map((s) => s.label);\n\treturn found.length > 0 ? found : undefined;\n}\n\nexport function readJson<T>(file: string): T | null {\n\ttry {\n\t\treturn JSON.parse(fs.readFileSync(file, \"utf8\")) as T;\n\t} catch {\n\t\treturn null;\n\t}\n}\n\nexport function dirIfExists(root: string, name: string): string | undefined {\n\tconst p = path.join(root, name);\n\treturn fs.existsSync(p) && fs.statSync(p).isDirectory() ? p : undefined;\n}\n\n/**\n * Resolve a capability directory: an explicit manifest override — a path (or\n * list of paths, first existing wins) relative to the plugin root, e.g.\n * `\"skills\": \"./skills/\"` — beats the conventional directory.\n */\nexport function resolveCapabilityDir(root: string, override: unknown, conventional: string): string | undefined {\n\tconst candidates = Array.isArray(override) ? override : [override];\n\tfor (const c of candidates) {\n\t\tif (typeof c !== \"string\" || !c.trim()) continue;\n\t\tconst p = path.resolve(root, c.trim());\n\t\tif (fs.existsSync(p) && fs.statSync(p).isDirectory()) return p;\n\t}\n\treturn dirIfExists(root, conventional);\n}\n\n/** Coerce an author field (string or `{ name }`) to a plain string. */\nexport function parseAuthor(author: RawManifest[\"author\"]): string | undefined {\n\treturn typeof author === \"string\" ? author : author?.name;\n}\n\n/** Accept either `{ hooks: {...} }` or a bare event map, falling back to `hooks/hooks.json`. */\nexport function normalizeHooks(raw: RawManifest[\"hooks\"], root: string): PluginHooksConfig | undefined {\n\tlet config: PluginHooksConfig | undefined;\n\tif (raw && typeof raw === \"object\") {\n\t\tconfig = \"hooks\" in raw && raw.hooks ? (raw.hooks as PluginHooksConfig) : (raw as PluginHooksConfig);\n\t}\n\t// Fall back to the conventional hooks/hooks.json file.\n\tif (!config) {\n\t\tconst hooksFile = path.join(root, \"hooks\", \"hooks.json\");\n\t\tif (fs.existsSync(hooksFile)) {\n\t\t\tconst fileRaw = readJson<RawManifest[\"hooks\"]>(hooksFile);\n\t\t\tif (fileRaw && typeof fileRaw === \"object\") {\n\t\t\t\tconfig =\n\t\t\t\t\t\"hooks\" in fileRaw && fileRaw.hooks\n\t\t\t\t\t\t? (fileRaw.hooks as PluginHooksConfig)\n\t\t\t\t\t\t: (fileRaw as PluginHooksConfig);\n\t\t\t}\n\t\t}\n\t}\n\treturn config && Object.keys(config).length > 0 ? config : undefined;\n}\n\n/**\n * Resolve inline `mcpServers`, a manifest-supplied file path, or a `.mcp.json`\n * file at `root`.\n *\n * `raw` may be:\n *  - a string path (Claude Code form: `\"mcpServers\": \"./.mcp.json\"`), resolved\n *    relative to the plugin root and read as a server file;\n *  - an inline server map (returned as-is);\n *  - undefined, falling back to `<root>/<mcpFileName>`.\n *\n * Server files use `{ mcpServers }` (Claude/native) or `{ servers }` (Code/Copilot);\n * both keys are accepted.\n */\nexport function normalizeMcp(\n\traw: Record<string, unknown> | string | undefined,\n\troot: string,\n\tmcpFileName = \".mcp.json\",\n): Record<string, unknown> | undefined {\n\t// Claude Code form: a path to a server file, relative to the plugin root.\n\tif (typeof raw === \"string\") {\n\t\treturn raw.trim() ? readMcpFile(path.resolve(root, raw.trim())) : undefined;\n\t}\n\tif (raw && Object.keys(raw).length > 0) return raw;\n\treturn readMcpFile(path.join(root, mcpFileName));\n}\n\n/** Read a server file, accepting both `{ mcpServers }` and `{ servers }` keys. */\nfunction readMcpFile(mcpFile: string): Record<string, unknown> | undefined {\n\tif (!fs.existsSync(mcpFile)) return undefined;\n\tconst fileRaw = readJson<{ mcpServers?: Record<string, unknown>; servers?: Record<string, unknown> }>(mcpFile);\n\tconst servers = fileRaw?.mcpServers ?? fileRaw?.servers;\n\treturn servers && Object.keys(servers).length > 0 ? servers : undefined;\n}\n\n// ============================================================================\n// Emit helpers (used by the writer / scaffolding half of each adapter)\n// ============================================================================\n\n/** Serialize a small set of frontmatter fields to YAML. Values are strings only. */\n/**\n * Render frontmatter, or nothing at all when every field is empty.\n *\n * Emitting bare `---\\n---` looks harmless and is not: YAML parses an empty\n * document as `null`, and Claude Code rejects a component whose frontmatter is\n * not a mapping. A command authored without a description hit exactly that, and\n * the artifact was invalid in the ecosystem it was written for while\n * round-tripping happily through our own reader.\n */\nfunction emitFrontmatter(fields: Record<string, string | undefined>): string {\n\tconst lines: string[] = [];\n\tfor (const [key, value] of Object.entries(fields)) {\n\t\tif (value === undefined || value === \"\") continue;\n\t\t// Quote values that could be misparsed as YAML (contain a colon-space or start punctuation).\n\t\tconst needsQuote = /[:#]|^[\\s>|@`&*!%]/.test(value) || value.includes(\"\\n\");\n\t\tlines.push(`${key}: ${needsQuote ? JSON.stringify(value) : value}`);\n\t}\n\treturn lines.length > 0 ? [\"---\", ...lines, \"---\"].join(\"\\n\") : \"\";\n}\n\n/** A markdown capability file: frontmatter block + body. */\nexport function emitMarkdown(fields: Record<string, string | undefined>, body: string): string {\n\tconst frontmatter = emitFrontmatter(fields);\n\treturn frontmatter ? `${frontmatter}\\n\\n${body.trimEnd()}\\n` : `${body.trimEnd()}\\n`;\n}\n\n/** Pretty-print a JSON manifest with a trailing newline (matches repo convention). */\nexport function emitJson(value: unknown): string {\n\treturn `${JSON.stringify(value, null, 2)}\\n`;\n}\n\n/** Convert authored hooks into the on-disk `hooks.json` event-map shape shared by all formats. */\nexport function authoredHooksToConfig(hooks: AuthoredHook[]): Record<string, unknown> {\n\tconst byEvent: Record<string, Array<{ matcher?: string; hooks: unknown[] }>> = {};\n\tfor (const h of hooks) {\n\t\tconst group = byEvent[h.event] ?? [];\n\t\tbyEvent[h.event] = group;\n\t\tgroup.push({\n\t\t\t...(h.matcher ? { matcher: h.matcher } : {}),\n\t\t\thooks: [{ type: \"command\", command: h.command, ...(h.timeout ? { timeout: h.timeout } : {}) }],\n\t\t});\n\t}\n\treturn byEvent;\n}\n\n/** Slug-safe filename component (no path separators, conservative charset). */\nexport function slug(name: string): string {\n\treturn name\n\t\t.trim()\n\t\t.replace(/[^a-zA-Z0-9._-]+/g, \"-\")\n\t\t.replace(/^-+|-+$/g, \"\")\n\t\t.slice(0, 80);\n}\n\n/**\n * Render a comma-separated tool allowlist (Claude Code convention, e.g.\n * \"read, grep\") as a YAML flow sequence (`['read', 'grep']`) for formats whose\n * frontmatter takes tools as a list (Copilot custom agents / prompt files).\n */\nexport function toolsYamlList(tools: string | undefined): string | undefined {\n\tconst items = (tools ?? \"\")\n\t\t.split(/[,\\s]+/)\n\t\t.map((t) => t.trim())\n\t\t.filter(Boolean);\n\tif (items.length === 0) return undefined;\n\treturn `[${items.map((t) => `'${t.replace(/'/g, \"''\")}'`).join(\", \")}]`;\n}\n\n/**\n * The workspace layout shared by the `.agents/` and `.claude/` conventions:\n * `<root>/skills/<name>/SKILL.md`, `<root>/agents/<name>.md`,\n * `<root>/commands/<name>.md`. Copilot's differs and lives in its adapter.\n */\nexport function claudeStyleWorkspace(root: string): WorkspaceLayout {\n\treturn {\n\t\troot,\n\t\temitSkill: (s) => ({\n\t\t\tpath: path.join(root, \"skills\", slug(s.name), \"SKILL.md\"),\n\t\t\tcontent: emitMarkdown({ name: s.name, description: s.description }, s.body),\n\t\t}),\n\t\temitAgent: (a) => ({\n\t\t\tpath: path.join(root, \"agents\", `${slug(a.name)}.md`),\n\t\t\tcontent: emitMarkdown({ name: a.name, description: a.description, tools: a.tools, model: a.model }, a.body),\n\t\t}),\n\t\temitCommand: (c) => ({\n\t\t\tpath: path.join(root, \"commands\", `${slug(c.name)}.md`),\n\t\t\tcontent: emitMarkdown({ name: c.name, description: c.description }, c.body),\n\t\t}),\n\t};\n}\n\n/** The manifest keys the adapters model, as a sorted list. Input to the drift check (§2.2). */\nexport function modelledManifestKeys(): string[] {\n\treturn [...MODELLED_MANIFEST_KEYS].sort();\n}\n\n/**\n * Surfaces we know about and deliberately do not load, as the **relative paths**\n * they actually occupy rather than their display labels.\n *\n * The distinction bit once already: the labels are written for humans\n * (`\"monitors/\"`), the vendor reference writes the real path\n * (`monitors/monitors.json`), and a drift check comparing the two reported a\n * known surface as a new finding.\n */\nexport function knownUnsupportedSurfaces(): string[] {\n\treturn UNSUPPORTED_SURFACES.map((s) => s.rel.replace(/\\\\/g, \"/\")).sort();\n}\n"]}

package/dist/core/extensions/plugins/formats/shared.js

@@ -8,6 +8,58 @@
  */
 import * as fs from "node:fs";
 import * as path from "node:path";
+/**
+ * Manifest keys every adapter models; anything else is preserved via
+ * `unknownFields`.
+ *
+ * Surfaced through {@link declaredVocabulary} so the Tier 2 drift check has
+ * something to diff the vendor references against. A checker that inferred our
+ * coverage by reading the parser would be checking its own inference; a declared
+ * set is a claim the codebase makes and the check can falsify.
+ */
+const MODELLED_MANIFEST_KEYS = new Set([
+    "name",
+    "version",
+    "description",
+    "author",
+    "skills",
+    "commands",
+    "agents",
+    "themes",
+    "hooks",
+    "mcpServers",
+    "providers",
+    "$schema",
+]);
+/** Manifest keys not modelled here, preserved verbatim so a re-emit cannot drop them. */
+export function unknownManifestFields(raw) {
+    const out = {};
+    for (const [key, value] of Object.entries(raw)) {
+        if (!MODELLED_MANIFEST_KEYS.has(key))
+            out[key] = value;
+    }
+    return Object.keys(out).length > 0 ? out : undefined;
+}
+/**
+ * On-disk component surfaces hoocode does not load, checked against both vendors'
+ * file-location tables. The files are left alone; naming them is what makes the
+ * gap visible (see NormalizedPlugin.unsupportedSurfaces).
+ */
+const UNSUPPORTED_SURFACES = [
+    { rel: "workflows", label: "workflows/" },
+    { rel: "output-styles", label: "output-styles/" },
+    { rel: "monitors/monitors.json", label: "monitors/" },
+    { rel: "bin", label: "bin/" },
+    { rel: "settings.json", label: "settings.json" },
+    { rel: ".lsp.json", label: ".lsp.json" },
+    { rel: "lsp.json", label: "lsp.json" },
+    { rel: path.join(".github", "lsp.json"), label: ".github/lsp.json" },
+];
+/** Which unsupported surfaces are present under `root`. */
+export function detectUnsupportedSurfaces(root) {
+    const found = UNSUPPORTED_SURFACES.filter((s) => fs.existsSync(path.join(root, s.rel))).map((s) => s.label);
+    return found.length > 0 ? found : undefined;
+}
 export function readJson(file) {
     try {
         return JSON.parse(fs.readFileSync(file, "utf8"));
@@ -95,8 +147,17 @@ function readMcpFile(mcpFile) {
 // Emit helpers (used by the writer / scaffolding half of each adapter)
 // ============================================================================
 /** Serialize a small set of frontmatter fields to YAML. Values are strings only. */
+/**
+ * Render frontmatter, or nothing at all when every field is empty.
+ *
+ * Emitting bare `---\n---` looks harmless and is not: YAML parses an empty
+ * document as `null`, and Claude Code rejects a component whose frontmatter is
+ * not a mapping. A command authored without a description hit exactly that, and
+ * the artifact was invalid in the ecosystem it was written for while
+ * round-tripping happily through our own reader.
+ */
 function emitFrontmatter(fields) {
-    const lines = ["---"];
+    const lines = [];
     for (const [key, value] of Object.entries(fields)) {
         if (value === undefined || value === "")
             continue;
@@ -104,12 +165,12 @@ function emitFrontmatter(fields) {
         const needsQuote = /[:#]|^[\s>|@`&*!%]/.test(value) || value.includes("\n");
         lines.push(`${key}: ${needsQuote ? JSON.stringify(value) : value}`);
     }
-    lines.push("---");
-    return lines.join("\n");
+    return lines.length > 0 ? ["---", ...lines, "---"].join("\n") : "";
 }
 /** A markdown capability file: frontmatter block + body. */
 export function emitMarkdown(fields, body) {
-    return `${emitFrontmatter(fields)}\n\n${body.trimEnd()}\n`;
+    const frontmatter = emitFrontmatter(fields);
+    return frontmatter ? `${frontmatter}\n\n${body.trimEnd()}\n` : `${body.trimEnd()}\n`;
 }
 /** Pretty-print a JSON manifest with a trailing newline (matches repo convention). */
 export function emitJson(value) {
@@ -172,4 +233,20 @@ export function claudeStyleWorkspace(root) {
         }),
     };
 }
+/** The manifest keys the adapters model, as a sorted list. Input to the drift check (§2.2). */
+export function modelledManifestKeys() {
+    return [...MODELLED_MANIFEST_KEYS].sort();
+}
+/**
+ * Surfaces we know about and deliberately do not load, as the **relative paths**
+ * they actually occupy rather than their display labels.
+ *
+ * The distinction bit once already: the labels are written for humans
+ * (`"monitors/"`), the vendor reference writes the real path
+ * (`monitors/monitors.json`), and a drift check comparing the two reported a
+ * known surface as a new finding.
+ */
+export function knownUnsupportedSurfaces() {
+    return UNSUPPORTED_SURFACES.map((s) => s.rel.replace(/\\/g, "/")).sort();
+}
 //# sourceMappingURL=shared.js.map