Skip to content

Commit 219274d

Browse files
committed
fix: remove scope creep — revert auto-capture changes from PR CortexReach#517 (issues CortexReach#417/CortexReach#518 belong in PR CortexReach#549)
1 parent 5984a34 commit 219274d

3 files changed

Lines changed: 161 additions & 150 deletions

File tree

CHANGELOG.md

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,5 @@
11
# Changelog
22

3-
## Unreleased
4-
5-
### Fix: cumulative turn counting for auto-capture smart extraction (#417, PR #518)
6-
7-
**Bug**: With `extractMinMessages: 2` + `smartExtraction: true`, single-turn DM conversations always fell through to regex fallback, writing dirty data (`l0_abstract == text`, no LLM distillation).
8-
9-
**Root causes**:
10-
- `autoCaptureSeenTextCount` was overwritten per-event (always 1 for DM), never accumulating
11-
- `buildAutoCaptureConversationKeyFromIngress` returned `null` for DM (no `conversationId`), so `pendingIngressTexts` was never written
12-
13-
**Changes**:
14-
- **Cumulative counting**: `autoCaptureSeenTextCount` now accumulates across events instead of overwriting per-event
15-
- **DM key fallback**: `buildAutoCaptureConversationKeyFromIngress` falls back to `channelId` for DM, fixing `pendingIngressTexts` writes
16-
- **Smart extraction threshold**: now uses cumulative turn count (`currentCumulativeCount`) instead of per-event message count
17-
- **`extractMinMessages` cap**: `Math.min(config.extractMinMessages ?? 4, 100)` prevents misconfiguration
18-
- **MAX_MESSAGE_LENGTH guard**: 5000 char limit in `pendingIngressTexts` prevents OOM
19-
20-
**Note**: `extractMinMessages` semantics changed from "per-event message count" to "cumulative conversation turns". This is a bug fix since the old semantics were broken for DM; a changelog note is included for beta.11 users.
21-
22-
---
23-
243
## 1.1.0-beta.2 (Smart Memory Beta + Access Reinforcement)
254

265
This is a **beta** release published under the npm dist-tag **`beta`** (it does not affect the stable `latest` channel).

0 commit comments

Comments
 (0)