Skip to content

feat: unify UI preference persistence (DB source of truth)#37

Merged
clFaster merged 5 commits intomainfrom
feature/unify-ui-preference-persistence
Apr 12, 2026
Merged

feat: unify UI preference persistence (DB source of truth)#37
clFaster merged 5 commits intomainfrom
feature/unify-ui-preference-persistence

Conversation

@clFaster
Copy link
Copy Markdown
Owner

@clFaster clFaster commented Apr 12, 2026

Summary #23

  • add a shared useUiPreferences hook to centralize UI preference state and persistence
  • switch sidebar collapsed, hide export row, and show timestamps preferences to DB-backed settings
  • remove duplicated localStorage writes for preferences now persisted in DB
  • add migration fallback from legacy localStorage values when DB values are missing
  • extend system e2e coverage with sidebar collapse persistence across app restart

Verification

  • pnpm format
  • pnpm lint
  • pnpm test:system

Notes

  • CodeScene MCP analysis could not be executed in this environment because no access token is configured.

codescene-delta-analysis[bot]

This comment was marked as outdated.

codescene-delta-analysis[bot]

This comment was marked as outdated.

codescene-delta-analysis[bot]

This comment was marked as outdated.

codescene-delta-analysis[bot]

This comment was marked as outdated.

codescene-delta-analysis[bot]

This comment was marked as outdated.

codescene-delta-analysis[bot]

This comment was marked as outdated.

@clFaster clFaster force-pushed the feature/unify-ui-preference-persistence branch from a84aa90 to ce8edbb Compare April 12, 2026 13:05
@clFaster clFaster enabled auto-merge April 12, 2026 13:06
@clFaster clFaster merged commit 491c74a into main Apr 12, 2026
5 checks passed
@clFaster clFaster deleted the feature/unify-ui-preference-persistence branch April 12, 2026 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant