Update eslint and code to satisfy linter

.eslintrc.js

@@ -4,8 +4,8 @@ module.exports = {
   plugins: ['@typescript-eslint'],
   root: true,
   rules: {
-    prettier: {
-      'space-before-function-paren': ['error', 'never']
-    }
+    'func-call-spacing': 'off',
+    'space-before-function-paren': 'off',
+    '@typescript-eslint/no-var-requires': 'off'
   }
 };

package.json

@@ -8,7 +8,7 @@
     "build": "tsup",
     "build:watch": "tsup --watch",
     "test": "cd tests/nextjs && next build && cd ../.. && vitest",
-    "test:ui": "vitest --ui"
+    "test:ui": "cd tests/nextjs && next build && cd ../.. && vitest --ui"
   },
   "license": "Apache-2.0",
   "author": "Last9 team <hi@last9.io>",

src/OpenAPM.ts

@@ -18,7 +18,7 @@ import { instrumentExpress } from './clients/express';
 import { instrumentMySQL } from './clients/mysql2';
 import { instrumentNestFactory } from './clients/nestjs';
 import { LevitateConfig, LevitateEvents } from './levitate/events';
-import { instrumentNextjs } from './clients/nextjs/nextjs';
+import { instrumentNextjs } from './clients/nextjs';
 
 export type ExtractFromParams = {
   from: 'params';
@@ -294,7 +294,7 @@ export class OpenAPM extends LevitateEvents {
         : pathname;
 
       const labels: Record<string, string> = {
-        path: path,
+        path,
         status: res.statusCode.toString(),
         method: req.method as string,
         ...parsedLabelsFromPathname

src/clients/nextjs/nextjs.ts → src/clients/nextjs.ts

@@ -1,7 +1,7 @@
 import type NextNodeServer from 'next/dist/server/next-server';
 import chokidar from 'chokidar';
 import type { Counter, Histogram } from 'prom-client';
-import { wrap } from '../../shimmer';
+import { wrap } from '../shimmer';
 import { loadManifest } from 'next/dist/server/load-manifest';
 import { join } from 'path';
 import { getRouteRegex } from 'next/dist/shared/lib/router/utils/route-regex';
@@ -33,7 +33,9 @@ const PATHS_CACHE = {
         false
       )) as Record<string, string>;
 
-      for (const [key, _] of Object.entries(appPathsManifest)) {
+      const appPathsKeys = Object.keys(appPathsManifest);
+      for (let i = 0; i < appPathsKeys.length; i++) {
+        const key = appPathsKeys[i];
         const path = key.replace(
           /\/(page|not-found|layout|loading|head|route)$/,
           ''
@@ -48,7 +50,9 @@ const PATHS_CACHE = {
         });
       }
 
-      for (const key of Object.keys(pagesManifest)) {
+      const keys = Object.keys(pagesManifest);
+      for (let i = 0; i < keys.length; i++) {
+        const key = keys[i];
         const reg = getRouteRegex(key);
         const matcher = getRouteMatcher(reg);
 
@@ -132,7 +136,9 @@ export const instrumentNextjs = (
   PATHS_CACHE.setValue();
   PATHS_CACHE.keepUpdated();
   const getParameterizedRoute = (route: string) => {
-    for (const page of PATHS_CACHE.value) {
+    const values = Array.from(PATHS_CACHE.value);
+    for (let i = 0; i < values.length; i++) {
+      const page = values[i];
       if (page.matcher(route) !== false) {
         return page.route;
       }