Skip to content

apollo_propeller: rename DuplicateShard/ReceivedSelfPublishedShard variants#13525

Open
sirandreww-starkware wants to merge 1 commit into03-26-apollo_propeller_fix_unitvalidationerror_error_messages_to_say_unit_instead_of_shard_from
03-26-apollo_propeller_rename_duplicateshard_to_duplicateunit_receivedselfpublishedshard_to_receivedselfpublishedunit
Open

apollo_propeller: rename DuplicateShard/ReceivedSelfPublishedShard variants#13525
sirandreww-starkware wants to merge 1 commit into03-26-apollo_propeller_fix_unitvalidationerror_error_messages_to_say_unit_instead_of_shard_from
03-26-apollo_propeller_rename_duplicateshard_to_duplicateunit_receivedselfpublishedshard_to_receivedselfpublishedunit

Conversation

@sirandreww-starkware
Copy link
Copy Markdown
Contributor

@sirandreww-starkware sirandreww-starkware commented Mar 26, 2026

Note

Low Risk
Low risk: this is a pure rename of UnitValidationError variants (and associated call sites/tests) with no behavior changes, but it may break downstream pattern matches if any external crates rely on the old names.

Overview
Renames UnitValidationError variants to use unit terminology (ReceivedSelfPublishedShard -> ReceivedSelfPublishedUnit, DuplicateShard -> DuplicateUnit) and updates all internal references in PropellerScheduleManager::validate_origin, UnitValidator::validate_unit, and the unit validator tests to match.

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

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@sirandreww-starkware sirandreww-starkware self-assigned this Mar 26, 2026
@sirandreww-starkware sirandreww-starkware marked this pull request as ready for review March 26, 2026 15:59
@sirandreww-starkware sirandreww-starkware changed the base branch from 03-26-apollo_propeller_fix_unitvalidationerror_error_messages_to_say_unit_instead_of_shard_ to graphite-base/13525 April 6, 2026 07:14
@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_rename_duplicateshard_to_duplicateunit_receivedselfpublishedshard_to_receivedselfpublishedunit branch from dee5dc1 to 8e3aabe Compare April 6, 2026 07:15
@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_rename_duplicateshard_to_duplicateunit_receivedselfpublishedshard_to_receivedselfpublishedunit branch from 8e3aabe to dc80eaa Compare April 6, 2026 08:35
@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_rename_duplicateshard_to_duplicateunit_receivedselfpublishedshard_to_receivedselfpublishedunit branch from dc80eaa to 6e1686b Compare April 6, 2026 12:03
Copy link
Copy Markdown
Contributor

@guy-starkware guy-starkware left a comment

Choose a reason for hiding this comment

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

@guy-starkware reviewed 4 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on noamsp-starkware and ShahakShama).

@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_rename_duplicateshard_to_duplicateunit_receivedselfpublishedshard_to_receivedselfpublishedunit branch from 6e1686b to decd756 Compare April 9, 2026 13:09
@sirandreww-starkware sirandreww-starkware changed the base branch from graphite-base/13525 to 03-26-apollo_propeller_fix_unitvalidationerror_error_messages_to_say_unit_instead_of_shard_ April 9, 2026 13:09
@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_fix_unitvalidationerror_error_messages_to_say_unit_instead_of_shard_ branch from 06592ea to 3ff06da Compare April 12, 2026 06:36
@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_rename_duplicateshard_to_duplicateunit_receivedselfpublishedshard_to_receivedselfpublishedunit branch from decd756 to 37d2616 Compare April 12, 2026 06:36
@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_rename_duplicateshard_to_duplicateunit_receivedselfpublishedshard_to_receivedselfpublishedunit branch from 37d2616 to 25a3bc0 Compare April 20, 2026 05:57
@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_fix_unitvalidationerror_error_messages_to_say_unit_instead_of_shard_ branch from 3ff06da to 3704d5e Compare April 20, 2026 05:57
@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 20, 2026

PR Summary

Low Risk
Purely a rename of error variants and call sites with no behavioral changes; risk is limited to missing references in downstream code.

Overview
Renames UnitValidationError variants to use unit terminology: ReceivedSelfPublishedShard -> ReceivedSelfPublishedUnit and DuplicateShard -> DuplicateUnit.

Updates origin validation (PropellerScheduleManager::validate_origin), unit validation (UnitValidator::validate_unit), and the corresponding validator test assertion to use the new error names.

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

@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_fix_unitvalidationerror_error_messages_to_say_unit_instead_of_shard_ branch from 3704d5e to 977ce48 Compare April 20, 2026 09:54
@sirandreww-starkware sirandreww-starkware force-pushed the 03-26-apollo_propeller_rename_duplicateshard_to_duplicateunit_receivedselfpublishedshard_to_receivedselfpublishedunit branch from 25a3bc0 to 2ecd0b8 Compare April 20, 2026 09:54
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