Skip to content

M0/M1: Add the orchestration retained-result foundation - #1510

Draft
Paul Lizer (paullizer) wants to merge 2 commits into
microsoft:paullizer-react-v2-uifrom
paullizer:paullizer-harness-result-foundation
Draft

Paul Lizer (paullizer) wants to merge 2 commits into
microsoft:paullizer-react-v2-uifrom
paullizer:paullizer-harness-result-foundation

Conversation

@paullizer

Copy link
Copy Markdown
Collaborator

Scope

Refs #1509. M0 + M1 only: executable contracts and private retained-result infrastructure. This does not finish the Gather / Reason / Render epic or activate file rendering, new runtime scheduling, UI, or retries.

Intended base: paullizer-react-v2-ui at 1f6b444c1db3d97aaf0b6e004eed6345c6a859be. The implementation branch was explicitly fast-forwarded to that published prerequisite before any edits; Development is not the review baseline.

Changes

  • Add strict versioned task/result/completeness/named-binding contracts, immutable path-free result references, and ordered record schemas. Gather / Reason / Render are purpose labels, not a new stage sorter.
  • Add an injected orchestration-owned facade for private persistence, full readers, bounded previews, source/upstream lineage, current access/screening checks, producer/attempt validation, and fail-closed binding resolution.
  • Reuse the existing result transport, bounded record trees, and cancellation/deletion fences through three additive generic orchestration commit methods. No new Cosmos/Blob client or fabricated workflow.
  • Preserve real saved Analyze readers and the existing exact_records_v1 JSON protocol without treating prose/Compare as analyze-final-v1.
  • Document foundation interfaces and deferred milestones; bump application version from 0.261.124 to 0.261.125.

Production v1 plan/executor/RunContext/checkpoint behavior, existing fingerprints, standalone tools, workflows, rendering implementations, requirements, deployers, and release notes are unchanged. There are no new settings, routes, or user-facing controls.

Validation

  • New contracts/facade/cold-import suites: 115 passed, including fresh normal/optimized real-module imports, separate web/scheduler bootstraps, and blocked network I/O.
  • Existing Analyze storage/source access, orchestration integration/recovery/access, and saved-record export suites: 164 passed + 309 subtests.
  • Existing v1 plan-schema standalone runner: 6/6 passed; not counted from pytest boolean-return behavior.
  • Documentation inventory coverage: 7/7; documentation source-quality checks: 6/6 (no built-site claim).
  • Broken-access-control and XSS guardrails passed for all three result modules; staged whitespace check passed.

Coverage includes two complete consumers after persistence/restart, a 30,000-row result exceeding 8 MiB, first/last records, Unicode/ordered schemas, pending/partial/invalid/unavailable states, source revocation/change/screening holds, exact lineage snapshots, corrupt counts/digests, missing final records, fake storage references, cross-attempt binding, cleanup/cancellation races, and zero facade file publication. One existing Pydantic deprecation warning remains. External storage/provider I/O is doubled; no live cloud/model work was performed.

Commit: 18fc08ac9dc21066a324afbdb499de523717be65.

Implement M0/M1 typed named outputs, strict completeness and bindings, authorized private retained-result readers, and lifecycle-fenced commits. Preserve production v1 execution, saved Analyze, workflows, and exact_records_v1 JSON behavior. Document foundation-only scope at version 0.261.125.

Refs microsoft#1509

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@paullizer
Paul Lizer (paullizer) changed the base branch from Development to paullizer-react-v2-ui September 21, 2026 16:16
Comment on lines +18 to +34
from functions_orchestration_result_contracts import (
Completeness,
Coverage,
InputBinding,
InputSpec,
OutputSpec,
ProducerIdentity,
RecordColumn,
ResultContractError,
ResultRef,
StepBindings,
TaskResult,
canonical_digest,
validate_input_bindings,
validate_json,
validate_record,
)
Preserve resumed native attempt guards, atomically bind committed results to exact producer/input receipts, and admit bounded external provenance with current access callbacks. Preserve v1 result bytes and checkpoint fingerprints; leave version ownership and runtime/export integration to the parent.

Refs microsoft#1509

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants