romdevtools 0.21.0 → 0.22.0

package/src/host/chafa-render.js

@@ -24,6 +24,7 @@ let lastSettings = "";
 // I had BLOCK=1, ASCII=2; real BLOCK=0x8, ASCII=0x4000. The bad
 // values silently picked an unrelated tag, which is why "ascii"
 // mode was still rendering Unicode block glyphs.
+/* eslint-disable no-unused-vars -- the full chafa tag enum is kept for reference; not all are used. */
 const TAG_SPACE     = 0x1;
 const TAG_SOLID     = 0x2;
 const TAG_STIPPLE   = 0x4;
@@ -35,6 +36,7 @@ const TAG_BRAILLE   = 0x800;
 const TAG_ASCII     = 0x4000;
 const TAG_SEXTANT   = 0x400000;
 const TAG_OCTANT    = 0x4000000;
+/* eslint-enable no-unused-vars */
 
 const SYMBOL_TAGS = {
   // Pure ASCII glyphs (space + printable 7-bit) — most text-shaped,

package/src/host/dsp-state.js

@@ -102,8 +102,8 @@ export function decodeSnes9xDSP(state) {
       bufLastSamples.push(u >= 0x8000 ? u - 0x10000 : u);
     }
     p += 24;
-    const interpPos = state[p] | (state[p + 1] << 8); p += 2;
-    const brrAddr = state[p] | (state[p + 1] << 8); p += 2;
+    p += 2; // interpPos (decoded field, not surfaced)
+    p += 2; // brrAddr (decoded field, not surfaced)
     const env = state[p] | (state[p + 1] << 8); p += 2;
     const hiddenEnvU = state[p] | (state[p + 1] << 8); p += 2;
     const hiddenEnv = hiddenEnvU >= 0x8000 ? hiddenEnvU - 0x10000 : hiddenEnvU;

package/src/host/gpgx-state.js

@@ -17,7 +17,11 @@ const formatCpuState = (s) => s;
 // entry is 5 × 4 = 20 bytes, so memory_map takes 256 × 20 = 5120 bytes.
 // After that come the fields we want.
 const M68K_BASE = 5120; // start of cpu_idle_t poll
+// M68K_POLL / M68K_CYCLES document the struct layout (consumed implicitly by the
+// next offset) — keep them named even though nothing reads them directly.
+// eslint-disable-next-line no-unused-vars
 const M68K_POLL = M68K_BASE + 0;       // 12 bytes
+// eslint-disable-next-line no-unused-vars
 const M68K_CYCLES = M68K_BASE + 12;    // 12 bytes (cycles + refresh_cycles + cycle_end)
 const M68K_DAR = M68K_BASE + 24;       // uint dar[16] — D0..D7 then A0..A7
 const M68K_PC = M68K_DAR + 64;         // uint pc

package/src/http/routes.js

@@ -18,7 +18,7 @@
 // (the app already mounts localhostHostValidation()).
 
 import { buildToolRegistry, runTool, toolJsonSchema } from "./tool-registry.js";
-import { skillPreamble, skillToolReference, buildSkillDoc } from "./skill-doc.js";
+import { buildSkillDoc } from "./skill-doc.js";
 import { swaggerHtml, swaggerAsset } from "./swagger.js";
 import { observer } from "../observer/bus.js";
 import { log } from "../mcp/log.js";

package/src/mcp/server.js

@@ -140,7 +140,7 @@ async function main() {
     // in verbose mode (log.debug handles that split). This keeps the console
     // quiet in prod while /log stays rich enough to diagnose what an agent did.
     if (req.method === "POST" && req.body) {
-      const { method, id, params } = req.body;
+      const { method, params } = req.body;
       if (method === "tools/call") {
         const argKeys = params?.arguments ? Object.keys(params.arguments) : [];
         const summary = argKeys.map((k) => {

package/src/mcp/state.js

@@ -14,6 +14,24 @@ import { LibretroHost } from "../host/index.js";
 /** @type {Map<string, LibretroHost>} */
 const hosts = new Map();
 
+// What this session last loaded, kept OUTSIDE the host map so it SURVIVES a
+// host eviction (server restart / session reconnect / unload). The host itself
+// is gone in those cases, so the "No ROM loaded" error has nothing to read —
+// this is the breadcrumb that lets the error tell the agent exactly how to
+// recover ("you last loaded <X>; re-run loadMedia to pick back up") instead of
+// a generic wipe. Set by loadMedia on success; never cleared on eviction.
+/** @type {Map<string, {platform?: string, path?: string, fromBase64?: boolean}>} */
+const lastMedia = new Map();
+
+/** Record the media a session last loaded (for recovery hints). @param {string} sessionKey */
+export function rememberLastMedia(sessionKey, info) {
+  lastMedia.set(sessionKey, info);
+}
+/** @param {string} sessionKey */
+export function getLastMedia(sessionKey) {
+  return lastMedia.get(sessionKey) ?? null;
+}
+
 /**
  * @param {string} sessionKey
  * @returns {LibretroHost}
@@ -21,6 +39,24 @@ const hosts = new Map();
 export function getHost(sessionKey) {
   const host = hosts.get(sessionKey);
   if (!host) {
+    // If THIS session loaded media before, the host was evicted (restart /
+    // reconnect / unload) — lead with the exact recovery call instead of the
+    // generic "you're in the wrong session" guidance, which doesn't apply here.
+    const prev = lastMedia.get(sessionKey);
+    if (prev && (prev.path || prev.fromBase64)) {
+      const recall = prev.path
+        ? `loadMedia({ platform: "${prev.platform}", path: "${prev.path}" })`
+        : `loadMedia({ platform: "${prev.platform}", base64: ... })  (your ROM came from base64 — re-supply the bytes)`;
+      throw new Error(
+        "No ROM loaded in this session — the host was evicted (the server restarted, " +
+        "your session reconnected, or the media was unloaded). Emulator state lives in " +
+        "server memory only, so it did not survive. RECOVER by re-running your last load:\n  " +
+        recall +
+        "\nThen replay any boot/navigate steps to get back to where you were. " +
+        "(If instead you expected a DIFFERENT session, you may be sending an inconsistent " +
+        "`x-romdev-session` header — reuse one stable id on every call.)",
+      );
+    }
     throw new Error(
       "No ROM loaded in this session — call loadMedia({path}) first. " +
       "If you DID loadMedia and still see this, your calls are landing in DIFFERENT " +

package/src/mcp/tools/address-to-symbol.js

@@ -18,7 +18,6 @@
 //   - Future: vasm listings, asar symbol tables
 
 import { readFile } from "node:fs/promises";
-import { jsonContent, safeTool } from "../util.js";
 import { parseGnuLdMap, isGnuLdMap } from "../../toolchains/gnu-ld-map.js";
 
 function parseSdldStyle(text) {

package/src/mcp/tools/art-loaders.js

@@ -407,7 +407,7 @@ function aseCelToRgba(cel, colorDepth, palette) {
   return rgba;
 }
 
-async function loadAsepriteSheetImpl({ path: asePath, platform, tile_size = 8, outputDir, slice_strategy = "slices", emit = "raw", emitDefines = false }) {
+async function loadAsepriteSheetImpl({ path: asePath, platform, _tile_size = 8, outputDir, slice_strategy = "slices", emit = "raw", emitDefines = false }) {
   const buf = await readFile(asePath);
   const ase = new Aseprite(buf, path.basename(asePath));
   ase.parse();

package/src/mcp/tools/cart-parts.js

@@ -131,7 +131,6 @@ function extractNes(data) {
  */
 function extractSnes(data) {
   const copierOff = (data.length % 0x8000 === 0x200) ? 0x200 : 0;
-  const loMapper = data[copierOff + 0x7FC0 + 0x15];
   const hiMapper = data[copierOff + 0xFFC0 + 0x15];
   const isLo = !(hiMapper === 0x21 || hiMapper === 0x31);
   const internalHeaderBase = copierOff + (isLo ? 0x7FC0 : 0xFFC0);

package/src/mcp/tools/classify-region.js

@@ -51,7 +51,7 @@ function zeroRatio(bytes) {
  * @param {boolean} [opts.bigEndian] platform endianness (for pointer-table guess)
  * @returns {{ looksLike: string, printableRatio:number, entropy:number, zeroRatio:number, longestAsciiRun:number, asciiPreview:string|null, confidence:string, note:string }}
  */
-export function classifyBytes(bytes, opts = {}) {
+export function classifyBytes(bytes, _opts = {}) {
   const pr = printableRatio(bytes);
   const ent = entropy(bytes);
   const zr = zeroRatio(bytes);

package/src/mcp/tools/diff-roms.js

@@ -49,7 +49,7 @@ function regionMapForRom(platform, data) {
  * live anywhere the developer wants. We tag the header so diffs there
  * are obvious.
  */
-function regionMapSms(data, platform) {
+function regionMapSms(data, _platform) {
   const regions = [];
   // Header lives at $7FF0 IF the file is at least 32 KB. Some homebrew
   // skips the header entirely (just runs from $0000) — flag with kind

package/src/mcp/tools/disasm-rebuild.js

@@ -135,7 +135,7 @@ const PLANNERS = {
   // a one-call build() rebuild. planRegions strips the 2-byte load address
   // (fileOffset 2); we re-emit it via a synthesized LOADADDR segment + a custom
   // 2-area .cfg (LOADADDR then the body). PROVEN byte-identical via build().
-  c64(data, regions) {
+  c64(data, _regions) {
     const loadAddr = data[0] | (data[1] << 8);
     const bodyLen = data.length - 2;
     const loadaddrSrc =

package/src/mcp/tools/disasm.js

@@ -208,7 +208,6 @@ function nesVectors(data) {
  */
 function snesVectors(data) {
   const copierOff = (data.length % 0x8000 === 0x200) ? 0x200 : 0;
-  const loMapper = data[copierOff + 0x7FC0 + 0x15];
   const hiMapper = data[copierOff + 0xFFC0 + 0x15];
   const isLo = !(hiMapper === 0x21 || hiMapper === 0x31);
   const headerBase = copierOff + (isLo ? 0x7FC0 : 0xFFC0);
@@ -513,7 +512,7 @@ export function mapGenesisAddress(data, cpuAddr, length) {
  *   48 KB: $4000-$FFFF (rare)
  *   144 KB SuperGame: bank-switched at $8000-$BFFF + fixed at $C000
  */
-export function mapAtari7800Address(data, cpuAddr, length, bank = 0) {
+export function mapAtari7800Address(data, cpuAddr, length, _bank = 0) {
   // Detect header. "ATARI7800" magic at offset 1.
   const hasHeader =
     data.length > 128 &&
@@ -557,7 +556,7 @@ export function mapAtari7800Address(data, cpuAddr, length, bank = 0) {
  * docs); not implemented here — pass `bank` instead and call with the raw
  * binary if you're hand-mapping.
  */
-export function mapC64Address(data, cpuAddr, length, bank = 0) {
+export function mapC64Address(data, cpuAddr, length, _bank = 0) {
   // Detect .prg by reading the load address and seeing if it makes sense.
   // (Anything is a valid load addr in theory, so we just trust the first
   // 2 bytes here.)

package/src/mcp/tools/find-references.js

@@ -7,8 +7,7 @@
 // though they're not "instructions" per se.
 
 import { readFile } from "node:fs/promises";
-import { jsonContent, safeTool } from "../util.js";
-import { mapNesAddress, mapSnesAddress, mapAtari2600Address, mapAtari7800Address, mapC64Address } from "./disasm.js";
+import { mapSnesAddress, mapAtari2600Address, mapAtari7800Address, mapC64Address } from "./disasm.js";
 
 /**
  * Classify a referring instruction by its mnemonic.

package/src/mcp/tools/font-map.js

@@ -16,7 +16,7 @@
 // in findEncodedText (NES/GB/GBC bank-aware, Genesis flat; SNES is mapper-
 // dependent and left to fileOffset).
 
-import { readFile, writeFile } from "node:fs/promises";
+import { readFile } from "node:fs/promises";
 import { jsonContent, safeTool } from "../util.js";
 import { getHost } from "../state.js";
 

package/src/mcp/tools/index.js

@@ -304,52 +304,3 @@ export function registerTools(server, z, sessionKey) {
   }
 }
 
-// ---- helper: which category owns a tool name? ----
-// Used so describeTool's error message can name the category to load.
-// Maintained by hand for now; if categories explode we can derive this
-// by registering each category into a dummy server and comparing diffs.
-const TOOL_OWNER = {
-  // platforms category
-  platform: "platforms",
-  // run category
-  loadMedia: "run", host: "run",
-  frame: "run",
-  // input category
-  input: "input",
-  // state category
-  state: "state",
-  // memory category
-  memory: "memory",
-  // debug category
-  tiles: "debug", sprites: "debug",
-  background: "debug", encodeArt: "assets",
-  cpu: "debug", audioDebug: "debug",
-  symbols: "debug",
-  disasm: "debug",
-  cheats: "debug",
-  inspectTile: "debug",
-  // assets category
-  encodeAudio: "assets",
-  cart: "assets",
-  listRoms: "assets", romPatch: "assets", validateRom: "assets",
-  assembleSnippet: "assets",
-
-  text: "assets",
-  importArt: "assets",
-  palette: "debug",
-  // project category
-  scaffold: "project",
-  // show category (was: advanced)
-  playtest: "show",
-  // advanced category
-  runUntil: "advanced",
-  watch: "advanced", breakpoint: "advanced",
-  recordSession: "advanced",
-  // entry tier itself
-  catalog: "entry",
-  build: "entry", listRunnableFormats: "entry",
-};
-
-function ownerCategoryOf(name) {
-  return TOOL_OWNER[name] ?? null;
-}

package/src/mcp/tools/input-layout.js

@@ -3,7 +3,6 @@
 // know exactly what each bit/id means.
 
 import { FACE_BUTTON_MAP } from "../../host/types.js";
-import { jsonContent, safeTool } from "../util.js";
 
 const HARDWARE_LAYOUTS = {
   nes: {

package/src/mcp/tools/input.js

@@ -65,18 +65,45 @@ function buttonShape(z) {
       start: z.boolean().optional(),
       select: z.boolean().optional(),
     })
+    // passthrough (not the zod default of stripping) so a TYPO'd button name
+    // ({jump:true}, {aa:true}) survives into the handler and can be reported as
+    // ignored — instead of being silently dropped, leaving the agent believing
+    // it pressed something it didn't.
+    .passthrough()
     .describe(
       "Per-port controller state. Prefer the spatial face-button names (north/east/south/west) for cross-platform code — they map to the physical button in that compass position on each platform's controller (e.g. on NES east=A, on SNES east=A, on Genesis east=C). Raw libretro names (a/b/x/y/l/r/...) also work if you need direct control. Omitted buttons are released.",
     );
 }
 
+// Every button key portInputToMask + the spatial resolver actually honor.
+// Anything else in a `ports` object is a typo and gets reported, not pressed.
+const KNOWN_BUTTONS = new Set([
+  "up", "down", "left", "right",
+  "north", "east", "south", "west",
+  "a", "b", "x", "y", "l", "r", "l2", "r2", "l3", "r3", "start", "select",
+]);
+
 // ── *Core functions for the `input` tool ──
 
 /** op:'set' — set held controller state (persists until changed). */
 function inputSetCore({ ports }, sessionKey) {
+      // Flag any key that isn't a real button BEFORE we set input — a typo
+      // ({jump:true}) would otherwise resolve to nothing and press silently.
+      const ignoredButtons = [];
+      ports.forEach((p, port) => {
+        for (const k of Object.keys(p)) {
+          if (p[k] === true && !KNOWN_BUTTONS.has(k)) ignoredButtons.push({ port, name: k });
+        }
+      });
       getHost(sessionKey).setInput({ ports });
-      const requested = ports.map((p) => Object.keys(p).filter((k) => p[k] === true));
-      return { inputSet: true, requested };
+      const requested = ports.map((p) => Object.keys(p).filter((k) => p[k] === true && KNOWN_BUTTONS.has(k)));
+      return {
+        inputSet: true,
+        requested,
+        ...(ignoredButtons.length
+          ? { ignoredButtons, ignoredNote: `Ignored ${ignoredButtons.length} unknown button name(s) — not pressed. Valid: ${[...KNOWN_BUTTONS].join(", ")}.` }
+          : {}),
+      };
 }
 
 /** op:'press' — press one named button N frames then release (port 0 default). */
@@ -168,7 +195,10 @@ export function registerInputTools(server, z, sessionKey) {
   server.tool(
     "input",
     "Drive the controller. `op`: 'set' | 'press' | 'sequence' | 'navigate' | 'layout'.\n" +
-    "'set': hold controller state (persists until changed) via `ports:[{a:true,...},{...}]`.\n" +
+    "'set': hold controller state (persists until changed) via `ports:[{a:true,...},{...}]`. " +
+    "The held state is honored by frame({op:'step'}) AND by watch/breakpoint runs that have NO `pressDuring` " +
+    "schedule (they inherit it). If a watch/breakpoint IS given `pressDuring`, that schedule OWNS the pad for " +
+    "the run and this set state is ignored — so drive a watched window with `pressDuring`, not a prior `set`.\n" +
     "'press': press one named `button` for `frames` then release (port 0 default).\n" +
     "'sequence': scripted frame-by-frame `steps:[{input:{ports}, frames}]` for replays/tests.\n" +
     "'navigate': walk a menu by advancing on SCREEN CHANGE — `steps:[{button, holdFrames?, maxWaitFrames?, " +

package/src/mcp/tools/lifecycle.js

@@ -1,5 +1,5 @@
 import { resolveCore } from "../../cores/registry.js";
-import { clearHost, getHost, getHostOrNull, resetHost } from "../state.js";
+import { clearHost, getHost, getHostOrNull, rememberLastMedia, resetHost } from "../state.js";
 import { jsonContent, safeTool, textContent } from "../util.js";
 import { resolveCheatCodeForApply } from "./cheats.js";
 
@@ -44,6 +44,13 @@ export function registerLifecycleTools(server, z, sessionKey) {
         }
       });
     }
+    // Remember what we loaded so a later host eviction (restart/reconnect) can
+    // tell the agent the exact loadMedia call to recover with. Survives reset.
+    rememberLastMedia(sessionKey, {
+      platform,
+      ...(bytes ? { fromBase64: true } : { path: host.status.mediaPath ?? path }),
+    });
+
     // Framebuffer dimensions are NOT known until the core has run at least one
     // frame — before that, fbWidth/fbHeight hold a pre-boot default (e.g.
     // 256×192 on Genesis) that does NOT match the real output resolution
@@ -103,7 +110,12 @@ export function registerLifecycleTools(server, z, sessionKey) {
       switch (op) {
         case "unload": {
           const host = getHostOrNull(sessionKey);
-          if (host) host.unloadMedia();
+          if (!host || !host.status.loaded) {
+            // Don't claim success when there was nothing loaded — that masks a
+            // session/state mix-up (the agent thinks it unloaded media it never had).
+            return textContent("nothing to unload — no media is loaded in this session");
+          }
+          host.unloadMedia();
           return textContent("unloaded");
         }
         case "shutdown":

package/src/mcp/tools/lospec.js

@@ -4,7 +4,6 @@
 // per-palette overrides.
 
 import { jsonContent, safeTool } from "../util.js";
-import { resolveIntent } from "../../platforms/common/intent.js";
 import { inspectPaletteCore, getPlatformMasterPaletteCore } from "./platform-tools.js";
 
 /** lospec.com hosts each palette at https://lospec.com/palette-list/<id>.json. */
@@ -61,24 +60,6 @@ async function fetchLospecPalette(id) {
   };
 }
 
-/**
- * Snap each color in `palette` to the nearest entry in `master`.
- * Used when the caller wants a lospec palette but the platform has a
- * fixed hardware palette (NES 2C02 master, etc.).
- */
-function snapToMaster(palette, master) {
-  return palette.map(([r, g, b]) => {
-    let best = master[0];
-    let bestD = Infinity;
-    for (const [mr, mg, mb] of master) {
-      const dr = r - mr, dg = g - mg, db = b - mb;
-      const d = dr * dr + dg * dg + db * db;
-      if (d < bestD) { bestD = d; best = [mr, mg, mb]; }
-    }
-    return best;
-  });
-}
-
 /**
  * Programmatic equivalent of the MCP tool. Exported for tests.
  *

package/src/mcp/tools/platform-docs.js

@@ -16,7 +16,7 @@ import { fileURLToPath } from "node:url";
 import path from "node:path";
 import { readFile, stat } from "node:fs/promises";
 
-import { jsonContent, safeTool, textContent } from "../util.js";
+import { jsonContent } from "../util.js";
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);

package/src/mcp/tools/platform-tools.js

@@ -6,7 +6,7 @@ import { mkdir, readFile, writeFile } from "node:fs/promises";
 import path from "node:path";
 import { PNG } from "pngjs";
 import { getHost } from "../state.js";
-import { imageContent, jsonContent, safeTool, textContent } from "../util.js";
+import { imageContent, jsonContent } from "../util.js";
 
 // Consolidation: several handlers in this big shared file are extracted as
 // *Core functions that the consolidated domain tools (palette/tiles/background/
@@ -37,10 +37,10 @@ import { getNesApuState } from "../../host/nes-apu-state.js";
 import { decodeGenesisPSG, decodeGenesisYM2612 } from "../../host/gpgx-state.js";
 import { decodeGbApu, decodeGbaApu } from "../../host/gb-apu-state.js";
 import { decodeC64Sid } from "../../host/c64-sid-state.js";
-import { decodeLynxMikey, decodeLynxPalette, decodeLynxRenderingContext } from "../../host/lynx-mikey-state.js";
+import { decodeLynxMikey, decodeLynxPalette } from "../../host/lynx-mikey-state.js";
 import { getPcePsgState } from "../../host/pce-psg-state.js";
 import { getMsxAyState } from "../../host/msx-ay-state.js";
-import { decodeGbaSprites, decodeGbaPalette, decodeGbaRenderingContext } from "../../host/gba-video-state.js";
+import { decodeGbaSprites, decodeGbaPalette } from "../../host/gba-video-state.js";
 
 /** Resolve the platform to inspect: explicit arg → currently loaded host. */
 function resolvePlatform(host, requested) {
@@ -119,7 +119,7 @@ export function registerPlatformTools(server, z, sessionKey) {
         // SMS tiles live in VRAM at runtime — the cart has no fixed CHR
         // region. Render all 448 tiles (the entire 16KB VRAM mapped to
         // tiles), using the live first-BG-palette so colors look right.
-        const { snapshotPatternTiles, snapshotPalette } = await import("../../platforms/sms/vdp.js");
+        const { snapshotPalette } = await import("../../platforms/sms/vdp.js");
         const { colors } = snapshotPalette(host, p);
         // Use BG palette (entries 0..15) for rendering.
         const bgPal = colors.slice(0, 16).map((c) => [c.r, c.g, c.b]);

package/src/mcp/tools/project.js

@@ -8,7 +8,6 @@
 // the project elsewhere and rebuild with cc65/sdcc directly, every byte
 // that compiles is in the directory.
 
-import { readFile, writeFile } from "node:fs/promises";
 import { jsonContent, safeTool } from "../util.js";
 import { starterSnippetsCore, copyStarterSnippetsCore } from "./snippets.js";
 
@@ -1776,7 +1775,6 @@ Compiles **C89**, not C99/C11. Stick to:
   // runtime tmpl.runtime list typically includes them) are skipped.
   const snippetFiles = [];
   if (withSnippets) {
-    const { listSnippetsForPlatform } = await import("./snippets.js").catch(() => ({}));
     // Inline minimal duplicate of listSnippetsForPlatform since snippets.js
     // doesn't export it. Keep this in sync with snippets.js.
     const LIB_DIR = path.join(PLATFORM_LIB_DIR, "lib");

package/src/mcp/tools/reinject.js

@@ -16,7 +16,6 @@
 // dev wiki (citations inline).
 
 import { readFile, writeFile } from "node:fs/promises";
-import { jsonContent, safeTool } from "../util.js";
 
 // ───────────────────────────────────────────────────────────────────────────
 // Pointer encoding per platform.

package/src/mcp/tools/rom-id.js

@@ -1,5 +1,5 @@
 import { imageContent, jsonContent, safeTool } from "../util.js";
-import { intentZod, resolveIntent } from "../../platforms/common/intent.js";
+import { resolveIntent } from "../../platforms/common/intent.js";
 import { getDefaultPalette, DEFAULT_PALETTES } from "../../platforms/common/default-palette.js";
 import { spliceChrCore } from "./splice-chr.js";
 import { relocateBlockCore, makeStoredBlockCore, findPointerToCore, PLATFORM_REGISTRY } from "./reinject.js";
@@ -65,7 +65,7 @@ export async function gbHeaderCore({ path: inPath, outputPath, cgb, title, cartT
   };
 }
 
-export function registerRomIdTools(server, z, sessionKey) {
+export function registerRomIdTools(server, z, _sessionKey) {
   // identifyRom folded into `cart`; patchFile/patchRom/spliceCHR/relocate/etc.
   // folded into the `romPatch` tool (router below).
   const PLATFORMS = Object.keys(PLATFORM_REGISTRY);

package/src/mcp/tools/snippets.js

@@ -13,7 +13,7 @@ import { fileURLToPath } from "node:url";
 import path from "node:path";
 import { readdir, readFile, stat, mkdir, writeFile } from "node:fs/promises";
 
-import { jsonContent, safeTool, textContent, writeOutput } from "../util.js";
+import { jsonContent, textContent, writeOutput } from "../util.js";
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
@@ -87,7 +87,7 @@ async function listSnippetsForPlatform(platform) {
   return out;
 }
 
-export function registerSnippetTools(server, z) {
+export function registerSnippetTools(_server, _z) {
   // ── Shared implementations for the three snippet modes ──────────
   async function snippetsList(platform, language) {
     const all = await listSnippetsForPlatform(platform);

package/src/mcp/tools/sprite-pipeline.js

@@ -15,7 +15,6 @@ import { readFile, writeFile } from "node:fs/promises";
 import { PNG } from "pngjs";
 import { jsonContent, safeTool } from "../util.js";
 import { intentZod, resolveIntent, intentError } from "../../platforms/common/intent.js";
-import { getDefaultPalette } from "../../platforms/common/default-palette.js";
 import { convertImageToTilesCore, imageToTilemapCore } from "./platform-tools.js";
 import { validateGenesisTilesCore } from "./metasprite-tools.js";
 
@@ -542,7 +541,7 @@ async function crossPlatformSpriteImportImpl(args) {
 
 // ── Register all three tools ─────────────────────────────────────────
 
-export function registerSpritePipelineTools(server, z, sessionKey) {
+export function registerSpritePipelineTools(server, z, _sessionKey) {
   server.tool(
     "encodeArt",
     "Encode a PNG into a platform's native art format, one tool keyed by `stage` — the PNG→tiles pipeline. " +

package/src/mcp/tools/tile-inspect.js

@@ -8,7 +8,7 @@
 // always reports `source: "file" | "emulator"` so the caller knows which.
 
 import { readFile } from "node:fs/promises";
-import { getHostOrNull, getHost } from "../state.js";
+import { getHost } from "../state.js";
 import { jsonContent, imageContent, textContent, safeTool } from "../util.js";
 import { inspectPatternTilesCore } from "./platform-tools.js";
 import { extractSpriteSheetCore } from "./rom-id.js";