mongodb-js / devtools-shared / 20265794165

78%

Build Type

push

github

Committed by web-flow

Commit Message

feat(eslint-plugin-devtools): add rule forbidding non-null prototypes for `Record`s (#604)

`Record` types in TS are typically used to indicate that an object
may have arbitrary keys. This means that using `{}` or similar object
literals to initialize them is a poor choice -- if user code can
cause keys like `hasOwnProperty` or `__proto__` to be used as
keys on those objects, they will not behave as intended.

This has been the cause of many security issues in the JS
ecosystem, so let's add a lint rule to limit impact
of these issues and force ourselves to stick to best practices.

Run Details

1789 of 2775 branches covered (64.47%)

Branch coverage included in aggregate %.

6 of 6 new or added lines in 4 files covered. (100.0%)

25 existing lines in 2 files now uncovered.

3829 of 5010 relevant lines covered (76.43%)

549.16 hits per line