Skip to content

apollo_consensus_orchestrator: rename snip35 module to dynamic_gas_price, scrub SNIP-35 comments#14216

Merged
sirandreww-starkware merged 1 commit into
mainfrom
05-27-apollo_consensus_orchestrator_rename_snip35_module_to_dynamic_gas_price_scrub_snip-35_comments
May 27, 2026
Merged

apollo_consensus_orchestrator: rename snip35 module to dynamic_gas_price, scrub SNIP-35 comments#14216
sirandreww-starkware merged 1 commit into
mainfrom
05-27-apollo_consensus_orchestrator_rename_snip35_module_to_dynamic_gas_price_scrub_snip-35_comments

Conversation

@sirandreww-starkware

Copy link
Copy Markdown
Contributor

No description provided.

@reviewable-StarkWare

Copy link
Copy Markdown

This change is Reviewable

@github-actions

github-actions Bot commented May 27, 2026

Copy link
Copy Markdown

sirandreww-starkware commented May 27, 2026

Copy link
Copy Markdown
Contributor Author

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

@graphite-app graphite-app Bot changed the base branch from 04-20-apollo_consensus_orchestrator_add_snip-35_proposer-validator_symmetry_tests to graphite-base/14216 May 27, 2026 11:52
@sirandreww-starkware sirandreww-starkware self-assigned this May 27, 2026

@matanl-starkware matanl-starkware left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matanl-starkware partially reviewed 26 files and all commit messages, and made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on sirandreww-starkware).


crates/apollo_dashboard/src/panels/consensus.rs line 726 at r1 (raw file):

fn get_panel_snip35_fee_actual() -> Panel {
    Panel::new(
        "SNIP-35 - fee_actual (GFri)",

Remove this prefix from all the panels, it's already under the SNIP-35 row.

Code quote:

SNIP-35 - 

crates/apollo_dashboard/src/panels/consensus.rs line 797 at r1 (raw file):

pub(crate) fn get_snip35_row() -> Row {
    Row::new(
        "SNIP-35 - Dynamic Gas Price",

Suggestion:

Dynamic Gas Price (SNIP-35)

@sirandreww-starkware sirandreww-starkware marked this pull request as ready for review May 27, 2026 12:52
@cursor

cursor Bot commented May 27, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Comment, module path, and symbol renames only; no algorithm or protocol field changes in the diff.

Overview
This PR is a naming and documentation cleanup for the consensus dynamic L2 gas fee mechanism: the orchestrator module snip35 is exposed as dynamic_gas_price, and imports across consensus orchestrator, cende, tests, protobuf comments, storage, API types, and related crates are updated accordingly.

Behavior is unchanged. Fee math, validation, and wire formats stay the same; only labels move from “SNIP-35” toward neutral terms (e.g. fee_proposals window, V0_14_3+) and one internal helper is renamed compute_snip35_fee_proposalcompute_proposer_fee_proposal. Validator messaging drops the “SNIP-35 bounds” wording; a TODO tag is retargeted to AndrewL.

Grafana/dashboard copy is refreshed (row “Dynamic Gas Price (SNIP-35)”, shorter panel titles); Prometheus metric names such as snip35_fee_actual are not renamed in this diff.

Reviewed by Cursor Bugbot for commit 8d2400d. Bugbot is set up for automated code reviews on this repo. Configure here.

@sirandreww-starkware sirandreww-starkware force-pushed the 05-27-apollo_consensus_orchestrator_rename_snip35_module_to_dynamic_gas_price_scrub_snip-35_comments branch from 477d3d8 to 9d8027d Compare May 27, 2026 13:31
@sirandreww-starkware sirandreww-starkware changed the base branch from graphite-base/14216 to 04-20-apollo_consensus_orchestrator_add_snip-35_proposer-validator_symmetry_tests May 27, 2026 13:31

@sirandreww-starkware sirandreww-starkware left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sirandreww-starkware made 2 comments.
Reviewable status: 24 of 26 files reviewed, 2 unresolved discussions (waiting on matanl-starkware).


crates/apollo_dashboard/src/panels/consensus.rs line 726 at r1 (raw file):

Previously, matanl-starkware (Matan Lior) wrote…

Remove this prefix from all the panels, it's already under the SNIP-35 row.

Done.


crates/apollo_dashboard/src/panels/consensus.rs line 797 at r1 (raw file):

pub(crate) fn get_snip35_row() -> Row {
    Row::new(
        "SNIP-35 - Dynamic Gas Price",

Done.

@matanl-starkware matanl-starkware left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matanl-starkware reviewed 2 files and all commit messages, made 1 comment, and resolved 2 discussions.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on sirandreww-starkware).


crates/apollo_dashboard/src/panels/consensus.rs line 726 at r2 (raw file):

fn get_panel_snip35_fee_actual() -> Panel {
    Panel::new(
        "fee_actual (GFri)",

All panels should have the same style (Title Case).
e.g. "Seconds Since Last Successful STRK→USD Rate Update"

Code quote:

fee_actual (GFri)

@matanl-starkware matanl-starkware left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on sirandreww-starkware).

@sirandreww-starkware sirandreww-starkware changed the base branch from 04-20-apollo_consensus_orchestrator_add_snip-35_proposer-validator_symmetry_tests to graphite-base/14216 May 27, 2026 14:47
@sirandreww-starkware sirandreww-starkware force-pushed the 05-27-apollo_consensus_orchestrator_rename_snip35_module_to_dynamic_gas_price_scrub_snip-35_comments branch from 9d8027d to 30aed32 Compare May 27, 2026 14:47
@sirandreww-starkware sirandreww-starkware changed the base branch from graphite-base/14216 to main May 27, 2026 14:47
@sirandreww-starkware sirandreww-starkware force-pushed the 05-27-apollo_consensus_orchestrator_rename_snip35_module_to_dynamic_gas_price_scrub_snip-35_comments branch from 30aed32 to 8d2400d Compare May 27, 2026 14:47

@sirandreww-starkware sirandreww-starkware left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sirandreww-starkware made 1 comment and resolved 1 discussion.
Reviewable status: 23 of 26 files reviewed, all discussions resolved (waiting on matanl-starkware).


crates/apollo_dashboard/src/panels/consensus.rs line 726 at r2 (raw file):

Previously, matanl-starkware (Matan Lior) wrote…

All panels should have the same style (Title Case).
e.g. "Seconds Since Last Successful STRK→USD Rate Update"

Done

@sirandreww-starkware sirandreww-starkware left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sirandreww-starkware partially reviewed 26 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on sirandreww-starkware).

@sirandreww-starkware sirandreww-starkware added this pull request to the merge queue May 27, 2026
Merged via the queue into main with commit 2d4a537 May 27, 2026
29 checks passed
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.

3 participants