[P1][errs] Donner priorité à la classe du fallback effectif #17

New issue

Open

opened 2026-07-31 21:05:12 +00:00 by mathieu · 2 comments

mathieu commented

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

Owner

Constat

failedAttempts enveloppe la classe du fallback mais garde les deux causes (internal/stt/stt.go:673-705). errs.Classify cherche d'abord context.DeadlineExceeded (internal/errs/errs.go:64-78): un timeout primaire peut donc masquer un auth_failed fallback alors que le provider effectif reste fallback.

Tâche

Définir une priorité explicite pour les erreurs agrégées de providers, sans modifier la classification des contextes ordinaires.

Critères d'acceptation

Timeout primaire + auth/503 fallback prend la classe fallback.
ProviderFromError reste cohérent avec la classe.
errors.Is retrouve toujours les deux causes.
Un deadline nu reste timeout.

Validation

go test -race ./internal/errs ./internal/stt ./internal/session ./internal/cli

## Constat `failedAttempts` enveloppe la classe du fallback mais garde les deux causes (`internal/stt/stt.go:673-705`). `errs.Classify` cherche d'abord `context.DeadlineExceeded` (`internal/errs/errs.go:64-78`): un timeout primaire peut donc masquer un `auth_failed` fallback alors que le provider effectif reste fallback. ## Tâche Définir une priorité explicite pour les erreurs agrégées de providers, sans modifier la classification des contextes ordinaires. ## Critères d'acceptation - Timeout primaire + auth/503 fallback prend la classe fallback. - `ProviderFromError` reste cohérent avec la classe. - `errors.Is` retrouve toujours les deux causes. - Un deadline nu reste `timeout`. ## Validation `go test -race ./internal/errs ./internal/stt ./internal/session ./internal/cli`