Skip to content

fix(docker): copy and build @openwork/email package for den-api image#1825

Open
yashranaway wants to merge 1 commit into
different-ai:devfrom
yashranaway:fix/dockerfile-den-email-package
Open

fix(docker): copy and build @openwork/email package for den-api image#1825
yashranaway wants to merge 1 commit into
different-ai:devfrom
yashranaway:fix/dockerfile-den-email-package

Conversation

@yashranaway
Copy link
Copy Markdown
Contributor

Closes #1804

Summary

  • Copy packages/email/package.json into the den image before pnpm install --frozen-lockfile so the workspace package resolves.
  • Copy the packages/email source after install and add a build step before den-db and den-api so the compiled dist/ exists when den-api imports @openwork/email at runtime.
  • Keep the existing build order otherwise unchanged; @openwork/email has no workspace deps so it can build right after utils.

Verification

  • docker build -f packaging/docker/Dockerfile.den . not run from this branch (no Docker on the dev box); the diff matches the exact patch the reporter posted in the issue.
  • grep '@openwork/email' ee/apps/den-api/package.json confirms den-api lists the workspace dep, and packages/email/package.json exports ./dist/index.js as the production entry, which makes the missing build step the failure path described in Dockerfile.den missing @openwork/email package — build fails #1804.

Notes

  • packages/email has no workspace dependencies, so it can build independently of den-db.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-landing Ready Ready Preview, Comment, Open in v0 May 22, 2026 4:06pm

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 17, 2026

@yashranaway is attempting to deploy a commit to the Different AI Team on Vercel.

A member of the Team first needs to authorize it.

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.

Dockerfile.den missing @openwork/email package — build fails

1 participant