[TituxMetal]

Add a console.warn in storage.ts when a stored version is older than the current version and no migrate function was provided, making the silent data-wipe path visible during development.

Add a comment above the createVersionedStorage call in coachStore.ts reminding future developers that bumping version without a migrate function discards all stored progress.

Closes #19

Generated with Claude Code

[Copilot]

Pull request overview

This PR makes the destructive “version bump without migrate” path in the web app’s versioned localStorage helper more visible, so future schema bumps don’t silently reset persisted state (notably Coach progress).

Changes:

• Add a warning when stored data is older than the current version and no migrate function is provided.
• Add an inline comment at Coach progress persistence reminding that bumping the version without migrate discards stored progress.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
apps/web/src/lib/storage.ts	Emits a warning on the destructive fallback path when version is bumped without migrate.
apps/web/src/features/coach/stores/coachStore.ts	Documents the destructive reset footgun next to Coach progress persistence configuration.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[TituxMetal]

@claude Please address the Copilot review comment on this PR:

In apps/web/src/lib/storage.ts line 66: gate the console.warn behind import.meta.env.DEV so it only runs in development builds and doesn't pollute the production console.

Follow project conventions: no semicolons, arrow functions only, named exports, bun run format:check + bun run lint:check + bun run typecheck must pass.

[claude]

Claude finished @TituxMetal's task in 2m 12s —— View job

---

Addressing Copilot review comment

• Read apps/web/src/lib/storage.ts to understand current code
• Gate console.warn behind import.meta.env.DEV
• Run bun run format:check, lint:check, typecheck — all green
• Commit and push — 170 tests pass, build succeeds

Change made in apps/web/src/lib/storage.ts:

- if (parsed.version < version && !migrate) {
+ if (parsed.version < version && !migrate && import.meta.env.DEV) {

The console.warn now only fires in Vite development builds (import.meta.env.DEV). In production Vite replaces this with false, so the branch is dead-code-eliminated — zero console noise for end users while the footgun warning still shows up during development.
·

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.