Skip to content

refactor: split localStorageService into domain modules#118

Merged
AdamJ merged 1 commit into
mainfrom
claude/issue-104-20260510-2040
May 10, 2026
Merged

refactor: split localStorageService into domain modules#118
AdamJ merged 1 commit into
mainfrom
claude/issue-104-20260510-2040

Conversation

@AdamJ
Copy link
Copy Markdown
Owner

@AdamJ AdamJ commented May 10, 2026

Converts the monolithic localStorageService.ts into a directory of focused, single-responsibility modules under src/services/localStorageService/.

Closes #104

Generated with Claude Code

@AdamJ AdamJ added enhancement New feature or request performance For performance related issues labels May 10, 2026
@AdamJ AdamJ force-pushed the claude/issue-104-20260510-2040 branch from 354ade0 to a7c61b8 Compare May 10, 2026 23:11
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 10, 2026

Deploying timetrackerpro with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3f8394b
Status: ✅  Deploy successful!
Preview URL: https://ce2f5b8b.timetrackerpro.pages.dev
Branch Preview URL: https://claude-issue-104-20260510-20.timetrackerpro.pages.dev

View logs

Converts the monolithic localStorageService.ts into a directory of
focused, single-responsibility modules under
src/services/localStorageService/:

- constants.ts   — STORAGE_KEYS and SCHEMA_VERSION
- utils.ts       — readVersioned<T>, hydrateTask, hydrateDay helpers
- currentDay.ts  — saveCurrentDay / getCurrentDay
- archivedDays.ts — CRUD for archived day records
- projects.ts    — saveProjects / getProjects
- categories.ts  — saveCategories / getCategories
- todos.ts       — saveTodos / getTodos
- index.ts       — LocalStorageService class (thin delegation) + re-exports

All existing consumers import from @/services/localStorageService and
resolve to index.ts automatically — no other files required changes.

Closes #104
@AdamJ AdamJ force-pushed the claude/issue-104-20260510-2040 branch from a7c61b8 to 3f8394b Compare May 10, 2026 23:15
@AdamJ AdamJ merged commit c0b5048 into main May 10, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request performance For performance related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

localStorageService rewrite

1 participant