TS 4.1 should support `require` with nested binding patterns

[sandersn]

prettier uses nested binding patterns to import multiple things from "namespaces":

Code

const {
  builders: {
    concat,
    join
  },
  utils: {
    nub,
    intercalate
  }
} = require('../document')

(Identifier names have been changed to make fun of Haskell.)

Expected behavior:

nub :: Eq a => [a] -> [a]

Actual behavior:

error "Module '../document' has no exported member 'nub'.", and nub: any.

This worked before switching const x = require to create real aliases, and needs to work afterwards.

[sandersn]

Update from surveying user tests+DT for this pattern:

1. Prettier uses it in 21 places, all from document.js. Maybe they're treating it like an index.js?
2. Eslint and discord use it once each.
3. Webpack has 2 tests that use it.
4. zipkin has a source file and a test file that each use it once.

[sandersn]

We can re-open this if prettier wants it, or if somebody else starts using this pattern (they shouldn't, it's a bad idea).