Repository navigation
#47: decouple brain autosave commit from auto-push per-brain - #8
Merged
xianxu merged 2 commits intoJun 8, 2026
Merged
Conversation
xianxu
commented
Jun 8, 2026
xianxu
commented
Owner
- #47: close — fix stale FindAllSharedBrainsInWorkspace doc ref (review minor)
- #47: decouple brain autosave commit from auto-push per-brain
Every brain now gets the local autosave commit safety net (even with no
remote); auto-push stays opt-in for plain remotes. Replaces the single
shared-OR-gcrypt "watchable" predicate with a pure per-brain BrainPolicy
{Commit, Push, Pull, KeysAdmit} derived from (manifest, remote kind) —
ARCH-PURE (table-tested without a daemon) + ARCH-DRY (remoteKind reuses
brain.ReadOriginURL; deleted the bespoke hasGcryptRemote git exec).
New manifest field `publish: on|off` (default derived): `on` is the
"private but published" opt-in for a plain-remote brain; `off` pauses
only the push half (pull keeps running). gcrypt/shared brains push with
no field — no regression.
Also fixes a latent writer bug: renderFrontmatter never emitted
`autosave:`/`publish:`, so a recipient op (Read→mutate→RewriteFrontmatter)
silently dropped them. Both now round-trip.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
… minor) Boundary review (SHIP) flagged DiscoverAll's comment still naming the pre-rename FindAllSharedBrainsInWorkspace. Plus the sdlc-close issue-file flip (status: done, actual 1.27h). Review-Verdict: SHIP Review-Window: f8f6731..HEAD Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
xianxu
added a commit
that referenced
this pull request
Jun 9, 2026
Personal Apple-account hygiene (FIDO2 keys, Advanced Data Protection, recovery posture) belongs with the shared-brain security family that lives in nous (nous#3/#8/#10/#11), not in brain's repo tracker — it has no brain code surface. Renumbered brain#10 → nous#51 (nous#10 taken). brain/data/project/shared-brain.md pointer updated separately in brain. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.