forgedock 1.0.10 → 1.0.11

package/bin/init-enrich-api.mjs

@@ -111,12 +111,22 @@ export function parseEnrichedDraft(output, draft) {
           // each section must be a plain non-null object. The typeof+null guard
           // rejects null and primitives; Array.isArray() additionally rejects
           // arrays since typeof [] === 'object' would otherwise pass them.
+          const isPlainObject = (v) =>
+            typeof v === "object" && v !== null && !Array.isArray(v);
           if (
-            typeof enriched.project !== "object" || enriched.project === null || Array.isArray(enriched.project) ||
-            typeof enriched.paths !== "object" || enriched.paths === null || Array.isArray(enriched.paths) ||
-            typeof enriched.branches !== "object" || enriched.branches === null || Array.isArray(enriched.branches) ||
-            typeof enriched.meta !== "object" || enriched.meta === null || Array.isArray(enriched.meta)
+            !isPlainObject(enriched.project) ||
+            !isPlainObject(enriched.paths) ||
+            !isPlainObject(enriched.branches) ||
+            !isPlainObject(enriched.meta)
           ) {
+            if (process.env.FORGEDOCK_DEBUG) {
+              const failing = ["project", "paths", "branches", "meta"]
+                .filter((k) => !isPlainObject(enriched[k]))
+                .join(", ");
+              console.error(
+                `  ${dim("[debug]")} api enrichment: parseEnrichedDraft rejected LLM output — missing/invalid sections: ${failing}. Falling back to baseline draft.`,
+              );
+            }
             return draft;
           }
           return enriched;

package/bin/tui.mjs

@@ -743,11 +743,14 @@ export function truncateVisible(str, maxWidth) {
   if (remaining > 0) {
     result += str.slice(lastIndex, lastIndex + remaining);
   }
-  // If truncation occurred and the result contains any ANSI sequences,
-  // append a full reset to prevent color from bleeding into adjacent columns.
-  // Truncation is detected by checking whether the original string has more
-  // visible characters than maxWidth (i.e. stripAnsi(str).length > maxWidth).
-  if (stripAnsi(str).length > maxWidth && result.includes("\x1b[")) {
+  // If truncation occurred (or visible length exactly equals maxWidth) and the
+  // result contains any ANSI sequences, append a full reset to prevent color from
+  // bleeding into adjacent columns.  The guard uses >= rather than > to cover the
+  // exact-boundary case: when visible === maxWidth the loop guard (visible < maxWidth)
+  // suppresses the original trailing RESET token, so the post-loop guard must fire
+  // to close any open sequences.  Inputs shorter than maxWidth are unaffected because
+  // stripAnsi(str).length < maxWidth makes the condition false.
+  if (stripAnsi(str).length >= maxWidth && result.includes("\x1b[")) {
     result += "\x1b[0m";
   }
   return result;