Skip to content

fix(ambient-ui): fix Docker build for standalone output#1619

Merged
jsell-rh merged 1 commit into
mainfrom
jsell/feat/ambient-ui-live-preview-v2
May 30, 2026
Merged

fix(ambient-ui): fix Docker build for standalone output#1619
jsell-rh merged 1 commit into
mainfrom
jsell/feat/ambient-ui-live-preview-v2

Conversation

@jsell-rh
Copy link
Copy Markdown
Contributor

@jsell-rh jsell-rh commented May 30, 2026

Summary

  • Build SDK dist/ in Docker (ts-sdk dist/ is gitignored, CI has no pre-built output)
  • Copy standalone output from correct nested path (outputFileTracingRoot nests files under components/ambient-ui/)
  • Use webpack for production build (Turbopack can't resolve file: linked dependencies)

Fixes the Cannot find module '/app/server.js' crash at container startup and the Module not found: Can't resolve 'ambient-sdk' build failure.

Test plan

  • npm run build passes locally with webpack
  • server.js confirmed at .next/standalone/components/ambient-ui/server.js
  • 167 tests pass
  • CI Docker build passes and pushes to quay.io
  • Container starts without MODULE_NOT_FOUND error

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Optimized application packaging in the build process.

- Build SDK dist in Docker (ts-sdk dist/ is gitignored)
- Copy standalone output from correct nested path
  (outputFileTracingRoot nests under components/ambient-ui/)
- Use webpack for production build (Turbopack can't resolve file: deps)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 30, 2026

Deploy Preview for cheerful-kitten-f556a0 canceled.

Name Link
🔨 Latest commit 872f6d2
🔍 Latest deploy log https://app.netlify.com/projects/cheerful-kitten-f556a0/deploys/6a1b0916958b8800088a60cd

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 30, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: c8efb687-5d8b-46bd-a41f-410a30aaef8e

📥 Commits

Reviewing files that changed from the base of the PR and between 5402772 and 872f6d2.

📒 Files selected for processing (1)
  • components/ambient-ui/Dockerfile

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting


📝 Walkthrough

Walkthrough

Builder stage in components/ambient-ui/Dockerfile refactored to copy only the components/ambient-ui portion of .next/standalone into /app-output rather than the entire standalone directory, with optional and non-fatal node_modules placement.

Changes

Builder Output Packaging

Layer / File(s) Summary
Builder output copy strategy
components/ambient-ui/Dockerfile
Destination directories created (/app-output/public, /app-output/.next/static), and copy logic changed to include only ./.next/standalone/components/ambient-ui/ into /app-output/, with separate optional placement of ./.next/standalone/node_modules tolerating failures (2>/dev/null || true).
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jsell/feat/ambient-ui-live-preview-v2
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch jsell/feat/ambient-ui-live-preview-v2

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 and usage tips.

@jsell-rh jsell-rh merged commit b5e6290 into main May 30, 2026
37 of 38 checks passed
@jsell-rh jsell-rh deleted the jsell/feat/ambient-ui-live-preview-v2 branch May 30, 2026 15:58
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