fix(nx-plugin): fix plugin build output and enhance caching

- fix typo in vitest.config output
- change vitest.config to mts for better type support
- list vite and vitest as a dep if enabled
- change build to only output cjs to support nx plugins
- add caching and nx linking if spec file is in another project

Author: seriouslag

packages/nx-plugin/executors.json

@@ -1,8 +1,8 @@
 {
   "executors": {
     "update-api": {
-      "implementation": "./dist/updateApi.js",
-      "schema": "./dist/executors/update-api/schema.json",
+      "implementation": "./dist/updateApi.cjs",
+      "schema": "./dist/executors/update-api/updateApi.schema.json",
       "description": "Updates the OpenAPI spec file and generates new client code if needed."
     }
   }

packages/nx-plugin/generators.json

@@ -3,8 +3,8 @@
   "version": "0.0.1",
   "generators": {
     "openapi-client": {
-      "factory": "./dist/openapiClient.js",
-      "schema": "./dist/generators/openapi-client/schema.json",
+      "factory": "./dist/openapiClient.cjs",
+      "schema": "./dist/generators/openapi-client/openapiClient.schema.json",
       "description": "Generate an OpenAPI client library from an OpenAPI spec file."
     }
   }

packages/nx-plugin/package.json

@@ -25,33 +25,28 @@
     "nx",
     "swagger"
   ],
-  "type": "module",
+  "type": "commonjs",
   "main": "./dist/index.cjs",
-  "module": "./dist/index.js",
+  "module": "./dist/index.cjs",
   "types": "./dist/index.d.ts",
   "scripts": {
-    "build": "tsup && pnpm check-exports",
-    "check-exports": "attw --pack .",
+    "build": "tsup",
     "test:watch": "vitest watch --config vitest.config.ts",
     "test": "vitest run --config vitest.config.ts",
     "typecheck": "tsc --noEmit",
     "prepublishOnly": "pnpm build"
   },
   "exports": {
     ".": {
-      "import": {
-        "types": "./dist/index.d.ts",
-        "default": "./dist/index.js"
-      },
       "require": {
         "types": "./dist/index.d.cts",
         "default": "./dist/index.cjs"
       }
     },
     "./package.json": "./package.json"
   },
-  "executors": "./dist/executors.json",
-  "generators": "./dist/generators.json",
+  "executors": "./executors.json",
+  "generators": "./generators.json",
   "dependencies": {
     "@hey-api/json-schema-ref-parser": "1.0.4",
     "@hey-api/openapi-ts": "workspace:*",
@@ -67,6 +62,8 @@
     "vitest": "3.1.1"
   },
   "files": [
+    "executors.json",
+    "generators.json",
     "package.json",
     "dist",
     "LICENSE.md"

packages/nx-plugin/src/executors/update-api/schema.json renamed to packages/nx-plugin/src/executors/update-api/updateApi.schema.json

@@ -7,11 +7,11 @@ import { type ExecutorContext, logger } from '@nx/devkit';
 import { randomUUID } from 'crypto';
 import { afterAll, describe, expect, it, vi } from 'vitest';
 
-import generator from '../../generators/openapi-client';
+import generator from '../../generators/openapi-client/openapiClient';
 import { getGeneratorOptions, TestOptions } from '../../test-utils';
 import { CONSTANTS } from '../../vars';
-import executor from '.';
 import type { UpdateApiExecutorSchema } from './schema';
+import executor from './updateApi';
 
 vi.mock('@hey-api/openapi-ts/internal', async (importOriginal) => {
   const actual =

packages/nx-plugin/src/executors/update-api/index.spec.ts renamed to packages/nx-plugin/src/executors/update-api/updateApi.spec.ts

@@ -160,7 +160,6 @@ const runExecutor: PromiseExecutor<UpdateApiExecutorSchema> = async (
       logger.info('No changes detected in the API spec.');
       if (!force) {
         logger.info('Force flag is false. Skipping client code generation.');
-        await cleanup(absoluteTempFolder);
         return { success: true };
       } else {
         logger.info('Force flag is true. Generating new client code...');
@@ -237,13 +236,13 @@ const runExecutor: PromiseExecutor<UpdateApiExecutorSchema> = async (
     });
 
     logger.info('Successfully updated API client and spec files.');
-    await cleanup(absoluteTempFolder);
     return { success: true };
   } catch (error) {
     const errorMessage = error instanceof Error ? error.message : String(error);
     logger.debug(`Error details: ${errorMessage}.`);
-    await cleanup(absoluteTempFolder);
     return { success: false };
+  } finally {
+    await cleanup(absoluteTempFolder);
   }
 };
 

packages/nx-plugin/src/executors/update-api/index.ts renamed to packages/nx-plugin/src/executors/update-api/updateApi.ts

@@ -8,13 +8,13 @@ import { afterAll, beforeEach, describe, expect, it, vi } from 'vitest';
 
 import { getGeneratorOptions } from '../../test-utils';
 import { generateClientCode } from '../../utils';
-import generator, { updateTsConfig } from './index';
+import generator, { updateTsConfig } from './openapiClient';
 import {
   generateApi,
   generateNxProject,
   normalizeOptions,
   updatePackageJson,
-} from './index';
+} from './openapiClient';
 
 vi.mock('@hey-api/openapi-ts', async (importOriginal) => {
   const actual = await importOriginal<typeof import('@hey-api/openapi-ts')>();
@@ -128,7 +128,7 @@ describe('openapi-client generator', () => {
       });
       const normalizedOptions = normalizeOptions(options);
 
-      generateNxProject({ clientPlugins: {}, normalizedOptions, tree });
+      await generateNxProject({ clientPlugins: {}, normalizedOptions, tree });
 
       const config = readJson(
         tree,
@@ -147,7 +147,7 @@ describe('openapi-client generator', () => {
       });
       const normalizedOptions = normalizeOptions(options);
 
-      generateNxProject({ clientPlugins: {}, normalizedOptions, tree });
+      await generateNxProject({ clientPlugins: {}, normalizedOptions, tree });
 
       expect(
         tree.exists(`${normalizedOptions.projectRoot}/tsconfig.json`),