Repository navigation
fix(ci): resolve inherited release/v3.8.51 base-red CI failures - #11608
Merged
diegosouzapw merged 3 commits intoAug 26, 2026
Conversation
…ed connections The generic retry-budget exemption (isRecoverableExpiredWithRetryBudget) was accidentally widening the diegosouzapw#8182 boundary: GitHub access-token-only connections expired for a non-no_refresh_token reason (e.g. invalid_grant) were being kept eligible for probes instead of remaining terminal. Exclude GitHub access-token-only connections from the generic retry-budget exemption. Only the dedicated isRecoverableGithubCopilotNoRefresh path (exact no_refresh_token shape) self-heals those connections. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Align tests, config, and docs with intentional base-branch behavior changes: - env-doc-sync: document OMNIROUTE_CORPUS_CACHE_SIZE in .env.example + ENVIRONMENT.md - route-guard: LOCAL_ONLY_API_GET_EXEMPTIONS now has 2 entries (cloudflared GET diegosouzapw#11531) - known-symbols: fixture matches typed const lazyExecutors declaration - mutation-test-coverage: register routeGuard + quotaScoring/quotaStrategies test files - bundler flag: resolveNextBuildBundlerFlag now defaults to --turbopack - pack-artifact: include bin/cli/utils/volatileEnvPath.mjs - i18n: fill remaining Vietnamese translations (no ___ markers/empty fallbacks) - opencode executor: tools array passes through intact (truncation moved to chatCore) - qwen model alias: qwen-cloud-token-plan/qwen-web serve bare qwen3.8-max (no -preview rewrite) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Merged
4 of 5 tasks
diegosouzapw
pushed a commit
that referenced
this pull request
Aug 26, 2026
…es registration (#9282) (#11609) Merged via /merge-batch (lote 2026-08-26 batch 2, v3.8.51). Dois conflitos, ambos triviais: (1) stryker.conf.json — mesma linha com indentação diferente, mantida a indentação correta; (2) ts7-executor-shared-shapes.test.ts — mesma correção já coberta por #11591/#11608 (mergeados neste lote), mantido o describe/comentário melhor deste PR sobre a asserção já validada. Validado: 4/4 testes passando. Obrigado pela contribuição.
# Conflicts: # scripts/build/pack-artifact-policy.ts # src/i18n/messages/vi.json # tests/unit/authz/route-guard-version-get-exemption.test.ts # tests/unit/bun-support.test.ts # tests/unit/check-known-symbols.test.ts # tests/unit/qwen38-max-bare-id-alias.test.ts # tests/unit/ts7-executor-shared-shapes.test.ts
diegosouzapw
merged commit Aug 26, 2026
1d476e4
into
diegosouzapw:release/v3.8.51
4 of 7 checks passed
This was referenced Aug 26, 2026
diegosouzapw
added a commit
to HouMinXi/OmniRoute
that referenced
this pull request
Aug 29, 2026
…in after merge Merging origin/release/v3.8.51 showed the skip assertion from diegosouzapw#11611 no longer matches production: GitHub access-token-only expired connections with remaining retry budget are probed via refreshCopilotToken (fetch is called; lastHealthCheckAt is set). That boundary was pinned by diegosouzapw#11592 after the diegosouzapw#11608 carve-out. Keep the probe assertion. Refs diegosouzapw#11611 diegosouzapw#11592 Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
muhamadgalihsaputra
pushed a commit
to niyatna/NiyatnaRoute
that referenced
this pull request
Sep 27, 2026
…es registration (diegosouzapw#9282) (diegosouzapw#11609) Merged via /merge-batch (lote 2026-08-26 batch 2, v3.8.51). Dois conflitos, ambos triviais: (1) stryker.conf.json — mesma linha com indentação diferente, mantida a indentação correta; (2) ts7-executor-shared-shapes.test.ts — mesma correção já coberta por diegosouzapw#11591/diegosouzapw#11608 (mergeados neste lote), mantido o describe/comentário melhor deste PR sobre a asserção já validada. Validado: 4/4 testes passando. Obrigado pela contribuição.
muhamadgalihsaputra
pushed a commit
to niyatna/NiyatnaRoute
that referenced
this pull request
Sep 27, 2026
…osouzapw#11608) Merged via /merge-batch (lote 2026-08-26 batch 2, v3.8.51). 7 conflitos, todos triviais/duplicados (mesmos base-reds já corrigidos por PRs paralelas mergeadas neste lote — diegosouzapw#11580/diegosouzapw#11582/diegosouzapw#11583/diegosouzapw#11585/diegosouzapw#11588/diegosouzapw#11589/diegosouzapw#11590/diegosouzapw#11591/diegosouzapw#11609): mantida a versão já validada nesses casos. Validado: 68/68 testes passando. Obrigado por resolver os base-reds.
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
3 participants
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.
Fixes all 10 inherited CI failures on
release/v3.8.51that block PR #11567. The PR's own objective (typecheck:core) passes; every failing job here is inherited from the base branch.Fixes
OMNIROUTE_CORPUS_CACHE_SIZEin.env.exampleanddocs/reference/ENVIRONMENT.md(commit d6fd55c).LOCAL_ONLY_API_GET_EXEMPTIONSlegitimately has 2 entries now (/api/system/version+/api/tunnels/cloudflaredfix(authz): restrict tunnel process routes locally #11531); test updated and doc comment expanded (d6fd55c).const lazyExecutors: Record<...>declaration (d6fd55c).route-guard+quotaScoring/quotaStrategiestest files instryker.conf.json(d6fd55c).resolveNextBuildBundlerFlagnow defaults to--turbopack; test updated (d6fd55c).bin/cli/utils/volatileEnvPath.mjstoPACK_ARTIFACT_REQUIRED_PATHS(d6fd55c).___markers/empty fallbacks (d6fd55c).no_refresh_tokenreason stay terminal (2af5854).qwen-cloud-token-plan/qwen-webserve bareqwen3.8-max(no-previewrewrite); test updated to per-provider expectation (d6fd55c).Verification
Coverage note
test:coverageexits non-zero only due to pre-existing, Windows-environment-specific failures inherited from the base branch (cert store query,zcode-executorEPERM temp cleanup,adobe-firefly,api-key-lifecycle,auggieCLI spawning,binaryManagersymlink/rollback). None relate to these 10 fixes; all changed test files pass. Coverage itself is well above the 60/60/60/60 floor.Off-limits (owned by PR #11567)
open-sse/translator/response/openai-responses/pureHelpers.tsconfig/quality/eslint-suppressions.json