fix(memory): typer aussi les schémas de SORTIE des outils MCP

[cyberlife-coder]

Le correctif précédent n'avait traité que l'entrée. Dette identifiée à ce moment-là, refermée ici.

Le rouge

11 untyped `items` across 18 advertised output schemas: {
    "list_working_contexts: $.sessions.items = {"$ref":"#/$defs/WorkingContextSession"}",
    "recall: $.memories.items = {"$ref":"#/$defs/RecollectionDto"}",
    "why: $.edges.items = {"$ref":"#/$defs/MemoryEdgeDto"}",
    ...
}

Pourquoi ce n'est pas cosmétique

Les SDK MCP officiels valident structuredContent contre l'outputSchema annoncé. Un $ref que le client ne sait pas résoudre est un résultat qu'il peut rejeter — plus dur que le cas de l'entrée, où le serveur avait au moins pu répondre.

Deux causes, pas une

wire_safe_output_schema n'appliquait que widen_id_properties, jamais l'inliner récursif. Corriger ça a fait tomber 11 → 6.

Les 6 restants venaient d'ailleurs : cinq outils ne déclaraient aucun output_schema. rmcp en dérivait un automatiquement, qui échappait donc à tout post-traitement. recall, recall_fused, recall_where, why et list_working_contexts le déclarent maintenant explicitement.

Vérification

11 → 0. Le test générique parcourt tous les outils, donc le prochain qui rend un tableau d'objets est couvert par construction, pas parce que quelqu'un aura pensé à ajouter un cas.

20 suites vertes, clippy pedantic propre, fmt, version-sync et les contrats passés en local avant le push.

--features extract échoue encore sur cette branche : c'est la dette traitée par #1610, indépendante.

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity

Metric	Results
Complexity	0

View in Codacy

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.}