Skip to content

fix(compute): add Hono template entrypoint - #8564

Merged
AmanVarshney01 merged 2 commits into
latestfrom
aman/fix-hono-compute-entrypoint
Jul 28, 2026
Merged

fix(compute): add Hono template entrypoint#8564
AmanVarshney01 merged 2 commits into
latestfrom
aman/fix-hono-compute-entrypoint

Conversation

@AmanVarshney01

@AmanVarshney01 AmanVarshney01 commented Jul 28, 2026

Copy link
Copy Markdown
Member

Purpose of change

Make the Hono Prisma Compute template deployable by giving the Bun build strategy an explicit source entrypoint.

What's changed and why

  • Add entry: "src/index.ts" to compute/hono/prisma.compute.json. Without it, the template installs, applies its database schema, and compiles successfully, but artifact creation fails with Entrypoint is required because package.json has no main or module field.

Testing notes / Before-after

  • Before: app build failed after TypeScript compilation with Entrypoint is required. Pass --entrypoint or define package.json main.
  • After: app build --json --no-interactive succeeds with build type bun and packaged entrypoint index.js.
  • npx vitest run tests/compute.test.ts
  • npx tsc --noEmit
  • DATABASE_URL=postgresql://user:pass@localhost:5432/db bun run build

@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

The Hono Prisma Compute configuration now declares src/index.ts as its application entry point. Compute template tests parse this configuration and verify the Hono entry against the expected smoke route.

Changes

Hono Compute Entry

Layer / File(s) Summary
Configure and validate the Hono entry point
compute/hono/prisma.compute.json, tests/compute.test.ts
The Hono configuration adds app.entry: "src/index.ts", and the compute test validates this framework-specific entry value.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: adding the Hono template entrypoint for the compute config.

Comment @coderabbitai help to get the list of available commands.

@AmanVarshney01

Copy link
Copy Markdown
Member Author

@coderabbitai approve

@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor
✅ Action performed

Comments resolved and changes approved.

@AmanVarshney01
AmanVarshney01 merged commit eb8f432 into latest Jul 28, 2026
8 of 10 checks passed
@AmanVarshney01
AmanVarshney01 deleted the aman/fix-hono-compute-entrypoint branch July 28, 2026 08:57
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