fix: auto-capture smart extraction — issue #417 full resolution (supersedes #518, #534) #549

Open

jlin53882 wants to merge 19 commits intoCortexReach:masterfrom

jlin53882:fix/issue-417-mustfixes

Commits on Apr 8, 2026

fix: auto-capture cumulative turn counting for smart extraction (issue #417)

James
authored and
jlin53882
committed
fix: re-apply all 7 fixes for issue #417 + add cumulative turn counting test + changelog

James
authored and
jlin53882
committed
docs: update changelog - add test file reference and improve breaking change label

James
authored and
jlin53882
committed
fix: Phase 1 - createMockApi accepts pluginConfigOverrides param + remove dead isExplicitRememberCommand guard (PR #518 review fixes)
jlin53882
committed
fix: resolve all Must Fix items from PR #534 review (issue #417)
jlin53882
committed
fix: move currentCumulativeCount reset inside success block (Fix #9)
jlin53882
committed
fix: add try-catch around extractAndPersist to prevent hook crash on extraction failure (Fix #10)
jlin53882
committed
fix: clear pendingIngressTexts in catch block on extraction failure (Fix #10 extended)
jlin53882
committed
fix: add conversationKey guard to Fix #8 + restore test comment
jlin53882
committed
fix: Must Fix 1/2/5 from PR #549 review - counter reset always, newTexts counting, Fix#8 assertion
jlin53882
committed
fix: Must Fix 1 revised - reset counter to previousSeenCount on all-dedup (reviewer suggestion)
jlin53882
committed
fix: revert Must Fix #2 (eligibleTexts.length counting restored) - preserves extractMinMessages semantics
jlin53882
committed
fix: correct test expectation - collected 1 not 2 text(s) after counter formula revert (e5b5e5b)
jlin53882
committed
fix: replace throw in hook with safe return (Fix-Must5)
jlin53882
committed
fix: remove unreachable conversationKey guard (Claude Code review)
jlin53882
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: auto-capture smart extraction — issue #417 full resolution (supersedes #518, #534) #549

fix: auto-capture smart extraction — issue #417 full resolution (supersedes #518, #534) #549
jlin53882 wants to merge 19 commits intoCortexReach:masterfrom
jlin53882:fix/issue-417-mustfixes

Commits on Apr 8, 2026

fix: auto-capture cumulative turn counting for smart extraction (issue #417)

fix: re-apply all 7 fixes for issue #417 + add cumulative turn counting test + changelog

docs: update changelog - add test file reference and improve breaking change label

fix: Phase 1 - createMockApi accepts pluginConfigOverrides param + remove dead isExplicitRememberCommand guard (PR #518 review fixes)

fix: resolve all Must Fix items from PR #534 review (issue #417)

fix: move currentCumulativeCount reset inside success block (Fix #9)

fix: add try-catch around extractAndPersist to prevent hook crash on extraction failure (Fix #10)

fix: clear pendingIngressTexts in catch block on extraction failure (Fix #10 extended)

fix: add conversationKey guard to Fix #8 + restore test comment

fix: Must Fix 1/2/5 from PR #549 review - counter reset always, newTexts counting, Fix#8 assertion

fix: Must Fix 1 revised - reset counter to previousSeenCount on all-dedup (reviewer suggestion)

fix: revert Must Fix #2 (eligibleTexts.length counting restored) - preserves extractMinMessages semantics

fix: correct test expectation - collected 1 not 2 text(s) after counter formula revert (e5b5e5b)

fix: replace throw in hook with safe return (Fix-Must5)

fix: remove unreachable conversationKey guard (Claude Code review)

Commits on Apr 9, 2026

fix(issue-417): skip regex fallback when all candidates skipped with no boundary texts (Fix-Must1b)

test(issue-417): add Fix-Must1b DM fallback regression test

Commits on Apr 12, 2026

fix(issue-417): F1 success block counter reset + rate limiter inside success path (rwmjhb review)

fix(issue-417): document intentional non-reset of counter after regex fallback