Conversation
…lence audit rule (round-9 peer-harness save)
Multi-AI feedback packet (round 9) preserved a load-bearing
safety rule about AceHack→LFG hard-reset: a prior audit used
`git diff --diff-filter=A` to check for AceHack-only FILES,
found none, and inferred 19/23 PRs were ALREADY-COVERED. The
peer-harness check (Codex sampling, Grok attacking
methodology) caught the gap: file-presence is not content-
equivalence. Deeper probe of .github/workflows/gate.yml +
.github/codeql/codeql-config.yml found shared files with
unique AceHack content NOT on LFG.
Hard-reset is now BLOCKED until content-equivalence audit
runs cleanly.
Landed as addendum to existing
memory/feedback_lfg_master_acehack_zero_divergence_fork_double_hop_aaron_2026_04_27.md
(consolidation discipline — no new memory island; addendum
to canonical fork-topology home).
Key rules captured verbatim:
"No AceHack-only files ≠ no AceHack-only content."
"Same path is not same substrate."
"Hard reset is allowed only after content-loss proof,
not content-loss confidence."
Required audit methodology:
- both --diff-filter=A AND --diff-filter=M shared-file
content-compare
- git merge-base --is-ancestor for reachability
- per-PR classification with explicit rationale
- hard-reset gate: all NEEDS-FORWARD-SYNC forwarded +
all OBSOLETE/INTENTIONAL-DROP rationale recorded +
NEEDS-HUMAN-REVIEW resolved + final audit clean
Anti-overcorrection: 145 commits is not the work queue;
unverified content-loss candidates are the work queue.
MEMORY.md paired-edit comment updated.
Per the consolidation discipline: this is an addendum to
an existing memory file, not a new conceptual island.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the fork-topology “canonical home” memory to include a hard-reset safety addendum clarifying that “no AceHack-only files” is not sufficient evidence of content-equivalence, and records a required audit procedure before any destructive reset.
Changes:
- Added a hard-reset safety addendum describing the prior audit gap and the corrected methodology (including required diff/reachability commands).
- Updated
memory/MEMORY.mdfast-path note to reflect the paired edit and point readers at the updated canonical memory.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| memory/feedback_lfg_master_acehack_zero_divergence_fork_double_hop_aaron_2026_04_27.md | Adds the load-bearing “content-equivalence audit” requirement and hard-reset gate criteria as an addendum. |
| memory/MEMORY.md | Updates the fast-path header comment to reflect the paired edit and keep the index current. |
3 tasks
AceHack
added a commit
that referenced
this pull request
Apr 29, 2026
…n note + slot-semantics on MEMORY.md HTML comment (Amara round-10) (#827) * memory(fork-audit): R/C/T diff-filter coverage + plumbing-vs-porcelain note + slot-semantics on MEMORY.md HTML comment (Amara round-10) Three corrections from Amara round-10 review of PR #826 (now-merged fork-topology safety addendum): 1. Audit was missing rename/copy/type-change coverage. The slogan "same path is not same substrate" has an inverse: "different path can still be same substrate." Renames and copies need content comparison, not path equality. Replaced `--diff-filter=A` + `--diff-filter=M` with a single comprehensive `--diff-filter=ACMRT --find-renames` call + per-status interpretation guide. 2. Plumbing-vs-porcelain note added: human audit is fine with `git diff`; scripted tooling should prefer `git diff-tree -r --name-status --find-renames` (plumbing) for output stability + config-immunity, OR add `-c core.quotepath=false --no-ext-diff` to porcelain for predictability. 3. MEMORY.md HTML comment renamed `paired-edit:` → `latest-paired-edit:` per Amara: the comment is acting as a single-slot ledger that loses prior markers each time; making the slot semantics explicit prevents future confusion that only one paired edit exists. (The prior round-9 marker was overwritten without ledger preservation — a chronic drift class. Future enhancement: paired-edit log file if multi-slot history becomes necessary.) Round-10 keepers preserved in audit prose: Added files are a clue. Modified shared files are the trap. Content equivalence is the gate. The four meta-rules from the round-10 packet that are NOT landing as new substrate (per B-0105 consolidation gate): - "Paired-edit failure must stop before commit" — process discipline; will absorb into existing paired-edit lint documentation when consolidation pass picks it up. - "Safety blockers should land as soon as green" — discipline-level rule; honored in this PR's intent. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * fix(fork-audit): enable copy detection + D semantic-regression separate pass + CI cost note (Amara round-11) Three corrections to PR #827's fork-audit additions: 1. --diff-filter=ACMRT requires BOTH --find-renames AND --find-copies. Previous form had `C` in the filter but only enabled rename detection, so copied entries would never appear. Added --find-copies. Plumbing form likewise uses `-M -C`. 2. D=deleted moved to a SEPARATE pass. Not AceHack-content- loss in the AceHack→LFG hard-reset direction (LFG already lacks the file), but can be SEMANTIC REGRESSION if AceHack intentionally removed bad/stale/unsafe content. Review separately when the deleted path is workflow / security config / tooling / governance substrate. 3. Copy-detection cost note for CI tooling. -C has an O(n²) fallback comparing each added file to each deleted file. Added two-pass tooling pattern: fast pass — A/M/R/T with rename detection only slow pass — bounded -C (e.g. -l200) for suspicious clusters Human one-shot audit can stay unbounded. Best round-11 keeper: Renames need -M. Copies need -C. Deletes need judgment. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 <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.
Summary
Round-9 multi-AI feedback packet preserved a load-bearing safety rule about AceHack→LFG hard-reset.
What the peer-harness check caught
A prior reset-readiness audit ran
git diff --diff-filter=Aonorigin/main..acehack/main, found no AceHack-only files, and inferred 19/23 PRs were ALREADY-COVERED. The peer-harness check (Codex independently sampling, Grok attacking methodology) caught the gap: file-presence is not content-equivalence. Deeper probe of.github/workflows/gate.yml+.github/codeql/codeql-config.ymlfound shared files with unique AceHack content NOT on LFG.Hard-reset is BLOCKED until content-equivalence audit runs cleanly.
Where it lands
Addendum to existing
memory/feedback_lfg_master_acehack_zero_divergence_fork_double_hop_aaron_2026_04_27.md(the canonical fork-topology home). No new memory island — consolidation discipline (B-0105) honored.Key rules (verbatim from the packet)
Required content-equivalence audit (BEFORE any hard-reset)
Hard-reset gate (binding)
Anti-overcorrection
Test plan
🤖 Generated with Claude Code