Task: Store a run's errors once, and keep the rows the vocabulary names

Table of Contents

This page documents a task in the Prove the report grid run end to end — the remaining work story. It captures the goal, current status, acceptance, and any notes or results.

1. Goal

A run's errors are written once however often its output is read, and one unknown scope costs only its own rows.

2. Status

Field Value
State DONE
Parent story Prove the report grid run end to end — the remaining work
Now Nothing.
Waiting on Nothing.
Next Nothing.
Last touched 2026-10-25

3. Acceptance

4. Plan

(Implementation strategy. Written when work starts; key decisions are distilled into the parent story's * Decisions at close, but the plan itself stays — it is the historical record of what we did.)

5. Measurements

Every number this task claims about a tree, with the command that produced it and the commit it was measured at. A number without both is not a measurement, because a clean-up record describes a tree and trunk moves under a branch: a consumer count, a gate's component count and a suite's size all change with no code changing, and the reader of a bare number cannot tell which tree it came from. Where the raw output matters, put it in an evidence/ file beside the task and name that file in the Command column.

What Value Command Commit
       

6. Notes

7. Test Scenarios

Manual QA scenarios (scaffolded via compass add test_scenario) that verify this task. Link new ones here as they're created; the scenario doc itself links back via its "Verifies task" field.

Scenario State Notes
     

8. PRs

PR Title
#3810 [reporting] Store a run's errors once, and keep the rows the vocabulary names

9. Review

Comment summary File Decision Notes
       

10. Result

A redelivery of the step that reads a run's output no longer stores the same reasons twice. The recorder asks the run whether it already holds errors and returns before writing when it does. The guard covers the run-level failure row as well.

An engine scope the vocabulary does not name no longer costs the whole batch. The rows the vocabulary does name are stored, and the scope that could not be placed is said on the run as a report-scope warning naming it, so the loss is visible on the screen rather than only in a log.

Proven by the reporting suites: a batch with one known and one unknown scope stores two rows, the curve and the warning, and recording the same errors twice stores one.

Emacs 29.3 (Org mode 9.6.15)