Skip to content

fix: add WORKDIR /build in builder stage to fix uv 0.11 cache error#389

Merged
MaStr merged 1 commit into
mainfrom
claude/action-run-121-docker-5ovf4h
Jun 27, 2026
Merged

fix: add WORKDIR /build in builder stage to fix uv 0.11 cache error#389
MaStr merged 1 commit into
mainfrom
claude/action-run-121-docker-5ovf4h

Conversation

@MaStr

@MaStr MaStr commented Jun 27, 2026

Copy link
Copy Markdown
Owner

uv 0.11 refuses to build if its cache dir (/root/.cache/uv) is inside
the build source directory. Without an explicit WORKDIR the builder stage
defaulted to /, making / the source root and triggering the new safety
check.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_018Zhp8eDqpSFxMru61fBKXK

uv 0.11 refuses to build if its cache dir (/root/.cache/uv) is inside
the build source directory. Without an explicit WORKDIR the builder stage
defaulted to /, making / the source root and triggering the new safety
check.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018Zhp8eDqpSFxMru61fBKXK
Copilot AI review requested due to automatic review settings June 27, 2026 15:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Docker build to explicitly set the builder stage working directory to /build, avoiding uv 0.11’s safety check failure when its cache directory is detected inside the build source root (which previously defaulted to /).

Changes:

  • Add WORKDIR /build to the builder stage before copying project files.
  • Ensure subsequent COPY and uv build steps execute within a dedicated build directory rather than /.

@MaStr MaStr merged commit 6ee7155 into main Jun 27, 2026
14 checks passed
@MaStr MaStr deleted the claude/action-run-121-docker-5ovf4h branch June 27, 2026 15:31
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.

3 participants