Summary

Addresses https://discord.com/channels/678763474494423051/978209399120216076/1126496823587115018

This PR modifies the noNonoctalDecimalEscape rule and narrows the scope of its fixes. This change eliminates the unusual circumstance of multiple diagnostics being performed on behalf of the user and also rectifies issues with the cargo lintdoc command.

If I understand correctly, the current analyzer should only be able to return one action for each diagnostic.
In this rule, below example \0\8 should have multiple diagnoses and suggest multiple fixes.

const x = "\0\8";
// `\0\8` should be reported and fixed to `\u00008` 
// `\8` should be reported and fixed to `\0\u0038`
// `\8` should be reported and fixed to `\0\\8` (this fix will be removed by this PR because of diagnostic duplication and EscapeBackslash is uncommon change)
// `\8` should be reported and fixed to `\08` (this fix will be removed by this PR because of diagnostic duplication)

Ideally, this rule would allow a unique TextRange to propose a single diagnostic and multiple actions (fixes) for it.
However, this rule in its current state has diagnosed the same place many times.

I remove the code regarding addition of EscapeBackslash for reducing multiple diagnoses anyway. EscapeBackslash is valid change. But fixing the mistake to simple number (8 or 9) Refator is more strait-forward.

Test Plan

cargo test -p rome_js_analyze -- no_nonoctal_decimal_escape

Thought

RuleAdvice can have SuggestionList and CodeSuggestionList. But this isn't used by Rule directly and they need MarkupBuf instead of properties that JsRuleAction requires.