@sendbird/actionbook-core 0.10.12 → 0.10.14

package/dist/index.js

@@ -2858,6 +2858,20 @@ function collectAllVariables(structures) {
 }
 
 // src/jinja/evaluator.ts
+var ERROR_MESSAGES = {
+  UNTERMINATED_STRING: "Unfinished statement.",
+  EMPTY_EXPRESSION: "Empty expression.",
+  INVALID_EXPRESSION: "Invalid expression.",
+  unexpectedChar: (char) => `Invalid character. ${char}`,
+  expectedToken: (expected, got) => `Expected ${expected}, got ${got}`,
+  unexpectedToken: (value) => `Invalid text at the end of the statement. ${value}`,
+  unexpectedTokenInPrimary: (type, value) => {
+    if (type === "RPAREN") {
+      return "Missing closing parenthesis.";
+    }
+    return `Missing value.`;
+  }
+};
 var KEYWORDS2 = {
   and: "AND",
   or: "OR",
@@ -2892,7 +2906,7 @@ function tokenize(input) {
           i++;
         }
       }
-      if (i >= input.length) return { tokens: [], error: "Unterminated string literal" };
+      if (i >= input.length) return { tokens: [], error: ERROR_MESSAGES.UNTERMINATED_STRING };
       i++;
       tokens.push({ type: "STRING", value: str });
       continue;
@@ -2980,7 +2994,7 @@ function tokenize(input) {
       i++;
       continue;
     }
-    return { tokens: [], error: `Unexpected character: ${input[i]}` };
+    return { tokens: [], error: ERROR_MESSAGES.unexpectedChar(input[i]) };
   }
   tokens.push({ type: "EOF", value: "" });
   return { tokens };
@@ -3004,7 +3018,7 @@ var Parser = class {
   expect(type) {
     const t = this.peek();
     if (t.type !== type) {
-      throw new Error(`Expected ${type}, got ${t.type}`);
+      throw new Error(ERROR_MESSAGES.expectedToken(type, t.type));
     }
     return this.advance();
   }
@@ -3018,7 +3032,7 @@ var Parser = class {
   evaluate() {
     const result = this.orExpr();
     if (this.peek().type !== "EOF") {
-      throw new Error(`Unexpected token: ${this.peek().value}`);
+      throw new Error(ERROR_MESSAGES.unexpectedToken(this.peek().value));
     }
     return result;
   }
@@ -3130,7 +3144,7 @@ var Parser = class {
         return val;
       }
       default:
-        throw new Error(`Unexpected token in primary: ${t.type} "${t.value}"`);
+        throw new Error(ERROR_MESSAGES.unexpectedTokenInPrimary(t.type, t.value));
     }
   }
   resolveVariable(name) {
@@ -3187,13 +3201,13 @@ function validateCondition(condition) {
   if (trimmed === "") return { valid: true };
   const { tokens, error: tokenError } = tokenize(trimmed);
   if (tokenError) return { valid: false, error: tokenError };
-  if (tokens.length === 0) return { valid: false, error: "Empty expression" };
+  if (tokens.length === 0) return { valid: false, error: ERROR_MESSAGES.EMPTY_EXPRESSION };
   try {
     const parser = new Parser(tokens, /* @__PURE__ */ new Map());
     parser.evaluate();
     return { valid: true };
   } catch (e) {
-    const msg = e instanceof Error ? e.message : "Invalid expression";
+    const msg = e instanceof Error ? e.message : ERROR_MESSAGES.INVALID_EXPRESSION;
     return { valid: false, error: msg };
   }
 }
@@ -5019,12 +5033,9 @@ function actionbookToAST(manual) {
 }
 
 // src/graph/builder.ts
-var idCounter = 0;
-function nextId() {
-  return `vn_${++idCounter}`;
-}
-function resetIds() {
-  idCounter = 0;
+function createIdGenerator() {
+  let counter = 0;
+  return () => `vn_${++counter}`;
 }
 function truncate(s, max = 40) {
   const t = s.trim();
@@ -5127,10 +5138,29 @@ function merge(target, source) {
   }
   target.annotations.push(...source.annotations);
 }
+function extractFirstLabel(blocks) {
+  for (const b of blocks) {
+    if (b.type === "paragraph" || b.type === "heading") {
+      return inlineSummary2(b.content || []);
+    }
+    if (b.type === "bulletList" || b.type === "orderedList") {
+      const firstLi = b.content[0];
+      if (firstLi) {
+        const result = extractFirstLabel(firstLi.content);
+        if (result) return result;
+      }
+    }
+  }
+  return null;
+}
 function isBulletListBranching(block) {
   if (block.type !== "bulletList") return false;
   for (const li of block.content) {
     if (li.content.length > 1) return true;
+    if (li.content.length === 1) {
+      const only = li.content[0];
+      if (only.type === "bulletList" || only.type === "orderedList" || only.type === "jinjaIfBlock") return true;
+    }
     const analysis = analyzeBlock(li);
     if (analysis.hasHandoff || analysis.hasEndCall || analysis.hasToolCall || analysis.hasGoto) return true;
   }
@@ -5153,7 +5183,7 @@ function processBlocks(blocks, startBlockIndex, ctx) {
   function flushContent() {
     if (!contentAccum) return null;
     const node = {
-      id: nextId(),
+      id: ctx.nextId(),
       type: "CONTENT",
       label: truncate(contentAccum.lines[0] || ""),
       lines: contentAccum.lines,
@@ -5211,7 +5241,7 @@ function processBlocks(blocks, startBlockIndex, ctx) {
       if (analysis.hasHandoff) {
         flushContent();
         const node = {
-          id: nextId(),
+          id: ctx.nextId(),
           type: "HANDOFF",
           label: "Handoff",
           lines: [text2],
@@ -5228,7 +5258,7 @@ function processBlocks(blocks, startBlockIndex, ctx) {
       if (analysis.hasEndCall) {
         flushContent();
         const node = {
-          id: nextId(),
+          id: ctx.nextId(),
           type: "END_CONVERSATION",
           label: "End",
           lines: [text2],
@@ -5255,33 +5285,13 @@ function processBlocks(blocks, startBlockIndex, ctx) {
     if (block.type === "orderedList") {
       const items = block.content;
       for (const li of items) {
-        const text2 = inlineSummary2(li.content[0]?.content || [], 60);
-        const analysis = analyzeBlock(li);
-        if (analysis.hasHandoff) {
+        const sub = processBlocks(li.content, blockIndex, ctx);
+        if (sub.firstId) {
           flushContent();
-          const node = {
-            id: nextId(),
-            type: "HANDOFF",
-            label: "Handoff",
-            lines: [text2],
-            section: ctx.currentSection,
-            annotations: analysis.annotations,
-            sourceBlockRange: { start: blockIndex, end: blockIndex }
-          };
-          ctx.nodes.push(node);
-          for (const pid of prevLastIds) addEdge(ctx, pid, node.id, "SEQUENTIAL");
-          if (!firstId) firstId = node.id;
-          prevLastIds = [node.id];
-          continue;
-        }
-        if (!contentAccum) {
-          contentAccum = { lines: [], annotations: [], startIdx: blockIndex, endIdx: blockIndex, jumpPointIds: [] };
+          for (const pid of prevLastIds) addEdge(ctx, pid, sub.firstId, "SEQUENTIAL");
+          if (!firstId) firstId = sub.firstId;
+          prevLastIds = sub.lastIds;
         }
-        contentAccum.lines.push(text2);
-        contentAccum.annotations.push(...analysis.annotations);
-        contentAccum.endIdx = blockIndex;
-        contentAccum.jumpPointIds.push(...analysis.jumpPointIds);
-        if (analysis.hasGoto) contentAccum.gotoTarget = analysis.gotoTarget ?? void 0;
       }
       continue;
     }
@@ -5304,9 +5314,9 @@ function processBlocks(blocks, startBlockIndex, ctx) {
       for (let li = 0; li < items.length; li++) {
         const item = items[li];
         const firstBlock = item.content[0];
-        const conditionText = firstBlock && (firstBlock.type === "paragraph" || firstBlock.type === "heading") ? inlineSummary2(firstBlock.content || []) : `Branch ${li + 1}`;
+        const conditionText = firstBlock && (firstBlock.type === "paragraph" || firstBlock.type === "heading") ? inlineSummary2(firstBlock.content || []) : extractFirstLabel(item.content) || `Branch ${li + 1}`;
         const condNode = {
-          id: nextId(),
+          id: ctx.nextId(),
           type: "BRANCH",
           label: conditionText,
           lines: [conditionText],
@@ -5338,7 +5348,7 @@ function processBlocks(blocks, startBlockIndex, ctx) {
     if (block.type === "jinjaIfBlock") {
       flushContent();
       const branchNode = {
-        id: nextId(),
+        id: ctx.nextId(),
         type: "BRANCH",
         label: "Jinja If",
         lines: [],
@@ -5370,13 +5380,13 @@ function processBlocks(blocks, startBlockIndex, ctx) {
   return { firstId, lastIds: prevLastIds };
 }
 function buildVizGraph(doc2) {
-  resetIds();
   const ctx = {
     nodes: [],
     edges: [],
     anchorMap: {},
     currentSection: "_default",
-    edgeCounter: 0
+    edgeCounter: 0,
+    nextId: createIdGenerator()
   };
   const startNode = {
     id: "START",
@@ -5392,20 +5402,23 @@ function buildVizGraph(doc2) {
   if (result.firstId) {
     addEdge(ctx, "START", result.firstId, "SEQUENTIAL");
   }
-  for (const lastId of result.lastIds) {
-    const lastNode = ctx.nodes.find((n) => n.id === lastId);
-    if (lastNode && lastNode.type !== "HANDOFF" && lastNode.type !== "END_CONVERSATION") {
-      const endNode = {
-        id: nextId(),
-        type: "END_CONVERSATION",
-        label: "End",
-        lines: [],
-        section: lastNode.section,
-        annotations: [],
-        sourceBlockRange: { start: -1, end: -1 }
-      };
-      ctx.nodes.push(endNode);
-      addEdge(ctx, lastId, endNode.id, "SEQUENTIAL");
+  const danglingIds = result.lastIds.filter((id) => {
+    const n = ctx.nodes.find((node) => node.id === id);
+    return n && n.type !== "HANDOFF" && n.type !== "END_CONVERSATION";
+  });
+  if (danglingIds.length > 0) {
+    const endNode = {
+      id: ctx.nextId(),
+      type: "END_CONVERSATION",
+      label: "End",
+      lines: [],
+      section: ctx.currentSection,
+      annotations: [],
+      sourceBlockRange: { start: -1, end: -1 }
+    };
+    ctx.nodes.push(endNode);
+    for (const id of danglingIds) {
+      addEdge(ctx, id, endNode.id, "SEQUENTIAL");
     }
   }
   for (const node of ctx.nodes) {
@@ -5414,7 +5427,7 @@ function buildVizGraph(doc2) {
     if (targetNodeId) {
       addEdge(ctx, node.id, targetNodeId, "GOTO", node.gotoTarget);
     } else {
-      addEdge(ctx, node.id, node.id, "BROKEN_REF", node.gotoTarget);
+      node.annotations = [...node.annotations, { kind: "jump_point", label: `\u26A0 ${node.gotoTarget} (broken)` }];
     }
   }
   const sectionMap = /* @__PURE__ */ new Map();