Skip to content

fix(mcp): make open(path) round-trip and return not_found details - #33

Closed
Razpines wants to merge 1 commit into
feat/mcp-source-coverage-search-diagnosticsfrom
fix/mcp-open-path-roundtrip
Closed

Razpines wants to merge 1 commit into
feat/mcp-source-coverage-search-diagnosticsfrom
fix/mcp-open-path-roundtrip

Conversation

@Razpines

Copy link
Copy Markdown
Owner

Summary

  • make DocStore.open_doc(path=...) resolve docs by exact/normalized origin_path
  • add canonical URL lookup support in open_doc(path=...)
  • keep doc-id lookup behavior and add path-derived fallback doc-id normalization
  • change MCP open() missing-doc response to structured
    ot_found payload with attempted inputs
  • add tests covering origin_path round-trip and canonical URL lookup

Issues

Testing

  • python -m py_compile src/unity_docs_mcp/mcp_server.py src/unity_docs_mcp/tools/ops.py tests/test_mcp_meta.py tests/test_ops_open_path_lookup.py
  • $env:PYTHONPATH='C:\projects\UnityRAG\src'; python -m pytest tests/test_ops_open_path_lookup.py -q

@Razpines
Razpines deleted the branch feat/mcp-source-coverage-search-diagnostics February 28, 2026 20:23
@Razpines Razpines closed this Feb 28, 2026
@Razpines
Razpines deleted the fix/mcp-open-path-roundtrip branch February 28, 2026 20:24
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