[P2][cleanup] Supprimer les API spéculatives et mortes #26

New issue

Open

opened 2026-07-31 21:05:14 +00:00 by mathieu · 1 comment

mathieu commented

2026-07-31 21:05:14 +00:00

Owner

Constat

Trois surfaces n'ont aucun caller de production: plugin External/Capability et son transport (internal/plugin/plugin.go:22-55, :68-123), textpipe.Text/Stage (internal/textpipe/pipeline.go:32-42), sysexec.New/FilterEnvironment (internal/sysexec/sysexec.go:45, :145-163). Elles dupliquent des politiques ou promettent des extensions inexistantes.

Tâche

Supprimer ces API internes et leurs tests synthétiques. Garder le runtime local plugin, les contrats concrets du pipeline et config.CredentialHelperEnvironment. Ne pas introduire de couche de remplacement.

Critères d'acceptation

Aucun code plugin n'importe HTTP/netip ni construit de netguard.
Aucun contrat générique de stage inutilisé ne reste.
Une seule implémentation du filtrage d'environnement subsiste.
Plain/Markdown gardent timeout, panic recovery et erreurs typées.

Validation

go test -race ./internal/plugin ./internal/plugin/markdown ./internal/textpipe ./internal/sysexec ./internal/config ./internal/core

## Constat Trois surfaces n'ont aucun caller de production: plugin `External`/`Capability` et son transport (`internal/plugin/plugin.go:22-55`, `:68-123`), `textpipe.Text`/`Stage` (`internal/textpipe/pipeline.go:32-42`), `sysexec.New`/`FilterEnvironment` (`internal/sysexec/sysexec.go:45`, `:145-163`). Elles dupliquent des politiques ou promettent des extensions inexistantes. ## Tâche Supprimer ces API internes et leurs tests synthétiques. Garder le runtime local plugin, les contrats concrets du pipeline et `config.CredentialHelperEnvironment`. Ne pas introduire de couche de remplacement. ## Critères d'acceptation - Aucun code plugin n'importe HTTP/netip ni construit de netguard. - Aucun contrat générique de stage inutilisé ne reste. - Une seule implémentation du filtrage d'environnement subsiste. - Plain/Markdown gardent timeout, panic recovery et erreurs typées. ## Validation `go test -race ./internal/plugin ./internal/plugin/markdown ./internal/textpipe ./internal/sysexec ./internal/config ./internal/core`