@tea-agent/loop-agent 0.41.1-beta.0 → 0.41.2

package/dist/executors/pi-executor.js

@@ -1014,28 +1014,17 @@ export function classifyPiFailure(input) {
     if (input.outputTooLarge)
         return "output-too-large";
     const combined = `${input.stderr}\n${input.stdout}`.toLowerCase();
-    // Context overflow (400 request-too-large / context window exceeded).
-    // Must be detected BEFORE the generic rate-limit/quota phrases: an overflow
-    // error often contains "too many tokens" / "maximum context" which would
-    // otherwise be misread as rate-limit or quota. Kept in sync with
-    // CONTEXT_OVERFLOW_PHRASES in commands/client-recovery.ts.
-    if (/请求上下文过大|context_length_exceeded|context overflow|context length overflow|too many tokens|maximum context|prompt is too long|request_too_large|context window/.test(combined))
-        return "context-overflow";
     // Prefer explicit rate-limit / HTTP 429 before generic "limit" quota phrases.
     // Chinese gateways often say "请求频率已达到限制" without English "rate limit".
-    // Patterns need error-shaped context: the captured stream also contains
-    // serialized session events, where a bare "429" is an output-token counter,
-    // bare "rpm"/"tpm" are base64 substrings of encrypted reasoning, and bare
-    // "network" is a "no-network" UI state name.
-    if (/rate.?limit|too many requests|(?:http|status)[^\d]{0,4}429\b|"code"\s*:\s*"?429\b|(?:^|\n)429[\s:]|请求频率|频率.*限制|\b(?:rpm|tpm)\s*(?:limit|quota|exceeded)/.test(combined))
+    if (/rate.?limit|too many requests|\b429\b|请求频率|频率.*限制|rpm|tpm/.test(combined))
         return "rate-limit";
-    if (/quota|usage limit|reached[^\n.]{0,60}limit|5\s*小时|5小时|insufficient_quota|billing/.test(combined))
+    if (/quota|usage limit|reached.*limit|5\s*小时|5小时|insufficient_quota|billing/.test(combined))
         return "quota";
     if (/\bunauthorized\b|\bhttp\s*40[13]\b|invalid api key|authentication (?:failed|required|error)|auth(?:entication)? failed/.test(combined))
         return "auth";
-    if (/unknown provider|unknown model|(?:provider|model|service|gateway)\s+(?:is\s+)?unavailable|overloaded|at capacity|temporarily unavailable/.test(combined))
+    if (/unknown provider|unknown model|model.*unavailable|provider.*unavailable|\bunavailable\b|overloaded|capacity|temporarily unavailable/.test(combined))
         return "unavailable";
-    if (/network\s*(?:error|failure|issue|timeout|unreachable|reset|abort)|networkerror|connection (?:reset|error|refused|closed|timed\s*out)|request timed out|socket hang up|econn\w*|etimedout|fetch failed|dns (?:error|failure|lookup failed)/.test(combined))
+    if (/network|econnreset|etimedout|socket hang up|connection reset|connection error|request timed out|dns|fetch failed/.test(combined))
         return "network";
     if (input.exitCode === 0 && !input.assistantText.trim())
         return "empty-output";

package/dist/executors/pi-extension-resolver.js

@@ -62,9 +62,21 @@ function readSettingsPackages(settingsPath) {
         return undefined;
     }
 }
-/** Strip an `npm:` prefix from a settings package spec, e.g. `npm:pi-lens`. */
+/**
+ * Strip an `npm:` prefix and a trailing `@<version>`/`@<dist-tag>` suffix
+ * from a settings package spec, e.g. `npm:pi-lens@3.8.72` → `pi-lens`,
+ * `npm:@scope/pkg@1.0.0` → `@scope/pkg`. The leading `@` of a scoped
+ * package name is preserved (the version separator is the LAST `@` at a
+ * positive index). Mirrors the console-side spec parsing semantics so a
+ * version-pinned install (e.g. via `pi install npm:pi-lens@3.8.72`) resolves
+ * identically in both surfaces.
+ */
 export function packageNameFromSpec(spec) {
-    return spec.startsWith("npm:") ? spec.slice(4) : spec;
+    const base = spec.startsWith("npm:") ? spec.slice(4) : spec;
+    const at = base.lastIndexOf("@");
+    if (at > 0)
+        return base.slice(0, at);
+    return base;
 }
 function loadPackageInventories(repoRoot) {
     const agentDir = getPiAgentDir();