fix(profile): do not delete stored reports on profile save

[cursor]

What changed

• Profile Save always called POST /api/profile/invalidate-cache, which deleted comprehensiveMysticalProfiles/{uid} — every stored natal chart and catalog reading.
• That wipe did not reset mysticalProfileGenerated or allReportsReady. After a non-hash save (for example a birth-time note), Generate treated the profile as already complete and catalog resume never ran, so the readings stayed gone.
• Invalidate-cache now only clears the in-memory divination cache. Stored reports persist until the user clicks Generate and the pipeline rewrites them for a new profile hash. Admin workspace clear is unchanged.

Why this change is safe

• Root cause: handleSave on /profile always hits invalidate-cache after a successful client write; the route used Admin deleteDocument on the comprehensive profile.
• Scope is one API route plus a regression test. Generate, catalog fill, and the intentional admin wipe path are untouched.
• Distinct from open Fix stale tool reports after birth-data regeneration #53 (stale display after hash-change Generate), fix(security): keep profile saves working after Generate and billing #70 (save rejected by hasOnly), and fix(generation): refill catalog after birth-data hash change #69/fix(generation): do not advance profile hash without natal regen #71 (hash/catalog persist).

Verification

• ESLint on touched files (pnpm exec eslint app/api/profile/invalidate-cache/route.ts tests/integration/invalidate-cache.test.ts)
• pnpm test (74 suites / 396 tests)
• pnpm run lint:security
• Playwright smoke (auth-gated Profile Save; not run in this environment)
• Note: pnpm run security still fails on pre-existing high/critical advisories on main (next, sharp, js-yaml); not introduced here.

Regression prevention

• Added tests/integration/invalidate-cache.test.ts: authenticated call clears in-memory cache and must not call deleteDocument.
• 401 paths still reject missing/invalid tokens without touching storage.

Release checklist (solo-friendly)

• CI is green:
  • CI / Lint + Jest
  • CI / Playwright smoke
• Ready to merge to main after review.
• Post-merge: generate a full report, save a birth-time note (or re-save unchanged details), open Tarot/Vedic — stored readings must still be there.

  

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
future-seer	Ready	Preview	Sep 10, 2026 11:13am UTC

[AndyOliverR]

Superseded by #78 — unique fix cherry-picked onto current main (profile save no longer deletes stored reports).