[no-unused-modules] Dynamic import doesn't mark exports as used

[phaux]

Repro:

// foo.ts
export const foo = "foo"

// index.ts
async function test() {
  const { foo } = await import("./foo")
  const foo2 = await import("./foo").then(mod => mod.foo)
  const mod = await import("./foo") // should mark all non-type exports as used
}

My specific use case is:

import { lazy } from "react"
const Foo = lazy(() => import("./Foo"))

exported declaration 'foo' not used within other modules

Config:

{
  "extends": [
    "plugin:import/recommended",
    "plugin:import/typescript",
    // and more ...
  ],
  "rules": {
    "import/no-unused-modules": ["error", { "unusedExports": true }],
  },
}

Versions:

@typescript-eslint/parser: 4.8.0
eslint: 7.13.0
eslint-plugin-import: 2.22.1

[ljharb]

This was intentional during the design of the rule (#1142, cc @rfermann).

When a dynamic import is done with a static, hardcoded path, it does seem possible for this plugin to detect it, but it would have to ignore any other forms of import(). A PR that detects these would be appreciated.

[taoliujun]

me too.
components are lazy imported.

[silverwind]

Related PR: #1660