fix: smart retry classification, Bedrock 200K context cap, batch error details

- retry.ts: respect isRetryable:false and 4xx status in JSON error bodies (Issue #7)
- provider.ts: cap Bedrock Anthropic models at 200K context (Issue #4)
- batch.ts: include per-tool error details in failure output (Issue #9)
- retry.test.ts: 5 new tests for catch-all retry classification

Author: praxstack

packages/opencode/src/provider/provider.ts

@@ -740,6 +740,17 @@ export namespace Provider {
 
     m.variants = mapValues(ProviderTransform.variants(m), (v) => v)
 
+    // Bedrock enforces 200K context unless the context-1m beta header is sent.
+    // models-snapshot.ts (auto-generated) lists capability (1M) not runtime limit.
+    const BEDROCK_CONTEXT_CAP = 200_000
+    if (
+      provider.id === "amazon-bedrock" &&
+      m.limit.context > BEDROCK_CONTEXT_CAP &&
+      m.id.includes("anthropic")
+    ) {
+      m.limit.context = BEDROCK_CONTEXT_CAP
+    }
+
     return m
   }
 
@@ -885,6 +896,16 @@ export namespace Provider {
           pickBy(merged, (v) => !v.disabled),
           (v) => omit(v, ["disabled"]),
         )
+        // Bedrock enforces 200K context unless the context-1m beta header is sent.
+        // models-snapshot.ts (auto-generated) lists capability (1M) not runtime limit.
+        const BEDROCK_CONTEXT_CAP = 200_000
+        if (
+          providerID === "amazon-bedrock" &&
+          parsedModel.limit.context > BEDROCK_CONTEXT_CAP &&
+          parsedModel.id.includes("anthropic")
+        ) {
+          parsedModel.limit.context = BEDROCK_CONTEXT_CAP
+        }
         parsed.models[modelID] = parsedModel
       }
       database[providerID] = parsed

packages/opencode/src/session/retry.ts

@@ -93,6 +93,11 @@ export namespace SessionRetry {
       if (json.type === "error" && json.error?.code?.includes("rate_limit")) {
         return "Rate Limited"
       }
+      // Respect explicit isRetryable: false from provider SDKs (e.g. Bedrock)
+      if (json.isRetryable === false) return undefined
+      // 4xx errors are client errors — not retryable
+      const status = typeof json.status === "number" ? json.status : typeof json.statusCode === "number" ? json.statusCode : undefined
+      if (status !== undefined && status >= 400 && status < 500) return undefined
       return JSON.stringify(json)
     } catch {
       return undefined

packages/opencode/src/tool/batch.ts

@@ -161,7 +161,14 @@ export const BatchTool = Tool.define("batch", async () => {
 
       const outputMessage =
         failedCalls > 0
-          ? `Executed ${successfulCalls}/${results.length} tools successfully. ${failedCalls} failed.`
+          ? [
+              `Executed ${successfulCalls}/${results.length} tools successfully. ${failedCalls} failed.`,
+              "",
+              "Failed tools:",
+              ...results
+                .filter((r) => !r.success)
+                .map((r) => `- ${r.tool}: ${r.error instanceof Error ? r.error.message : String(r.error)}`),
+            ].join("\n")
           : `All ${successfulCalls} tools executed successfully.\n\nKeep using the batch tool for optimal performance in your next response!`
 
       return {

packages/opencode/test/session/retry.test.ts

@@ -113,6 +113,31 @@ describe("session.retry.retryable", () => {
     expect(SessionRetry.retryable(error)).toBeUndefined()
   })
 
+  test("does not retry when json has isRetryable: false", () => {
+    const error = wrap(JSON.stringify({ isRetryable: false, message: "prompt is too long" }))
+    expect(SessionRetry.retryable(error)).toBeUndefined()
+  })
+
+  test("does not retry 400 status in json body", () => {
+    const error = wrap(JSON.stringify({ status: 400, message: "Bad request" }))
+    expect(SessionRetry.retryable(error)).toBeUndefined()
+  })
+
+  test("does not retry 403 statusCode in json body", () => {
+    const error = wrap(JSON.stringify({ statusCode: 403, message: "Forbidden" }))
+    expect(SessionRetry.retryable(error)).toBeUndefined()
+  })
+
+  test("retries 500 status in json body", () => {
+    const error = wrap(JSON.stringify({ status: 500, message: "Internal Server Error" }))
+    expect(SessionRetry.retryable(error)).toBe(JSON.stringify({ status: 500, message: "Internal Server Error" }))
+  })
+
+  test("retries json without status or isRetryable fields", () => {
+    const error = wrap(JSON.stringify({ error: { message: "some_unknown_error" } }))
+    expect(SessionRetry.retryable(error)).toBe(JSON.stringify({ error: { message: "some_unknown_error" } }))
+  })
+
   test("does not retry context overflow errors", () => {
     const error = new MessageV2.ContextOverflowError({
       message: "Input exceeds context window of this model",