Skip to content

fix(analytics): share contract revision with generated gateway - #271

Merged
yaacovcorcos merged 2 commits into
mainfrom
codex/health-contract-revision-20260909
Sep 9, 2026
Merged

fix(analytics): share contract revision with generated gateway#271
yaacovcorcos merged 2 commits into
mainfrom
codex/health-contract-revision-20260909

Conversation

@yaacovcorcos

@yaacovcorcos yaacovcorcos commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Move the existing analytics contract revision constant into wireContract.ts and re-export it through the existing contract API. This makes the constant available to the generated website gateway, allowing /health to report the producer revision without a duplicate hard-coded label.

No event payload, supported revision, consent, activation, notification, or public API behavior changes. Companion website PR: ScientFactory/ScientFactory-website#40.

Validation: unchanged conformance fixture and generated-source parity; focused contract/conformance tests and analytics typecheck. CI provides full repository qualification.

Documentation impact: None — internal constant ownership refactor with no user-facing behavior change.

Reviewed by GPT-6 in Codex desktop.

@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Sep 9, 2026
@yaacovcorcos
yaacovcorcos enabled auto-merge (squash) September 9, 2026 07:29
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

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.6 KiB +89 B (+0.6%) 15.1 KiB
Codex Thread snapshot wire 7.1 KiB 7.1 KiB −9 B (−0.1%) 7.3 KiB
Codex Live turn WebSocket wire 6.4 KiB 6.5 KiB +98 B (+1.5%) 7.8 KiB
Codex Live turn WebSocket decoded 56.2 KiB 57.0 KiB +822 B (+1.4%) 66.4 KiB
Codex Live turn messages 8 8 0 (0.0%) 21
Claude Total thread wire 13.5 KiB 13.6 KiB +89 B (+0.6%) 15.1 KiB
Claude Thread snapshot wire 7.1 KiB 7.1 KiB +4 B (+0.1%) 7.3 KiB
Claude Live turn WebSocket wire 6.4 KiB 6.5 KiB +85 B (+1.3%) 7.8 KiB
Claude Live turn WebSocket decoded 57.0 KiB 57.8 KiB +840 B (+1.4%) 66.4 KiB
Claude Live turn messages 8 8 0 (0.0%) 21

Baseline: 8ddf71c · PR result: 44a4960 · 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: 114.0 KiB
  • Claude decoded thread snapshot: 114.7 KiB

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

@yaacovcorcos
yaacovcorcos merged commit 083993b into main Sep 9, 2026
24 checks passed
@yaacovcorcos
yaacovcorcos deleted the codex/health-contract-revision-20260909 branch September 9, 2026 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant