@bike4mind/cli 0.2.3-feat-questmaster-gpt5-function-calling.17055 → 0.2.3-feat-questmaster-gpt5-function-calling.17057

package/dist/{chunk-KDBSH6HC.js → chunk-6AJODFPC.js}

@@ -7,7 +7,7 @@ import {
   getSettingsMap,
   getSettingsValue,
   secureParameters
-} from "./chunk-NH74I6GX.js";
+} from "./chunk-LYNNVNEC.js";
 import {
   KnowledgeType,
   SupportedFabFileMimeTypes

package/dist/{chunk-O2622ACK.js → chunk-LEG2UM5Q.js}

@@ -2,7 +2,7 @@
 import {
   BadRequestError,
   secureParameters
-} from "./chunk-NH74I6GX.js";
+} from "./chunk-LYNNVNEC.js";
 import {
   GenericCreditDeductTransaction,
   ImageEditUsageTransaction,

package/dist/{chunk-NH74I6GX.js → chunk-LYNNVNEC.js}

@@ -7985,6 +7985,13 @@ function extractQuestMasterData(reply, options = {}) {
 }
 
 // ../../b4m-core/packages/utils/dist/src/questMasterToolSchema.js
+var MAX_GOAL_LENGTH = 500;
+var MAX_TITLE_LENGTH = 200;
+var MAX_DESCRIPTION_LENGTH = 2e3;
+var MAX_TAG_LENGTH = 50;
+var MAX_QUESTS = 20;
+var MAX_SUBQUESTS_PER_QUEST = 20;
+var MAX_TAGS = 10;
 var GPT5_MODELS_WITH_TOOL_SUPPORT = [
   ChatModels.GPT5,
   ChatModels.GPT5_MINI,
@@ -8070,6 +8077,9 @@ ALWAYS call this function to respond. Do NOT respond with plain text.`,
   }
 };
 function parseQuestPlanFunctionCall(args) {
+  if (!args || typeof args !== "string" || args.trim() === "") {
+    throw new Error("Function call arguments are empty or invalid");
+  }
   let parsed;
   try {
     parsed = JSON.parse(args);
@@ -8080,8 +8090,11 @@ function parseQuestPlanFunctionCall(args) {
     throw new Error("Function call arguments must be an object");
   }
   const result = parsed;
-  if (!result.goal || typeof result.goal !== "string") {
-    throw new Error("Function call missing required field: goal");
+  if (!result.goal || typeof result.goal !== "string" || result.goal.trim() === "") {
+    throw new Error("Function call missing required field: goal (must be non-empty string)");
+  }
+  if (result.goal.length > MAX_GOAL_LENGTH) {
+    throw new Error(`Goal exceeds maximum length of ${MAX_GOAL_LENGTH} characters`);
   }
   if (!Array.isArray(result.quests)) {
     throw new Error("Function call missing required field: quests (must be an array)");
@@ -8089,45 +8102,82 @@ function parseQuestPlanFunctionCall(args) {
   if (result.quests.length === 0) {
     throw new Error("Function call returned empty quests array");
   }
+  if (result.quests.length > MAX_QUESTS) {
+    throw new Error(`Too many quests: ${result.quests.length} (max ${MAX_QUESTS})`);
+  }
   for (let i = 0; i < result.quests.length; i++) {
     const quest = result.quests[i];
-    if (!quest.id || typeof quest.id !== "string") {
-      throw new Error(`Quest ${i} missing required field: id`);
+    if (!quest || typeof quest !== "object") {
+      throw new Error(`Quest ${i} is not a valid object`);
+    }
+    const questObj = quest;
+    if (!questObj.id || typeof questObj.id !== "string" || questObj.id.trim() === "") {
+      throw new Error(`Quest ${i} missing required field: id (must be non-empty string)`);
+    }
+    if (!questObj.title || typeof questObj.title !== "string" || questObj.title.trim() === "") {
+      throw new Error(`Quest ${i} missing required field: title (must be non-empty string)`);
+    }
+    if (questObj.title.length > MAX_TITLE_LENGTH) {
+      throw new Error(`Quest ${i} title exceeds maximum length of ${MAX_TITLE_LENGTH} characters`);
     }
-    if (!quest.title || typeof quest.title !== "string") {
-      throw new Error(`Quest ${i} missing required field: title`);
+    if (!questObj.description || typeof questObj.description !== "string" || questObj.description.trim() === "") {
+      throw new Error(`Quest ${i} missing required field: description (must be non-empty string)`);
     }
-    if (!quest.description || typeof quest.description !== "string") {
-      throw new Error(`Quest ${i} missing required field: description`);
+    if (questObj.description.length > MAX_DESCRIPTION_LENGTH) {
+      throw new Error(`Quest ${i} description exceeds maximum length of ${MAX_DESCRIPTION_LENGTH} characters`);
     }
-    if (!quest.complexity || !["Easy", "Medium", "Hard"].includes(quest.complexity)) {
-      throw new Error(`Quest ${i} has invalid complexity: ${quest.complexity} (must be Easy, Medium, or Hard)`);
+    if (!questObj.complexity || !["Easy", "Medium", "Hard"].includes(questObj.complexity)) {
+      throw new Error(`Quest ${i} has invalid complexity: ${questObj.complexity} (must be Easy, Medium, or Hard)`);
     }
-    if (!Array.isArray(quest.subQuests)) {
-      throw new Error(`Quest "${quest.id}" missing required field: subQuests (must be an array)`);
+    if (!Array.isArray(questObj.subQuests)) {
+      throw new Error(`Quest "${questObj.id}" missing required field: subQuests (must be an array)`);
     }
-    if (quest.subQuests.length === 0) {
-      throw new Error(`Quest "${quest.id}" has no subQuests (minimum 2 required)`);
+    if (questObj.subQuests.length === 0) {
+      throw new Error(`Quest "${questObj.id}" has no subQuests (minimum 2 required)`);
     }
-    for (let j = 0; j < quest.subQuests.length; j++) {
-      const subQuest = quest.subQuests[j];
-      if (!subQuest.id || typeof subQuest.id !== "string") {
-        throw new Error(`Quest "${quest.id}" subQuest ${j} missing required field: id`);
+    if (questObj.subQuests.length > MAX_SUBQUESTS_PER_QUEST) {
+      throw new Error(`Quest "${questObj.id}" has too many subQuests: ${questObj.subQuests.length} (max ${MAX_SUBQUESTS_PER_QUEST})`);
+    }
+    for (let j = 0; j < questObj.subQuests.length; j++) {
+      const subQuest = questObj.subQuests[j];
+      if (!subQuest || typeof subQuest !== "object") {
+        throw new Error(`Quest "${questObj.id}" subQuest ${j} is not a valid object`);
+      }
+      const subQuestObj = subQuest;
+      if (!subQuestObj.id || typeof subQuestObj.id !== "string" || subQuestObj.id.trim() === "") {
+        throw new Error(`Quest "${questObj.id}" subQuest ${j} missing required field: id (must be non-empty string)`);
+      }
+      if (!subQuestObj.title || typeof subQuestObj.title !== "string" || subQuestObj.title.trim() === "") {
+        throw new Error(`Quest "${questObj.id}" subQuest ${j} missing required field: title (must be non-empty string)`);
       }
-      if (!subQuest.title || typeof subQuest.title !== "string") {
-        throw new Error(`Quest "${quest.id}" subQuest ${j} missing required field: title`);
+      if (subQuestObj.title.length > MAX_TITLE_LENGTH) {
+        throw new Error(`Quest "${questObj.id}" subQuest ${j} title exceeds maximum length of ${MAX_TITLE_LENGTH} characters`);
       }
     }
   }
-  if (result.tags !== void 0 && !Array.isArray(result.tags)) {
-    throw new Error("Function call field tags must be an array if provided");
+  if (result.tags !== void 0) {
+    if (!Array.isArray(result.tags)) {
+      throw new Error("Function call field tags must be an array if provided");
+    }
+    if (result.tags.length > MAX_TAGS) {
+      throw new Error(`Too many tags: ${result.tags.length} (max ${MAX_TAGS})`);
+    }
+    for (let i = 0; i < result.tags.length; i++) {
+      if (typeof result.tags[i] !== "string") {
+        throw new Error(`Function call field tags[${i}] must be a string`);
+      }
+      if (result.tags[i].length > MAX_TAG_LENGTH) {
+        throw new Error(`Tag ${i} exceeds maximum length of ${MAX_TAG_LENGTH} characters`);
+      }
+    }
   }
   return result;
 }
 
 // ../../b4m-core/packages/utils/dist/src/questMaster.js
-var sanitizeId = (id) => {
-  return id.replace(/\s+/g, "-").replace(/[^a-zA-Z0-9_.-]/g, "").replace(/-+/g, "-").replace(/^-|-$/g, "");
+var sanitizeId = (id, fallback) => {
+  const sanitized = id.replace(/\s+/g, "-").replace(/[^a-zA-Z0-9_.-]/g, "").replace(/-+/g, "-").replace(/^-|-$/g, "");
+  return sanitized || fallback;
 };
 var QuestMaster = class {
   llm;
@@ -8157,6 +8207,9 @@ var QuestMaster = class {
   }
   async processQuestPlan(input) {
     try {
+      if (!this.quest) {
+        throw new Error("No quest context available");
+      }
       await this.onStatusUpdate(this.quest, "Processing quest plan...");
       const defaultResponse = {
         type: "narrative",
@@ -8174,11 +8227,8 @@ var QuestMaster = class {
           }
         ]
       };
-      if (!this.quest) {
-        throw new Error("No quest context available");
-      }
       let questResponse;
-      if (typeof input === "object") {
+      if (input !== null && typeof input === "object") {
         this.logger.log("Processing quest plan from function calling result");
         questResponse = {
           type: "quest_plan",
@@ -8293,7 +8343,7 @@ var QuestMaster = class {
                   title: item.quest,
                   description: item.details,
                   complexity: this.determineComplexity(item.details),
-                  status: "Not Started",
+                  status: "not_started",
                   subQuests: []
                   // This will trigger a retry since we require subquests
                 }))
@@ -8342,18 +8392,21 @@ var QuestMaster = class {
         goal: questResponse.meta.goal,
         userId: this.userId,
         tags: questResponse.meta.tags || [],
-        quests: questResponse.quests.map((quest) => ({
-          id: sanitizeId(quest.id),
-          title: quest.title,
-          description: quest.description,
-          complexity: quest.complexity,
-          subQuests: quest.subQuests.map((sub) => ({
-            id: sanitizeId(sub.id),
-            title: sub.title,
-            status: sub.status
-            // questId intentionally omitted - will be set when task is started and linked to a chat message
-          }))
-        }))
+        quests: questResponse.quests.map((quest, questIndex) => {
+          const questId = sanitizeId(quest.id, `quest-${questIndex + 1}`);
+          return {
+            id: questId,
+            title: quest.title,
+            description: quest.description,
+            complexity: quest.complexity,
+            subQuests: quest.subQuests.map((sub, subIndex) => ({
+              id: sanitizeId(sub.id, `${questId}-sub-${subIndex + 1}`),
+              title: sub.title,
+              status: sub.status
+              // questId intentionally omitted - will be set when task is started and linked to a chat message
+            }))
+          };
+        })
       });
       this.quest.questMasterPlanId = questMasterPlan.id;
       this.quest.reply = null;
@@ -8404,7 +8457,9 @@ var QuestMaster = class {
       }
       throw error;
     } finally {
-      await this.onStatusUpdate(this.quest, null);
+      if (this.quest) {
+        await this.onStatusUpdate(this.quest, null);
+      }
     }
   }
   validateQuestResponse(response) {
@@ -8563,8 +8618,8 @@ Guidelines:
       toolSchema: createQuestPlanToolSchema
     };
     const options = {
-      temperature: retryCount > 0 ? 0.7 : 0.9,
-      // Lower temperature on retries
+      temperature: retryCount > 0 ? 0.5 : 0.7,
+      // Lower temperature for structured JSON outputs; even lower on retries
       n: 1,
       stream: false,
       tools: [toolDef],
@@ -8583,7 +8638,12 @@ Guidelines:
           const toolCall = completionInfo.toolsUsed[0];
           if (toolCall.name === "create_quest_plan" && toolCall.arguments) {
             this.logger.log("GPT-5 function call received, parsing arguments...");
-            functionResult = parseQuestPlanFunctionCall(toolCall.arguments);
+            try {
+              functionResult = parseQuestPlanFunctionCall(toolCall.arguments);
+            } catch (parseError) {
+              lastError = parseError instanceof Error ? parseError : new Error(String(parseError));
+              this.logger.error("Failed to parse function call arguments:", parseError);
+            }
           }
         }
       });
@@ -8600,9 +8660,13 @@ Guidelines:
     }
     const result = functionResult;
     const questsWithoutSubquests = result.quests.filter((q) => !q.subQuests || q.subQuests.length === 0);
-    if (questsWithoutSubquests.length > 0 && retryCount < MAX_RETRIES) {
-      this.logger.warn(`Some quests missing subquests: ${questsWithoutSubquests.map((q) => q.id).join(", ")}, retrying...`);
-      return this.createQuestPlanWithFunctionCalling(model, prompt, retryCount + 1);
+    if (questsWithoutSubquests.length > 0) {
+      if (retryCount < MAX_RETRIES) {
+        this.logger.warn(`Some quests missing subquests: ${questsWithoutSubquests.map((q) => q.id).join(", ")}, retrying...`);
+        return this.createQuestPlanWithFunctionCalling(model, prompt, retryCount + 1);
+      } else {
+        this.logger.warn(`Returning quest plan with ${questsWithoutSubquests.length} quests missing subquests after max retries (${MAX_RETRIES}): ${questsWithoutSubquests.map((q) => q.id).join(", ")}`);
+      }
     }
     this.logger.log("Successfully received GPT-5 function call result:", {
       goal: result.goal,
@@ -8613,6 +8677,9 @@ Guidelines:
   }
   async createQuestPlan(model, prompt) {
     try {
+      if (!this.quest) {
+        throw new Error("No quest context available");
+      }
       await this.onStatusUpdate(this.quest, "Creating quest plan...");
       if (isGPT5ModelWithToolSupport(model)) {
         this.logger.log(`Using function calling approach for GPT-5 model: ${model}`);

package/dist/{chunk-44WCHZA2.js → chunk-WNBKV4FX.js}

@@ -6,7 +6,7 @@ import {
   getSettingsByNames,
   obfuscateApiKey,
   secureParameters
-} from "./chunk-NH74I6GX.js";
+} from "./chunk-LYNNVNEC.js";
 import {
   ApiKeyType,
   MementoTier,

package/dist/{create-LKM7JKFD.js → create-LOAAI36O.js}

@@ -2,8 +2,8 @@
 import {
   createFabFile,
   createFabFileSchema
-} from "./chunk-KDBSH6HC.js";
-import "./chunk-NH74I6GX.js";
+} from "./chunk-6AJODFPC.js";
+import "./chunk-LYNNVNEC.js";
 import "./chunk-AMDXHL6S.js";
 import "./chunk-WR6YZRQV.js";
 import "./chunk-PDX44BCA.js";

package/dist/index.js

@@ -4,9 +4,9 @@ import {
   getEffectiveApiKey,
   getOpenWeatherKey,
   getSerperKey
-} from "./chunk-44WCHZA2.js";
-import "./chunk-O2622ACK.js";
-import "./chunk-KDBSH6HC.js";
+} from "./chunk-WNBKV4FX.js";
+import "./chunk-LEG2UM5Q.js";
+import "./chunk-6AJODFPC.js";
 import {
   BFLImageService,
   BaseStorage,
@@ -15,7 +15,7 @@ import {
   OpenAIBackend,
   OpenAIImageService,
   XAIImageService
-} from "./chunk-NH74I6GX.js";
+} from "./chunk-LYNNVNEC.js";
 import {
   Logger
 } from "./chunk-AMDXHL6S.js";
@@ -9178,7 +9178,7 @@ import { isAxiosError as isAxiosError2 } from "axios";
 // package.json
 var package_default = {
   name: "@bike4mind/cli",
-  version: "0.2.3-feat-questmaster-gpt5-function-calling.17055+efa3267ae",
+  version: "0.2.3-feat-questmaster-gpt5-function-calling.17057+6a6bd60d2",
   type: "module",
   description: "Interactive CLI tool for Bike4Mind with ReAct agents",
   license: "UNLICENSED",
@@ -9280,10 +9280,10 @@ var package_default = {
   },
   devDependencies: {
     "@bike4mind/agents": "0.1.0",
-    "@bike4mind/common": "2.37.1-feat-questmaster-gpt5-function-calling.17055+efa3267ae",
-    "@bike4mind/mcp": "1.19.5-feat-questmaster-gpt5-function-calling.17055+efa3267ae",
-    "@bike4mind/services": "2.31.1-feat-questmaster-gpt5-function-calling.17055+efa3267ae",
-    "@bike4mind/utils": "2.0.4-feat-questmaster-gpt5-function-calling.17055+efa3267ae",
+    "@bike4mind/common": "2.37.1-feat-questmaster-gpt5-function-calling.17057+6a6bd60d2",
+    "@bike4mind/mcp": "1.19.5-feat-questmaster-gpt5-function-calling.17057+6a6bd60d2",
+    "@bike4mind/services": "2.31.1-feat-questmaster-gpt5-function-calling.17057+6a6bd60d2",
+    "@bike4mind/utils": "2.0.4-feat-questmaster-gpt5-function-calling.17057+6a6bd60d2",
     "@types/diff": "^5.0.9",
     "@types/jsonwebtoken": "^9.0.4",
     "@types/node": "^22.9.0",
@@ -9295,7 +9295,7 @@ var package_default = {
     typescript: "^5.9.3",
     vitest: "^3.2.4"
   },
-  gitHead: "efa3267ae1a98e505e89bb3b7e40caf68a8a1347"
+  gitHead: "6a6bd60d237fe79e8af9694dfa94421fbc0e438c"
 };
 
 // src/config/constants.ts

package/dist/{mementoService-EQIPY6HP.js → mementoService-XGFAWC5F.js}

@@ -2,8 +2,8 @@
 import {
   findMostSimilarMemento,
   getRelevantMementos
-} from "./chunk-44WCHZA2.js";
-import "./chunk-NH74I6GX.js";
+} from "./chunk-WNBKV4FX.js";
+import "./chunk-LYNNVNEC.js";
 import "./chunk-AMDXHL6S.js";
 import "./chunk-WR6YZRQV.js";
 import "./chunk-PDX44BCA.js";

package/dist/{src-U3N2PGNX.js → src-32Z2VZMC.js}

@@ -120,7 +120,7 @@ import {
   validateMermaidSyntax,
   warmUpSettingsCache,
   withRetry
-} from "./chunk-NH74I6GX.js";
+} from "./chunk-LYNNVNEC.js";
 import {
   Logger,
   NotificationDeduplicator,

package/dist/{subtractCredits-7PKKJISU.js → subtractCredits-VMGWNJS7.js}

@@ -2,8 +2,8 @@
 import {
   SubtractCreditsSchema,
   subtractCredits
-} from "./chunk-O2622ACK.js";
-import "./chunk-NH74I6GX.js";
+} from "./chunk-LEG2UM5Q.js";
+import "./chunk-LYNNVNEC.js";
 import "./chunk-AMDXHL6S.js";
 import "./chunk-WR6YZRQV.js";
 import "./chunk-PDX44BCA.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bike4mind/cli",
-  "version": "0.2.3-feat-questmaster-gpt5-function-calling.17055+efa3267ae",
+  "version": "0.2.3-feat-questmaster-gpt5-function-calling.17057+6a6bd60d2",
   "type": "module",
   "description": "Interactive CLI tool for Bike4Mind with ReAct agents",
   "license": "UNLICENSED",
@@ -102,10 +102,10 @@
   },
   "devDependencies": {
     "@bike4mind/agents": "0.1.0",
-    "@bike4mind/common": "2.37.1-feat-questmaster-gpt5-function-calling.17055+efa3267ae",
-    "@bike4mind/mcp": "1.19.5-feat-questmaster-gpt5-function-calling.17055+efa3267ae",
-    "@bike4mind/services": "2.31.1-feat-questmaster-gpt5-function-calling.17055+efa3267ae",
-    "@bike4mind/utils": "2.0.4-feat-questmaster-gpt5-function-calling.17055+efa3267ae",
+    "@bike4mind/common": "2.37.1-feat-questmaster-gpt5-function-calling.17057+6a6bd60d2",
+    "@bike4mind/mcp": "1.19.5-feat-questmaster-gpt5-function-calling.17057+6a6bd60d2",
+    "@bike4mind/services": "2.31.1-feat-questmaster-gpt5-function-calling.17057+6a6bd60d2",
+    "@bike4mind/utils": "2.0.4-feat-questmaster-gpt5-function-calling.17057+6a6bd60d2",
     "@types/diff": "^5.0.9",
     "@types/jsonwebtoken": "^9.0.4",
     "@types/node": "^22.9.0",
@@ -117,5 +117,5 @@
     "typescript": "^5.9.3",
     "vitest": "^3.2.4"
   },
-  "gitHead": "efa3267ae1a98e505e89bb3b7e40caf68a8a1347"
+  "gitHead": "6a6bd60d237fe79e8af9694dfa94421fbc0e438c"
 }