eslint-plugin harden-exports rule mishandles many exports

[gibson042]

(Originally posted by @gibson042 in #2387 (comment) )

Non-comprehensive list:

• export const { propName: exportName } = objWithPropName (prop.key.name of propName: exportName pulls propName, whereas we should pull prop.value.name to ensure the presence of harden(exportName)) and
• export const { name, ...rest } = obj (object rest) and
• export const { name, parent: { subName } } (nested patterns) and
• export const [ exportName1, … ] = array (declaration.id.type === 'ArrayPattern' is just skipped entirely)

The linked comment proposes a fix in the form of a pushDeclaredNames helper that handles default assignment and arbitrarily nested destructuring patterns, to be used in combination with post hoc verification of the style already employed for "should be followed by a call to 'harden'" complaints (i.e., collect all declarations in one phase and validate their conformance in a second phase).