@rainbow-robotics/muscat-plugin-kit 1.3.6-dev.7 → 1.3.6-dev.8

package/cli/src/mcp.mjs

@@ -13,9 +13,7 @@
 //
 //   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
-//                            (lang: "python" | "cpp" — cpp reads the headers of the
-//                            rainbow-rb-sdk-dev apt package out of the cpp dev image).
+//   - list_rb_sdk            classes/functions of the installed rainbow.rb_sdk.
 //   - 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:
@@ -24,26 +22,15 @@ 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 {
-  appendFileSync,
-  existsSync,
-  mkdirSync,
-  readdirSync,
-  readFileSync,
-  renameSync,
-  statSync,
-  writeFileSync,
-} from "node:fs";
+import { appendFileSync, existsSync, mkdirSync, readdirSync, readFileSync, statSync } 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 { PYTHON_DEV_IMAGE } from "./python.mjs";
 import { SHIPPED_STORIES_SRC, SHIPPED_DESIGN_TOKENS } from "./storybook.mjs";
-import { CPP_DEV_IMAGE } from "./cpp.mjs";
 
-const TEMPLATE_READMES = { python: "common/README.md", cpp: "readme-cpp/README.md" };
+const TEMPLATE_README = join(kitRoot, "cli/templates/common/README.md");
 // Design tokens: prefer the shipped copy, fall back to the monorepo source.
 const DESIGN_TOKENS_CSS = existsSync(SHIPPED_DESIGN_TOKENS)
   ? SHIPPED_DESIGN_TOKENS
@@ -53,10 +40,14 @@ const DESIGN_TOKENS_CSS = existsSync(SHIPPED_DESIGN_TOKENS)
 /* react.d.ts introspection — component/hook list + prop types                */
 /* -------------------------------------------------------------------------- */
 
-// 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() {
+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;
+
   const entryUrl = await import.meta.resolve("@rainbow-robotics/plugin-ui/react");
   const dtsPath = fileURLToPath(entryUrl).replace(/\.js$/, ".d.ts");
   if (!existsSync(dtsPath)) {
@@ -64,23 +55,19 @@ async function reactDtsPath() {
       `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) : [];
-  return { checker, sourceFile, exports };
+
+  typesCache = { checker, sourceFile, exports };
+  return typesCache;
 }
 
 // The barrel's trailing `export { A, B, ... }` produces Alias symbols whose
@@ -96,124 +83,6 @@ 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                                              */
 /* -------------------------------------------------------------------------- */
@@ -261,29 +130,10 @@ function splitSections(text) {
   return sections;
 }
 
-// Standalone binaries have no templates/ on disk.
-async function readTemplateReadme(rel) {
-  const onDisk = join(kitRoot, "cli/templates", rel);
-  if (existsSync(onDisk)) return readFileSync(onDisk, "utf8");
-  try {
-    const { TEMPLATES } = await import("./templates-embedded.mjs");
-    return TEMPLATES[rel] ? Buffer.from(TEMPLATES[rel], "base64").toString("utf8") : "";
-  } catch {
-    return "";
-  }
-}
-
-async function searchReadme(query, lang) {
-  const langs = lang ? [lang] : ["python", "cpp"];
-  const seen = new Set();
-  const sections = [];
-  for (const l of langs) {
-    for (const section of splitSections(await readTemplateReadme(TEMPLATE_READMES[l]))) {
-      if (seen.has(section)) continue; // UI/manifest sections are shared by both READMEs
-      seen.add(section);
-      sections.push(langs.length > 1 && l === "cpp" ? `[C++ backend]\n${section}` : section);
-    }
-  }
+function searchReadme(query) {
+  if (!existsSync(TEMPLATE_README)) return [];
+  const text = readFileSync(TEMPLATE_README, "utf8");
+  const sections = splitSections(text);
   const terms = query.toLowerCase().split(/\s+/).filter(Boolean);
 
   return sections
@@ -377,6 +227,9 @@ function pythonCmd() {
 // backend dependency). Returns a clear hint if it isn't importable.
 function introspectRbSdk(script) {
   const py = pythonCmd();
+  if (!py) {
+    return { error: "Python not found on PATH. Install Python 3 to use rb-sdk introspection." };
+  }
   const probe = `
 import json, importlib
 try:
@@ -385,44 +238,10 @@ except Exception as e:
     print(json.dumps({"__notinstalled__": str(e)})); raise SystemExit(0)
 ${script}
 `;
-  if (!py) {
-    return introspectInDocker(probe) ?? { error: "Python not found on PATH. Install Python 3 to use rb-sdk introspection." };
-  }
   const r = spawnSync(py, ["-c", probe], { encoding: "utf8", maxBuffer: 8 << 20 });
   if (r.status !== 0) {
     return { error: `python introspection failed: ${(r.stderr || "").trim().slice(0, 500)}` };
   }
-  const local = parseProbe(r);
-  return local.error && local.notInstalled ? introspectInDocker(probe) ?? local : local;
-}
-
-// No macOS wheels: import the plugin's pinned SDK inside the python dev image (cached).
-function introspectInDocker(probe) {
-  if (spawnSync("docker", ["image", "inspect", PYTHON_DEV_IMAGE], { encoding: "utf8" }).status !== 0) return null;
-  let ver = "";
-  try {
-    const req = readFileSync(join(findPluginRoot(), "requirements.txt"), "utf8");
-    ver = req.match(/^rainbow-rb-sdk==(\S+)/m)?.[1] ?? "";
-  } catch {
-    /* not a python plugin — latest */
-  }
-  // ponytail: "latest" cache is never refreshed; delete ~/.cache/... rb-sdk-py/latest to update.
-  const cache = join(UI_INDEX_CACHE_DIR, "rb-sdk-py", ver || "latest");
-  mkdirSync(cache, { recursive: true });
-  const pkg = ver ? `rainbow-rb-sdk==${ver}` : "rainbow-rb-sdk";
-  const script =
-    `[ -f /c/.ok ] || { pip3 install -q --break-system-packages --target /c '${pkg}' >&2 && touch /c/.ok; } && ` +
-    `PYTHONPATH=/c python3 -c "$PROBE"`;
-  const r = spawnSync("docker", ["run", "--rm", "-v", `${cache}:/c`, "-e", "PROBE", PYTHON_DEV_IMAGE, "bash", "-c", script], {
-    encoding: "utf8",
-    maxBuffer: 8 << 20,
-    env: { ...process.env, PROBE: probe },
-  });
-  if (r.status !== 0) return { error: `python introspection (docker) failed: ${(r.stderr || "").trim().slice(-500)}` };
-  return parseProbe(r);
-}
-
-function parseProbe(r) {
   // Importing rainbow.rb_sdk has side effects that print to stdout (e.g. the
   // Zenoh client logs its connection endpoints), ahead of our own final
   // print(json.dumps(...)) — so parse the last line, not the whole output.
@@ -435,7 +254,6 @@ function parseProbe(r) {
   }
   if (data && data.__notinstalled__) {
     return {
-      notInstalled: true,
       error:
         "rainbow-rb-sdk is not installed in this environment. Install it with `pip install rainbow-rb-sdk` " +
         "(it's already in a plugin's requirements.txt), then retry.",
@@ -444,114 +262,6 @@ function parseProbe(r) {
   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+(?:[A-Z][A-Z0-9_]*_API\s+)?(\w+)\b(?![^\n]*;\s*$)/; // skip export macros (RB_SDK_API)
-
-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>`                    */
 /* -------------------------------------------------------------------------- */
@@ -603,8 +313,11 @@ export async function runMcp() {
       inputSchema: {},
     },
     async () => {
-      const { components } = await loadUiIndex();
-      return { content: [{ type: "text", text: components }] };
+      const { checker, exports } = await loadReactTypes();
+      const lines = exports
+        .map((sym) => `${sym.getName()} (${exportKind(checker, sym)})`)
+        .sort();
+      return { content: [{ type: "text", text: lines.join("\n") }] };
     },
   );
 
@@ -618,8 +331,9 @@ export async function runMcp() {
       inputSchema: { name: z.string().describe('Exported name, e.g. "RBSlider"') },
     },
     async ({ name }) => {
-      const { props } = await loadUiIndex();
-      if (!Object.hasOwn(props, name)) {
+      const { checker, sourceFile, exports } = await loadReactTypes();
+      const symbol = exports.find((sym) => sym.getName() === name);
+      if (!symbol) {
         return {
           isError: true,
           content: [
@@ -630,7 +344,15 @@ export async function runMcp() {
           ],
         };
       }
-      return { content: [{ type: "text", text: props[name] }] };
+      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 }] };
     },
   );
 
@@ -677,29 +399,20 @@ 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 (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), video streaming (MediaMTX: backend publishes RTSP to " +
-        "127.0.0.1:8554, UI plays WebRTC from :8889), ROS 2 topics (plugin namespace, rb.rosPeers to allow " +
-        "an external PC), contribution points, and backend runtimes " +
-        "(Python/C++/ROS2). Covers BOTH backends: the Python README (py/main.py, rainbow.rb_sdk) and the " +
-        "C++ README (cpp/main.cpp or an rclcpp node: rb_plugin::RBPluginCall serve/subscribe/publish/" +
-        "router(), rb_plugin::PluginProgramContext, RBManipulateSDK, CMake rb_sdk::rb_sdk). Pass lang " +
-        "\"python\" or \"cpp\" to match the plugin's manifest runtime (cpp or ros2+rclcpp → \"cpp\"); " +
-        "omit it to search both. IMPORTANT: " +
+        "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 the backend. Only a program.panel command uses PluginProgramContext; a toolbar/move.panel " +
+        "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.",
       inputSchema: {
         query: z.string().describe('Keywords, e.g. "save form data" or "rb.action"'),
-        lang: z.enum(["python", "cpp"]).optional().describe("Backend language; omit to search both"),
       },
     },
-    async ({ query, lang }) => {
-      const sections = await searchReadme(query, lang);
+    async ({ query }) => {
+      const sections = searchReadme(query);
       if (sections.length === 0) {
         return { content: [{ type: "text", text: `No matching section found for "${query}".` }] };
       }
@@ -733,10 +446,7 @@ export async function runMcp() {
       const other = { shadow: [], radius: [], spacing: [] };
       for (const t of vars) {
         const m = t.group?.match(/^color-(bg|fg|border|fill)$/);
-        // 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);
+        if (m && t.class) colorGroups[m[1]].push(`${t.class}   (${t.value})`);
         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);
@@ -799,29 +509,14 @@ export async function runMcp() {
   server.registerTool(
     "list_rb_sdk",
     {
-      title: "List rainbow-rb-sdk classes and functions (Python or C++)",
+      title: "List rainbow-rb-sdk (Python) classes and functions",
       description:
-        "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`). RBPluginCall (UI pub/sub/serve, " +
-        "router) exists in both: Python rainbow.rb_sdk.RBPluginCall, C++ rb_plugin::RBPluginCall " +
-        "(plugin/rb_plugin.hpp).",
-      inputSchema: { lang: z.enum(["python", "cpp"]).optional().describe('Default "python"') },
+        "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: {},
     },
-    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")}` }] };
-      }
+    async () => {
       const { data, error } = introspectRbSdk(`
 import inspect, json
 mod = importlib.import_module("rainbow.rb_sdk")
@@ -841,22 +536,19 @@ print(json.dumps(out))
   server.registerTool(
     "get_rb_sdk_api",
     {
-      title: "Get a rainbow-rb-sdk class/function signature + docs (Python or C++)",
+      title: "Get a rainbow-rb-sdk class/function signature + docs",
       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`. 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.",
+        "`method` alongside `name` — do not put both in `name`. Requires rainbow-rb-sdk to be " +
+        "pip-installed.",
       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, lang }) => {
+    async ({ name, method }) => {
       // Tolerate a combined "ClassName method_name" in `name` (a common agent
       // mistake) by splitting it ourselves instead of failing validation.
       let className = name;
@@ -866,42 +558,6 @@ 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

@@ -17,7 +17,7 @@ import { color, ensureIdeVenv, hasBin, kitVersion, parseArgs, prompt, templatesD
 // Like cpSync(src, dst, {recursive:true}) but reads from the embedded snapshot
 // when the templates dir doesn't exist on disk (compiled binary mode).
 // ponytail: import() is module-cached by the runtime — no manual cache needed.
-export async function copyTemplate(srcDir, dst) {
+async function copyTemplate(srcDir, dst) {
   if (existsSync(templatesDir)) { cpSync(srcDir, dst, { recursive: true }); return; }
   const { TEMPLATES } = await import("./templates-embedded.mjs");
   const prefix = srcDir.slice(templatesDir.length + 1).replace(/\\/g, "/") + "/";
@@ -87,7 +87,6 @@ export function runtimeForChoice(choice, slug) {
 
 export function applyTokens(dir, tokens) {
   for (const entry of readdirSync(dir)) {
-    if (entry === "node_modules" || entry === ".git" || entry === ".venv") continue;
     const full = resolve(dir, entry);
     if (statSync(full).isDirectory()) {
       applyTokens(full, tokens);
@@ -107,12 +106,6 @@ export function applyTokens(dir, tokens) {
 }
 
 // Derive naming tokens from a plugin directory name.
-// One id part (see PLUGIN_ID_RE): lowercase, single "-", starts with a letter.
-function idPart(name) {
-  const part = name.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "");
-  return /^[a-z]/.test(part) ? part : part ? `plugin-${part}` : "plugin";
-}
-
 export function deriveNames(target) {
   const base = basename(target);
   const slug = base.toLowerCase().replace(/[^a-z0-9]+/g, "_");
@@ -122,7 +115,7 @@ export function deriveNames(target) {
     slug,
     pluginName,
     pluginClass: pluginName.replace(/\s+/g, ""),
-    pluginId: `com.example.${idPart(base)}`,
+    pluginId: `com.example.${base.toLowerCase().replace(/[^a-z0-9]+/g, "-")}`,
   };
 }
 
@@ -233,9 +226,6 @@ export async function runNew(args) {
     rmSync(resolve(target, "py"), { recursive: true, force: true });
     rmSync(resolve(target, "requirements.txt"), { force: true });
   }
-  if (runtimeChoice.type === "cpp" || runtimeChoice.rosClient === "rclcpp") {
-    await copyTemplate(resolve(templatesDir, "readme-cpp"), target);
-  }
 
   applyTokens(target, {
     __PLUGIN_ID__: pluginId,
@@ -260,7 +250,7 @@ export async function runNew(args) {
   // gitignore (templates can't ship a literal .gitignore via npm reliably)
   writeFileSync(
     resolve(target, ".gitignore"),
-    ["node_modules", "dist", ".muscat-dev", ".venv", "*.zip", "ros2_ws/install", "ros2_ws/build*", "ros2_ws/log", "cpp/build*", ""].join("\n"),
+    ["node_modules", "dist", ".muscat-dev", ".venv", "*.zip", "ros2_ws/install", "cpp/build", ""].join("\n"),
   );
 
   if (!opts["no-install"]) {