ui/src/composables/useDataPoint.ts (1)

108-114: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Remove the ref mutation inside the datasetsProcessed computed getter.

Mutating datasets.value from within the getter that reads it is a Vue anti-pattern (computed getters are expected to be side-effect free). It's also now vestigial: classifyPayload (line 22/28/33) already normalizes every load path into { mode: 'full', datasets: [...] } / { mode: 'catalog', entries: [...] }, and datasets is typed shallowRef<Dataset[]>, so every writer (payload.entries.map(...) / payload.datasets.map(...)) always assigns an array. The !Array.isArray branch is unreachable given the current contract, yet it still triggers an extra reactive write on every evaluation path that would hit it.

♻️ Proposed fix: make the getter pure

 const datasetsProcessed = computed<Dataset[]>(() => {
-  if (!Array.isArray(datasets.value)) {
-    datasets.value = [datasets.value]
-  }
-
-  return datasets.value
+  return Array.isArray(datasets.value) ? datasets.value : [datasets.value]
 })

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@ui/src/composables/useDataPoint.ts` around lines 108 - 114, Remove the
!Array.isArray branch and its assignment from the datasetsProcessed computed
getter. Return datasets.value directly, keeping the getter pure and relying on
the existing array normalization performed by classifyPayload and the typed
writers.

ui/src/views/Dashboard.vue (1)

1-1: 📐 Maintainability & Code Quality | 🔵 Trivial

Run task build:ui before building the CLI or running Go tests.

As per coding guidelines, please ensure you run task build:ui after this ui/ change before running Go tests or CLI builds.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@ui/src/views/Dashboard.vue` at line 1, Run task build:ui after the
Dashboard.vue change and before building the CLI or running Go tests; no source
changes are required.

Source: Coding guidelines