refactor(drivers): split codex driver into focused modules behind barrel - #1416
Closed
bradhallett wants to merge 64 commits into
Closed
bradhallett wants to merge 64 commits into
bradhallett wants to merge 64 commits into
Conversation
|
Important Review skippedToo many files! This PR contains 358 files, which is 58 over the limit of 300. To get a review, reduce the PR to 300 files or fewer by splitting it into smaller PRs or changing its base branch. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (358)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
bradhallett
force-pushed
the
refactor/code-health-49
branch
from
September 18, 2026 01:58
f40cccb to
a10e878
Compare
|
@bradhallett is attempting to deploy a commit to the SupaMaus Team on Vercel. A member of the Team first needs to authorize it. |
bradhallett
force-pushed
the
refactor/code-health-49
branch
13 times, most recently
from
September 18, 2026 20:22
f3147aa to
3e985ed
Compare
… dead browser-panel-operation module, and the webhooks/routines re-export shims (importers now use shared/ directly; client-only helpers moved to their owners)
…— the server no longer imports renderer code; unify the byte formatters in src/lib/format-bytes.ts and give the paste-policy test its real home
…dule — boxagent/openai-chat/pi/acp-core drop their hand-rolled skeletons; antigravity reuses the ACP connection instead of re-implementing it
… refreshModels consolidated across claude/codex/acp-core, and codex approval config extracted to codex-approvals.ts
…ons from index.ts — /api/events SSE machinery and the routine routes move to server/routes/ behind the workspace-backup-http factory pattern; json/readBody/stderrOf deduplicated into server/http.ts
…ckends — ComputerStatusCommon and the computerStatusProblem ladder, shared probeCuaDesktop, and computerBackendFor(bot); index.ts dispatches through the resolver
…07-line facade — records, context, migrations, messages, groups, bots, and tasks; the public surface is unchanged and both legacy load catches stay verbatim
…e — an ordered OverlayKind list plus an OVERLAY_EXCLUDES table in store.tsx; thirty call sites move to openOverlay/closeOverlay and the testing previews follow
…s into server/routes/internal.ts — 31 path branches move verbatim behind the chain factory pattern; index.ts shrinks 17,135 to 15,505 lines
…and the streaming context move to src/state/overlays.ts and stream-context.tsx behind unchanged re-exports; no consumer changes
…Url and the OPENMAUSBOT_URL/OMB_PORT resolution move to shared/server-endpoint.ts; mcp-server and control-omb drop their private copies, credential guards byte-identical
…l-capability minting moves to server/internal-capabilities.ts and the fold maps plus requestBehavior to server/turn-fold.ts; turn admission stays pending a composition root for the index-local singletons
…ove verbatim to src/components/sidebar/ behind unchanged re-exports; all Sidebar tests pass unmodified (2,367 to 998 lines)
…the 680-line branch chain moves to eleven domain modules under server/drivers/agents-proxy/ with compile-time sync between the tool table and handlers; wire output byte-identical (1,753 to 1,062 lines)
…al flow from index
bradhallett
force-pushed
the
refactor/code-health-49
branch
from
September 18, 2026 21:32
3e985ed to
e1ea70c
Compare
…ndow Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
bradhallett
force-pushed
the
refactor/code-health-49
branch
from
September 19, 2026 02:20
9a11fe8 to
d4f97c2
Compare
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
The route-scan test listed server/request-handler.ts and server/route-wiring.ts unconditionally, but those files only exist once the route-extraction refactor lands, so every branch before it fails the drift test with ENOENT. Filter the module list to files that exist so the scan covers the routes each branch actually serves. Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com> # Conflicts: # server/browser-runtime.test.ts # server/drivers/agents-proxy.ts # server/index.ts # server/routine-requests.ts # src/components/RoutineCalendarPage.tsx
# Conflicts: # server/box.ts # server/browser-runtime.test.ts # server/index.ts # src/components/ComputerPanel.tsx # src/state/store.tsx
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com> # Conflicts: # server/index.ts
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.
Part 49 of 91 in the refactor/code-health-campaign stack.
Merges after refactor/code-health-48 (part 48).
Until earlier parts of the stack merge, the Files changed view also shows their commits — review only this PRs head commit in the Commits tab.
Stack queue cleanup — 2026-09-20
Closed at the maintainer’s request in favor of the single draft consolidation PR #1582. This is not a merge and does not assert that every change is included. See the commit reconciliation checklist before merging the consolidation. The source branch and this PR’s history are preserved; this PR can be reopened if needed.
Saved head: 3b70935.