Fix detection of installed package versions

package.json

@@ -126,7 +126,8 @@
       "@typescript-eslint"
     ],
     "rules": {
-      "@typescript-eslint/no-explicit-any": "off"
+      "@typescript-eslint/no-explicit-any": "off",
+      "@typescript-eslint/no-var-requires": "off"
     },
     "env": {
       "node": true

src/common/packageVersions.ts

@@ -1,10 +1,28 @@
 import { createRequire } from "module";
 
+function getRequire() {
+  if (typeof require !== "undefined") {
+    return require;
+  } else {
+    return createRequire(import.meta.url);
+  }
+}
+
 export function getPackageVersion(name: string): string | null {
   try {
-    const _require = createRequire(import.meta.url);
+    const _require = getRequire();
     return _require(`${name}/package.json`).version || null;
   } catch (error) {
     return null;
   }
 }
+
+export function isPackageInstalled(name: string): boolean {
+  try {
+    const _require = getRequire();
+    _require.resolve(name);
+    return true;
+  } catch (error) {
+    return false;
+  }
+}

src/express/middleware.ts

@@ -3,7 +3,10 @@ import { performance } from "perf_hooks";
 
 import { ApitallyClient } from "../common/client.js";
 import { consumerFromStringOrObject } from "../common/consumerRegistry.js";
-import { getPackageVersion } from "../common/packageVersions.js";
+import {
+  getPackageVersion,
+  isPackageInstalled,
+} from "../common/packageVersions.js";
 import {
   ApitallyConfig,
   ApitallyConsumer,
@@ -32,10 +35,10 @@ export const useApitally = (
 };
 
 const getMiddleware = (app: Express | Router, client: ApitallyClient) => {
-  const validatorInstalled = getPackageVersion("express-validator") !== null;
-  const celebrateInstalled = getPackageVersion("celebrate") !== null;
-  const nestInstalled = getPackageVersion("@nestjs/core") !== null;
-  const classValidatorInstalled = getPackageVersion("class-validator") !== null;
+  const validatorInstalled = isPackageInstalled("express-validator");
+  const celebrateInstalled = isPackageInstalled("celebrate");
+  const nestInstalled = isPackageInstalled("@nestjs/core");
+  const classValidatorInstalled = isPackageInstalled("class-validator");
   let errorHandlerConfigured = false;
 
   return (req: Request, res: Response, next: NextFunction) => {
@@ -256,10 +259,14 @@ const getAppInfo = (
     ["nodejs", process.version.replace(/^v/, "")],
   ];
   const expressVersion = getPackageVersion("express");
+  const nestjsVersion = getPackageVersion("@nestjs/core");
   const apitallyVersion = getPackageVersion("../..");
   if (expressVersion) {
     versions.push(["express", expressVersion]);
   }
+  if (nestjsVersion) {
+    versions.push(["nestjs", nestjsVersion]);
+  }
   if (apitallyVersion) {
     versions.push(["apitally", apitallyVersion]);
   }

tests/common/packageVersions.test.ts

@@ -0,0 +1,18 @@
+import { describe, expect, it } from "vitest";
+
+import {
+  getPackageVersion,
+  isPackageInstalled,
+} from "../../src/common/packageVersions.js";
+
+describe("Package versions", () => {
+  it("Check if package is installed", async () => {
+    expect(isPackageInstalled("vitest")).toBe(true);
+    expect(isPackageInstalled("nonexistent")).toBe(false);
+  });
+
+  it("Get package version", () => {
+    expect(getPackageVersion("vitest")).not.toBeNull();
+    expect(getPackageVersion("nonexistent")).toBeNull();
+  });
+});

tsup.config.ts

@@ -2,6 +2,7 @@ import { defineConfig } from "tsup";
 
 export default defineConfig({
   format: ["cjs", "esm"],
+  platform: "node",
   dts: true,
   sourcemap: true,
   splitting: false,