[Kutadelios]

What was seen

companion-compact can store the same Companion text more than once in the monthly archive (main@d7a6875):

1. Retry after an interrupted compact. If the archive append succeeds but the live file rewrite does not (process killed, lock timeout, disk error), the next compact plans the same move again and appends the same block a second time.
2. Small edits to an over-limit file. Each compact of a changed Threads.md appends a new full backup, so an unchanged heading ends up stored once per edit.

Reproduction with a synthetic vault: let a Threads.md exceed its limit, run compact four times with a one-line edit before each run. The archive then holds the unchanged heading four times (45,137 bytes in the synthetic fixture). For the retry case, make the live rewrite fail after the archive append, then run compact again: the moved block appears twice.

What changed

• Moved blocks carry a beyin-compact-v1 marker with the source path, a stable operation id (derived from the source path and the full moved text, not from time or month), the length and a SHA-256. The marker and the text are written in the same atomic archive write; no extra state file is needed. A retry recognises a verified record and skips the append. A marker alone, or a damaged or partial record, does not count.
• Backups keep one full base plus difflib.SequenceMatcher deltas whose reconstruction is verified by SHA-256 before they are used. If a delta is not smaller than the text, a full copy is written. Line endings, BOM, Unicode, repeated lines and the trailing newline are preserved. The same synthetic run above stores 13,282 bytes, and all four versions rebuild byte for byte.
• Reuse of the previous month's backup for an unchanged file (fix(compact): aylar arasında değişmemiş tam yedeği yeniden kullan #460) keeps its 12-month / 8 MiB lookback and now shares a single archive reader with the new records. Older unmarked full backups are still recognised and reused, but are not used as a delta base.
• The shared lock order from fix(notlar): note-edit ve companion-compact aynı kilidi paylaşsın (kayıp güncelleme) #527 (.beyin-note-write/writer.lock → .beyin-compact.lock) is unchanged.
• scripts/restore_v3_compact_backup.py: a read-only tool that lists verified backup versions and writes a chosen version to a new file outside the vault.

Still stdlib-only, with no model or network calls. Existing archives are not rewritten.

Tests

Run on Windows 11 with Python 3.12, from this branch:

Suite	Result
v3_note_compact_lock_test.py	7 passed
v3_compact_retention_test.py (new)	24 passed
v3_compact_archive_reuse_test.py	8 passed
v3_hygiene_test.py	36 passed
v3_companion*test.py	137 passed, 1 skipped (POSIX-only lock fault injection)

v3_compact_retention_test.py fails against main's beyin_v3_compact.py (6 failures, 13 errors), so it covers both regressions. Two assertions in v3_compact_archive_reuse_test.py now check the reconstructed full version instead of raw bytes, because a changed backup may now be stored as a delta. I did not run the suite on macOS or Linux.

🤖 Generated with Claude Code

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	⚠️ Failed 2026-10-27T07:26:34.290431Z We couldn’t complete this review. Comment @codex review to try again.	eef646b	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[avenoxai]

Katkı için teşekkürler. Kesinti sonrası arşiv tekrarını önlemek yararlı; ancak beyin_v3_compact.py:697 normal taşımalarda history_now=None geçiriyor, :110 bütün aylık dosyaları sınırsız okuyor. Gerçek geçici dosyayla 2000 yılı arşivinin bu yolda okunduğu, sınırlı yolda okunmadığı doğrulandı. Ortak yazma kilidi altındaki tarama sınırlanmalı; aylar arası bağımlı delta yedek biçimi ayrı ele alınmalı. Bu nedenle mevcut sürüm #562 bakım adayına alınmadı.