Skip to content

Fix collection and publication integrity; remove coverage accounting - #70

Merged
jrh-mann merged 4 commits into
claude/integratefrom
codex/sweep-findings
Oct 23, 2026
Merged

jrh-mann merged 4 commits into
claude/integratefrom
codex/sweep-findings

Conversation

@jrh-mann

Copy link
Copy Markdown
Contributor

Changes

  • Remove required_coverage, the separate coverage.json completion gate, and the staged coverage exporter. Retain per-cell missing/unresolved work and question-label validation.
  • Mark a child collected only after its verdict import and findings sync succeed.
  • Commit publication state after validation, rendering and persistence complete.
  • Refuse duplicate sample references; show benchmark-wide findings without a false 0/N count.
  • Replace exact LaTeX template placeholders once, preserving brackets in findings and evidence.
  • Reject non-finite model prices.
  • Untrack docs/ and ignore it, retaining local files, per James's instruction.

Validation

58 focused tests passed; 6 excluded. No local Docker, TeX, browser publishing, paid calls or Hawk submissions. Commit hooks pass. Rebased onto 7dadabe.

Coordination

Targets claude/integrate, not main. Claude retains ownership of dispatch/local.py package fidelity (Job.package and extra_packages currently ignored), the task split, and integrated CI. This PR intentionally avoids that work. A local ownership note is at /Users/james/Documents/generality/inspect-audit-coordination.md; PR #69 comments are the notification channel.

LaTeX graph components and moving benchmark enumeration out of the auditor host remain separate architecture work, not hidden in this fix.

@jrh-mann
jrh-mann merged commit 53d4fc1 into claude/integrate Oct 23, 2026
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