@ibelick updated the PR

• Dockerfile migrated to pnpm
• apps/webclaw/package.json added a missing dep, clsx for css

thanks @deblanco!

Two nits:

• Build/run should target the workspace: pnpm -C apps/webclaw build and pnpm -C apps/webclaw preview --host --port 3000, otherwise it relies on root scripts that aren’t shown.
• Runtime image currently copies apps/webclaw/src and full node_modules, if not required for vite preview, consider dropping src and/or installing only prod deps to reduce size and exposure

thanks @deblanco!

Two nits:

• Build/run should target the workspace: pnpm -C apps/webclaw build and pnpm -C apps/webclaw preview --host --port 3000, otherwise it relies on root scripts that aren’t shown.
• Runtime image currently copies apps/webclaw/src and full node_modules, if not required for vite preview, consider dropping src and/or installing only prod deps to reduce size and exposure

@ibelick pushed small "optimizations" in the Dockerfile

This is funny because running vite preview requires the node_modules folder, since the vite binary is in node_modules, and it also complains if tsconfig.json is not there.

@deblanco thanks! could we switch the runtime to nginx/Caddy serving only dist/ so we can drop node_modules and dev tooling from production?

@deblanco thanks! could we switch the runtime to nginx/Caddy serving only dist/ so we can drop node_modules and dev tooling from production?

The project cannot serve /dist because this app has been done with @tanstack/react-start which by default uses SSR and that requires running the project with node, and with it requires the node_modules

Producing a SPA (a full static site in /dist) requires changing vite.config.js, and I don't think the project should be open to that.

@deblanco ok I see, and can we replace pnpm preview with pnpm start (real production command)? thanks

https://github.com/ibelick/webclaw/pull/17/changes

this looks better