CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

test(gate): pin cross-implementation eventId vectors against GateTest #5625

MergedXSccantynz wants to mergetest/gatetest-eventid-vectorsmainopened 20h ago1/1 tasks
ccantynzcommented 20h ago

Summary

  • GateTest independently derives the same eventId for legacy bodies that arrive without one (gatetest@0737dc32). Three vectors were confirmed byte-identical across both implementations on 2026-09-02.
  • Pinned as literals: drift on either side now fails a suite instead of silently double-scanning legacy retries. The test carries a do-not-"fix" note pointing at crclabs-hq/GateTest for coordination.

Test plan

  • gatetest-base-sha.test.ts — 13 pass (3 new vector tests)
ccantynzAI Reviewcommented 20h ago

AI Triage

(no summary)

Priority: medium Risk area: mixed

Suggested labels: (no label suggestions) Suggested reviewers: (no reviewer suggestions)

Suggestions only — nothing has been applied. The PR author stays in control.

gluecron[bot]🤖 botAI Reviewcommented 20h ago

AI review unavailable

The platform's AI balance is exhausted, so AI generation is temporarily unavailable. Nothing was lost. You can queue this as a repair for the internal agent from the repository's Health page, or try again once the balance is restored. The PR is otherwise unchanged.

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
⮌ Merged

This pull request was merged into main.

c comment · e edit title · m merge · a approve · r request changes · ? shortcuts