Skip to content

Unify memory presentation through InformationView - #112

Merged
wefio merged 1 commit into
mainfrom
codex/unified-memory-information-views
Oct 27, 2026
Merged

wefio merged 1 commit into
mainfrom
codex/unified-memory-information-views

Conversation

@wefio

@wefio wefio commented Oct 27, 2026

Copy link
Copy Markdown
Owner

Change summary

What: Route shared search, exact-evidence and save-reply presentation through InformationView while preserving existing tool text and domain budgets.

Why: A common presentation protocol gives text and structured consumers the same projection. Immutable snapshots and source fingerprints add cost, so disclosure metadata is computed only when requested and memoized within the prepared output.

Changes:

  • Add prepared search, compact search, evidence and save surfaces; preserve atomic follow-up guidance and save acknowledgements.
  • Deduplicate exact text by source and defer per-source fingerprints until included is accessed. Confirmation still checks the exact text and reader/session on each call.
  • Record rendering and metadata parity, synthetic timing measurements, and updated presentation contracts.

Not verified

No new live-host integration, recall benchmark, model-answer quality evaluation, or end-to-end retrieval-latency measurement was run for this revision. Other mutation-action confirmations, host errors and JSON/RPC business results remain outside the migrated presentation scope. The synthetic timing results do not establish an end-to-end speedup.

One prior full verification run failed while cleaning up an STG temporary directory on Windows with EPERM after the assertions completed; 10 isolated reruns reproduced it 0 times. The cause remains undetermined. A subsequent full run passed. The first pre-push mutation sweep also failed with a Windows file-write error and left one injected mutation; it was restored to HEAD, and the full retry passed with all 23 targets restored byte-identically.

Completion checklist

Local quality checks

  • npm run verify:static passes (the equivalent 16 blocking checks in agent:verify passed; the aggregate command itself was not run)
  • npm run test:product passes (through agent:verify: 2,071 passed, 2 skipped, 0 failed)
  • Dependencies and lockfiles unchanged (dependency-specific audit requirement not applicable)
  • No subpackage changed; verify:packages passed
  • Documentation checks and applicable decision/design maintenance completed
  • Not verified scope is stated and decision record updated
  • complexity:gate passed; the full pre-push mutation:teeth retry passed
  • No reproducible build artifacts committed

RCP (Repository Control Plane)

  • In-flight goal registered and resolved
  • Changed route verified; evidence recorded in .nmg/verification/latest.json
  • All-green CI confirmed through nmg-rcp forge-status
  • Only owned files committed

CI completion confirmation

  • All checks passed is SUCCESS
  • CodeFactor passes
  • Static job passes

Route shared search, exact evidence and save replies through InformationView while preserving existing text and domain budgets. Memoize disclosure metadata on each immutable output so text-only callers avoid source hashing.

Record 15 text parity cases and 12 complete metadata comparisons. All 16 blocking agent:verify checks passed; product tests reported 2071 passed, 2 skipped and 0 failed. Retain the earlier Windows cleanup failure and its 0/10 isolated reproduction result in the experiment report.
@wefio
wefio marked this pull request as ready for review October 27, 2026 05:48
@wefio
wefio merged commit c3bb5c8 into main Oct 27, 2026
7 checks passed
@wefio
wefio deleted the codex/unified-memory-information-views branch October 27, 2026 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant