imports inside module declaration are moved outside, which can break types

[floriancargoet]

Describe the bug

Since 3.8, this ambient declaration in foo.d.ts :

declare module "foo" {
  import { Bar } from "bar";
  type Foo = {
    bar: Bar
  };
  export = Foo;
}

is transformed into:

import { Bar } from "bar"; // This import moved

declare module "foo" {
  type Foo = {
    bar: Bar
  };
  export = Foo;
}

This top-level import turns an ambient module into a standard module and it breaks the global declare module "foo".

Reproduction

.

Steps to reproduce

No response

System Info

System:
    OS: macOS 13.6.6
    CPU: (4) x64 Intel(R) Core(TM) i7-7660U CPU @ 2.50GHz
    Memory: 40.82 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.10.0 - ~/.nvm/versions/node/v20.10.0/bin/node
    npm: 10.2.3 - ~/.nvm/versions/node/v20.10.0/bin/npm
    pnpm: 8.15.5 - ~/.nvm/versions/node/v20.10.0/bin/pnpm
    bun: 1.1.0 - ~/.bun/bin/bun
    Watchman: 2023.12.04.00 - /usr/local/bin/watchman
  Browsers:
    Chrome: 123.0.6312.107
    Safari: 17.4.1
  npmPackages:
    @vitejs/plugin-react: 4.2.1 => 4.2.1
    vite: 5.2.8 => 5.2.8
    vite-plugin-dts: 3.8.1 => 3.8.1

Validations

• Read the FAQ.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• The provided reproduction is a minimal reproducible example of the bug.

[floriancargoet]

Thanks! I'll test it in the next release.