Skip to content

fix: build design-agent package + copy its manifest in Docker#233

Closed
FluidDynamicsAI wants to merge 1 commit into
Anil-matcha:mainfrom
FluidDynamicsAI:fix/docker-design-agent-build
Closed

fix: build design-agent package + copy its manifest in Docker#233
FluidDynamicsAI wants to merge 1 commit into
Anil-matcha:mainfrom
FluidDynamicsAI:fix/docker-design-agent-build

Conversation

@FluidDynamicsAI

Copy link
Copy Markdown

Problem

design-agent (main = dist/index.js) is imported by packages/studio (DesignAgentStudio.jsximport { CreativeCanvas } from 'design-agent'), but:

  1. build:packages never builds it (build:workflow && build:agent && build:studio only).
  2. The Dockerfile deps stage never copies its package.json.

Without dist/ output, module resolution of design-agent fails — breaking both npm run build (Next.js) and the Docker image build.

Fix

  • package.json: add build:design, wire into build:packages.
  • Dockerfile: copy design-agent package*.json before npm install.

Verification

docker compose up -d --build → exit 0. Container serves /studio → HTTP 200 on :3001.

🤖 Generated with Claude Code

design-agent (main=dist/index.js) is imported by packages/studio but was
never built by build:packages and its package.json was not copied in the
Dockerfile deps stage. Both the Next.js build and the Docker build fail to
resolve the 'design-agent' module without dist output.

- package.json: add build:design, wire into build:packages
- Dockerfile: copy design-agent package*.json before npm install

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PWLUy7F91kmo98T8D8tLmD
@Anil-matcha

Copy link
Copy Markdown
Owner

Superseded — I implemented this exact fix directly in commit f02bb30 (add build:design script wired into build:packages, plus copy the design-agent submodule's package*.json in the Dockerfile deps stage), matching this PR's approach. Closing as no longer needed on top of main; thanks for the diagnosis and fix though, it was correct.

@Anil-matcha Anil-matcha closed this Jul 7, 2026
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.

3 participants