Skip to content

refactor(reconciliation): retire unconsumed publisher port - #73

Open
iperev wants to merge 1 commit into
fix/proof-integrity-residualsfrom
refactor/remove-unused-publisher
Open

iperev wants to merge 1 commit into
fix/proof-integrity-residualsfrom
refactor/remove-unused-publisher

Conversation

@iperev

@iperev iperev commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

Stacked PR 18 — base: #72 (fix/proof-integrity-residuals). Do not merge yet. Review only the top commit.

Context

CI-056 asks for unused abstractions to be checked against real consumers before removal. ReconciliationPublisher and its CheckSummary/summarize_check projection had no implementation, runtime wiring, provider adapter, requirement or mutation consumer. Only package exports and one unit test used them. The reconciliation owner doc already says the non-enforcing runtime has no provider success publisher.

What changed

  • Removed the port, the projection and the one test that used it.
  • Retired the check_summary.py proof owner under its exact baseline binding digest. REQ-CI-RUNTIME-003 keeps its live witnesses.
  • Regenerated the self-CI and Proofkit hash artifacts.
  • Recorded the H-20, H-21, H-22, H-23, H-25 and M-38 dispositions in the audit hardening plan and updated the CI-056 roadmap row.
  • Provisional owner decisions (2026-10-11): Q1 (a) remove both the port and the projection; Q2 (b) keep the 12 _exact_object copies (ABSTAIN).

Expected result

No runtime behavior changes. The reconciliation package no longer exposes a port for writing to the provider. Static, Proofkit, ownership and documentation gates pass.

Non-claims:

  • No native test or mutation result is claimed.
  • No module-ownership semantic verdict, test-duplication review or contract-size review is claimed.
  • No H-25 reachability result is claimed.
  • CI-056 stays open; closure needs merged, exact-source qualification.

Independent review: gpt-6-astra (xhigh); findings addressed or rejected with reasons in the single commit. Provisional owner decisions (2026-10-11) are labelled in the changed documents.

Remove ReconciliationPublisher and its CheckSummary projection, which had no
runtime, composition, provider, requirement or mutation consumer. Retire the
check_summary.py proof owner under its exact baseline binding digest, record
CI-056 abstraction dispositions and keep the _exact_object copies (ABSTAIN).

Provisional owner decisions (2026-10-11): Q1 (a) and Q2 (b).
@iperev
iperev force-pushed the refactor/remove-unused-publisher branch from b92f95d to 3b76155 Compare October 12, 2026 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant