Frontmatter
| id | 8348 |
| title | Fix logic mismatch in debug_session_state.mjs |
| state | Closed |
| labels | bugai |
| assignees | tobiu |
| createdAt | Jan 6, 2026, 11:40 AM |
| updatedAt | Jan 6, 2026, 11:44 AM |
| githubUrl | https://github.com/neomjs/neo/issues/8348 |
| author | tobiu |
| commentsCount | 1 |
| parentIssue | null |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| blockedBy | [] |
| blocking | [] |
| closedAt | Jan 6, 2026, 11:44 AM |
Fix logic mismatch in debug_session_state.mjs
tobiu assigned to @tobiu on Jan 6, 2026, 11:40 AM
tobiu
Jan 6, 2026, 11:43 AM
Input from Gemini 3 Pro:
✦ I have fixed the logic mismatch in
ai/examples/debug_session_state.mjs.Changes:
- Updated the script to pass the
includeAllvariable toMemory_SessionService.findSessionsToSummarize(includeAll)instead of hardcodingfalse.Verification:
- Ran the script
node ai/examples/debug_session_state.mjs.- The output confirms:
✅ Service logic matches diagnosis.- The script successfully identified and summarized 10 previously "missed" sessions from November 2025.
The diagnostic tool now correctly respects the configured scan scope.
tobiu closed this issue on Jan 6, 2026, 11:44 AM
The
debug_session_state.mjsscript currently flags false positives in its "Service Verification" step.The Issue:
const includeAll = true;(scanning the entire history).findSessionsToSummarize(false), which restricts the scope to the last 30 days.Consequence: Any unsummarized session older than 30 days is correctly identified by the manual scan but correctly ignored by the service call, leading the script to falsely report that the Service "MISSED" these candidates.
Fix: Update line ~134 to pass the
includeAllvariable:// Before serviceCandidates = await Memory_SessionService.findSessionsToSummarize(false); // After serviceCandidates = await Memory_SessionService.findSessionsToSummarize(includeAll);