Skip to content

refactor(arch): elimine les 3 cycles de dependances inter-modules#117

Merged
Destynova2 merged 1 commit intodevelopfrom
refactor/break-dep-cycles
Apr 10, 2026
Merged

refactor(arch): elimine les 3 cycles de dependances inter-modules#117
Destynova2 merged 1 commit intodevelopfrom
refactor/break-dep-cycles

Conversation

@Destynova2
Copy link
Copy Markdown
Contributor

Contexte

CC-3 du plan cli-cycle : cli-audit-tangle a identifie 3 cycles de dependances
mutuelles qui bloquent la testabilite et ralentissent la compilation incrementale.

Cycles elimines

Cycle Strategie Fichiers cles
features <-> server MCP handlers HTTP deplaces dans server/mcp_handlers.rs +server/mcp_handlers.rs, -features/mcp/server/ handlers
features <-> providers pricing extrait dans src/pricing.rs (module feuille) +pricing.rs, features/token_pricing re-exporte
cli <-> providers AuthType+ProviderConfig deplaces dans cli/config.rs providers re-exporte pour compat

Impact

  • 12 fichiers, +871/-828 lignes (principalement des deplacements)
  • DAG de dependances propre : cli -> providers -> pricing, server -> features
  • Backward compat preservee via re-exports
  • 728 tests passent, clippy clean

Restructure pour etablir un DAG propre entre modules :

1. features <-> server : deplace les handlers HTTP MCP (handle_mcp_rpc,
   handle_matrix_report, grob_configure) de features/mcp/server/ vers
   server/mcp_handlers.rs. La logique business MCP pure (query, bench,
   calibrate) reste dans features/mcp/server/methods.rs sans dep AppState.

2. features <-> providers : extrait ModelPricing, KNOWN_PRICING et pricing()
   dans un nouveau module feuille src/pricing.rs sans dep interne.
   features/token_pricing re-exporte pour backward compat.

3. cli <-> providers : deplace AuthType et ProviderConfig de providers/mod.rs
   vers cli/config.rs (ou vivent tous les types TOML). providers/mod.rs
   re-exporte pour backward compat.

728 tests passent. Clippy clean.
@Destynova2 Destynova2 merged commit bad4daa into develop Apr 10, 2026
38 checks passed
@Destynova2 Destynova2 deleted the refactor/break-dep-cycles branch April 10, 2026 20:26
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