Repository navigation
Conversation
…nce#3501) (cheaper-inference#4826) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 1/13)
…heaper-inference#3501) (cheaper-inference#4824) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 2/13)
…e#3501) (cheaper-inference#4811) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 3/13)
… completo, cheaper-inference#3501) (cheaper-inference#4817) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 4/13)
…3501) (cheaper-inference#4827) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 5/13)
… usage non-streaming, cheaper-inference#3501) (cheaper-inference#4832) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 6/13)
…ardrail post-call, cheaper-inference#3501) (cheaper-inference#4831) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 7/13)
…n-streaming, cheaper-inference#3501) (cheaper-inference#4828) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 8/13)
…de resposta non-streaming, cheaper-inference#3501) (cheaper-inference#4835) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 9/13)
…ce#3089 JSON→SSE streaming, cheaper-inference#3501) (cheaper-inference#4833) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 10/13)
…de resposta streaming, cheaper-inference#3501) (cheaper-inference#4836) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 11/13)
…-store streaming, cheaper-inference#3501) (cheaper-inference#4829) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 12/13)
…orms streaming, cheaper-inference#3501) (cheaper-inference#4837) Integrated into release/v3.8.36 (cheaper-inference#3501 chatCore extraction stack 13/13)
…ease-acceleration) (cheaper-inference#4857) * feat(quality): add check:test-runner-api gate (vitest-only dirs must use vitest API) * feat(release): reusable CHANGELOG i18n-mirror sync script * chore(ops): add prune-stale-worktrees.sh (dry-run by default) * ci(quality): run test-runner-api + docs-all + vitest + full unit suite on PR->release fast-path --------- Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
…ce#4806) + limite EPSILON não bloqueia (cheaper-inference#4830) Integrated into release/v3.8.36 — quota-exclusive qtSd/ listing (cheaper-inference#4806) + EPSILON placeholder no longer blocks; rebuilt from stale base (3 defining commits cherry-picked clean over release tip)
…ce#4569) (cheaper-inference#4769) Integrated into release/v3.8.36 — Google Flow video-generation provider (cheaper-inference#4569), release-green validated (typecheck + 21 tests + file-size)
…_CREDENTIALS (cheaper-inference#4694, cheaper-inference#4720) (cheaper-inference#4796) Integrated into release/v3.8.36 — auth on compression run-telemetry + OMNIROUTE_EVAL_CREDENTIALS doc, release-green validated (typecheck + 3 tests + env-doc-sync)
…rough (port from 9router#1157) (cheaper-inference#4624) Integrated into release/v3.8.36 — port (rebuilt from stale base; defining commit cherry-picked clean over release tip, release-green validated)
…rmat providers (cheaper-inference#4625) Integrated into release/v3.8.36 — port (rebuilt from stale base; defining commit cherry-picked clean over release tip, release-green validated)
…ocks (cheaper-inference#4633) Integrated into release/v3.8.36 — port (rebuilt from stale base; defining commit cherry-picked clean over release tip, release-green validated)
…thropic providers (cheaper-inference#4650) Integrated into release/v3.8.36 — port (rebuilt from stale base; defining commit cherry-picked clean over release tip, release-green validated)
…heaper-inference#4651) Integrated into release/v3.8.36 — port (rebuilt from stale base; defining commit cherry-picked clean over release tip, release-green validated)
…nference#4654) Integrated into release/v3.8.36 — port (rebuilt from stale base; defining commit cherry-picked clean over release tip, release-green validated)
…heaper-inference#4656) Integrated into release/v3.8.36 — port (rebuilt from stale base; defining commit cherry-picked clean over release tip, release-green validated)
…ttings (cheaper-inference#4659) Integrated into release/v3.8.36 — port (rebuilt from stale base; defining commit cherry-picked clean over release tip, release-green validated)
) (cheaper-inference#4629) Integrated into release/v3.8.36 — kiro region SSRF guard (GHSA-6mwv-4mrm-5p3m), port rebuilt clean over release tip
…eaper-inference#4628) Integrated into release/v3.8.36 — port rebuilt clean over release tip, release-green validated
…sages (cheaper-inference#4657) Integrated into release/v3.8.36 — anthropic-compat validation via POST /v1/messages (port 584cf66a), rebuilt clean + baseline; release-green
…cheaper-inference#4658) Integrated into release/v3.8.36 — port rebuilt clean over release tip, release-green validated
…eaper-inference#4540) (cheaper-inference#4990) * fix(sse): soft-penalize exhausted providers in auto-combo scoring (cheaper-inference#4540) * chore(quality): document STATUS_SOFT_DEPRIORITIZE_FACTOR + rebaseline combo.ts for cheaper-inference#4540 --------- Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
…els in test-all (cheaper-inference#4887) (cheaper-inference#4991) * fix(dashboard): switch to visible filter after auto-hiding failed models in OAuth provider test-all (cheaper-inference#4887) * test(dashboard): move cheaper-inference#4887 test into tests/unit/ui so a CI runner collects it --------- Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
…cheaper-inference#3981) (cheaper-inference#5009) Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
…ative Gemini paths (cheaper-inference#5003) (cheaper-inference#5008) * fix(antigravity): default safetySettings to all-OFF for parity with native Gemini paths (cheaper-inference#5003) * docs(changelog): restore cheaper-inference#3981 pollinations entry eaten by merge --------- Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
…vent silent model substitution (cheaper-inference#4665) (cheaper-inference#5010) * fix(chatgpt-web): map advertised gpt-5.5/5.4-pro/5.2-pro slugs to prevent silent model substitution (cheaper-inference#4665) MODEL_MAP was missing the advertised catalog ids gpt-5.5, gpt-5.5-pro, gpt-5.4-pro and gpt-5.2-pro, so MODEL_MAP[model] ?? model sent the dot-form id verbatim to the ChatGPT backend-api, which silently rejected it and served the default Plus model. Map each to its dash-form slug. gpt-4-5 is already dash-form and falls through correctly, so it is intentionally left unmapped. Extends the executor MODEL_MAP test with the four ids and adds a drift guard asserting every advertised dot-form catalog id reaches the backend in dash-form (never verbatim), guarding future catalog<->map drift. file-size: tests/unit/chatgpt-web.test.ts frozen baseline 2809->2855 (+46) for the added test cases and drift-guard test; executor source unchanged in baseline. * docs(changelog): restore cheaper-inference#3981/cheaper-inference#5003 entries eaten by merge --------- Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
…/api/combos (cheaper-inference#5005) (cheaper-inference#5011) * feat(combos): add editable per-combo description field persisted via /api/combos (cheaper-inference#5005) * docs(changelog): restore cheaper-inference#3981/cheaper-inference#5003/cheaper-inference#4665 entries eaten by merge --------- Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
Integrated into release/v3.8.36
…tion (cheaper-inference#5024) Integrated into release/v3.8.36
…heaper-inference#5027) Integrated into release/v3.8.36
…inference#5022) Integrated into release/v3.8.36
…mpression (cheaper-inference#5023) Integrated into release/v3.8.36
…rence#5024, cheaper-inference#5027 (release notes credit)
…r-inference#5006) (cheaper-inference#5028) * fix(api): stop /api/system/env/repair 500 on packaged install — lazy createRequire in sync-env.mjs (cheaper-inference#5006) scripts/dev/sync-env.mjs ran createRequire(import.meta.url) at module top-level. When webpack bundles it into the standalone env-repair route, import.meta.url is frozen to the build-machine path (file:///home/runner/...) and createRequire throws during module evaluation, so the whole route module fails to load and every GET returns HTTP 500 — breaking the onboarding wizard on packaged/global installs. - Move createRequire into the guarded better-sqlite3 block (only place that needs it); a bad import.meta.url now returns the safe default. - resolveRootDir() falls back to process.cwd() when fileURLToPath throws. - route.ts passes an explicit rootDir (process.cwd()) so the helper never derives the root from the frozen import.meta.url, matching the .env target used by createEnvBackup(). - Regression guard: assert sync-env.mjs has no top-level createRequire + getEnvSyncPlan(oauth) works with explicit rootDir without throwing. * docs(changelog): restore cheaper-inference#4993/cheaper-inference#5023/cheaper-inference#5024/cheaper-inference#5027 + custom-system-prompt/headroom entries eaten by release merge * chore(quality): rebaseline 3 inherited base-reds from release merge Files NOT touched by this PR — grew on release/v3.8.36 via --admin merges and inherited here through 'git merge origin/release': - open-sse/executors/base.ts 1414->1416 (cheaper-inference#4993 Ollama Cloud max-effort) - src/lib/db/settings.ts 1149->1151 (cheaper-inference#5023 custom system prompt) - src/app/(dashboard)/.../endpoint/EndpointPageClient.tsx 2570->2612 (custom system prompt UI)
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request refactors the codebase by removing several OAuth service files (including Antigravity, Claude, Codex, Gemini, GitHub, OpenAI, Qoder, and Qwen services) and their main export index from the open-sse/services/ directory. Additionally, references in comments within antigravityProjectBootstrap.ts and usage.ts have been updated to point to the new location under src/lib/oauth/providers/. There are no review comments to evaluate, and I have no additional feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
JxnLexn
marked this pull request as ready for review
June 25, 2026 21:34
Contributor
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Collaborator
|
Thanks for this work, @JxnLexn 🙏 — closing after a maintainer review of all 46 open refactor PRs in this batch (yours included). This is not a rejection of the effort. Why closed (applies to the whole batch): every branch here is 44–50 commits behind This PR specifically: ✅ SOUND (with VPS caveat) — the legacy class-based OAuth services have only a doc-comment reference; the real flows live in What happens to it: ✅ Sound — going into the "salvage" batch. We will reimplement it cleanly on top of the current release (with tests per our Rule #18), crediting you as co-author ( We ran a full case-by-case triage. The approaches worth keeping are being recorded internally with author credit, so when we pick them up the attribution travels with the code. Thank you for pushing on this. 🙌 |
diegosouzapw
added a commit
that referenced
this pull request
Jul 2, 2026
The src/lib/oauth/services/ service-class hierarchy is superseded — the live OAuth flow runs through src/lib/oauth/providers.ts + providers/. The old per-provider 'class *Service extends OAuthService' implementations and their barrel had zero production or test references. Removed oauth/openai/github/claude/codex/antigravity/ qwen/qoder + the index barrel (-1559 LOC). Kept kiro.ts, cursor.ts, codexImport.ts (routes import them directly by path, never via the deleted barrel). Proven safe by typecheck:core staying green (a live reference would fail the build) + a filesystem guard test pinning the removal. Salvage of closed PR #5039. gaps v3.8.42 - T10 (5.7).
Merged
tkgo11
pushed a commit
to tkgo11/OmniRoute
that referenced
this pull request
Sep 23, 2026
…ference#5838) The src/lib/oauth/services/ service-class hierarchy is superseded — the live OAuth flow runs through src/lib/oauth/providers.ts + providers/. The old per-provider 'class *Service extends OAuthService' implementations and their barrel had zero production or test references. Removed oauth/openai/github/claude/codex/antigravity/ qwen/qoder + the index barrel (-1559 LOC). Kept kiro.ts, cursor.ts, codexImport.ts (routes import them directly by path, never via the deleted barrel). Proven safe by typecheck:core staying green (a live reference would fail the build) + a filesystem guard test pinning the removal. Salvage of closed PR cheaper-inference#5039. gaps v3.8.42 - T10 (5.7).
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.
Summary
Validation
Split out of #4964 and rebased onto release/v3.8.36.