fix(hosted): harden versioned local run reporting#160
Merged
Conversation
Member
Author
|
Protected exact-head CI is green. Release remains intentionally held: the first real managed-run compatibility trial against Cloud deploy 6a5cc853 exposed PostgreSQL 42804 in Cloud's apply_run_callback (text status assigned to run_status enum). No workflow action or false success was recorded; the run remains queued and single-flight. Cloud's additive callback repair, deployment, stranded-run reconciliation, and a fresh 3/3 hosted trial matrix will complete before this PR is marked ready or Flow 1.18 is released. |
abrichr
force-pushed
the
fix/run-summary-egress-20260719
branch
from
July 19, 2026 17:04
31bd336 to
165d4d2
Compare
abrichr
marked this pull request as ready for review
July 19, 2026 20:06
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
Hardens the versioned local run and break reporting rails for the hosted
control plane while preserving the public Flow 1.18.0 caller and CLI surface.
Contract
run-summary/v2andbreak-summary/v2payloads.bundle digest; no names, URLs, parameters, screenshots, free-form report
content, or effect hashes.
report_run(..., deployment_kind=..., org_id=...)keyword inputs and
report-run --deployment-kind/--org-idflags asdeprecated compatibility no-ops. Authenticated server identity remains
authoritative and neither value enters the v2 payload.
outbound payloads at 16 KiB.
.report_run_idrace-safely with0600, exclusive creation, fsync,descriptor-based
O_NOFOLLOWwhere available, and descriptor/inode bindingon every platform before reading.
canonical run UUID, exact expected status, boolean
duplicatewhen present,and canonical halt/teach bindings for break responses. Malformed or
non-object JSON cannot become
emitted=true.Verification
29696104223passed every required job, including full unit/coverage, browser E2E, Windows
mock, Linux AT-SPI, Python compatibility, typing, PHI, and wheel validation.
Claims run
29696104222also passed.
an environment-sensitive ONNX/OCR test stalled; the run was stopped rather
than allowed to take over the machine indefinitely. The initial sandboxed
attempt reached 2,270 passing but its localhost-dependent tests were refused
by the execution sandbox, so neither partial run is represented as a full
suite pass.
checks pass.
reports one pre-existing
benchmark/dom_arm.pyerror on exactmain; theprotected PR matrix remains authoritative.
openIMIS, AGPL, or repository-only benchmark files.
Release sequencing
Flow 1.18.0 is already public. This PR must therefore release, if accepted,
under the next patch or minor version selected by the semantic-release result;
it does not pre-assign that version.
The completed managed-production qualification used the immutable Flow 1.17.1
/ Modal 1.5.2
report-v2managed runner. That qualification validates themanaged callback/report path, but it does not directly qualify these local
run-summary/v2/break-summary/v2reporting rails. Their evidence is thisPR's paired Cloud contract, adversarial tests, protected CI, and a separately
scoped local-report qualification before release.