@theme-registry/refract-mcp 0.1.5 → 0.1.7

package/README.md

@@ -8,6 +8,11 @@
 It is **project-scoped**: it loads your `theme.config.(ts|js|mjs)` once at startup and serves queries
 against it, so the agent asks about *your* theme without ever resending it. It reloads on change.
 
+The reload watches the config's directory recursively — so a theme split across `./tokens/*.ts` reloads
+too, not just an edit to the config itself. Hidden paths are never watched (that includes `.git`,
+`.next`, `.turbo`, and the hidden `.mjs` files a `.ts` config's graph-compile emits beside your sources
+and unlinks again), and neither are `node_modules` / `dist` / `out` / `coverage`. Reloads never overlap.
+
 ## Tools
 
 | Tool | Answers |

package/dist/server.js

@@ -227,10 +227,25 @@ function diffHeld(state, candidateRaw) {
     }
     return { ok: validation.ok, diff, targets: validation.perTarget };
 }
-// Dirs never worth reloading on (vendored deps, build output, VCS). Skipped under the recursive watch.
-const WATCH_IGNORE = /(^|[/\\])(node_modules|dist|out|\.git|coverage)([/\\]|$)/;
+// Dirs never worth reloading on (vendored deps, build output). Skipped under the recursive watch.
+const WATCH_IGNORE = /(^|[/\\])(node_modules|dist|out|coverage)([/\\]|$)/;
 // Source files a theme graph is authored in — a change to any of them may change the built theme.
 const WATCH_SOURCE = /\.(ts|mjs|cjs|js|json)$/;
+/**
+ * Any hidden path segment. A theme graph is never authored in one, and this is the rule that keeps the
+ * server from waking on its own output: loading a `.ts` config graph-compiles it to hidden
+ * `.<base>.<pid>-<n>.mjs` files emitted **beside each compiled source** (adjacency is load-bearing —
+ * it's what keeps relative sibling specifiers resolvable), which are imported and then unlinked. Those
+ * writes and deletes match {@link WATCH_SOURCE}, so without this a load wakes the watcher, which
+ * reloads, which emits them again — an endless self-triggered loop with no user edit involved, churning
+ * the config's directory (visible in an editor as a file tree that never stops refreshing). Also covers
+ * `.git`, `.next`, `.turbo`, `.cache` and friends, whose build churn caused spurious reloads too.
+ */
+const WATCH_HIDDEN = /(^|[/\\])\./;
+/** Is this watch event worth rebuilding the theme for? Pure, so the loop guard above is unit-testable. */
+function shouldReload(filename) {
+    return WATCH_SOURCE.test(filename) && !WATCH_HIDDEN.test(filename) && !WATCH_IGNORE.test(filename);
+}
 /**
  * Watch the config's directory **recursively** and rebuild on any source-file change — so a theme split
  * across `./tokens/*.ts` (imported transitively by the config) reloads, not just an edit to the config
@@ -241,21 +256,40 @@ function watchConfig() {
     if (!held)
         return;
     let timer;
+    // Second line of defence behind {@link WATCH_HIDDEN}: never overlap two loads, so anything a load
+    // writes can't start another one. Edits landing mid-load aren't lost — they re-arm the debounce once.
+    let reloading = false;
+    let pending = false;
+    const reload = () => {
+        if (reloading) {
+            pending = true;
+            return;
+        }
+        reloading = true;
+        loadTheme(configPathArg)
+            .then((next) => {
+            held = next;
+            process.stderr.write(`refract-mcp: reloaded theme from ${next.path}\n`);
+        })
+            .catch((e) => process.stderr.write(`refract-mcp: reload failed (kept previous theme): ${e.message}\n`))
+            .finally(() => {
+            reloading = false;
+            if (pending) {
+                pending = false;
+                schedule();
+            }
+        });
+    };
+    const schedule = () => {
+        clearTimeout(timer);
+        timer = setTimeout(reload, 150);
+    };
     const onChange = (_event, filename) => {
         if (!filename)
             return;
-        const f = filename.toString();
-        if (WATCH_IGNORE.test(f) || !WATCH_SOURCE.test(f))
+        if (!shouldReload(filename.toString()))
             return;
-        clearTimeout(timer);
-        timer = setTimeout(() => {
-            loadTheme(configPathArg)
-                .then((next) => {
-                held = next;
-                process.stderr.write(`refract-mcp: reloaded theme from ${next.path}\n`);
-            })
-                .catch((e) => process.stderr.write(`refract-mcp: reload failed (kept previous theme): ${e.message}\n`));
-        }, 150);
+        schedule();
     };
     try {
         const dir = dirname(held.path);
@@ -479,4 +513,4 @@ function isBinEntry(invokedAs, selfPath, real = realpathOr) {
     }
 }
 
-export { callTool, createServer, guideFiles, isBinEntry, loadTheme, serverVersion, start };
+export { callTool, createServer, guideFiles, isBinEntry, loadTheme, serverVersion, shouldReload, start };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@theme-registry/refract-mcp",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "description": "Model Context Protocol server for refract — query + validate a project's theme from an AI agent.",
   "author": {
     "name": "Petyo Stoyanov",
@@ -25,7 +25,7 @@
   ],
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.29.0",
-    "@theme-registry/refract": "^0.1.5"
+    "@theme-registry/refract": "^0.1.7"
   },
   "peerDependencies": {
     "typescript": "*"
@@ -42,7 +42,7 @@
     "rollup": "^3.29.4",
     "typescript": "^5.4.2",
     "vitest": "^4.1.10",
-    "@theme-registry/refract-css": "^0.1.5"
+    "@theme-registry/refract-css": "^0.1.7"
   },
   "scripts": {
     "build": "rollup -c",