Skip to content

feat: skip assistant boilerplate as recall-primary episodes - #105

Merged
cursor[bot] merged 1 commit into
devfrom
pr/assistant-extract-policy-a6c7
Aug 13, 2026
Merged

feat: skip assistant boilerplate as recall-primary episodes#105
cursor[bot] merged 1 commit into
devfrom
pr/assistant-extract-policy-a6c7

Conversation

@sdntsng

@sdntsng sdntsng commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Wave D: assistant conversation_episode rows drown user facts (gold “4 weeks” vs congratulations “3 weeks”). LME single-session-assistant is 0/2, but the indicated fix is not “extract more assistant chit-chat” — it is to stop persisting boilerplate as recall-primary episodes while still keeping assistant-stated facts.

Changes

  • Deterministic extract: skip phatic assistant turns; store non-phatic assistant utterances as assistant_stated facts (not PrimitiveEpisode).
  • Provider extract: same filter after merge so async worker cannot reintroduce congratulations episodes.
  • User free-dialogue episodes unchanged.
  • Tests: congratulations dropped; FCC/distillation assistant fact retained without conversation_episode.

Test plan

  • go test ./internal/memory/
  • OpMem 13/13 + marketing after merge (extract filter is conversation-path)
  • Do not treat as LME quality lift until a dedicated pin

Linear / tracking

Wave D inverted PR9 after #100.

Risk / follow-ups

  • Evidence plane still captures assistant actor_role.
  • Phatic cue list is product-general (congratulations / you're welcome / …), not LoCoMo/LME-named.
  • No SOTA claim. dev only; do not merge to main.
Open in Web Open in Cursor 

Wave D: congratulations/listicle assistant turns drown user facts.
Keep evidence-plane actor_role and still extract assistant-stated
facts; do not persist phatic assistant conversation_episode rows.

Co-authored-by: Siddhant Singh <sdntsng@users.noreply.github.com>
@sdntsng
sdntsng marked this pull request as ready for review August 13, 2026 12:48
@cursor
cursor Bot merged commit a7a5184 into dev Aug 13, 2026
2 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.

1 participant