fix: use current SDK version in telemetry meter

[aaguiarz]

Summary

Replaces the hardcoded meter version with the SDK version constant.

Changes

• Added regression test asserting metrics.getMeter receives SdkConstants.SdkVersion
• Updated MetricRecorder.meter() to use SdkConstants.SdkVersion

Potential Breaking Change

• Telemetry meter version tag changes from 0.6.3 to the actual SDK version.
• Existing dashboards/alerts/queries filtering by the previous version value may need to be updated.

Fixes #334

[coderabbitai]

Warning

Rate limit exceeded

@aaguiarz has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 23 minutes and 55 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/issue-334-meter-version

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[dosubot]

Related Documentation

Checked 8 published document(s) in 1 knowledge base(s). No updates required.

^{How did I do? Any feedback?}  

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.68%. Comparing base (a7090e2) to head (3e626cb).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #335   +/-   ##
=======================================
  Coverage   89.67%   89.68%           
=======================================
  Files          25       25           
  Lines        1492     1493    +1     
  Branches      279      279           
=======================================
+ Hits         1338     1339    +1     
  Misses         94       94           
  Partials       60       60           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull request overview

This PR fixes a bug where the OpenTelemetry meter was being registered with a hardcoded, stale version ("0.6.3") instead of the current SDK version. The fix replaces the hardcoded version with the centralized SdkConstants.SdkVersion constant, which currently holds version "0.9.2". This ensures telemetry metadata accurately reflects the SDK version being used, enabling proper version-based filtering and analysis.

Changes:

• Updated MetricRecorder.meter() to use SdkConstants.SdkVersion instead of hardcoded "0.6.3"
• Added regression test to verify meter is registered with the correct SDK version

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
tests/telemetry/metrics.test.ts	Added imports for SdkConstants and metrics module, and new test case to verify meter registration uses current SDK version
telemetry/metrics.ts	Imported SdkConstants and replaced hardcoded version string with SdkConstants.SdkVersion in getMeter call

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.