Skip to content

fix(shell): allow default local API URL bootstrap#2

Open
skulidropek wants to merge 5 commits into
rikohomeless:codex/controller-host-runtime-skillerfrom
ProverCoderAI:codex/default-local-api-url-bootstrap-pr351
Open

fix(shell): allow default local API URL bootstrap#2
skulidropek wants to merge 5 commits into
rikohomeless:codex/controller-host-runtime-skillerfrom
ProverCoderAI:codex/default-local-api-url-bootstrap-pr351

Conversation

@skulidropek

Copy link
Copy Markdown

Summary

  • Treat default-local DOCKER_GIT_API_URL values as bootstrap hints instead of strict remote controller overrides.
  • Preserve strict failure behavior for custom explicit API URLs.
  • Cover the Windows-like browser startup path in unit tests and browser e2e.

Context

This is a one-commit follow-up for ProverCoderAI#351. It fixes the case where a persisted Windows env var like DOCKER_GIT_API_URL=http://127.0.0.1:3334 prevents local Docker bootstrap.

Verification

  • bun run --cwd packages/app typecheck
  • bun run --cwd packages/app lint
  • bun run --cwd packages/app lint:tests
  • bun run --cwd packages/app lint:effect
  • bun run --cwd packages/app vitest run tests/docker-git/controller.test.ts tests/docker-git/controller-ready.test.ts tests/docker-git/browser-frontend.test.ts
  • DOCKER_GIT_E2E_REUSE_WORKSPACE_INSTALL=1 DOCKER_GIT_E2E_BROWSER_STARTUP_ATTEMPTS=360 bash scripts/e2e/browser-command.sh

rikohomeless and others added 4 commits May 29, 2026 09:33
* fix(shell): prepare host-backed controller builds

- Default controller runtime to host Docker with socket mount

- Initialize the pinned Skiller submodule before controller revision/build

- Include Skiller inputs in controller revision hashing

* test(shell): cover controller compose preparation

* fix(shell): support containerized remote Docker controller startup

* fix(shell): preserve remote and isolated controller bootstrap

* fix(shell): make isolated controller fallback self-contained

* fix(shell): address controller runtime review comments

---------

Co-authored-by: skulidropek <66840575+skulidropek@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
# Conflicts:
#	packages/app/src/docker-git/controller.ts
#	packages/app/tests/docker-git/controller-ready.test.ts
#	packages/app/tests/docker-git/controller.test.ts
@skulidropek skulidropek force-pushed the codex/default-local-api-url-bootstrap-pr351 branch from 5386e55 to db0594b Compare May 29, 2026 09:03
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.

2 participants