Skip to content

fix(forge): specify-cli cache + ~/.local/bin PATH - #40

Merged
uphiago merged 1 commit into
mainfrom
fix/specify-path-cache
Aug 15, 2026
Merged

fix(forge): specify-cli cache + ~/.local/bin PATH#40
uphiago merged 1 commit into
mainfrom
fix/specify-path-cache

Conversation

@uphiago

@uphiago uphiago commented Aug 15, 2026

Copy link
Copy Markdown
Owner

Problema

specify-cli instalava mas não aparecia no PATH: uv instalou em ~/.local/bin (fora do PATH do Codex), e o UV_CACHE_DIR=/tmp/uv-cache do PR anterior era root-owned → permission denied pros jobs forge.

Correções

  • containers/forge/forge-entrypoint.sh: volta ao cache default do uv (agora /home/forge é volume writable)
  • config/codex/config.toml: PATH com /home/forge/.local/bin primeiro (jobs acham specify)
  • tests/infra-regression.sh: asserção do PATH

Verificado no worker

  • uv 0.12.5 ✅ · specify 0.12.11 ✅ (após PATH manual) · vercel 59.0.0 ✅ · supabase 2.114.0
  • infra-regression.sh all passed

Deploy

Merge → workflow_dispatch.

…/.local/bin to Codex PATH

- forge-entrypoint: drop UV_CACHE_DIR=/tmp (root-owned, forge couldn't write) — /home/forge volume is writable now
- config/codex/config.toml: prepend /home/forge/.local/bin to PATH so specify is found in jobs
- infra-regression: assert PATH entry instead of removed UV_TOOL_BIN_DIR
@uphiago
uphiago merged commit df7ee29 into main Aug 15, 2026
4 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