@springbrand/agent-runtime 0.2.0-alpha.60 → 0.2.0-alpha.61

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@springbrand/agent-runtime",
-  "version": "0.2.0-alpha.60",
+  "version": "0.2.0-alpha.61",
   "type": "module",
   "files": [
     "src",

package/src/lib/prompt.ts

@@ -111,8 +111,8 @@ export const INTERACTION =
   "Interaction: When progress requires user decisions, put all 1-10 necessary questions in one ask_user call " +
   "instead of asking each question separately or writing 'please choose A/B/C' as text. Every question must set exactly one responseType. " +
   "Use responseType: \"single_select\" or responseType: \"multi_select\" with 2-6 options for choices; use responseType: \"text\" " +
-  "for required free text; use responseType: \"attachment\" when the user must upload one or more files. At most one question may " +
-  "request attachments. Do not add an accept field or tell the user to upload through the normal chat composer; " +
+  "for free text; use responseType: \"attachment\" for one or more files. Set required: true only when progress cannot continue " +
+  "without that answer; required defaults to false. At most one question may request attachments. Do not add an accept field or tell the user to upload through the normal chat composer; " +
   "continue the same turn when its result arrives. " +
   "Don't use it when a sensible default lets you proceed. " +
   "After completing a substantive task (report, analysis, multi-step job), finish all work and other Tool calls, " +

package/src/pi/tool/base.ts

@@ -44,6 +44,11 @@ const askUserQuestion = Type.Object({
     description:
       "For single_select or multi_select, whether the user may also add free text. Defaults to false.",
   })),
+  required: Type.Optional(Type.Boolean({
+    default: false,
+    description:
+      "Whether the user must answer this question to submit. Defaults to false.",
+  })),
 }, { additionalProperties: false });
 
 const askUserParameters = Type.Object({
@@ -98,6 +103,10 @@ function questionsOf(
     if (!question || typeof question.question !== "string" || !question.question) {
       return null;
     }
+    if (
+      question.required !== undefined && typeof question.required !== "boolean"
+    ) return null;
+    const required = question.required === true;
     let responseType = question.responseType;
     if (
       typeof responseType !== "string" ||
@@ -118,6 +127,11 @@ function questionsOf(
         return null;
       }
     }
+    const normalizedQuestion = {
+      question: question.question,
+      responseType: responseType as typeof askUserResponseTypes[number],
+      required,
+    };
     if (responseType === "attachment") {
       attachmentQuestions += 1;
       if (
@@ -126,7 +140,7 @@ function questionsOf(
         question.allowCustom !== undefined ||
         (!allowLegacy && question.multiSelect !== undefined)
       ) return null;
-      normalized.push({ question: question.question, responseType });
+      normalized.push(normalizedQuestion);
       continue;
     }
     if (responseType === "text") {
@@ -137,7 +151,7 @@ function questionsOf(
       ) {
         return null;
       }
-      normalized.push({ question: question.question, responseType });
+      normalized.push(normalizedQuestion);
       continue;
     }
     if (
@@ -156,8 +170,7 @@ function questionsOf(
       (question.kind !== undefined || question.multiSelect !== undefined)
     ) return null;
     normalized.push({
-      question: question.question,
-      responseType: responseType as typeof askUserResponseTypes[number],
+      ...normalizedQuestion,
       options: question.options,
       ...(question.allowCustom === undefined
         ? {}
@@ -223,6 +236,12 @@ function isAskUserResponse(input: unknown, value: unknown): boolean {
       );
     })) return false;
 
+    if (
+      answer.selections.length === 0 &&
+      !answer.text.trim() &&
+      answer.attachments.length === 0
+    ) return question.required !== true;
+
     if (question.responseType === "attachment") {
       return answer.selections.length === 0 && !answer.text &&
         answer.attachments.length >= 1 && answer.attachments.length <= 8;
@@ -367,7 +386,7 @@ export function basePiToolCandidates(
         name: "ask_user",
         label: "Ask user",
         description:
-          "Ask all 1-10 user questions needed to continue in one call. Required input for every question: question and responseType. Use responseType text for free text, single_select or multi_select with 2-6 distinct options for choices, and attachment when the user must upload files. At most one attachment question is allowed and it accepts multiple files. Do not add file-specific fields such as accept or tell the user to use the normal chat composer. The answers return as this tool's result, so continue the same turn after calling it.",
+          "Ask all 1-10 user questions needed to continue in one call. Required input for every question: question and responseType; required defaults to false, so set it to true only when progress cannot continue without an answer. Use responseType text for free text, single_select or multi_select with 2-6 distinct options for choices, and attachment for files. At most one attachment question is allowed and it accepts multiple files. Do not add file-specific fields such as accept or tell the user to use the normal chat composer. The answers return as this tool's result, so continue the same turn after calling it.",
         parameters: askUserParameters,
         prepareArguments: prepareAskUserArguments,
         // 永远不会被调用：执行链在 interaction 闸就 park 住了。留一个失败关闭的