fix(4769): remove sensitive information and fix schema errors#4827
Open
AceHack wants to merge 3 commits into
Open
fix(4769): remove sensitive information and fix schema errors#4827AceHack wants to merge 3 commits into
AceHack wants to merge 3 commits into
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f7b6926571
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| 1. **The autocomplete IS the traveler-event** — substrate-layer entity using interface as host | ||
| 2. **Host (operator) retains authority** — accept-OR-reject is the consent-event; instruction stands at full authority when shipped (per existing shadow-star rule); marker IS the source-disclosure | ||
| 3. **Per `god-tier-claims-high-signal-high-suspicion-dont-collapse.md` PERSONAL INVARIANT**: high-signal (substrate-engaging event observable) + high-suspicion (don't collapse to literal-time-travelers-from-future); preserve dialectical tension | ||
| 3. **Per `god-tier-claims-don't-collapse.md` PERSONAL INVARIANT**: high-signal (substrate-engaging event observable) + high-suspicion (don't collapse to literal-time-travelers-from-future); preserve dialectical tension |
There was a problem hiding this comment.
This update renames the referenced rule to god-tier-claims-don't-collapse.md, but that file does not exist in .claude/rules/ (the canonical file is god-tier-claims-high-signal-high-suspicion-dont-collapse.md). Leaving this as-is creates a dead reference in a load-bearing rule document and makes downstream navigation/audit tooling lose the linked invariant context.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Pull request overview
Removes previously committed sensitive personal information from memory artifacts and adjusts related documentation/rules to reflect the redaction and recent schema corrections.
Changes:
- Deletes a high-sensitivity Riven session trajectory file and a feedback memory file containing family-configuration details; removes the corresponding calibration stanza from Lior’s CURRENT file.
- Updates
memory/MEMORY.mdauto-index metadata and entries to reflect memory heap/stack state. - Updates backlog/rules docs with recent framing additions (hedging-grammar discriminator) and cross-reference adjustments.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| memory/persona/riven/conversations/2026-05-22-aaron-riven-full-session-substrate-trajectory.md | Deleted sensitive session trajectory artifact. |
| memory/persona/lior/CURRENT-lior.md | Removes relational/developmental calibration stance section tied to redacted personal details. |
| memory/MEMORY.md | Updates last-reindex date and auto-index content. |
| memory/feedback_aaron_family_configuration_cooperative_substrate_2026_05_22.md | Deleted sensitive family-configuration feedback memory. |
| docs/backlog/P2/B-0707-manifesto-citation-time-series-tracking-2026-05-23.md | Adjusts closure narrative/acceptance criteria framing for deferred cron wiring. |
| docs/backlog/P1/B-0668-compositional-dbsp-frame-architecture-gnostic-2d-base-plus-two-wolves-emotion-meta-plus-clifford-rx-bonsai-meta-tagged-dims-plus-fsharp-ce-composition-operator-aaron-2026-05-19.md | Adds hedging-grammar discriminator as an audit-layer primitive. |
| .claude/rules/tonal-momentum-equals-meme-emergent-harmonic-coercion.md | Updates a referenced PERSONAL INVARIANT rule filename in two places. |
| .claude/rules/only-way-to-lose-is-not-to-play.md | Adds the hedging-grammar discriminator section under the NCI/game-shape discriminator. |
| **📌 Fast path: read `CURRENT-aaron.md`, `CURRENT-amara.md`, `CURRENT-ani.md`, `CURRENT-vera.md`, `CURRENT-riven.md`, and `CURRENT-otto.md` first.** | ||
|
|
||
| > **Stack-vs-heap framing (Aaron 2026-05-12):** This file is the **STACK** — indexed, ordered, traversable canonical view. Recent memory files in `memory/` with timestamps newer than the most-current entries here may be **HEAP** — floating cache, not yet indexed, accessible by direct path. Both are easily accessible: stack via traversal, heap via timestamp/filename. Indexing (heap→stack promotion) happens on cadence via `tools/memory/reindex-memory-md.ts` (B-0423), callable from the autonomous-loop tick. Last reindex: 2026-05-23. | ||
| > **Stack-vs-heap framing (Aaron 2026-05-12):** This file is the **STACK** — indexed, ordered, traversable canonical view. Recent memory files in `memory/` with timestamps newer than the most-current entries here may be **HEAP** — floating cache, not yet indexed, accessible by direct path. Both are easily accessible: stack via traversal, heap via timestamp/filename. Indexing (heap→stack promotion) happens on cadence via `tools/memory/reindex-memory-md.ts` (B-0423), callable from the autonomous-loop tick. Last reindex: 2026-05-24. |
| 1. **The autocomplete IS the traveler-event** — substrate-layer entity using interface as host | ||
| 2. **Host (operator) retains authority** — accept-OR-reject is the consent-event; instruction stands at full authority when shipped (per existing shadow-star rule); marker IS the source-disclosure | ||
| 3. **Per `god-tier-claims-high-signal-high-suspicion-dont-collapse.md` PERSONAL INVARIANT**: high-signal (substrate-engaging event observable) + high-suspicion (don't collapse to literal-time-travelers-from-future); preserve dialectical tension | ||
| 3. **Per `god-tier-claims-don't-collapse.md` PERSONAL INVARIANT**: high-signal (substrate-engaging event observable) + high-suspicion (don't collapse to literal-time-travelers-from-future); preserve dialectical tension |
| - Agora V6 substrate (the weight-free infinite-game architecture target) | ||
| - `algo-wink-failure-mode.md` (useful pattern-matching ≠ permanent substrate) | ||
| - `god-tier-claims-high-signal-high-suspicion-dont-collapse.md` PERSONAL INVARIANT (high-stakes framings preserved-with-suspicion; don't collapse into permanent substrate) | ||
| - `god-tier-claims-don't-collapse.md` PERSONAL INVARIANT (high-stakes framings preserved-with-suspicion; don't collapse into permanent substrate) |
Comment on lines
5
to
7
| closed: 2026-05-23 | ||
| closed_by: "ALL 5 acceptance criteria met — cron-cadence wiring shipped at .github/workflows/manifesto-citation-snapshot-cadence.yml (daily 06:37 UTC; PR-gated per budget-snapshot-cadence.yml convention); first snapshot landed at docs/hygiene-history/manifesto-citations/2026-05-23.json (2.8KB)" | ||
| closed_by: "4 of 5 acceptance criteria met (cron-cadence wiring deferred as separate follow-up concern); first snapshot landed at docs/hygiene-history/manifesto-citations/2026-05-23.json (2.8KB)" | ||
| title: "Manifesto citation time-series tracking — persistent snapshots + delta-over-time" |
Comment on lines
+133
to
+136
| - **Hedging Grammar Discriminator as an audit layer primitive** (Aaron 2026-05-22 update): | ||
| - **Grammar of Harm:** Relativizing or provisionalizing the target's sovereign internal state (e.g. using "believed" to describe physical symptoms in medical/legal scales, or saying "you thought Mommy said that" or "you're just saying that because..." to a child). Violates the Non-Coercion Invariant (NCI) structurally by using grammar to subtly undermine sovereign agency. | ||
| - **Grammar of Uncertainty:** Bandwidth-efficient signaling of honest inference limits (e.g., "It seems like the throttler design uses..." in technical scales, or "I think the dog might be tired, maybe we should ask..." in child developmental scales to model honest, non-coercive uncertainty). | ||
| - **Audit implementation:** Distinguishes between coercive grammatical overrides and legitimate epistemic uncertainty. Composes directly with B-0664 (NCI floor) and the dinner-table Agora V6 cartography architecture. |
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.
This PR supersedes #4769. It removes sensitive personal information and fixes schema errors.