Skip to content

feat(mcp): reduce noise in MCP tools — strip nulls, fix bugs, unify errors - #478

Merged
Gabrymi93 merged 11 commits into
mainfrom
feat/mcp-noise-reduction
Aug 29, 2026
Merged

feat(mcp): reduce noise in MCP tools — strip nulls, fix bugs, unify errors#478
Gabrymi93 merged 11 commits into
mainfrom
feat/mcp-noise-reduction

Conversation

@Gabrymi93

Copy link
Copy Markdown
Member

Riduzione rumore nei tool MCP per AI agenti

9 commit che migliorano l'UX dei tool MCP per AI: meno token sprecati, errori uniformi, bug fixes.

Bug fixes

  • Error handling unificato: tutti i return {"error": ...} sostituiti con raise ToolkitClientError(...) — logging uniforme via guard_timed
  • mode='schema'/'profile' funzionano: il guard SQL ora è mode-aware, non blocca più i mode non-SQL
  • by_dataset filtra i bridge: filter_graph(by_dataset=...) ora filtra anche i bridge, non solo le entità

Noise reduction

  • shape() utility: strip_nulls + strip_empty_lists applicati a find, overview, preview, probe, status, preflight
  • toolkit_contract eliminato: duplicato identico di toolkit_pipeline(action='contract')
  • status droppa paths_info: sezione sovrapposta con summary rimossa
  • preflight strip nulls: ogni source entry pulita dai campi null
  • probe routed=True di default: meno parametri da ricordare, routing automatico sempre attivo

Output prima/dopo (char count)

tool                        prima    dopo    delta
─────────────────────────────────────────────────
dataset(preflight)         21,607   ~18K    -16%
dataset(status)            ~12K     ~3.7K   -70%
dataset(find, limit=3)     ~5K      ~3.5K   -30%
query(preview)             ~400     ~256    -36%

Test

77/77 test passano (test_mcp_server + test_mcp_toolkit_client + test_registry_graph).

File changed

  • toolkit/mcp/response.py — nuove utility (strip_nulls, compact, shape)
  • toolkit/mcp/server.py — error handling, shape application, remove toolkit_contract
  • toolkit/registry/graph.py — by_dataset bridge filtering
  • tests/test_mcp_server.py — test aggiornati per nuovi error patterns
  • tests/test_registry_graph.py — test per bridge filtering

@Gabrymi93
Gabrymi93 merged commit 123d322 into main Aug 29, 2026
3 checks passed
@Gabrymi93
Gabrymi93 deleted the feat/mcp-noise-reduction branch August 29, 2026 10:00
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