Skip to content

fix(core): restore Sidecar public export + sidecar-client subpath#112

Open
abcxff wants to merge 1 commit into
mainfrom
fix/sidecar-public-export
Open

fix(core): restore Sidecar public export + sidecar-client subpath#112
abcxff wants to merge 1 commit into
mainfrom
fix/sidecar-public-export

Conversation

@abcxff

@abcxff abcxff commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

What

@secure-exec/core was renamed internally (sidecar-client.tssidecar-process.ts, class SidecarSidecarProcess) but the public surface wasn't updated, so on main:

  • @secure-exec/core no longer exports Sidecar (only SidecarProcess)
  • the ./sidecar-client export still points at the now-nonexistent dist/sidecar-client.js (the source builds to dist/sidecar-process.js)

Consumers that import Sidecar / @secure-exec/core/sidecar-client (the secure-exec facade, the native-client example, and @rivet-dev/agentos-core) fail to build with TS2305 / TS2307.

Fix (3 small edits, no consumer changes)

  • sidecar-process.ts: export { SidecarProcess as Sidecar }
  • index.ts: also export Sidecar from the package root
  • package.json: repoint the ./sidecar-client export to ./dist/sidecar-process.{js,d.ts}

sidecar-process.ts already re-exports the full prior sidecar-client surface (SidecarProcessError, SidecarProcessExited, SidecarEventBufferOverflow, the types), so the public API is preserved exactly.

Verification

With this change, @secure-exec/core builds, the secure-exec facade builds, and the native-client example typechecks clean.

🤖 Generated with Claude Code

@railway-app

railway-app Bot commented Jun 22, 2026

Copy link
Copy Markdown

🚅 Deployed to the secure-exec-pr-112 environment in rivet-frontend

Service Status Web Updated (UTC)
secure-exec 😴 Sleeping (View Logs) Jun 22, 2026 at 9:21 pm

🚅 Deployed to the secure-exec-pr-112 environment in secure-exec

Service Status Web Updated (UTC)
secure-exec 😴 Sleeping (View Logs) Web Jun 24, 2026 at 7:22 pm

NathanFlurry added a commit that referenced this pull request Jun 23, 2026
@secure-exec/core dropped the public `Sidecar` export (the public name for
`SidecarProcess`) and the ./sidecar-client subpath still pointed at a
non-existent sidecar-client.* file. The `secure-exec` package imports
`Sidecar` from @secure-exec/core, so its build failed with TS2305 during
publish. Re-export `SidecarProcess as Sidecar` and repoint the subpath to
sidecar-process.*. (Mirrors PR #112.)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant