Emit opt-in catalog location notices after CLI saves - #368
Open
Brian Krabach (bkrabach) wants to merge 1 commit into
Open
Brian Krabach (bkrabach) wants to merge 1 commit into
Brian Krabach (bkrabach) wants to merge 1 commit into
Conversation
Brian Krabach (bkrabach)
marked this pull request as ready for review
October 4, 2026 06:43
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
1 participant
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.
Successful CLI native saves and renames did not announce their changed locations to the running catalog. The opt-in environment variable now emits the existing bounded location-only notice after canonical save success.
SessionStore save, new, transcript, metadata, update and rename wrappers are covered. Publication uses a validated private directory descriptor, a unique mode-0600 file, fsync, atomic coalescing and directory fsync. Notice failure cannot veto canonical save. No process activation, provider call or history watcher is added.
Validation: 43 source and 43 independent installed checks with zero skips. All 103 Python runtime files match source, wheel and installed package. Actual installed CLI save and rename plus the installed catalog qualified new discovery, five coalesced updates, one metadata read, native transcript preservation and failed inbox delivery. Foundation qualification revision is 64d9b8086094b1da25924650509a6df4ca5b1834.
Physical CLI deletion remains separately reviewed. Configured native-owner lifecycle proofs stay separate from location hints; this patch does not create deletion authority. No live CLI reinstall, browser/device acceptance, merge or deployment.