Skip to content

fix(hono): point start script at .output/server/index.mjs#46

Draft
VaguelySerious wants to merge 1 commit into
mainfrom
peter/fix-hono-start-script
Draft

fix(hono): point start script at .output/server/index.mjs#46
VaguelySerious wants to merge 1 commit into
mainfrom
peter/fix-hono-start-script

Conversation

@VaguelySerious

Copy link
Copy Markdown
Member

Summary

Nitro writes its production server bundle to `.output/server/index.mjs`, but the hono example's `start` script ran `node dist/index.js`:

```

hono@ start /Users/.../hono
node dist/index.js

Error: Cannot find module '/Users/.../hono/dist/index.js'
code: 'MODULE_NOT_FOUND'
```

Pre-existing — `pnpm start` has never worked from this script. Surfaced when smoke-testing the v2-flow tarballs.

Fix

Point the start script at the path nitro actually emits.

Test plan

  • `pnpm build` produces `.output/server/index.mjs`
  • `pnpm start` listens on port 3000
  • `POST /api/signup` runs the workflow end-to-end (`Started run wrun_…`, `Creating user with email: …`, `Sending welcome email to user: …`)
  • CI green

🤖 Generated with Claude Code

Nitro emits the production server to `.output/server/index.mjs`, but
`pnpm start` was running `node dist/index.js`, which doesn't exist:

  Error: Cannot find module '/Users/.../hono/dist/index.js'

Use the path nitro actually writes. Verified locally with the latest
v2-flow tarballs — `pnpm build && pnpm start` now serves on port 3000
and `POST /api/signup` runs the workflow end-to-end (`Started run …`,
`Creating user …`, `Sending welcome email …`).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented May 6, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
birthday-card-generator Ready Ready Preview, Comment May 6, 2026 4:07am
flight-booking-app Ready Ready Preview, Comment May 6, 2026 4:07am

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