From 9be49acbfe39e8d5ddbd03706aa2ea08e1783951 Mon Sep 17 00:00:00 2001 From: saravanakumardb1 Date: Sat, 23 May 2026 16:08:41 -0700 Subject: [PATCH] fix(eslint): ignore python venv + playwright artifacts MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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. --- eslint.config.js | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/eslint.config.js b/eslint.config.js index 40fd2b28..13f552e3 100644 --- a/eslint.config.js +++ b/eslint.config.js @@ -12,6 +12,17 @@ export default [ '*.config.ts', '**/dist/**', '__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,