refactor: move vue-data-ui css in nuxt.config

[MatteoGabriele]

Summary by CodeRabbit

• Refactor
  • Optimized stylesheet loading across chart components for improved performance.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5d748efe-339e-45b5-8a27-7a417d208d45

📥 Commits

Reviewing files that changed from the base of the PR and between 7812ea1 and ba061fc.

📒 Files selected for processing (8)

• app/components/Chart/AccountEventsTimeline.vue
• app/components/Chart/GlobalEventsBreakdown.vue
• app/components/Chart/GlobalEventsEvolution.vue
• app/components/Chart/GlobalEventsHeatmap.vue
• app/components/Chart/GlobalEventsSplitSparklines.vue
• app/components/Chart/GlobalScatter.vue
• app/components/Chart/GlobalScoreGauge.vue
• nuxt.config.ts

💤 Files with no reviewable changes (7)

• app/components/Chart/GlobalEventsEvolution.vue
• app/components/Chart/GlobalScatter.vue
• app/components/Chart/GlobalScoreGauge.vue
• app/components/Chart/GlobalEventsSplitSparklines.vue
• app/components/Chart/GlobalEventsHeatmap.vue
• app/components/Chart/AccountEventsTimeline.vue
• app/components/Chart/GlobalEventsBreakdown.vue

---

📝 Walkthrough

Walkthrough

Seven Vue chart components remove distributed vue-data-ui/style.css imports from their script blocks, and nuxt.config.ts adds that stylesheet to the global CSS array for centralized loading. Vite's dependency pre-bundling config is also updated to remove vue-data-ui/vue-ui-icon.

Changes

Style Loading Centralization

Layer / File(s)	Summary
Remove component-level CSS imports app/components/Chart/AccountEventsTimeline.vue, app/components/Chart/GlobalEventsBreakdown.vue, app/components/Chart/GlobalEventsEvolution.vue, app/components/Chart/GlobalEventsHeatmap.vue, app/components/Chart/GlobalEventsSplitSparklines.vue, app/components/Chart/GlobalScatter.vue, app/components/Chart/GlobalScoreGauge.vue	All seven chart components remove the vue-data-ui/style.css side-effect import, leaving their component logic and rendering logic unchanged.
Centralize stylesheet loading in Nuxt config nuxt.config.ts	Adds vue-data-ui/style.css to the global css array alongside ~/assets/main.css; also removes vue-data-ui/vue-ui-icon from vite.optimizeDeps.include.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• MatteoGabriele/agentscan#89: Earlier introduction of AccountEventsTimeline.vue chart component that is now refactored by this PR's CSS import removal.
• MatteoGabriele/agentscan#96: Adds vue-data-ui/style.css imports to chart components, directly opposing this PR's consolidation strategy for the same stylesheet.
• MatteoGabriele/agentscan#88: Introduces or modifies GlobalScatter.vue, which is refactored by this PR's removal of the component-level CSS import.

Poem

🐰 Stylesheets scattered far and wide,
Now bundled at the config's side,
Seven charts let go their weight,
One global load—so clean, so great! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: moving vue-data-ui CSS imports from individual Vue components to the central nuxt.config.ts file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this 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 refactor/move-css-in-nuxt-config

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.

---

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

[netlify]

❌ Deploy Preview for agentscan failed.

Name	Link
🔨 Latest commit	ba061fc
🔍 Latest deploy log	https://app.netlify.com/projects/agentscan/deploys/6a0ca36b7800dd0008da1fc2