[codex] Normalize verifier telemetry contracts

[stranske]

Source: Issue #1836

Related to campaign issue #1836

Automated Status Summary

Scope

Sync/Dependabot Campaign Queue

Remote GitHub Actions owns discovery for sync-generated and Dependabot PR rounds. Local Codex should only claim items from this issue when needs-local-codex work is queued.

Summary

Context for Agent

Related Issues/PRs

• stranske/Template#625
• stranske/Template#623
• stranske/Template#622
• stranske/Template#619
• stranske/Template#617
• stranske/Template#616
• stranske/Template#615
• stranske/Template#614
• stranske/Template#613
• stranske/Template#612
• stranske/Travel-Plan-Permission#931
• stranske/Workflows#1911

Tasks

• Updated: 2026-04-26T21:19:27.410Z
• Repos checked: 11/11
• Open sync PRs: 598
• Open Dependabot PRs: 0
• Active review threads queued: 1162
• Items needing local Codex: 0
• Actionable local Codex items: 0
• Claimable local Codex items: 0
• Source-fixed candidates: 0
• Superseded sync candidates: 120
• Finished local results without published source changes: 0
• Claimed local Codex items: 0
• Next claim lease expires: -

Acceptance criteria

• Status: local-codex-superseded-sync-candidate
• Source repo: stranske/Workflows
• Source sync state: superseded (PR 8c201f72e3ed / current d663bb30e7f5)
  • .github/scripts/terminal_disposition_coverage.js:178 (copilot-pull-request-reviewer): verifier_mode is cleaned to an empty string when absent, but requiresCodexModel now treats empty/unknown as requiring...
  • scripts/aggregate_agent_metrics.py:571 (copilot-pull-request-reviewer): verifier_mode is normalized to an empty string when missing, but this condition now treats the empty value as "not ev...

Head SHA: 83b09b9
Latest Runs: ⏹️ cancelled — Gate
Required: gate: ⏹️ cancelled

Workflow / Job	Result	Logs
Agents Auto-Pilot	⏭️ skipped	View run
Agents Bot Comment Handler	✅ success	View run
Agents Keepalive Loop	⏹️ cancelled	View run
Agents Verifier	✅ success	View run
Auto-label Dependabot PRs	⏭️ skipped	View run
CI Autofix Loop	⏹️ cancelled	View run
Copilot code review	✅ success	View run
Create Issue from Verification (Enhanced)	⏭️ skipped	View run
Create New PR from Verification	⏭️ skipped	View run
Gate	⏹️ cancelled	View run
Health 40 Sweep	✅ success	View run
Health 44 Gate Branch Protection	✅ success	View run
Health 45 Agents Guard	⏹️ cancelled	View run
Health 50 Security Scan	✅ success	View run
Health 72 Template Sync	✅ success	View run
Maint 52 Validate Workflows	✅ success	View run
PR 11 - Minimal invariant CI	✅ success	View run
Selftest CI	✅ success	View run
Validate Sync Manifest	✅ success	View run

[agents-workflows-bot]

Automated Status Summary

Head SHA: 90f51bb
Latest Runs: ⏳ pending — Gate
Required contexts: Gate / gate, Health 45 Agents Guard / guard
Required: core tests (3.12): ⏳ pending, core tests (3.13): ⏳ pending, docker smoke: ⏳ pending, gate: ⏳ pending

Workflow / Job	Result	Logs
(no jobs reported)	⏳ pending	—

Coverage Overview

• Coverage history entries: 1

Coverage Trend

Metric	Value
Current	93.12%
Baseline	85.00%
Delta	+8.12%
Minimum	70.00%
Status	✅ Pass

Top Coverage Hotspots (lowest coverage)

File	Coverage	Missing
src/cli_parser.py	81.8%	4
src/percentile_calculator.py	95.0%	1
src/aggregator.py	95.0%	2
src/__init__.py	100.0%	0
src/ndjson_parser.py	100.0%	0

Updated automatically; will refresh on subsequent CI/Docker completions.

---

Keepalive checklist

Scope

No scope information available

Tasks

• No tasks defined

Acceptance criteria

• No acceptance criteria defined

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 83b09b90d6

ℹ️ 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".

[chatgpt-codex-connector]

Count explicit unknown modes in unknown-mode metric

The new unknown-mode counter only increments when verifier_mode is blank, so records that explicitly set verifier_mode to 'unknown' are added to missing_model_records but not included in missing_model_unknown_mode_record_count. That creates an inconsistent machine-readable contract (and markdown summary) for the exact category this field is meant to report, which can undercount unknown-mode gaps in post-contract enforcement windows.

Useful? React with 👍 / 👎.