what-devtools-mcp 0.13.4 → 0.13.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "what-devtools-mcp",
-  "version": "0.13.4",
+  "version": "0.13.6",
   "description": "MCP server bridging AI agents to live What Framework app state via WebSocket",
   "type": "module",
   "bin": {
@@ -26,8 +26,8 @@
   ],
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.0.0",
-    "ws": "^8.18.0",
-    "zod": "^3.25.0"
+    "ws": "^8.21.3",
+    "zod": "^4.4.3"
   },
   "peerDependencies": {
     "what-devtools": ">=0.7.0"

package/src/bridge.js

@@ -23,6 +23,7 @@ export function createBridge({ port = 9229, host = '127.0.0.1', onError } = {})
   let latestSnapshot = null;
   const eventLog = [];
   const errorLog = [];
+  /** @type {import('ws').WebSocket | null} */
   let browserSocket = null;
   let correlationCounter = 0;
   const pendingCommands = new Map();
@@ -33,6 +34,7 @@ export function createBridge({ port = 9229, host = '127.0.0.1', onError } = {})
   const SNAPSHOT_CACHE_MS = 100;
 
   // Baseline snapshot for diff tool
+  /** @type {any} */
   let baselineSnapshot = null;
 
   // Use a fixed token from env if provided, otherwise generate a random one.

package/src/client-commands.js

@@ -49,6 +49,7 @@ function getComponentElement(entry) {
 
 const MAX_WRITE_LOG = 200;
 let _signalWriteLog = [];       // { signalId, signalName, previousValue, newValue, timestamp, writerEffect }
+/** @type {{ id: any, name: any, timestamp: number } | null} */
 let _lastRunningEffect = null;  // { id, name, timestamp } — most recent effect:run event
 let _trackingInitialized = false;
 let _unsubTracker = null;
@@ -111,11 +112,12 @@ function findLastWrite(signalId) {
  * Handle extended commands sent from the MCP server via the bridge.
  *
  * @param {string} command - The command name
- * @param {object} args - Command arguments
- * @param {object|null} devtools - window.__WHAT_DEVTOOLS__ reference
- * @returns {Promise<object|null>} Result object, or null if command not handled
+ * @param {Record<string, any>} [args] - Command arguments
+ * @param {any} [devtools] - window.__WHAT_DEVTOOLS__ reference (live hook bag)
+ * @returns {Promise<Record<string, any>|null>} Result object, or null if command not handled
  */
 export async function handleExtendedCommand(command, args, devtools) {
+  args = args || /** @type {Record<string, any>} */ ({});
   switch (command) {
 
     // -------------------------------------------------------------------------

package/src/client.js

@@ -31,9 +31,11 @@ export function connectDevToolsMCP({ port = 9229, token = '', discoveryUrl = ''
     }
   } catch {}
 
+  /** @type {WebSocket | null} */
   let ws = null;
   let connected = false;
   let stopped = false;
+  /** @type {ReturnType<typeof setTimeout> | null} */
   let probeTimer = null;
   // Probe back-off: first retry after 10s, then ×3 each time, capped at 5min.
   // With no bridge running this yields probes at ~0s / 10s / 40s / 130s / 430s…
@@ -46,7 +48,9 @@ export function connectDevToolsMCP({ port = 9229, token = '', discoveryUrl = ''
   let eventCount = 0;
   let hasLoggedMissingBridge = false;
   let hasShownBanner = false;
+  /** @type {(() => void) | null} */
   let unsubscribeFn = null;
+  /** @type {(() => void) | null} */
   let flushEventBatch = null; // Hoisted — set during subscription, called by set-signal
   let discoveredToken = token;
   let discoveredPort = port;
@@ -178,10 +182,11 @@ export function connectDevToolsMCP({ port = 9229, token = '', discoveryUrl = ''
       }
       if (devtools) {
         let eventBatch = [];
+        /** @type {ReturnType<typeof setTimeout> | null} */
         let batchTimer = null;
 
         // Hoisted so set-signal can call it to flush after programmatic writes
-        flushEventBatch = function() {
+        const flush = function() {
           if (eventBatch.length === 0) return;
           if (eventBatch.length === 1) {
             const item = eventBatch[0];
@@ -192,12 +197,13 @@ export function connectDevToolsMCP({ port = 9229, token = '', discoveryUrl = ''
           eventBatch = [];
           if (batchTimer) { clearTimeout(batchTimer); batchTimer = null; }
         };
+        flushEventBatch = flush;
 
         unsubscribeFn = devtools.subscribe((event, data) => {
           eventCount++;
           eventBatch.push({ event, data: devtools.safeSerialize(data) });
           if (!batchTimer) {
-            batchTimer = setTimeout(flushEventBatch, 16);
+            batchTimer = setTimeout(flush, 16);
           }
         });
         log('MCP', BADGE, 'Subscribed to reactive events — streaming to bridge (batched)');
@@ -328,6 +334,7 @@ export function connectDevToolsMCP({ port = 9229, token = '', discoveryUrl = ''
       }
       default: {
         // Try extended command handlers
+        /** @type {Record<string, any> | null} */
         let extResult = null;
         try {
           const { handleExtendedCommand, initEventTracking } = await import('./client-commands.js');

package/src/tools-extended.js

@@ -189,6 +189,7 @@ export function registerExtendedTools(server, bridge) {
       }
 
       // Optional filter — keep subtrees that contain a matching node
+      /** @type {RegExp | null} */
       let filterRe = null;
       if (filter) {
         try {
@@ -308,6 +309,9 @@ export function registerExtendedTools(server, bridge) {
 
       // If a specific signal or effect is requested, filter the graph
       let filteredEdges = allEdges;
+      // `any` rather than `Set | null`: the set is read inside filter
+      // callbacks, and TS 7 does not narrow a closed-over nullable.
+      /** @type {any} */
       let nodeIds = null; // Set of "type:id" strings we want to include
 
       if (signalId != null) {
@@ -933,6 +937,7 @@ export function registerExtendedTools(server, bridge) {
       }));
 
       // DOM output (if requested and bridge connected)
+      /** @type {{ html: any, structure: any } | null} */
       let dom = null;
       if (includeDOM) {
         try {

package/src/tools.js

@@ -63,6 +63,7 @@ export function registerTools(server, bridge) {
       const componentCount = snapshot?.components?.length || 0;
 
       // Try to get app metadata from the browser
+      /** @type {Record<string, any> | null} */
       let appInfo = null;
       if (connected) {
         try {