docs(rulebook): PHP MCP rules MCP-019/020 + regenerate indexes#13
Merged
Conversation
Document the two field-based Go MCP rules (MCP-015 no-description, MCP-016 ambiguous-name) in mcp/tool_definition.md, mirroring the Python/TS description and ambiguous-name rules. The Go SDKs (mark3labs/mcp-go and the official go-sdk) are added to the policy's coverage and "does not cover" sections — untyped-params has no analog for statically-typed Go. gen_index.py regenerates every POLICY_INDEX.md: alongside the two new MCP rows it also picks up prior recalibration severity drift in several per-SDK indexes that had not been regenerated after the severity recalibration.
Document the two field-based C# MCP rules (MCP-017 no-description, MCP-018 ambiguous-name) in mcp/tool_definition.md, mirroring the Python/TS/Go rules, and add the official ModelContextProtocol C# SDK to the coverage and "does not cover" sections. Regenerate the MCP and top-level POLICY_INDEX.md.
Document the two field-based PHP MCP rules (MCP-019 no-description, MCP-020 ambiguous-name) in mcp/tool_definition.md, mirroring the Python/TS/Go/C# rules, and add the PHP MCP SDKs (official mcp/sdk + community php-mcp/server) to the coverage and "does not cover" sections. Note that, unlike Go/C#, an untyped-params analog is meaningful for PHP (optional type hints) and is a fast-follow. Regenerate the MCP and top-level POLICY_INDEX.md.
This was referenced Jun 7, 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
1 participant
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.
Documents the two field-based PHP MCP rules (MCP-019 no-description, MCP-020 ambiguous-name) in
docs/Policy/mcp/tool_definition.md, mirroring the Python/TS/Go/C# rules, and adds the PHP MCP SDKs (officialmcp/sdk+ communityphp-mcp/server) to the coverage and "does not cover" sections. Notes that, unlike Go/C#, an untyped-params analog is meaningful for PHP (optional type hints) and is a fast-follow. Regenerates the MCP and top-levelPOLICY_INDEX.md.check_rulebook.py --rules-repo ../trustabl-rulespasses with 0 warnings.Merge order
Stacked on the C# rulebook branch. Merge in order: #11 (Go) → #12 (C#) → this.
Paired with the engine PHP discovery PR and the rules PHP rules PR.