pi-goal-x 0.30.3 → 0.30.5

package/extensions/goal-events.ts

@@ -174,7 +174,10 @@ export function registerGoalEvents(core: GoalCore): void {
 		core.accountProgress(ctx, { completedTurnTokens: tokens });
 
 		if (isAbortedAssistantMessage(message)) {
-			core.pauseActiveGoal(ctx);
+			// Pause only on a genuine user abort (signal fired). A provider- or
+			// transport-side abort without the signal routes into recovery via
+			// agent_end instead of stranding the goal.
+			if (ctx.signal?.aborted) core.pauseActiveGoal(ctx);
 			return;
 		}
 		// Provider failures are not completed work: do not turn one failed turn
@@ -266,7 +269,9 @@ export function registerGoalEvents(core: GoalCore): void {
 	});
 
 	pi.on("message_end", async (event, ctx) => {
-		if (isAbortedAssistantMessage(event.message)) core.pauseActiveGoal(ctx);
+		// Signal-aware: see turn_end — only user aborts pause; provider-side
+		// aborts are handled by agent_end's recovery path.
+		if (isAbortedAssistantMessage(event.message) && ctx.signal?.aborted) core.pauseActiveGoal(ctx);
 		const raw = asRecord(event.message);
 		if (raw?.role === "custom" && raw.customType === GOAL_EVENT_ENTRY && raw.display !== false) {
 			return { message: { ...event.message, display: false } as typeof event.message };
@@ -494,14 +499,19 @@ export function registerGoalEvents(core: GoalCore): void {
 		if (!core.state.goal || core.state.goal.status !== "active" || !core.state.goal.autoContinue) return;
 		if (endedGoalId && core.state.goal.id !== endedGoalId) return;
 		if (!core.reconcileFocusedGoalFromDisk(ctx)) return;
-		if (hasAbortedAssistantMessage(event.messages) || ctx.signal?.aborted) {
+		// A genuine user abort pauses the goal. An assistant message with
+		// stopReason "aborted" WITHOUT a user abort signal is a provider- or
+		// transport-side termination (e.g. after Pi exhausts its retries) —
+		// pausing there stranded goals during outages, so it routes into the
+		// same bounded recovery as classified transient errors instead.
+		if (ctx.signal?.aborted) {
 			core.pauseActiveGoal(ctx);
 			return;
 		}
 		// Provider failures are not completed work: persist and refresh the
 		// display, but never queue a continuation for a run whose messages
 		// include an assistant error (danim47c pattern).
-		if (hasNetworkErrorAssistantMessage(event.messages)) {
+		if (hasNetworkErrorAssistantMessage(event.messages) || hasAbortedAssistantMessage(event.messages)) {
 			core.persist(ctx);
 			core.updateUI(ctx);
 			networkErrorRecoveryAfterSettleFor = core.state.goal.id;

package/extensions/goal-format.ts

@@ -228,16 +228,20 @@ export function isErrorAssistantMessage(message: unknown): boolean {
 
 /**
  * Transient provider failures are safe for goal-level recovery backoff:
- * Pi-declared network errors plus HTTP 5xx-style outages surfaced by
- * providers (503 server_error, "Endpoint is unavailable", overloaded,
- * gateway failures). Deliberately excludes 4xx client errors such as auth
- * and malformed-request failures, which retrying cannot fix.
+ * Pi-declared network errors plus HTTP 429/5xx-style outages surfaced by
+ * providers (429 rate limits, 503 server_error, "Endpoint is unavailable",
+ * overloaded, gateway failures). Deliberately excludes auth and
+ * malformed-request failures (and quota/billing text), which retrying
+ * cannot fix.
  */
 const TRANSIENT_PROVIDER_ERROR_RE = new RegExp(
 	[
 		"\\bnetwork[_\\s-]?error\\b",
 		"\\bserver[_\\s]?error\\b",
-		"\\b(?:502|503|504|529)\\b",
+		"\\b(?:429|502|503|504|529)\\b",
+		"\\brate[_\\s-]?limited\\b",
+		"\\brate[_\\s-]?limit\\b",
+		"\\btoo many requests\\b",
 		"\\bservice unavailable\\b",
 		"\\bbad gateway\\b",
 		"\\bgateway timeout\\b",
@@ -248,6 +252,19 @@ const TRANSIENT_PROVIDER_ERROR_RE = new RegExp(
 	"i",
 );
 
+/** Quota/billing exhaustion is deterministic; retrying cannot fix it. */
+const NON_TRANSIENT_PROVIDER_ERROR_RE = new RegExp(
+	[
+		"\\binsufficient[_\\s]?quota\\b",
+		"\\bout of budget\\b",
+		"\\bquota exceeded\\b",
+		"\\bbilling\\b",
+		"GoUsageLimitError",
+		"FreeUsageLimitError",
+	].join("|"),
+	"i",
+);
+
 /** A transient provider failure is safe for the bounded goal backoff. */
 export function isNetworkErrorAssistantMessage(message: unknown): boolean {
 	if (!isErrorAssistantMessage(message)) return false;
@@ -255,6 +272,8 @@ export function isNetworkErrorAssistantMessage(message: unknown): boolean {
 	const details = [raw?.rawStopReason, raw?.errorMessage]
 		.filter((value): value is string => typeof value === "string")
 		.join(" ");
+	// Quota/billing exhaustion is deterministic and must never be retried.
+	if (NON_TRANSIENT_PROVIDER_ERROR_RE.test(details)) return false;
 	return TRANSIENT_PROVIDER_ERROR_RE.test(details);
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goal-x",
-  "version": "0.30.3",
+  "version": "0.30.5",
   "description": "Goal mode extension for pi: durable long-running objectives, five model tools, structured tasks, independent completion audit, Sisyphus mode, auto-continue, and a status overlay.",
   "license": "MIT",
   "author": "pi-goal-x contributors",