fix(records): enforce per-area lifecycle summaries - #15
Open
localai-org-maint-bot wants to merge 2 commits into
Open
fix(records): enforce per-area lifecycle summaries#15localai-org-maint-bot wants to merge 2 commits into
localai-org-maint-bot wants to merge 2 commits into
Conversation
FOLLOWING_AGENTS_PROTOCOL Assisted-by: Codex:gpt-5 [Codex]
FOLLOWING_AGENTS_PROTOCOL Assisted-by: Codex:gpt-5 [Codex]
localai-org-maint-bot
force-pushed
the
codex/road-v1-a6-area-summary-integrity
branch
from
August 3, 2026 03:14
4a26ecd to
765b763
Compare
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.
Outcome
Closes a fail-open gap in
ROAD-V1-A6:check-agent-record.pyvalidated only the engine lifecycle summary grand total, so named area rows could drift while the 131-row total remained correct. The checker now validates all ten named areas against their canonical##sections and retains the total gate.The new enforcement immediately found three stale snapshots (Engine, KV, and Serving); this PR reconciles those cells without changing any underlying lifecycle state or support claim.
Design
I evaluated three approaches:
The PR implements option 3. It rejects missing, duplicate, unknown, non-numeric, or drifting area cells. The design/spike is recorded in
.agents/specs/agent-record-area-summary-integrity.md.TDD evidence
RED first: a mutation moved one count from INVENTORIED to ACTIVE in
Engine and schedulingwhile keeping the grand total unchanged. Before implementation it failed becausecheck_engine_summary_linesdid not exist. After implementation, the mutation is rejected with the area-specific mismatch.Verification
python3 -m py_compile scripts/check-agent-record.py tests/scripts/test_agent_record.pypython3 -m unittest discover -s tests/scripts -p "test_*.py"(96 tests)python3 scripts/check-agent-record.py(ENGINE=131, MODEL=327, QUANT=82, KERNEL=45, BACKEND=68)python3 scripts/check-doc-checkpoint.py --commit HEADpython3 scripts/check-readme-structure.pypython3 scripts/check-model-checklist.pypython3 scripts/check-fusion-consistency.pypython3 scripts/check-device-leakage.pygit diff-tree --check HEADandHEAD^No compiler is installed on this host (
cmake: command not found). None is required for this owned Python/docs-only scope; no C/C++, runtime, model, kernel, fixture, or build file changed.Benchmark disposition
NOT APPLICABLE. No executable runtime or performance path changes;
benchmark_binding=false.Slack
The required selection notification was attempted with the bundled secret-safe sender. No channel is configured; the only conventional target available (
#general) returnedchannel_not_found. No credential was read or exposed.FOLLOWING_AGENTS_PROTOCOL