Skip to content

refactor(L1): drop slow prefix from selected interval locals - #445

Closed
roger-bai-coinbase wants to merge 1 commit into
mainfrom
refactor/aggregate-verifier-interval-return-names
Closed

roger-bai-coinbase wants to merge 1 commit into
mainfrom
refactor/aggregate-verifier-interval-return-names

Conversation

@roger-bai-coinbase

Copy link
Copy Markdown
Contributor

Summary

  • Rename locals that hold _intervalsAt / intervalsForStartingBlock results to blockInterval and intermediateBlockInterval, since those values may be the fast cadence pair.
  • Refresh snapshots/semver-lock.json for the AggregateVerifier source hash.

Test plan

  • Confirm the rename is local-only (no ABI or behavior change)
  • CI: existing AggregateVerifier tests

Made with Cursor

The values from _intervalsAt may be the fast cadence pair, so naming them slow* is misleading.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@roger-bai-coinbase
roger-bai-coinbase deleted the refactor/aggregate-verifier-interval-return-names branch September 21, 2026 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants