@skillsmith/core 0.12.0 → 0.12.2

package/dist/src/install/agent-pack-installer.cursor-mcp.d.ts

@@ -0,0 +1,53 @@
+/**
+ * Cursor-specific MCP registration step for `sklx agent install`
+ * (SMI-6279 Wave 9).
+ *
+ * Split out of `agent-pack-installer.harness.ts`'s generic `installMcpConfig`
+ * — mirrors the existing `agent-pack-installer.cursor-hooks.ts` split
+ * (SMI-5893 Wave 8a) for the same reason: Cursor's on-disk shape/resolution
+ * needs diverge structurally from the other 6 MCP-capable harnesses, so
+ * folding cursor-only logic into the shared function would either bloat it
+ * past the 500-line gate or force every other harness through cursor-only
+ * branches it doesn't need.
+ *
+ * Two independent, confirmed-broken things this fixes (GH#2368 V3 repro —
+ * see the SMI-6266 Wave 9 plan doc):
+ *   1. `buildAgentMcpEntryValue()` wrote the `npx`/`args` form into Cursor's
+ *      `mcp.json` — the exact shape two live UAT passes proved ENOENTs
+ *      inside Cursor's bundled Node (GH#2368 C-01) — and never set
+ *      `SKILLSMITH_CLIENT`, so re-running `agent install` (the very
+ *      remediation `formatUpdateNotification()` tells a user to run)
+ *      silently rewrote the SAME broken entry, even after the website docs
+ *      snippet was corrected (SMI-5893 Wave 11). {@link installCursorMcpConfig}
+ *      now writes `buildCursorMcpEntryValue()` instead — resolved binary
+ *      path (or the documented paste-here placeholder) plus
+ *      `SKILLSMITH_CLIENT=cursor`.
+ *   2. The installer wrote its entry under the JSON key `skillsmith`, while
+ *      the website/CLI-template docs snippet (which users are told to
+ *      hand-paste) uses the package-scoped key `@skillsmith/mcp-server` — a
+ *      user who did both ended up with TWO different server entries
+ *      coexisting in the same file, neither ever recognized as "the
+ *      other's" by the installer's own conflict-detection (which matches
+ *      purely on key). {@link installCursorMcpConfig} now writes under
+ *      `CURSOR_MCP_ENTRY_KEY` (`'@skillsmith/mcp-server'`) so a docs-paste
+ *      and an `agent install` reconcile as ONE entry, and
+ *      {@link stripStaleLegacyMcpKey} removes a pre-fix install's stale
+ *      `'skillsmith'`-keyed entry (mirroring the cursor-hooks legacy-key
+ *      cleanup precedent) so a re-install doesn't leave both keys present.
+ *
+ * Scoped to Cursor ONLY — `mergeJsonMcpEntry`'s default `entryKey`
+ * (`'skillsmith'`) and `buildAgentMcpEntryValue()`'s `npx` form are
+ * UNCHANGED for claude-code/copilot/windsurf/opencode/codex/hermes; nothing
+ * in this module is reachable from any other harness's install path.
+ *
+ * @module @skillsmith/core/install/agent-pack-installer.cursor-mcp
+ */
+import type { HarnessInstallCtx } from './agent-pack-installer.harness.js';
+import type { HarnessInstallReport } from './agent-pack-installer.types.js';
+/**
+ * Merge Cursor's `skillsmith` MCP server registration into `~/.cursor/mcp.json`
+ * under {@link CURSOR_MCP_ENTRY_KEY}, then clean up a stale legacy-keyed
+ * entry from a pre-SMI-6279 install (see module header).
+ */
+export declare function installCursorMcpConfig(ctx: HarnessInstallCtx, report: HarnessInstallReport): void;
+//# sourceMappingURL=agent-pack-installer.cursor-mcp.d.ts.map

package/dist/src/install/agent-pack-installer.cursor-mcp.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"agent-pack-installer.cursor-mcp.d.ts","sourceRoot":"","sources":["../../../src/install/agent-pack-installer.cursor-mcp.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2CG;AAeH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,mCAAmC,CAAA;AAC1E,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,iCAAiC,CAAA;AAQ3E;;;;GAIG;AACH,wBAAgB,sBAAsB,CAAC,GAAG,EAAE,iBAAiB,EAAE,MAAM,EAAE,oBAAoB,GAAG,IAAI,CAqDjG"}

package/dist/src/install/agent-pack-installer.cursor-mcp.js

@@ -0,0 +1,207 @@
+/**
+ * Cursor-specific MCP registration step for `sklx agent install`
+ * (SMI-6279 Wave 9).
+ *
+ * Split out of `agent-pack-installer.harness.ts`'s generic `installMcpConfig`
+ * — mirrors the existing `agent-pack-installer.cursor-hooks.ts` split
+ * (SMI-5893 Wave 8a) for the same reason: Cursor's on-disk shape/resolution
+ * needs diverge structurally from the other 6 MCP-capable harnesses, so
+ * folding cursor-only logic into the shared function would either bloat it
+ * past the 500-line gate or force every other harness through cursor-only
+ * branches it doesn't need.
+ *
+ * Two independent, confirmed-broken things this fixes (GH#2368 V3 repro —
+ * see the SMI-6266 Wave 9 plan doc):
+ *   1. `buildAgentMcpEntryValue()` wrote the `npx`/`args` form into Cursor's
+ *      `mcp.json` — the exact shape two live UAT passes proved ENOENTs
+ *      inside Cursor's bundled Node (GH#2368 C-01) — and never set
+ *      `SKILLSMITH_CLIENT`, so re-running `agent install` (the very
+ *      remediation `formatUpdateNotification()` tells a user to run)
+ *      silently rewrote the SAME broken entry, even after the website docs
+ *      snippet was corrected (SMI-5893 Wave 11). {@link installCursorMcpConfig}
+ *      now writes `buildCursorMcpEntryValue()` instead — resolved binary
+ *      path (or the documented paste-here placeholder) plus
+ *      `SKILLSMITH_CLIENT=cursor`.
+ *   2. The installer wrote its entry under the JSON key `skillsmith`, while
+ *      the website/CLI-template docs snippet (which users are told to
+ *      hand-paste) uses the package-scoped key `@skillsmith/mcp-server` — a
+ *      user who did both ended up with TWO different server entries
+ *      coexisting in the same file, neither ever recognized as "the
+ *      other's" by the installer's own conflict-detection (which matches
+ *      purely on key). {@link installCursorMcpConfig} now writes under
+ *      `CURSOR_MCP_ENTRY_KEY` (`'@skillsmith/mcp-server'`) so a docs-paste
+ *      and an `agent install` reconcile as ONE entry, and
+ *      {@link stripStaleLegacyMcpKey} removes a pre-fix install's stale
+ *      `'skillsmith'`-keyed entry (mirroring the cursor-hooks legacy-key
+ *      cleanup precedent) so a re-install doesn't leave both keys present.
+ *
+ * Scoped to Cursor ONLY — `mergeJsonMcpEntry`'s default `entryKey`
+ * (`'skillsmith'`) and `buildAgentMcpEntryValue()`'s `npx` form are
+ * UNCHANGED for claude-code/copilot/windsurf/opencode/codex/hermes; nothing
+ * in this module is reachable from any other harness's install path.
+ *
+ * @module @skillsmith/core/install/agent-pack-installer.cursor-mcp
+ */
+import { existsSync, readFileSync, writeFileSync } from 'node:fs';
+import { AGENT_MCP_TARGETS } from './agent-harness-targets.js';
+import { mergeJsonMcpEntry } from './agent-config-merge.json.js';
+import { writeBackup } from './agent-config-merge.json-array.js';
+import { deepEqualJson, markBackedUp, shouldBackup } from './agent-config-merge.types.js';
+import { CURSOR_MCP_ENTRY_KEY, LEGACY_MCP_ENTRY_KEY, buildAgentMcpEntryValue, buildCursorMcpEntryValue, } from './agent-pack-installer.entry.js';
+import { relocateUnderHome } from './agent-home-relocate.js';
+/** Same predicate as `agent-pack-installer.harness.ts`'s private `mergeSucceeded` — duplicated rather than exported to keep that module's helper private. */
+function mergeSucceeded(status) {
+    return status === 'created' || status === 'updated' || status === 'unchanged';
+}
+/**
+ * Merge Cursor's `skillsmith` MCP server registration into `~/.cursor/mcp.json`
+ * under {@link CURSOR_MCP_ENTRY_KEY}, then clean up a stale legacy-keyed
+ * entry from a pre-SMI-6279 install (see module header).
+ */
+export function installCursorMcpConfig(ctx, report) {
+    const target = AGENT_MCP_TARGETS.cursor;
+    const path = relocateUnderHome(target.path, ctx.homeDir);
+    const entryValue = buildCursorMcpEntryValue();
+    const result = mergeJsonMcpEntry({
+        path,
+        keyPath: target.keyPath,
+        entryKey: CURSOR_MCP_ENTRY_KEY,
+        entryValue,
+        backupDir: ctx.backupDir,
+        force: ctx.force,
+        alreadyBackedUpPaths: ctx.backedUpPaths,
+    });
+    report.mcpConfig = result;
+    if (result.status === 'conflict') {
+        report.notes.push(`MCP config at ${path} already has a '${CURSOR_MCP_ENTRY_KEY}' entry that doesn't look like ours — left untouched. Re-run with --force to overwrite, or edit ${path} manually.`);
+    }
+    if (result.status === 'error') {
+        report.notes.push(`MCP config merge failed at ${path}: ${result.errorMessage}`);
+    }
+    // Only attempt the legacy-key cleanup once our own write succeeded (or was
+    // already correct) — same data-loss guard as the cursor-hooks precedent:
+    // a 'conflict'/'error' write means we deliberately left the file alone (or
+    // couldn't touch it), so removing the legacy key too would still mutate a
+    // file the real merge just refused to touch.
+    const cleanup = mergeSucceeded(result.status)
+        ? stripStaleLegacyMcpKey(path, target.keyPath, ctx)
+        : null;
+    if (cleanup?.status === 'updated') {
+        report.notes.push(`Removed stale legacy '${LEGACY_MCP_ENTRY_KEY}' MCP entry at ${path} — superseded by '${CURSOR_MCP_ENTRY_KEY}' (aligns with the website/CLI docs snippet key, SMI-6279 Wave 9).`);
+    }
+    if (cleanup?.status === 'error') {
+        report.notes.push(`Found a stale legacy '${LEGACY_MCP_ENTRY_KEY}' MCP entry at ${path} but could not remove it: ${cleanup.errorMessage}. The new '${CURSOR_MCP_ENTRY_KEY}' entry above was written successfully; the stale entry was left in place — remove it manually.`);
+    }
+    if (mergeSucceeded(result.status) || cleanup?.status === 'updated') {
+        ctx.entries.push({
+            path,
+            kind: 'mcp-config',
+            harness: 'cursor',
+            backupPath: result.backupPath ?? cleanup?.backupPath ?? null,
+            executable: false,
+        });
+    }
+}
+/**
+ * Strict, purpose-built ownership fingerprint for the destructive delete in
+ * {@link stripStaleLegacyMcpKey} — deliberately NOT the shared
+ * `looksLikeOurMcpEntry` heuristic (`agent-config-merge.types.ts`).
+ *
+ * `looksLikeOurMcpEntry` is intentionally loose (substring-matches an
+ * `args` element, or merely checks a `SKILLSMITH_TOOL_PROFILE` key is
+ * PRESENT regardless of its value) because its only job is gating a
+ * same-key OVERWRITE — a false positive there is non-destructive, the
+ * entry we'd write next replaces whatever was there either way. Deleting a
+ * DIFFERENT key entirely (this function) is a strictly higher-risk
+ * operation: a false positive here permanently removes a user's config
+ * with nothing to replace it — e.g. a custom wrapper script whose `args`
+ * happen to reference the package name in one element among several, or a
+ * user's own hand-authored `skillsmith`-keyed entry that happens to set a
+ * `SKILLSMITH_TOOL_PROFILE` env var with a DIFFERENT value for unrelated
+ * reasons (code-review finding, GPT-5.6-Sol / SMI-6279).
+ *
+ * The bar here is "structurally identical to exactly what
+ * `buildAgentMcpEntryValue()` itself would have written" — checked via
+ * `deepEqualJson` against that function's actual return value (not a
+ * hand-duplicated literal, so this can never drift out of sync with the
+ * real shape): exact `command`, exact `args` sequence (not "contains"),
+ * exact `env` key SET (not just key presence) and exact value.
+ */
+function looksLikeExactLegacyNpxEntry(value) {
+    return deepEqualJson(value, buildAgentMcpEntryValue());
+}
+/**
+ * Remove a stale `LEGACY_MCP_ENTRY_KEY` (`'skillsmith'`)-keyed MCP entry
+ * left behind by a pre-SMI-6279 `agent install` run, but ONLY when it is
+ * structurally IDENTICAL to what that installer itself would have written
+ * (`looksLikeExactLegacyNpxEntry` above). A user's own hand-written
+ * `skillsmith`-named entry — even one that superficially resembles ours —
+ * is left completely untouched — this never silently deletes a foreign or
+ * near-miss entry just because it shares the old key name.
+ *
+ * Read-modify-write is scoped to exactly the `mcpServers.skillsmith` key —
+ * every other key in the file (including the entry this function's caller
+ * just wrote under `CURSOR_MCP_ENTRY_KEY`) is preserved byte-for-byte.
+ *
+ * @returns `null` when there's genuinely nothing to clean up (file missing,
+ *   no `mcpServers` object, no legacy key, or the legacy key isn't an EXACT
+ *   match for our own legacy shape) — a real no-op, not surfaced as a report
+ *   entry. A `MergeResult` with `status: 'updated'` when the legacy key was
+ *   removed, or `status: 'error'` when a match WAS found but reading,
+ *   backing up, or writing the file failed — this must reach the caller as
+ *   a reported failure, not collapse into the same `null` "nothing to do"
+ *   return as the routine no-match case (PR-07 finding, GPT-5.6-Sol /
+ *   SMI-6279): a swallowed write failure here would silently leave a
+ *   confirmed-stale, confirmed-broken legacy entry in place with no
+ *   indication anything went wrong.
+ */
+function stripStaleLegacyMcpKey(path, keyPath, ctx) {
+    if (!existsSync(path))
+        return null;
+    let doc;
+    try {
+        const parsed = JSON.parse(readFileSync(path, 'utf-8'));
+        if (!parsed || typeof parsed !== 'object' || Array.isArray(parsed))
+            return null;
+        doc = parsed;
+    }
+    catch {
+        // Unparsable/unreadable — not a match we can act on either way; the
+        // main merge above already succeeded reading+writing this same file
+        // moments earlier, so this is a genuine no-op, not a swallowed failure.
+        return null;
+    }
+    const containerKey = keyPath[0];
+    if (containerKey === undefined)
+        return null;
+    const container = doc[containerKey];
+    if (!container || typeof container !== 'object' || Array.isArray(container))
+        return null;
+    const containerObj = container;
+    const legacyEntry = containerObj[LEGACY_MCP_ENTRY_KEY];
+    if (legacyEntry === undefined)
+        return null;
+    // Foreign OR near-miss — never touch, even if named 'skillsmith'.
+    if (!looksLikeExactLegacyNpxEntry(legacyEntry))
+        return null;
+    // Past this point we've confirmed a real match to delete — any failure
+    // from here on is a genuine error, not a no-op, and must be reported.
+    try {
+        delete containerObj[LEGACY_MCP_ENTRY_KEY];
+        const backupPath = shouldBackup(path, ctx.backedUpPaths)
+            ? writeBackup(path, ctx.backupDir)
+            : null;
+        markBackedUp(path, ctx.backedUpPaths);
+        writeFileSync(path, JSON.stringify(doc, null, 2) + '\n', { mode: 0o600 });
+        return { status: 'updated', path, backupPath };
+    }
+    catch (error) {
+        return {
+            status: 'error',
+            path,
+            backupPath: null,
+            errorMessage: error instanceof Error ? error.message : String(error),
+        };
+    }
+}
+//# sourceMappingURL=agent-pack-installer.cursor-mcp.js.map

package/dist/src/install/agent-pack-installer.cursor-mcp.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"agent-pack-installer.cursor-mcp.js","sourceRoot":"","sources":["../../../src/install/agent-pack-installer.cursor-mcp.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2CG;AAEH,OAAO,EAAE,UAAU,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,SAAS,CAAA;AAEjE,OAAO,EAAE,iBAAiB,EAAE,MAAM,4BAA4B,CAAA;AAC9D,OAAO,EAAE,iBAAiB,EAAE,MAAM,8BAA8B,CAAA;AAChE,OAAO,EAAE,WAAW,EAAE,MAAM,oCAAoC,CAAA;AAChE,OAAO,EAAE,aAAa,EAAE,YAAY,EAAE,YAAY,EAAE,MAAM,+BAA+B,CAAA;AACzF,OAAO,EACL,oBAAoB,EACpB,oBAAoB,EACpB,uBAAuB,EACvB,wBAAwB,GACzB,MAAM,iCAAiC,CAAA;AACxC,OAAO,EAAE,iBAAiB,EAAE,MAAM,0BAA0B,CAAA;AAK5D,6JAA6J;AAC7J,SAAS,cAAc,CAAC,MAA6B;IACnD,OAAO,MAAM,KAAK,SAAS,IAAI,MAAM,KAAK,SAAS,IAAI,MAAM,KAAK,WAAW,CAAA;AAC/E,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,sBAAsB,CAAC,GAAsB,EAAE,MAA4B;IACzF,MAAM,MAAM,GAAG,iBAAiB,CAAC,MAAM,CAAA;IACvC,MAAM,IAAI,GAAG,iBAAiB,CAAC,MAAM,CAAC,IAAI,EAAE,GAAG,CAAC,OAAO,CAAC,CAAA;IACxD,MAAM,UAAU,GAAG,wBAAwB,EAAE,CAAA;IAE7C,MAAM,MAAM,GAAG,iBAAiB,CAAC;QAC/B,IAAI;QACJ,OAAO,EAAE,MAAM,CAAC,OAAO;QACvB,QAAQ,EAAE,oBAAoB;QAC9B,UAAU;QACV,SAAS,EAAE,GAAG,CAAC,SAAS;QACxB,KAAK,EAAE,GAAG,CAAC,KAAK;QAChB,oBAAoB,EAAE,GAAG,CAAC,aAAa;KACxC,CAAC,CAAA;IAEF,MAAM,CAAC,SAAS,GAAG,MAAM,CAAA;IACzB,IAAI,MAAM,CAAC,MAAM,KAAK,UAAU,EAAE,CAAC;QACjC,MAAM,CAAC,KAAK,CAAC,IAAI,CACf,iBAAiB,IAAI,mBAAmB,oBAAoB,mGAAmG,IAAI,YAAY,CAChL,CAAA;IACH,CAAC;IACD,IAAI,MAAM,CAAC,MAAM,KAAK,OAAO,EAAE,CAAC;QAC9B,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,8BAA8B,IAAI,KAAK,MAAM,CAAC,YAAY,EAAE,CAAC,CAAA;IACjF,CAAC;IAED,2EAA2E;IAC3E,yEAAyE;IACzE,2EAA2E;IAC3E,0EAA0E;IAC1E,6CAA6C;IAC7C,MAAM,OAAO,GAAG,cAAc,CAAC,MAAM,CAAC,MAAM,CAAC;QAC3C,CAAC,CAAC,sBAAsB,CAAC,IAAI,EAAE,MAAM,CAAC,OAAO,EAAE,GAAG,CAAC;QACnD,CAAC,CAAC,IAAI,CAAA;IACR,IAAI,OAAO,EAAE,MAAM,KAAK,SAAS,EAAE,CAAC;QAClC,MAAM,CAAC,KAAK,CAAC,IAAI,CACf,yBAAyB,oBAAoB,kBAAkB,IAAI,qBAAqB,oBAAoB,oEAAoE,CACjL,CAAA;IACH,CAAC;IACD,IAAI,OAAO,EAAE,MAAM,KAAK,OAAO,EAAE,CAAC;QAChC,MAAM,CAAC,KAAK,CAAC,IAAI,CACf,yBAAyB,oBAAoB,kBAAkB,IAAI,6BAA6B,OAAO,CAAC,YAAY,cAAc,oBAAoB,iGAAiG,CACxP,CAAA;IACH,CAAC;IAED,IAAI,cAAc,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,OAAO,EAAE,MAAM,KAAK,SAAS,EAAE,CAAC;QACnE,GAAG,CAAC,OAAO,CAAC,IAAI,CAAC;YACf,IAAI;YACJ,IAAI,EAAE,YAAY;YAClB,OAAO,EAAE,QAAQ;YACjB,UAAU,EAAE,MAAM,CAAC,UAAU,IAAI,OAAO,EAAE,UAAU,IAAI,IAAI;YAC5D,UAAU,EAAE,KAAK;SAClB,CAAC,CAAA;IACJ,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;;;GAwBG;AACH,SAAS,4BAA4B,CAAC,KAAc;IAClD,OAAO,aAAa,CAAC,KAAK,EAAE,uBAAuB,EAAE,CAAC,CAAA;AACxD,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;;;GAwBG;AACH,SAAS,sBAAsB,CAC7B,IAAY,EACZ,OAA0B,EAC1B,GAAsB;IAEtB,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC;QAAE,OAAO,IAAI,CAAA;IAElC,IAAI,GAA4B,CAAA;IAChC,IAAI,CAAC;QACH,MAAM,MAAM,GAAY,IAAI,CAAC,KAAK,CAAC,YAAY,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC,CAAA;QAC/D,IAAI,CAAC,MAAM,IAAI,OAAO,MAAM,KAAK,QAAQ,IAAI,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC;YAAE,OAAO,IAAI,CAAA;QAC/E,GAAG,GAAG,MAAiC,CAAA;IACzC,CAAC;IAAC,MAAM,CAAC;QACP,oEAAoE;QACpE,oEAAoE;QACpE,wEAAwE;QACxE,OAAO,IAAI,CAAA;IACb,CAAC;IAED,MAAM,YAAY,GAAG,OAAO,CAAC,CAAC,CAAC,CAAA;IAC/B,IAAI,YAAY,KAAK,SAAS;QAAE,OAAO,IAAI,CAAA;IAC3C,MAAM,SAAS,GAAG,GAAG,CAAC,YAAY,CAAC,CAAA;IACnC,IAAI,CAAC,SAAS,IAAI,OAAO,SAAS,KAAK,QAAQ,IAAI,KAAK,CAAC,OAAO,CAAC,SAAS,CAAC;QAAE,OAAO,IAAI,CAAA;IACxF,MAAM,YAAY,GAAG,SAAoC,CAAA;IAEzD,MAAM,WAAW,GAAG,YAAY,CAAC,oBAAoB,CAAC,CAAA;IACtD,IAAI,WAAW,KAAK,SAAS;QAAE,OAAO,IAAI,CAAA;IAC1C,kEAAkE;IAClE,IAAI,CAAC,4BAA4B,CAAC,WAAW,CAAC;QAAE,OAAO,IAAI,CAAA;IAE3D,uEAAuE;IACvE,sEAAsE;IACtE,IAAI,CAAC;QACH,OAAO,YAAY,CAAC,oBAAoB,CAAC,CAAA;QACzC,MAAM,UAAU,GAAG,YAAY,CAAC,IAAI,EAAE,GAAG,CAAC,aAAa,CAAC;YACtD,CAAC,CAAC,WAAW,CAAC,IAAI,EAAE,GAAG,CAAC,SAAS,CAAC;YAClC,CAAC,CAAC,IAAI,CAAA;QACR,YAAY,CAAC,IAAI,EAAE,GAAG,CAAC,aAAa,CAAC,CAAA;QACrC,aAAa,CAAC,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,GAAG,EAAE,IAAI,EAAE,CAAC,CAAC,GAAG,IAAI,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC,CAAA;QACzE,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,IAAI,EAAE,UAAU,EAAE,CAAA;IAChD,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO;YACL,MAAM,EAAE,OAAO;YACf,IAAI;YACJ,UAAU,EAAE,IAAI;YAChB,YAAY,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;SACrE,CAAA;IACH,CAAC;AACH,CAAC"}

package/dist/src/install/agent-pack-installer.cursor-mcp.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=agent-pack-installer.cursor-mcp.test.d.ts.map

package/dist/src/install/agent-pack-installer.cursor-mcp.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"agent-pack-installer.cursor-mcp.test.d.ts","sourceRoot":"","sources":["../../../src/install/agent-pack-installer.cursor-mcp.test.ts"],"names":[],"mappings":""}

package/dist/src/install/agent-pack-installer.cursor-mcp.test.js

@@ -0,0 +1,285 @@
+/**
+ * Cursor `mcp.json` entry-shape + key-alignment regression coverage
+ * (SMI-6279 Wave 9, GH#2368 V3).
+ *
+ * Split from agent-pack-installer.test.ts, mirroring the existing
+ * agent-pack-installer.cursor-hooks.test.ts split (same temp-HOME +
+ * manifest-env fixture shape). Covers the two independent things
+ * `agent-pack-installer.cursor-mcp.ts` fixes:
+ *   1. Entry SHAPE: a resolved `skillsmith-mcp` binary path (or the
+ *      documented paste-here placeholder when resolution fails) instead of
+ *      the `npx`/`args` form two live UAT passes proved ENOENTs inside
+ *      Cursor's bundled Node, plus `SKILLSMITH_CLIENT=cursor`.
+ *   2. Entry KEY: `@skillsmith/mcp-server` (matching the website/CLI docs
+ *      snippet) instead of `skillsmith`, with a legacy-key cleanup step for
+ *      a pre-fix install that still has the old key.
+ *
+ * `node:child_process`'s `execFileSync` is mocked so `resolveSkillsmithMcpBinPath`
+ * never depends on whether `skillsmith-mcp` happens to be on the test
+ * runner's real PATH.
+ *
+ * @module @skillsmith/core/install/agent-pack-installer.cursor-mcp.test
+ */
+import { mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+const { execFileSyncMock } = vi.hoisted(() => ({ execFileSyncMock: vi.fn() }));
+vi.mock('node:child_process', async (importOriginal) => {
+    const actual = await importOriginal();
+    return { ...actual, execFileSync: execFileSyncMock };
+});
+// Defaults to the real writeFileSync so every test in this file gets real
+// disk writes unmodified; only the one PR-07-regression test below swaps in
+// a conditional throw, keyed by path so it can't accidentally intercept the
+// unrelated skill-pack-file writes installAgentPack also performs in the
+// same run (confirmed necessary: a naive call-order-based override fired on
+// one of those instead, on the first pass at writing this test).
+const { writeFileSyncMock, realWriteFileSyncRef } = vi.hoisted(() => ({
+    writeFileSyncMock: vi.fn(),
+    realWriteFileSyncRef: { current: null },
+}));
+vi.mock('node:fs', async (importOriginal) => {
+    const actual = await importOriginal();
+    realWriteFileSyncRef.current = actual.writeFileSync;
+    writeFileSyncMock.mockImplementation(actual.writeFileSync);
+    return { ...actual, writeFileSync: writeFileSyncMock };
+});
+import { installAgentPack } from './agent-pack-installer.js';
+import { AGENT_INSTALL_DIR_ENV_VAR } from './agent-manifest.js';
+let homeDir;
+let manifestDir;
+let prevInstallDirEnv;
+const RESOLVED_BIN_PATH = '/usr/local/bin/skillsmith-mcp';
+function mcpJsonPath(home) {
+    return join(home, '.cursor', 'mcp.json');
+}
+function readCursorMcpJson(home) {
+    return JSON.parse(readFileSync(mcpJsonPath(home), 'utf-8'));
+}
+function seedCursorPresent() {
+    mkdirSync(join(homeDir, '.cursor', 'skills'), { recursive: true });
+}
+beforeEach(() => {
+    homeDir = mkdtempSync(join(tmpdir(), 'skillsmith-agent-install-home-'));
+    manifestDir = mkdtempSync(join(tmpdir(), 'skillsmith-agent-install-manifest-'));
+    prevInstallDirEnv = process.env[AGENT_INSTALL_DIR_ENV_VAR];
+    process.env[AGENT_INSTALL_DIR_ENV_VAR] = manifestDir;
+    execFileSyncMock.mockReset();
+    // Reset to the real passthrough (see the vi.mock('node:fs', ...) setup
+    // above) in case a prior test overrode it to simulate a write failure.
+    writeFileSyncMock.mockClear();
+});
+afterEach(() => {
+    rmSync(homeDir, { recursive: true, force: true });
+    rmSync(manifestDir, { recursive: true, force: true });
+    if (prevInstallDirEnv !== undefined)
+        process.env[AGENT_INSTALL_DIR_ENV_VAR] = prevInstallDirEnv;
+    else
+        delete process.env[AGENT_INSTALL_DIR_ENV_VAR];
+    vi.restoreAllMocks();
+});
+describe('installAgentPack — cursor mcp.json entry shape (SMI-6279 Wave 9)', () => {
+    it('writes the entry under the @skillsmith/mcp-server key with a resolved binary path + SKILLSMITH_CLIENT=cursor', () => {
+        execFileSyncMock.mockReturnValue(`${RESOLVED_BIN_PATH}\n`);
+        seedCursorPresent();
+        const result = installAgentPack({ homeDir });
+        const doc = readCursorMcpJson(homeDir);
+        const entry = doc.mcpServers['@skillsmith/mcp-server'];
+        expect(entry?.command).toBe(RESOLVED_BIN_PATH);
+        expect(entry?.env?.SKILLSMITH_CLIENT).toBe('cursor');
+        expect(entry?.env?.SKILLSMITH_TOOL_PROFILE).toBe('agent');
+        // No `args` — unlike the broken npx form, Cursor's entry invokes the
+        // resolved binary directly.
+        expect(entry?.args).toBeUndefined();
+        // No stray legacy key on a first-ever install.
+        expect(doc.mcpServers.skillsmith).toBeUndefined();
+        const report = result.harnessReports.find((r) => r.harness === 'cursor');
+        expect(report?.mcpConfig?.status).toBe('created');
+    });
+    it('falls back to the documented paste-here placeholder when the binary cannot be resolved', () => {
+        execFileSyncMock.mockImplementation(() => {
+            throw new Error('not found');
+        });
+        seedCursorPresent();
+        installAgentPack({ homeDir });
+        const doc = readCursorMcpJson(homeDir);
+        const entry = doc.mcpServers['@skillsmith/mcp-server'];
+        expect(entry?.command).toContain('which skillsmith-mcp');
+        expect(entry?.command).toContain('where skillsmith-mcp');
+    });
+    it('falls back to the placeholder (never hangs) when the which/where lookup times out — code-review finding, GPT-5.6-Sol / SMI-6279', () => {
+        // execFileSync throws a synchronous ETIMEDOUT-shaped error when its own
+        // `timeout` option elapses — the existing blanket catch already handles
+        // this the same as "not found", so simulating that error is sufficient
+        // to prove the timeout-case fallback without an actual hung process.
+        const timeoutError = Object.assign(new Error('spawnSync which ETIMEDOUT'), {
+            code: 'ETIMEDOUT',
+            killed: true,
+            signal: 'SIGTERM',
+        });
+        execFileSyncMock.mockImplementation(() => {
+            throw timeoutError;
+        });
+        seedCursorPresent();
+        installAgentPack({ homeDir });
+        const doc = readCursorMcpJson(homeDir);
+        const entry = doc.mcpServers['@skillsmith/mcp-server'];
+        expect(entry?.command).toContain('which skillsmith-mcp');
+        // And the bound is actually wired to the real execFileSync call, not
+        // just documented — a `timeout` option keeps a hung/substituted
+        // which/where from blocking `agent install` indefinitely.
+        expect(execFileSyncMock).toHaveBeenCalledWith(expect.any(String), ['skillsmith-mcp'], expect.objectContaining({ timeout: expect.any(Number) }));
+        const [, , options] = execFileSyncMock.mock.calls[0];
+        expect(options.timeout).toBeGreaterThan(0);
+        // A few seconds is plenty for a local PATH lookup — not a network call.
+        expect(options.timeout).toBeLessThanOrEqual(5000);
+    });
+    it('does not regress claude-code’s own entry — still the npx form under the "skillsmith" key', () => {
+        execFileSyncMock.mockReturnValue(`${RESOLVED_BIN_PATH}\n`);
+        seedCursorPresent();
+        installAgentPack({ homeDir });
+        const claudeDoc = JSON.parse(readFileSync(join(homeDir, '.claude', 'settings.json'), 'utf-8'));
+        expect(claudeDoc.mcpServers.skillsmith?.command).toBe('npx');
+        expect(claudeDoc.mcpServers.skillsmith?.env?.SKILLSMITH_TOOL_PROFILE).toBe('agent');
+        expect(claudeDoc.mcpServers.skillsmith?.env?.SKILLSMITH_CLIENT).toBeUndefined();
+    });
+});
+describe('installAgentPack — cursor mcp.json idempotency (SMI-6279 Wave 9)', () => {
+    it('re-install does not create a duplicate second server entry', () => {
+        execFileSyncMock.mockReturnValue(`${RESOLVED_BIN_PATH}\n`);
+        seedCursorPresent();
+        installAgentPack({ homeDir });
+        const second = installAgentPack({ homeDir });
+        const doc = readCursorMcpJson(homeDir);
+        expect(Object.keys(doc.mcpServers)).toEqual(['@skillsmith/mcp-server']);
+        const report = second.harnessReports.find((r) => r.harness === 'cursor');
+        expect(report?.mcpConfig?.status).toBe('unchanged');
+    });
+});
+describe('installAgentPack — cursor mcp.json stale legacy-key cleanup (SMI-6279 Wave 9)', () => {
+    it('removes a stale "skillsmith"-keyed entry from a pre-fix install, leaving exactly one entry', () => {
+        execFileSyncMock.mockReturnValue(`${RESOLVED_BIN_PATH}\n`);
+        seedCursorPresent();
+        writeFileSync(mcpJsonPath(homeDir), JSON.stringify({
+            mcpServers: {
+                skillsmith: {
+                    command: 'npx',
+                    args: ['-y', '@skillsmith/mcp-server'],
+                    env: { SKILLSMITH_TOOL_PROFILE: 'agent' },
+                },
+            },
+        }, null, 2));
+        installAgentPack({ homeDir });
+        const doc = readCursorMcpJson(homeDir);
+        expect(doc.mcpServers.skillsmith).toBeUndefined();
+        expect(doc.mcpServers['@skillsmith/mcp-server']?.command).toBe(RESOLVED_BIN_PATH);
+        expect(Object.keys(doc.mcpServers)).toEqual(['@skillsmith/mcp-server']);
+    });
+    it('preserves a foreign entry under the legacy "skillsmith" key — never deletes a user-owned unrelated entry', () => {
+        execFileSyncMock.mockReturnValue(`${RESOLVED_BIN_PATH}\n`);
+        seedCursorPresent();
+        const foreignEntry = { command: 'some-other-tool', args: ['--flag'] };
+        writeFileSync(mcpJsonPath(homeDir), JSON.stringify({ mcpServers: { skillsmith: foreignEntry } }, null, 2));
+        installAgentPack({ homeDir });
+        const doc = readCursorMcpJson(homeDir);
+        expect(doc.mcpServers.skillsmith).toEqual(foreignEntry);
+        expect(doc.mcpServers['@skillsmith/mcp-server']?.command).toBe(RESOLVED_BIN_PATH);
+    });
+    it('preserves a near-miss entry that references the package name in an extra description field (code-review finding, GPT-5.6-Sol / SMI-6279)', () => {
+        execFileSyncMock.mockReturnValue(`${RESOLVED_BIN_PATH}\n`);
+        seedCursorPresent();
+        // Structurally close to our own legacy shape (same command/args/env
+        // value) but NOT byte-identical — one extra key. The old loose
+        // `looksLikeOurMcpEntry` heuristic (substring-matches args, checks only
+        // key presence) would have matched this and deleted it; the tightened
+        // exact-shape check must not.
+        const nearMissEntry = {
+            command: 'npx',
+            args: ['-y', '@skillsmith/mcp-server'],
+            env: { SKILLSMITH_TOOL_PROFILE: 'agent' },
+            description: 'Custom wrapper — see @skillsmith/mcp-server docs for context',
+        };
+        writeFileSync(mcpJsonPath(homeDir), JSON.stringify({ mcpServers: { skillsmith: nearMissEntry } }, null, 2));
+        installAgentPack({ homeDir });
+        const doc = readCursorMcpJson(homeDir);
+        expect(doc.mcpServers.skillsmith).toEqual(nearMissEntry);
+        expect(doc.mcpServers['@skillsmith/mcp-server']?.command).toBe(RESOLVED_BIN_PATH);
+    });
+    it('preserves a near-miss "skillsmith"-keyed entry whose SKILLSMITH_TOOL_PROFILE value differs from what we would have written (code-review finding, GPT-5.6-Sol / SMI-6279)', () => {
+        execFileSyncMock.mockReturnValue(`${RESOLVED_BIN_PATH}\n`);
+        seedCursorPresent();
+        // Same command/args/key SET as our legacy shape, but a different env
+        // VALUE — the old loose heuristic only checked key presence, not value,
+        // and would have matched+deleted this too.
+        const nearMissEntry = {
+            command: 'npx',
+            args: ['-y', '@skillsmith/mcp-server'],
+            env: { SKILLSMITH_TOOL_PROFILE: 'debug' },
+        };
+        writeFileSync(mcpJsonPath(homeDir), JSON.stringify({ mcpServers: { skillsmith: nearMissEntry } }, null, 2));
+        installAgentPack({ homeDir });
+        const doc = readCursorMcpJson(homeDir);
+        expect(doc.mcpServers.skillsmith).toEqual(nearMissEntry);
+        expect(doc.mcpServers['@skillsmith/mcp-server']?.command).toBe(RESOLVED_BIN_PATH);
+    });
+    it('is idempotent: a second install run on an already-migrated file makes no further legacy-key change', () => {
+        execFileSyncMock.mockReturnValue(`${RESOLVED_BIN_PATH}\n`);
+        seedCursorPresent();
+        writeFileSync(mcpJsonPath(homeDir), JSON.stringify({
+            mcpServers: {
+                skillsmith: {
+                    command: 'npx',
+                    args: ['-y', '@skillsmith/mcp-server'],
+                    env: { SKILLSMITH_TOOL_PROFILE: 'agent' },
+                },
+            },
+        }, null, 2));
+        installAgentPack({ homeDir }); // first run: writes the new key, cleans the legacy one
+        const afterFirst = readFileSync(mcpJsonPath(homeDir), 'utf-8');
+        const second = installAgentPack({ homeDir }); // second run: nothing left to clean
+        const afterSecond = readFileSync(mcpJsonPath(homeDir), 'utf-8');
+        expect(afterSecond).toBe(afterFirst);
+        const report = second.harnessReports.find((r) => r.harness === 'cursor');
+        expect(report?.mcpConfig?.status).toBe('unchanged');
+    });
+    it('reports a cleanup write failure via report.notes instead of swallowing it as "nothing to clean up" (PR-07 finding, GPT-5.6-Sol / SMI-6279)', () => {
+        execFileSyncMock.mockReturnValue(`${RESOLVED_BIN_PATH}\n`);
+        seedCursorPresent();
+        // Seed BOTH the already-correct new entry and the stale legacy entry, so
+        // the main merge sees 'unchanged' (no write attempted there) and only
+        // the cleanup step's own write is exercised below.
+        writeFileSync(mcpJsonPath(homeDir), JSON.stringify({
+            mcpServers: {
+                skillsmith: {
+                    command: 'npx',
+                    args: ['-y', '@skillsmith/mcp-server'],
+                    env: { SKILLSMITH_TOOL_PROFILE: 'agent' },
+                },
+                '@skillsmith/mcp-server': {
+                    command: RESOLVED_BIN_PATH,
+                    env: { SKILLSMITH_TOOL_PROFILE: 'agent', SKILLSMITH_CLIENT: 'cursor' },
+                },
+            },
+        }, null, 2));
+        // Target only the cleanup's write to THIS file — installAgentPack also
+        // writes several unrelated skill-pack files earlier in the same run, and
+        // must keep writing those for real.
+        const targetPath = mcpJsonPath(homeDir);
+        writeFileSyncMock.mockImplementation((...args) => {
+            if (args[0] === targetPath) {
+                throw new Error('EACCES: permission denied (simulated)');
+            }
+            return realWriteFileSyncRef.current(...args);
+        });
+        const result = installAgentPack({ homeDir });
+        // The legacy key must still be on disk — the failed write must not have
+        // corrupted or partially applied the delete.
+        const doc = readCursorMcpJson(homeDir);
+        expect(doc.mcpServers.skillsmith).toBeDefined();
+        const report = result.harnessReports.find((r) => r.harness === 'cursor');
+        expect(report?.notes.some((n) => n.includes('could not remove it'))).toBe(true);
+        expect(report?.notes.some((n) => n.includes('EACCES'))).toBe(true);
+    });
+});
+//# sourceMappingURL=agent-pack-installer.cursor-mcp.test.js.map

package/dist/src/install/agent-pack-installer.cursor-mcp.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"agent-pack-installer.cursor-mcp.test.js","sourceRoot":"","sources":["../../../src/install/agent-pack-installer.cursor-mcp.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,OAAO,EAAE,SAAS,EAAE,WAAW,EAAE,YAAY,EAAE,MAAM,EAAE,aAAa,EAAE,MAAM,SAAS,CAAA;AACrF,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AAChC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAA;AAChC,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AAExE,MAAM,EAAE,gBAAgB,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,gBAAgB,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAA;AAC9E,EAAE,CAAC,IAAI,CAAC,oBAAoB,EAAE,KAAK,EAAE,cAAc,EAAE,EAAE;IACrD,MAAM,MAAM,GAAG,MAAM,cAAc,EAAuC,CAAA;IAC1E,OAAO,EAAE,GAAG,MAAM,EAAE,YAAY,EAAE,gBAAgB,EAAE,CAAA;AACtD,CAAC,CAAC,CAAA;AAEF,0EAA0E;AAC1E,4EAA4E;AAC5E,4EAA4E;AAC5E,yEAAyE;AACzE,4EAA4E;AAC5E,iEAAiE;AACjE,MAAM,EAAE,iBAAiB,EAAE,oBAAoB,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IACpE,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC1B,oBAAoB,EAAE,EAAE,OAAO,EAAE,IAA6C,EAAE;CACjF,CAAC,CAAC,CAAA;AACH,EAAE,CAAC,IAAI,CAAC,SAAS,EAAE,KAAK,EAAE,cAAc,EAAE,EAAE;IAC1C,MAAM,MAAM,GAAG,MAAM,cAAc,EAA4B,CAAA;IAC/D,oBAAoB,CAAC,OAAO,GAAG,MAAM,CAAC,aAAwD,CAAA;IAC9F,iBAAiB,CAAC,kBAAkB,CAAC,MAAM,CAAC,aAAa,CAAC,CAAA;IAC1D,OAAO,EAAE,GAAG,MAAM,EAAE,aAAa,EAAE,iBAAiB,EAAE,CAAA;AACxD,CAAC,CAAC,CAAA;AAEF,OAAO,EAAE,gBAAgB,EAAE,MAAM,2BAA2B,CAAA;AAC5D,OAAO,EAAE,yBAAyB,EAAE,MAAM,qBAAqB,CAAA;AAE/D,IAAI,OAAe,CAAA;AACnB,IAAI,WAAmB,CAAA;AACvB,IAAI,iBAAqC,CAAA;AAEzC,MAAM,iBAAiB,GAAG,+BAA+B,CAAA;AAEzD,SAAS,WAAW,CAAC,IAAY;IAC/B,OAAO,IAAI,CAAC,IAAI,EAAE,SAAS,EAAE,UAAU,CAAC,CAAA;AAC1C,CAAC;AASD,SAAS,iBAAiB,CAAC,IAAY;IACrC,OAAO,IAAI,CAAC,KAAK,CAAC,YAAY,CAAC,WAAW,CAAC,IAAI,CAAC,EAAE,OAAO,CAAC,CAAiB,CAAA;AAC7E,CAAC;AAED,SAAS,iBAAiB;IACxB,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;AACpE,CAAC;AAED,UAAU,CAAC,GAAG,EAAE;IACd,OAAO,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,gCAAgC,CAAC,CAAC,CAAA;IACvE,WAAW,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,oCAAoC,CAAC,CAAC,CAAA;IAC/E,iBAAiB,GAAG,OAAO,CAAC,GAAG,CAAC,yBAAyB,CAAC,CAAA;IAC1D,OAAO,CAAC,GAAG,CAAC,yBAAyB,CAAC,GAAG,WAAW,CAAA;IACpD,gBAAgB,CAAC,SAAS,EAAE,CAAA;IAC5B,uEAAuE;IACvE,uEAAuE;IACvE,iBAAiB,CAAC,SAAS,EAAE,CAAA;AAC/B,CAAC,CAAC,CAAA;AAEF,SAAS,CAAC,GAAG,EAAE;IACb,MAAM,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAA;IACjD,MAAM,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAA;IACrD,IAAI,iBAAiB,KAAK,SAAS;QAAE,OAAO,CAAC,GAAG,CAAC,yBAAyB,CAAC,GAAG,iBAAiB,CAAA;;QAC1F,OAAO,OAAO,CAAC,GAAG,CAAC,yBAAyB,CAAC,CAAA;IAClD,EAAE,CAAC,eAAe,EAAE,CAAA;AACtB,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,kEAAkE,EAAE,GAAG,EAAE;IAChF,EAAE,CAAC,8GAA8G,EAAE,GAAG,EAAE;QACtH,gBAAgB,CAAC,eAAe,CAAC,GAAG,iBAAiB,IAAI,CAAC,CAAA;QAC1D,iBAAiB,EAAE,CAAA;QAEnB,MAAM,MAAM,GAAG,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAE5C,MAAM,GAAG,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACtC,MAAM,KAAK,GAAG,GAAG,CAAC,UAAU,CAAC,wBAAwB,CAAC,CAAA;QACtD,MAAM,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;QAC9C,MAAM,CAAC,KAAK,EAAE,GAAG,EAAE,iBAAiB,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACpD,MAAM,CAAC,KAAK,EAAE,GAAG,EAAE,uBAAuB,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;QACzD,qEAAqE;QACrE,4BAA4B;QAC5B,MAAM,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC,aAAa,EAAE,CAAA;QACnC,+CAA+C;QAC/C,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,UAAU,CAAC,CAAC,aAAa,EAAE,CAAA;QAEjD,MAAM,MAAM,GAAG,MAAM,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,QAAQ,CAAC,CAAA;QACxE,MAAM,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;IACnD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wFAAwF,EAAE,GAAG,EAAE;QAChG,gBAAgB,CAAC,kBAAkB,CAAC,GAAG,EAAE;YACvC,MAAM,IAAI,KAAK,CAAC,WAAW,CAAC,CAAA;QAC9B,CAAC,CAAC,CAAA;QACF,iBAAiB,EAAE,CAAA;QAEnB,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAE7B,MAAM,GAAG,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACtC,MAAM,KAAK,GAAG,GAAG,CAAC,UAAU,CAAC,wBAAwB,CAAC,CAAA;QACtD,MAAM,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAA;QACxD,MAAM,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAA;IAC1D,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iIAAiI,EAAE,GAAG,EAAE;QACzI,wEAAwE;QACxE,wEAAwE;QACxE,uEAAuE;QACvE,qEAAqE;QACrE,MAAM,YAAY,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,2BAA2B,CAAC,EAAE;YACzE,IAAI,EAAE,WAAW;YACjB,MAAM,EAAE,IAAI;YACZ,MAAM,EAAE,SAAS;SAClB,CAAC,CAAA;QACF,gBAAgB,CAAC,kBAAkB,CAAC,GAAG,EAAE;YACvC,MAAM,YAAY,CAAA;QACpB,CAAC,CAAC,CAAA;QACF,iBAAiB,EAAE,CAAA;QAEnB,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAE7B,MAAM,GAAG,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACtC,MAAM,KAAK,GAAG,GAAG,CAAC,UAAU,CAAC,wBAAwB,CAAC,CAAA;QACtD,MAAM,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAA;QAExD,qEAAqE;QACrE,gEAAgE;QAChE,0DAA0D;QAC1D,MAAM,CAAC,gBAAgB,CAAC,CAAC,oBAAoB,CAC3C,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,EAClB,CAAC,gBAAgB,CAAC,EAClB,MAAM,CAAC,gBAAgB,CAAC,EAAE,OAAO,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,CACzD,CAAA;QACD,MAAM,CAAC,EAAE,AAAD,EAAG,OAAO,CAAC,GAAG,gBAAgB,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAA4C,CAAA;QAC/F,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;QAC1C,wEAAwE;QACxE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,mBAAmB,CAAC,IAAI,CAAC,CAAA;IACnD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0FAA0F,EAAE,GAAG,EAAE;QAClG,gBAAgB,CAAC,eAAe,CAAC,GAAG,iBAAiB,IAAI,CAAC,CAAA;QAC1D,iBAAiB,EAAE,CAAA;QAEnB,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAE7B,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAC1B,YAAY,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,eAAe,CAAC,EAAE,OAAO,CAAC,CACjD,CAAA;QACjB,MAAM,CAAC,SAAS,CAAC,UAAU,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC5D,MAAM,CAAC,SAAS,CAAC,UAAU,CAAC,UAAU,EAAE,GAAG,EAAE,uBAAuB,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;QACnF,MAAM,CAAC,SAAS,CAAC,UAAU,CAAC,UAAU,EAAE,GAAG,EAAE,iBAAiB,CAAC,CAAC,aAAa,EAAE,CAAA;IACjF,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,kEAAkE,EAAE,GAAG,EAAE;IAChF,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACpE,gBAAgB,CAAC,eAAe,CAAC,GAAG,iBAAiB,IAAI,CAAC,CAAA;QAC1D,iBAAiB,EAAE,CAAA;QAEnB,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAC7B,MAAM,MAAM,GAAG,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAE5C,MAAM,GAAG,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACtC,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,wBAAwB,CAAC,CAAC,CAAA;QAEvE,MAAM,MAAM,GAAG,MAAM,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,QAAQ,CAAC,CAAA;QACxE,MAAM,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAA;IACrD,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,+EAA+E,EAAE,GAAG,EAAE;IAC7F,EAAE,CAAC,4FAA4F,EAAE,GAAG,EAAE;QACpG,gBAAgB,CAAC,eAAe,CAAC,GAAG,iBAAiB,IAAI,CAAC,CAAA;QAC1D,iBAAiB,EAAE,CAAA;QACnB,aAAa,CACX,WAAW,CAAC,OAAO,CAAC,EACpB,IAAI,CAAC,SAAS,CACZ;YACE,UAAU,EAAE;gBACV,UAAU,EAAE;oBACV,OAAO,EAAE,KAAK;oBACd,IAAI,EAAE,CAAC,IAAI,EAAE,wBAAwB,CAAC;oBACtC,GAAG,EAAE,EAAE,uBAAuB,EAAE,OAAO,EAAE;iBAC1C;aACF;SACF,EACD,IAAI,EACJ,CAAC,CACF,CACF,CAAA;QAED,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAE7B,MAAM,GAAG,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACtC,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,UAAU,CAAC,CAAC,aAAa,EAAE,CAAA;QACjD,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,wBAAwB,CAAC,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;QACjF,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,wBAAwB,CAAC,CAAC,CAAA;IACzE,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0GAA0G,EAAE,GAAG,EAAE;QAClH,gBAAgB,CAAC,eAAe,CAAC,GAAG,iBAAiB,IAAI,CAAC,CAAA;QAC1D,iBAAiB,EAAE,CAAA;QACnB,MAAM,YAAY,GAAG,EAAE,OAAO,EAAE,iBAAiB,EAAE,IAAI,EAAE,CAAC,QAAQ,CAAC,EAAE,CAAA;QACrE,aAAa,CACX,WAAW,CAAC,OAAO,CAAC,EACpB,IAAI,CAAC,SAAS,CAAC,EAAE,UAAU,EAAE,EAAE,UAAU,EAAE,YAAY,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CACtE,CAAA;QAED,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAE7B,MAAM,GAAG,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACtC,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,UAAU,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAA;QACvD,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,wBAAwB,CAAC,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;IACnF,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0IAA0I,EAAE,GAAG,EAAE;QAClJ,gBAAgB,CAAC,eAAe,CAAC,GAAG,iBAAiB,IAAI,CAAC,CAAA;QAC1D,iBAAiB,EAAE,CAAA;QACnB,oEAAoE;QACpE,+DAA+D;QAC/D,wEAAwE;QACxE,sEAAsE;QACtE,8BAA8B;QAC9B,MAAM,aAAa,GAAG;YACpB,OAAO,EAAE,KAAK;YACd,IAAI,EAAE,CAAC,IAAI,EAAE,wBAAwB,CAAC;YACtC,GAAG,EAAE,EAAE,uBAAuB,EAAE,OAAO,EAAE;YACzC,WAAW,EAAE,8DAA8D;SAC5E,CAAA;QACD,aAAa,CACX,WAAW,CAAC,OAAO,CAAC,EACpB,IAAI,CAAC,SAAS,CAAC,EAAE,UAAU,EAAE,EAAE,UAAU,EAAE,aAAa,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CACvE,CAAA;QAED,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAE7B,MAAM,GAAG,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACtC,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,UAAU,CAAC,CAAC,OAAO,CAAC,aAAa,CAAC,CAAA;QACxD,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,wBAAwB,CAAC,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;IACnF,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0KAA0K,EAAE,GAAG,EAAE;QAClL,gBAAgB,CAAC,eAAe,CAAC,GAAG,iBAAiB,IAAI,CAAC,CAAA;QAC1D,iBAAiB,EAAE,CAAA;QACnB,qEAAqE;QACrE,wEAAwE;QACxE,2CAA2C;QAC3C,MAAM,aAAa,GAAG;YACpB,OAAO,EAAE,KAAK;YACd,IAAI,EAAE,CAAC,IAAI,EAAE,wBAAwB,CAAC;YACtC,GAAG,EAAE,EAAE,uBAAuB,EAAE,OAAO,EAAE;SAC1C,CAAA;QACD,aAAa,CACX,WAAW,CAAC,OAAO,CAAC,EACpB,IAAI,CAAC,SAAS,CAAC,EAAE,UAAU,EAAE,EAAE,UAAU,EAAE,aAAa,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CACvE,CAAA;QAED,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAE7B,MAAM,GAAG,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACtC,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,UAAU,CAAC,CAAC,OAAO,CAAC,aAAa,CAAC,CAAA;QACxD,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,wBAAwB,CAAC,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;IACnF,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,oGAAoG,EAAE,GAAG,EAAE;QAC5G,gBAAgB,CAAC,eAAe,CAAC,GAAG,iBAAiB,IAAI,CAAC,CAAA;QAC1D,iBAAiB,EAAE,CAAA;QACnB,aAAa,CACX,WAAW,CAAC,OAAO,CAAC,EACpB,IAAI,CAAC,SAAS,CACZ;YACE,UAAU,EAAE;gBACV,UAAU,EAAE;oBACV,OAAO,EAAE,KAAK;oBACd,IAAI,EAAE,CAAC,IAAI,EAAE,wBAAwB,CAAC;oBACtC,GAAG,EAAE,EAAE,uBAAuB,EAAE,OAAO,EAAE;iBAC1C;aACF;SACF,EACD,IAAI,EACJ,CAAC,CACF,CACF,CAAA;QAED,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA,CAAC,uDAAuD;QACrF,MAAM,UAAU,GAAG,YAAY,CAAC,WAAW,CAAC,OAAO,CAAC,EAAE,OAAO,CAAC,CAAA;QAC9D,MAAM,MAAM,GAAG,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA,CAAC,oCAAoC;QACjF,MAAM,WAAW,GAAG,YAAY,CAAC,WAAW,CAAC,OAAO,CAAC,EAAE,OAAO,CAAC,CAAA;QAE/D,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,QAAQ,CAAC,CAAA;QACxE,MAAM,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAA;IACrD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4IAA4I,EAAE,GAAG,EAAE;QACpJ,gBAAgB,CAAC,eAAe,CAAC,GAAG,iBAAiB,IAAI,CAAC,CAAA;QAC1D,iBAAiB,EAAE,CAAA;QACnB,yEAAyE;QACzE,sEAAsE;QACtE,mDAAmD;QACnD,aAAa,CACX,WAAW,CAAC,OAAO,CAAC,EACpB,IAAI,CAAC,SAAS,CACZ;YACE,UAAU,EAAE;gBACV,UAAU,EAAE;oBACV,OAAO,EAAE,KAAK;oBACd,IAAI,EAAE,CAAC,IAAI,EAAE,wBAAwB,CAAC;oBACtC,GAAG,EAAE,EAAE,uBAAuB,EAAE,OAAO,EAAE;iBAC1C;gBACD,wBAAwB,EAAE;oBACxB,OAAO,EAAE,iBAAiB;oBAC1B,GAAG,EAAE,EAAE,uBAAuB,EAAE,OAAO,EAAE,iBAAiB,EAAE,QAAQ,EAAE;iBACvE;aACF;SACF,EACD,IAAI,EACJ,CAAC,CACF,CACF,CAAA;QAED,uEAAuE;QACvE,yEAAyE;QACzE,oCAAoC;QACpC,MAAM,UAAU,GAAG,WAAW,CAAC,OAAO,CAAC,CAAA;QACvC,iBAAiB,CAAC,kBAAkB,CAAC,CAAC,GAAG,IAAe,EAAE,EAAE;YAC1D,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,UAAU,EAAE,CAAC;gBAC3B,MAAM,IAAI,KAAK,CAAC,uCAAuC,CAAC,CAAA;YAC1D,CAAC;YAED,OAAO,oBAAoB,CAAC,OAAQ,CAAC,GAAG,IAAI,CAAC,CAAA;QAC/C,CAAC,CAAC,CAAA;QAEF,MAAM,MAAM,GAAG,gBAAgB,CAAC,EAAE,OAAO,EAAE,CAAC,CAAA;QAE5C,wEAAwE;QACxE,6CAA6C;QAC7C,MAAM,GAAG,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACtC,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC,UAAU,CAAC,CAAC,WAAW,EAAE,CAAA;QAE/C,MAAM,MAAM,GAAG,MAAM,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,QAAQ,CAAC,CAAA;QACxE,MAAM,CAAC,MAAM,EAAE,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC/E,MAAM,CAAC,MAAM,EAAE,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IACpE,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/src/install/agent-pack-installer.d.ts

@@ -24,6 +24,14 @@
  *   - The `skillsmith` MCP server registration (`SKILLSMITH_TOOL_PROFILE=agent`)
  *     for every detected MCP-capable harness (all 7), backup-first,
  *     idempotent, preserve-and-prompt on a foreign entry.
+ *   - AntiGravity (SMI-6275 Wave 5): a workspace-scoped 8th target, handled
+ *     OUTSIDE the 7-harness loop above — see the `antigravity` block near
+ *     the end of `installAgentPack`. Whenever its workspace scope resolves
+ *     (an existing `.agents/` marker auto-detects, or `--scope workspace`
+ *     creates one), writes the skill pack AND `.agents/mcp_config.json`
+ *     (`installAntigravityMcpConfig`, `agent-pack-installer.antigravity-mcp.ts`)
+ *     — MCP config only, no hooks/shim this wave (stated decision, see
+ *     `HOOK_HARNESSES`'s doc comment in `services/agent-pack/types.ts`).
  *
  * Every write funnels through `writeOwnedArtifactFile` (owned files) or the
  * `agent-config-merge.*` helpers (shared config files), both of which record

package/dist/src/install/agent-pack-installer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"agent-pack-installer.d.ts","sourceRoot":"","sources":["../../../src/install/agent-pack-installer.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAkCG;AAWH,OAAO,EAGL,iBAAiB,EAGlB,MAAM,qBAAqB,CAAA;AAU5B,OAAO,EAEL,KAAK,mBAAmB,EACxB,KAAK,kBAAkB,EAExB,MAAM,iCAAiC,CAAA;AA4FxC;;;GAGG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,GAAE,mBAAwB,GAAG,kBAAkB,CAqInF;AAGD,OAAO,EAAE,iBAAiB,EAAE,CAAA"}
+{"version":3,"file":"agent-pack-installer.d.ts","sourceRoot":"","sources":["../../../src/install/agent-pack-installer.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA0CG;AAYH,OAAO,EAGL,iBAAiB,EAGlB,MAAM,qBAAqB,CAAA;AAY5B,OAAO,EAEL,KAAK,mBAAmB,EACxB,KAAK,kBAAkB,EAExB,MAAM,iCAAiC,CAAA;AA4FxC;;;GAGG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,GAAE,mBAAwB,GAAG,kBAAkB,CA0NnF;AAGD,OAAO,EAAE,iBAAiB,EAAE,CAAA"}