Skip to content

fix: preserve reporting acceptance and submission evidence - #272

Merged
pcvantol merged 6 commits into
mainfrom
codex/r270-reporting-integrity
Sep 17, 2026
Merged

pcvantol merged 6 commits into
mainfrom
codex/r270-reporting-integrity

Conversation

@pcvantol

@pcvantol pcvantol commented Sep 17, 2026

Copy link
Copy Markdown
Owner

Scope

Closes only R270-1 and R270-2. This is intentionally stacked on PR #271 at 1c30ccf; it does not duplicate or replace the telemetry/export implementation. No version or updater change is included.

R270-1

  • removes prompt-keyword-driven YES/PASS/GO conclusions
  • projects technical delivery, EP Run Qualification, and Mission/autonomy acceptance as separate facts
  • keeps COMPLETE + NOT_QUALIFIED valid and visible
  • keeps Quality/Security PASS scoped to the reviewed candidate
  • aligns readable report, Engineering Evidence Summary, receipt and consistency validation
  • fails closed on any unscoped YES/PASS/GO in Deliverable Answer and on an absent machine-readable Evidence Summary

R270-2

  • adds typed submission-evidence states: valid modern, legacy absent, missing modern binding, storage unavailable, corrupt, and identity conflict
  • permits prompt fallback only for positively established legacy absence
  • preserves and cross-validates retry attempt, canonical root, retry parent and qualification lineage identities
  • accepts historical parent-without-attempt compatibility only for an explicit RETRIED dispatch with matching run, resolution and producer correlation
  • stops report publication on modern integrity/read failures using safe diagnostic codes
  • passes one resolved submission snapshot through report consumers instead of rereading and hiding later failures

Qualification — candidate b5c8967e6517c70ba691ef9c850cc27711bc304b

  • focused ReportingIntegrity regressions: 14 passed
  • reporting + retry lifecycle: 45 passed
  • existing reporting/contract subset: 299 passed, 1 skipped
  • managed-autonomy recovery: 22 passed
  • full combined source suite: 1,881 passed, 2 skipped
  • independent read-only review: no findings after two reproduced gaps were fixed and re-reviewed
  • Bandit high-severity scan: PASS
  • bytecode compile and git diff --check: PASS

Coordination

The base is PR #271 so this PR contains only the R270-owned reporting/storage/test/documentation delta. After #271 merges, this branch can be retargeted to main and requalified on the resulting exact head. No release number is allocated here, preventing a competing 2.3.80 artifact or updater operation.

Safety

No Mission, provider run, retry/resume, operational state mutation, historical evidence rewrite, SQL repair, version change, release, or installation is performed.

pcvantol added a commit that referenced this pull request Sep 17, 2026
…UMP) (#273)

Owner-requested companion to Forge #133 (merged 03f9514e0f1e66a79ccb684c9fdbe51b28b88090) and Workspace #34. Seven PLANNED EP nodes, typed operation authority/identity, local Genesis, Managed birth/adoption/readback and history-preserving promotion. Exact candidate validation/trusted-delivery evidence, CodeQL, security, Golden Qualification and TDE observe succeeded. Existing documentation-profile browser/localization skips retained; no new runtime qualification claimed. No code, schema, workflow, version, grant, installation or Mission change; #271/#272 left untouched.
Base automatically changed from codex/r268-telemetry-export-followup to main September 17, 2026 13:42
…integrity

# Conflicts:
#	docs/engineering/PROVIDER_USAGE_SEMANTICS.md
#	docs/engineering/examples/telemetry-followup/telemetry-detail-example.json
#	docs/engineering/examples/telemetry-followup/telemetry-detail-example.md
#	docs/engineering/examples/telemetry-followup/telemetry-overview-example.json
#	docs/engineering/examples/telemetry-followup/telemetry-overview-example.md
#	src/engineering_platform/assets/dashboard.js
#	src/engineering_platform/execution_timing.py
#	src/engineering_platform/provider_usage.py
#	src/engineering_platform/server.py
#	src/engineering_platform/telemetry_contract.py
#	src/engineering_platform/telemetry_export.py
#	src/engineering_platform/telemetry_metrics.py
#	tests/engineering/dashboard.spec.mjs
#	tests/engineering/test_server_foundation.py
#	tests/engineering/test_telemetry_contract_v2.py
@pcvantol
pcvantol merged commit f45694d into main Sep 17, 2026
18 checks passed
@pcvantol
pcvantol deleted the codex/r270-reporting-integrity branch September 17, 2026 13:53
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.

1 participant