c64bridge 0.9.1 → 0.9.2

package/mcp/tools.json

@@ -8125,147 +8125,266 @@
       "description": "Keyboard buffer injection (cross-platform) and joystick simulation (VICE only).",
       "inputSchema": {
         "type": "object",
-        "description": "Input operations: write_text, key, joystick.",
-        "oneOf": [
+        "properties": {
+          "op": {
+            "type": "string",
+            "enum": [
+              "write_text",
+              "key",
+              "joystick"
+            ],
+            "description": "Selects the operation."
+          },
+          "text": {
+            "type": "string",
+            "description": "Text to type. Use {RETURN}, {CLR}, {F1}…{F8}, {UP/DOWN/LEFT/RIGHT}, colour names, etc.",
+            "minLength": 1,
+            "maxLength": 512
+          },
+          "delayMs": {
+            "type": [
+              "integer",
+              "null"
+            ],
+            "description": "Delay in ms after sending (default 0).",
+            "minimum": 0,
+            "maximum": 10000,
+            "default": 0
+          },
+          "key": {
+            "type": "string",
+            "description": "Key to press: a printable character, or a PETSCII token name (RETURN, F1…F8, UP, etc.).",
+            "minLength": 1,
+            "maxLength": 32
+          },
+          "durationMs": {
+            "type": [
+              "integer",
+              "null"
+            ],
+            "description": "Delay in ms after each queued key press before the next press (default 0 = no delay).",
+            "minimum": 0,
+            "maximum": 5000,
+            "default": 0
+          },
+          "count": {
+            "type": [
+              "integer",
+              "null"
+            ],
+            "description": "Number of times to press the key (default 1).",
+            "minimum": 1,
+            "maximum": 100,
+            "default": 1
+          },
+          "port": {
+            "type": "integer",
+            "description": "Joystick port (1 = $DC01, 2 = $DC00).",
+            "minimum": 1,
+            "maximum": 2
+          },
+          "controls": {
+            "type": "array",
+            "description": "List of controls to activate simultaneously.",
+            "items": {
+              "type": "string",
+              "description": "Control to activate.",
+              "enum": [
+                "up",
+                "down",
+                "left",
+                "right",
+                "fire"
+              ]
+            },
+            "minItems": 0
+          },
+          "action": {
+            "type": "string",
+            "description": "press = hold until a release call; release = restore all bits; tap = press then release after durationMs.",
+            "enum": [
+              "press",
+              "release",
+              "tap"
+            ]
+          }
+        },
+        "required": [
+          "op"
+        ],
+        "description": "Input operations: write_text, key, joystick.\n\nOperations:\n- write_text: Send a text string to the keyboard buffer, with PETSCII token expansion. Required inputs: text. Optional inputs: delayMs.\n- key: Tap a single key or hold it for a duration. Required inputs: key. Optional inputs: durationMs, count.\n- joystick: Simulate joystick input by writing directly to CIA1 Port A/B registers. Required inputs: port, controls, action. Optional inputs: durationMs.",
+        "additionalProperties": false,
+        "x-c64bridge-operations": [
           {
-            "type": "object",
+            "op": "write_text",
             "description": "Send a text string to the keyboard buffer, with PETSCII token expansion.",
-            "properties": {
-              "op": {
-                "const": "write_text"
-              },
-              "text": {
-                "type": "string",
-                "description": "Text to type. Use {RETURN}, {CLR}, {F1}…{F8}, {UP/DOWN/LEFT/RIGHT}, colour names, etc.",
-                "minLength": 1,
-                "maxLength": 512
-              },
-              "delayMs": {
-                "type": [
-                  "integer",
-                  "null"
-                ],
-                "description": "Delay in ms after sending (default 0).",
-                "minimum": 0,
-                "maximum": 10000,
-                "default": 0
-              }
-            },
             "required": [
-              "op",
               "text"
             ],
-            "additionalProperties": false
+            "optional": [
+              "delayMs"
+            ],
+            "inputSchema": {
+              "type": "object",
+              "description": "Send a text string to the keyboard buffer, with PETSCII token expansion.",
+              "properties": {
+                "op": {
+                  "const": "write_text"
+                },
+                "text": {
+                  "type": "string",
+                  "description": "Text to type. Use {RETURN}, {CLR}, {F1}…{F8}, {UP/DOWN/LEFT/RIGHT}, colour names, etc.",
+                  "minLength": 1,
+                  "maxLength": 512
+                },
+                "delayMs": {
+                  "type": [
+                    "integer",
+                    "null"
+                  ],
+                  "description": "Delay in ms after sending (default 0).",
+                  "minimum": 0,
+                  "maximum": 10000,
+                  "default": 0
+                }
+              },
+              "required": [
+                "op",
+                "text"
+              ],
+              "additionalProperties": false
+            }
           },
           {
-            "type": "object",
+            "op": "key",
             "description": "Tap a single key or hold it for a duration.",
-            "properties": {
-              "op": {
-                "const": "key"
-              },
-              "key": {
-                "type": "string",
-                "description": "Key to press: a printable character, or a PETSCII token name (RETURN, F1…F8, UP, etc.).",
-                "minLength": 1,
-                "maxLength": 32
-              },
-              "durationMs": {
-                "type": [
-                  "integer",
-                  "null"
-                ],
-                "description": "Delay in ms after each queued key press before the next press (default 0 = no delay).",
-                "minimum": 0,
-                "maximum": 5000,
-                "default": 0
-              },
-              "count": {
-                "type": [
-                  "integer",
-                  "null"
-                ],
-                "description": "Number of times to press the key (default 1).",
-                "minimum": 1,
-                "maximum": 100,
-                "default": 1
-              }
-            },
             "required": [
-              "op",
               "key"
             ],
-            "additionalProperties": false
-          },
-          {
-            "type": "object",
-            "description": "Simulate joystick input by writing directly to CIA1 Port A/B registers.",
-            "properties": {
-              "op": {
-                "const": "joystick"
-              },
-              "port": {
-                "type": "integer",
-                "description": "Joystick port (1 = $DC01, 2 = $DC00).",
-                "minimum": 1,
-                "maximum": 2
-              },
-              "controls": {
-                "type": "array",
-                "description": "List of controls to activate simultaneously.",
-                "items": {
+            "optional": [
+              "durationMs",
+              "count"
+            ],
+            "inputSchema": {
+              "type": "object",
+              "description": "Tap a single key or hold it for a duration.",
+              "properties": {
+                "op": {
+                  "const": "key"
+                },
+                "key": {
                   "type": "string",
-                  "description": "Control to activate.",
-                  "enum": [
-                    "up",
-                    "down",
-                    "left",
-                    "right",
-                    "fire"
-                  ]
+                  "description": "Key to press: a printable character, or a PETSCII token name (RETURN, F1…F8, UP, etc.).",
+                  "minLength": 1,
+                  "maxLength": 32
                 },
-                "minItems": 0
-              },
-              "action": {
-                "type": "string",
-                "description": "press = hold until a release call; release = restore all bits; tap = press then release after durationMs.",
-                "enum": [
-                  "press",
-                  "release",
-                  "tap"
-                ]
+                "durationMs": {
+                  "type": [
+                    "integer",
+                    "null"
+                  ],
+                  "description": "Delay in ms after each queued key press before the next press (default 0 = no delay).",
+                  "minimum": 0,
+                  "maximum": 5000,
+                  "default": 0
+                },
+                "count": {
+                  "type": [
+                    "integer",
+                    "null"
+                  ],
+                  "description": "Number of times to press the key (default 1).",
+                  "minimum": 1,
+                  "maximum": 100,
+                  "default": 1
+                }
               },
-              "durationMs": {
-                "type": [
-                  "integer",
-                  "null"
-                ],
-                "description": "Duration to hold for tap action in ms (default 80).",
-                "minimum": 10,
-                "maximum": 5000,
-                "default": 80
-              }
-            },
+              "required": [
+                "op",
+                "key"
+              ],
+              "additionalProperties": false
+            }
+          },
+          {
+            "op": "joystick",
+            "description": "Simulate joystick input by writing directly to CIA1 Port A/B registers.",
             "required": [
-              "op",
               "port",
               "controls",
               "action"
             ],
-            "additionalProperties": false
+            "optional": [
+              "durationMs"
+            ],
+            "inputSchema": {
+              "type": "object",
+              "description": "Simulate joystick input by writing directly to CIA1 Port A/B registers.",
+              "properties": {
+                "op": {
+                  "const": "joystick"
+                },
+                "port": {
+                  "type": "integer",
+                  "description": "Joystick port (1 = $DC01, 2 = $DC00).",
+                  "minimum": 1,
+                  "maximum": 2
+                },
+                "controls": {
+                  "type": "array",
+                  "description": "List of controls to activate simultaneously.",
+                  "items": {
+                    "type": "string",
+                    "description": "Control to activate.",
+                    "enum": [
+                      "up",
+                      "down",
+                      "left",
+                      "right",
+                      "fire"
+                    ]
+                  },
+                  "minItems": 0
+                },
+                "action": {
+                  "type": "string",
+                  "description": "press = hold until a release call; release = restore all bits; tap = press then release after durationMs.",
+                  "enum": [
+                    "press",
+                    "release",
+                    "tap"
+                  ]
+                },
+                "durationMs": {
+                  "type": [
+                    "integer",
+                    "null"
+                  ],
+                  "description": "Duration to hold for tap action in ms (default 80).",
+                  "minimum": 10,
+                  "maximum": 5000,
+                  "default": 80
+                }
+              },
+              "required": [
+                "op",
+                "port",
+                "controls",
+                "action"
+              ],
+              "additionalProperties": false
+            }
           }
-        ],
-        "discriminator": {
-          "propertyName": "op"
-        }
+        ]
       },
       "_meta": {
         "domain": "input",
         "summary": "Types text, taps keys, and simulates joystick movements.",
         "lifecycle": "request-response",
         "resources": [
-          "c64://specs/assembly",
-          "c64://specs/memory-map"
+          "c64://guide/bootstrap",
+          "c64://memory/map",
+          "c64://io/cia/spec"
         ],
         "prompts": [],
         "examples": [

package/mcp.json

@@ -1,6 +1,6 @@
 {
   "name": "c64bridge",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "Stdio-based Model Context Protocol (MCP) server that bridges Commodore 64 Ultimate hardware with AI agents.",
   "entry": "dist/index.js",
   "type": "stdio",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "c64bridge",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "mcpName": "io.github.chrisgleissner/c64bridge",
   "description": "C64 Bridge MCP server for Commodore 64 Ultimate hardware and VICE emulator control",
   "type": "module",
@@ -18,7 +18,7 @@
     "mcp": "node scripts/invoke-bun.mjs src/mcp-server.ts",
     "mcp:node": "node dist/mcp-server.js",
     "mcp:build": "npm run build && node dist/mcp-server.js",
-    "build": "node scripts/prebuild.mjs && tsc -p tsconfig.json && node scripts/postbuild.mjs && node scripts/invoke-bun.mjs scripts/update-readme.ts && node scripts/invoke-bun.mjs scripts/generate-mcp-interface.ts",
+    "build": "VICE_VISIBLE=false node scripts/prebuild.mjs && VICE_VISIBLE=false tsc -p tsconfig.json && VICE_VISIBLE=false node scripts/postbuild.mjs && VICE_VISIBLE=false node scripts/invoke-bun.mjs scripts/update-readme.ts && VICE_VISIBLE=false node scripts/invoke-bun.mjs scripts/generate-mcp-interface.ts",
     "lint": "npm run check",
     "test": "node scripts/invoke-bun.mjs scripts/run-tests.ts",
     "test:node": "node scripts/run-tests.mjs",

package/scripts/c64-cli.mjs

@@ -18,7 +18,7 @@ async function loadCoreModules() {
   }
 
   if (typeof globalThis.Bun !== "undefined") {
-    ({ basicToPrg } = await import("../src/basicConverter.ts"));
+    ({ basicToPrg } = await import("../src/tools/translation/basicTokenizer.ts"));
     ({ loadConfig } = await import("../src/config.ts"));
     ({ C64Client } = await import("../src/c64Client.ts"));
     return;
@@ -26,7 +26,7 @@ async function loadCoreModules() {
 
   const distPath = resolve(projectRoot, "dist");
   try {
-    ({ basicToPrg } = await import(pathToFileURL(resolve(distPath, "basicConverter.js")).href));
+    ({ basicToPrg } = await import(pathToFileURL(resolve(distPath, "tools/translation/basicTokenizer.js")).href));
     ({ loadConfig } = await import(pathToFileURL(resolve(distPath, "config.js")).href));
     ({ C64Client } = await import(pathToFileURL(resolve(distPath, "c64Client.js")).href));
   } catch (error) {

package/scripts/run-coverage.mjs

@@ -162,7 +162,8 @@ async function listRepoTestFiles(root) {
   return files;
 }
 
-function runCoverageLeg(leg, legDir, label, files) {
+async function runCoverageLeg(leg, legDir, label, files) {
+  const knownTempFiles = new Set(await listCoverageTempFiles(coverageDir));
   return new Promise((resolve, reject) => {
     const outputPath = path.join(legDir, `${label}.lcov.info`);
     const args = [runner, "scripts/run-tests.ts", "--coverage", ...leg.args, ...files];
@@ -178,7 +179,7 @@ function runCoverageLeg(leg, legDir, label, files) {
         return;
       }
       try {
-        await fs.copyFile(path.join(coverageDir, "lcov.info"), outputPath);
+        await copyCoverageOutput(coverageDir, outputPath, knownTempFiles);
         resolve(outputPath);
       } catch (error) {
         reject(error);
@@ -187,6 +188,56 @@ function runCoverageLeg(leg, legDir, label, files) {
   });
 }
 
+export async function copyCoverageOutput(sourceDir, outputPath, knownTempFiles = new Set()) {
+  const stablePath = path.join(sourceDir, "lcov.info");
+  try {
+    await fs.copyFile(stablePath, outputPath);
+    return outputPath;
+  } catch (error) {
+    if (!isNodeErrorCode(error, "ENOENT")) {
+      throw error;
+    }
+  }
+
+  const tempFiles = await listCoverageTempFiles(sourceDir);
+  const newTempFiles = tempFiles.filter((filePath) => !knownTempFiles.has(filePath));
+  if (newTempFiles.length === 0) {
+    throw new Error(`Coverage run completed but no LCOV report was produced in ${sourceDir}`);
+  }
+
+  const newest = await newestFile(newTempFiles);
+  await fs.copyFile(newest, outputPath);
+  return outputPath;
+}
+
+async function listCoverageTempFiles(dir) {
+  let entries;
+  try {
+    entries = await fs.readdir(dir);
+  } catch (error) {
+    if (isNodeErrorCode(error, "ENOENT")) {
+      return [];
+    }
+    throw error;
+  }
+  return entries
+    .filter((entry) => /^\.lcov\.info\.[^.]+\.tmp$/.test(entry))
+    .map((entry) => path.join(dir, entry));
+}
+
+async function newestFile(filePaths) {
+  const stats = await Promise.all(filePaths.map(async (filePath) => ({
+    filePath,
+    mtimeMs: (await fs.stat(filePath)).mtimeMs,
+  })));
+  stats.sort((a, b) => b.mtimeMs - a.mtimeMs || a.filePath.localeCompare(b.filePath));
+  return stats[0].filePath;
+}
+
+function isNodeErrorCode(error, code) {
+  return Boolean(error && typeof error === "object" && error.code === code);
+}
+
 async function readReport(filePath) {
   const text = await fs.readFile(filePath, "utf8");
   return parseReport(text);
@@ -313,4 +364,4 @@ function globToRegExp(glob) {
   }
   regex += "$";
   return new RegExp(regex);
-}
+}

package/scripts/run-tests.mjs

@@ -18,6 +18,7 @@ if (!fs.existsSync(defaultEmbeddingsDir)) {
 const env = {
   ...process.env,
   C64_TEST_TARGET: target,
+  VICE_VISIBLE: "false",
 };
 
 if (!env.RAG_EMBEDDINGS_DIR) {

package/scripts/run-tests.ts

@@ -142,6 +142,7 @@ export function buildMatrixEnv(
   const env: Record<string, string> = { ...envSource } as Record<string, string>;
   env.C64_MODE = platform;
   env.C64_TEST_TARGET = target === "device" ? "real" : "mock";
+  env.VICE_VISIBLE = "false";
 
   if (platform === "vice") {
     const useViceMock = target !== "device";

package/server.json

@@ -8,12 +8,12 @@
     "source": "github"
   },
   "websiteUrl": "https://github.com/chrisgleissner/c64bridge",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "c64bridge",
-      "version": "0.9.1",
+      "version": "0.9.2",
       "transport": {
         "type": "stdio"
       },