fix(memory): persist decision content (B-MCP-4 critical data loss)

[h2devx]

Que cambia

Cierra B-MCP-4 (issue #3, critical, data loss) via Option B
(decision tomada en el cuadro comparativo previo, alineada con la
regla durable "siempre priorizar la estabilidad"):

• Migracion 008: agrega columna decisions.content TEXT NOT NULL DEFAULT '', backfill content = rationale para filas existentes,
  rebuild de decisions_fts virtual table incluyendo la nueva
  columna, triggers actualizados con UPDATE OF column-scope.
• Domain: nuevo VO DecisionContent (max 50K chars vs 5K de
  Rationale). Aggregate Decision lleva el campo en record y
  rehydrate.
• Application: RecordDecision port + use case aceptan content
  optional con fallback a rationale (defensa para flujos internos
  no-wire). Importer de handoff + JSON tambien.
• Infrastructure: SqliteDecisionRepository lee/escribe la
  columna; SqliteMemoryProjectionRepository (lado recall) la
  surface en el wire response. JsonMemoryExporter la incluye para
  round-trip lossless.
• Composition: RememberFacadeAdapter ya no descarta
  silenciosamente input.content para kind=decision — lo pasa
  intacto al use case.

Por que

Fixes #3. Wire schema documentaba content como campo top-level
para todas las kinds desde v0.1.0; la tabla decisions no tenia la
columna; la facade descartaba el valor sin error. Recall retornaba
rationale en el campo wire content para empeorar la confusion.

Eleccion entre Option A (drop content del wire) y Option B
(agregar columna + migracion): Option B porque preservar el
contrato publico documentado tiene prioridad sobre la velocidad.

Audit turns/tasks confirmado: ambos tienen sus columnas
dedicadas (summary/description) que reciben content correctamente.
Solo decisions tenia el silent-drop. Sin scope creep.

Tipo de cambio

• fix — bug fix critical (data loss)
• test — agrega tests
• schema — migracion 008

Checklist

• npm run typecheck EXIT=0
• npm run lint y npm run lint:tests EXIT=0
• npm run validate:modules EXIT=0
• npm run build EXIT=0
• npm run test EXIT=0 — 2519 tests passing en 208 archivos (was 2512 in 207)
• Cero any, cero as any, cero // @ts-ignore
• Tests nuevos cubren el cambio
• Wire/protocolo MCP documentado en docs/02 §4.4 (sin cambio — el contrato existente AHORA se honra)
• N/A — no introduce ADR (Option B es el cumplimiento del contrato, no un nuevo principio)
• HANDOFF.md se actualiza al cierre de v0.1.2-beta.3

E2E que validan VALORES

• Los tests asertan valores reales (no solo shape)

tests/integration/N-decision-content-roundtrip.test.ts (2 casos)
sigue la metodologia de Phase-9:

1. Estado conocido: workspace sin decisions.
2. Insertar via wire: mem.remember(kind=decision) con
   rationale SHORT y content LONG, distintos.
3. SQL inspection: SELECT content FROM decisions WHERE id=?
   debe contener el content largo, NO el rationale.
4. Wire round-trip: mem.recall con un token que aparece SOLO
   en content (no en title ni rationale) — debe traer el hit, y
   result.content debe ser el content largo (no el rationale,
   que era el comportamiento pre-fix).
5. Defensive default: insertar via use case interno sin
   content debe persistir content = rationale.

Pre-fix: result.content === rationale (siempre). Post-fix:
result.content === content (lo que el cliente envio).

Notas para el reviewer

Migracion 008 sobre datos existentes:

• Backfill content = rationale (no empty string) para
  preservar searchability via FTS5 sobre la dogfood DB del usuario
  (27 decisions reales). Documentado en el SQL header.
• DROP TABLE + CREATE de decisions_fts necesario porque FTS5 no
  soporta ALTER. Operacion idempotente porque la migration runner
  honra _meta-based applied versions (la migracion no se reaplica).
• Los triggers se reescriben con UPDATE OF title, rationale, content
  para preservar la optimizacion de migration 007 (no reindexar FTS
  en updates de confidence).

Forward-compat de exports legacy:

• JsonMemoryImporter.DecisionSchema marca content como
  optional para consumir snapshots v0.1.0/v0.1.1 que no lo tienen.
  Cuando absent → buildDecision usa rationale.
• SqliteMemoryProjectionRepository.DecisionRowSchema igual:
  content: z.string().optional() con fallback a rationale en el
  preview.

Por que NO se toca turns ni tasks:

• Audit del facade routing:
  • turn: summary: input.content → tiene su columna summary.
  • task: description: input.content → tiene su columna description.
  • Ambos persisten correctamente. Solo decisions tenia el bug.