@rubytech/create-realagent 1.0.864 → 1.0.866

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rubytech/create-realagent",
-  "version": "1.0.864",
+  "version": "1.0.866",
   "description": "Install Real Agent — Built for agents. By agents.",
   "bin": {
     "create-realagent": "./dist/index.js"

package/payload/platform/plugins/admin/mcp/dist/__tests__/plugin-read-skill-resolution.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=plugin-read-skill-resolution.test.d.ts.map

package/payload/platform/plugins/admin/mcp/dist/__tests__/plugin-read-skill-resolution.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"plugin-read-skill-resolution.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/plugin-read-skill-resolution.test.ts"],"names":[],"mappings":""}

package/payload/platform/plugins/admin/mcp/dist/__tests__/plugin-read-skill-resolution.test.js

@@ -0,0 +1,67 @@
+// Task 964 — plugin-skill resolution gate.
+//
+// Locks in the contract for `findSkillOwners` (the helper backing the
+// new `skill-find` MCP tool) and the sibling-hint that `plugin-read`
+// appends to its "not found" error when the agent guesses the wrong
+// pluginName for a skill that DOES live somewhere else under
+// `plugins/*/skills/<name>/SKILL.md`.
+//
+// Tests build a temp PLATFORM_ROOT tree, point the helpers at it, and
+// assert exact result shapes — no stdio transport, no SDK harness.
+import { describe, it, expect, beforeEach, afterEach } from "vitest";
+import { mkdtempSync, mkdirSync, writeFileSync, rmSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { findSkillOwners, computePluginReadHint } from "../skill-resolution.js";
+let root;
+beforeEach(() => {
+    root = mkdtempSync(join(tmpdir(), "skill-find-"));
+});
+afterEach(() => {
+    rmSync(root, { recursive: true, force: false });
+});
+function seedSkill(plugin, skill) {
+    const dir = join(root, "plugins", plugin, "skills", skill);
+    mkdirSync(dir, { recursive: true });
+    writeFileSync(join(dir, "SKILL.md"), `# ${skill}\n`);
+}
+describe("findSkillOwners", () => {
+    it("returns a unique owner when exactly one plugin holds the skill", () => {
+        seedSkill("cloudflare", "setup-tunnel");
+        seedSkill("admin", "onboarding");
+        const result = findSkillOwners(root, "setup-tunnel");
+        expect(result).toEqual({
+            status: "unique",
+            candidates: [{ pluginName: "cloudflare", file: "skills/setup-tunnel/SKILL.md" }],
+        });
+    });
+    it("returns every candidate when the skill name is shared across plugins", () => {
+        seedSkill("admin", "shared-name");
+        seedSkill("cloudflare", "shared-name");
+        seedSkill("memory", "unrelated");
+        const result = findSkillOwners(root, "shared-name");
+        expect(result.status).toBe("ambiguous");
+        expect(result.candidates).toHaveLength(2);
+        const names = result.candidates.map(c => c.pluginName).sort();
+        expect(names).toEqual(["admin", "cloudflare"]);
+        expect(result.candidates.every(c => c.file === "skills/shared-name/SKILL.md")).toBe(true);
+    });
+    it("returns not-found when no plugin owns the skill", () => {
+        seedSkill("admin", "onboarding");
+        const result = findSkillOwners(root, "absent-skill");
+        expect(result).toEqual({ status: "not-found", candidates: [] });
+    });
+});
+describe("computePluginReadHint", () => {
+    it("returns the owning plugin name when exactly one sibling owns the skill", () => {
+        seedSkill("cloudflare", "setup-tunnel");
+        const hint = computePluginReadHint(root, "admin", "skills/setup-tunnel/SKILL.md");
+        expect(hint).toBe("cloudflare");
+    });
+    it("returns null when no plugin owns the requested skill", () => {
+        seedSkill("admin", "onboarding");
+        const hint = computePluginReadHint(root, "admin", "skills/imaginary/SKILL.md");
+        expect(hint).toBeNull();
+    });
+});
+//# sourceMappingURL=plugin-read-skill-resolution.test.js.map

package/payload/platform/plugins/admin/mcp/dist/__tests__/plugin-read-skill-resolution.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"plugin-read-skill-resolution.test.js","sourceRoot":"","sources":["../../src/__tests__/plugin-read-skill-resolution.test.ts"],"names":[],"mappings":"AAAA,2CAA2C;AAC3C,EAAE;AACF,sEAAsE;AACtE,qEAAqE;AACrE,oEAAoE;AACpE,6DAA6D;AAC7D,sCAAsC;AACtC,EAAE;AACF,sEAAsE;AACtE,mEAAmE;AACnE,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACrE,OAAO,EAAE,WAAW,EAAE,SAAS,EAAE,aAAa,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AACxE,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AACjC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,eAAe,EAAE,qBAAqB,EAAE,MAAM,wBAAwB,CAAC;AAEhF,IAAI,IAAY,CAAC;AAEjB,UAAU,CAAC,GAAG,EAAE;IACd,IAAI,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,aAAa,CAAC,CAAC,CAAC;AACpD,CAAC,CAAC,CAAC;AAEH,SAAS,CAAC,GAAG,EAAE;IACb,MAAM,CAAC,IAAI,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC,CAAC;AAClD,CAAC,CAAC,CAAC;AAEH,SAAS,SAAS,CAAC,MAAc,EAAE,KAAa;IAC9C,MAAM,GAAG,GAAG,IAAI,CAAC,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,KAAK,CAAC,CAAC;IAC3D,SAAS,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IACpC,aAAa,CAAC,IAAI,CAAC,GAAG,EAAE,UAAU,CAAC,EAAE,KAAK,KAAK,IAAI,CAAC,CAAC;AACvD,CAAC;AAED,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAC/B,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;QACxE,SAAS,CAAC,YAAY,EAAE,cAAc,CAAC,CAAC;QACxC,SAAS,CAAC,OAAO,EAAE,YAAY,CAAC,CAAC;QAEjC,MAAM,MAAM,GAAG,eAAe,CAAC,IAAI,EAAE,cAAc,CAAC,CAAC;QAErD,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;YACrB,MAAM,EAAE,QAAQ;YAChB,UAAU,EAAE,CAAC,EAAE,UAAU,EAAE,YAAY,EAAE,IAAI,EAAE,8BAA8B,EAAE,CAAC;SACjF,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sEAAsE,EAAE,GAAG,EAAE;QAC9E,SAAS,CAAC,OAAO,EAAE,aAAa,CAAC,CAAC;QAClC,SAAS,CAAC,YAAY,EAAE,aAAa,CAAC,CAAC;QACvC,SAAS,CAAC,QAAQ,EAAE,WAAW,CAAC,CAAC;QAEjC,MAAM,MAAM,GAAG,eAAe,CAAC,IAAI,EAAE,aAAa,CAAC,CAAC;QAEpD,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QACxC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC1C,MAAM,KAAK,GAAG,MAAM,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,IAAI,EAAE,CAAC;QAC9D,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,YAAY,CAAC,CAAC,CAAC;QAC/C,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,6BAA6B,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC5F,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QACzD,SAAS,CAAC,OAAO,EAAE,YAAY,CAAC,CAAC;QAEjC,MAAM,MAAM,GAAG,eAAe,CAAC,IAAI,EAAE,cAAc,CAAC,CAAC;QAErD,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,WAAW,EAAE,UAAU,EAAE,EAAE,EAAE,CAAC,CAAC;IAClE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;IACrC,EAAE,CAAC,wEAAwE,EAAE,GAAG,EAAE;QAChF,SAAS,CAAC,YAAY,EAAE,cAAc,CAAC,CAAC;QAExC,MAAM,IAAI,GAAG,qBAAqB,CAAC,IAAI,EAAE,OAAO,EAAE,8BAA8B,CAAC,CAAC;QAElF,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,SAAS,CAAC,OAAO,EAAE,YAAY,CAAC,CAAC;QAEjC,MAAM,IAAI,GAAG,qBAAqB,CAAC,IAAI,EAAE,OAAO,EAAE,2BAA2B,CAAC,CAAC;QAE/E,MAAM,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE,CAAC;IAC1B,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/payload/platform/plugins/admin/mcp/dist/index.js

@@ -19,6 +19,7 @@ import { homedir, hostname as osHostname } from "node:os";
 import QRCode from "qrcode";
 import { getSession, closeDriver } from "./lib/neo4j.js";
 import { getOnboardingState, completeOnboardingStep } from "./lib/onboarding.js";
+import { findSkillOwners, computePluginReadHint } from "./skill-resolution.js";
 const server = new McpServer({
     name: "admin",
     version: "0.1.0",
@@ -1581,14 +1582,20 @@ server.tool("plugin-read", "Read a plugin definition (PLUGIN.md) or one of its r
 }, async ({ pluginName, file }) => {
     try {
         const resolvedFile = file ?? "PLUGIN.md";
-        console.log(`[plugin-read] ${pluginName}/${resolvedFile}`);
         const pluginPath = resolve(PLATFORM_ROOT, "plugins", pluginName, resolvedFile);
         if (!existsSync(pluginPath)) {
+            // Task 964 — when the agent guesses the wrong pluginName for a file
+            // path shaped like `skills/<n>/...`, surface the owning plugin so
+            // the next call is deterministic instead of falling through to Glob.
+            const owner = computePluginReadHint(PLATFORM_ROOT, pluginName, resolvedFile);
+            const hintText = owner ? ` Did you mean pluginName="${owner}"?` : "";
+            console.log(`[plugin-read] ${pluginName}/${resolvedFile} not-found hint=${owner ?? "none"}`);
             return {
-                content: [{ type: "text", text: `Plugin file not found: ${pluginPath}` }],
+                content: [{ type: "text", text: `Plugin file not found: ${pluginPath}.${hintText}` }],
                 isError: true,
             };
         }
+        console.log(`[plugin-read] ${pluginName}/${resolvedFile}`);
         const rawContent = await readFile(pluginPath, "utf-8");
         // Brand-substitute every plugin file before it reaches the agent —
         // PLUGIN.md, skills/<name>/SKILL.md, references/*.md all flow through
@@ -1642,6 +1649,41 @@ server.tool("plugin-read", "Read a plugin definition (PLUGIN.md) or one of its r
         };
     }
 });
+// Task 964 — agent-side `Glob` fallthrough when "set up cloudflare" maps a
+// SKILL.md to the wrong plugin: skill-find walks
+// `${PLATFORM_ROOT}/plugins/*/skills/<skillName>/SKILL.md` once and returns
+// the owner. With one tool call the agent learns the correct plugin name
+// and follows up with a deterministic `plugin-read`.
+server.tool("skill-find", "Find which plugin owns a skill by name. Walks plugins/*/skills/<skillName>/SKILL.md and returns the owning plugin (unique), every candidate (ambiguous), or not-found.", {
+    skillName: z.string().regex(/^[a-z0-9][a-z0-9-]*$/, "skillName must be lowercase kebab-case (a-z, 0-9, -)"),
+}, async ({ skillName }) => {
+    const start = Date.now();
+    const result = findSkillOwners(PLATFORM_ROOT, skillName);
+    const ms = Date.now() - start;
+    console.log(`[skill-find] skillName=${skillName} result=${result.status} candidates=${result.candidates.length} ms=${ms}`);
+    if (result.status === "unique") {
+        const [{ pluginName, file }] = result.candidates;
+        return {
+            content: [{
+                    type: "text",
+                    text: `pluginName="${pluginName}" file="${file}"\nFollow up with: plugin-read pluginName="${pluginName}" file="${file}"`,
+                }],
+        };
+    }
+    if (result.status === "ambiguous") {
+        const lines = result.candidates.map(c => `  pluginName="${c.pluginName}" file="${c.file}"`).join("\n");
+        return {
+            content: [{
+                    type: "text",
+                    text: `Ambiguous — skill "${skillName}" exists in ${result.candidates.length} plugins:\n${lines}`,
+                }],
+        };
+    }
+    return {
+        content: [{ type: "text", text: `not_found — no plugin owns a skill named "${skillName}"` }],
+        isError: true,
+    };
+});
 // Task 940 — `"rendered"` is a sentinel, not a persistence ack. The platform
 // UI's stream-parser intercepts this tool_use upstream of the SDK reply
 // (platform/ui/app/lib/claude-agent/stream-parser.ts:362) and emits a typed