Repository navigation
memory-lifecycle-v2 repairs: ai-memory's own upstream suites at v2.3.2/v2.4.0/v2.4.1, repaired fixture (GPT-6 reviewed) - #330
Merged
seathatflowsinourveins merged 1 commit intoSep 26, 2026
Conversation
… repaired fixture, retained evidence Repairs the GPT-6 cross-family findings on #292: analyzer rules, retained sanitized native responses with publication records, preserved failed attempts, corrected timestamps and evidence classes; adds ai-memory's own cargo test --workspace --all-targets at v2.3.2, v2.4.0 and v2.4.1. Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
seathatflowsinourveins
deleted the
claude/memory-lifecycle-v2-repairs-pr-20260926
branch
September 26, 2026 10:41
This was referenced Sep 26, 2026
seathatflowsinourveins
added a commit
that referenced
this pull request
Sep 26, 2026
…und (#340) memory-lifecycle-v2 and retrieval-quality-v2 cite their repaired 2026-09-26 results (#330, #335); token-efficiency-wiring states the retained 2026-09-26 coverage run (#333). Co-authored-by: Scout <scout@local> Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]>
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
This PR repairs the GPT-6 cross-family findings on merged #292 (
blueprints/memory-lifecycle-v2/) and adds ai-memory's own upstream test suites at the pinned release tags.Upstream verification is
native_proven: unchanged upstream suites at the release tags, run with the upstream CI command. Each receipt is underblueprints/memory-lifecycle-v2/upstream-20260926/.cargo test --workspace --all-targetscargo test --workspace --all-targetscargo test --workspace --all-targetscargo test --workspace --docThe v2.4.1 doc-test run selected no tests, so it is recorded as untested. Round 1 of these runs was superseded and is kept under
superseded-round-1/. A discriminating control, filed separately underdiscriminating_controland labelledlocal_integration, runs pass, fail, fail, pass against mutated checkouts.control.pynow exits nonzero unless that exact sequence occurs and the checkout is restored.Repaired fixture rerun (
local_integration: a locally authored fixture that drives the official binaries over stdio MCP insidebwrapwith no network). Each binary passed on its first attempt; the source isresults-20260926.json:What was wrong, and the fix (GPT-6 findings F1–F8, all verified)
expired_at.runs-20260926/) and the 2 surviving 2026-09-25 runs (runs-20260925/). Each run has apublication.jsonbinding every public file to its private source's sha256.history/attempts-20260925.json. The original preregistration (ca81e9b4…, before any run) is kept byte-identical beside the amended one (fb82e0aa…).local_integration.assemble_results.pyreports every attempt.run_upstream_tests.pyrefuses to overwrite an earlier log or receipt.setup.sh's header no longer claims a pre-run rustup-init checksum; the binary was compared after the runs (rustup-init-verification.json).<scratch>substitution.Review
The #292 description is corrected in a comment on #292.
Scope notes
memory-lifecycle-v2changes in a separatelane:sharedPR, becauseobservability/grand-dashboard/state.jsonis a shared hot file.DashboardCheckpointTestschecks the row's date once it cites the 2026-09-26 results, and skips until then.memory-lifecyclegate incatalogs/us-equities/convergence-review.jsonstays open. Live-store policy, scoped retrieval quality and evaluated learning were not exercised.Verification
python -m unitteston this tree:Ran 6041 tests,OK (skipped=628).scripts/validate.py,host_receipts.py validate,component_matrix.py --check,new_host_grand_list.py --check,evidence_manifest.py --checkandvalidate_catalogs.py: all pass.SOTA sources
cargo testtargets: https://doc.rust-lang.org/cargo/commands/cargo-test.html🤖 Generated with Claude Code