refactor: use built-in vue-data-ui composable

[graphieros]

🔗 Linked issue

Resolves#2950

📚 Description

Replace the useChartTooltipPosition composable with the built-in useTooltipPosition from vue-data-ui.
No impact on functionality.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
npmx.dev	Ready	Preview, Comment	Jun 23, 2026 1:58pm

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored		Jun 23, 2026 1:58pm
npmx-lunaria	Ignored		Jun 23, 2026 1:58pm

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c3e07462-a83f-46ac-9678-8239cab702c9

📥 Commits

Reviewing files that changed from the base of the PR and between a11045f and 4f98377.

📒 Files selected for processing (4)

• app/components/Package/TimelineChart.vue
• app/components/Package/TrendsChart.vue
• app/composables/useChartTooltipPosition.ts
• test/unit/app/composables/use-chart-tooltip-position.spec.ts

💤 Files with no reviewable changes (2)

• test/unit/app/composables/use-chart-tooltip-position.spec.ts
• app/composables/useChartTooltipPosition.ts

---

📝 Walkthrough

Summary by CodeRabbit

• Refactor
  • Streamlined chart tooltip positioning logic across timeline and trends charts through internal composable consolidation.

Walkthrough

The custom useChartTooltipPosition composable and its unit test file are deleted. Both TimelineChart.vue and TrendsChart.vue replace the import and usage of that composable with useTooltipPosition from vue-data-ui/composables, keeping the same call signature useTooltipPosition(chartRef).

Changes

Tooltip position composable migration

Layer / File(s)	Summary
Replace composable in chart components app/components/Package/TimelineChart.vue, app/components/Package/TrendsChart.vue	Both components remove the import of useChartTooltipPosition and add useTooltipPosition from vue-data-ui/composables; tooltipPosition is initialised by calling useTooltipPosition(chartRef) in each component.
Delete local composable and tests app/composables/useChartTooltipPosition.ts, test/unit/app/composables/use-chart-tooltip-position.spec.ts	The now-redundant local composable (mouse-position-based tooltip placement logic) and all 122 lines of its Vitest unit tests are removed.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely summarises the main change: refactoring to use a built-in vue-data-ui composable instead of a custom one.
Description check	✅ Passed	The description is directly related to the changeset, explaining the purpose of replacing the custom composable with the built-in one and noting no functional impact.
Linked Issues check	✅ Passed	The PR fully meets all objectives from issue #2950: the custom useChartTooltipPosition composable and tests are removed, and both TimelineChart.vue and TrendsChart.vue are updated to use the built-in useTooltipPosition composable with correct import statements.
Out of Scope Changes check	✅ Passed	All changes are directly scoped to the refactoring objective; no unrelated modifications are present in the pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch 2950-use-tooltip-position

---

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.}