Workaround bundled circular referenced types (#1657)

* chore: update rollup config plugin with right typescript config file

* fix: add a workaround to ensure types are properly bundled

* docs: update changelog

CHANGELOG.md

@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 ### Fixed
+- Updated ESM bundled types to solve circularly referenced types.
 
 ## [6.1.5-beta.1]
 

rollup.config.mjs

@@ -48,6 +48,10 @@ export default [
   {
     input: 'lib/esm/index.d.ts',
     output: [{ file: 'lib/esm/index.d.ts', format: 'es' }],
-    plugins: [dts()],
+    plugins: [
+      dts({
+        tsconfig: './tsconfig.esm.json',
+      }),
+    ],
   },
 ];

src/interfaces/interfaces.ts

@@ -1,11 +1,11 @@
 /* eslint-disable @typescript-eslint/naming-convention */
-import {
-  Newable as CommonNewable,
-  ServiceIdentifier as CommonServiceIdentifier,
-} from '@inversifyjs/common';
 import { LegacyTarget } from '@inversifyjs/core';
 
 import { FactoryType } from '../utils/factory_type';
+import {
+  CommonNewable,
+  CommonServiceIdentifier,
+} from './interfaces_common_exports';
 
 // eslint-disable-next-line @typescript-eslint/no-namespace
 namespace interfaces {

src/interfaces/interfaces_common_exports.ts

@@ -0,0 +1,12 @@
+import { Newable, ServiceIdentifier } from '@inversifyjs/common';
+
+// Unnecesary types to workaround https://github.com/Swatinem/rollup-plugin-dts/issues/325#issuecomment-2507540892
+
+export type CommonNewable<
+  TInstance = unknown,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  TArgs extends unknown[] = any[],
+> = Newable<TInstance, TArgs>;
+
+export type CommonServiceIdentifier<TInstance = unknown> =
+  ServiceIdentifier<TInstance>;