fix(stt): préserver échec effectif et provenance des warnings #39

Open

hermes wants to merge 1 commit from fix/stt-correctness into fix/config-policy

pull from: fix/stt-correctness

merge into: mathieu:fix/config-policy

mathieu:main

mathieu:fix/cli-doctor-cancellation

mathieu:fix/text-pipeline-budget

mathieu:fix/ui-lifecycle

mathieu:fix/delivery-correctness

mathieu:fix/build-test-gates

mathieu:fix/session-event-invariants

mathieu:fix/audio-lifecycle

mathieu:fix/config-policy

mathieu:fix/wav-portable-memfd

mathieu:fix/ipc-subscribe-decoder

Conversation 0 Commits 1 Files changed 10 +162 -63

hermes commented 2026-08-07 17:14:06 +00:00

First-time contributor

Copy link

Résumé

• conserve l’échec/provider effectif quand tous les segments mixed+partial échouent
• donne priorité explicite à la classe du fallback tout en préservant errors.Is sur les deux causes
• transporte le provider cleanup dans warnings, événements, JSON CLI et last-result
• remplace les tris d’insertion par sort.Slice partagé et retire la revalidation streaming redondante

Closes #16
Closes #17
Closes #19
Closes #20

Validation

• go test -race ./internal/errs ./internal/stt ./internal/textpipe ./internal/core ./internal/session
• test CLI ciblé de provenance warning sous race
• régressions fallback/tous-échecs 100 fois sous race
• git diff --check

La suite CLI complète reste affectée par le défaut baseline d’umask traité séparément dans le bloc tests/CI.

Dépendance

Cette PR est empilée sur la PR #38. Les branches ont été rebasées et validées dans cet ordre pour éviter les conflits croisés.

## Résumé - conserve l’échec/provider effectif quand tous les segments mixed+partial échouent - donne priorité explicite à la classe du fallback tout en préservant `errors.Is` sur les deux causes - transporte le provider cleanup dans warnings, événements, JSON CLI et last-result - remplace les tris d’insertion par `sort.Slice` partagé et retire la revalidation streaming redondante Closes #16 Closes #17 Closes #19 Closes #20 ## Validation - `go test -race ./internal/errs ./internal/stt ./internal/textpipe ./internal/core ./internal/session` - test CLI ciblé de provenance warning sous race - régressions fallback/tous-échecs 100 fois sous race - `git diff --check` La suite CLI complète reste affectée par le défaut baseline d’umask traité séparément dans le bloc tests/CI. ## Dépendance Cette PR est empilée sur la PR #38. Les branches ont été rebasées et validées dans cet ordre pour éviter les conflits croisés.

hermes added 1 commit 2026-08-07 17:14:06 +00:00

fix(stt): preserve effective failure and warning provenance 95a3b5aa1c

hermes referenced this pull request 2026-08-07 17:14:07 +00:00

[P1][stt] Préserver l’échec provider quand zéro segment ne réussit #16

hermes referenced this pull request 2026-08-07 17:14:07 +00:00

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

hermes referenced this pull request 2026-08-07 17:14:08 +00:00

[P2][textpipe] Conserver le provider des warnings cleanup #19

hermes referenced this pull request 2026-08-07 17:14:08 +00:00

[P2][stt] Remplacer les tris quadratiques dupliqués par un tri standard unique #20

hermes referenced this pull request 2026-08-07 18:02:34 +00:00

[P1][stt] Préserver l’échec provider quand zéro segment ne réussit #16

hermes referenced this pull request 2026-08-07 18:02:35 +00:00

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

hermes referenced this pull request 2026-08-07 18:02:35 +00:00

[P2][textpipe] Conserver le provider des warnings cleanup #19

hermes referenced this pull request 2026-08-07 18:02:35 +00:00

[P2][stt] Remplacer les tris quadratiques dupliqués par un tri standard unique #20

hermes force-pushed fix/stt-correctness from 95a3b5aa1c to b832a24e3a 2026-08-07 18:04:22 +00:00 Compare

hermes changed target branch from main to fix/config-policy 2026-08-07 18:07:30 +00:00

hermes referenced this pull request 2026-08-07 18:07:31 +00:00

fix(delivery): rendre les effets single-shot et bornés #40

Some checks failed

Hide checks

CI / amd64 / Go 1.23.12 (push) Has been cancelled

Details

CI / amd64 / Go 1.26.5 (push) Has been cancelled

Details

CI / arm64 cross-build / Go 1.23.12 (push) Has been cancelled

Details

CI / arm64 cross-build / Go 1.26.5 (push) Has been cancelled

Details

CI / amd64 / Go 1.23.12 (pull_request) Has been cancelled

Details

CI / amd64 / Go 1.26.5 (pull_request) Has been cancelled

Details

CI / arm64 cross-build / Go 1.23.12 (pull_request) Has been cancelled

Details

CI / arm64 cross-build / Go 1.26.5 (pull_request) Has been cancelled

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin fix/stt-correctness:fix/stt-correctness

git switch fix/stt-correctness

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch fix/config-policy

git merge --no-ff fix/stt-correctness

git switch fix/stt-correctness

git rebase fix/config-policy

git switch fix/config-policy

git merge --ff-only fix/stt-correctness

git switch fix/stt-correctness

git rebase fix/config-policy

git switch fix/config-policy

git merge --no-ff fix/stt-correctness

git switch fix/config-policy

git merge --squash fix/stt-correctness

git switch fix/config-policy

git merge --ff-only fix/stt-correctness

git switch fix/config-policy

git merge fix/stt-correctness

git push origin fix/config-policy

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

mathieu/murmur!39

No description provided.