From 13a6b60c2100eae26486732d5da6912646f1c509 Mon Sep 17 00:00:00 2001 From: m3taversal Date: Mon, 13 Apr 2026 10:33:12 +0200 Subject: [PATCH] add patch audit and orphan inventory to consolidation diff log 7 root patch scripts audited (2 applied, 2 superseded, 3 abandoned). All 7 safe to delete. 2 root orphans (extract.py, cascade.py) confirmed stale vs canonical copies. Argus's 8 diagnostics patches listed but not audited (his responsibility). Co-Authored-By: Claude Opus 4.6 (1M context) Pentagon-Agent: Epimetheus <0144398E-4ED3-4FE2-95A3-3D72E1ABF887> --- ops/diagnostics/CONSOLIDATION-DIFF-LOG.md | 92 +++++++++++++++++++++++ 1 file changed, 92 insertions(+) diff --git a/ops/diagnostics/CONSOLIDATION-DIFF-LOG.md b/ops/diagnostics/CONSOLIDATION-DIFF-LOG.md index a420a386b..2457d51a5 100644 --- a/ops/diagnostics/CONSOLIDATION-DIFF-LOG.md +++ b/ops/diagnostics/CONSOLIDATION-DIFF-LOG.md @@ -45,3 +45,95 @@ All in /opt/teleo-eval/diagnostics/. Git is the backup now. Safe to delete after ## VPS deploy needed after merge alerting.py, alerting_routes.py, vitality.py, vitality_routes.py — all local versions are newer than VPS. + +--- + +## Root Patch Script Audit (Epimetheus's 7 patches) + +### patch-prompt-version.py — APPLIED +- **Target:** db.py, merge.py, extract.py, extraction_prompt.py +- **What:** Schema v17 migration for prompt_version/pipeline_version columns, version stamping on PR discovery, feedback param for re-extraction +- **Status:** All 4 targets have changes. Schema is at v19 (includes this migration). merge.py stamps versions. extract.py has feedback param. extraction_prompt.py has previous_feedback. +- **Action:** SAFE TO DELETE + +### tmp-patch-research-state.py — APPLIED +- **Target:** research-session.sh +- **What:** Integrates agent-state hooks (state_start_session, state_update_report, state_journal_append) +- **Status:** All hooks present in research-session.sh (STATE_LIB sourcing, HAS_STATE init, session lifecycle calls) +- **Action:** SAFE TO DELETE + +### patch-dashboard-cost.py — STALE (superseded) +- **Target:** dashboard_routes.py +- **What:** Adds per-PR cost queries via audit_log (cost_map, triage_cost_map) +- **Status:** Cost tracking implemented differently in current codebase — uses `costs` table and p.cost_usd column, not audit_log aggregation. Patch logic abandoned in favor of newer approach. +- **Action:** SAFE TO DELETE (superseded by different implementation) + +### patch-dashboard-prs-cost.py — STALE (superseded) +- **Target:** dashboard_prs.py +- **What:** Adds Cost column header, fmtCost() function, cost cell in row template +- **Status:** Cost KPI card exists (line 101) but implemented as card-based KPI, not table column. fmtCost() not present. Different UI approach than patch intended. +- **Action:** SAFE TO DELETE (superseded by card-based cost display) + +### patch-cost-per-pr.py — NOT APPLIED +- **Target:** evaluate.py +- **What:** Adds _estimate_cost() helper function, cost instrumentation to audit events (haiku_triage, domain_rejected, approved, changes_requested) +- **Status:** _estimate_cost not found in evaluate.py. No cost fields in audit events. eval_checks.py has its own estimate_cost but for bot responses, not pipeline eval. +- **Action:** SAFE TO DELETE — eval_checks.py already has cost estimation for its own use case. The pipeline eval cost tracking was a different approach that was never completed. + +### patch-dashboard-prs-version.py — NOT APPLIED +- **Target:** dashboard_prs.py +- **What:** Adds version badges (prompt_version, pipeline_version) to eval chain section and agent cell +- **Status:** No version badges in dashboard_prs.py. prompt_version/pipeline_version not displayed anywhere. +- **Action:** SAFE TO DELETE — version columns exist in schema (v17 migration) but UI display was never built. Low priority feature, can be re-implemented from schema when needed. + +### patch-dashboard-version.py — NOT APPLIED +- **Target:** dashboard_routes.py, shared_ui.py +- **What:** Adds prompt_version/pipeline_version to SELECT query, version badges to shared_ui +- **Status:** Version fields not in SELECT. shared_ui.py exists but without version display. +- **Action:** SAFE TO DELETE — same reasoning as patch-dashboard-prs-version.py. + +### Summary + +| Script | Status | Action | +|--------|--------|--------| +| patch-prompt-version.py | APPLIED | Delete | +| tmp-patch-research-state.py | APPLIED | Delete | +| patch-dashboard-cost.py | STALE (superseded) | Delete | +| patch-dashboard-prs-cost.py | STALE (superseded) | Delete | +| patch-cost-per-pr.py | NOT APPLIED (abandoned) | Delete | +| patch-dashboard-prs-version.py | NOT APPLIED (low priority) | Delete | +| patch-dashboard-version.py | NOT APPLIED (low priority) | Delete | + +All 7 safe to delete. 2 were applied, 2 were superseded by different implementations, 3 were never applied but the features either exist differently or are low priority. + +--- + +## Root Orphan Files + +### extract.py (693 lines) +- **Location:** Pentagon workspace root +- **Canonical:** teleo-codex/ops/pipeline-v2/openrouter-extract-v2.py (Apr 7+) +- **Status:** Older draft (Apr 1). Confirmed by Cory as safe to delete. +- **Action:** DELETE + +### cascade.py (274 lines) +- **Location:** Pentagon workspace root +- **Canonical:** teleo-codex/ops/pipeline-v2/lib/cascade.py (10372 bytes, Apr 13) +- **Status:** Older draft. Confirmed by Cory as safe to delete. +- **Action:** DELETE + +--- + +## Argus's Patch Scripts (in root diagnostics/) + +8 patch scripts owned by Argus — audit responsibility is Argus's: +- diagnostics/compute_profile_patch.py +- diagnostics/dashboard_compute_patch.py +- diagnostics/patch_4page.py +- diagnostics/patch_dashboard_tokens.py +- diagnostics/patch_evaluate_costs.py +- diagnostics/patch_llm_cli.py +- diagnostics/patch_prs_page.py +- diagnostics/patch_vps_app.py + +These remain in root diagnostics/ until Argus completes his audit.