fix(ci): accept V2 transfer reports without cross-scenario comparisons

[juliusmarminge]

The trusted transfer-report publisher rejects V2's rebuilt measurement scenario. Accept its fixed scenario ID and keep V1/V2 results separate: no impact percentages or ceiling-change claims across different fixtures.

All seven publisher tests pass, including V2 validation, incompatible-baseline rendering, and rejection of unknown scenarios. The measurement itself remains on the V2 branch. No UI change.

Model: GPT-6. Harness: Codex.

---

Summary by CodeRabbit

• Bug Fixes

  • Reports now support both thread-transfer scenario versions.
  • Ceiling-change notices are shown only when baseline comparisons are valid.
  • Version 2 reports no longer display misleading ceiling-change or percentage-impact messages.
  • Unsupported scenario identifiers continue to be rejected.

• Tests

  • Added coverage for version 2 validation and reporting behavior.

[macroscopeapp]

Approvability

Verdict: Approved at 3d2a420

Macroscope's review found this PR approvable — This is a narrowly scoped CI reporting fix that accepts the V2 artifact format and prevents misleading comparisons across different fixtures. The changes are confined to the trusted workflow harness, preserve existing validation and ceiling enforcement, and include focused regression coverage.

^{You can add or adjust custom eligibility rules. Learn more.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.5 KiB	13.5 KiB	+34 B (+0.2%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.1 KiB	+10 B (+0.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.5 KiB	+24 B (+0.4%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	56.2 KiB	56.3 KiB	+44 B (+0.1%)	66.4 KiB	✅
Codex	Live turn messages	9	10	+1 (+11.1%)	21	✅
Claude	Total thread wire	13.5 KiB	13.5 KiB	+11 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.1 KiB	7.1 KiB	+6 B (+0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+5 B (+0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.0 KiB	57.0 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	9	9	0 (0.0%)	21	✅

Baseline: c03b240 · PR result: 3d2a420 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.9 KiB
• Claude decoded thread snapshot: 114.6 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 3cca74de-0d61-499b-870f-e587f801303b

📥 Commits

Reviewing files that changed from the base of the PR and between c03b240 and 3d2a420.

📒 Files selected for processing (2)

• .github/scripts/thread-transfer-report.cjs
• .github/scripts/thread-transfer-report.test.cjs

Limit details: You’ve used all 10 included reviews currently available.

---

📝 Walkthrough

Walkthrough

The report validator now accepts thread-transfer-v2. The renderer compares baseline ceilings only when scenarios match. Tests cover independent V2 validation, V2 output, and unsupported scenarios.

Changes

Thread transfer scenario handling

Layer / File(s)	Summary
Scenario validation and reporting .github/scripts/thread-transfer-report.cjs, .github/scripts/thread-transfer-report.test.cjs	validateResult accepts V1 and V2 scenario IDs. renderComment reports ceiling changes only for matching scenarios. Tests verify V2 behavior and reject unsupported IDs.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 3d2a4

The V1/V2 report separation behaves as intended for reachable production inputs, so the change is mergeable.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: accepting V2 transfer reports without cross-scenario comparisons.
Description check	✅ Passed	The description explains what changed, why it changed, test coverage, and that there are no UI changes. It does not include the template headings or checklist, but the core information is complete.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

Usage-based review receipt

• Mode: Continue automatically
• Reviewed files: 2
• Charged: $0.50
• View usage details

Note

This review was completed with usage-based billing: files reviewed beyond your plan's included limits are billed at $0.25/file. View usage-based billing.

---

_{Comment @coderabbitai help to get the list of available commands.}