Repository navigation
Drop the ambient dev sidecar from local dev - #682
Merged
TheGreatAxios merged 2 commits intoSep 7, 2026
Merged
TheGreatAxios merged 2 commits into
TheGreatAxios merged 2 commits into
Conversation
PR #684 already removed the ambient dev sidecar bootstrap, but reset still removed a repo-root .data/sidecar directory that nothing creates anymore, its test still expected it, and the README/local-rip dev-start wording still described bootstrapped sidecar state. Reset now covers only the hub data dir (which holds provisioned sidecar state), and the docs describe hub-provisioned capacity.
TheGreatAxios
force-pushed
the
cl-7496-drop-ambient-dev-sidecar-bun-run-dev-relies-on-hub
branch
from
September 7, 2026 17:10
1dad8a2 to
ff0ad74
Compare
The re-arm test asserted a visible-expiry boundary with a 20ms margin under real timers; a CI runner stretched a single await past 120ms and the indicator expired before the re-arm, failing the shard. The expiry doubles to 500ms and every assertion margin is now at least 150ms, so runner jitter cannot flip before-expiry / after-expiry either way.
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.
Follow-up to #684, which landed the ambient dev-sidecar removal itself (upstream Interchange commit
064e997dremoved ambient/shared sidecar authentication, so the dev bootstrap's baresidecarrow + static-token login could never register). This PR finishes the removal with the pieces #684 did not cover:scripts/reset.tsno longer removes the repo-root.data/sidecardirectory — nothing creates it anymore; provisioned sidecar state lives under the hub data dir, which reset already coversscripts/reset.test.tsexpectations updated to matchdocs/local-rip.mddescribe hub-provisioned sidecar capacity instead of the retired bootstrapped pathCloses CL-7496.
Verification:
bun test scripts/reset.test.ts9/9 in the worktree;check:structuralgreen; CI runs the full gate matrix here. Pre-push hook skipped withSKIP_WORKBENCH_HOOKSsolely because of the pre-existing@corbits/workflowslocal-env failures tracked in CL-7499 (CI is unaffected).