This is a comment left during a code review.
Path: src/infra/unhandled-rejections.ts
Line: 68-74

Comment:
**`SQLITE_CANTOPEN` is not always transient**

`SQLITE_CANTOPEN` typically means SQLite cannot open the database file due to a wrong path, missing directory, or a permanent permission denial — these won't resolve on their own. Treating it as transient means the gateway will silently swallow and continue past what might be a hard misconfiguration (e.g., wrong `store.path`), making it much harder to diagnose.

Consider separating it from the genuinely transient codes or at least logging it at a higher severity level than the others:

```suggestion
// SQLite error codes that indicate transient failures (shouldn't crash the gateway)
const TRANSIENT_SQLITE_CODES = new Set([
  "SQLITE_BUSY",
  "SQLITE_LOCKED",
  "SQLITE_IOERR",
]);

// Permanent-ish codes that should still be logged loudly but not crash
const NONFATAL_SQLITE_CODES = new Set([
  "SQLITE_CANTOPEN",
]);
```

At a minimum, the `SQLITE_CANTOPEN` branch might deserve a `console.error` rather than `console.warn` so operators notice it.

How can I resolve this? If you propose a fix, please make it concise.

This is a comment left during a code review.
Path: src/agents/transcript-policy.ts
Line: 124-126

Comment:
**Missing test coverage for Copilot Claude `dropThinkingBlocks` and `preserveSignatures`**

The PR updates the `dropThinkingBlocks` flag to be `true` for all Anthropic models (including Copilot), and changes `preserveSignatures` to be `true` **only** for Copilot (previously it was `false` for Copilot). Neither of these Copilot-specific outcomes is tested.

The old tests that were removed checked `preserveSignatures` for `provider: "anthropic"` and `provider: "amazon-bedrock"`. No test now verifies the Copilot case for either field. Consider adding:

```ts
it("drops thinking blocks and preserves signatures for GitHub Copilot Claude", () => {
  const policy = resolveTranscriptPolicy({
    provider: "github-copilot",
    modelId: "claude-sonnet-4-5",
    modelApi: "anthropic-messages",
  });
  expect(policy.dropThinkingBlocks).toBe(true);
  expect(policy.preserveSignatures).toBe(true);
});
```

The `preserveSignatures = true` for Copilot is functionally harmless (thinking blocks are dropped by `dropThinkingBlocks` anyway), but the change is undocumented and untested.

How can I resolve this? If you propose a fix, please make it concise.