Skip to content

fix: Enable deduplication for trailing extraction indexing#173

Open
abrookins wants to merge 1 commit intomainfrom
fix/trailing-extraction-dedup
Open

fix: Enable deduplication for trailing extraction indexing#173
abrookins wants to merge 1 commit intomainfrom
fix/trailing-extraction-dedup

Conversation

@abrookins
Copy link
Collaborator

@abrookins abrookins commented Feb 28, 2026

Summary

  • pass deduplicate=True when delayed extraction indexes memories
  • preserve existing delayed extraction behavior while preventing duplicate long-term entries
  • assert the deduplication flag in test_extraction_logic_fix

Note

Medium Risk
Enables deduplication in the delayed/trailing extraction indexing path, which can change what long-term memory records are persisted (skip/merge/delete duplicates). Impact is limited to trailing extraction but touches data integrity behavior.

Overview
Trailing-edge (delayed) extraction now indexes with deduplication enabled. run_delayed_extraction passes deduplicate=True into index_long_term_memories, preventing duplicate long-term entries when delayed extraction runs.

Tests are updated to assert the deduplicate flag is set on the indexing call during delayed extraction.

Written by Cursor Bugbot for commit c0dcc47. This will update automatically on new commits. Configure here.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

@jit-ci
Copy link

jit-ci bot commented Feb 28, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@abrookins abrookins changed the title Enable deduplication for trailing extraction indexing fix: Enable deduplication for trailing extraction indexing Feb 28, 2026
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