Skip to content

Personalized memories - #551

Merged
Yavaren merged 9 commits into
mainfrom
personalized-memories
Sep 18, 2026
Merged

Yavaren merged 9 commits into
mainfrom
personalized-memories

Conversation

@Yavaren

@Yavaren Yavaren commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Add chat personalization and optional profile export

Adds a Personalization page in Settings where users can enter their preferred name, occupation, and information about themselves.

Users can choose a chat style: Default, Concise, Friendly, Professional, or Detailed. They can also control emoji usage with None, Default, or More, and choose Minimal, Default, or Structured Markdown formatting.

Personalization applies to new chats. Each chat keeps the personalization it started with, so changing these settings does not change existing conversations.

Chat exports exclude personalization by default. An “Include personalization” checkbox in the save dialog lets users include their saved profile and response preferences. The checkbox starts unchecked for every export. If included, personalization is restored when the chat is imported.

When exporting, users have an option to include/exclude their personalization
image

Validated with 98 passing tests and checks of the save dialog, including both checkbox states and the default reset behavior.

image image

@Yavaren
Yavaren force-pushed the personalized-memories branch from de0b450 to 5e474b5 Compare September 15, 2026 08:35
@Yavaren
Yavaren marked this pull request as draft September 15, 2026 15:47
@Yavaren
Yavaren marked this pull request as ready for review September 16, 2026 10:15
@lucasnewman

Copy link
Copy Markdown
Collaborator

Overall this looks good. I think we can go ahead and remove the "Default —" prefix from these, it doesn't add anything.

Screenshot 2026-09-16 at 8 51 26 AM

@Yavaren

Yavaren commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator Author

@lucasnewman I removed the prefixes completely, I agree, they don't make much sense
Screenshot 2026-09-18 at 12 06 45 PM
Screenshot 2026-09-18 at 12 06 52 PM

@lucasnewman lucasnewman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

case .default:
return ""
case .concise:
instructions = """

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hm, I think this is a fair pr, but i hope this doesn't crush small models, can u test to verify that given lengthy responses + this hard coded prompt, model doesn't degrade or loose focus.

ideally I don't for see a problem with this, but just worth mentioning :)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Smaller models don't really follow instructions. I tested on 0.5 and 0.8 B qwen models. Also tried gemma-3-270m and the same problem. The model starts telling me about itself when I ask it who am I.

The only exposed issue is the emoji prompt, it starts working only when you ask the model in the chat. I will update the prompt and merge the PR, it is ready.

@Yavaren
Yavaren merged commit 4e47295 into main Sep 18, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants