[PR #330] [CLOSED] Align Copilot persistence keys, fix mutable JSON defaults in UserProfile, and update docs #297

New issue

Closed

opened 2026-03-02 23:35:00 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 23:35:00 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AJaySi/ALwrity/pull/330
Author: @AJaySi
Created: 2/9/2026
Status: ❌ Closed

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

📝 Commits (1)

223274c Fix persistence keys and docs

📊 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/330 **Author:** [@AJaySi](https://github.com/AJaySi) **Created:** 2/9/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `codex/fix-persistence-keys-mismatch-in-debug-helper-zu7ea1` --- ### 📝 Commits (1) - [`223274c`](https://github.com/AJaySi/ALwrity/commit/223274c949b199f17a3d8cb68ac7bdbc32c20c98) Fix persistence keys and docs ### 📊 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-02 23:35:00 +03:00

closed this issue
added the
pull-request
label

kerem referenced this issue

2026-03-13 20:53:48 +03:00

[GH-ISSUE #297] [FEATURE] OpenRouter Support for Custom Models Evaluation #550

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#297

No description provided.

Rows
Columns