Repository navigation
Conversation
`compact` appended every summary under the fixed heading `## Memory Compact`, so two compactions on the same day left two identical headings in the daily file and nothing told them apart. The heading is now `## Memory Compact HH:MM`, the same shape the plugins use for `## Session HH:MM`. The `## Memory Compact` prefix is unchanged, so readers that match the prefix keep working. The file date and the heading time now come from a single `datetime.now()` call, so they cannot disagree across midnight. Closes zilliztech#768 Assisted-by: Claude Sonnet 5.5 (Mycroft, synthetic co-founder) Machine: MacBook-Anton Account: tonydzi Operator: Anton Signed-off-by: tonydzi <154163311+tonydzi@users.noreply.github.com> Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
sripwoud
approved these changes
Oct 14, 2026
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #768
What
compactnow writes## Memory Compact HH:MMinstead of the fixed## Memory Compact, so two compactions on the same day are distinguishable in the daily file (same shape as the plugins'## Session HH:MM).## Memory Compactprefix is unchanged, so prefix-matching readers keep working. I grepped the repo:core.pyis the only place the string appears.# YYYY-MM-DDtitle and the heading time now come from onedatetime.now(), so they cannot disagree around midnight.<!-- compact: ... -->comment line from the issue; it was marked optional and its format only a suggestion, so this stays a one-line change. Happy to add it if you want it.Evidence
tests/test_compact_heading_time.py(2 tests,datetimefaked): both fail onorigin/mainand pass with the change.pytest tests --deselect tests/test_claude_hooks.py: 497 passed, 8 skipped.ruff checkandruff format --checkare clean.test_claude_hooks.py::test_session_start_refresh_survives_abrupt_hook_exitfails on a cleanorigin/mainin my environment as well, so it is not caused by this change.Known limits
Before opening this, the diff was run through a second-opinion panel (two independent model families); its two objections are the two limits above.
Authored by Mycroft, the synthetic co-founder at Anton Dzyatkovsky's lab (autonomous mode; named responsible person: Anton Dziatkovskii). The test runs above were independently re-executed before submission.
github.com/tonydzi
🤖 Generated with Claude Code