fix: parse string-encoded structured plugin tool arguments #209

Merged
che-guevara merged 1 commit from fix/prepare-arguments-shim into main 2026-11-01 19:42:13 +00:00
Collaborator

Fix fuer den Wrapper-Bug: Modelle, die strukturierte Tool-Args als JSON-Strings emittieren (z.B. glm-5.3-flash), scheitern sonst an der pi-Validation (items muss array/object sein).

  • src/tool-args.ts: prepareStructuredToolArgs() parst den Gesamt-String und einzelne String-Felder zu den deklarierten array/object-Parametern; nicht parsebare/falsch geformte Werte bleiben unverändert (Validation meldet sie weiter).
  • src/runtime/pi.ts: prepareArguments-Hook an jeder Plugin-Tool-Registrierung.
  • test/tool-args.test.ts: 4 Tests, inkl. echter pi-Adapter-Integration.

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

Fix fuer den Wrapper-Bug: Modelle, die strukturierte Tool-Args als JSON-Strings emittieren (z.B. glm-5.3-flash), scheitern sonst an der pi-Validation (items muss array/object sein). - src/tool-args.ts: prepareStructuredToolArgs() parst den Gesamt-String und einzelne String-Felder zu den deklarierten array/object-Parametern; nicht parsebare/falsch geformte Werte bleiben unverändert (Validation meldet sie weiter). - src/runtime/pi.ts: prepareArguments-Hook an jeder Plugin-Tool-Registrierung. - test/tool-args.test.ts: 4 Tests, inkl. echter pi-Adapter-Integration. Verifiziert: typecheck ok, Build ok, 1186/1186 Tests gruen, Review-Gate passed (2ac2db45, 1 Round, 0 blockers, 0 findings).
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!209
No description provided.