Skip to content

Derive MCP error context from tool names - #57

Open
kennethbruskiewicz wants to merge 3 commits into
mainfrom
codex/refactor-mcp-error-prefixes
Open

Derive MCP error context from tool names#57
kennethbruskiewicz wants to merge 3 commits into
mainfrom
codex/refactor-mcp-error-prefixes

Conversation

@kennethbruskiewicz

Copy link
Copy Markdown
Collaborator

Summary

  • remove the MCP server's per-tool error-prefix registry
  • derive human-readable error context from the reflected tool function name
  • keep protocol conversion focused on translating shared invocation errors
  • cover existing and newly added tool names without synchronized metadata

Testing

  • ============================= test session starts ==============================
    platform darwin -- Python 3.13.5, pytest-8.4.2, pluggy-1.6.0
    rootdir: /Users/kenneth/Sync/ken/active/90_work/Translator_component_toolkit
    configfile: pyproject.toml
    plugins: anyio-4.10.0, cov-6.2.1
    collected 21 items

tests/test_invocation.py ............ [ 57%]
tests/test_server.py ...... [ 85%]
tests/test_mcp_contract.py ... [100%]

================================ tests coverage ================================
_______________ coverage: platform darwin, python 3.13.5-final-0 _______________

Name Stmts Miss Cover Missing

TCT/TCT.py 893 805 10% 59, 63-68, 79-84, 103, 107-311, 335-346, 355-358, 368-383, 386-394, 398-402, 410-426, 449-496, 509-609, 622-722, 728-762, 771-798, 803-842, 857-900, 961, 989-998, 1011, 1033-1036, 1053-1055, 1072, 1103-1123, 1154-1156, 1188-1189, 1218, 1245-1286, 1291-1356, 1360-1406, 1413-1455, 1465-1512, 1523-1575, 1582-1595, 1600-1611, 1615-1620, 1624-1637, 1643-1683, 1688-1711, 1733-1763, 1768-1772, 1776-1823, 1827-1874, 1878-1925, 1929-1951, 1954-1966, 1970-2016, 2019-2037, 2081-2087, 2091-2215, 2218-2237, 2240-2258
TCT/TCT_Visualization.py 55 55 0% 2-99
TCT/TCT_neighborhood_finder.py 155 146 6% 47-126, 138-217, 276-338
TCT/TCT_network_annotator.py 100 100 0% 1-263
TCT/TCT_pathfinder.py 169 152 10% 66-115, 121-153, 161-191, 204-313, 332-363, 381-384, 403-412, 429-432, 450-459, 523-593
TCT/init.py 4 0 100%
TCT/config.py 45 21 53% 22-26, 72-77, 81-83, 97-98, 110-111, 116, 122, 127
TCT/graph_downloader.py 38 38 0% 3-64
TCT/interfaces/init.py 0 0 100%
TCT/interfaces/cli.py 115 115 0% 3-191
TCT/interfaces/invocation.py 73 15 79% 70-71, 76-78, 84-95
TCT/interfaces/mcp.py 23 1 96% 52
TCT/interfaces/tools.py 48 17 65% 40, 72, 94, 116, 131, 143, 169, 193, 203, 221, 242, 264, 287, 305-306, 330-331
TCT/kg_loader.py 237 237 0% 3-378
TCT/name_resolver.py 84 74 12% 18-20, 59-78, 96-113, 118-121, 151-180, 198-217
TCT/node_annotator.py 21 15 29% 17-19, 23, 49-64
TCT/node_normalizer.py 94 86 9% 18-20, 53-96, 116-135, 148-195
TCT/server.py 2 0 100%
TCT/translator_kpinfo.py 75 68 9% 17-19, 43-153
TCT/translator_metakg.py 193 185 4% 8-41, 65-97, 131-138, 168-331, 348-355
TCT/translator_node.py 64 18 72% 64, 69, 73, 78-94
TCT/translator_query.py 89 78 12% 26-44, 55-65, 87-132, 157-169, 176-194, 219-251
TCT/trapi.py 22 16 27% 55-79, 118-135

TOTAL 2599 2242 14%
Coverage HTML written to dir htmlcov
Coverage XML written to file coverage.xml
============================== 21 passed in 7.51s ============================== (21 passed)

  • full suite: 83 passed, 1 skipped, 13 unrelated live-service failures (NodeNorm CI returned 404; node-annotation responses no longer matched two fixtures)

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