Skip to content

breaking: rimosso il server MCP — migrato a dataciviclab/dcl-mcp - #96

Merged
Gabrymi93 merged 2 commits into
mainfrom
migrate/remove-mcp-server
Oct 16, 2026
Merged

Gabrymi93 merged 2 commits into
mainfrom
migrate/remove-mcp-server

Conversation

@Gabrymi93

Copy link
Copy Markdown
Member

Sintesi

Il server MCP di ACB (agent-context-mcp, tool dataciviclab-context) migra in dataciviclab/dcl-mcp — nuovo server unico del Lab (context + discussions + gcs_query, 15 tool, 3 passate di audit newcomer). ACB resta il builder.

Contesto collegato

Completamento migrazione approvata: dcl-mcp pubblico e collaudato; rimozione concordata (doppio running vecchio+nuovo da evitare).

Cosa cambia

  • Modifica builder (src/) — rimosso mcp_server.py; navigation.py tiene solo i build_* (query-side già migrato)
  • Rimossi agent-context-mcp entry point e extra [mcp] (lab-connectors base, serve HttpClient)
  • Rimossi tests/test_mcp_server.py e test query-side di test_navigation.py (coperti da dcl-mcp/tests)
  • README: sezione Tool MCP → "migrato a dcl-mcp"

Contratti pubblici

  • Nomi tool e server (dataciviclab): invariati in dcl-mcp
  • Breaking: entry point agent-context-mcp rimosso; tool legacy topic_index/search già droppati in dcl-mcp
  • ACB_REPO/ACB_BRANCH: invariate, puntano ancora a questa repo

Verifica

pytest tests/ -q -m "not smoke"   # 100 passed
ruff check src/ tests/             # pass
mypy src/agent_context_builder/    # pass
  • pytest passa
  • grep zero riferimenti mcp_server/agent-context-mcp/dataciviclab-context nel package

Checklist PR

  • Perimetro stretto: una PR = la rimozione del MCP
  • Migrazione concordata in sessione Lab

Note per chi revisiona

Dopo il merge: workspace.toml demuove ACB da essential (PR dedicata in dataciviclab) — il builder non serve più in locale per il MCP.

Il lato serving (mcp_server.py, entry point agent-context-mcp, extra
[mcp]) e il query-side di navigation.py (search_index, explore_ref,
lab_map_summary + helpers) sono migrati in dataciviclab/dcl-mcp insieme
ai tool Discussions e al nuovo gcs_query (3 passate di audit newcomer).

ACB resta il builder: CLI + CI che producono gli artifact sul branch
context, consumati da remoto dal nuovo server.

Rimosso anche:
- tests/test_mcp_server.py (coperti da dcl-mcp/tests)
- test_navigation.py: via i test query-side (coperti da dcl-mcp)
- lab-connectors[mcp] → lab-connectors base (serve solo HttpClient)
@Gabrymi93
Gabrymi93 merged commit 42a9e43 into main Oct 16, 2026
2 checks passed
@Gabrymi93
Gabrymi93 deleted the migrate/remove-mcp-server branch October 16, 2026 11:30
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