docs: record CTO approval of ADR-0002 platform domain boundaries

[ja573]

Summary

Records the CTO's approval of ADR-0002 - Distribution and Metrics Platform Domain Boundaries exactly as written, and reconciles the engineering-control, Publisher Services and Metrics control records with that approval. Documentation and control-record changes only — no runtime effect.

CTO decision:

I approve ADR-0002 - Distribution and Metrics Platform Domain Boundaries as written.
Do not amend the architectural decision.

Task specification: docs/engineering/ai-delivery/tasks/ADR-0002-APPROVE.md

Scope

• ADR-0002 status → APPROVED (approval metadata only; architectural decision body unchanged)
• Reconcile the engineering decision register, engineering README and CG-06 control gap
• Reconcile Publisher Services README, decisions, task-status and rollout-plan
• Reconcile Metrics README, decisions and task-status
• Bounded Changed CHANGELOG entry
• Implementation report with issue Publisher Services: packages, distribution configuration and controlled rollout #765/Thoth Metrics: canonical ingestion, Sphinx orchestration and client cutover #766 baselines, hashes and proposed post-merge issue bodies (issues are not written by this task)

Invariants preserved

ADR-0001: PROPOSED
ADR-0002: APPROVED only after this PR merges
Publisher Services ADR-01: unapproved
Platform inventory: FINAL ENUM NOT APPROVED
MET-CTRL-01: CHANGES REQUIRED
All Publisher Services implementation tasks: BLOCKED
All Metrics work packages: BLOCKED
No shared platform abstraction or mapping
No runtime effect

Approving ADR-0002 removes one dependency. It does not by itself make ADR-01, BE-02, Metrics WP1 or any other task ready.

Review

Draft until an independent, non-implementing reviewer with high reasoning returns APPROVED. The implementing context may not approve or merge this PR, and may not edit issue #765 or #766.

🤖 Generated with Claude Code

[ja573]

Independent review — APPROVED

Reviewed head: 78307578f680050581f1a4e16a9668d9dfcc037a

Decision: APPROVED

Independence attestation: The reviewer acted solely as an independent reviewer and did not modify the branch, create or amend commits, edit the PR, edit issues #765/#766, mark the PR ready or merge it. Conclusions were independently verified against the repository, GitHub CI and the live issues rather than accepted from the implementing-agent handoff.

Findings:

• P0: NONE
• P1: NONE
• P2: NONE

The earlier pre-review evidence findings were resolved by correction commit 78307578f680050581f1a4e16a9668d9dfcc037a, which changes only the implementation report.

CI: PASS at the exact reviewed head.

• 30342715466 — build-test-and-check
  • build: success
  • lint: success
  • test: success
  • format_check: success
• 30342715472 — run-migrations
  • run_migrations: success
• 30342715538 — check-changelog
  • check-changelog: success
• 30342715481 — publish-to-dockerhub
  • build_and_push_staging_docker_image: success

ADR integrity: PASS. ADR-0002 changes only from PROPOSED to APPROVED and records the CTO approver, approval date and approval note. Sections 1–9 remain unchanged. The separate DistributionPlatform and MetricPlatform domains, absence of name-based conversion, distribution-only OAPEN/DOAB linkage and absence of an initial cross-domain mapping remain intact.

Control records: PASS. ADR-0001 remains PROPOSED; Publisher Services ADR-01 and the final platform inventory remain unapproved; MET-CTRL-01 remains CHANGES REQUIRED; all Publisher Services implementation tasks and Metrics work packages remain BLOCKED; no task becomes READY.

Issues: PASS. Issues #765 and #766 remained open and unchanged during this task. Their proposed synchronization is recorded but not authorized by this review or merge.

Runtime, migration and authorization effects: NONE. The cumulative diff contains exactly 14 allowlisted documentation/control files and no Rust, SQL, migration, GraphQL, workflow, generated-code, deployment or repository-protection changes.

Merge recommendation: Eligible to merge as a documentation/control-record-only change. No unresolved P0, P1 or P2 finding remains.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 78307578f6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".