Skip to content

fix: restore recent file history with partial session rows - #11

Merged
davidrobertson merged 1 commit into
mainfrom
fix/restore-file-history-session-selection
Oct 10, 2026
Merged

davidrobertson merged 1 commit into
mainfrom
fix/restore-file-history-session-selection

Conversation

@davidrobertson

Copy link
Copy Markdown
Owner

File-history retrieval could report no history for recently changed files when the session store contained partial legacy rows. Missing start times made the date comparator return NaN before the handler selected its 15 sessions, and null rows could crash retrieval.

Reuse the existing validated session selector in the shared file-context handler. Keep the current project filter, current-session exclusion, and 15-session limit. Add regressions for null/partial rows and filtering before the limit.

Validation:

  • Reproduced three failures before the fix; all four regression cases now pass.
  • Build, skill generation/checks, and docs synchronization passed.
  • Full suite: 2,687 passed, four skipped.
  • TypeScript reports the same pre-existing diagnostics as unchanged main, with byte-identical baseline and candidate logs.
  • Applied the candidate build to the local daemon after backing up the original runtime. The same native MCP file-history query changed from no history to five stored observations, including the upgrade's restart-handling change.

The patch does not migrate or delete stored data. Local verification artifacts are under /private/tmp/agentmemory-file-history-evidence.

@davidrobertson
davidrobertson merged commit 823df5d into main Oct 10, 2026
10 checks passed
@davidrobertson
davidrobertson deleted the fix/restore-file-history-session-selection branch October 10, 2026 18:31
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