Repository navigation
t27mcp: rename tool_schema's Tool to McpTool (types ratchet red on master) (Closes #10459) - #10460
Merged
Merged
Conversation
… with provider/schema's Tool (Closes #10459) #10446 added a second type named Tool beside specs/provider/schema.t27's Tool, and the types ratchet on master turned red ("+ Tool NEW conflict"). The newer one is renamed; the spec is resealed (12/12 tests, 0 vacuous) and tri types ratchet is CLEAN on the lab. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
gHashTag
enabled auto-merge (squash)
October 18, 2026 12:48
This was referenced Oct 18, 2026
Merged
Contributor
This was referenced Oct 18, 2026
This was referenced Oct 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #10459
#10446 added
pub struct Toolinspecs/t27mcp/tool_schema.t27.specs/provider/schema.t27already declares aTooltype, so the types ratchet on master turned red with "+ Tool NEW conflict", and every open PR inherits that red check. This PR renames the newer type toMcpTool, which touches 30 lines: the struct, theTOOLSarray type, its 27 rows and onelet. No other spec in the tree names the type.Checked on the lab against master 7552b89:
t27c checkis OK, with 0 errors and 0 warnings.t27c test-reportpasses 12 of 12, with 0 vacuous.t27c seal --saveregenerated.trinity/seals/t27mcp_T27mcpToolSchema.json. Itsspec_hashmatches the file.tri types ratchetis CLEAN, with 107 names in the ledger and 107 observed.There is one effect downstream. gHashTag/t27-mcp#24 carries the
gen-jstable of this spec, and that table exportsTool. Nothing there imports the name, but the table and seal in #24 must be regenerated after this PR merges.Refs #10444
🤖 Generated with Claude Code