@karmaniverous/jeeves-meta 0.10.1 → 0.11.0

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

@@ -46,11 +46,11 @@ const metaConfigSchema = z.object({
     /** Maximum lines of context to include in subprocess prompts. */
     maxLines: z.number().int().min(50).default(500),
     /** Architect subprocess timeout in seconds. */
-    architectTimeout: z.number().int().min(30).default(120),
+    architectTimeout: z.number().int().min(30).default(180),
     /** Builder subprocess timeout in seconds. */
-    builderTimeout: z.number().int().min(60).default(600),
+    builderTimeout: z.number().int().min(60).default(360),
     /** Critic subprocess timeout in seconds. */
-    criticTimeout: z.number().int().min(30).default(300),
+    criticTimeout: z.number().int().min(30).default(240),
     /** Thinking level for spawned synthesis sessions. */
     thinking: z.string().default('low'),
     /** Resolved architect system prompt text. Falls back to built-in default. */
@@ -2151,6 +2151,17 @@ async function synthesizeNode(node, currentMeta, config, executor, watcher, onPr
     const steerChanged = hasSteerChanged(currentMeta._steer, latestArchive?._steer, Boolean(latestArchive));
     // Step 7: Compute context (includes scope files and delta files)
     const ctx = await buildContextPackage(node, currentMeta, watcher, logger);
+    // Skip empty-scope entities that have no prior content.
+    // Without scope files, child metas, or cross-refs there is nothing for
+    // the architect/builder to work with and the cycle will either time out
+    // or produce empty output.
+    const hasScope = ctx.scopeFiles.length > 0 ||
+        Object.keys(ctx.childMetas).length > 0 ||
+        Object.keys(ctx.crossRefMetas).length > 0;
+    if (!hasScope && !currentMeta._content) {
+        logger?.debug({ path: node.ownerPath }, 'Skipping empty-scope entity');
+        return { synthesized: false };
+    }
     // Step 5 (deferred): Structure hash from context scope files
     const newStructureHash = computeStructureHash(ctx.scopeFiles);
     const structureChanged = newStructureHash !== currentMeta._structureHash;
@@ -2447,21 +2458,41 @@ function formatSeconds(durationMs) {
 function titleCasePhase(phase) {
     return phase.charAt(0).toUpperCase() + phase.slice(1);
 }
-/** Build a link to the entity's meta.json output file. */
-function buildEntityLink(path, serverBaseUrl) {
-    // Normalize backslashes, then convert drive letter to URL path segment
+/**
+ * URL-encode each path segment individually so that spaces and special
+ * characters are safe while preserving the `/` separators.
+ */
+function encodePathSegments(p) {
+    return p
+        .split('/')
+        .map((seg) => encodeURIComponent(seg))
+        .join('/');
+}
+/** Build a link (or plain path) to the owner directory. */
+function buildDirectoryLink(path, serverBaseUrl) {
+    const normalized = normalizePath(path).replace(/^([A-Za-z]):/, '/$1');
+    const encoded = encodePathSegments(normalized);
+    if (!serverBaseUrl)
+        return normalized;
+    const base = serverBaseUrl.replace(/\/+$/, '');
+    return `${base}/path${encoded}`;
+}
+/** Build a link (or plain path) to the entity's meta.json output file. */
+function buildMetaJsonLink(path, serverBaseUrl) {
     const normalized = normalizePath(path).replace(/^([A-Za-z]):/, '/$1');
     const metaJsonPath = `${normalized}/.meta/meta.json`;
+    const encoded = encodePathSegments(metaJsonPath);
     if (!serverBaseUrl)
         return metaJsonPath;
     const base = serverBaseUrl.replace(/\/+$/, '');
-    return `${base}/path${metaJsonPath}`;
+    return `${base}/path${encoded}`;
 }
 function formatProgressEvent(event, serverBaseUrl) {
-    const pathDisplay = buildEntityLink(event.path, serverBaseUrl);
     switch (event.type) {
-        case 'synthesis_start':
-            return `🔬 Started meta synthesis: ${pathDisplay}`;
+        case 'synthesis_start': {
+            const dirLink = buildDirectoryLink(event.path, serverBaseUrl);
+            return `🔬 Started meta synthesis: ${dirLink}`;
+        }
         case 'phase_start': {
             if (!event.phase) {
                 return '  ⚙️ Phase started';
@@ -2475,16 +2506,18 @@ function formatProgressEvent(event, serverBaseUrl) {
             return `  ✅ ${phase} complete (${formatNumber(tokens)} tokens / ${duration})`;
         }
         case 'synthesis_complete': {
+            const metaLink = buildMetaJsonLink(event.path, serverBaseUrl);
             const tokens = event.tokens ?? 0;
             const duration = event.durationMs !== undefined
                 ? formatSeconds(event.durationMs)
                 : '0.0s';
-            return `✅ Completed: ${pathDisplay} (${formatNumber(tokens)} tokens / ${duration})`;
+            return `✅ Completed: ${metaLink} (${formatNumber(tokens)} tokens / ${duration})`;
         }
         case 'error': {
+            const dirLink = buildDirectoryLink(event.path, serverBaseUrl);
             const phase = event.phase ? `${titleCasePhase(event.phase)} ` : '';
             const error = event.error ?? 'Unknown error';
-            return `❌ Synthesis failed at ${phase}phase: ${pathDisplay}\n   Error: ${error}`;
+            return `❌ Synthesis failed at ${phase}phase: ${dirLink}\n   Error: ${error}`;
         }
         default: {
             return 'Unknown progress event';
@@ -11087,6 +11120,7 @@ async function startService(config, configPath) {
                 await progress.report({
                     type: 'error',
                     path: ownerPath,
+                    phase: result.error.step,
                     error: result.error.message,
                 });
             }

package/dist/index.js

@@ -254,11 +254,11 @@ const metaConfigSchema = z.object({
     /** Maximum lines of context to include in subprocess prompts. */
     maxLines: z.number().int().min(50).default(500),
     /** Architect subprocess timeout in seconds. */
-    architectTimeout: z.number().int().min(30).default(120),
+    architectTimeout: z.number().int().min(30).default(180),
     /** Builder subprocess timeout in seconds. */
-    builderTimeout: z.number().int().min(60).default(600),
+    builderTimeout: z.number().int().min(60).default(360),
     /** Critic subprocess timeout in seconds. */
-    criticTimeout: z.number().int().min(30).default(300),
+    criticTimeout: z.number().int().min(30).default(240),
     /** Thinking level for spawned synthesis sessions. */
     thinking: z.string().default('low'),
     /** Resolved architect system prompt text. Falls back to built-in default. */
@@ -2143,6 +2143,17 @@ async function synthesizeNode(node, currentMeta, config, executor, watcher, onPr
     const steerChanged = hasSteerChanged(currentMeta._steer, latestArchive?._steer, Boolean(latestArchive));
     // Step 7: Compute context (includes scope files and delta files)
     const ctx = await buildContextPackage(node, currentMeta, watcher, logger);
+    // Skip empty-scope entities that have no prior content.
+    // Without scope files, child metas, or cross-refs there is nothing for
+    // the architect/builder to work with and the cycle will either time out
+    // or produce empty output.
+    const hasScope = ctx.scopeFiles.length > 0 ||
+        Object.keys(ctx.childMetas).length > 0 ||
+        Object.keys(ctx.crossRefMetas).length > 0;
+    if (!hasScope && !currentMeta._content) {
+        logger?.debug({ path: node.ownerPath }, 'Skipping empty-scope entity');
+        return { synthesized: false };
+    }
     // Step 5 (deferred): Structure hash from context scope files
     const newStructureHash = computeStructureHash(ctx.scopeFiles);
     const structureChanged = newStructureHash !== currentMeta._structureHash;
@@ -2439,21 +2450,41 @@ function formatSeconds(durationMs) {
 function titleCasePhase(phase) {
     return phase.charAt(0).toUpperCase() + phase.slice(1);
 }
-/** Build a link to the entity's meta.json output file. */
-function buildEntityLink(path, serverBaseUrl) {
-    // Normalize backslashes, then convert drive letter to URL path segment
+/**
+ * URL-encode each path segment individually so that spaces and special
+ * characters are safe while preserving the `/` separators.
+ */
+function encodePathSegments(p) {
+    return p
+        .split('/')
+        .map((seg) => encodeURIComponent(seg))
+        .join('/');
+}
+/** Build a link (or plain path) to the owner directory. */
+function buildDirectoryLink(path, serverBaseUrl) {
+    const normalized = normalizePath(path).replace(/^([A-Za-z]):/, '/$1');
+    const encoded = encodePathSegments(normalized);
+    if (!serverBaseUrl)
+        return normalized;
+    const base = serverBaseUrl.replace(/\/+$/, '');
+    return `${base}/path${encoded}`;
+}
+/** Build a link (or plain path) to the entity's meta.json output file. */
+function buildMetaJsonLink(path, serverBaseUrl) {
     const normalized = normalizePath(path).replace(/^([A-Za-z]):/, '/$1');
     const metaJsonPath = `${normalized}/.meta/meta.json`;
+    const encoded = encodePathSegments(metaJsonPath);
     if (!serverBaseUrl)
         return metaJsonPath;
     const base = serverBaseUrl.replace(/\/+$/, '');
-    return `${base}/path${metaJsonPath}`;
+    return `${base}/path${encoded}`;
 }
 function formatProgressEvent(event, serverBaseUrl) {
-    const pathDisplay = buildEntityLink(event.path, serverBaseUrl);
     switch (event.type) {
-        case 'synthesis_start':
-            return `🔬 Started meta synthesis: ${pathDisplay}`;
+        case 'synthesis_start': {
+            const dirLink = buildDirectoryLink(event.path, serverBaseUrl);
+            return `🔬 Started meta synthesis: ${dirLink}`;
+        }
         case 'phase_start': {
             if (!event.phase) {
                 return '  ⚙️ Phase started';
@@ -2467,16 +2498,18 @@ function formatProgressEvent(event, serverBaseUrl) {
             return `  ✅ ${phase} complete (${formatNumber(tokens)} tokens / ${duration})`;
         }
         case 'synthesis_complete': {
+            const metaLink = buildMetaJsonLink(event.path, serverBaseUrl);
             const tokens = event.tokens ?? 0;
             const duration = event.durationMs !== undefined
                 ? formatSeconds(event.durationMs)
                 : '0.0s';
-            return `✅ Completed: ${pathDisplay} (${formatNumber(tokens)} tokens / ${duration})`;
+            return `✅ Completed: ${metaLink} (${formatNumber(tokens)} tokens / ${duration})`;
         }
         case 'error': {
+            const dirLink = buildDirectoryLink(event.path, serverBaseUrl);
             const phase = event.phase ? `${titleCasePhase(event.phase)} ` : '';
             const error = event.error ?? 'Unknown error';
-            return `❌ Synthesis failed at ${phase}phase: ${pathDisplay}\n   Error: ${error}`;
+            return `❌ Synthesis failed at ${phase}phase: ${dirLink}\n   Error: ${error}`;
         }
         default: {
             return 'Unknown progress event';
@@ -11079,6 +11112,7 @@ async function startService(config, configPath) {
                 await progress.report({
                     type: 'error',
                     path: ownerPath,
+                    phase: result.error.step,
                     error: result.error.message,
                 });
             }

package/package.json

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