[PR #331] [MERGED] Align Copilot persistence keys, fix mutable JSON defaults in UserProfile, and update docs #634

New issue

Closed

opened 2026-03-13 21:01:02 +03:00 by kerem · 0 comments

kerem commented

2026-03-13 21:01:02 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AJaySi/ALwrity/pull/331
Author: @AJaySi
Created: 2/9/2026
Status: ✅ Merged
Merged: 2/10/2026
Merged by: @AJaySi

Base: main ← Head: codex/fix-persistence-keys-mismatch-in-debug-helper-0olkiz

📝 Commits (2)

ac9cf3f Fix persistence keys and docs
3d1416b Merge branch 'main' into codex/fix-persistence-keys-mismatch-in-debug-helper-0olkiz

📊 Changes

4 files changed (+20 additions, -18 deletions)

View changed files

📝 backend/models/user_profiles.py (+8 -8)
📝 docs/ALwrity Researcher/RESEARCH_PERSONA_DATA_SOURCES.md (+1 -1)
📝 docs/ALwrity_vision.md (+1 -1)
📝 frontend/src/components/LinkedInWriter/utils/persistenceTest.ts (+10 -8)

📄 Description

Motivation

Ensure the Copilot persistence debug helper reads and clears the actual keys used by the persistence layer so debugging/clearing works correctly.
Prevent shared mutable defaults in the UserProfile SQLAlchemy model to avoid cross-instance state leakage.
Keep documentation accurate and clean by fixing a punctuation typo and updating a stale frontend file reference.

Description

Updated frontend/src/components/LinkedInWriter/utils/persistenceTest.ts to import and use the shared STORAGE_KEYS from enhancedPersistence and to iterate Object.values(STORAGE_KEYS) when clearing stored data.
Replaced mutable JSON defaults in backend/models/user_profiles.py (e.g., default={} / default=[]) with callable defaults like default=dict and default=list for all JSON columns.
Fixed the extra period in the FastAPI sentence in docs/ALwrity_vision.md and updated the frontend file reference in docs/ALwrity Researcher/RESEARCH_PERSONA_DATA_SOURCES.md to point to frontend/src/pages/ResearchDashboard/ResearchDashboard.tsx.

Testing

No automated tests were run for these changes.

Codex Task

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/AJaySi/ALwrity/pull/331 **Author:** [@AJaySi](https://github.com/AJaySi) **Created:** 2/9/2026 **Status:** ✅ Merged **Merged:** 2/10/2026 **Merged by:** [@AJaySi](https://github.com/AJaySi) **Base:** `main` ← **Head:** `codex/fix-persistence-keys-mismatch-in-debug-helper-0olkiz` --- ### 📝 Commits (2) - [`ac9cf3f`](https://github.com/AJaySi/ALwrity/commit/ac9cf3fd9bc11e93a0ec41842a38237a907215a0) Fix persistence keys and docs - [`3d1416b`](https://github.com/AJaySi/ALwrity/commit/3d1416b695648d88690941d7384201b992fc63a7) Merge branch 'main' into codex/fix-persistence-keys-mismatch-in-debug-helper-0olkiz ### 📊 Changes **4 files changed** (+20 additions, -18 deletions) <details> <summary>View changed files</summary> 📝 `backend/models/user_profiles.py` (+8 -8) 📝 `docs/ALwrity Researcher/RESEARCH_PERSONA_DATA_SOURCES.md` (+1 -1) 📝 `docs/ALwrity_vision.md` (+1 -1) 📝 `frontend/src/components/LinkedInWriter/utils/persistenceTest.ts` (+10 -8) </details> ### 📄 Description ### Motivation - Ensure the Copilot persistence debug helper reads and clears the actual keys used by the persistence layer so debugging/clearing works correctly. - Prevent shared mutable defaults in the `UserProfile` SQLAlchemy model to avoid cross-instance state leakage. - Keep documentation accurate and clean by fixing a punctuation typo and updating a stale frontend file reference. ### Description - Updated `frontend/src/components/LinkedInWriter/utils/persistenceTest.ts` to import and use the shared `STORAGE_KEYS` from `enhancedPersistence` and to iterate `Object.values(STORAGE_KEYS)` when clearing stored data. - Replaced mutable JSON defaults in `backend/models/user_profiles.py` (e.g., `default={}` / `default=[]`) with callable defaults like `default=dict` and `default=list` for all JSON columns. - Fixed the extra period in the FastAPI sentence in `docs/ALwrity_vision.md` and updated the frontend file reference in `docs/ALwrity Researcher/RESEARCH_PERSONA_DATA_SOURCES.md` to point to `frontend/src/pages/ResearchDashboard/ResearchDashboard.tsx`. ### Testing - No automated tests were run for these changes. ------ [Codex Task](https://chatgpt.com/codex/tasks/task_e_6989fceb20588328978015d9337872f0) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-13 21:01:02 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ALwrity#634

No description provided.

Rows
Columns