docs(exceptions): propose 90% coverage for the print agent's Windows layer (EX-0016) #58

Open
james.maes wants to merge 4 commits from docs/ex-0016-print-agent-windows-coverage into main
pull from: docs/ex-0016-print-agent-windows-coverage
Owner

What

Adds EX-0016 to the exception register: QRun-IO/qqq-print-agent gates its Windows adapter assembly (Qqq.PrintAgent.Windows) at 90% line coverage for version 2.0, where K22-TEST-10 asks for 100% of UI-free application logic. Its Core, view-model and command-line assemblies stay at 100% and are not covered by the exception.

Why

The owner chose this target on 2026-11-10 while planning 2.0 and asked for it to be recorded as a formal exception rather than left as a silent deviation. The entry gives the reason, the risk, the compensating controls and an end date of 2027-05-10.

Status

Proposed. Merging this pull request is the approval; it is for the maintainer to merge or decline. The gate the entry refers to is on that repository's release/2.0 branch (build/Verify.ps1 with build/minimum-coverage.json, decision record docs/adr/0003-version-2.md).

Written with Claude Code (Claude Fable 5.2) under the owner's direction of 2026-11-10.

🤖 Generated with Claude Code

## What Adds EX-0016 to the exception register: `QRun-IO/qqq-print-agent` gates its Windows adapter assembly (`Qqq.PrintAgent.Windows`) at 90% line coverage for version 2.0, where `K22-TEST-10` asks for 100% of UI-free application logic. Its Core, view-model and command-line assemblies stay at 100% and are not covered by the exception. ## Why The owner chose this target on 2026-11-10 while planning 2.0 and asked for it to be recorded as a formal exception rather than left as a silent deviation. The entry gives the reason, the risk, the compensating controls and an end date of 2027-05-10. ## Status Proposed. Merging this pull request is the approval; it is for the maintainer to merge or decline. The gate the entry refers to is on that repository's `release/2.0` branch (`build/Verify.ps1` with `build/minimum-coverage.json`, decision record `docs/adr/0003-version-2.md`). Written with Claude Code (Claude Fable 5.2) under the owner's direction of 2026-11-10. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
james.maes added 1 commit 2026-11-10 18:15:07 +00:00
K22-TEST-10 asks for 100% line coverage of UI-free application logic.
Version 2.0 of QRun-IO/qqq-print-agent gates its Windows adapter layer at
90% and its Core, view-model and command-line assemblies at 100%. The
owner chose that target on 2026-11-10; this entry records the deviation,
its reason, risk, compensating controls and end date. It is proposed, not
approved: merging this pull request is the approval.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Signed-off-by: James Maes <james@kof22.com>
james.maes added 1 commit 2026-11-10 23:17:40 +00:00
james.maes added 1 commit 2026-11-11 01:29:22 +00:00
An independent review of the record found it wrong in four places:

- it said the assembly is gated at 90%, while the gate today is 73 and
  90 is the bar 2.0.0 must reach before release;
- it did not list K22-CODE-CS-11, which is not met in part;
- it stated two plans as the current state (exclusions in the Windows
  assembly, and verification as if it ran in CI);
- its expiry named the release the exception exists for.

The wording changes what is being accepted, so the status goes back to
proposed until the maintainer approves the corrected text. The evidence
link is pinned to a commit.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Signed-off-by: James Maes <james@kof22.com>
Author
Owner

Status after an independent review (2026-11-10).

  • Reviewed head cb3108f: 1 blocker, 3 should-fix, 4 nits. The blocker: the record said the assembly is "gated at 90%", while the gate today is 73 and 90 is the bar 2.0.0 must reach before release.
  • New head a3c1453 corrects the record: states the bar and the current minimum, lists K22-CODE-CS-11 as not met in part, marks two controls as plans, fixes the expiry, pins the evidence link to a commit.
  • Because the corrected text changes what is being accepted, the status is back to "Proposed, awaiting approval". The maintainer has not approved this wording. Not merged; it waits for that approval and for the reviewer to confirm the new head.
Status after an independent review (2026-11-10). - Reviewed head `cb3108f`: 1 blocker, 3 should-fix, 4 nits. The blocker: the record said the assembly is "gated at 90%", while the gate today is 73 and 90 is the bar 2.0.0 must reach before release. - New head `a3c1453` corrects the record: states the bar and the current minimum, lists `K22-CODE-CS-11` as not met in part, marks two controls as plans, fixes the expiry, pins the evidence link to a commit. - Because the corrected text changes what is being accepted, the status is back to "Proposed, awaiting approval". The maintainer has not approved this wording. Not merged; it waits for that approval and for the reviewer to confirm the new head.
james.maes added 1 commit 2026-11-11 01:32:11 +00:00
The confirmation review could not tie "the owner made a check on real
printers a condition" to anything written down. The record now states
the condition itself, so approving the record approves the condition.
Figures are tied to the cited commit instead of a local date, and the
index row separates its two requirements with a comma like the others.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Signed-off-by: James Maes <james@kof22.com>
You are not authorized to merge this pull request.
This pull request can be merged automatically.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin docs/ex-0016-print-agent-windows-coverage:docs/ex-0016-print-agent-windows-coverage
git checkout docs/ex-0016-print-agent-windows-coverage
Sign in to join this conversation.