Repository navigation
merge: upstream 0b7403c (token speed + M3.1 Flash fallback) — en attente du fix upstream - #80
Merged
Merged
Conversation
(cherry picked from commit 51ac3b2) Co-authored-by: tournierjc <tournierjc@users.noreply.github.com>
Node 24's process.exit() returns before a piped stderr stream is fully flushed. On a busy Ubuntu runner the desktop bash truncation test then captured about 32KiB and missed the tail row. Exit via process.exitCode so the runtime drains both streams before the command ends. Co-authored-by: tournierjc <tournierjc@users.noreply.github.com>
This was referenced Sep 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sync amont
0b7403c(MiniMax-AI/minimax-code, 28/09)Commits amont intégrés (2)
51ac3b2fix: measure token speed over model generation time (fix: measure token speed over model generation time MiniMax-AI/minimax-code#373) — pris. La vitesse affichée (ligne d'activité et résumé de tour) est le nombre de tokens de sortie divisé par le temps de génération du modèle, hors attente du premier token et hors exécution d'outils.docs/tui-capabilities.mdreçoit la section « Output speed » sans réintroduire le libellé MCode : les titres de terminal restent « Kinetick Code ».0b7403cfeat(config): add M3.1 Flash Preview to the fallback catalog (feat(config): add M3.1 Flash Preview to the fallback catalog MiniMax-AI/minimax-code#375) — pris. AjouteMiniMax-M3.1-Flash-Previewau catalogue de repliMINIMAX_MODELS(fenêtre de contexte, effort, capabilities fichiers) et le testbuiltin-model-fallback.test.ts. Donnée de catalogue, pas une fonctionnalité réservée au produit MiniMax.Commit amont ignoré (1)
836580echore: bump version to 0.5.6 (chore: bump version to 0.5.6 MiniMax-AI/minimax-code#374) — skip. Ne fait que passer minimax-code / @minimax/code de 0.5.5 à 0.5.6. Le fork restekinetick-code/@mavis/codeen 0.6.6.Déjà dans main (ne pas réappliquer)
4198174,258adde,a30a898,0f6ad52via merge: upstream 4198174 (bash guidance) #76 / merge: upstream 0f6ad52 (0.5.5 runtime improvements) — en attente du fix upstream #79 (tip forka80e00c).Crible télémétrie / identité
Aucun commit de télémétrie dans cet intervalle. Invariants du fork conservés : pas de
docs/telemetry.md, pas derequireTelemetry/createTuiBusinessEventTracker, pas de réintroduction deminimax-code,mcode(identité produit),MiniMax Codeou@minimax/code. Versions racine et TUI inchangées (0.6.6). Le test de catalogue reprend les conventions déjà utilisées par le fork (MINIMAX_DATA_DIR,__MAVIS_RUNTIME_DATA_DIR, préfixe temporairemcode-model-fallback-, commemcode-config-permissions-).Conflits résolus
test/vitest-suites.json: union des suites fork (copilot, headless exit, session cost, status metrics, etc.) et des nouveaux tests amont (decode-timing, stream-events, turn-output-rate, content-timing, builtin-model-fallback).release/public-source.json: union des chemins, puis régénération vianode scripts/source-inventory.mjs --write(4296 fichiers).docs/tui-capabilities.md,chat-controller.tset son test : merge automatique. Le wording Kinetick Code /KCODE_DEFAULT_AGENT_NAMEdu fork est conservé ; seul le calcul de débit de tokens vient de l'amont.packages/config/src/config.ts: merge automatique de l'entrée de modèle, sans toucherBRAND(.kinetick) ni la version.Change
Mesure du débit de tokens sur le temps de génération réel, et ajout de MiniMax-M3.1-Flash-Preview au catalogue de repli de premier lancement. Pas de changement de version ni d'identité produit.
9628780corrige le verify Ubuntu déjà rouge sur main : le test de troncature bash à sortie d'erreur large quitte viaprocess.exitCodepour que Node 24 vide stderr avant la fin du processus.Validation
pnpm verify(profilfull) PASS — 15/15 gates surc9b95cb, Linux, Node.js 22.22.2, pnpm 9.12.0.check:sourcePASS (4296 fichiers)check:tsconfigPASS (129 exports)test:release-toolsPASS (50 pass / 1 skip / 0 fail)lint:tuiPASStypecheckPASSbuildPASScheck:standalonePASScheck:egressPASStest:artifactPASStest:capabilitiesPASS (207 fichiers, 5350 passés, 15 skipped, 0 failed)test:status-contractPASS (9)test:smokePASS (20 pass / 1 skip / 0 fail)test:byokPASS (4/4)test:policyPASS (142)git diff --checkproprec9b95cb(run 36386311146) :verify (macos-latest, 24)PASS ;verify (ubuntu-latest, 24)FAIL surpackages/agent-tools/test/desktop/local-bash-output.test.ts(« caps a large nonzero-exit result… »,error-row-399absent,original_bytes=32020). Même échec déjà présent sur maina80e00c(run 36344499821,original_bytes=32125).9628780: le fixture appelaitprocess.exit(7). Sur Node 24, cet appel revient avant que stderr (pipe) soit entièrement vidé. Un runner Ubuntu chargé ne recevait qu'environ 32 KiB, donc la preview head/tail s'arrêtait verserror-row-305et l'assertionerror-row-399échouait. Le script fixe maintenantprocess.exitCode = 7, ce qui laisse le runtime vider stdout et stderr. L'assertion head/tail est inchangée. Contrôle local Node 24.21.0 : 15/15 puis 5/5 sous charge CPU surpackages/agent-tools/test/desktop/local-bash-output.test.ts.test:windows,test:sandbox(macOS) ettest:release-package(archive npm). Pas d'appel fournisseur réel.Publication and contribution checks
release/public-source.json; new tests are declared intest/vitest-suites.jsonwhere applicable.Maintainer handoff
Publication scope or license changes (if any): none.
Shared-source port: this PR is the reviewed fork port of upstream
51ac3b2and0b7403c.836580eis intentionally not ported.9628780is a fork-side CI fix for the Node 24 stderr flush race; it is not an upstream port.