fix(ppt): dep injection (#3732)

Author: icecrasher321

apps/sim/lib/execution/pptx-vm.ts

@@ -33,7 +33,11 @@ let cachedWorkerPath: string | undefined
 function getWorkerPath(): string {
   if (cachedWorkerPath) return cachedWorkerPath
   const candidates = [
+    path.join(currentDir, '..', '..', 'dist', 'pptx-worker.cjs'),
     path.join(currentDir, 'pptx-worker.cjs'),
+    path.join(process.cwd(), 'apps', 'sim', 'dist', 'pptx-worker.cjs'),
+    path.join(process.cwd(), 'apps', 'sim', 'lib', 'execution', 'pptx-worker.cjs'),
+    path.join(process.cwd(), 'dist', 'pptx-worker.cjs'),
     path.join(process.cwd(), 'lib', 'execution', 'pptx-worker.cjs'),
   ]
   const found = candidates.find((p) => fs.existsSync(p))

apps/sim/next.config.ts

@@ -90,14 +90,7 @@ const nextConfig: NextConfig = {
   ],
   outputFileTracingIncludes: {
     '/api/tools/stagehand/*': ['./node_modules/ws/**/*'],
-    '/*': [
-      './node_modules/sharp/**/*',
-      './node_modules/@img/**/*',
-      // pptxgenjs and the PPTX worker are required at runtime by the subprocess.
-      // Neither has a static import that Next.js can trace, so we include them explicitly.
-      './node_modules/pptxgenjs/**/*',
-      './lib/execution/pptx-worker.cjs',
-    ],
+    '/*': ['./node_modules/sharp/**/*', './node_modules/@img/**/*', './dist/pptx-worker.cjs'],
   },
   experimental: {
     optimizeCss: true,

apps/sim/package.json

@@ -12,7 +12,8 @@
     "dev:webpack": "next dev --webpack",
     "dev:sockets": "bun run socket/index.ts",
     "dev:full": "bunx concurrently -n \"App,Realtime\" -c \"cyan,magenta\" \"bun run dev\" \"bun run dev:sockets\"",
-    "build": "next build",
+    "build": "bun run build:pptx-worker && next build",
+    "build:pptx-worker": "bun build ./lib/execution/pptx-worker.cjs --target=node --format=cjs --outfile ./dist/pptx-worker.cjs",
     "start": "next start",
     "prepare": "cd ../.. && bun husky",
     "test": "vitest run",

docker/app.Dockerfile

@@ -108,14 +108,11 @@ COPY --from=builder --chown=nextjs:nodejs /app/apps/sim/.next/static ./apps/sim/
 # Copy isolated-vm native module (compiled for Node.js in deps stage)
 COPY --from=deps --chown=nextjs:nodejs /app/node_modules/isolated-vm ./node_modules/isolated-vm
 
-# Copy PPTX runtime dependency
-COPY --from=deps --chown=nextjs:nodejs /app/node_modules/pptxgenjs ./node_modules/pptxgenjs
-
 # Copy the isolated-vm worker script
 COPY --from=builder --chown=nextjs:nodejs /app/apps/sim/lib/execution/isolated-vm-worker.cjs ./apps/sim/lib/execution/isolated-vm-worker.cjs
 
-# Copy the PPTX worker script
-COPY --from=builder --chown=nextjs:nodejs /app/apps/sim/lib/execution/pptx-worker.cjs ./apps/sim/lib/execution/pptx-worker.cjs
+# Copy the bundled PPTX worker artifact
+COPY --from=builder --chown=nextjs:nodejs /app/apps/sim/dist/pptx-worker.cjs ./apps/sim/dist/pptx-worker.cjs
 
 # Guardrails setup with pip caching
 COPY --from=builder --chown=nextjs:nodejs /app/apps/sim/lib/guardrails/requirements.txt ./apps/sim/lib/guardrails/requirements.txt