Skip to content

fix(docker): preserve vendor/*.tgz in manifests stage - #73

Merged
hblanken merged 1 commit into
collabfrom
upstream-sync
Aug 18, 2026
Merged

fix(docker): preserve vendor/*.tgz in manifests stage#73
hblanken merged 1 commit into
collabfrom
upstream-sync

Conversation

@hblanken

Copy link
Copy Markdown
Collaborator

Summary

  • The upstream merge introduced @opencode-ai/client as a vendored local tarball (packages/app/vendor/opencode-ai-client-1.17.13-v2.tgz)
  • The Dockerfile manifests stage strips all non-manifest files before bun install, which caused ENOENT when bun tried to resolve the file: reference
  • Fix: add ! -path '*/vendor/*.tgz' to the find whitelist in the manifests stage

Test plan

  • Docker build completes without ENOENT on @opencode-ai/client
  • Deploy workflow succeeds end-to-end

🤖 Generated with Claude Code

The upstream merge introduced @opencode-ai/client as a vendored local
tarball (packages/app/vendor/). The manifests stage strips all non-manifest
files before bun install, which caused ENOENT when bun tried to resolve the
file: reference. Add *.tgz under */vendor/* to the find whitelist.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d75ae1e1-5858-4ef2-9ac3-99852ced765d


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hblanken
hblanken merged commit c7d408a into collab Aug 18, 2026
4 of 8 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.

2 participants