fix: use committed .js wrapper + separate _handler.js bundle to avoid Vercel TS compilation and file-not-found errors

Agent-Logs-Url: https://github.com/objectstack-ai/framework/sessions/656a5b4f-383c-4318-a657-a952f4445583

Co-authored-by: xuyushun441-sys <255036401+xuyushun441-sys@users.noreply.github.com>

Author: Copilot

CHANGELOG.md

@@ -10,13 +10,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 - **Studio Vercel API routes returning HTML instead of JSON** — Adopted the
   same Vercel deployment pattern used by `hotcrm`: committed
-  `api/[[...route]].ts` catch-all route so Vercel detects it pre-build,
+  `api/[[...route]].js` catch-all route so Vercel detects it pre-build,
   switched esbuild output from CJS to ESM (fixes `"type": "module"` conflict),
-  and changed the output path from `api/index.js` to `api/[[...route]].js`.
+  and changed the bundle output to `api/_handler.js` (a separate file that
+  the committed wrapper re-exports).  This avoids both Vercel's TS
+  compilation overwriting the bundle (`ERR_MODULE_NOT_FOUND`) and the
+  "File not found" error from deleting source files during build.
   Updated rewrites to match: `/api/:path*` → `/api/[[...route]]`.
-  Added post-bundle cleanup to remove the `.ts` stub so Vercel's
-  `@vercel/node` builder uses the self-contained esbuild bundle instead of
-  compiling the stub (which would fail with `ERR_MODULE_NOT_FOUND`).
 - **Studio CORS error on Vercel temporary/preview domains** — Changed
   `VITE_SERVER_URL` from hardcoded `https://play.objectstack.ai` to `""`
   (empty string / same-origin) in `vercel.json` so each deployment — including

apps/studio/.gitignore

@@ -5,8 +5,8 @@ pnpm-lock.yaml
 # Build outputs
 dist
 build
-api/*.js
-api/*.js.map
+api/_handler.js
+api/_handler.js.map
 *.tsbuildinfo
 
 # Development

apps/studio/api/[[...route]].js

@@ -0,0 +1,17 @@
+// Vercel Serverless Function — Catch-all API route.
+//
+// This file MUST be committed to the repository so Vercel can detect it
+// as a serverless function during the pre-build phase.
+//
+// It delegates to the esbuild bundle (`_handler.js`) generated by
+// `scripts/bundle-api.mjs` during the Vercel build step.  A separate
+// bundle file is used (rather than overwriting this file) so that:
+//   1. Vercel always finds this committed entry point (no "File not found").
+//   2. Vercel does not TypeScript-compile a .ts stub that references
+//      source files absent at runtime (no ERR_MODULE_NOT_FOUND).
+//
+// @see ../server/index.ts    — the actual server entrypoint
+// @see ../scripts/bundle-api.mjs — the esbuild bundler
+// @see https://github.com/objectstack-ai/hotcrm/blob/main/vercel.json
+
+export { default, config } from './_handler.js';

apps/studio/api/[[...route]].ts

@@ -4,8 +4,9 @@ set -euo pipefail
 # Build script for Vercel deployment of @objectstack/studio.
 #
 # Follows the same Vercel deployment pattern as hotcrm:
-#   - api/[[...route]].ts is committed to the repo (Vercel detects it pre-build)
-#   - esbuild bundles server/index.ts → api/[[...route]].js (replaces TS at deploy)
+#   - api/[[...route]].js is committed to the repo (Vercel detects it pre-build)
+#   - esbuild bundles server/index.ts → api/_handler.js (self-contained bundle)
+#   - The committed .js wrapper re-exports from _handler.js at runtime
 #   - Vite SPA output is copied to public/ for CDN serving
 #
 # Vercel routing (framework: null, no outputDirectory):
@@ -14,7 +15,7 @@ set -euo pipefail
 #
 # Steps:
 #   1. Turbo build (Vite SPA → dist/)
-#   2. Bundle the API serverless function (→ api/[[...route]].js)
+#   2. Bundle the API serverless function (→ api/_handler.js)
 #   3. Copy Vite output to public/ for Vercel CDN serving
 
 echo "[build-vercel] Starting studio build..."
@@ -26,15 +27,10 @@ cd apps/studio
 
 # 2. Bundle API serverless function
 node scripts/bundle-api.mjs
-# Remove the TS source file so Vercel's @vercel/node builder uses the
-# pre-bundled JS directly.  Without this, Vercel compiles the TS stub
-# itself, producing a thin re-export that references ../server/index —
-# a file that doesn't exist at runtime (ERR_MODULE_NOT_FOUND).
-rm -f "api/[[...route]].ts"
 
 # 3. Copy Vite build output to public/ for static file serving
 rm -rf public
 mkdir -p public
 cp -r dist/* public/
 
-echo "[build-vercel] Done. Static files in public/, serverless function in api/[[...route]].js"
+echo "[build-vercel] Done. Static files in public/, serverless function in api/[[...route]].js → api/_handler.js"

apps/studio/scripts/build-vercel.sh

@@ -37,7 +37,7 @@ await build({
   platform: 'node',
   format: 'esm',
   target: 'es2020',
-  outfile: 'api/[[...route]].js',
+  outfile: 'api/_handler.js',
   sourcemap: true,
   external: EXTERNAL,
   // Silence warnings about optional/unused require() calls in knex drivers
@@ -48,4 +48,4 @@ await build({
   banner: { js: '// Bundled by esbuild — see scripts/bundle-api.mjs' },
 });
 
-console.log('[bundle-api] Bundled server/index.ts → api/[[...route]].js');
+console.log('[bundle-api] Bundled server/index.ts → api/_handler.js');