@rainbow-robotics/muscat-plugin-kit 1.1.5 → 1.3.0

package/cli/src/mcp.mjs

@@ -13,7 +13,9 @@
 //
 //   Plugin SDK docs + Python backend:
 //   - search_rb_sdk          search the plugin-author README (useRb(), rb.*, manifest, runtimes).
-//   - list_rb_sdk            classes/functions of the installed rainbow.rb_sdk.
+//   - list_rb_sdk            classes/functions of the installed rainbow.rb_sdk
+//                            (lang: "python" | "cpp" — cpp reads the headers of the
+//                            rainbow-rb-sdk-dev apt package out of the cpp dev image).
 //   - get_rb_sdk_api         signature + docstring for an rb_sdk class/function.
 //
 // Add to an MCP client config, e.g. Claude Code's .mcp.json:
@@ -22,12 +24,23 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { z } from "zod";
 import ts from "typescript";
-import { existsSync, readdirSync, readFileSync, statSync } from "node:fs";
+import {
+  appendFileSync,
+  existsSync,
+  mkdirSync,
+  readdirSync,
+  readFileSync,
+  renameSync,
+  statSync,
+  writeFileSync,
+} from "node:fs";
 import { spawnSync } from "node:child_process";
+import { homedir } from "node:os";
 import { join, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 import { kitRoot, kitVersion, findPluginRoot, venvPython } from "./util.mjs";
 import { SHIPPED_STORIES_SRC, SHIPPED_DESIGN_TOKENS } from "./storybook.mjs";
+import { CPP_DEV_IMAGE } from "./cpp.mjs";
 
 const TEMPLATE_README = join(kitRoot, "cli/templates/common/README.md");
 // Design tokens: prefer the shipped copy, fall back to the monorepo source.
@@ -39,14 +52,10 @@ const DESIGN_TOKENS_CSS = existsSync(SHIPPED_DESIGN_TOKENS)
 /* react.d.ts introspection — component/hook list + prop types                */
 /* -------------------------------------------------------------------------- */
 
-let typesCache = null;
-
-// Parse plugin-ui/react's dist/react.d.ts. It's self-contained (rb-components
-// types are inlined at build — see plugin-ui/build-dist.mjs), so this one file
-// is enough even in an external install.
-async function loadReactTypes() {
-  if (typesCache) return typesCache;
-
+// plugin-ui/react's dist/react.d.ts. It's self-contained (rb-components types
+// are inlined at build — see plugin-ui/build-dist.mjs), so this one file is
+// enough even in an external install.
+async function reactDtsPath() {
   const entryUrl = await import.meta.resolve("@rainbow-robotics/plugin-ui/react");
   const dtsPath = fileURLToPath(entryUrl).replace(/\.js$/, ".d.ts");
   if (!existsSync(dtsPath)) {
@@ -54,19 +63,23 @@ async function loadReactTypes() {
       `plugin-ui type declarations not found at ${dtsPath} — reinstall or rebuild @rainbow-robotics/plugin-ui.`,
     );
   }
+  return dtsPath;
+}
 
+function loadReactTypes(dtsPath) {
+  // Without moduleResolution the `import ... from "react"` inside react.d.ts
+  // never resolves and every React-derived type collapses to `any`.
   const program = ts.createProgram([dtsPath], {
     target: ts.ScriptTarget.ES2022,
     module: ts.ModuleKind.ESNext,
+    moduleResolution: ts.ModuleResolutionKind.Bundler,
     skipLibCheck: true,
   });
   const sourceFile = program.getSourceFile(dtsPath);
   const checker = program.getTypeChecker();
   const moduleSymbol = sourceFile && checker.getSymbolAtLocation(sourceFile);
   const exports = moduleSymbol ? checker.getExportsOfModule(moduleSymbol) : [];
-
-  typesCache = { checker, sourceFile, exports };
-  return typesCache;
+  return { checker, sourceFile, exports };
 }
 
 // The barrel's trailing `export { A, B, ... }` produces Alias symbols whose
@@ -82,6 +95,124 @@ function exportKind(checker, symbol) {
   return resolved.flags & ts.SymbolFlags.Value ? "value" : "type";
 }
 
+// typeToString alone often prints an opaque alias (`ForwardRefExoticComponent<PropsType$s>`).
+// The first call signature works for plain components and forwardRef alike; its
+// first parameter is the props object.
+function ownProps(checker, type) {
+  const [signature] = type.getCallSignatures?.() ?? [];
+  const param = signature?.getParameters?.()[0];
+  if (!param) return [];
+  const declaration = param.valueDeclaration ?? param.declarations?.[0];
+  if (!declaration) return [];
+  const propsType = checker.getTypeOfSymbolAtLocation(param, declaration);
+
+  return checker
+    .getPropertiesOfType(propsType)
+    // Without this, ~220 inherited React DOM attributes bury the component's own API.
+    .filter((prop) =>
+      (prop.declarations ?? []).some(
+        (d) => !d.getSourceFile().fileName.includes("node_modules"),
+      ),
+    )
+    .map((prop) => {
+      const d = prop.valueDeclaration ?? prop.declarations[0];
+      const optional = prop.flags & ts.SymbolFlags.Optional ? "?" : "";
+      return `${prop.getName()}${optional}: ${checker.typeToString(
+        checker.getTypeOfSymbolAtLocation(prop, d),
+      )}`;
+    });
+}
+
+// Kept out of the tool handler so every export can be precomputed and cached.
+function propsText(checker, sourceFile, name, symbol) {
+  const resolved = resolveExport(checker, symbol);
+  const declaration = resolved.valueDeclaration ?? resolved.declarations?.[0] ?? sourceFile;
+  const type = checker.getTypeOfSymbolAtLocation(resolved, declaration);
+  const signature = checker.typeToString(
+    type,
+    undefined,
+    ts.TypeFormatFlags.NoTruncation | ts.TypeFormatFlags.MultilineObjectLiterals,
+  );
+
+  const props = ownProps(checker, type);
+  if (props.length) {
+    // Keep the signature line: RBSlider's value/min/max come from Radix and the
+    // own-props filter drops them.
+    return `# ${name} — own props\n${props.sort().join("\n")}\n\nfull type: ${signature}`;
+  }
+  // No own props means either a Radix wrapper or unresolved React types. The two
+  // look identical, so name the second case explicitly.
+  if (/<any>/.test(signature)) {
+    return (
+      `# ${name} — prop types could not be resolved\n${signature}\n\n` +
+      "React's types didn't resolve, so this collapsed to `any`. Install `@types/react` " +
+      "in this project (`npm i -D @types/react`) and retry for the real props."
+    );
+  }
+  return (
+    `# ${name} — full type (no plugin-ui-specific props; it forwards a third-party ` +
+    `component's props)\n${signature}`
+  );
+}
+
+// Rendering every export up front is what makes the result cacheable: once the
+// text exists no TypeChecker is needed.
+function buildUiIndex(dtsPath) {
+  const { checker, sourceFile, exports } = loadReactTypes(dtsPath);
+  const props = {};
+  for (const symbol of exports) {
+    props[symbol.getName()] = propsText(checker, sourceFile, symbol.getName(), symbol);
+  }
+  return {
+    components: exports
+      .map((sym) => `${sym.getName()} (${exportKind(checker, sym)})`)
+      .sort()
+      .join("\n"),
+    props,
+  };
+}
+
+// ts.createProgram pulls in ~1100 source files (~1.5s) and an MCP server is a fresh
+// process per session. The key covers the .d.ts identity and the kit version.
+const UI_INDEX_CACHE_DIR = join(homedir(), ".cache", "muscat-pk");
+const UI_INDEX_CACHE = join(UI_INDEX_CACHE_DIR, "ui-index.json");
+
+let uiIndexCache = null;
+
+async function loadUiIndex() {
+  if (uiIndexCache) return uiIndexCache;
+  const dtsPath = await reactDtsPath();
+  const { mtimeMs, size } = statSync(dtsPath);
+  const key = `${dtsPath}|${mtimeMs}|${size}|${kitVersion()}`;
+
+  try {
+    const cached = JSON.parse(readFileSync(UI_INDEX_CACHE, "utf8"));
+    // A half-written file still parses as JSON, so check the shape too.
+    if (
+      cached.key === key &&
+      typeof cached.components === "string" &&
+      cached.props &&
+      typeof cached.props === "object"
+    ) {
+      return (uiIndexCache = cached);
+    }
+  } catch {
+    /* absent, unreadable or corrupt — rebuild below */
+  }
+
+  uiIndexCache = { key, ...buildUiIndex(dtsPath) };
+  try {
+    mkdirSync(UI_INDEX_CACHE_DIR, { recursive: true });
+    // rename is atomic, so a concurrent server never reads a half-written file.
+    const tmp = `${UI_INDEX_CACHE}.${process.pid}.tmp`;
+    writeFileSync(tmp, JSON.stringify(uiIndexCache));
+    renameSync(tmp, UI_INDEX_CACHE);
+  } catch {
+    /* read-only fs / sandbox: the cache is an optimisation, never a requirement */
+  }
+  return uiIndexCache;
+}
+
 /* -------------------------------------------------------------------------- */
 /* Storybook story source lookup                                              */
 /* -------------------------------------------------------------------------- */
@@ -111,10 +242,28 @@ function findStorySource(name) {
 /* -------------------------------------------------------------------------- */
 
 // Split the README on headings and keyword-score each section.
+// Headings inside fenced code blocks (```…```) don't count — a column-0
+// Python comment would otherwise start a bogus section.
+function splitSections(text) {
+  const sections = [];
+  let current = [];
+  let inFence = false;
+  for (const line of text.split("\n")) {
+    if (/^\s*```/.test(line)) inFence = !inFence;
+    if (!inFence && /^#{1,4} /.test(line) && current.length) {
+      sections.push(current.join("\n"));
+      current = [];
+    }
+    current.push(line);
+  }
+  if (current.length) sections.push(current.join("\n"));
+  return sections;
+}
+
 function searchReadme(query) {
   if (!existsSync(TEMPLATE_README)) return [];
   const text = readFileSync(TEMPLATE_README, "utf8");
-  const sections = text.split(/\n(?=#{1,4} )/);
+  const sections = splitSections(text);
   const terms = query.toLowerCase().split(/\s+/).filter(Boolean);
 
   return sections
@@ -243,6 +392,147 @@ ${script}
   return { data };
 }
 
+/* -------------------------------------------------------------------------- */
+/* rainbow-rb-sdk-dev (C++) headers — copied out of the cpp dev image           */
+/* -------------------------------------------------------------------------- */
+
+const CPP_INCLUDE_CACHE = join(UI_INDEX_CACHE_DIR, "rb-sdk-include");
+// Headers we don't want to surface: third-party vendored code, pybind glue,
+// flatbuffers-generated schemas, and the industrial comm stacks.
+const CPP_SKIP = /^(vendor|binding|rb_comm_source)\/|_generated\.h$/;
+
+// The apt package is only installed inside the muscat-pk-cpp-dev image, so
+// copy /usr/include/rainbow/rb_sdk out once per image build (cache key =
+// image id) with `docker cp` — no host tar needed.
+function cppIncludeDir() {
+  const insp = spawnSync("docker", ["image", "inspect", "-f", "{{.Id}}", CPP_DEV_IMAGE], { encoding: "utf8" });
+  if (insp.error) return { error: "Docker not found — the C++ SDK headers live in the muscat-pk cpp dev image." };
+  if (insp.status !== 0) {
+    return {
+      error:
+        `Docker image ${CPP_DEV_IMAGE} not built yet. Run \`muscat-pk dev:back\` (or \`muscat-pk build\`) once ` +
+        "in a cpp / rclcpp plugin to build it, then retry.",
+    };
+  }
+  const dir = join(CPP_INCLUDE_CACHE, insp.stdout.trim().replace(/^sha256:/, "").slice(0, 12));
+  const out = join(dir, "rb_sdk");
+  if (existsSync(out)) return { dir: out };
+  mkdirSync(dir, { recursive: true });
+  const ctr = spawnSync("docker", ["create", CPP_DEV_IMAGE], { encoding: "utf8" });
+  if (ctr.status !== 0) return { error: `docker create failed: ${(ctr.stderr || "").trim().slice(0, 300)}` };
+  const id = ctr.stdout.trim();
+  const cp = spawnSync("docker", ["cp", `${id}:/usr/include/rainbow/rb_sdk`, dir], { encoding: "utf8" });
+  spawnSync("docker", ["rm", "-f", id], { encoding: "utf8" });
+  if (cp.status !== 0 || !existsSync(out)) {
+    return { error: `docker cp failed: ${(cp.stderr || "").trim().slice(0, 300)}` };
+  }
+  return { dir: out };
+}
+
+function cppHeaders(root) {
+  const files = [];
+  const walk = (d, rel) => {
+    for (const name of readdirSync(d)) {
+      const r = rel ? `${rel}/${name}` : name;
+      if (CPP_SKIP.test(r)) continue;
+      const full = join(d, name);
+      if (statSync(full).isDirectory()) walk(full, r);
+      else if (/\.hpp$/.test(name)) files.push(r);
+    }
+  };
+  walk(root, "");
+  return files.sort();
+}
+
+// "class Foo {" / "class Foo : public Bar {" — not forward declarations.
+const CLASS_RE = /^\s*(?:class|struct)\s+(\w+)\b(?![^\n]*;\s*$)/;
+
+function cppClassesIn(text) {
+  return [...new Set(text.split("\n").map((l) => l.match(CLASS_RE)?.[1]).filter(Boolean))];
+}
+
+// Return the class body (brace-matched) plus the file's leading comment block,
+// which in these headers usually carries a full usage example.
+function cppExtract(text, className, method) {
+  const lines = text.split("\n");
+  const start = lines.findIndex((l) => CLASS_RE.test(l) && l.match(CLASS_RE)[1] === className);
+  if (start < 0) return null;
+  let depth = 0, end = start, seen = false;
+  for (let i = start; i < lines.length; i++) {
+    for (const ch of lines[i]) {
+      if (ch === "{") { depth++; seen = true; }
+      else if (ch === "}") depth--;
+    }
+    if (seen && depth === 0) { end = i; break; }
+  }
+  const body = lines.slice(start, end + 1);
+  if (method) {
+    // Declarations sit at class-member indent (<= 2 spaces) and the name is
+    // not preceded by `.`/`->`/`::` — that skips call sites inside bodies.
+    const re = new RegExp(`^ {0,2}[^(]*(?<![\\w.>:])${method.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\s*\\(`);
+    const hits = [];
+    for (let i = 0; i < body.length; i++) {
+      if (!re.test(body[i]) || /^\s*\/\//.test(body[i])) continue;
+      let a = i;
+      while (a > 0 && /^\s*\/\//.test(body[a - 1])) a--;
+      let b = i;
+      while (b < body.length - 1 && !/[{;]/.test(body[b])) b++;
+      hits.push(body.slice(a, b + 1).join("\n"));
+    }
+    return hits.length ? hits.join("\n\n") : null;
+  }
+  const head = [];
+  for (const l of lines) {
+    if (/^\s*\/\//.test(l)) head.push(l);
+    else if (l.trim() === "" || /^#pragma/.test(l)) continue;
+    else break;
+  }
+  return (head.length > 3 ? head.join("\n") + "\n\n" : "") + body.join("\n");
+}
+
+function cppFind(className) {
+  const { dir, error } = cppIncludeDir();
+  if (error) return { error };
+  for (const rel of cppHeaders(dir)) {
+    const text = readFileSync(join(dir, rel), "utf8");
+    if (cppClassesIn(text).includes(className)) return { rel, text };
+  }
+  return { error: `No class "${className}" in the C++ SDK headers. Call list_rb_sdk with lang "cpp" for valid names.` };
+}
+
+/* -------------------------------------------------------------------------- */
+/* client registration — `muscat-pk mcp --install <client>`                    */
+/* -------------------------------------------------------------------------- */
+
+// Claude Code needs no install step: `muscat-pk new` scaffolds a project-level
+// .mcp.json. Codex only reads the global ~/.codex/config.toml, so we append
+// the server block there (idempotent).
+export function installMcp(target) {
+  if (target !== "codex") {
+    throw new Error(
+      `Unknown --install target ${JSON.stringify(target ?? "")} — supported: "codex". ` +
+        "(Claude Code works out of the box via the scaffolded .mcp.json.)",
+    );
+  }
+  const cfgDir = join(homedir(), ".codex");
+  const cfgPath = join(cfgDir, "config.toml");
+  const existing = existsSync(cfgPath) ? readFileSync(cfgPath, "utf8") : "";
+  if (/^\s*\[mcp_servers\.muscat\]/m.test(existing)) {
+    console.log(`Codex already has the muscat MCP server registered (${cfgPath}) — nothing to do.`);
+    return;
+  }
+  // ponytail: Windows Codex spawns without a shell, so it needs the .cmd shim.
+  const command = process.platform === "win32" ? "muscat-pk.cmd" : "muscat-pk";
+  mkdirSync(cfgDir, { recursive: true });
+  const sep = existing.length === 0 ? "" : existing.endsWith("\n") ? "\n" : "\n\n";
+  appendFileSync(
+    cfgPath,
+    `${sep}[mcp_servers.muscat]\ncommand = "${command}"\nargs = ["mcp"]\n`,
+  );
+  console.log(`Registered the muscat MCP server for Codex → ${cfgPath}`);
+  console.log("Restart Codex (or start a new session) to pick it up.");
+}
+
 /* -------------------------------------------------------------------------- */
 /* server                                                                      */
 /* -------------------------------------------------------------------------- */
@@ -261,11 +551,8 @@ export async function runMcp() {
       inputSchema: {},
     },
     async () => {
-      const { checker, exports } = await loadReactTypes();
-      const lines = exports
-        .map((sym) => `${sym.getName()} (${exportKind(checker, sym)})`)
-        .sort();
-      return { content: [{ type: "text", text: lines.join("\n") }] };
+      const { components } = await loadUiIndex();
+      return { content: [{ type: "text", text: components }] };
     },
   );
 
@@ -279,9 +566,8 @@ export async function runMcp() {
       inputSchema: { name: z.string().describe('Exported name, e.g. "RBSlider"') },
     },
     async ({ name }) => {
-      const { checker, sourceFile, exports } = await loadReactTypes();
-      const symbol = exports.find((sym) => sym.getName() === name);
-      if (!symbol) {
+      const { props } = await loadUiIndex();
+      if (!Object.hasOwn(props, name)) {
         return {
           isError: true,
           content: [
@@ -292,15 +578,7 @@ export async function runMcp() {
           ],
         };
       }
-      const resolved = resolveExport(checker, symbol);
-      const declaration = resolved.valueDeclaration ?? resolved.declarations?.[0] ?? sourceFile;
-      const type = checker.getTypeOfSymbolAtLocation(resolved, declaration);
-      const text = checker.typeToString(
-        type,
-        undefined,
-        ts.TypeFormatFlags.NoTruncation | ts.TypeFormatFlags.MultilineObjectLiterals,
-      );
-      return { content: [{ type: "text", text }] };
+      return { content: [{ type: "text", text: props[name] }] };
     },
   );
 
@@ -347,8 +625,13 @@ export async function runMcp() {
       title: "Search the Muscat plugin-author docs",
       description:
         "Search the Muscat plugin-author README for how to use useRb(), rb.pub/sub/action/serve, " +
+        "plugin.router (external endpoints: router.on/subscribe/emit exposed over REST/Socket.IO/TCP/" +
+        "MQTT/WebRTC at common/plugin/{plugin_id}/{path}, PluginRouteError, meta.transport/principal), " +
         "rb.toast/alert/confirm/prompt, useRbFormData/useRbSave (save/restore form data through the " +
-        "host), manifest.json, contribution points, and backend runtimes (Python/C++/ROS2). IMPORTANT: " +
+        "host), manifest.json (including the \"variables\" declaration for the pyfm variable picker — " +
+        "names require the RBP_ prefix), program variables & executor state (ctx.get_variable/" +
+        "set_variable/executor_state/check_stop), contribution points, and backend runtimes " +
+        "(Python/C++/ROS2). IMPORTANT: " +
         "the backend pattern depends on the contribution point — search 'which backend pattern' before " +
         "writing py/main.py. Only a program.panel command uses PluginProgramContext; a toolbar/move.panel " +
         "plugin controls the robot with the SDK (RBManipulateSDK/RBAmrSDK) directly.",
@@ -391,7 +674,10 @@ export async function runMcp() {
       const other = { shadow: [], radius: [], spacing: [] };
       for (const t of vars) {
         const m = t.group?.match(/^color-(bg|fg|border|fill)$/);
-        if (m && t.class) colorGroups[m[1]].push(`${t.class}   (${t.value})`);
+        // Class name only: the value is just an indirection to another CSS var
+        // (var(--color-scale-zinc-200)), and each class is declared twice (light/dark)
+        // with different values — dropping the value lets the Set dedup collapse them.
+        if (m && t.class) colorGroups[m[1]].push(t.class);
         else if (t.name.startsWith("shadow-")) other.shadow.push(`shadow-${t.name.slice(7)}`);
         else if (t.name.startsWith("radius-")) other.radius.push(`rounded-${t.name.slice(7)}`);
         else if (t.name.startsWith("spacing-")) other.spacing.push(t.name);
@@ -454,14 +740,28 @@ export async function runMcp() {
   server.registerTool(
     "list_rb_sdk",
     {
-      title: "List rainbow-rb-sdk (Python) classes and functions",
+      title: "List rainbow-rb-sdk classes and functions (Python or C++)",
       description:
-        "List the public classes and functions of the installed rainbow.rb_sdk Python package — the " +
-        "plugin BACKEND SDK (RBPluginCall, RBManipulateSDK, RBAmrSDK, PluginProgramContext, ...). Call " +
-        "before writing py/main.py. Requires rainbow-rb-sdk to be pip-installed.",
-      inputSchema: {},
+        "List the public classes and functions of the plugin BACKEND SDK. lang \"python\" (default): the " +
+        "installed rainbow.rb_sdk package (RBPluginCall, RBManipulateSDK, RBAmrSDK, PluginProgramContext, " +
+        "...) — call before writing py/main.py; requires rainbow-rb-sdk to be pip-installed. lang \"cpp\": " +
+        "the C++ SDK headers (apt rainbow-rb-sdk-dev: RBManipulateSDK, rb_plugin::PluginProgramContext, " +
+        "...) grouped by header path — call before writing cpp/main.cpp or an rclcpp node; requires the " +
+        "muscat-pk-cpp-dev Docker image (built by `muscat-pk dev:back`). Note: RBPluginCall (UI pub/sub/" +
+        "serve) exists in Python only.",
+      inputSchema: { lang: z.enum(["python", "cpp"]).optional().describe('Default "python"') },
     },
-    async () => {
+    async ({ lang }) => {
+      if (lang === "cpp") {
+        const { dir, error } = cppIncludeDir();
+        if (error) return { isError: true, content: [{ type: "text", text: error }] };
+        const out = [];
+        for (const rel of cppHeaders(dir)) {
+          const classes = cppClassesIn(readFileSync(join(dir, rel), "utf8"));
+          if (classes.length) out.push(`${rel}: ${classes.join(", ")}`);
+        }
+        return { content: [{ type: "text", text: `#include <rainbow/rb_sdk/...>\n\n${out.join("\n")}` }] };
+      }
       const { data, error } = introspectRbSdk(`
 import inspect, json
 mod = importlib.import_module("rainbow.rb_sdk")
@@ -481,19 +781,22 @@ print(json.dumps(out))
   server.registerTool(
     "get_rb_sdk_api",
     {
-      title: "Get a rainbow-rb-sdk class/function signature + docs",
+      title: "Get a rainbow-rb-sdk class/function signature + docs (Python or C++)",
       description:
         "Get the signature(s) and docstring for a rainbow.rb_sdk class or function (e.g. " +
         "'PluginProgramContext', 'RBManipulateSDK'). For a class, lists its public methods with " +
         "signatures. To drill into ONE method of a class (e.g. RBManipulateSDK.call_move_j), pass " +
-        "`method` alongside `name` — do not put both in `name`. Requires rainbow-rb-sdk to be " +
-        "pip-installed.",
+        "`method` alongside `name` — do not put both in `name`. lang \"python\" (default) introspects the " +
+        "pip-installed package. lang \"cpp\" returns the class declaration from the C++ SDK header " +
+        "(rainbow-rb-sdk-dev) with its header comment — usually a full usage example — and the " +
+        "#include path; with `method`, just that method's declaration and comment.",
       inputSchema: {
         name: z.string().describe('Class or function name, e.g. "RBManipulateSDK"'),
         method: z.string().optional().describe('Optional: a specific method on that class, e.g. "call_move_j"'),
+        lang: z.enum(["python", "cpp"]).optional().describe('Default "python"'),
       },
     },
-    async ({ name, method }) => {
+    async ({ name, method, lang }) => {
       // Tolerate a combined "ClassName method_name" in `name` (a common agent
       // mistake) by splitting it ourselves instead of failing validation.
       let className = name;
@@ -503,6 +806,42 @@ print(json.dumps(out))
         if (parts.length === 2) [className, methodName] = parts;
       }
 
+      if (lang === "cpp") {
+        const cls = className.replace(/^.*::/, "");
+        let { rel, text, error } = cppFind(cls);
+        if (error) return { isError: true, content: [{ type: "text", text: error }] };
+        let body = cppExtract(text, cls, methodName);
+        let owner = cls;
+        if (!body && methodName) {
+          // Like the Python path: most calls live on sub-SDK members
+          // (RBManipulateMoveSDK move; -> sdk.move.call_move_j) or on the base
+          // class — look there before giving up.
+          const decl = cppExtract(text, cls);
+          const cands = [
+            ...[...decl.matchAll(/^\s*(\w+SDK)\s+(\w+);/gm)].map((m) => [m[1], `${cls}.${m[2]}`]),
+            ...[...decl.matchAll(/:\s*public\s+(\w+)/g)].map((m) => [m[1], `${cls} (base ${m[1]})`]),
+          ];
+          for (const [c, label] of cands) {
+            const f = cppFind(c);
+            if (f.error) continue;
+            const b = cppExtract(f.text, c, methodName);
+            if (b) { ({ rel, text } = f); body = b; owner = label; break; }
+          }
+        }
+        if (!body) {
+          return {
+            isError: true,
+            content: [{ type: "text", text: `No method "${methodName}" declared in class "${cls}" (${rel}) or its sub-SDK members / base class. Call get_rb_sdk_api with just \`name\` to see the class.` }],
+          };
+        }
+        const ns = text.match(/^namespace\s+(\w+)\s*\{/m)?.[1];
+        const header =
+          `// #include <rainbow/rb_sdk/${rel}>${ns ? `   (namespace ${ns})` : ""}\n` +
+          `// link: find_package(rb_sdk) + rb_sdk::rb_sdk\n` +
+          (methodName ? `// ${owner}.${methodName}\n` : "") + "\n";
+        return { content: [{ type: "text", text: header + body.slice(0, 60_000) }] };
+      }
+
       const { data, error } = introspectRbSdk(`
 import inspect, json
 mod = importlib.import_module("rainbow.rb_sdk")

package/cli/src/new.mjs

@@ -65,6 +65,9 @@ function printNewSuccess({ targetName, type, pm }) {
   log.info(`  ${color.cyan("muscat-pk dev")}      ${color.dim("# preview in mini-Muscat + back end")}`);
   log.info(`  ${color.cyan("muscat-pk build")}    ${color.dim("# build + distributable ZIP")}`);
   log.info("");
+  log.info(color.dim("AI assistant? Claude Code reads the scaffolded .mcp.json;"));
+  log.info(color.dim(`Codex users run ${color.cyan("muscat-pk mcp --install codex")} once.`));
+  log.info("");
 }
 
 // Build the manifest runtime object for a given type choice + plugin slug.

package/cli/src/ports.mjs

@@ -28,9 +28,9 @@ function reservedPortReason(port) {
   return `${range.name} (${range.from}-${range.to})`;
 }
 
-function isPluginManagedPort(port) {
-  return reservedPortReason(port) || (port >= PLUGIN_PORT_MIN && port <= PLUGIN_PORT_MAX);
-}
+// function isPluginManagedPort(port) {
+//   return reservedPortReason(port) || (port >= PLUGIN_PORT_MIN && port <= PLUGIN_PORT_MAX);
+// }
 
 function isInt(value) {
   return Number.isInteger(value);

package/cli/src/ros2.mjs

@@ -37,9 +37,11 @@ export async function ensureImage(image = ROS2_DEV_IMAGE, { platform = null } =
   const dockerfile = await resolveDockerfile("Dockerfile.dev");
   log.step(`docker build -t ${image}${platform ? ` (${platform})` : ""} (first run is slow, then cached)…`);
   const platformFlags = platform ? ["--platform", platform] : [];
+  const sdkVer = process.env.MUSCAT_PK_RB_SDK_VERSION;
+  const buildArgs = sdkVer ? ["--build-arg", `RB_SDK_VERSION=${sdkVer}`] : [];
   const r = spawnSync(
     "docker",
-    ["build", ...platformFlags, "-t", image, "-f", dockerfile, dirname(dockerfile)],
+    ["build", ...platformFlags, ...buildArgs, "-t", image, "-f", dockerfile, dirname(dockerfile)],
     { stdio: "inherit" },
   );
   if (r.status !== 0) throw new Error("docker build failed.");