@karmaniverous/jeeves-meta 0.11.0 → 0.11.2

package/dist/cli/jeeves-meta/index.js

@@ -834,6 +834,32 @@ async function listMetas(config, watcher) {
     };
 }
 
+/**
+ * Escape special glob characters in a path so it can be used as a literal
+ * prefix in glob patterns.
+ *
+ * Glob metacharacters `* ? [ ] { } ( ) !` are escaped with a backslash so
+ * that paths containing parentheses (e.g. Slack channel IDs) or other
+ * special characters are matched literally by the watcher's walk endpoint.
+ *
+ * @module escapeGlob
+ */
+/**
+ * Escape glob metacharacters in a string using character-class wrapping.
+ *
+ * Backslash escaping (`\(`) does not work reliably on Windows where `\` is
+ * the path separator. Instead, each metacharacter is wrapped in a character
+ * class (e.g. `(` → `[(]`) which is universally supported by glob libraries.
+ *
+ * Square brackets themselves are escaped as `[[]` and `[]]`.
+ *
+ * @param s - Raw path string.
+ * @returns String with glob metacharacters wrapped in character classes.
+ */
+function escapeGlob(s) {
+    return s.replace(/[*?[\]{}()!]/g, (ch) => `[${ch}]`);
+}
+
 /**
  * Filter file paths by modification time.
  *
@@ -934,7 +960,7 @@ function filterInScope(node, files) {
  */
 async function getScopeFiles(node, watcher, logger) {
     const walkStart = Date.now();
-    const rawFiles = await watcher.walk([`${node.ownerPath}/**`]);
+    const rawFiles = await watcher.walk([`${escapeGlob(node.ownerPath)}/**`]);
     const allFiles = rawFiles.map(normalizePath);
     const scopeFiles = filterInScope(node, allFiles);
     logger?.debug({
@@ -1746,7 +1772,7 @@ async function buildMinimalNode(metaPath, watcher) {
     // We include only *direct* children (nearest descendants in the ownership tree)
     // to match the ownership semantics used elsewhere.
     const rawMetaJsonPaths = await watcher.walk([
-        `${ownerPath}/**/.meta/meta.json`,
+        `${escapeGlob(ownerPath)}/**/.meta/meta.json`,
     ]);
     const candidateMetaPaths = [
         ...new Set(rawMetaJsonPaths.map((p) => normalizePath(dirname(p)))),
@@ -1882,7 +1908,7 @@ function discoverStalestPath(candidates, depthWeight) {
 async function isStale(scopePrefix, meta, watcher) {
     if (!meta._generatedAt)
         return true; // Never synthesized = stale
-    const files = await watcher.walk([`${scopePrefix}/**`]);
+    const files = await watcher.walk([`${escapeGlob(scopePrefix)}/**`]);
     return hasModifiedAfter(files, new Date(meta._generatedAt).getTime());
 }
 /** Maximum staleness for never-synthesized metas (1 year in seconds). */
@@ -2159,6 +2185,14 @@ async function synthesizeNode(node, currentMeta, config, executor, watcher, onPr
         Object.keys(ctx.childMetas).length > 0 ||
         Object.keys(ctx.crossRefMetas).length > 0;
     if (!hasScope && !currentMeta._content) {
+        // Bump _generatedAt so this entity doesn't keep winning the staleness
+        // race every cycle. It will be re-evaluated when files appear.
+        // Uses lock-staging for atomic write consistency.
+        currentMeta._generatedAt = new Date().toISOString();
+        const lockPath = join(node.metaPath, '.lock');
+        const metaJsonPath = join(node.metaPath, 'meta.json');
+        await writeFile(lockPath, JSON.stringify(currentMeta, null, 2));
+        await copyFile(lockPath, metaJsonPath);
         logger?.debug({ path: node.ownerPath }, 'Skipping empty-scope entity');
         return { synthesized: false };
     }
@@ -11111,6 +11145,11 @@ async function startService(config, configPath) {
             // orchestrate() always returns exactly one result
             const result = results[0];
             const durationMs = Date.now() - startMs;
+            if (!result.synthesized) {
+                // Entity was skipped (e.g. empty scope) — no progress to report.
+                logger.debug({ path: ownerPath }, 'Synthesis skipped');
+                return;
+            }
             // Update stats
             stats.totalSyntheses++;
             stats.lastCycleDurationMs = durationMs;

package/dist/index.js

@@ -826,6 +826,32 @@ async function listMetas(config, watcher) {
     };
 }
 
+/**
+ * Escape special glob characters in a path so it can be used as a literal
+ * prefix in glob patterns.
+ *
+ * Glob metacharacters `* ? [ ] { } ( ) !` are escaped with a backslash so
+ * that paths containing parentheses (e.g. Slack channel IDs) or other
+ * special characters are matched literally by the watcher's walk endpoint.
+ *
+ * @module escapeGlob
+ */
+/**
+ * Escape glob metacharacters in a string using character-class wrapping.
+ *
+ * Backslash escaping (`\(`) does not work reliably on Windows where `\` is
+ * the path separator. Instead, each metacharacter is wrapped in a character
+ * class (e.g. `(` → `[(]`) which is universally supported by glob libraries.
+ *
+ * Square brackets themselves are escaped as `[[]` and `[]]`.
+ *
+ * @param s - Raw path string.
+ * @returns String with glob metacharacters wrapped in character classes.
+ */
+function escapeGlob(s) {
+    return s.replace(/[*?[\]{}()!]/g, (ch) => `[${ch}]`);
+}
+
 /**
  * Filter file paths by modification time.
  *
@@ -926,7 +952,7 @@ function filterInScope(node, files) {
  */
 async function getScopeFiles(node, watcher, logger) {
     const walkStart = Date.now();
-    const rawFiles = await watcher.walk([`${node.ownerPath}/**`]);
+    const rawFiles = await watcher.walk([`${escapeGlob(node.ownerPath)}/**`]);
     const allFiles = rawFiles.map(normalizePath);
     const scopeFiles = filterInScope(node, allFiles);
     logger?.debug({
@@ -1738,7 +1764,7 @@ async function buildMinimalNode(metaPath, watcher) {
     // We include only *direct* children (nearest descendants in the ownership tree)
     // to match the ownership semantics used elsewhere.
     const rawMetaJsonPaths = await watcher.walk([
-        `${ownerPath}/**/.meta/meta.json`,
+        `${escapeGlob(ownerPath)}/**/.meta/meta.json`,
     ]);
     const candidateMetaPaths = [
         ...new Set(rawMetaJsonPaths.map((p) => normalizePath(dirname(p)))),
@@ -1874,7 +1900,7 @@ function discoverStalestPath(candidates, depthWeight) {
 async function isStale(scopePrefix, meta, watcher) {
     if (!meta._generatedAt)
         return true; // Never synthesized = stale
-    const files = await watcher.walk([`${scopePrefix}/**`]);
+    const files = await watcher.walk([`${escapeGlob(scopePrefix)}/**`]);
     return hasModifiedAfter(files, new Date(meta._generatedAt).getTime());
 }
 /** Maximum staleness for never-synthesized metas (1 year in seconds). */
@@ -2151,6 +2177,14 @@ async function synthesizeNode(node, currentMeta, config, executor, watcher, onPr
         Object.keys(ctx.childMetas).length > 0 ||
         Object.keys(ctx.crossRefMetas).length > 0;
     if (!hasScope && !currentMeta._content) {
+        // Bump _generatedAt so this entity doesn't keep winning the staleness
+        // race every cycle. It will be re-evaluated when files appear.
+        // Uses lock-staging for atomic write consistency.
+        currentMeta._generatedAt = new Date().toISOString();
+        const lockPath = join(node.metaPath, '.lock');
+        const metaJsonPath = join(node.metaPath, 'meta.json');
+        await writeFile(lockPath, JSON.stringify(currentMeta, null, 2));
+        await copyFile(lockPath, metaJsonPath);
         logger?.debug({ path: node.ownerPath }, 'Skipping empty-scope entity');
         return { synthesized: false };
     }
@@ -11103,6 +11137,11 @@ async function startService(config, configPath) {
             // orchestrate() always returns exactly one result
             const result = results[0];
             const durationMs = Date.now() - startMs;
+            if (!result.synthesized) {
+                // Entity was skipped (e.g. empty scope) — no progress to report.
+                logger.debug({ path: ownerPath }, 'Synthesis skipped');
+                return;
+            }
             // Update stats
             stats.totalSyntheses++;
             stats.lastCycleDurationMs = durationMs;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@karmaniverous/jeeves-meta",
-  "version": "0.11.0",
+  "version": "0.11.2",
   "author": "Jason Williscroft",
   "description": "Fastify HTTP service for the Jeeves Meta synthesis engine",
   "license": "BSD-3-Clause",