nicojs / typed-inject

Type safe dependency injection for TypeScript
Apache License 2.0
448 stars 23 forks source link

Improve type inference performance by simplifying child context type #66

Open mayorandrew opened 1 year ago

mayorandrew commented 1 year ago

This PR improves the TypeScript/IDE performance of long provider chains, as in this issue: https://github.com/nicojs/typed-inject/issues/50, by explicitly simplifying the ChildContext type.

It also fixes the "Type instantiation is excessively deep and possibly infinite" issue:

Without Simplify: Screen Shot 2023-08-04 at 11 27 10

With Simplify: Screen Shot 2023-08-04 at 11 28 52

nirweiner2 commented 11 months ago

Hi guys, Is it getting merged anytime soon? Thanks.