@autonoma-ai/planner 0.1.8 → 0.1.9

package/dist/index.js

@@ -3585,7 +3585,7 @@ function buildClassifierPrompt(args) {
   const statusLine = args.httpStatus != null ? `HTTP status: ${args.httpStatus}. (A 5xx usually means the handler threw; a 4xx is more often a rejected payload, but use the error text \u2014 status alone is not decisive.)` : `No HTTP status (the request threw before a response \u2014 often a network/server-process problem, lean implementation or unclear).`;
   return `${PRIMER}
 
-- RECIPE DATA is wrong \u2014 the JSON the tool sent doesn't fit the developer's (correct) schema. Examples of the *kind* of problem (not an exhaustive list): a _ref points at an alias that does not exist among the valid targets listed below, a field holds a value the backend rejects, a required field is missing or an unknown field was sent, or a value has the wrong type. These are fixable by regenerating the data \u2014 the developer's code is fine.
+- RECIPE DATA is wrong \u2014 the JSON the tool sent doesn't fit the developer's (correct) schema. Examples of the *kind* of problem (not an exhaustive list): the request body references an alias (_ref) that no record in THIS SAME request declares with an _alias, a field holds a value the backend rejects, a required field is missing or an unknown field was sent, or a value has the wrong type. These are fixable by regenerating the data \u2014 the developer's code is fine. NOTE: an error like "references unknown alias(es): X" is recipe-side whenever X is not declared by an _alias in the test data shown below \u2014 the SDK resolves _refs WITHIN the request body, so it does not matter whether X appears in the "valid targets" list (that list is historical context about other entities' own runs, NOT the contents of this request).
 - IMPLEMENTATION is wrong \u2014 the developer's own handler/factory code is broken. Examples of the *kind* of problem: the factory for this entity is not registered, the handler references a column or table that does not exist (even though the recipe never mentioned it), the insert/delete logic has a bug, or the server threw an unhandled exception. No change to the test data can fix this; the developer must edit code.
 
 ## How to decide
@@ -3595,7 +3595,7 @@ Ask: "Would sending DIFFERENT, corrected test data \u2014 still matching the int
 - If the data shown below looks valid and the error points at the server's own logic, a missing factory, or a column/table the recipe never referenced \u2192 **implementation**.
 - If you genuinely cannot tell from the evidence \u2192 **unclear**. Prefer "unclear" over a confident guess; a wrong confident answer is worse than admitting uncertainty.
 
-Cross-check the error against the actual data below before deciding \u2014 e.g. only call a _ref invalid if its target really is absent from the valid targets list.
+Cross-check the error against the actual data sent below before deciding. For an "unknown alias" error, look at whether a record in the test data declares that alias with an _alias \u2014 NOT at the valid-targets list. If the alias the error names is missing from (or spelled differently in) the data we sent, it is recipe-side, even when that alias appears in the valid-targets list.
 
 ## Evidence
 
@@ -3906,8 +3906,7 @@ function seedFeedbackFromError(errorContext, reason) {
 async function promptOnFailure(entityName, errorBody, ctx, phase, httpStatus) {
   notify("Autonoma", `${entityName} \u2014 failed, action needed`);
   const errorContext = formatErrorContext(errorBody);
-  const budgetLeft = ctx.budget.attempts < MAX_AUTOFIX_ATTEMPTS;
-  const { side, reason } = await classifyFailure(ctx.model, {
+  const { reason } = await classifyFailure(ctx.model, {
     entityName,
     phase,
     httpStatus,
@@ -3916,31 +3915,25 @@ async function promptOnFailure(entityName, errorBody, ctx, phase, httpStatus) {
     validRefAliases: ctx.validRefAliases,
     entityAudit: ctx.entityAudit
   });
-  if (side === "recipe" && budgetLeft) {
+  if (ctx.budget.attempts < MAX_AUTOFIX_ATTEMPTS) {
     ctx.budget.attempts++;
+    p5.log.info(`Triage: ${reason}`);
     p5.log.info(
-      `Triaged as a recipe-data issue \u2014 fixing automatically (attempt ${ctx.budget.attempts}/${MAX_AUTOFIX_ATTEMPTS}): ${reason}`
+      `Handing the failure to the agent to fix from the error (attempt ${ctx.budget.attempts}/${MAX_AUTOFIX_ATTEMPTS})...`
     );
     return seedFeedbackFromError(errorContext, reason);
   }
-  const offerAutofix = side !== "implementation" && budgetLeft;
-  if (side === "implementation") {
-    p5.log.warn(`This looks like a handler/code issue, not the test data: ${reason}`);
-  } else if (side === "recipe" && !budgetLeft) {
-    p5.log.warn(`Autofix ran ${MAX_AUTOFIX_ATTEMPTS}\xD7 without resolving it \u2014 over to you. Latest read: ${reason}`);
-  }
+  p5.log.warn(`The agent tried ${MAX_AUTOFIX_ATTEMPTS}\xD7 without resolving it. Latest triage: ${reason}`);
   const action = await p5.select({
     message: "What would you like to do?",
     options: [
       { value: "retry", label: "Yes, retry \u2014 I fixed my handler code", hint: "Send the same request again" },
-      ...offerAutofix ? [
-        {
-          value: "autofix",
-          label: "Yes, let the agent fix it from the error",
-          hint: "The error explains itself \u2014 hand it to the agent, no typing"
-        }
-      ] : [],
-      { value: "feedback", label: "Yes, fix the recipe data \u2014 I'll explain what's wrong", hint: "The request data is wrong and I'll describe the change" },
+      {
+        value: "autofix",
+        label: "Let the agent try again from the error",
+        hint: "Hand the raw error back to the agent, no typing"
+      },
+      { value: "feedback", label: "Let me explain what's wrong", hint: "Describe the change and the agent will apply it" },
       { value: "skip", label: "No, skip this entity", hint: "Move on to the next entity" }
     ]
   });
@@ -3949,7 +3942,7 @@ async function promptOnFailure(entityName, errorBody, ctx, phase, httpStatus) {
   if (action === "retry") return "retry";
   if (action === "autofix") {
     ctx.budget.attempts++;
-    return seedFeedbackFromError(errorContext);
+    return seedFeedbackFromError(errorContext, reason);
   }
   const fb = await p5.text({
     message: "What's wrong with the recipe data?",