Preserve useful evaluation context in rejected-update evidence #1

Merged
nimmo merged 1 commit from fix/evaluation-evidence into main 2026-09-09 06:46:07 +01:00
Owner

Long Nix evaluation traces could exhaust the excerpt budget on outer module frames and omit the actual error. Reserve part of the existing line budget for final source locations and the rest for named evaluation contexts and errors, preserving the final diagnostic under the byte limit. Keep shared redaction and omit numbered source snippets.

Refs nimmo/nixos-config#205. Companion configuration change: 7bc9a85 on fix/remediation-evaluation-evidence enables --show-trace and tightens advisory reporting.

Validated with incident protocol regressions (long traces, generic-frame noise, redaction, missing/empty logs, byte limits), remediation integration/regression suites using the built package, ShellCheck, and combined nixos-config flake checks. Subsequent real-model replay with the companion text-file read fix correctly identified the pi-coding-agent rename failure; recorded in config issue #205. Replaying the exact #204 candidate showed why retaining a simple trace tail was insufficient: pi-coding-agent nativeBuildInputs context precedes many generic alias frames. The final selector retains that named context and the final rename error within existing limits.

No incident schema, fingerprint, automatic repair eligibility, publication authority, or deployment behavior changes. Deployment and new scheduled model evidence remain pending integration.

Long Nix evaluation traces could exhaust the excerpt budget on outer module frames and omit the actual error. Reserve part of the existing line budget for final source locations and the rest for named evaluation contexts and errors, preserving the final diagnostic under the byte limit. Keep shared redaction and omit numbered source snippets. Refs https://git.nimmog.uk/nimmo/nixos-config/issues/205. Companion configuration change: 7bc9a85 on fix/remediation-evaluation-evidence enables --show-trace and tightens advisory reporting. Validated with incident protocol regressions (long traces, generic-frame noise, redaction, missing/empty logs, byte limits), remediation integration/regression suites using the built package, ShellCheck, and combined nixos-config flake checks. Subsequent real-model replay with the companion text-file read fix correctly identified the pi-coding-agent rename failure; recorded in config issue #205. Replaying the exact #204 candidate showed why retaining a simple trace tail was insufficient: pi-coding-agent nativeBuildInputs context precedes many generic alias frames. The final selector retains that named context and the final rename error within existing limits. No incident schema, fingerprint, automatic repair eligibility, publication authority, or deployment behavior changes. Deployment and new scheduled model evidence remain pending integration.
fix: retain named evaluation context and final errors
All checks were successful
Check / validate (push) Successful in 27s
Check / validate (pull_request) Successful in 5s
cd61d13c0a
Refs nimmo/nixos-config#205
nimmo merged commit a514b03855 into main 2026-09-09 06:46:07 +01:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
nimmo/nixos-auto-update!1
No description provided.