@hono/cli 0.1.1 → 0.1.2

package/dist/cli.js

@@ -70,61 +70,30 @@ import { dirname, join, resolve } from "path";
 
 // src/utils/build.ts
 import * as esbuild from "esbuild";
-async function* buildAndImportApp(filePath, options = {}) {
-  let resolveApp;
-  let appPromise;
-  const preparePromise = () => {
-    appPromise = new Promise((resolve4) => {
-      resolveApp = resolve4;
+import { extname } from "path";
+import { pathToFileURL } from "url";
+async function buildAndImportApp(filePath, options = {}) {
+  const ext = extname(filePath);
+  if ([".ts", ".tsx", ".jsx"].includes(ext)) {
+    const result = await esbuild.build({
+      entryPoints: [filePath],
+      bundle: true,
+      write: false,
+      format: "esm",
+      target: "node20",
+      jsx: "automatic",
+      jsxImportSource: "hono/jsx",
+      platform: "node",
+      external: options.external || []
     });
-  };
-  preparePromise();
-  const context2 = await esbuild.context({
-    entryPoints: [filePath],
-    bundle: true,
-    write: false,
-    format: "esm",
-    target: "node20",
-    jsx: "automatic",
-    jsxImportSource: "hono/jsx",
-    platform: "node",
-    external: options.external || [],
-    plugins: [
-      {
-        name: "watch",
-        setup(build2) {
-          build2.onEnd(async (result) => {
-            try {
-              const code = result.outputFiles?.[0]?.text || "";
-              const dataUrl = `data:text/javascript;base64,${Buffer.from(code).toString("base64")}`;
-              const module = await import(dataUrl);
-              const app = module.default;
-              if (!app) {
-                throw new Error("Failed to build app");
-              }
-              if (!app || typeof app.request !== "function") {
-                throw new Error("No valid Hono app exported from the file");
-              }
-              try {
-                resolveApp(app);
-              } catch {
-              }
-            } catch (error) {
-              console.error("Error building app", error);
-            }
-          });
-        }
-      }
-    ]
-  });
-  await context2.watch();
-  if (!options.watch) {
-    await context2.dispose();
+    const code = result.outputFiles[0].text;
+    const dataUrl = `data:text/javascript;base64,${Buffer.from(code).toString("base64")}`;
+    const module = await import(dataUrl);
+    return module.default;
+  } else {
+    const module = await import(pathToFileURL(filePath).href);
+    return module.default;
   }
-  do {
-    yield await appPromise;
-    preparePromise();
-  } while (options.watch);
 }
 
 // src/commands/optimize/index.ts
@@ -139,10 +108,9 @@ function optimizeCommand(program2) {
       throw new Error(`Entry file ${entry} does not exist`);
     }
     const appFilePath = realpathSync(appPath);
-    const buildIterator = buildAndImportApp(appFilePath, {
+    const app = await buildAndImportApp(appFilePath, {
       external: ["@hono/node-server"]
     });
-    const app = (await buildIterator.next()).value;
     let routerName;
     let importStatement;
     let assignRouterStatement;
@@ -193,13 +161,13 @@ function optimizeCommand(program2) {
       plugins: [
         {
           name: "hono-optimize",
-          setup(build2) {
+          setup(build3) {
             const honoPseudoImportPath = "hono-optimized-pseudo-import-path";
-            build2.onResolve({ filter: /^hono$/ }, async (args) => {
+            build3.onResolve({ filter: /^hono$/ }, async (args) => {
               if (!args.importer) {
                 return void 0;
               }
-              const resolved = await build2.resolve(args.path, {
+              const resolved = await build3.resolve(args.path, {
                 kind: "import-statement",
                 resolveDir: args.resolveDir
               });
@@ -207,7 +175,7 @@ function optimizeCommand(program2) {
                 path: join(dirname(resolved.path), honoPseudoImportPath)
               };
             });
-            build2.onLoad({ filter: new RegExp(`/${honoPseudoImportPath}$`) }, async () => {
+            build3.onLoad({ filter: new RegExp(`/${honoPseudoImportPath}$`) }, async () => {
               return {
                 contents: `
 import { HonoBase } from 'hono/hono-base'
@@ -235,7 +203,7 @@ import { existsSync as existsSync2, realpathSync as realpathSync2 } from "fs";
 import { resolve as resolve2 } from "path";
 var DEFAULT_ENTRY_CANDIDATES2 = ["src/index.ts", "src/index.tsx", "src/index.js", "src/index.jsx"];
 function requestCommand(program2) {
-  program2.command("request").description("Send request to Hono app using app.request()").argument("[file]", "Path to the Hono app file").option("-P, --path <path>", "Request path", "/").option("-X, --method <method>", "HTTP method", "GET").option("-d, --data <data>", "Request body data").option("-w, --watch", "Watch for changes and re-run the request", false).option(
+  program2.command("request").description("Send request to Hono app using app.request()").argument("[file]", "Path to the Hono app file").option("-P, --path <path>", "Request path", "/").option("-X, --method <method>", "HTTP method", "GET").option("-d, --data <data>", "Request body data").option(
     "-H, --header <header>",
     "Custom headers",
     (value, previous) => {
@@ -244,15 +212,11 @@ function requestCommand(program2) {
     []
   ).action(async (file, options) => {
     const path = options.path || "/";
-    const watch = options.watch;
-    const buildIterator = getBuildIterator(file, watch);
-    for await (const app of buildIterator) {
-      const result = await executeRequest(app, path, options);
-      console.log(JSON.stringify(result, null, 2));
-    }
+    const result = await executeRequest(file, path, options);
+    console.log(JSON.stringify(result, null, 2));
   });
 }
-function getBuildIterator(appPath, watch) {
+async function executeRequest(appPath, requestPath, options) {
   let entry;
   let resolvedAppPath;
   if (appPath) {
@@ -266,12 +230,12 @@ function getBuildIterator(appPath, watch) {
     throw new Error(`Entry file ${entry} does not exist`);
   }
   const appFilePath = realpathSync2(resolvedAppPath);
-  return buildAndImportApp(appFilePath, {
-    external: ["@hono/node-server"],
-    watch
+  const app = await buildAndImportApp(appFilePath, {
+    external: ["@hono/node-server"]
   });
-}
-async function executeRequest(app, requestPath, options) {
+  if (!app || typeof app.request !== "function") {
+    throw new Error("No valid Hono app exported from the file");
+  }
   const url = new URL(requestPath, "http://localhost");
   const requestInit = {
     method: options.method || "GET"
@@ -307,7 +271,11 @@ async function executeRequest(app, requestPath, options) {
 function searchCommand(program2) {
   program2.command("search").argument("<query>", "Search query for Hono documentation").option("-l, --limit <number>", "Number of results to show (default: 5)", (value) => {
     const parsed = parseInt(value, 10);
-    if (isNaN(parsed) || parsed < 1 || parsed > 20) {
+    if (isNaN(parsed) || parsed < 1) {
+      throw new Error(`Limit must be a number between 1 and 20. Received: ${value}
+`);
+    }
+    if (parsed > 20) {
       console.warn("Limit must be a number between 1 and 20\n");
       return 5;
     }
@@ -502,7 +470,14 @@ var builtinMap = {
   }
 });
 function serveCommand(program2) {
-  program2.command("serve").description("Start server").argument("[entry]", "entry file").option("-p, --port <port>", "port number").option("--show-routes", "show registered routes").option(
+  program2.command("serve").description("Start server").argument("[entry]", "entry file").option("-p, --port <port>", "port number", (value) => {
+    const parsed = parseInt(value, 10);
+    if (!/^\d+$/.test(value) || parsed < 0 || parsed > 65535) {
+      throw new Error(`Port must be a number between 0 and 65535. Received: ${value}
+`);
+    }
+    return parsed;
+  }).option("--show-routes", "show registered routes").option(
     "--use <middleware>",
     "use middleware",
     (value, previous) => {
@@ -520,10 +495,9 @@ function serveCommand(program2) {
           app = new Hono();
         } else {
           const appFilePath = realpathSync3(appPath);
-          const buildIterator = buildAndImportApp(appFilePath, {
+          app = await buildAndImportApp(appFilePath, {
             external: ["@hono/node-server"]
           });
-          app = (await buildIterator.next()).value;
         }
       }
       const allFunctions = {};
@@ -556,7 +530,7 @@ function serveCommand(program2) {
       serve(
         {
           fetch: baseApp.fetch,
-          port: options.port ? Number.parseInt(options.port) : 7070
+          port: options.port ?? 7070
         },
         (info) => {
           console.log(`Listening on http://localhost:${info.port}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hono/cli",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "type": "module",
   "bin": {
     "hono": "dist/cli.js"