isolatedDeclaration strips module declarations if the file contains imports or exports

[MichaelMitchell-at]

Tested version: 0.23.0

import {} from 'bar';
declare module 'foo' {
    interface Foo {}
    const foo = 42;
}

require('oxc-transform').isolatedDeclaration('hello.ts', `
  import "bar";
  declare module "foo" {
    interface Foo {}
    const foo = 42;
  }
`)
{ sourceText: 'import "bar";\nexport {};\n', errors: [] }

Expected:

import "bar";
declare module "foo" {
  interface Foo {}
  const foo = 42;
}

Actual:

import "bar";
export {};

[MichaelMitchell-at]

Another case

function foo(): void {}
declare namespace foo {
    export let bar = 42;
}
foo.bar = 42;
export {foo};

{
  sourceText: 'declare function foo(): void;\n' +
    'declare namespace foo {\n' +
    '\texport let bar = 42;\n' +
    '}\n' +
    'export { foo };\n',
  errors: [
    '  x TS9023: Assigning properties to functions without declaring them is not\n' +
      '  | supported with --isolatedDeclarations. Add an explicit declaration for the\n' +
      '  | properties assigned to this function.\n' +
      '   ,-[hello.ts:5:1]\n' +
      ' 4 | }\n' +
      ' 5 | foo.bar = 42;\n' +
      '   : ^^^^^^^\n' +
      ' 6 | export {foo};\n' +
      '   `----\n'
  ]
}

[MichaelMitchell-at]

Another case

This was actually a separate issue. Addressed by #4910.

[MichaelMitchell-at]

In working on #4911 I realized that the issue actually happens with any import, not just an import with the same name. Updated the title of this issue to reflect that.

[Dunqing]

Reopen, because #4916