Repository navigation
feat(checkpoint): defer_seal — card answers as drafts for a host that seals at the session's end - #47
Merged
Merged
Conversation
… seals at the session's end A host that seals once, at the end of a session (willow-bot's one script, with Nestor's new session seal), now passes defer_seal=True through open_bite -> run_checkpoint. A fresh Socratic answer, or a recognized answer the maker confirms, is then proposed as a Nestor draft (CheckpointMemory.propose -> EntityResolver.propose) instead of sealed, and is not attested: nothing was committed for an attestation to sign off on yet. The outcome says so (drafted=True, sealed=False, pair_id of the draft). The default is unchanged: seal and attest, as before. The operator: "there is really nothing to seal yet because no real decision has been made other than hey I want to build this thing that's not a decision that's an idea." Tests attempt the forbidden act (a mid-session seal: CheckpointMemory.seal patched to fail) on both bands and through the entry, and pin that the default still seals. Mutations, each caught: defer ignored (3 red), the socratic band dropping the flag (2), the recognize band dropping it (1), the entry dropping it (1), drafts attested (2), the outcome claiming sealed (2). Persona: willow Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WpF9VeQij17q9eaens5cNb
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
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.
What
open_bite(..., defer_seal=True)→run_checkpoint(defer_seal=True)is for a host that seals once, at the end of a session: willow-bot's one script, with Nestor's new session seal.With the flag on:
CheckpointMemory.propose→EntityResolver.propose) instead of being sealed.drafted=True,sealed=False, andpair_idis the draft's.With the flag off (the default), nothing changes: the answer is sealed and attested as before.
Why
The operator: "there is really nothing to seal yet because no real decision has been made other than hey I want to build this thing that's not a decision that's an idea." An answer given mid-session is one piece of the session, and the person's one seal is over the session's whole.
Proof
pytest tests/tools/vendor_sync_check.pyruff check ./ruff format --check .(0.16.7)tests/test_checkpoint_defer_seal.pyCheckpointMemory.sealis patched to fail (a mid-session seal) and the run must not call it. A fourth pins that the default still seals and attestsforgewith this branch onONESCRIPT_FORGE:populated, majorgamesealedCompanion PRs: Die-Namic-Systems/Nestor (session_seal) and willow-memory/willow-bot#143 (one-script).
Ratified-by: sean-campbell — "Yes, build it"
Generated by Claude Code