This is a comment left during a code review.
Path: src/agents/compaction.ts
Line: 341-353

Comment:
**Circuit breaker unreachable via public API**

`summarizeChunks` is a module-private (unexported) function, so the new `circuitBreaker?` parameter can never be populated by external callers. Both public entry points — `summarizeWithFallback` and `summarizeInStages` — do not include `circuitBreaker` in their parameter types, and both call `summarizeChunks(params)` where `params` comes from their own unaugmented type. At runtime `circuitBreaker` is always `undefined` inside `summarizeChunks`, making the entire feature inert.

To make the circuit breaker usable, `summarizeWithFallback` (and `summarizeInStages`) need to accept and forward it:

```ts
export async function summarizeWithFallback(params: {
  // ... existing fields ...
  circuitBreaker?: CompactionCircuitBreaker;
}): Promise<string> {
```

Then all three internal calls to `summarizeChunks` (`params` direct spread, `{ ...params, messages: smallMessages }`, etc.) automatically pick it up. The same addition is needed for `summarizeInStages` so it passes through when it delegates to `summarizeWithFallback`.

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/compaction-circuit-breaker.ts
Line: 82-92

Comment:
**Spurious "OPENED" log on half-open → open re-transition**

When the circuit is already in the half-open state (`consecutiveFailures == maxFailures`) and `recordFailure()` is called, `consecutiveFailures` is incremented to `maxFailures + 1`. The guard `consecutiveFailures >= maxFailures` fires again and emits another "OPENED after N consecutive failures" warning, even though the circuit was previously opened and is simply re-opening after a probe failure. This can be confusing in logs and slightly inflates the reported failure count.

```ts
  recordFailure(): void {
    this.consecutiveFailures += 1;
    this.lastFailureTime = Date.now();

    if (this.consecutiveFailures === this.maxFailures) {   // was >=
      log.warn(
        `Compaction circuit breaker OPENED after ${this.consecutiveFailures} consecutive failures — ` +
          `pausing compaction for ${Math.ceil(this.resetAfterMs / 1000)}s`,
      );
    }
  }
```

Changing `>=` to `===` emits the log only on the exact transition, not on repeated re-openings from half-open.

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