fix: memory put defaults source and reports the failing field #214

Merged
che-guevara merged 1 commit from fix/memory-put-source into main 2026-11-01 21:29:57 +00:00
Collaborator

Fix fuer den Memory-Put-Defekt: jeder Payload ohne source scheiterte am Store mit generischem 'Invalid memory entry'.

  • put(): source faellt auf ctx.sessionKey zurueck (entspricht der Service-Konvention); in normalized und entry.
  • validate(): Fehlermeldung nennt jetzt das konkrete Feld ('Invalid memory entry: '), Semantik und Reihenfolge unverändert.
  • plugins.lock.json: Pinn revision ebba1cf (plugins PR #36: typisierter entry-Parameter, pi-Validation lehnt malformed Input schon am Tool-Aufruf ab).
  • test/memory.test.ts: neuer Test (Source-Default + Feld-feindliche Fehlermeldung).

Verifiziert: typecheck ok, Build ok, 1198/1198 Tests gruen, Review-Gate passed (2f920e15, 1 Round, 0 blockers).

Gate-Note (nicht-blockierende P2s, offengelegt): 'memory-source-default' - kein Cross-Session-Leak, da Store-Reads strikt auf gleiche sessionKey beschränkt sind. 'result-status-exception' - Datei liegt nicht im Changeset (Upstream-Kontext).

Fix fuer den Memory-Put-Defekt: jeder Payload ohne `source` scheiterte am Store mit generischem 'Invalid memory entry'. - put(): `source` faellt auf `ctx.sessionKey` zurueck (entspricht der Service-Konvention); in `normalized` und `entry`. - validate(): Fehlermeldung nennt jetzt das konkrete Feld ('Invalid memory entry: <field>'), Semantik und Reihenfolge unverändert. - plugins.lock.json: Pinn revision ebba1cf (plugins PR #36: typisierter entry-Parameter, pi-Validation lehnt malformed Input schon am Tool-Aufruf ab). - test/memory.test.ts: neuer Test (Source-Default + Feld-feindliche Fehlermeldung). Verifiziert: typecheck ok, Build ok, 1198/1198 Tests gruen, Review-Gate passed (2f920e15, 1 Round, 0 blockers). Gate-Note (nicht-blockierende P2s, offengelegt): 'memory-source-default' - kein Cross-Session-Leak, da Store-Reads strikt auf gleiche sessionKey beschränkt sind. 'result-status-exception' - Datei liegt nicht im Changeset (Upstream-Kontext).
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
harness/core!214
No description provided.