Repository navigation
refactor: convert the shared pinia store to a setup store (CM-1564) - #5003
Conversation
Refs linuxfoundation/lfx-cdp-insights-ops#264 Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Refs linuxfoundation/lfx-cdp-insights-ops#264 Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Refs linuxfoundation/lfx-cdp-insights-ops#264 Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
PR SummaryLow Risk Overview The modal payload is now modeled as Reviewed by Cursor Bugbot for commit 52992d7. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
🟢 Approval recommended
Static review supports consumer compatibility and found no blocking issues; typechecking was not rerun.
0 open findings
What changed in this PR
Consolidates the shared Pinia store into a setup store while preserving its existing API for all nine consumers.
Changes:
- Combines state and actions in one file and removes obsolete modules.
- Adds typed report payloads, including project affiliations.
- Lowers the typecheck error baseline from 31 to 30.
| File | Description |
|---|---|
| frontend/typecheck-baseline.json | Updates the expected error count to 30. |
| frontend/src/shared/pinia/shared.store.ts | Implements the setup store and typed payload. |
| frontend/src/shared/pinia/shared.state.ts | Removes the separate state module. |
| frontend/src/shared/pinia/shared.getters.ts | Removes the empty getters module. |
| frontend/src/shared/pinia/shared.actions.ts | Removes actions now implemented in the store. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Summary
Converts the shared Pinia store (id
shared,reportDataModalandsetReportDataModal) to a one-file setup store. The nine consumers are untouched and keep working.shared.actions.ts,shared.getters.tsandshared.state.tsare deleted. Type errors onmaindrop from 31 to 30, sinceshared.actions.tscarried the last error CM-1536 deferred.Closes linuxfoundation/lfx-cdp-insights-ops#264