FE-1251: Publish traceable comparison reports

[kostandinang]

Why

Comparison reports need to retain the release and controller commit actually tested, even when publication happens from a later checkout.

What

Capture write-once provenance after setup approval and before the first lane, then publish retained validity-first reports through an explicit guarded Notion workflow. Publication upserts by Run ID + Phase, preserves redaction boundaries, and verifies every mutation.

How to test

• npm test -- src/dev/comparison-provenance.test.ts
• npm run check:skills
• npm run verify
• Create, update, and duplicate-detection smoke exercise against the Comparison Reports database

[kostandinang]

• FE-1251: Publish traceable comparison reports #364 👈 (View in Graphite)
• next

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

PR Summary

Medium Risk
Touches external Notion mutation and attribution of published results, but stays dev-only with strict validation, no overwrite of provenance, and verified idempotent upsert semantics.

Overview
Comparison runs can now freeze controller and release identity at approval time instead of inferring it from whatever checkout publishes the report later.

A new dev CLI (src/dev/comparison-provenance.ts) writes collision-failing provenance.json (package version, exact tag, full commit URL, branch, dirty flag, run id, kind) after setup approval and before the first lane. Elicitation (/compare-specs), execution (/compare-execution), and end-to-end flows are updated to require that capture; promotion must carry the snapshot unchanged.

Publication is explicit via the new .agents/skills/comparison-publish skill (/comparison-publish <run-directory>): validate report.md + provenance + kind-specific contracts, redact controller-only material, map live Notion properties, create or update exactly one row keyed by Run ID + Phase, stop on duplicate matches, and re-fetch to verify. check:reporting-skills now enforces the third skill contract; docs/praxis/comparison-runs.md and memory/PLAN.md document FE-1251.

^{Reviewed by Cursor Bugbot for commit f6591b1. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit f6591b1. Configure here.}