fix: optional access to Reflect.metadata

[sirenkovladd]

🔗 Linked issue

#145

❓ Type of change

replace babel-plugin-transform-typescript-metadata

• 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Replace all call Reflect.metadata with optional call (if not install reflect-metadata)

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[pi0]

Thanks for this PR i really appreciate it.

Before moving forward with adding a fork, have you considered to make an upstream PR directly to babel?

Also in the meantime, would be nice if we inline the forked version of patch in jiti codebase (same as other plugin forks) but better maintainability.

Adding tests to the fixtures would be also awesome!

[sirenkovladd]

I already did this (leonardfactory/babel-plugin-transform-typescript-metadata#70) over a month ago

the last activity in this repository was on March 4, 2022

[sirenkovladd]

We can move this repo to the UnJS organization

[pi0]

Moving to src/plugins/babel-plugin-transform-typescript-metadata.ts should be doable i guess. let me know if need help on this this.

[sirenkovladd]

Do you recommend moving all the code to jiti?

[pi0]

Source data only yes. (there are too many repos to maintain in unjs already and i hope we don't need constant update for this until your PR lands upstream -- hopefully)

[sirenkovladd]

One remark, there is a problem with pnpm, I suspect that it is install the types incorrectly
build works only with npm install and not pnpm install
do you have any idea?

[sirenkovladd]

Ok, I just hate pnpm lock, it's ok now

[sirenkovladd]

@pi0 what do you think about it now?

[pi0]

Thanks for the updates! Will check it soon locally 👍🏼

[sirenkovladd]

@pi0 hi, how's your check going?