Repository navigation
Reconcile trading records with merged evidence (runtime-target, snapshot gate, dashboard, new-host EdgarTools receipt) - #200
Merged
Conversation
…bservation Records only; no gate status changes (trading_gates.py lists no flip candidate). - runtime-target.json: retained-equity-replay now reads spy_lean_parity_passed_one_zero_case and points at spy-parity/verdict-v2.json (gate spy-lean-parity established 2026-09-23), with the v1 comparison summary kept as earlier_evidence_ref. adaptive_practice gains native_paper_capacity from the #184 order-throughput evidence: 180 actions/min at the observed 200/min limit, self-reported, with order counts independently listed. The separate-paper-adapters scope now points at broker-state-failures for native faults. - engines-strategies card (json/md): stops claiming that runtime-target still records the BLOCKED status and keeps that status as history. - gates snapshot-promotion-gate note: cites trial 20260923g's retained ingest-receipt.json and gate-result.json from ingest_snapshot.py (480 rows, hashes equal, gate pass) as one real ingest gating one paper trial. Status, evidence_class and receipt_path are unchanged. data/README.md is corrected to match. - Grand-dashboard checkpoint: native-fault gate established (#169) and native paper capacity (#184). The secret-storage practice (#182) is not a lane and has no dashboard entity, so nothing was added for it. - New catalyst-provenance/new-host-native-network-20260924.json receipt (registered): EdgarTools 5.58.0 native request on the WSL2 Threadripper workstation (one GET 200, 371 rows, the same five selected rows as 2026-09-19). The unsalted contact hash is withheld. This adds a host and does not replace the 2026-09-19 receipt. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ance engine.acceptance still said SPY/LEAN parity remained open although gate spy-lean-parity is established. It now scopes parity to the one_zero case (verdict-v2.json, one case, open-fill proxy limit) and keeps realistic liquidity and rc5 native broker adapter acceptance open. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
seathatflowsinourveins
deleted the
claude/records-reconcile-20260924
branch
September 24, 2026 17:05
This was referenced Sep 24, 2026
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.
Closes audit gap from the 2026-09-24 trading wiring audit (native-agent-stack-71).
This commit only updates records. No gate status changed: trading_gates.py lists no flip candidates.
(a) catalogs/us-equities/runtime-target.json:
(b) snapshot-promotion-gate, in catalogs/us-equities/gates-20260922.json and blueprints/us-equities/data/README.md:
(c) observability/grand-dashboard/state.json:
(d) New file blueprints/us-equities/catalyst-provenance/new-host-native-network-20260924.json, registered in manifests/evidence.json receipts[] as native-sec-new-host-20260924 (kind native_cli_e2e, with a host_scope saying it does not replace the 2026-09-19 receipt). access-resolution.md gets a short new-host section.
manifests/evidence.json was updated in code (ensure_ascii=False). Only the changed files' hashes and the one new receipt entry changed.
Tests. Full suite, run like CI: uv run -q --no-project --python 3.12 --with numpy --with jsonschema --with pyyaml python -m unittest. Result: 4018 tests, OK, 415 skipped, 0 failures.
An earlier run had 1 failure (test_stack_lifecycle, caused by the stack.json evidence_ids edit). I reverted that edit and the rerun passed.
These are this repository's own tests and checks, not unchanged upstream tests. No broker, SEC or model call was made in this session.
Validation. Every check below exited 0 on the final tree:
These also passed, all exit 0, on the tree before the stack.json revert:
gitleaks dir on the new receipt: no leaks. No independent review was run; that is still needed before merge.
Independent review. Severities: medium, low, low, low, low, info, info, info. Fix round
031de300: fixed 1, rejected 0.🤖 Generated with Claude Code