Skip to content

fix(docker): copy testing/e2e/package.json for frozen lockfile install - #222

Merged
07prajwal2000 merged 1 commit into
Fluxify-rest:mainfrom
07prajwal2000:fix/docker-frozen-lockfile-e2e-workspace
Aug 7, 2026
Merged

fix(docker): copy testing/e2e/package.json for frozen lockfile install#222
07prajwal2000 merged 1 commit into
Fluxify-rest:mainfrom
07prajwal2000:fix/docker-frozen-lockfile-e2e-workspace

Conversation

@07prajwal2000

@07prajwal2000 07prajwal2000 commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • testing/e2e is declared as a workspace in root package.json, but no Dockerfile's deps stage copied testing/e2e/package.json, so bun install --frozen-lockfile failed with Workspace not found "testing/e2e" in every image build (kit, admin, worker-compiled).
  • Adds COPY testing/e2e/package.json testing/e2e/package.json to all three Dockerfiles' deps stages, alongside the other workspace manifests.
  • Second, independent bug found while verifying the fix with a real docker build: bun run build (invoked by every Dockerfile after bun install) also failed, in apps/web, on a genuine next build type-check error unrelated to the lockfile. whereConditionSchema's attribute/value had become a tagged { kind, value } union (column vs literal), but the db block settings panels (delete.tsx, getAll.tsx, getSingle.tsx, update.tsx) still passed those tagged objects straight through to ConditionsEditor, which expects flat string | number | boolean. Fixed by unwrapping .value on read and re-tagging (column for lhs, literal for rhs) on write — the same approach apps/portal's serializeDbConditions/parseDbConditions already use for the current db block settings panels.

Test plan

  • Reproduced the lockfile failure by simulating the deps stage COPY list in an isolated dir and running bun install --frozen-lockfile — failed with Workspace not found "testing/e2e"; fix resolved it.
  • bun run --cwd apps/web build and full bun run build (all 4 workspace build tasks) pass after the apps/web fix.
  • Built docker/worker-compiled/Dockerfile end-to-end locally (docker build) — succeeded.
  • Built docker/kit/Dockerfile end-to-end locally (docker build, all 3 stages: deps → build → production) — succeeded. Test images removed afterward (docker rmi).

🤖 Generated with Claude Code

… works

root package.json declares testing/e2e as a workspace, but no Dockerfile's
deps stage copied its package.json, so `bun install --frozen-lockfile`
failed with "Workspace not found \"testing/e2e\"" in every image build.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@07prajwal2000
07prajwal2000 added this pull request to the merge queue Aug 7, 2026
Merged via the queue into Fluxify-rest:main with commit 4449eed Aug 7, 2026
12 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