@pellux/goodvibes-agent 1.5.9 → 1.6.0

package/src/tools/agent-local-registry-memory.ts

@@ -12,6 +12,20 @@
  * search so an unavailable index degrades honestly (stated reason, literal
  * fallback) instead of silently returning zero matches that read as "nothing
  * was ever stored."
+ *
+ * MEMORY-SPINE WIRING (SDK 1.2.0 full-detach catalog). Every action below now
+ * routes through `memorySpine` — local by default, or over the wire the moment
+ * the agent has adopted a daemon (see services.ts's memorySpineClient).
+ * get/create/review/stale/delete and the literal `search` path use the five
+ * 1.1.0 core verbs (get/add/updateReview/updateReview/delete/honestSearch);
+ * `list` and `update` use the 1.2.0 extended verbs of the same name
+ * (list/update); the SEMANTIC search path uses the 1.2.0 `searchSemantic`
+ * extended verb. Only the index-health check (`registry.vectorStats()`) stays
+ * on the raw local `registry` — vector-index diagnostics are maintenance a
+ * store performs on its OWN index, so reporting them honestly means reading
+ * this process's own index, not a wire-forwarded view of the daemon's (see the
+ * CLI's identical ruling in memory-command-wire.ts). That is a deliberate,
+ * stated scope limit, not an oversight.
  */
 import {
   HASHED_MEMORY_EMBEDDING_PROVIDER,
@@ -22,6 +36,7 @@ import {
   type MemorySemanticSearchResult,
   type MemoryVectorStats,
 } from '@pellux/goodvibes-sdk/platform/state';
+import type { MemoryAccess } from '@pellux/goodvibes-sdk/platform/runtime/memory-spine';
 import { assertNoSecretLikeMemoryText } from '../agent/memory-safety.ts';
 import { formatAgentRecordReference, formatAgentRecordReviewState } from '../agent/record-labels.ts';
 import {
@@ -105,9 +120,16 @@ function renderMemorySearch(mode: string, query: string, lines: readonly string[
   return [`Agent-local memory search (${mode})`, `query ${query || '(all)'}`, ...lines].join('\n');
 }
 
-export async function handleMemory(registry: MemoryRegistry, action: AgentLocalRegistryAction, args: AgentLocalRegistryToolArgs): Promise<string> {
+export async function handleMemory(
+  registry: MemoryRegistry,
+  memorySpine: MemoryAccess,
+  action: AgentLocalRegistryAction,
+  args: AgentLocalRegistryToolArgs,
+): Promise<string> {
   if (action === 'list') {
-    const records = registry.getAll();
+    // list() is a 1.2.0 extended verb — routes through the spine (no filter =
+    // getAll semantics, matching the previous registry.getAll() call exactly).
+    const records = await memorySpine.list();
     return records.length === 0
       ? 'Agent-local memory\nNo Agent-local memory records.'
       : ['Agent-local memory', ...records.map(formatMemory)].join('\n');
@@ -116,20 +138,27 @@ export async function handleMemory(registry: MemoryRegistry, action: AgentLocalR
     const query = readString(args.query);
     const wantsLiteral = args.semantic === false;
     if (wantsLiteral) {
-      const records = registry.search({ query, limit: 10 });
+      // Literal search IS wire-covered — routes through the spine (honestSearch
+      // with no `semantic` flag is exactly a literal scan; see
+      // runHonestMemorySearch in the SDK).
+      const { records } = await memorySpine.honestSearch({ query, limit: 10 });
       return renderMemorySearch('literal — explicitly requested', query, records.map(formatMemory));
     }
     // SEMANTIC BY DEFAULT: natural-language recall must not depend on the query text
     // appearing as a literal substring in the stored summary/detail. Check index
     // health FIRST so an unavailable index degrades honestly instead of silently
-    // returning zero matches that read as "nothing was ever stored."
+    // returning zero matches that read as "nothing was ever stored." The health
+    // check itself stays on the raw local registry (vector-index diagnostics are
+    // this process's own — see the file-header note); the actual search, once the
+    // index is known healthy, is the 1.2.0 searchSemantic extended verb and
+    // routes through the spine.
     const stats = registry.vectorStats();
     const unavailable = describeMemoryIndexUnavailable(stats);
     if (unavailable) {
       const records = registry.search({ query, limit: 10 });
       return renderMemorySearch(`literal fallback — ${unavailable}`, query, records.map(formatMemory));
     }
-    const results = registry.searchSemantic({ query, limit: 10 });
+    const results = await memorySpine.searchSemantic({ query, limit: 10 });
     const consultedSemanticIndex = query.length > 0 && results.some((entry) => entry.similarity > 0);
     const caveat = describeMemoryIndexCaveat(stats);
     const mode = !consultedSemanticIndex && query.length > 0
@@ -140,7 +169,7 @@ export async function handleMemory(registry: MemoryRegistry, action: AgentLocalR
     return renderMemorySearch(mode, query, results.map(formatMemoryMatch));
   }
   if (action === 'get') {
-    const record = registry.get(requireId(args));
+    const record = await memorySpine.get(requireId(args));
     if (!record) return `Unknown Agent-local memory ${readString(args.id)}`;
     return [
       formatMemory(record),
@@ -158,7 +187,7 @@ export async function handleMemory(registry: MemoryRegistry, action: AgentLocalR
     const tags = readStringList(args.tags);
     const confidence = readOptionalConfidence(args.confidence);
     assertNoSecretLikeMemoryText([summary, detail, ...tags]);
-    const record = await registry.add({
+    const record = await memorySpine.add({
       scope: readMemoryScope(args),
       cls: requireMemoryClass(args),
       summary,
@@ -174,11 +203,13 @@ export async function handleMemory(registry: MemoryRegistry, action: AgentLocalR
     ].join('\n');
   }
   if (action === 'update') {
+    // update() is a 1.2.0 extended verb (summary/detail/tags/scope patch) — routes
+    // through the spine.
     const summary = readString(args.summary || args.description);
     const detail = readString(args.detail || args.body);
     const tags = args.tags === undefined ? undefined : [...readStringList(args.tags)];
     assertNoSecretLikeMemoryText([summary, detail, ...(tags ?? [])]);
-    const record = registry.update(requireId(args), {
+    const record = await memorySpine.update(requireId(args), {
       summary: summary || undefined,
       detail: detail || undefined,
       tags,
@@ -192,7 +223,7 @@ export async function handleMemory(registry: MemoryRegistry, action: AgentLocalR
     ].join('\n');
   }
   if (action === 'review') {
-    const record = registry.review(requireId(args), {
+    const record = await memorySpine.updateReview(requireId(args), {
       state: 'reviewed',
       confidence: readOptionalConfidence(args.confidence),
       reviewedBy: 'agent',
@@ -204,7 +235,7 @@ export async function handleMemory(registry: MemoryRegistry, action: AgentLocalR
     ].join('\n');
   }
   if (action === 'stale') {
-    const record = registry.review(requireId(args), {
+    const record = await memorySpine.updateReview(requireId(args), {
       state: 'stale',
       staleReason: readString(args.reason) || 'Marked stale by Agent.',
     });
@@ -217,7 +248,7 @@ export async function handleMemory(registry: MemoryRegistry, action: AgentLocalR
   if (action === 'delete') {
     const id = requireId(args);
     requireConfirmedDelete(args, 'Agent-local memory');
-    if (!registry.delete(id)) return `Unknown Agent-local memory ${id}`;
+    if (!(await memorySpine.delete(id))) return `Unknown Agent-local memory ${id}`;
     return [
       'Deleted Agent-local memory',
       `  id ${id}`,

package/src/tools/agent-local-registry-tool.ts

@@ -2,6 +2,7 @@ import type { Tool } from '@pellux/goodvibes-sdk/platform/types';
 import type { ToolRegistry } from '@pellux/goodvibes-sdk/platform/tools';
 import type { ShellPathService } from '@/runtime/index.ts';
 import { MemoryRegistry } from '@pellux/goodvibes-sdk/platform/state';
+import type { MemoryAccess } from '@pellux/goodvibes-sdk/platform/runtime/memory-spine';
 import { AgentPersonaRegistry, type AgentPersonaRecord } from '../agent/persona-registry.ts';
 import { AgentNoteRegistry, type AgentNoteRecord } from '../agent/note-registry.ts';
 import { AgentRoutineRegistry, type AgentRoutineRecord } from '../agent/routine-registry.ts';
@@ -512,7 +513,7 @@ function handleRoutine(shellPaths: ShellPathService, action: AgentLocalRegistryA
   throw new Error(`Action ${action} is not valid for routines.`);
 }
 
-export function createAgentLocalRegistryTool(shellPaths: ShellPathService, memoryRegistry: MemoryRegistry): Tool {
+export function createAgentLocalRegistryTool(shellPaths: ShellPathService, memoryRegistry: MemoryRegistry, memorySpine: MemoryAccess): Tool {
   return {
     definition: {
       name: 'agent_local_registry',
@@ -560,7 +561,7 @@ export function createAgentLocalRegistryTool(shellPaths: ShellPathService, memor
       if (!isDomain(args.domain)) return registryError(`Unknown domain. Valid values ${DOMAINS.join(', ')}.`);
       if (!isAction(args.action)) return registryError(`Unknown action. Valid values ${ACTIONS.join(', ')}.`);
       try {
-        if (args.domain === 'memory') return registryOutput(await handleMemory(memoryRegistry, args.action, args));
+        if (args.domain === 'memory') return registryOutput(await handleMemory(memoryRegistry, memorySpine, args.action, args));
         if (args.domain === 'note') return registryOutput(handleNote(shellPaths, args.action, args));
         if (args.domain === 'persona') return registryOutput(handlePersona(shellPaths, args.action, args));
         if (args.domain === 'skill') return registryOutput(handleSkill(shellPaths, args.action, args));
@@ -573,6 +574,6 @@ export function createAgentLocalRegistryTool(shellPaths: ShellPathService, memor
   };
 }
 
-export function registerAgentLocalRegistryTool(registry: ToolRegistry, shellPaths: ShellPathService, memoryRegistry: MemoryRegistry): void {
-  registry.register(createAgentLocalRegistryTool(shellPaths, memoryRegistry));
+export function registerAgentLocalRegistryTool(registry: ToolRegistry, shellPaths: ShellPathService, memoryRegistry: MemoryRegistry, memorySpine: MemoryAccess): void {
+  registry.register(createAgentLocalRegistryTool(shellPaths, memoryRegistry, memorySpine));
 }

package/src/version.ts

@@ -6,7 +6,7 @@ import { join } from 'node:path';
 // The prebuild script updates the fallback value before compilation.
 // Uses import.meta.dir (Bun) to locate package.json relative to this file,
 // which is correct regardless of the process working directory.
-let _version = '1.5.9';
+let _version = '1.6.0';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8')) as {
     readonly version?: unknown;

package/src/audio/spoken-turn-controller.ts

@@ -1,455 +0,0 @@
-import type { ConfigManager } from '@pellux/goodvibes-sdk/platform/config';
-import type { ConfigKey } from '@pellux/goodvibes-sdk/platform/config';
-import type { TurnEvent } from '@/runtime/index.ts';
-import type { VoiceService, VoiceSynthesisStreamResult } from '@pellux/goodvibes-sdk/platform/voice';
-import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
-import { TtsTextChunker } from './text-chunker.ts';
-import type { StreamingAudioPlayer } from './player.ts';
-
-/**
- * How many synthesis requests may sit in the pipeline at once (synthesizing,
- * waiting to play, or playing). 2 = the chunk being played plus ONE prefetch,
- * so the next audio is ready the moment the current sink drains. Bounding
- * this is what keeps a streaming answer from bursting N concurrent requests
- * at the voice provider — ElevenLabs plans allow as few as 3 concurrent, and
- * an unbounded burst 429s the whole turn. The SDK config schema has no tts.*
- * key for pipeline tuning, so this is a constant by design.
- */
-const SYNTHESIS_PIPELINE_WINDOW = 2;
-
-/**
- * Upper bound for one merged synthesis request's text. The ElevenLabs
- * provider passes request text through verbatim (no cap of its own); the API
- * caps text per request by plan — 2,500 chars on the lowest tiers, 5,000 on
- * most others. 1,500 stays safely under every plan while still folding a
- * multi-paragraph answer into one or two requests.
- */
-const SYNTHESIS_MERGE_MAX_CHARS = 1500;
-
-/**
- * Backoff schedule for transient synthesis failures (429 rate/concurrency
- * limits, transient 5xx, network drops): first retry after 1s, second after
- * 2.5s, then the chunk is skipped honestly and the turn continues. The SDK's
- * provider errors are plain Error strings with the HTTP status embedded — no
- * Retry-After header is exposed — so the schedule is fixed, not server-driven.
- */
-const SYNTHESIS_RETRY_DELAYS_MS = [1000, 2500] as const;
-
-export interface SpokenTurnControllerOptions {
-  readonly voiceService: Pick<VoiceService, 'synthesizeStream'>;
-  readonly configManager: Pick<ConfigManager, 'get'>;
-  readonly player: StreamingAudioPlayer;
-  readonly notify?: (message: string) => void;
-  readonly now?: () => number;
-  readonly setInterval?: typeof setInterval;
-  readonly clearInterval?: typeof clearInterval;
-  /** Injectable clock for retry backoff timers (tests use fakes). */
-  readonly setTimeout?: typeof setTimeout;
-  readonly clearTimeout?: typeof clearTimeout;
-}
-
-export class SpokenTurnController {
-  private pendingPrompt: string | null = null;
-  private activeTurnId: string | null = null;
-  private chunker: TtsTextChunker | null = null;
-  private chunkSequence = 0;
-  private playbackChain: Promise<void> = Promise.resolve();
-  private readonly abortControllers = new Set<AbortController>();
-  private timer: ReturnType<typeof setInterval> | null = null;
-  private errorReportedForTurn = false;
-  private noPlayerNoticed = false;
-  /** Chunker output waiting to be merged into a synthesis request. */
-  private pendingTexts: string[] = [];
-  /** Requests currently in the pipeline (synthesizing / waiting / playing). */
-  private pipelineDepth = 0;
-  /** Bumped on every teardown so stale pipeline releases are ignored. */
-  private pipelineGeneration = 0;
-  private pumpScheduled = false;
-  /** Set when TURN_COMPLETED arrives; the turn releases once the pipeline drains. */
-  private completedTurnId: string | null = null;
-  private readonly voiceService: Pick<VoiceService, 'synthesizeStream'>;
-  private readonly configManager: Pick<ConfigManager, 'get'>;
-  private readonly player: StreamingAudioPlayer;
-  private readonly notify?: (message: string) => void;
-  private readonly now: () => number;
-  private readonly setIntervalImpl: typeof setInterval;
-  private readonly clearIntervalImpl: typeof clearInterval;
-  private readonly setTimeoutImpl: typeof setTimeout;
-  private readonly clearTimeoutImpl: typeof clearTimeout;
-
-  constructor(options: SpokenTurnControllerOptions) {
-    this.voiceService = options.voiceService;
-    this.configManager = options.configManager;
-    this.player = options.player;
-    this.notify = options.notify;
-    this.now = options.now ?? (() => Date.now());
-    this.setIntervalImpl = options.setInterval ?? setInterval;
-    this.clearIntervalImpl = options.clearInterval ?? clearInterval;
-    this.setTimeoutImpl = options.setTimeout ?? setTimeout;
-    this.clearTimeoutImpl = options.clearTimeout ?? clearTimeout;
-  }
-
-  submitNextTurn(prompt: string): boolean {
-    const normalized = prompt.trim();
-    if (!normalized) return false;
-    this.stop();
-    if (!this.player.available) {
-      if (!this.noPlayerNoticed) {
-        this.noPlayerNoticed = true;
-        this.notify?.('[TTS] Text response will continue, but live audio is unavailable. Install mpv or ffplay.');
-      }
-      return false;
-    }
-    // Reset the no-player notice if player becomes available again.
-    this.noPlayerNoticed = false;
-    this.pendingPrompt = normalized;
-    return true;
-  }
-
-  /**
-   * Returns whether speech was actually ACTIVE when stopped. The notice only
-   * prints in that case — stop() on an idle controller used to notify anyway,
-   * spamming "[TTS] Spoken output stopped." on every Ctrl+C (an earlier replay
-   * fix); callers use the return to decide whether the press "did a
-   * job" (see handleCtrlC's consume-on-speech-stop).
-   */
-  stop(message?: string): boolean {
-    const wasActive = this.pendingPrompt !== null || this.activeTurnId !== null
-      || this.chunker !== null || this.abortControllers.size > 0;
-    this.pendingPrompt = null;
-    this.activeTurnId = null;
-    this.chunker?.reset();
-    this.chunker = null;
-    this.stopTimer();
-    this.resetPipeline();
-    for (const controller of this.abortControllers) controller.abort();
-    this.abortControllers.clear();
-    this.player.stop();
-    this.playbackChain = Promise.resolve();
-    this.errorReportedForTurn = false;
-    if (message && wasActive) this.notify?.(`[TTS] ${message}`);
-    return wasActive;
-  }
-
-  /**
-   * Exit-path teardown: drops everything not yet audible (pending arm,
-   * buffered text, queued chunks) but lets the audio the user is already
-   * hearing finish naturally, capped at `drainTimeoutMs`, before the hard
-   * stop. Deliberate interrupts (Ctrl+C, /tts stop, turn cancel) keep their
-   * instant path through stop(); this is only for exiting the app while the
-   * final audio of a completed response is still draining.
-   */
-  async stopForExit(drainTimeoutMs = 2000): Promise<void> {
-    this.pendingPrompt = null;
-    this.activeTurnId = null;
-    this.chunker?.reset();
-    this.chunker = null;
-    this.stopTimer();
-    this.resetPipeline();
-    // Cancel chunks that have not started playing; the chunk currently in the
-    // sink is not in this set (its controller is released before playback).
-    for (const controller of this.abortControllers) controller.abort();
-    this.abortControllers.clear();
-    await this.player.waitForDrain(drainTimeoutMs);
-    // Backstop: anything still alive after the window is torn down hard.
-    this.stop();
-  }
-
-  handleTurnEvent(event: TurnEvent): void {
-    if (event.type === 'TURN_SUBMITTED') {
-      this.maybeStartTurn(event.turnId, event.prompt);
-      return;
-    }
-    if (!this.activeTurnId || event.turnId !== this.activeTurnId) return;
-
-    if (event.type === 'STREAM_DELTA') {
-      this.queueTexts(this.chunker?.push(event.content) ?? []);
-      return;
-    }
-    if (event.type === 'STREAM_END') {
-      return;
-    }
-    if (event.type === 'TURN_COMPLETED') {
-      this.finishTurn(event.turnId);
-      return;
-    }
-    if (event.type === 'TURN_CANCEL' || event.type === 'TURN_ERROR' || event.type === 'PREFLIGHT_FAIL') {
-      this.stop(event.type === 'TURN_CANCEL' ? 'Spoken output stopped.' : 'Spoken output stopped because the turn did not complete.');
-    }
-  }
-
-  private maybeStartTurn(turnId: string, prompt: string): void {
-    if (!this.pendingPrompt) return;
-    if (prompt.trim() !== this.pendingPrompt) return;
-    this.pendingPrompt = null;
-    this.activeTurnId = turnId;
-    this.chunkSequence = 0;
-    this.errorReportedForTurn = false;
-    this.chunker = new TtsTextChunker({ now: this.now });
-    this.playbackChain = Promise.resolve();
-    this.resetPipeline();
-    this.startTimer();
-    this.notify?.(`[TTS] Live playback queued through ${this.player.label}.`);
-  }
-
-  private finishTurn(turnId: string): void {
-    if (turnId !== this.activeTurnId) return;
-    this.queueTexts(this.chunker?.flushAll() ?? []);
-    this.stopTimer();
-    this.completedTurnId = turnId;
-    // Nothing pending and nothing in flight releases immediately; otherwise
-    // the last pipeline slot to free performs the release.
-    this.maybeReleaseTurn();
-  }
-
-  private resetPipeline(): void {
-    this.pendingTexts = [];
-    this.pipelineDepth = 0;
-    this.pipelineGeneration++;
-    this.completedTurnId = null;
-  }
-
-  private startTimer(): void {
-    this.stopTimer();
-    this.timer = this.setIntervalImpl(() => {
-      if (!this.activeTurnId || !this.chunker) return;
-      this.queueTexts(this.chunker.flushDue());
-    }, 250);
-  }
-
-  private stopTimer(): void {
-    if (!this.timer) return;
-    this.clearIntervalImpl(this.timer);
-    this.timer = null;
-  }
-
-  /**
-   * Chunker output does NOT map 1:1 to synthesis requests. Text queues here
-   * and the pump merges everything pending into one request whenever a
-   * pipeline slot is free — so the request count tracks how often the model
-   * out-paces the audio, not how many sentences it wrote. A short answer that
-   * arrives before the first pump tick is exactly one request.
-   */
-  private queueTexts(chunks: readonly string[]): void {
-    for (const chunk of chunks) {
-      if (chunk.trim()) this.pendingTexts.push(chunk);
-    }
-    if (this.pendingTexts.length > 0) this.schedulePump();
-  }
-
-  /**
-   * Deferred one tick so text delivered in the same synchronous burst (fast
-   * deltas, or a turn that completes instantly) coalesces into a single
-   * request instead of firing per sentence boundary.
-   */
-  private schedulePump(): void {
-    if (this.pumpScheduled) return;
-    this.pumpScheduled = true;
-    queueMicrotask(() => {
-      this.pumpScheduled = false;
-      this.pump();
-    });
-  }
-
-  private pump(): void {
-    while (this.activeTurnId && this.pendingTexts.length > 0 && this.pipelineDepth < SYNTHESIS_PIPELINE_WINDOW) {
-      this.dispatchChunk(this.takeMergedText());
-    }
-    this.maybeReleaseTurn();
-  }
-
-  /** Merge everything pending into one request, capped at the per-request text limit. */
-  private takeMergedText(): string {
-    let merged = '';
-    while (this.pendingTexts.length > 0) {
-      const next = this.pendingTexts[0]!;
-      if (!merged && next.length > SYNTHESIS_MERGE_MAX_CHARS) {
-        // A single oversized entry (e.g. a large end-of-turn flush): split at
-        // a word boundary under the per-request cap; the rest stays queued.
-        const cut = findSplitIndex(next, SYNTHESIS_MERGE_MAX_CHARS);
-        this.pendingTexts[0] = next.slice(cut).trim();
-        return next.slice(0, cut).trim();
-      }
-      if (merged && merged.length + 1 + next.length > SYNTHESIS_MERGE_MAX_CHARS) break;
-      merged = merged ? `${merged} ${next}` : next;
-      this.pendingTexts.shift();
-    }
-    return merged;
-  }
-
-  private dispatchChunk(text: string): void {
-    const turnId = this.activeTurnId;
-    if (!turnId || !text.trim()) return;
-    const sequence = ++this.chunkSequence;
-    const generation = this.pipelineGeneration;
-    this.pipelineDepth++;
-    const abortController = new AbortController();
-    this.abortControllers.add(abortController);
-    const resultPromise = this.synthesizeWithRetry(text, turnId, sequence, abortController.signal)
-      .then((result) => ({ ok: true as const, result }))
-      .catch((error: unknown) => ({ ok: false as const, error }));
-
-    this.playbackChain = this.playbackChain.then(async () => {
-      try {
-        if (abortController.signal.aborted) {
-          this.abortControllers.delete(abortController);
-          return;
-        }
-        const result = await resultPromise;
-        this.abortControllers.delete(abortController);
-        // Re-check after the await: an abort that landed while synthesis was
-        // in flight (deliberate stop or exit) makes the rejection expected —
-        // it must not be reported, and it must not hard-stop a sink that may
-        // still be draining the previous chunk.
-        if (abortController.signal.aborted) return;
-        if (!result.ok) {
-          // Retries are exhausted (or the failure was not transient). Skip
-          // just this chunk and keep speaking the rest of the turn — a gap in
-          // speech beats losing the whole response.
-          this.reportSkippedChunk(result.error);
-          return;
-        }
-        await this.player.play(result.result.chunks, {
-          format: String(result.result.format ?? 'mp3'),
-          signal: abortController.signal,
-        });
-      } finally {
-        this.releasePipelineSlot(generation);
-      }
-    }).catch((error: unknown) => {
-      this.abortControllers.delete(abortController);
-      this.reportError(error);
-    });
-  }
-
-  private releasePipelineSlot(generation: number): void {
-    if (generation !== this.pipelineGeneration) return;
-    this.pipelineDepth = Math.max(0, this.pipelineDepth - 1);
-    if (this.pendingTexts.length > 0) {
-      this.schedulePump();
-      return;
-    }
-    this.maybeReleaseTurn();
-  }
-
-  private maybeReleaseTurn(): void {
-    if (!this.completedTurnId || this.completedTurnId !== this.activeTurnId) return;
-    if (this.pendingTexts.length > 0 || this.pipelineDepth > 0 || this.pumpScheduled) return;
-    this.activeTurnId = null;
-    this.completedTurnId = null;
-    this.chunker = null;
-    this.abortControllers.clear();
-  }
-
-  private async synthesizeWithRetry(text: string, turnId: string, sequence: number, signal: AbortSignal): Promise<VoiceSynthesisStreamResult> {
-    for (let attempt = 0; ; attempt++) {
-      try {
-        return await this.synthesize(text, turnId, sequence, signal);
-      } catch (error) {
-        const retryable = attempt < SYNTHESIS_RETRY_DELAYS_MS.length
-          && !signal.aborted
-          && isTransientSynthesisError(error);
-        if (!retryable) throw error;
-        await this.delay(SYNTHESIS_RETRY_DELAYS_MS[attempt]!, signal);
-      }
-    }
-  }
-
-  /** Abortable backoff sleep — an abort clears the timer and rejects, so a stop mid-backoff leaves nothing running. */
-  private delay(ms: number, signal: AbortSignal): Promise<void> {
-    return new Promise((resolve, reject) => {
-      if (signal.aborted) {
-        reject(new Error('Synthesis retry cancelled'));
-        return;
-      }
-      const timer = this.setTimeoutImpl(() => {
-        signal.removeEventListener('abort', onAbort);
-        resolve();
-      }, ms);
-      const onAbort = () => {
-        this.clearTimeoutImpl(timer);
-        reject(new Error('Synthesis retry cancelled'));
-      };
-      signal.addEventListener('abort', onAbort, { once: true });
-    });
-  }
-
-  private synthesize(text: string, turnId: string, sequence: number, signal: AbortSignal): Promise<VoiceSynthesisStreamResult> {
-    return this.voiceService.synthesizeStream(readOptionalConfigString(this.configManager, 'tts.provider'), {
-      text,
-      voiceId: readOptionalConfigString(this.configManager, 'tts.voice'),
-      format: 'mp3',
-      speed: readOptionalConfigNumber(this.configManager, 'tts.speed'),
-      signal,
-      metadata: {
-        source: 'goodvibes-agent',
-        feature: 'live-tts',
-        turnId,
-        sequence,
-      },
-    });
-  }
-
-  /**
-   * One synthesis request failed after its retries. Report once per turn and
-   * keep going — the rest of the response still plays.
-   */
-  private reportSkippedChunk(error: unknown): void {
-    if (this.errorReportedForTurn) return;
-    this.errorReportedForTurn = true;
-    this.notify?.(`[TTS] Skipping part of the spoken response — synthesis kept failing (${summarizeError(error)}). Playback continues with the rest.`);
-  }
-
-  private reportError(error: unknown): void {
-    if (this.errorReportedForTurn) return;
-    this.errorReportedForTurn = true;
-    this.activeTurnId = null;
-    this.chunker = null;
-    this.stopTimer();
-    this.resetPipeline();
-    for (const controller of this.abortControllers) controller.abort();
-    this.abortControllers.clear();
-    this.player.stop();
-    this.playbackChain = Promise.resolve();
-    this.notify?.(`[TTS] Live playback stopped: ${summarizeError(error)}`);
-  }
-}
-
-/**
- * Transient = worth a bounded retry: rate/concurrency limits (HTTP 429),
- * transient server errors (5xx), and network-level drops. The SDK's voice
- * providers throw plain Error strings with the HTTP status embedded in the
- * message, so classification is by message content.
- */
-function isTransientSynthesisError(error: unknown): boolean {
-  const message = (error instanceof Error ? error.message : String(error)).toLowerCase();
-  if (message.includes('429') || message.includes('rate limit') || message.includes('rate_limit')
-    || message.includes('too many requests') || message.includes('concurrent')) return true;
-  if (/http 5\d\d/.test(message)) return true;
-  return message.includes('fetch failed') || message.includes('network')
-    || message.includes('timed out') || message.includes('timeout')
-    || message.includes('econnreset') || message.includes('socket');
-}
-
-/** Split point at or under `limit`, preferring the last word boundary. */
-function findSplitIndex(text: string, limit: number): number {
-  const space = text.lastIndexOf(' ', limit);
-  return space > 0 ? space : limit;
-}
-
-function readOptionalConfigString(configManager: Pick<ConfigManager, 'get'>, key: ConfigKey): string | undefined {
-  const value = String(configManager.get(key) ?? '').trim();
-  return value || undefined;
-}
-
-/**
- * readOptionalConfigNumber — reads a numeric config value by key.
- *
- * Accepts a string key and casts it, returning undefined when the value is
- * absent, zero, or not a finite positive number.
- */
-function readOptionalConfigNumber(configManager: Pick<ConfigManager, 'get'>, key: string): number | undefined {
-  const raw = configManager.get(key as ConfigKey);
-  const value = typeof raw === 'number' ? raw : parseFloat(String(raw ?? ''));
-  return isFinite(value) && value > 0 ? value : undefined;
-}