Skip to content

refactor(install-claude-sidecar): adopt host-wrapper + .sidecar/shado…#13

Open
mithredate wants to merge 1 commit into
mainfrom
feat/install-claude-sidecar-wrapper-flow
Open

refactor(install-claude-sidecar): adopt host-wrapper + .sidecar/shado…#13
mithredate wants to merge 1 commit into
mainfrom
feat/install-claude-sidecar-wrapper-flow

Conversation

@mithredate
Copy link
Copy Markdown
Owner

No description provided.

…w flow

The previous workflow performed compose-file surgery — adding claude and
socket-proxy services to the user's compose.yml, with inline /dev/null
shadow mounts per discovered credential file. That coupled the skill to
the user's compose layout and made cross-project sessions impossible
(every project an island).

New workflow targets the host-wrapper architecture in claude-sidecar:

- The user's compose.yml is NEVER touched. All claude-sidecar wiring goes
  into a generated compose.sidecar.yml emitted by `bridge gen-overlay`,
  merged via `docker compose -f` at `claude-sidecar up`.
- Sensitive paths live in a committed `.sidecar/shadow` (canonical per
  repo). Cross-project mounts honor the mounted repo's own shadow file.
- First-time setup installs ~/.local/bin/claude-sidecar and creates
  ~/.claude-sidecar/config.yaml; detected on subsequent runs and skipped.
- Sibling-repo extra_mounts are asked once and stored in user config; the
  skill ensures every extra-mounted repo has its own .sidecar/shadow
  before allowing the mount.
- .gitignore gains compose.sidecar.yml + compose.sidecar-local.yml +
  .credentials.json (the shadow file itself is committed).

references/credential-shadowing.md updated to describe pattern-based
discovery (the part still useful) and document the .sidecar/shadow format
+ the three-layer defense (mount, entrypoint check, drift warning).

Co-Authored-By: Claude Opus 4.7 (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