Skip to content

[replay 7239] fix(memory): characters joining a merged group chat no longer hide earlier memories - #431

Open
Promansis wants to merge 2 commits into
replay-base/7239-cleanfrom
replay/7239-gp3
Open

Promansis wants to merge 2 commits into
replay-base/7239-cleanfrom
replay/7239-gp3

Conversation

@Promansis

Copy link
Copy Markdown
Owner

Dottore replay shadow (fork only) of upstream PR 7239 at accf679, the first head CodeRabbit reviewed.

Original title: fix(memory): characters joining a merged group chat no longer hide earlier memories

SpicyMarinara and others added 2 commits October 7, 2026 12:54
…rlier memories

In merged Roleplay group chats the recall audience is every chat
character, and a scene was recalled only when all of them had been there.
After a character joined, no earlier scene qualified. Character-scoped
Chat Summaries were also rendered as whichever character's random id
sorted first, which could be the newcomer, so they rendered empty.

A merged reply can voice any character, so recall now keeps what any
present character remembers and marks what not everyone knows:
"Scene summary (known only to Alice):" for scenes and
"[Known only to Alice: ...]" for private summary sections. Memories no
present character has, including narrator-only scenes, stay out.
Individual group chats, single-reader recall, impersonation and narrator
rules are unchanged.

Closes Pasta-Devs#7237

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Keeps every CHANGELOG entry from both sides.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

🎭 Dottore Review — Concluded

Tip

Review posted. The specimen has left the observation table.

  • Commit: accf679 - Merge origin/staging into fix/merged-group-newcomer-recall

@github-actions

Copy link
Copy Markdown

🎭 Dottore Review

Dottore Verdict: Viable, With Notes

Warning

VIABLE, WITH NOTES
No actionable defects were isolated, but non-blocking notes remain.

Findings Nitpicks Controls Reviewed Head Mode
0 0 1 unknown accf6790 full

Note

Mode: full
Head: accf679 - Merge origin/staging into fix/merged-group-newcomer-recall
Base: c2f214a - Merge pull request Pasta-Devs#7236 from Pasta-Devs/feat/advanced-memory-fix-all

🧭 Specimen Summary

  • DOTTORE_LAB_STAGE=recheck: the checker judged recorded candidates; nothing was published.

🔎 Isolated Defects

Tip

No actionable defects isolated.

🧹 Minor Imperfections

  • None recorded.

✅ Control Checks

Status Type Check Detail
⚠️ UNKNOWN Review Limitation Lab Recheck Only DOTTORE_LAB_STAGE=recheck: the checker judged recorded candidates; nothing was published.

🧪 Observations

  • No model pass ran; the specimen remained unexamined.

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.

2 participants