Skip to content

refactor: rimuovi compat legacy inutilizzati dal parser config#25

Merged
Gabrymi93 merged 5 commits intomainfrom
feat/remove-legacy-config-compat
Mar 8, 2026
Merged

refactor: rimuovi compat legacy inutilizzati dal parser config#25
Gabrymi93 merged 5 commits intomainfrom
feat/remove-legacy-config-compat

Conversation

@Gabrymi93
Copy link
Copy Markdown
Member

Summary

Closes #24

Questa PR restringe il contract del parser config del toolkit alle forme canoniche realmente usate nei progetti attivi.

In particolare rimuove compatibilità legacy che oggi non porta più valore e aumenta la confusione del config model.

Cosa entra

Rimosso

  • raw.source
  • raw.sources[].plugin
  • raw.sources[].id
  • clean.read scalar form
  • clean.read.csv.*
  • clean.sql_path
  • mart.sql_dir

Comportamento nuovo

Le forme sopra non vengono più normalizzate o ignorate.
Ora falliscono con errore esplicito che indica la shape corretta.

Perché

Nel workspace non risultano config attive che usano queste forme.

Quindi:

  • il beneficio di compatibilità è basso
  • il costo di mantenimento e il rischio di ambiguità sono alti

Il caso peggiore erano clean.sql_path e mart.sql_dir:

  • non erano veri alias vivi
  • erano solo campi ignorati
  • potevano far credere che il config fosse valido quando in realtà non lo era

File principali

  • toolkit/core/config_models.py
  • toolkit/core/csv_read.py
  • toolkit/clean/duckdb_read.py
  • tests/test_config.py
  • docs/config-schema.md
  • docs/advanced-workflows.md
  • CHANGELOG.md

Commit inclusi

  • 70e16ab refactor: rimuovi compat legacy del parser config
  • 53aae93 refactor: rimuovi clean.sql_path e mart.sql_dir

Validation

Test passati:

  • python -m pytest tests/test_config.py -q

Scope

PR volutamente limitata al parser/config contract.
Non tocca:

  • cross_year
  • status / inspect
  • legacy_aliases
  • altri rami non direttamente coinvolti nel config parser

@Gabrymi93 Gabrymi93 linked an issue Mar 8, 2026 that may be closed by this pull request
@Gabrymi93 Gabrymi93 merged commit 6751250 into main Mar 8, 2026
5 checks passed
@Gabrymi93 Gabrymi93 deleted the feat/remove-legacy-config-compat branch March 8, 2026 19:07
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.

Rimuovere i compat legacy inutilizzati dal parser config

1 participant