npm - bini-router - Versions diffs - 1.0.30 → 1.0.32 - Mend

bini-router 1.0.30 → 1.0.32

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (6) hide show

package/README.md CHANGED Viewed

@@ -246,6 +246,8 @@ export default function Docs() {
 Layouts wrap all pages in their directory and subdirectories. bini-router walks up the directory tree from each page to collect the full layout chain, stopping at the `appDir` root.
+All layouts — including the root layout — are rendered as React Router `<Route element>` wrappers using `<Outlet />`. The root layout receives child routes via `<Outlet />` exactly like nested layouts do.
 ```tsx
 // src/app/layout.tsx — root layout
 export const metadata = {
@@ -253,13 +255,13 @@ export const metadata = {
   description: 'Built with bini-router',
 }
-export default function RootLayout({ children }: { children: React.ReactNode }) {
-  return <>{children}</>
+export default function RootLayout() {
+  return <Outlet />
 }
 ```
 ```tsx
-// src/app/dashboard/layout.tsx — nested layout (Outlet auto-imported)
+// src/app/dashboard/layout.tsx — nested layout
 export const metadata = {
   title: 'Dashboard',
 }
@@ -274,8 +276,6 @@ export default function DashboardLayout() {
 }
 ```
-> **Root layout** uses `{children}` — it wraps `<BrowserRouter>` from outside.
-> **Nested layouts** use `<Outlet />` — they are React Router route wrappers.
 > Layouts that contain an `<html>` tag are automatically excluded from the chain (treated as HTML shell files, not route layouts).
 > Layouts without a default export are also excluded from the chain.
 > Circular layout dependencies are detected and throw a `CircularLayoutError`.
@@ -376,7 +376,7 @@ API handlers are loaded on-demand and cached by `mtime` — touching a file in d
 import { Hono } from 'hono'
 import nodemailer from 'nodemailer'
-const app = new Hono().basePath('/api')
+const app = new Hono()
 const transporter = nodemailer.createTransport({
   host: 'smtp-relay.brevo.com',
@@ -396,7 +396,7 @@ app.post('/email', async (c) => {
 export default app
 ```
-> Hono apps are detected by checking for `from 'hono'` in the file source. Detected Hono apps are mounted with `app.route('/', handler)` in the production entry — they manage their own path prefixes.
+> Hono apps are detected by checking for `from 'hono'` in the file source. Do **not** call `.basePath('/api')` — bini-router already mounts the app under `/api` in the production entry. Adding it yourself will result in double-prefixed routes (`/api/api/...`).
 ### Plain function handlers
@@ -415,7 +415,7 @@ Route params are passed via the `x-bini-params` request header as a JSON string
 // src/app/api/posts/[id].ts
 import { Hono } from 'hono'
-const app = new Hono().basePath('/api')
+const app = new Hono()
 app.get('/posts/:id', (c) => c.json({ id: c.req.param('id') }))
 export default app
 ```
@@ -450,6 +450,8 @@ biniroute({ platform: 'netlify' })
 Generates `netlify/edge-functions/api.ts` using Deno CDN URL imports (`hono@v4.3.11`) — no npm deps needed in the edge function.
+> ⚠️ **Netlify Edge Functions run on the Deno runtime, not Node.js.** Node-specific packages like `nodemailer`, `fs`, `path`, or anything that depends on Node built-ins will not work. Use Deno-compatible or Web API alternatives instead (e.g. `fetch` for HTTP, Deno CDN imports for utilities).
 Add `netlify.toml`:
 ```toml
@@ -518,9 +520,7 @@ directory = "./dist"
 binding = "ASSETS"
 ```
-```bash
-vite build && npx wrangler deploy
-```
+Run `vite build` — the worker file is generated automatically and picked up by the Cloudflare dashboard on deploy.
 ---
@@ -542,9 +542,20 @@ biniroute({ platform: 'deno' })
 Generates `server/index.ts` (or `server/index.js`) using Deno CDN imports (`hono@v4.3.11`) and `Deno.serve`. Port defaults to `3000` or reads from the `PORT` environment variable.
-```bash
-vite build && deno run --allow-net --allow-read --sloppy-imports server/index.ts
-```
+> ⚠️ **Deno Deploy does not run Node.js.** Node-specific packages like `nodemailer`, `fs`, `path`, or anything that depends on Node built-ins will not work. Use Deno-compatible or Web API alternatives instead (e.g. `fetch` for HTTP, Deno CDN imports for utilities).
+> ⚠️ **Deno Deploy reads `server/` before the build step runs.** You must commit the generated file:
+>
+> ```bash
+> git add server/index.ts
+> git commit -m "chore: update deno server entry"
+> git push
+> ```
+In Deno Console, set:
+- **Entrypoint**: `server/index.ts`
+- **Build Command**: `vite build`
+- **Runtime**: Dynamic App
 ---
@@ -564,7 +575,7 @@ With `basePath: '/app'`:
 - `src/app/api/users.ts` → `/app/api/users`
 - `BrowserRouter basename` is set to `"/app"` at build time
-> Without `basePath` set, `basename` defaults to `"/"`.
+> Without `basePath` set, `basename` falls back to `import.meta.env.BASE_URL` and then `"/"`.
 ---

package/dist/index.cjs CHANGED Viewed

@@ -1176,7 +1176,7 @@ var ADAPTERS = {
     importLine: `import { Hono } from 'https://deno.land/x/hono@v4.3.11/mod.ts';
 import { handle } from 'https://deno.land/x/hono@v4.3.11/adapter/netlify/index.ts';`,
     exportLine: `export default handle(app);`,
-    outFile: (cwd) => path__default.default.join(cwd, "netlify", "edge-functions", "api.ts"),
+    outFile: (cwd, ts) => path__default.default.join(cwd, "netlify", "edge-functions", ts ? "api.ts" : "api.js"),
     stripsApiPrefix: false,
     usesDenoRuntime: true
   },
@@ -1349,21 +1349,37 @@ function buildProductionEntry(srcApiDir, platform, enableCors, basePath = "") {
     lines.push(...[
       `// Serve static files from dist (must come AFTER API routes)`,
       `app.get('/*', async (c) => {`,
+      `  // Skip API routes - let them be handled first`,
+      `  if (c.req.path.startsWith('/api/')) {`,
+      `    return c.text('API route not found', 404);`,
+      `  }`,
+      `  `,
+      `  const filePath = c.req.path === '/' ? '/index.html' : c.req.path;`,
+      `  `,
       `  try {`,
-      `    const path = c.req.path === '/' ? '/index.html' : c.req.path;`,
-      `    const file = await Deno.readFile(\`./dist\${path}\`);`,
-      `    const ext = path.split('.').pop();`,
-      `    const contentType = ext === 'html' ? 'text/html' :`,
-      `                       ext === 'css' ? 'text/css' :`,
-      `                       ext === 'js' ? 'application/javascript' :`,
-      `                       'text/plain';`,
-      `    return new Response(file, { headers: { 'Content-Type': contentType } });`,
+      `    const file = await Deno.readFile(\`./dist\${filePath}\`);`,
+      `    const ext = filePath.split('.').pop();`,
+      `    const contentType = `,
+      `      ext === 'html' ? 'text/html' :`,
+      `      ext === 'css' ? 'text/css' :`,
+      `      ext === 'js' ? 'application/javascript' :`,
+      `      ext === 'json' ? 'application/json' :`,
+      `      ext === 'png' ? 'image/png' :`,
+      `      ext === 'jpg' || ext === 'jpeg' ? 'image/jpeg' :`,
+      `      'text/plain';`,
+      `    `,
+      `    return new Response(file, {`,
+      `      headers: { 'Content-Type': contentType }`,
+      `    });`,
       `  } catch {`,
+      `    // SPA fallback`,
       `    try {`,
       `      const indexHtml = await Deno.readFile('./dist/index.html');`,
-      `      return new Response(indexHtml, { headers: { 'Content-Type': 'text/html' } });`,
+      `      return new Response(indexHtml, {`,
+      `        headers: { 'Content-Type': 'text/html' }`,
+      `      });`,
       `    } catch {`,
-      `      return new Response('File not found', { status: 404 });`,
+      `      return c.text('File not found', 404);`,
       `    }`,
       `  }`,
       `});`,
@@ -1394,36 +1410,56 @@ function buildProductionEntry(srcApiDir, platform, enableCors, basePath = "") {
     fs__default.default.writeFileSync(outFile, lines.join("\n") + "\n", "utf8");
     console.log(`\x1B[90m${(/* @__PURE__ */ new Date()).toLocaleTimeString()}\x1B[0m \x1B[36m[vite]\x1B[0m \x1B[32m\u2713 Generated\x1B[0m \x1B[90m${toPosixPath(path__default.default.relative(cwd, outFile))}\x1B[0m`);
     if (platform === "vercel") {
+      const serverFile = path__default.default.relative(cwd, outFile);
       console.log(`
 \x1B[90m${(/* @__PURE__ */ new Date()).toLocaleTimeString()}\x1B[0m \x1B[33m[vite]\x1B[0m \x1B[33m\u26A0\uFE0F  Vercel platform detected.\x1B[0m`);
       console.log(`  Vercel reads your api/ directory BEFORE the build step runs.
   You must commit the generated file to your repository:
-    git add ${toPosixPath(path__default.default.relative(cwd, outFile))}
+    git add ${toPosixPath(serverFile)}
     git commit -m "chore: update vercel api entry"
     git push
   Without this step, Vercel will not find your API routes.
 `);
     }
-    if (platform === "cloudflare") {
+    if (platform === "deno") {
+      const serverFile = path__default.default.relative(cwd, outFile);
       console.log(`
-\x1B[90m${(/* @__PURE__ */ new Date()).toLocaleTimeString()}\x1B[0m \x1B[36m[vite]\x1B[0m \x1B[32m\u2713 Cloudflare Worker generated successfully\x1B[0m \x1B[90m${toPosixPath(path__default.default.relative(cwd, outFile))}\x1B[0m
-  The worker includes SPA fallback for React Router.
-  Make sure your wrangler.toml has the ASSETS binding configured.
+\x1B[90m${(/* @__PURE__ */ new Date()).toLocaleTimeString()}\x1B[0m \x1B[33m[vite]\x1B[0m \x1B[33m\u26A0\uFE0F  Deno Deploy platform detected.\x1B[0m`);
+      console.log(`  Deno Deploy reads your server/ directory BEFORE the build step runs.
+  You must commit the generated file to your repository:
+    git add ${toPosixPath(serverFile)}
+    git commit -m "chore: update deno server entry"
+    git push
+  Without this step, Deno Deploy will not find your API routes.
+`);
+      console.log(`  In Deno Console, set:
+    - Entrypoint: ${toPosixPath(serverFile)}
+    - Build Command: vite build
+    - Runtime: Dynamic App
 `);
     }
-    if (platform === "deno") {
+    if (platform === "netlify") {
+      const functionFile = path__default.default.relative(cwd, outFile);
       console.log(`
-\x1B[90m${(/* @__PURE__ */ new Date()).toLocaleTimeString()}\x1B[0m \x1B[36m[vite]\x1B[0m \x1B[32m\u2713 Deno server generated successfully\x1B[0m \x1B[90m${toPosixPath(path__default.default.relative(cwd, outFile))}\x1B[0m
-  To run your Deno server:
-    deno run --allow-net --allow-read --allow-env ${toPosixPath(path__default.default.relative(cwd, outFile))}
+\x1B[90m${(/* @__PURE__ */ new Date()).toLocaleTimeString()}\x1B[0m \x1B[36m[vite]\x1B[0m \x1B[32m\u2713 Netlify function generated successfully\x1B[0m \x1B[90m${toPosixPath(functionFile)}\x1B[0m`);
+      console.log(`  The function is ready for deployment to Netlify Edge Functions.
+  Make sure your netlify.toml has:
+    [functions]
+      directory = "netlify/edge-functions"
 `);
     }
-    if (platform === "netlify") {
+    if (platform === "cloudflare") {
+      const workerFile = path__default.default.relative(cwd, outFile);
       console.log(`
-\x1B[90m${(/* @__PURE__ */ new Date()).toLocaleTimeString()}\x1B[0m \x1B[36m[vite]\x1B[0m \x1B[32m\u2713 Netlify function generated successfully\x1B[0m \x1B[90m${toPosixPath(path__default.default.relative(cwd, outFile))}\x1B[0m
-  The function is ready for deployment to Netlify Edge Functions.
+\x1B[90m${(/* @__PURE__ */ new Date()).toLocaleTimeString()}\x1B[0m \x1B[36m[vite]\x1B[0m \x1B[32m\u2713 Cloudflare Worker generated successfully\x1B[0m \x1B[90m${toPosixPath(workerFile)}\x1B[0m`);
+      console.log(`  The worker includes SPA fallback for React Router.
+  Make sure your wrangler.toml has the ASSETS binding configured.
 `);
     }
   } catch (error) {