nodejs / admin

Administrative space for policies of the TSC
148 stars 129 forks source link

Transfer elastic/require-in-the-middle to the org #890

Open trentm opened 1 week ago

trentm commented 1 week ago

Following on import-in-the-middle (IITM) being transferred to the nodejs org and discussion in today's Diagnostics WG meeting, this is a request to also move require-in-the-middle (RITM) from the Elastic org to the Node.js org. As with import-in-the-middle, this would be governed by the TSC, but delegated to the Diagnostics WG. The expectation is that there would be significant overlap in the maintainers of IITM and RITM.

require-in-the-middle is a fairly mature library for intercepting CommonJS module loading via require(...). It is widely used in the Observability/APM space -- OpenTelemetry, Elastic, Datadog (though I think they currently have a small fork), New Relic. Reasoning for wanting to donate this to a vendor-neutral org are the same as described for IITM.

/cc @nodejs/tsc @nodejs/diagnostics

Others, please indicate if you'd be interested in maintaining RITM.

Qard commented 1 week ago

SGTM. 👍🏻

legendecas commented 1 week ago

+1

AbhiPrasad commented 4 days ago

I would be interested in helping maintain! Aside from the open issues are there any areas that require-in-the-middle is looking to improve/refactor in the near future?

watson commented 4 days ago

Thanks for opening this issue @trentm. I'd love to help maintain the module under the Node.js org

pichlermarc commented 4 days ago

I'm also happy to help out with this one. :slightly_smiling_face:

timfish commented 1 day ago

I would be interested in helping maintain too!