Repository navigation
docs(curation): correct Cole Medin second-brain write authority - #522
Merged
Knapp-Kevin merged 1 commit intoOct 30, 2026
Merged
Conversation
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.
Scope and evidence
Pinned Cole Medin source
fb2e876f057c5356d6603ba0c52d6b4418d893ba, exactsecond-brain-auditmanifest blob7a3e971c0d6b5b9defc1d28182ef9b302e0e9252, package treec5fc704b1ae0fe67c12d7a02c5d7092fcf6278ad(SKILL.md + source-authored audit.py). Ten-package risk screen: all three previously VERIFIED source manifests directly inspected; seven already-rejected controls reconciled against existing exact-version evidence.Decisive correction
second-brain-auditVERIFIED 19/20 → REJECT unchanged 16/20. Phase 6's permission to convert one notes location does not authorize Phase 7's separate persistent CLAUDE.md/AGENTS.md write; broad always-loaded/private-notes reads also lack a bounded read scope and minimization gate. Preserve state-vs-event, freshness triage, no-bulk-conversion, diff-preview and read-only accelerator methods as ADAPT/EXTRACT/CHECKLIST.ast-grepandrules-check-driftremain VERIFIED semantic-only. Seven rejected controls retain their existing decisions. No upstream material was changed.Atomic accounting and hygiene
Cole Medin 33/33 → 33/33, zero gaps; VERIFIED 3→2, REJECT 30→31. No new reviews, denominator delta 0; persisted counts remain 44 first-party / 867 third-party / 911 total. All five public surfaces updated in the same commit. One broken relative link in SYSTEM_STATE corrected. Eight files total, including two exact-version companions and macro review.
Lifecycle and acceptance
No scanner or behavioral execution claimed; behavioral not-run. No repository-owned runtime/scripts/CI introduced. Pre-existing draft repairs #489/#492/#496 remain blocked by 36 exact-final-tree SkillSpector v2.12.0 reports plus independent executed adversarial acceptance and registry reconciliation. Admissions #412/#413 remain separate HOLD. This PR has no new external qualification dependency because it only removes an unsafe unchanged-use approval and corrects passive evidence.
Macro review.