Skip to content

cleanup: rimuovi 4 CLI comandi senza consumatori esterni#400

Merged
Gabrymi93 merged 2 commits into
mainfrom
cleanup/remove-unused-commands
Jun 24, 2026
Merged

cleanup: rimuovi 4 CLI comandi senza consumatori esterni#400
Gabrymi93 merged 2 commits into
mainfrom
cleanup/remove-unused-commands

Conversation

@Gabrymi93

Copy link
Copy Markdown
Member

Sintesi

Rimuove 4 comandi CLI deprecati che non hanno consumatori esterni (nessun riferimento in altri repo, skills, o CI).

Comando Sostituito da Motivo
toolkit query toolkit layer -m sql Nessun consumer esterno
toolkit preview toolkit scout <URL> Nessun consumer esterno
toolkit scaffold clean toolkit run init Nessun consumer CLI diretto
toolkit review-readiness toolkit run full / inspect summary Usato solo internamente

Cosa cambia

  • Refactor / performance

Impatto su contratti pubblici

Nessuno — tutti i comandi erano hidden=True.

Verifica

pytest tests/ -q --ignore=tests/test_smoke_e2e_flow.py --ignore=tests/test_smoke_scout.py
  • pytest -m core passa (1053 test, 0 fallimenti)

Note

  • I backend module rimangono (usati internamente da run.py e MCP)
  • DuckDB MCP (_local/mcp/duckdb/) aggiornato

@Gabrymi93

Copy link
Copy Markdown
Member Author

Fix applicati (commit 64b25e3)

Coverage 81% — CI should pass now:

  • Rimosso cmd_review_readiness.py (dead module, logica in readiness_ops.py)
  • Rimosso cmd_query.py, cmd_scaffold.py (dead code, già deregistrati)
  • Rimosso 503 linee di codice morto → copertura da 79.41% a 81%
  • Aggiunti 3 test per check_column_types() (copre la nuova funzione)

Stale message fixed:

  • cmd_run.py:789: toolkit scaffold cleantoolkit run init

@Gabrymi93 Gabrymi93 merged commit 81dd90f into main Jun 24, 2026
3 checks passed
@Gabrymi93 Gabrymi93 deleted the cleanup/remove-unused-commands branch June 24, 2026 16:49
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