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

fix(eslint): ignore python venv + playwright artifacts

Browse Source 
The root 'Build, Test & Typecheck' CI job was failing at lint with:

  services/extraction-service/python/.venv/lib/python3.13/site-packages/
    urllib3/contrib/emscripten/emscripten_fetch_worker.js
     10:21  error  'TextEncoder' is not defined  no-undef
     12:1   error  'self' is not defined         no-undef
     40:9   error  'console' is not defined      no-undef
     70:30  error  'fetch' is not defined        no-undef
     ...

urllib3 ships emscripten worker stubs that ESLint should never see.
The Python venv was added recently and the root eslint.config.js
didn't yet have any '**/.venv/**' / '**/__pycache__/**' ignore
patterns, so 'pnpm -r exec eslint . --ext .ts,.tsx' picked them up
when run from the extraction-service workspace.

Added four ignore patterns to the root config:
  '**/.venv/**'
  '**/venv/**'
  '**/__pycache__/**'
  '**/playwright-report/**'
  '**/test-results/**'

The Python venv globs are the actual fix; the playwright ones are
preventative — per-package configs already cover most cases but
the root config is what 'pnpm -r exec eslint' walks.
This commit is contained in:
saravanakumardb1 2026-05-23 16:08:41 -07:00
parent 5da706ead1
commit 9be49acbfe
1 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
eslint.config.js
View File

@ -12,6 +12,17 @@ export default [
'*.config.ts', '*.config.ts',
'**/dist/**', '**/dist/**',
'__LOCAL_LLMs/**', '__LOCAL_LLMs/**',
// Python virtual env in extraction-service/python ships with
// urllib3's emscripten worker stubs (JS files that reference
// browser globals like `self`, `fetch`, `TextEncoder`). Those
// files are NOT project source and must never be linted.
'**/.venv/**',
'**/venv/**',
'**/__pycache__/**',
// Playwright artifacts (per-package configs already exclude
// these, but the root config is what runs in CI).
'**/playwright-report/**',
'**/test-results/**',
], ],
}, },
js.configs.recommended, js.configs.recommended,