You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Define and test the §4 snapshot-compaction contract #130
Verified 2026-08-30 against comfy-multi-playerorigin/main at 5f880f6c0e534d94dd3d381c4f1cee69fbbd0dad and GitHub PR state.
PR #33 merged on 2026-08-21, so the original instruction to resolve this risk before merge is stale. The residual gap remains:
docs/multiplayer-schema.md §4 requires host snapshot compaction when bookkeeping exceeds 25% of document bytes or 10,000 __applied entries.
The same section records the digest representation at about 32% bookkeeping in the measured session, so the percentage trigger is crossed during ordinary operation.
src/doc.ts explicitly says per-actor watermark compaction is not implemented and delegates snapshot compaction to the host. The package exposes no tested host-facing compaction contract.
The bounded SHA-256 digest from PR #33 fixed unbounded payload-sized entries, but it did not bring bookkeeping below the schema trigger or implement that trigger. __applied therefore remains O(ops). This is no longer a merge blocker for an already-merged PR; it is current implementation debt requiring a shared-package contract plus consumer-host work.
XREF-1 current-state verification
Verified 2026-08-30 against
comfy-multi-playerorigin/mainat5f880f6c0e534d94dd3d381c4f1cee69fbbd0dadand GitHub PR state.PR #33 merged on 2026-08-21, so the original instruction to resolve this risk before merge is stale. The residual gap remains:
docs/multiplayer-schema.md§4 requires host snapshot compaction when bookkeeping exceeds 25% of document bytes or 10,000__appliedentries.src/doc.tsexplicitly says per-actor watermark compaction is not implemented and delegates snapshot compaction to the host. The package exposes no tested host-facing compaction contract.The bounded SHA-256 digest from PR #33 fixed unbounded payload-sized entries, but it did not bring bookkeeping below the schema trigger or implement that trigger.
__appliedtherefore remains O(ops). This is no longer a merge blocker for an already-merged PR; it is current implementation debt requiring a shared-package contract plus consumer-host work.Upstream tracking: #130
Glossary
__applied: replicated map used to recognize an already-consumed operation ID.