Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[opentelemetry/auto-instrumentations-web] esm directory missing in release 0.34.0 #1799

Closed
adamlyka opened this issue Nov 15, 2023 · 1 comment · Fixed by #1848
Closed

[opentelemetry/auto-instrumentations-web] esm directory missing in release 0.34.0 #1799

adamlyka opened this issue Nov 15, 2023 · 1 comment · Fixed by #1848
Assignees
Labels
bug Something isn't working pkg:auto-instrumentations-web priority:p1 Bugs which cause problems in end-user applications such as crashes, data inconsistencies

Comments

@adamlyka
Copy link

When attempting to use the latest version of opentelemetry/auto-instrumentations-web my project fails to build with the following error:
Failed to resolve entry for package "@opentelemetry/auto-instrumentations-web". The package may have incorrect main/module/exports specified in its package.json.

I noticed the esm directory was missing and causing the failure:
https://github.com/open-telemetry/opentelemetry-js-contrib/tree/main/metapackages/auto-instrumentations-web/src

@JamieDanielson
Copy link
Member

I was looking into this and was able to repro using Vite, which sets ESNext as the transform target. I used one of the web example apps with Vite instead of webpack (tsc && vite build). It looks like the esm files were built but not included in "files" in the package.json, and esnext was not built at all. The individual instrumentations were fixed up with #1776 , but the same is needed for the metapackage.

I've opened #1848 to fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pkg:auto-instrumentations-web priority:p1 Bugs which cause problems in end-user applications such as crashes, data inconsistencies
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants