Skip to content

Conversation

@joaquin-diaz
Copy link
Contributor

@joaquin-diaz joaquin-diaz commented Oct 24, 2025

Goal

Add the first iteration of the user action instrumentation only covering the mousedown event based on the current semantic conventions PR

Extras

Vitest

Simple vitest setup for running unit tests using jsdom

Initial monorepo setup

Preparing for a future turbopack integration, I set up some packages for the upcoming monorepo setup.

  • instrumentation-user-action: package where the new instrumentation lives
  • test-utils: a private shared package where test utilities can live
  • web-utils: a private shared package where cross instrumentation web utils can live

Each package has these set of scripts:

  • test: vitest run
  • test:watch: vitest
  • test:coverage: vitest --coverage

Once we set up a monorepo tool (possible turbopack) we can easily run these commands from root

TODO: make sure package can build

What's not included

  • CI/CD: still not hooked up, will wait until we have turbopack
  • Release/publishing: not yet; we need to review the release strategy with the sig

@joaquin-diaz joaquin-diaz requested a review from a team as a code owner October 24, 2025 19:33
@joaquin-diaz joaquin-diaz force-pushed the joaquin-diaz/feat/add-basic-user-action-instrumentation branch from ff62925 to fb3b29e Compare October 29, 2025 13:40
@joaquin-diaz joaquin-diaz force-pushed the joaquin-diaz/feat/add-basic-user-action-instrumentation branch from fb3b29e to e888059 Compare October 29, 2025 18:42
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is an existing web-common package. It might be confusing to have two similar packages published in npm. I suggest that we either move web-common to this repo, or add these utils there for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants