Skip to content

feat: parametro table per selezionare tabella mart in toolkit_layer - #423

Merged
Gabrymi93 merged 2 commits into
mainfrom
feat/mart-table-param
Jul 28, 2026
Merged

feat: parametro table per selezionare tabella mart in toolkit_layer#423
Gabrymi93 merged 2 commits into
mainfrom
feat/mart-table-param

Conversation

@Gabrymi93

Copy link
Copy Markdown
Member

Sintesi

Aggiunge parametro table a toolkit_layer per selezionare una specifica tabella mart quando uno slug ne ha più di una (es. conto-annuale ha 83 tabelle mart).

Contesto collegato

Continuazione del lavoro di unificazione iniziato con #419-#422. Il layer mart su GCS è supportato ma senza modo di scegliere quale tabella usare.

Cosa cambia

  • CatalogResolver.resolve_slug(slug, layer='mart', table='mart_top_sa') — filtra per filename senza estensione
  • toolkit_layer(datasets=[...], layer='mart', table='mart_top_sa') — passa il filtro al resolver
  • Se table non specificato, prende la prima tabella (backward compat)

Impatto

  • Documentazione o testi
  • Policy GitHub o template
  • Codice o automazioni
  • Pipeline dati o trasformazioni
  • Contenuti o metadati di dataset
  • Nessun impatto visibile per chi usa il repository

Verifica

  • Test reale: anac_appalti_master / mart_top_sa restituisce i top SA per importo
  • 67/67 test passanti
  • mypy pulito sui file modificati

Controlli

  • Questa PR e' nel repository giusto
  • Ho collegato issue o discussion quando serve
  • Ho verificato l'impatto su documentazione, codice o dati
  • Ho aggiornato solo quello che era davvero necessario
  • I test nuovi o modificati hanno marker (contract/policy/regression/adapter/pure_unit/smoke)
  • Se la PR fixa un bug: il test che lo protegge e' marcato regression con link all'issue

Note per chi revisiona

  • table matcha sul filename senza estensione (es. "mart_top_sa" matcha mart_top_sa.parquet)
  • Disponibile solo in catalog mode (datasets=...)
  • Pipeline mode ha già mart_index per selezionare output locale

- MCP README: 14→18 tool, signature aggiornate, list_candidates deprecato
- README.md: tabella tool completa, test count 55→85+
- CONTRIBUTING.md: test count 55→85+
@Gabrymi93
Gabrymi93 merged commit 19a1c80 into main Jul 28, 2026
2 checks passed
@Gabrymi93
Gabrymi93 deleted the feat/mart-table-param branch July 28, 2026 15:39
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