feat: rule no-unsafe-values

bmeck commented 1 month ago

Prerequisites checklist

[x] I have read the contributing guidelines.

What is the purpose of this pull request?

Introduce a new rule for checking common interchange problems

What changes did you make? (Give an overview)

Added a rule for:

non-finite (Infinity, null, NaN) values.
lone surrogates due to UTF16

Related Issues

fixes #29

Is there anything you'd like reviewers to focus on?

tests are somewhat limited by using already parsed values so I avoided trying to reparse for getting exact locations within strings and/or couldn't check array/string bounds since things error before getting to the rules processing phase.

linux-foundation-easycla[bot] commented 1 month ago

The committers listed above are authorized under a signed CLA.

:white_check_mark: login: bmeck / name: Bradley Farias (3f35f97178897ae5987c80fa9f4599d515541f03, fd0ac0bc87cc18acfd637bc9f4bc01e4525e4026, c7e2ba898d3c6f103be0a67c268bf49890910b30)

nzakas commented 1 month ago

@bmeck just one outstanding comment to address here.

eslint / json