Skip to content

fix: restore @datum-cloud/datum-ui to runtime dependencies#197

Merged
scotwells merged 2 commits into
mainfrom
fix/activity-ui-datum-ui-dependency
May 4, 2026
Merged

fix: restore @datum-cloud/datum-ui to runtime dependencies#197
scotwells merged 2 commits into
mainfrom
fix/activity-ui-datum-ui-dependency

Conversation

@scotwells
Copy link
Copy Markdown
Contributor

Problem

The activity-ui pod in staging is in CrashLoopBackOff (272+ restarts) with the following error:

Error: Cannot find package '@datum-cloud/datum-ui' imported from /app/build/server/index.js

Root Cause

PR #194 promoted @datum-cloud/datum-ui to a peer dependency of the @datum-cloud/activity-ui library (ui/package.json) — which is correct for the published package. However, the example app (ui/example/package.json) was never updated to include @datum-cloud/datum-ui in its own dependencies.

The Dockerfile builds the image using pnpm deploy --filter activity-ui-example --prod, which only copies dependencies (not devDependencies or peer dependencies of workspace siblings) into the production image at /deploy/node_modules. As a result, @datum-cloud/datum-ui is absent from the container, and the Remix server bundle can't resolve it at runtime.

Fix

Add @datum-cloud/datum-ui: ^0.8.0 to dependencies in ui/example/package.json. This ensures pnpm deploy --prod includes the package in the production image.

Test Plan

  • Verify Docker image builds successfully with the fix
  • Confirm the activity-ui pod starts without the Cannot find package error in staging
  • Verify activity UI renders correctly end-to-end

scotwells and others added 2 commits May 2, 2026 11:03
The activity-ui example app (served as the activity-ui pod) is crashing
in staging with CrashLoopBackOff (272+ restarts) because the Docker image
is missing the @datum-cloud/datum-ui package at runtime:

  Error: Cannot find package '@datum-cloud/datum-ui' imported from
  /app/build/server/index.js

The Dockerfile uses `pnpm deploy --filter activity-ui-example --prod`
which only copies `dependencies` into the production image. When PR #194
promoted @datum-cloud/datum-ui to a peer dependency of the activity-ui
library, it was never added to the example app's own `dependencies`, so
pnpm deploy excluded it from the container.

Fix: add @datum-cloud/datum-ui ^0.8.0 to dependencies in
ui/example/package.json so it is included in the production image build.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@scotwells scotwells requested review from a team and kevwilliams May 4, 2026 17:33
@scotwells scotwells enabled auto-merge May 4, 2026 17:37
@scotwells scotwells merged commit b7f92ab into main May 4, 2026
7 checks passed
@scotwells scotwells deleted the fix/activity-ui-datum-ui-dependency branch May 4, 2026 18:03
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