refactor(server): extract team and bot action routes from index - #1402
Closed
bradhallett wants to merge 94 commits into
Closed
bradhallett wants to merge 94 commits into
bradhallett wants to merge 94 commits into
Conversation
|
@bradhallett is attempting to deploy a commit to the SupaMaus Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedToo many files! This PR contains 447 files, which is 147 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 (447)
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 |
This was referenced Sep 17, 2026
Open
Closed
bradhallett
force-pushed
the
refactor/code-health-77
branch
14 times, most recently
from
September 18, 2026 20:23
b3d858a to
0855de7
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
…systems from main
…subsystems from main
bradhallett
force-pushed
the
refactor/code-health-77
branch
from
September 18, 2026 21:33
0855de7 to
f2a36d9
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-77
branch
from
September 19, 2026 02:22
44f87af to
4e2bb0c
Compare
Signed-off-by: Brad Hallett <53977268+bradhallett@users.noreply.github.com>
The section guard slices electron/main/company-backup.mjs on an end anchor that contains a literal newline, so a Windows checkout under core.autocrlf fails the anchor search before the guard runs. Normalise the source to LF after reading so the guard is checkout-agnostic. 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>
The main bundle imports DEV_URL from main/environments.mjs; without the export the packaged app fails to link at startup. 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> # 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
2 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 77 of 91 in the refactor/code-health-campaign stack.
Merges after refactor/code-health-76 (part 76).
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: 5ec549d.