refactor(ast): remove AstKind for AssignmentTarget

[taearls]

Part of #11490. Removes AssignmentTarget from AstKind.

I was able to get through everything and get the tests to all pass with minimal changes (as much as I could, anyway). Some of the rules required more work than others.

Affected files after removing AssignmentTarget from AstKind:

oxc_ast:

• crates/oxc_ast/src/ast_kind_impl.rs

oxc_linter:

• crates/oxc_linter/src/rules/eslint/no_extend_native.rs
• crates/oxc_linter/src/rules/eslint/no_unused_private_class_members.rs
• crates/oxc_linter/src/rules/eslint/no_unused_vars/allowed.rs
• crates/oxc_linter/src/rules/eslint/no_unused_vars/usage.rs
• crates/oxc_linter/src/rules/eslint/no_useless_rename.rs
• crates/oxc_linter/src/rules/import/no_commonjs.rs
• crates/oxc_linter/src/rules/oxc/no_accumulating_spread.rs
• crates/oxc_linter/src/rules/unicorn/no_accessor_recursion.rs
  crates/oxc_linter/src/rules/unicorn/no_unreadable_array_destructuring.rs
• crates/oxc_linter/src/rules/unicorn/prefer_dom_node_text_content.rs

oxc_semantic:

• crates/oxc_semantic/src/checker/javascript.rs

oxc_mangler:

• crates/oxc_mangler/src/keep_names.rs

ast_tools:

• tasks/ast_tools/src/generators/ast_kind.rs

Snapshot updates:

• the snapshot slightly changed for no_commonjs, but it seems to be correct from what I can tell. it looks like there might've been some duplicate reporting.
• the snapshot for no_accessor_recursion slightly updated, but only the column numbers in two of the diagnostic messages, so I think this is fine?

Other than those two snapshots, the others I think come from removing AssignmentTarget and then regenerating the ast types.

[codspeed-hq]

CodSpeed Instrumentation Performance Report

Merging #12252 will not alter performance

_{Comparing taearls:refactor/remove-assignment-target-from-ast-kind (6a8464f) with main (c2db117)}

Summary

✅ 34 untouched benchmarks

[overlookmotel]

Thanks very much for this. Either me or @camc314 will review as soon as we can. These AstKind changes often have quite subtle effects, so we need to do a proper review, and we're both quite tied up at the moment. I'd hope one of us can get to it before end of this week. Please excuse the delay.

[taearls]

Thanks very much for this. Either me or @camc314 will review as soon as we can. These AstKind changes often have quite subtle effects, so we need to do a proper review, and we're both quite tied up at the moment. I'd hope one of us can get to it before end of this week. Please excuse the delay.

No worries at all! That makes complete sense.

[camc314]

thanks!