feat: cli remove validate, absorb batch into run --batch

[Gabrymi93]

Sintesi

Rimosso toolkit validate (coperto da run --dry-run + validazione interna).
Assorbito toolkit batch in toolkit run --batch <file>.
Aggiunto smoke/batch.txt con smoke test offline.

Contesto collegato

Non ci sono issue collegate — Fase 1 del refactor CLI discusso in chat dopo il merge della PR #439.

Cosa cambia

• Bug fix
• Nuova funzionalità del motore
• Nuovo plugin sorgente
• Modifica contratto pubblico (dataset.yml, path output, schema parquet)
• Refactor / performance
• Documentazione
• Dipendenze o CI

Impatto su contratti pubblici

• Struttura dataset.yml (nuovo campo, cambio obbligatorietà)
• Path output (nuovo layer, cambio percorso artifact)
• Schema parquet (nuova colonna, rename, cambio tipo)
• CLI o MCP tool (nuovo comando, cambio parametro)
• API pubblica del toolkit (firma funzione, classe, eccezione)

Se segnato, hai aggiornato downstream? [ ] dataset-incubator — [ ] docs/

Verifica

pytest tests/test_batch_cli.py tests/test_cmd_batch.py tests/test_dataset_loader.py tests/test_cli_path_contract.py tests/test_smoke_templates_contract_years.py tests/test_smoke_templates_golden_path.py -x --tb=short -q
ruff check .

• pytest -m core passa
• ruff check . passa
• mypy toolkit/ passa (2 errori preesistenti in read_excel.py e profile/raw.py, non toccati)
• Modificato o aggiunto test con marker appropriato (contract)

Checklist PR

• Perimetro stretto: una PR = Fase 1 refactor CLI (validate + batch)
• Se nuovo plugin: test + docs inclusi (N/A)
• Issue collegata o motivazione dell'assenza
• Se rimuovo un modulo/funzione pubblica: verificata assenza di import con rg
  e lasciato shim backward compat con DeprecationWarning

Metriche (root-cause-change)

Tipo: cleanup reale + contract alignment

Problema reale: validate e batch duplicavano funzionalità già in run

Contratto riusato/sostituito: _run_batch() in cmd_run.py assorbe la logica batch;
run --dry-run copre già la validazione che faceva validate

Codice rimosso o duplicazione eliminata:

• cmd_validate.py eliminato (110 righe)
• cmd_batch.py da 272 a 65 righe (thin wrapper deprecato)
• Bilancio netto: -32 righe (425 aggiunte, 452 rimosse)
• 6 comandi root → 5 (batch è hidden/deprecato)

Test aggiunti: test_run_batch_end_to_end (@pytest.mark.contract)

Rischio residuo: batch comando hidden esiste ancora come backward compat;
utenti che usano toolkit validate in script devono passare a toolkit run --dry-run

Follow-up obbligatorio: Fase 2 (assorbire contract in inspect --contract)

Note per chi revisiona

Il grosso del refactor è in:

• toolkit/cli/cmd_run.py: nuovo _run_batch() + helper batch condivisi
• toolkit/cli/cmd_batch.py: riscritto come thin wrapper deprecato
• toolkit/cli/cmd_validate.py: eliminato
• smoke/batch.txt: batch list per smoke offline
• Test: aggiornati dove usavano CLI validate o importavano da cmd_validate