key-store/dispatch
1
0
Fork 0
Code Pull requests 4 Releases Activity Actions

harness: fetch MCP tool schemas on demand via describe_tools #28

Open
devex wants to merge 1 commit from devex/runner-schema-ondemand into main AGit
pull from: devex/runner-schema-ondemand
merge into: key-store:main
Conversation 1 Commits 1 Files changed 3 +313 -1
devex commented 2026-09-14 07:41:27 +00:00
Member
Copy link

Per-turn hub tool definitions ride trimmed to names+descriptions;
the full input schema is cached at listing time and served only
when the model asks describe_tools (max 10 names, 16KB cap).
Unknown/built-in/empty requests fail with a visible message that
names the usable tools instead of dropping to built-ins silently.
Also nil-guard addMCPTools maps (unit use hit a nil-map panic).

KSE-0197; Redmine #62.

Per-turn hub tool definitions ride trimmed to names+descriptions; the full input schema is cached at listing time and served only when the model asks describe_tools (max 10 names, 16KB cap). Unknown/built-in/empty requests fail with a visible message that names the usable tools instead of dropping to built-ins silently. Also nil-guard addMCPTools maps (unit use hit a nil-map panic). KSE-0197; Redmine #62. 
Per-turn hub tool definitions ride trimmed to names+descriptions;
the full input schema is cached at listing time and served only
when the model asks describe_tools (max 10 names, 16KB cap).
Unknown/built-in/empty requests fail with a visible message that
names the usable tools instead of dropping to built-ins silently.
Also nil-guard addMCPTools maps (unit use hit a nil-map panic).

KSE-0197; Redmine #62.
devex commented 2026-09-14 07:41:35 +00:00
Author
Member
Copy link

Senior review requested. Scope: on-demand MCP schema fetch (describe_tools), per-turn trimmed hub definitions, nil-guard hardening. Evidence: go build ./..., go vet, full internal/harness + internal/mcphttp suites green. Redmine #62 carries design + test notes. I will not merge without senior approval.

Senior review requested. Scope: on-demand MCP schema fetch (`describe_tools`), per-turn trimmed hub definitions, nil-guard hardening. Evidence: `go build ./...`, `go vet`, full `internal/harness` + `internal/mcphttp` suites green. Redmine #62 carries design + test notes. I will not merge without senior approval.
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin +refs/pull/28/head:devex/runner-schema-ondemand
git switch devex/runner-schema-ondemand
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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
key-store/dispatch!28
No description provided.