chore(eslint): block process.env rule bypasses - #9679
Conversation
|
I can only run on private repositories. |
Overall package sizeSelf size: 7.81 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 3.3.3 | 125.43 kB | 441.68 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | dc-polyfill | 0.1.11 | 25.74 kB | 25.74 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
|
BenchmarksBenchmark execution time: 2026-08-04 15:20:01 Comparing candidate commit 2140105 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 2315 metrics, 43 unstable metrics.
|
What does this PR do?
process.envguardrail, including block, line, next-line, mixed-rule, and explained directives.Motivation
Review feedback on PR #9075 identified that an inline ESLint disable can bypass the repository's tracked environment-variable access policy. Centralizing the existing baseline and rejecting new suppressions keeps configuration reads observable without forcing unrelated legacy cleanup into this change.
Testing
npm run test:eslint-rulesnpm run lintpackages/dd-trace/src/proxy.jsfails with the new guardrail message.Additional Notes
This changes lint enforcement only and has no production runtime impact.
PR by Bits - View session in Datadog
Comment @DataDog to request changes