Skip to content

refactor(hub,sidecar): modularize hub composition and retire obsolete sidecar deltas (CL-10992, CL-10993) - #1138

Merged
TheGreatAxios merged 3 commits into
mainfrom
cl-10992-slim-hub-and-sidecar
Oct 21, 2026
Merged

TheGreatAxios merged 3 commits into
mainfrom
cl-10992-slim-hub-and-sidecar

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Extract Corbits mount block (CL-10992): Split Workbench's Corbits mount block (@corbits/artifacts, live workflow-run SSE stream, @corbits/agent-token, @corbits/mailbox, @corbits/cron, @corbits/memory, @corbits/oauth-core, @corbits/mcp, @corbits/webhooks) into apps/hub/src/corbits-mount.ts (mountCorbits), leaving apps/hub/src/server.ts focused on stock @intx/hub-sessions + @intx/hub-api (createApp) composition and reusing databaseUrlFrom(dbConfig) and a shared workflowRunAuthenticator.
  • Collapse over-packaged sidecar provisioner (CL-10992): Fold apps/hub/src/provisioners/sandbox-sidecar.ts into apps/hub/src/provisioners/process.ts and delete sandbox-sidecar.ts, removing unused multi-backend abstractions (SidecarBackend, SIDECAR_ISOLATION_LEVELS, per-ensure tokenHashSha256 hashing, getRecord) now that local child-process provisioning is the sole backend.
  • Slim apps/hub and apps/sidecar dependencies & docs (CL-10992, CL-10993): Prune 19 unused direct dependencies and stale TypeScript project references from apps/hub and apps/sidecar left behind after upstream @intx/workflow-host absorbed workflow-substrate-factory.ts, step-agent-tools.ts, and sidecar-deploy-router.ts. Update apps/sidecar/README.md, apps/hub/README.md, apps/hub/AGENTS.md, ARCHITECTURE.md, and remove obsolete docs/sidecar-conversation-durability.md and docs/sidecar-tool-package-durability.md.

Test Plan

  • bun run check (typecheck, oxlint, oxfmt --check, unit tests across all workspaces)

@linear-code

linear-code Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

CL-10993

CL-10992

@TheGreatAxios
TheGreatAxios force-pushed the cl-10992-slim-hub-and-sidecar branch from 723a6d2 to d9b14e6 Compare October 21, 2026 04:32
@TheGreatAxios
TheGreatAxios enabled auto-merge (squash) October 21, 2026 04:32
@corbits-triage

corbits-triage Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Triage will approve, request review.

Changed since the last verdict (Fail at d9b14e6): 8 new commits.

@corbits-triage corbits-triage Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Triage requests changes:

  • No merge conflicts: merge conflicts with base

@TheGreatAxios
TheGreatAxios force-pushed the cl-10992-slim-hub-and-sidecar branch from d9b14e6 to 0f2cfca Compare October 21, 2026 04:58
@TheGreatAxios
TheGreatAxios force-pushed the cl-10992-slim-hub-and-sidecar branch from 0f2cfca to b39681e Compare October 21, 2026 04:58
@corbits-triage
corbits-triage Bot dismissed their stale review October 21, 2026 05:05

Nothing triage asked to change still fails, so this request no longer stands.

@TheGreatAxios
TheGreatAxios merged commit ad467a2 into main Oct 21, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant