Skip to content

test_run: carry the report's surface into the sidecar#86

Merged
Jammy2211 merged 1 commit into
mainfrom
feature/surface-recording
Jul 17, 2026
Merged

test_run: carry the report's surface into the sidecar#86
Jammy2211 merged 1 commit into
mainfrom
feature/surface-recording

Conversation

@Jammy2211

Copy link
Copy Markdown
Contributor

Summary

Consumer half of the evidence-chain audit's §5.3 fix (#83; campaign PyAutoBuild#155 Phase 2 satellite). PyAutoLabs/PyAutoHands#164 (merged) makes report.json state the surface it measured — projects, shards, run_types, env_profiles, script_count. This carries that block through into test_run.json, so the leg can state its own denominator rather than having counts compared across incomparable runs (the 07-15 "3 failed → 30 failed" false regression).

Pre-#164 reports have no surface → the field is None, never a crash.

Shipped under the standing corrective-PR exception (Heart RED). PR-open only; merge stays human.

API Changes

None — test_run.json gains an additive surface key.

Test Plan

  • Surface carried report → sidecar; absent-surface (old report) → None.
  • Full suite: 286 passed.

Generated by the PyAutoLabs agent workflow.

Consumer half of §5.3: PyAutoBuild#164 makes report.json state the surface it
measured; the leg now carries that block through to test_run.json, so the
verdict can cite its own denominator instead of comparing counts across
incomparable runs. Absent on pre-#164 reports -> None, never a crash.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@Jammy2211 Jammy2211 added the pending-release Merged changes awaiting the next release cycle label Jul 17, 2026
@Jammy2211
Jammy2211 merged commit 0470866 into main Jul 17, 2026
4 checks passed
@Jammy2211
Jammy2211 deleted the feature/surface-recording branch July 17, 2026 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pending-release Merged changes awaiting the next release cycle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant