fix(lint/noBlankTarget): don't hang when applying the code fix

[Conaclos]

Summary

Fix #2675

The code fix created a JSX string with a JSX_IDENT token.
inner_string_text is called to retrieve the content of the attribute.
inner_string_text removes quotes only when the token is a JSX_STRING_LITERAL.
Thus, on first run, the quotes were well removed, while on the following run, the quotes are not removed.
This led to a never fixed lint rule and thus, an infinite loop.

I also fixed a similar issue for noUselessFragments which was not observed yet.

To avoid this kind of issue we could:

• Add debug_assert in node factories (this should be generated by the grammar) to guarantee that every leaf node has the correct token kind.
• Avoid the infinite loop by setting a limit. We can expect to have some conflicting rules in the future.
  We should ensure that this doesn't lead to an infinite loop.

Test Plan

I manually tested the code.
We could add a test in biome_cli however I think it is not worth it because this could require to add a test for every rule that create jsx/js/css strings.

[codspeed-hq]

CodSpeed Performance Report

Merging #2687 will not alter performance

_{Comparing conaclos/noBlankTarget/2675 (db69dcf) with main (afa5004)}

Summary

✅ 87 untouched benchmarks