From a4ee36b681e2f076292f20704d7765961b45d6d5 Mon Sep 17 00:00:00 2001
From: saravanakumardb1 <saravanakumardb1@users.noreply.github.com>
Date: Sat, 23 May 2026 17:06:56 -0700
Subject: [PATCH] fix(admin-web/eslint): also ignore .pnpmfile.cjs at the
 dashboard level
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The previous root-level eslint.config.js .cjs ignore (commit
1be38bef) had no effect on CI because 'pnpm -r exec eslint .' walks
into each workspace and invokes the LOCAL eslint config. admin-web
has its own dashboards/admin-web/eslint.config.mjs (extending
eslint-config-next), which did NOT ignore .cjs files. So the same
.pnpmfile.cjs require() errors kept failing run 64 and run 65 with
identical output.

Fix at the consumer (dashboard) level so 'pnpm -r exec eslint' sees
the ignore regardless of which workspace it runs from:

  globalIgnores([
    ...
    '.pnpmfile.cjs',
    '**/*.cjs',
  ])

Verified locally:
  cd dashboards/admin-web && pnpm lint  -> 0 errors

tracker-web has no .pnpmfile.cjs and no other .cjs configs, so its
eslint config does not need the same change yet — adding it would
be defensive but not required.
---
 dashboards/admin-web/eslint.config.mjs | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/dashboards/admin-web/eslint.config.mjs b/dashboards/admin-web/eslint.config.mjs
index 51a8c89e..5a14310e 100644
--- a/dashboards/admin-web/eslint.config.mjs
+++ b/dashboards/admin-web/eslint.config.mjs
@@ -18,6 +18,12 @@ const eslintConfig = defineConfig([
     "out/**",
     "build/**",
     "next-env.d.ts",
+    // .pnpmfile.cjs is a pnpm install hook (CommonJS by design).
+    // The TypeScript no-require-imports rule would otherwise flag
+    // every require() call in this file. eslint-config-next does NOT
+    // ignore .cjs by default.
+    ".pnpmfile.cjs",
+    "**/*.cjs",
   ]),
 ]);