Move dead characters' locks through their estates (stage 6b)

[ryanbarlow97]

Stage 6b: a dead character's locks pass through their estate at the next restart.

The estate planner works on the same scan as the 6a lock pass and adds its entries to the same numbered journal, marked kind: estate with their deathId (6a entries are now marked kind: migrate). For every decided estate whose guard holds, PRIVATE and PUBLIC containers and entities and doors go to the property recipient, GUILD and FACTION ones to the guild-locks recipient. Waiting or frozen parts are not planned; lock state and accessMap are kept. The planner runs in its own try, so a failure (or TLibs older than 2.6.0) leaves the pass with its 6a entries and locks still reach FULL.

• Dedupe and double chests. Only owners marked CHARACTER are estate-planned, so a markerless file the 6a planner claims moves next boot. A container next to one the 6a planner gives the same testator waits too, with every container joined to it, so double-chest halves move together.
• Archive. A pass with no 6a entries archives only the files it journals.
• Replay. An estate entry is replayed only while its estate still applies; otherwise the file keeps (or gets back from the archive) its original bytes and the entry is marked skipped.
• Report. After the pass each decided estate is reported from the state after the pass: DONE with counts (locks for containers and displays, doors) and every moved path (from all journals) only when nothing of the testator's is left in a part with a character recipient; AT_RESTART when something is; RETRY on planner failure. Thievery registers as the thievery estate participant once locks are loaded; at runtime settle() answers from that boot's report (DONE still holds for a later revision whose new character parts held nothing at boot) or AT_RESTART, and RETRY when the decision's guard fails. Each estate is reported on its own, so one failing report does not drop the rest.
• Furniture passes lazily on first interaction for a dead owner with a decided estate whose guard holds.
• /thievery locks setowner refuses a dead or missing character by UUID.
• tools/thievery_rollback.py --estate-only restores estate entries only and keeps the 6a markers; an older jar that completes an estate journal drops kind/deathId, after which only a full rollback undoes those entries.

Pins TLibs 2.6.0 (not yet released, so CI will 404 until it is). Gate: tfmc-mvn -o -B clean verify, 1004 tests, 100% instruction/line/branch/method/class coverage; Python tool tests 8/8.

Docs: TF-Minecraft/Docs#217

🤖 Generated with Claude Code

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}