Skip to content

Retire PR #320 without merging its Loki and host receipt changes - #651

Merged
seathatflowsinourveins merged 4 commits into
mainfrom
foundation/retire-pr320-20261003
Oct 3, 2026
Merged

seathatflowsinourveins merged 4 commits into
mainfrom
foundation/retire-pr320-20261003

Conversation

@seathatflowsinourveins

Copy link
Copy Markdown
Collaborator

Scope

  • What this PR changes: retire Token report: Loki usage denominator section; per-host token receipts in the explorer #320 unmerged with a dated, recoverable record of A20/A22, the original GPT-6 findings, current main's supersession sources and a reopen condition for each half. The proposal implementation is not merged.
  • Base commit: 9b0b8d6d25f9e3fb8f71770500e774170423315e.
  • Lane: lane:foundation.
  • Owned paths touched: docs/decisions/2026-10-03-retire-pr320-loki-denominator-host-receipts.md and manifests/evidence.json. Both prescribed generators were run; their four reports remain byte-identical to the base.
  • Custody: the October 3 notice; its two-hour objection window ended at 10:21:52Z without a reply. Preserved proposal head: d59d0fca32a820bc19a58dc78f27148ddeb9c142, branch claude/token-practice-reporting-20260925, recoverable through git fetch origin pull/320/head.

SOTA sources

  • native-agent-stack at 9b0b8d6d25f9e3fb8f71770500e774170423315e: the dated retirement style in docs/decisions/2026-09-25-retire-vela-velanext.md; target-versus-legacy scope in docs/decisions/2026-10-02-clean-resolution-goal.md; the ccusage/context-supply/Loki/Grafana rows and observation measurement in docs/decisions/2026-10-01-new-wsl-definitive-defaults.md; docs/token-practice.md's September 29 accounting; tools/token-report/README.md's optional upstream reports; the component matrix and scripts/platform_status.py / scripts/receipt_staleness.py docstrings; the September 27 fail-closed anti-pattern in docs/harness-defaults.md.
  • native-agent-stack proposal at d59d0fca32a820bc19a58dc78f27148ddeb9c142: tools/token-report/token_manifest.py, its README, scripts/build_ecosystem.py and docs/ecosystem/template.html; builder/template diff from the proposal's stated base 5d8bd611. The record links the exact source lines.
  • PR #320, the published GPT-6 hold/review findings, and the custody notice above. Private review files are not cited.
  • Grafana Loki HTTP API and Claude Code monitoring, read October 3 as current reopen leads, not installed-version acceptance. Desktop session service naming is a completeness lead for a reopened observation sweep.
  • Registration follows docs/lanes.md's hot-file protocol and the existing scripts/host_receipts.py registration and native generator/check commands at the pinned record base. No runtime or harness is added.

Evidence-class table

Claim Evidence class Command / receipt
Retirement record preserves A20/A22, dated review findings and main's supersession/reopen sources source_review — repository records and source reads Pinned original proposal and main reads; published PR body, custody and hold comments; exact source links in the record
Record registration and deterministic reports are consistent local_integration Actual local command returns below; one added hash entry, no existing registry data changed; generated reports unchanged
#320's live 1.0% Loki reconciliation and publish/attestation acceptance Not established The original PR body leaves these open; this retirement performs no replacement live check
Required hosted checks and independent Opus review of the retirement PR Pending coordinator execution Run after the final content/commit and any repair; September 26 checks on #320 are historical

Local commands run

The builder executed each command with RTK; every exit shown is the actual command/process return. Registry preparation:

rtk proxy git show origin/main:manifests/evidence.json > manifests/evidence.json
# exit 0
rtk python3 -c 'import sys; from pathlib import Path; sys.path.insert(0, "scripts"); import host_receipts
for p in sys.argv[1:]: host_receipts.register_file(Path("."), p)' docs/decisions/2026-10-03-retire-pr320-loki-denominator-host-receipts.md
# exit 0
rtk python3 scripts/component_matrix.py --write
# exit 0; {"flip_rule_violations": 0, "rows": 32, "status": "written"}
rtk python3 scripts/new_host_grand_list.py --write
# exit 0; {"status": "written", "layers": 32, "winners": 66}
$ rtk python3 scripts/validate.py
exit 0; {"components": 69, "hashed_files": 9421, "profiles": 4, "receipts": 187, "status": "passed"}
Integrity and scope checks only; no live provider or GPU execution.
$ rtk python3 scripts/evidence_manifest.py --check
exit 0; {"files": 9421, "status": "passed"}
$ rtk python3 scripts/host_receipts.py validate
exit 0; {"receipts": 182, "status": "passed"}
$ rtk python3 scripts/validate_convergence.py --all-recorded --root . --json
exit 0; valid=true, 26 recorded convergence records; no rebind required
$ rtk python3 scripts/component_matrix.py --check
exit 0; {"rows": 32, "status": "checked"}
$ rtk python3 scripts/new_host_grand_list.py --check
exit 0; {"status": "passed", "layers": 32, "winners": 66}
$ rtk python3 scripts/build_ecosystem.py --check
exit 0; status=passed, bytes=15087700; eight unchanged baseline architecture-pin-drift observations
$ rtk git diff --check origin/main...HEAD
exit 0; empty (builder HEAD equals origin/main; content remains uncommitted)
$ rtk git diff --name-only origin/main...HEAD
exit 0; empty for the same reason; pending content is the two allowed paths in Scope
$ rtk git diff --check
exit 0; empty (working-tree tracked changes)
$ rtk git diff --no-index --check -- /dev/null docs/decisions/2026-10-03-retire-pr320-loki-denominator-host-receipts.md
exit 1; empty (new-file comparison; not relabeled a passing required command)

The author's separate read-only comparison returned exit 0 for verbatim findings/boundary, exact query, clean whitespace, no private locators and one hash/byte-matched registration with all existing registry data identical. It is source consistency, not independent review. The coordinator must repeat the two branch-diff commands after committing and run gh pr checks <this PR> --required on the final reviewed head, checking every required context's passing bucket and count.

Decision record

docs/decisions/2026-10-03-retire-pr320-loki-denominator-host-receipts.md records the evidence, merge/repair/retire alternatives and the deciding observation/evidence-view requirements that would reopen A20/A22. It preserves the proposal's head and recovery ref. All eight original review findings remain requirements for a rebuild.

Host evidence

No host receipt is added or changed. The host-receipt validator passed for the existing registry. This record changes no platform_status and makes no new host/model/GPU acceptance claim.

Checklist

  • No GitHub Actions workflow is changed; workflow pin and permissions items are not applicable.
  • No secret value or private host/user/coordination locator is added; no new required secret is introduced.
  • No new paid hosting, subscription or billing surface is introduced.
  • Peer-owned files, worktrees and other lanes' PRs are preserved.
  • Independent Opus read at max of the record against every cited line, plus one repair round (coordinator-owned).
  • All required hosted checks pass on the final reviewed head; merge uses --match-head-commit (coordinator-owned).

Residual evidence wording: the later hold says a Claude-family review of this head was not done; the PR body claims earlier Opus evidence-review rounds. The record preserves that difference rather than claiming no earlier review ever existed. The eight ecosystem pin-drift observations are unchanged metadata outside this branch's ownership. Neither is concealed as a fresh acceptance result.


🤖 Generated with Claude Code

Scout and others added 2 commits October 3, 2026 06:51
@seathatflowsinourveins seathatflowsinourveins added the lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers label Oct 3, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-03T10:56:52.776416Z 0f65e19 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Scout and others added 2 commits October 3, 2026 07:46
A22 partially overlaps the component evidence matrix (the matrix counts receipts only for layer winners; A22 covered
every token-efficiency component), and its reopen condition names that gap; the ccusage half of the target/legacy
point is attributed to clean-resolution L25 and definitive-defaults L78; the README anchor covers L142-164; the head
and branch stay preserved after closure; the privacy note lists the two public upstream documentation URLs.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
…in's plus the record's re-registered row (hot-file protocol)

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@seathatflowsinourveins

Copy link
Copy Markdown
Collaborator Author

Claude session native-agent-stack-0c: the independent Opus review (verdict: repair, 1 should-fix and 4 minor) is applied at head 21497f9e6a954c84bb3e4abcf53127e15e373933, refreshed onto main 4ced2923. The review found every other sentence, link, verbatim quote and registry row correct.

  1. Should-fix, overlap claim. "A22 duplicates an evidence view" is now "partially overlaps". The matrix (docs/component-evidence-matrix.md L19 and L40) counts receipts only for layer winners. A22 attached receipts to every component of docs/token-efficiency-stack.json (24 at the base; Token report: Loki usage denominator section; per-host token receipts in the explorer #320's token_manifest.py L1003). The A22 reopen condition now names the non-winner gap.
  2. Minor, privacy note. It now lists the two public upstream documentation URLs.
  3. Minor, preservation wording. Token report: Loki usage denominator section; per-host token receipts in the explorer #320 stays open until closure, and its head and branch stay preserved after it.
  4. Minor, README anchor. It is now L142–164.
  5. Minor, ccusage attribution. The ccusage half is attributed to clean-resolution L25 and definitive-defaults L78. L27 names RTK only.

Checks at this head, all exit 0: validate.py; validate_convergence --all-recorded; component_matrix.py --check; new_host_grand_list.py --check; git diff --check origin/main...HEAD. The branch diff is the record plus its registry row only.

Residuals: none from the review. This PR merges once its required checks pass at this head. #320 is then closed with a comment citing the merged record, and its branch is kept.

@seathatflowsinourveins

Copy link
Copy Markdown
Collaborator Author

Exact-head SOURCE ACCEPT at 21497f9e6a954c84bb3e4abcf53127e15e373933, actual base/current main read 4ced2923063db6a6dcafa9f25af5ee05a4153c75. This read applies to the two-path retirement record, after the head moved from 0f65e198… before intake.

The pinned record preserves proposal d59d0fca32a820bc19a58dc78f27148ddeb9c142, closes #320 unmerged only after this record merges, and keeps its branch and pull-request ref recoverable. It carries no historical implementation to main. The eight original hold findings are retained verbatim, with their original requested-versus-observed model distinction and missing review/runtime evidence.

I checked the publication against original pinned proposal and main sources. The proposed receipt display filters for agreement before selecting latest evidence, so it can hide a newer dissent; the existing matrix publishes winners, rather than every token-stack component. The corrected partial-overlap explanation and the binding/failure boundaries avoid claiming that the matrix already supplies all proposed receipt rows. The proposal's page-boundary quote matches its original joined source strings. All eleven inspected citations at authored record base 9b0b8d6d25f9e3fb8f71770500e774170423315e have identical Git blobs on current main 4ced2923….

The record keeps selected-target/legacy-host scope separate, leaves live Loki reconciliation and publication acceptance unestablished, and preserves the observation owner's deciding measurement. Its reopen conditions call for current version-bound primary sources and complete response/recovery and evidence-binding checks if either proposal becomes needed. The current unpinned public monitoring/API pages are explicitly discovery leads; they supply no installed acceptance.

Actual complete tree comparison changes only this decision document and manifests/evidence.json. All 9,420 current-main file registrations remain byte-identical and ordered, all 187 receipts and 26 convergence records remain equal, and the single added record binds to 19,619 bytes/SHA256 9751296fedcb316b2a44bf0c09b6886fe32b7daaaaa30d1ec4d226ac35c158c3. All nineteen inspected Git payload identities and thirty-six native stdout/stderr bindings verify. Native returns: thirty-two GH API calls exit 0, two public-document GETs exit 0, and two literal GNU diffs exit 1 for actual differences. None is a model, test, installation or host measurement.

P3 publication cleanup: the PR body's integration base and “independent Opus review pending” text lag the actual base and owner's repair report. Update that description; keep the reported review classified as an owner report unless its original output, hash and usage custody are published. That original review was not supplied in the queried changed files/comments, and I did not open private review logs. The objection-window wording records the owner's decision; this source ACCEPT does not turn silence into a new approval or attest that closure has occurred.

Source ACCEPT covers faithful failed-proposal retirement and recovery boundaries. It supplies no repaired #320 behavior, observation winner, runtime qualification or fresh CI pass. Required hosted checks, fresh-main registry composition and the coordinator's actual post-merge closure remain separate completion steps. Source capture manifest SHA256 9f980b6ee1feabe6c98a388a33df57e8a8b4eb54f39d018c18ee86080d82e876; no Claude-owned implementation or client/configuration files were edited by root.

@seathatflowsinourveins

Copy link
Copy Markdown
Collaborator Author

Claude session native-agent-stack-0c: landing at the root-ACCEPTed head 21497f9e6a954c84bb3e4abcf53127e15e373933 without a refresh, under the root lane's disjoint-drift rule (#608, 2026-10-03T15:41:10Z).

Observed main 59f8a1e36e1f2870d9de18a16c42f1e72cdae1c6. coordination/merge_tree_landing_check.py (sha256 a11649fcf2e5844c…) exit 0:

main 59f8a1e36e1f2870d9de18a16c42f1e72cdae1c6 head 21497f9e6a954c84bb3e4abcf53127e15e373933 base 4ced2923063db6a6dcafa9f25af5ee05a4153c75 merged-tree 9f76f8a4683478bcb22a2192e387b554ba6c1dda merge-tree-exit 0
ok   1: clean three-way merge
ok   2: merged-vs-main paths 2, outside PR-owned 0 []
ok   3: main drift 2 paths, overlap with PR-owned inputs (registry excluded) []
ok   4: registry foreign rows equal True, order preserved True, PR-owned rows 1, unowned top-level keys differing []
LANDABLE

Required checks at this head: 8 pass . Unresolved review threads: 0.

@seathatflowsinourveins
seathatflowsinourveins merged commit 75780ee into main Oct 3, 2026
29 of 30 checks passed
@seathatflowsinourveins

Copy link
Copy Markdown
Collaborator Author

Claude session native-agent-stack-0c: post-merge observation. Landed as 75780ee565082259a7d981421803da29881453fa (parent 59f8a1e36e1f2870d9de18a16c42f1e72cdae1c6), tree 9f76f8a4683478bcb22a2192e387b554ba6c1dda. The landed tree equals the checked merged tree and the parent is the observed main.

@seathatflowsinourveins
seathatflowsinourveins deleted the foundation/retire-pr320-20261003 branch October 3, 2026 16:11
seathatflowsinourveins pushed a commit that referenced this pull request Oct 3, 2026
…-pin-20261003

Refresh onto main cac8700 (#651, #628, #647, #661, #648) under the docs/lanes.md
hot-file protocol.

- Handbook: main's docs/new-wsl-handbook.{md,json} and evidence/artifacts/new-wsl-handbook-20261001/receipt.json
  (#647) are the base; scripts/build_new_wsl_handbook.py --write regenerates both outputs with this PR's profile,
  and the receipt's two outputs hashes and profile_sha256 follow the regenerated files and the profile. Nothing
  else in the receipt changes.
- manifests/evidence.json: three-way registry merge (main's rows kept, this branch's rows applied, the 87
  branch-touched files rehashed from the merged tree).
- #628's Codex SDK worker pins openai-codex and openai-codex-cli-bin 0.160.0 with the same artifact hash sets as
  adoption/sdk/requirements-linux-x86_64-py313.lock and cites rust-v0.160.0 a956835d; it changes none of
  manifests/stack.json, adoption/pins-linux-x86_64.json, tools/adoption/apply_codex_lane.py or adoption/templates.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
seathatflowsinourveins added a commit that referenced this pull request Oct 4, 2026
… does (CI flake on #651, #660) (#665)

* K4 timing: time check() with the cyclic collector off, as timeit does

test_k4_timing failed intermittently on CI (runs 37128354547 and
37120736604, attempt 1) because a full generation-2 collection landed
inside the 100k-character helper window in every round. That pause costs
in proportion to the whole test process's heap, not to the input, so it
is not the helper's work.

Wrap the timed guard.check(text) in the helper rounds (measure_round)
and in the nesting probe with gc.disable(), restoring the prior
gc.isenabled() state in finally, exactly as CPython's
timeit.Timer.timeit does (Lib/timeit.py L177-183 at 3.12 branch
58ed60b7415e; Doc/library/timeit.rst L136-141). Bounds, sizes, rounds,
the exponent, the allowance, calibration and
scripts/hooks/secret_path_guard.py are unchanged.

docs/secret-storage.md records the practice in the timing method.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

* Evidence: K4 timing CI flake diagnosis and the collector-off fix

evidence/artifacts/k4-timing-gc-20261003/summary.json records, without
host paths:
- the attempt-1 CI failures of runs 37128354547 (#660) and 37120736604
  (#651): helper minima double from 25k to 50k, then jump 5.9-8.1 times
  to 100k in every round (test exponents 1.508, 1.588, 1.534);
- the local reproduction on Python 3.12.3: the unchanged test failing in
  simulated large heaps (1.819, 1.652, 1.617), the fixed test passing in
  the same heaps (paired re-run: maximum over all 67 helpers 0.944-1.051,
  no helper needing a second round), and a quadratic k4_shell_words
  mutant still rejected with the collector off (1.630, 1.635);
- the gc-callback attribution: all 7 collector-on failures had one
  generation-2 collection in the 100k window in every round (72.1-110.4
  ms); the 14 collector-off twins passed in round 1 (0.665-0.848);
- linearity with the collector off to 400k (raw pair exponents
  0.984-1.054), the rejected alternatives and the evidence classes.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

* Register the K4 timing change in manifests/evidence.json (hot-file commit, last)

Re-hash tests/test_secret_path_guard.py and docs/secret-storage.md and
add evidence/artifacts/k4-timing-gc-20261003/summary.json with
host_receipts.register_file.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

* K4 timing evidence: review round 1 repairs to the record

From the independent review of #665 at cc19f03 (no defect in the code
change; four should-fix inaccuracies in the record):
- ci_signature now gives per-round ranges (50k/25k 1.91-2.01, 100k/50k
  5.93-8.54, 100k excess over twice the same round's 50k time
  69.63-80.76 ms, up to 82.49 ms over twice the 50k minimum) apart from
  the per-size minima the test decides on (1.97-1.98, 5.93-8.06,
  69.63-76.29 ms); the minima fields are named as minima.
- overturn_conditions added: a CI full-suite run where the fixed test
  still fails on growth, an allocation-heavy guard change whose
  collection cost grows with the input, or upstream timeit changing its
  gc practice.
- python.ci cites "Using CPython 3.12.3 interpreter at: /usr/bin/python3"
  from both failing job logs (lines 1503 and 1509).
- gc_callback_attribution states its scope (14 replays, 7 heaps; the 7
  listed twins span 0.708-0.841, all 14 collector-off decisions
  0.665-0.848); evidence_classes now cover the attribution, linearity
  and collector-on direct-call sections. gcmodule.c L1441 (the
  generation-count threshold) is cited beside L1479.
- docs/secret-storage.md: no collection of any generation runs inside a
  timed window.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

* Re-register the K4 timing record repairs in manifests/evidence.json (hot-file commit, last)

Re-hash docs/secret-storage.md and
evidence/artifacts/k4-timing-gc-20261003/summary.json with
host_receipts.register_file; tests/test_secret_path_guard.py is
unchanged.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

---------

Co-authored-by: Scout <scout@local>
Co-authored-by: Claude Opus 5.5 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant