Repository: TypeScript
Issue: Typescript has more trouble resolving circular class references when using mixins / through function invocations
Labels: Help Wanted,Possible Improvement

### 🔎 Search Terms

Mixin function invocation circular inherits references itself

Related issues:
https://github.com/microsoft/TypeScript/issues/29872
https://github.com/microsoft/TypeScript/issues/42383

### 🕗 Version & Regression Information

- This is the behavior in every version I tried, and I reviewed the FAQ for entries about limit of typescript and resolving circular references.


### ⏯ Playground Link

https://www.typescriptlang.org/play?ts=5.2.2#code/C4TwDgpgBAwg9gOwM7AE4FcDGw6oCrgQA8eAfFALxQCGARiqtdlAhAO5QAUAdL9agHMkALhoIQAbQC6ASkrk8AbgCwAKDUAzdAmwBLRFAC21ANYQAsroAeuhCQBC1JNAhXgEBABMksRAyw4+IRE1OKkADRQAGK21AA28MhoAbiknGpQmUbWtlHaeoiinI7OonglEHIU5DEI8Yn+2LhqMkUVAPxlFVU1sQl+yU2oUADeGVmoEMDoqAhcHV1OlfKj41lZAPQbUJ5wUHDAABYQw0e2QlBsukdQFZFnPq7UhmBx0Lo+SLovcboauhBPGt1lBJtNZtkbAg8jpgPoEMUllB2u0uAB5WgAKwgzCcYhAMhkKlU6wAvsTSWo1JgBlBLFCopQjKYLDkEQ4ka53F4fA1BoECJAQmE0hVFs4eqsSTR6GgmMAoJg4k4fPTbFAuR5vLckWNpZkNHA4JxDXBRGqEABBMCvAGeSJcxiiWhGt6hVp0tnW22AqAAHygLrgbrmI1BUxmc0d1CglOlccyYMjnqhalJMip6lUNKSKdsACZGVRjGYLZwOc4NW4tbyBhghoLiKEQKRRUtxctqlLMnQGPLFcqkKq2fmq9ztRVu1laPwTUbzV6bb9AQ63E7A66IO6F1DvcvPP6N8Gt6Hw+Co2uYwnY+MkxCLfm0xms0qVXmrUu7WOa+-C5wLVEnCEqMpJAA

### 💻 Code

```ts
type ConstructorType<T> = abstract new (...args: any[]) => T;

function makeMixin<TBase extends ConstructorType<any>, FinalConstructor>(
    mixinFunction: (Base: TBase) => FinalConstructor
): (Base?: TBase) => FinalConstructor {
    return (Base?: TBase) => {
        // do other things with Base, this example is simplified
        return mixinFunction(Base ?? (Object as any));
    };
}

const MixinF = makeMixin(<TBase extends ConstructorType<any>>(Base: TBase) => {
  abstract class Mixin extends Base {
    foo(foo: MixinApplied, extra: boolean): MixinApplied | boolean { return extra }
  }
  return Mixin
})


const Mixin2F = makeMixin(<TBase extends ConstructorType<any>>(Base: TBase) => {
  abstract class Mixin2 extends Base {
    bar(foo: MixinApplied, extra: boolean): MixinApplied | boolean { return extra }
  }
  return Mixin2
})

class MixinApplied extends Mixin2F(MixinF()) {}
```


### 🙁 Actual behavior

The declaration for either `Mixin1F` or `Mixin2F` (unclear why it's not deterministic) errors with "'Mixin2F' implicitly has type 'any' because it does not have a type annotation and is referenced directly or indirectly in its own initializer." and the declaration for `MixinApplied` errors with "Type 'MixinApplied' recursively references itself as a base type.(2310)
'MixinApplied' is referenced directly or indirectly in its own base expression.(2506)"

I imagine I'm just running into some limits of the TypeScript compiler, but I can come up with at least 3 different scenarios that  end with the same final result but without TypeScript errors, and this is preventing perfectly fine Javascript from being written. Mixins and function invocations seem to exacerbate the problem of recursive / circular references in class definitions.

For example, calling `makeMixin` within the function (instead of wrapping the function) and immediately invoking it works ([TS playground](https://www.typescriptlang.org/play?#code/C4TwDgpgBAwg9gOwM7AE4FcDGw6oCrgQA8eAfFALxQCGARiqtdlAhAO5QAUAdL9agHMkALhoIQAbQC6ASkrk8AbgCwAKDUAzdAmwBLRFAC21ANYQAsroAeuhCQBC1JNAhXgEBABMksRAyw4+IRE1OKkADRQAGK21AA28MhoAbiknGpQmUbWtlHaeoiinI7OonglEHIU5DEI8Yn+2LhqMkUVAPxlFVU1sQl+yU2oUADeGVmoEMDoqAhcHV1OlfKj41kTUzNzhjkIeTrA+gjFS1Dt7VwA8rQAVhDMTmIgMjIqqusAvm8famqYA1BLDY9pQoA5Tq53F4fA1BoECJAQmE0gsoOUlj1Vu8oJNprMjKYLLtOCdSmjuisxtjMnQGExgFBMHEnD4gbYoJCPN4oBUsetMho4HBOIK4KI2QgAIJgMBxXQQTyRSGMUS0IVxCChVqA3bS2XyzxQAA+UDVcA1oVGOM2+OV1CgP2pDrW1rxcwl4w+MlJlTUjt+qn+SR1wIATFFQeDnBy3FyYQMMEMEcRQiBSCilp1yRjKeNcVsCWYJSSKotnJiqetaWh6YzmUhWbtQzGodzeZX+bR+CKheLdTK5QqlW4Vab1ZqENqJXrB4aTWaLXMRq6C3bnU7Het8-iJaHPd7un6A0yWSHbDODS242eEOHOBKopxrndsC9Rh8oAB6T8sODuUQAJIaFAAFQJ4iAAOQMkg6ADiAUDPvcDIABYQJMkTAMhug+Bo1C6HEOG4DQYG6BoGhoR4DKTE4iDHgCVigiMH6PNOA4Gm8VjcKKnBWBhGCVBx3BdqgPGRLhBECUAA)):
```ts
type ConstructorType<T> = abstract new (...args: any[]) => T;

function makeMixin<TBase extends ConstructorType<any>, FinalConstructor>(
    mixinFunction: (Base: TBase) => FinalConstructor
): (Base?: TBase) => FinalConstructor {
    return (Base?: TBase) => {
        return mixinFunction(Base ?? (Object as any));
    };
}

const MixinF = <TBase extends ConstructorType<any>>(Base?: TBase) => {
  return makeMixin((Base: TBase) => {
    abstract class Mixin extends Base {
      foo(foo: MixinApplied, extra: boolean): MixinApplied | boolean { return extra }
    }
    return Mixin
  })(Base)
}


const Mixin2F = <TBase extends ConstructorType<any>>(Base?: TBase) => {
  return makeMixin((Base: TBase) => {
    abstract class Mixin2 extends Base {
      bar(foo: MixinApplied, extra: boolean): MixinApplied | boolean { return extra }
    }
    return Mixin2
  })(Base)
}

class MixinApplied extends Mixin2F(MixinF(Object)) {} // note: If I don't supply Object here, this fails for a different reason

const x = {} as MixinApplied;
x.foo(x, true);
x.bar(x, false);
```

Removing the `makeMixin` call also works ([TS playground](https://www.typescriptlang.org/play?#code/C4TwDgpgBAwg9gOwM7AE4FcDGw6oCrgQA8eAfFALxQCGARiqtdlAhAO5QAUAdL9agHMkALhoIQAbQC6ASkrk8AbgCwAKDWZEKKAFkAlgA89CAGKUoJAELUk0CAeAQEAEySwtaLDnyEi1caSknNa2onghEHIU5ADealA09GhMwFCYADY2bvpGCFD2ji5uEVBxqgkJAGZwcJzVcKI5xgCCYGDpehDOADT5DoyitDXpEP4yjYYtbR1dUAA+UENwI-6lUKgQwOioeQWMUAC+8YfHG1s7upMIakfqd5rIqU0IAExmVFY2dg5Oru6PGGwuAIkD8ASCETCESisWOdAYKTSmSQ2SuLz6hT+JTKFUW-DqNQmuVa7U6PQxA0Ww1GCHGl2J0zJ8ypyxpazO212-WoJ3KvISHIuzxeNzUGmRqIZpNmBV+kuMb04zxMnAA8rQAFYQbAyOQxA5AA)):
```ts
type ConstructorType<T> = abstract new (...args: any[]) => T;

const MixinF = <TBase extends ConstructorType<any>>(Base: TBase) => {
  abstract class Mixin extends Base {
    foo(foo: MixinApplied, extra: boolean): MixinApplied | boolean { return extra }
  }
  return Mixin
}


const Mixin2F = <TBase extends ConstructorType<any>>(Base: TBase) => {
  abstract class Mixin2 extends Base {
    bar(foo: MixinApplied, extra: boolean): MixinApplied | boolean { return extra }
  }
  return Mixin2
}

class MixinApplied extends Mixin2F(MixinF(Object)) {}
```

And of course the non-mixin version works as well ([TS playground](https://www.typescriptlang.org/play?#code/IYIwzgLgTsDGEAJYBthjAgsgSwB7YDsEBTXCYggEwwHkQArY+BAbwFgAoBBAMwHs+ACn58AXFjyEAggAcZybMUoAaEmRjiQA5MWAEAlOJz4Cs+YsoIAPgi18de1gijEIAVyhFS0YAgC+nAEcnKCQMMwoaBjGhABMauRU0ZJE7Fy2wFDCAkYpZgpKqt4attq6Brkm+RbWpfblTi7ungkw-oGcnJHoElVyBZbeFNS9cax+QA)):
```ts
abstract class Mixin extends Object {
  foo(foo: MixinApplied, extra: boolean): MixinApplied | boolean { return extra }
}

abstract class Mixin2 extends Mixin {
  bar(foo: MixinApplied, extra: boolean): MixinApplied | boolean { return extra }
}

class MixinApplied extends Mixin2 {}
```

### 🙂 Expected behavior

It'd be great if the mixin + function invocation example worked as well as the others.

### Additional information about the issue

_No response_