search

microsoft / tsyringe

Lightweight dependency injection container for JavaScript/TypeScript
MIT License
5.19k stars 172 forks source link

Bugfix for ESM5 autoInjectable. #234

Open Saneesh-B opened 10 months ago

Saneesh-B commented 10 months ago

Fix "Class constructor XXX cannot be invoked without 'new'" when in ESM5. This happens in Vue and React

Saneesh-B commented 10 months ago

@microsoft-github-policy-service agree company="Identy"

MeltingMosaic commented 10 months ago

This is kind of a big change to the basic autoInjectable functionality, and introduces reflection via Proxy - I wonder if we should have a new decorator for the Vue/React case?

Saneesh-B commented 10 months ago

@MeltingMosaic I wanted to make it generic to all frameworks. Im okay either way.

Saneesh-B commented 9 months ago

Ideas on how to proceed ?

risen228 commented 4 months ago

Should be working in my fork: tsyringe-neo

Saneesh-B commented 4 months ago

@risen228 , i have it working in my fork :). Just this is taking a whole lot of time to get some attention