bytelyst/learning_ai_common_plat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(eslint): also ignore .pnpmfile.cjs / .cjs (CommonJS by design)

Browse Source 
After the .venv ignore fix landed (commit 9be49acb), a different
lint error surfaced in CI run 64:

  /dashboards/admin-web/.pnpmfile.cjs
     1:12  error  A `require()` style import is forbidden  @typescript-eslint/no-require-imports
     2:14  error  A `require()` style import is forbidden  @typescript-eslint/no-require-imports

`.pnpmfile.cjs` is a pnpm hook that MUST be CommonJS; it cannot
use ESM imports because pnpm loads it via Node's require() and the
file is not part of the TypeScript source tree at all. The
@typescript-eslint/no-require-imports rule should never apply to
*.cjs files.

Added two ignore patterns to the root config:

  '**/.pnpmfile.cjs'
  '**/*.cjs'

The broader '*.cjs' glob also covers any future CommonJS config
files (next.config.cjs, postcss.config.cjs, etc.).
This commit is contained in:
saravanakumardb1 2026-05-23 16:58:45 -07:00
parent 539e64e5d6
commit 1be38bef28
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
eslint.config.js
View File

@ -25,6 +25,11 @@ export default [
// these, but the root config is what runs in CI). // these, but the root config is what runs in CI).
'**/playwright-report/**', '**/playwright-report/**',
'**/test-results/**', '**/test-results/**',
// .pnpmfile.cjs / .cjs config files use CommonJS require() by
// design and must not be linted with the TypeScript ESM rule set
// that forbids require-style imports.
'**/.pnpmfile.cjs',
'**/*.cjs',
], ],
}, },
js.configs.recommended, js.configs.recommended,