Skip to content

docs(risk-register): strategic-review curation + MetricFrame integrity cluster - #24

Merged
Polichinel merged 1 commit into
developmentfrom
docs/risk-register-curation
Jun 26, 2026
Merged

docs(risk-register): strategic-review curation + MetricFrame integrity cluster#24
Polichinel merged 1 commit into
developmentfrom
docs/risk-register-curation

Conversation

@Polichinel

Copy link
Copy Markdown
Collaborator

Curation of the technical risk register from a /review-rr strategic pass (no code change).

Changes

  • Demote C-21 (stale EvaluationManager docstring) and C-23 (vestigial adapters/ package) to the tech-debt backlog (technical_debt_backlog.md §5) — Tier-4, no risk dimension.
  • Re-tier C-22 (silent Pearson nan) 4 → 3 (reachable in normal use).
  • Register the MetricFrame emit-artifact blind spots as new Cluster D: C-24 (cross-repo contract drift, Tier 2), C-25 (provenance version-label integrity, Tier 3), C-26 (float32 cast + unenforced schema_version, Tier 3).
  • Document causal clusters A–D; sharpen C-05 trigger (action-based); flag the C-11/C-14 version-bump regression (reverted to 0.4.0 via PR fix wrong version number #20; tracked in Release views-evaluation (cut 0.5.0) shipping to_metric_frame — the evaluation-of-record emit #23).
  • Structural: remove duplicated C-10 from Open Concerns, normalize separators, add a Demoted Concerns table (IDs retired, never reused).

Notes

  • First commit of technical_risk_register.md — it was previously gitignored (reports/) and maintained locally; force-added here so the curated register is tracked. (The backlog was already tracked.)
  • Docs-only. ruff clean; 281 tests pass. Triage verdict: CLEAN (counts, IDs, cross-refs all consistent).

🤖 Generated with Claude Code

…y cluster

Curate the technical risk register per the 2026-06-26 strategic review:
- Demote C-21 (stale EvaluationManager docstring) and C-23 (vestigial adapters
  package) to the tech-debt backlog (§5) — Tier-4 items with no risk dimension.
- Re-tier C-22 (silent Pearson nan) 4 -> 3 (reachable in normal use).
- Register the MetricFrame emit-artifact blind spots as a new causal cluster D:
  C-24 (cross-repo contract drift, Tier 2), C-25 (provenance version-label
  integrity, Tier 3), C-26 (float32 + unenforced schema_version, Tier 3).
- Document causal clusters A-D; sharpen C-05 trigger; flag the C-11/C-14
  version-bump regression (reverted to 0.4.0, PR #20; tracked in #23).
- Structural: remove duplicated C-10 from Open Concerns, normalize separators,
  add Demoted Concerns table (IDs retired, never reused).

First commit of technical_risk_register.md (previously gitignored, force-added).
No code change; ruff clean, 281 tests pass.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Polichinel
Polichinel merged commit 639807b into development Jun 26, 2026
4 checks passed
@Polichinel
Polichinel deleted the docs/risk-register-curation branch June 26, 2026 17:40
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