Repository navigation
fix: make MCP server tools correct and testable - #49
Merged
Merged
Conversation
- Resolve data files next to server.py so tools work from any working directory, and fail at startup instead of returning empty results (#22) - shader_helper no longer labels URP-only HLSL as HDRP or Built-in code; validates pipeline and lists available effects (#24) - scaffold_script validates C# identifiers, replaces the TARGET_TYPE placeholder with an inferred or explicit target_type, gives each state machine its own state interface, and notes Editor/test assembly needs (#25) - lookup_api: multi-word ranked search, category validation, Unity 6 linearVelocity/linearDamping entries, and new input, ui, networking and editor entries so no advertised category is empty (#26) - Add pytest suite with a stdio MCP smoke test, run it in CI on Python 3.10 and 3.12, drop unused fastapi/uvicorn, bound mcp to <2, and add pip to Dependabot (#36) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MpEiWjv38TrdFcgqU8ER5L
The C# conventions rule told the AI to use file-scoped namespaces and primary constructors. Unity 6 compiles C# 9, so both fail to build. Replace them with an explicit C# 9 constraint (#30). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MpEiWjv38TrdFcgqU8ER5L
Remove the ECS scaffolding claim and the "200+ entries" count, and document target_type, ranked search, URP-only HLSL, data path behavior, and how to run the tests. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MpEiWjv38TrdFcgqU8ER5L
github-actions
Bot
added
rules
mcp-server
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
This was referenced Oct 9, 2026
Closed
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.
Summary
server.py(works from any working directory) and exits with a clear error if a data file is missingshader_helperno longer labels URP HLSL as HDRP or Built-in codescaffold_scriptvalidates names, fills in the inspected/drawn target type, gives each state machine its own state interface, and notes Editor folder and test assembly requirementslookup_api:Rigidbody.velocityreplaced withlinearVelocity, new deprecation warnings, 15 entries for the empty input/UI/networking/editor categories (75 total), multi-word ranked searchmcp<2, Dependabot for pipFixes #22, #24, #25, #26, #30
Refs #36 (linter and
tools/stub cleanup still open)Test plan
python -m pytest mcp-server/testspasses locally (48 tests; 41 fail against the old code)🤖 Generated with Claude Code
https://claude.ai/code/session_01MpEiWjv38TrdFcgqU8ER5L