c64bridge 1.0.7 → 1.1.0

package/mcp.json

@@ -1,6 +1,6 @@
 {
   "name": "c64bridge",
-  "version": "1.0.7",
+  "version": "1.1.0",
   "description": "Stdio-based Model Context Protocol (MCP) server that bridges Commodore 64 Ultimate hardware with AI agents.",
   "entry": "dist/index.js",
   "type": "stdio",
@@ -36,6 +36,26 @@
       "description": "Path to configuration JSON",
       "default": "~/.c64bridge.json"
     },
+    "C64BRIDGE_ALLOWED_DIRS": {
+      "description": "Additional directories allowed for host file paths in tool arguments, separated by the path delimiter; * disables the check",
+      "default": ""
+    },
+    "C64BRIDGE_ACCESS": {
+      "description": "Access mode limiting which operations may be called (full, no-destructive, read-only)",
+      "default": "full"
+    },
+    "C64BRIDGE_HTTP_TOKEN": {
+      "description": "Bearer token required by the optional HTTP transport (--http)",
+      "default": ""
+    },
+    "C64BRIDGE_DIAGNOSTICS_KEEP": {
+      "description": "Number of diagnostics files kept at startup; files older than 30 days are always removed",
+      "default": "200"
+    },
+    "C64BRIDGE_VERBOSE_SCHEMA": {
+      "description": "Set to 1 to include full per-operation schemas in tools/list",
+      "default": ""
+    },
     "C64_MODE": {
       "description": "Select active backend (c64u/U64 hardware, u2/U2-family cartridge, or vice emulator)",
       "default": "c64u"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "c64bridge",
-  "version": "1.0.7",
+  "version": "1.1.0",
   "mcpName": "io.github.chrisgleissner/c64bridge",
   "description": "C64 Bridge MCP server for Commodore 64 Ultimate hardware and VICE emulator control",
   "repository": {
@@ -23,6 +23,7 @@
     "mcp:node": "node dist/mcp-server.js",
     "mcp:build": "npm run build && node dist/mcp-server.js",
     "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",
+    "prepack": "npm run build",
     "lint": "npm run check",
     "test": "node scripts/invoke-bun.mjs scripts/run-tests.ts",
     "test:node": "node scripts/run-tests.mjs",
@@ -71,10 +72,8 @@
     "@modelcontextprotocol/sdk": "^1.32.1",
     "axios": "^1.20.0",
     "date-fns": "^4.4.0",
-    "fastify": "^5.12.5",
     "jimp": "^1.6.1",
     "meyda": "5.6.2",
-    "octokit": "^5.0.5",
     "pitchfinder": "^2.3.4",
     "reflect-metadata": "^0.2.2",
     "yaml": "^2.9.0"
@@ -83,10 +82,12 @@
     "naudiodon": "^2.3.6"
   },
   "overrides": {
-    "zod": "^4.6.6"
+    "zod": "^4.6.6",
+    "hono": "^4.13.16"
   },
   "devDependencies": {
     "@types/node": "^25.9.5",
+    "octokit": "^5.0.5",
     "tsx": "^4.23.11",
     "typescript": "^6.0.3"
   },
@@ -98,11 +99,18 @@
     "README.md",
     "mcp/**",
     "dist/**",
+    "!dist/.tsbuildinfo",
     "data/**",
-    "doc/**",
+    "doc/developer.md",
+    "doc/troubleshooting-mcp.md",
+    "doc/c64u/**",
+    "doc/u2/**",
+    "doc/vice/**",
+    "doc/img/**",
     "scripts/**",
+    "!scripts/compare-rest/**",
     "generated/**/*.js",
-    ".github/prompts/**"
-  ],
-  "prepack": "npm run build"
+    ".github/prompts/**",
+    ".github/skills/**"
+  ]
 }

package/scripts/mockC64Server.mjs

@@ -88,6 +88,12 @@ function createInitialState() {
     networkPassword: null,
     forcedErrors: null,
     forcedErrorsUrlMatch: null,
+    // Test hooks for request serialisation: every request is counted while it
+    // is being served, optionally delayed so overlap becomes observable.
+    responseDelayMs: 0,
+    inFlight: 0,
+    maxInFlight: 0,
+    requestOrder: [],
     hangOnMachineInputProbe: false,
     lastPrg: null,
     lastLoadedPrg: null,
@@ -405,6 +411,21 @@ export async function startMockC64Server(options = {}) {
     // Track last request metadata
     state.lastRequest = { method, url, headers: req.headers };
 
+    state.inFlight += 1;
+    state.maxInFlight = Math.max(state.maxInFlight, state.inFlight);
+    if (state.requestOrder.length < 10_000) state.requestOrder.push(`${method} ${url}`);
+    let released = false;
+    const releaseInFlight = () => {
+      if (!released) {
+        released = true;
+        state.inFlight -= 1;
+      }
+    };
+    res.once("close", releaseInFlight);
+    if (state.responseDelayMs > 0) {
+      await new Promise((resolve) => setTimeout(resolve, state.responseDelayMs));
+    }
+
     if (state.networkPassword && req.headers["x-password"] !== state.networkPassword) {
       sendJson(res, { message: "Forbidden", errors: ["invalid network password"] }, 403);
       return;
@@ -814,6 +835,29 @@ export async function startMockC64Server(options = {}) {
       }
     }
 
+    const categoryFlashRoute = /^\/v1\/configs\/([^/:]+):(load_from_flash|save_to_flash|reset_to_default)(?:\?|$)/.exec(url);
+    if (categoryFlashRoute && method === "PUT") {
+      const requested = decodeURIComponent(categoryFlashRoute[1]);
+      const action = categoryFlashRoute[2];
+      const category = Object.keys(state.configs).find((name) => name.toLowerCase() === requested.toLowerCase());
+      if (!category) {
+        sendJson(res, { errors: [`Category ${requested} not found`] }, 404);
+        return;
+      }
+      const clone = (value) => JSON.parse(JSON.stringify(value));
+      if (action === "load_from_flash") {
+        if (state.flashSnapshot?.[category]) state.configs[category] = clone(state.flashSnapshot[category]);
+      } else if (action === "save_to_flash") {
+        state.flashSnapshot = state.flashSnapshot ?? {};
+        state.flashSnapshot[category] = clone(state.configs[category]);
+      } else {
+        state.configs[category] = clone(createDefaultConfigs()[category] ?? {});
+      }
+      state.lastConfigAction = { action, category };
+      sendJson(res, { result: action, category });
+      return;
+    }
+
     if (url.startsWith("/v1/configs/")) {
       const routeUrl = new URL(url, "http://mock.local");
       const segments = routeUrl.pathname.split("/").filter(Boolean).slice(2).map((segment) => decodeURIComponent(segment)); // remove v1 + configs

package/scripts/run-coverage.mjs

@@ -107,6 +107,32 @@ export async function main() {
 
   const summary = summariseReport(finalReport);
   console.log(JSON.stringify({ lines: summary }, null, 2));
+
+  // The merged report only carries line data, so only the line threshold can be enforced here.
+  const minimum = resolveLineThreshold(coverageConfig, process.env);
+  try {
+    enforceLineThreshold(summary, minimum);
+  } catch (error) {
+    console.error(error instanceof Error ? error.message : String(error));
+    process.exitCode = 1;
+  }
+}
+
+/** Minimum line coverage in percent: C64BRIDGE_COVERAGE_MIN_LINES overrides `lines` from .c8rc.json. */
+export function resolveLineThreshold(config, env = process.env) {
+  const fromEnv = Number(env.C64BRIDGE_COVERAGE_MIN_LINES ?? "");
+  if (env.C64BRIDGE_COVERAGE_MIN_LINES !== undefined && env.C64BRIDGE_COVERAGE_MIN_LINES !== "" && Number.isFinite(fromEnv)) {
+    return fromEnv;
+  }
+  const fromConfig = Number(config?.lines);
+  return Number.isFinite(fromConfig) ? fromConfig : 0;
+}
+
+export function enforceLineThreshold(summary, minimum) {
+  const pct = Number(summary.pct);
+  if (!(pct >= minimum)) {
+    throw new Error(`Line coverage ${summary.pct}% is below the required ${minimum}%`);
+  }
 }
 
 export function buildCoverageBatches(testFiles, extraTests, env = process.env) {
@@ -327,7 +353,7 @@ function serializeReport(report) {
   return `${lines.join("\n")}\n`;
 }
 
-function summariseReport(report) {
+export function summariseReport(report) {
   let covered = 0;
   let total = 0;
   for (const record of report.values()) {

package/scripts/smoke-run.sh

@@ -28,7 +28,8 @@ case "$MODE" in
     fi
     (
       cd "$PACKAGE_DIR" || exit 1
-  timeout --signal=TERM "$DURATION" node dist/index.js
+      # The server exits when stdin closes, so keep stdin open until the timeout ends the run.
+      sleep $((DURATION + 1)) | timeout --signal=TERM "$DURATION" node dist/index.js
     ) 2>&1 | tee "$LOGFILE"
     STATUS=${PIPESTATUS[0]}
     ;;
@@ -43,7 +44,7 @@ case "$MODE" in
     fi
     (
       cd "$PACKAGE_DIR" || exit 1
-      timeout --signal=TERM "$DURATION" node dist/index.js
+      sleep $((DURATION + 1)) | timeout --signal=TERM "$DURATION" node dist/index.js
     ) 2>&1 | tee "$LOGFILE"
     STATUS=${PIPESTATUS[0]}
     ;;

package/scripts/start.mjs

@@ -79,8 +79,30 @@ async function runWithBun(entryPath) {
         },
       });
 
-      child.on('error', reject);
+      // Forward termination signals so the server child does not outlive the launcher.
+      const forwardedSignals = ['SIGTERM', 'SIGINT', 'SIGHUP'];
+      const forwarders = new Map();
+      for (const name of forwardedSignals) {
+        const forward = () => {
+          try {
+            child.kill(name);
+          } catch {}
+        };
+        forwarders.set(name, forward);
+        process.on(name, forward);
+      }
+      const removeForwarders = () => {
+        for (const [name, forward] of forwarders) {
+          process.removeListener(name, forward);
+        }
+      };
+
+      child.on('error', (error) => {
+        removeForwarders();
+        reject(error);
+      });
       child.on('exit', (code, signal) => {
+        removeForwarders();
         if (signal) {
           try {
             process.kill(process.pid, signal);

package/server.json

@@ -8,12 +8,12 @@
     "source": "github"
   },
   "websiteUrl": "https://github.com/chrisgleissner/c64bridge",
-  "version": "1.0.7",
+  "version": "1.1.0",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "c64bridge",
-      "version": "1.0.7",
+      "version": "1.1.0",
       "transport": {
         "type": "stdio"
       },
@@ -36,6 +36,39 @@
           "format": "filepath",
           "default": "~/.c64bridge.json"
         },
+        {
+          "name": "C64BRIDGE_ALLOWED_DIRS",
+          "description": "Additional directories allowed for host file paths in tool arguments, separated by the path delimiter; * disables the check.",
+          "format": "string"
+        },
+        {
+          "name": "C64BRIDGE_ACCESS",
+          "description": "Access mode limiting which operations may be called.",
+          "format": "string",
+          "default": "full",
+          "choices": [
+            "full",
+            "no-destructive",
+            "read-only"
+          ]
+        },
+        {
+          "name": "C64BRIDGE_HTTP_TOKEN",
+          "description": "Bearer token required by the optional HTTP transport.",
+          "format": "string",
+          "isSecret": true
+        },
+        {
+          "name": "C64BRIDGE_DIAGNOSTICS_KEEP",
+          "description": "Number of diagnostics files kept at startup (files older than 30 days are always removed).",
+          "format": "number",
+          "default": "200"
+        },
+        {
+          "name": "C64BRIDGE_VERBOSE_SCHEMA",
+          "description": "Set to 1 to include full per-operation schemas in tools/list.",
+          "format": "string"
+        },
         {
           "name": "C64_MODE",
           "description": "Active backend (c64u/U64 hardware, u2/U2-family cartridge, or vice emulator).",