Remove CLI ESBuild step

This fix aims to address various ESBuild issues of users by removing the ESBuild step from the CLI commands and parsing parsing the OpenAPI spec directly from TS/JS files without any build step by using tsx. The `generate` and `validate` functions are now also exposed as entry points for the ESBuild output, allowing those functions to be used for custom CLIs.
blomqma · Apr 15, 2024 · 2f09414 · 2f09414
1 parent 19e5e38
commit 2f09414
Show file tree

Hide file tree

Showing 13 changed files with 100 additions and 152 deletions.
diff --git a/apps/example/package.json b/apps/example/package.json
@@ -6,18 +6,19 @@
     "dev": "pnpm prebuild && next dev",
     "build": "pnpm prebuild && next build",
     "start": "next start",
-    "generate": "pnpm prebuild && next-rest-framework generate",
-    "validate": "pnpm prebuild && next-rest-framework validate",
+    "generate": "pnpm prebuild && NODE_OPTIONS='--import=tsx' next-rest-framework generate",
+    "validate": "pnpm prebuild && NODE_OPTIONS='--import=tsx' next-rest-framework validate",
     "lint": "tsc && next lint"
   },
   "dependencies": {
     "next-rest-framework": "workspace:*",
+    "tsx": "4.7.2",
     "zod-form-data": "2.0.2"
   },
   "devDependencies": {
     "autoprefixer": "10.0.1",
+    "eslint-config-next": "14.0.4",
     "postcss": "8.4.33",
-    "tailwindcss": "3.3.0",
-    "eslint-config-next": "14.0.4"
+    "tailwindcss": "3.3.0"
   }
 }
diff --git a/docs/docs/api-reference.md b/docs/docs/api-reference.md
@@ -207,10 +207,11 @@ The RPC operation handler function is a strongly-typed function to implement the
 
 ## [CLI](#cli)
 
-The Next REST Framework CLI supports generating and validating the `openapi.json` file:
+The CLI commands will parse your Next.js APIs and generate/validate the `openapi.json` file.
+If using TypeScript, you will need to install `tsx` and use it as the Node.js loader for the CLI commands below: `npm install --save-dev tsx`
 
-- `npx next-rest-framework generate` to generate the `openapi.json` file.
-- `npx next-rest-framework validate` to validate that the `openapi.json` file is up-to-date.
+- `NODE_OPTIONS='--import=tsx' npx next-rest-framework generate` to generate the `openapi.json` file.
+- `NODE_OPTIONS='--import=tsx' npx next-rest-framework validate` to validate that the `openapi.json` file is up-to-date.
 
 The `next-rest-framework validate` command is useful to have as part of the static checks in your CI/CD pipeline. Both commands support the following options:
 
@@ -222,7 +223,7 @@ A good practice is to set these in your `package.json` as both commands are need
 
 ```json
 "scripts": {
-  "generate": "next-rest-framework generate",
-  "validate": "next-rest-framework validate",
+  "generate": "NODE_OPTIONS='--import=tsx' next-rest-framework generate",
+  "validate": "NODE_OPTIONS='--import=tsx' next-rest-framework validate",
 }
 ```
diff --git a/docs/docs/getting-started.md b/docs/docs/getting-started.md
@@ -11,9 +11,10 @@ sidebar_position: 2
 You also need the following dependencies installed in you Next.js project:
 
 - [Next.js](https://github.com/vercel/next.js) >= v12
-- [Zod](https://github.com/colinhacks/zod) >= v3
-- [TypeScript](https://www.typescriptlang.org/) >= v3
-- Optional, needed if working with forms: [zod-form-data](https://www.npmjs.com/package/zod-form-data) >= v2
+- Optional (needed for validating input): [Zod](https://github.com/colinhacks/zod) >= v3
+- Optional: [TypeScript](https://www.typescriptlang.org/) >= v3
+- Optional (needed when using the CLI commands and using TypeScript): [tsx](https://github.com/privatenumber/tsx) >= v4
+- Optional (needed if working with forms): [zod-form-data](https://www.npmjs.com/package/zod-form-data) >= v2
 
 ## [Installation](#installation)
 

diff --git a/docs/docs/intro.md b/docs/docs/intro.md
@@ -30,9 +30,10 @@ Next REST Framework is an open-source, opinionated, lightweight, easy-to-use set
 You also need the following dependencies installed in you Next.js project:
 
 - [Next.js](https://github.com/vercel/next.js) >= v12
-- [Zod](https://github.com/colinhacks/zod) >= v3
-- [TypeScript](https://www.typescriptlang.org/) >= v3
-- Optional, needed if working with forms: [zod-form-data](https://www.npmjs.com/package/zod-form-data) >= v2
+- Optional (needed for validating input): [Zod](https://github.com/colinhacks/zod) >= v3
+- Optional: [TypeScript](https://www.typescriptlang.org/) >= v3
+- Optional (needed when using the CLI commands and using TypeScript): [tsx](https://github.com/privatenumber/tsx) >= v4
+- Optional (needed if working with forms): [zod-form-data](https://www.npmjs.com/package/zod-form-data) >= v2
 
 ### [Installation](#installation)
 

diff --git a/packages/next-rest-framework/README.md b/packages/next-rest-framework/README.md
@@ -101,9 +101,10 @@ This is a monorepo containing the following packages / projects:
 You also need the following dependencies installed in you Next.js project:
 
 - [Next.js](https://github.com/vercel/next.js) >= v12
-- [Zod](https://github.com/colinhacks/zod) >= v3
-- [TypeScript](https://www.typescriptlang.org/) >= v3
-- Optional, needed if working with forms: [zod-form-data](https://www.npmjs.com/package/zod-form-data) >= v2
+- Optional (needed for validating input): [Zod](https://github.com/colinhacks/zod) >= v3
+- Optional: [TypeScript](https://www.typescriptlang.org/) >= v3
+- Optional (needed when using the CLI commands and using TypeScript): [tsx](https://github.com/privatenumber/tsx) >= v4
+- Optional (needed if working with forms): [zod-form-data](https://www.npmjs.com/package/zod-form-data) >= v2
 
 ## [Installation](#installation)
 
@@ -886,10 +887,11 @@ The RPC operation handler function is a strongly-typed function to implement the
 
 ## [CLI](#cli)
 
-The Next REST Framework CLI supports generating and validating the `openapi.json` file:
+The CLI commands will parse your Next.js APIs and generate/validate the `openapi.json` file.
+If using TypeScript, you will need to install [tsx](https://github.com/privatenumber/tsx) and use it as the Node.js loader for the CLI commands below: `npm install --save-dev tsx`
 
-- `npx next-rest-framework generate` to generate the `openapi.json` file.
-- `npx next-rest-framework validate` to validate that the `openapi.json` file is up-to-date.
+- `NODE_OPTIONS='--import=tsx' npx next-rest-framework generate` to generate the `openapi.json` file.
+- `NODE_OPTIONS='--import=tsx' npx next-rest-framework validate` to validate that the `openapi.json` file is up-to-date.
 
 The `next-rest-framework validate` command is useful to have as part of the static checks in your CI/CD pipeline. Both commands support the following options:
 
@@ -901,8 +903,8 @@ A good practice is to set these in your `package.json` as both commands are need
 
 ```json
 "scripts": {
-  "generate": "next-rest-framework generate",
-  "validate": "next-rest-framework validate",
+  "generate": "NODE_OPTIONS='--import=tsx' next-rest-framework generate",
+  "validate": "NODE_OPTIONS='--import=tsx' next-rest-framework validate",
 }
 ```
 

diff --git a/packages/next-rest-framework/package.json b/packages/next-rest-framework/package.json
@@ -38,8 +38,6 @@
   "dependencies": {
     "chalk": "4.1.2",
     "commander": "10.0.1",
-    "esbuild": "0.19.11",
-    "fast-glob": "3.3.2",
     "formidable": "^3.5.1",
     "lodash": "4.17.21",
     "prettier": "3.0.2",
@@ -51,6 +49,7 @@
     "@types/jest": "29.5.4",
     "@types/lodash": "4.14.197",
     "@types/qs": "6.9.11",
+    "esbuild": "0.19.11",
     "jest": "29.6.4",
     "next": "*",
     "node-mocks-http": "1.13.0",

diff --git a/packages/next-rest-framework/src/cli/constants.ts b/packages/next-rest-framework/src/cli/constants.ts
@@ -1,2 +1 @@
 export const OPEN_API_VERSION = '3.0.1';
-export const NEXT_REST_FRAMEWORK_TEMP_FOLDER_NAME = '.next-rest-framework';
diff --git a/packages/next-rest-framework/src/cli/generate.ts b/packages/next-rest-framework/src/cli/generate.ts
@@ -35,11 +35,7 @@ const writeOpenApiSpec = async ({
 };
 
 // Regenerate the OpenAPI spec if it has changed.
-export const syncOpenApiSpecFromBuild = async ({
-  configPath
-}: {
-  configPath?: string;
-}) => {
+export const generate = async ({ configPath }: { configPath?: string }) => {
   const config = await findConfig({ configPath });
 
   if (!config) {

diff --git a/packages/next-rest-framework/src/cli/index.ts b/packages/next-rest-framework/src/cli/index.ts
@@ -2,9 +2,8 @@
 
 import { Command } from 'commander';
 import chalk from 'chalk';
-import { clearTmpFolder, compileEndpoints } from './utils';
-import { validateOpenApiSpecFromBuild } from './validate';
-import { syncOpenApiSpecFromBuild } from './generate';
+import { generate } from './generate';
+import { validate } from './validate';
 
 const program = new Command();
 
@@ -19,18 +18,15 @@ program
     const configPath: string = options.configPath ?? '';
 
     try {
-      await compileEndpoints();
       console.info(chalk.yellowBright('Generating OpenAPI spec...'));
 
-      await syncOpenApiSpecFromBuild({
+      await generate({
         configPath
       });
     } catch (e) {
       console.error(e);
       process.exit(1);
     }
-
-    await clearTmpFolder();
   });
 
 program
@@ -44,10 +40,9 @@ program
     const configPath: string = options.configPath ?? '';
 
     try {
-      await compileEndpoints();
       console.info(chalk.yellowBright('Validating OpenAPI spec...'));
 
-      const valid = await validateOpenApiSpecFromBuild({
+      const valid = await validate({
         configPath
       });
 
@@ -58,8 +53,6 @@ program
       console.error(e);
       process.exit(1);
     }
-
-    await clearTmpFolder();
   });
 
 program.parse(process.argv);
Original file line number	Diff line number	Diff line change
		@@ -1,2 +1 @@
		export const OPEN_API_VERSION = '3.0.1';
		export const NEXT_REST_FRAMEWORK_TEMP_FOLDER_NAME = '.next-rest-framework';