dangernoodle-io / mcpkit / 29206352684

96%

push

github

feat(mcpkit)!: deferred tool registration + Risk/Group primitives (MC-43) (#26)

- add mcpkit.Risk (ReadOnly/Write/Destructive), a required positional AddTool param that fail-closes on unannotated write/destructive tools
- auto-derive ToolAnnotations from Risk via mcpx.RiskAnnotations (MC-42) when the caller left Annotations nil; an explicit Annotations still wins
- add mcpkit.Group(name) ToolOption, arbitrary per-tool bookkeeping consumed by the registry's byGroup map
- defer tool registration out of AddTool into an unexported registry of pendingTool closures; App.finalize (called once, idempotently, from Run/Connect/HTTPHandler) registers every pending tool against the live mcpx.Server
- this is the load-bearing MC-43 step of the MC-40 tier/gate epic: it introduces the deferred-registration seam a later gate (MC-44) will filter before anything registers, but MC-43 itself is behavior-preserving -- every tool still ends up registered, just at Run/Connect/HTTPHandler time instead of at New
- update every AddTool caller (mcpkit_test.go, testkit/harness_test.go, examples/server, examples/minimal, examples/http) to pass an explicit Risk; all are ReadOnly query/demo tools

BREAKING CHANGE: AddTool now requires a Risk argument and registers tools at Run/Connect/HTTPHandler instead of at New.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

50 of 51 new or added lines in 1 file covered. (98.04%)

1345 of 1402 relevant lines covered (95.93%)

1.11 hits per line