Skip to content

fix(docker-git): prefer live runtime and block duplicate identities#212

Merged
skulidropek merged 1 commit intomainfrom
codex/c7f9577-runtime-identities
Apr 7, 2026
Merged

fix(docker-git): prefer live runtime and block duplicate identities#212
skulidropek merged 1 commit intomainfrom
codex/c7f9577-runtime-identities

Conversation

@skulidropek
Copy link
Copy Markdown
Member

Summary

  • return Docker stderr in command failures instead of a bare DockerCommandError
  • block duplicate docker-git container/service/volume identities unless --force replaces the existing project
  • make open prefer the live runtime owner and attach to the running container before falling back to up

Verification

  • pnpm -C packages/app exec vitest run tests/docker-git/create-project-identity-conflict.test.ts tests/docker-git/open-project.test.ts tests/docker-git/docker-runtime-info.test.ts
  • pnpm -C packages/lib exec vitest run tests/usecases/create-project-docker-identities.test.ts tests/usecases/errors.test.ts
  • pnpm -C packages/api exec vitest run tests/projects.test.ts
  • pnpm -C packages/app typecheck
  • pnpm -C packages/lib typecheck
  • pnpm -C packages/api typecheck
  • pnpm --filter ./packages/app build:docker-git

Return Docker stderr in command failures, prevent duplicate docker-git identities unless --force, and make open resolve the live runtime owner before falling back to up.

Verified with targeted vitest suites, typecheck for app/lib/api, and build:docker-git.
@skulidropek skulidropek merged commit 27f238d into main Apr 7, 2026
10 of 13 checks passed
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