Skip to content

spec(t27mcp): rename Tool to McpTool, one definition per type name (Refs #10416) - #10463

Closed
gHashTag wants to merge 2 commits into
masterfrom
rename-mcptool-10416
Closed

gHashTag wants to merge 2 commits into
masterfrom
rename-mcptool-10416

Conversation

@gHashTag

Copy link
Copy Markdown
Owner

Refs #10416

The types ratchet reports Tool as a NEW conflict on master: specs/provider/schema.t27 already defines Tool, and specs/t27mcp/tool_schema.t27 added a second definition. This renames the t27mcp one to McpTool (definition, TOOLS array type, 27 literals, one local). Nothing else in the repository refers to the type.

  • The diff is a pure rename: reversing McpTool to Tool reproduces the previous file byte for byte.
  • The seal was recomputed with t27c built from master 7552b89 and zig 0.16.0: 12 of 12 tests pass, as before.
  • The ledger is not blessed.

🤖 Generated with Claude Code

gHashTag and others added 2 commits October 18, 2026 19:44
…ion (Refs #10416)

specs/provider/schema.t27 already defines Tool; tool_schema.t27 added a second
definition and the types ratchet reports Tool as a NEW conflict on master.
Rename only; the seal is refreshed in the next commit.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-18 12:59:12 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 45
PRs with All Checks Green 5
READY 0
FAILING 45
PENDING 0
NO CHECKS YET 0

These columns do not partition: 0 + 45 + 0 + 0 = 45, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=fad12492937a != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@gHashTag

Copy link
Copy Markdown
Owner Author

Closing as a duplicate: #10460 (merged as 982894c) already renames tool_schema's Tool to McpTool, which is the same change this PR makes. The seal refresh here is superseded by the one that landed with it.

@gHashTag gHashTag closed this Oct 18, 2026
auto-merge was automatically disabled October 18, 2026 13:20

Pull request was closed

@gHashTag
gHashTag deleted the rename-mcptool-10416 branch October 18, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant