Description

Awhile back we decided to start allowing for iterator to be used (#34249). The logic was that it only adds 0.12kb to any bundle which uses it, and only .js builds as opposed to .mjs builds. We started implementation of this change, but ran into roadblocks (#34282 (review))

To do:

Update these plugins/rules to handle ArrayExpression:

• build-system/eslint-rules/no-deep-destructuring.js: eslint rule: no-deep-destructuring should handle ArrayPattern #35698
• build-system/eslint-rules/unused-private-field.js
• [Descoped] build-system/eslint-rules/prefer-destructuring.js

Can be removed:

• build-system/babel-plugins/babel-plugin-transform-simple-array-destructure/index.js (babel-plugins: remove simple-array-destructuring #35695)
• no-array-destructuring, no-rest, no-spread (lint: allow rest,spread, and array destructure #35699)

cc @ampproject/wg-performance