Skip to content

chore: remove broadcasted function events#21805

Merged
PhilWindle merged 3 commits intomerge-train/spartanfrom
palla/remove-broadcasted-function-events
Mar 20, 2026
Merged

chore: remove broadcasted function events#21805
PhilWindle merged 3 commits intomerge-train/spartanfrom
palla/remove-broadcasted-function-events

Conversation

@spalladino
Copy link
Contributor

@spalladino spalladino commented Mar 19, 2026

Motivation

The PrivateFunctionBroadcastedEvent and UtilityFunctionBroadcastedEvent were removed from the ContractClassRegistry Noir contract, making all supporting TypeScript infrastructure dead code. This cleans it up.

Approach

Removed the event types, membership proof creation and validation functions, the archiver data flow that processed these events (storeBroadcastedIndividualFunctions), the store methods that persisted them (addFunctions), and the privateFunctions/utilityFunctions fields from ContractClassPublic.

Bumps the ARCHIVER_DB_VERSION to account for the schema change in the store.

Changes

  • stdlib: Removed ExecutablePrivateFunctionWithMembershipProof, UtilityFunctionWithMembershipProof, and related types from ContractClassPublic. Deleted membership proof files and their tests.
  • protocol-contracts: Deleted PrivateFunctionBroadcastedEvent and UtilityFunctionBroadcastedEvent classes, their tests, snapshots, and hex fixtures.
  • archiver: Removed storeBroadcastedIndividualFunctions from the data store updater, addFunctions from the contract class store, and related serialization helpers and tests.
  • txe, end-to-end: Removed references to the deleted fields.

The PrivateFunctionBroadcastedEvent and UtilityFunctionBroadcastedEvent
were removed from the ContractClassRegistry contract, so all supporting
TypeScript infrastructure is now dead code. This removes the event types,
membership proof creation/validation, archiver data flows, store methods,
and the privateFunctions/utilityFunctions fields from ContractClassPublic.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@spalladino spalladino added the ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure label Mar 19, 2026
@spalladino spalladino changed the title refactor: remove broadcasted function events and membership proofs chore: remove broadcasted function events Mar 19, 2026
spalladino and others added 2 commits March 19, 2026 16:57
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ublic assertions

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@PhilWindle PhilWindle merged commit fa67469 into merge-train/spartan Mar 20, 2026
11 checks passed
@PhilWindle PhilWindle deleted the palla/remove-broadcasted-function-events branch March 20, 2026 09:27
github-merge-queue bot pushed a commit that referenced this pull request Mar 20, 2026
BEGIN_COMMIT_OVERRIDE
feat(p2p): add tx validator for contract instance deployment addresses
(#21771)
fix: always deploy IRM for testnet (#21755)
fix: avoid mutating caller's array via splice in snapshot sync (A-718)
(#21759)
chore: update network logs skill (#21785)
feat(archiver): validate contract instance addresses before storing
(#21787)
fix: ensure no division by 0 (#21786)
feat: support private fork releases via ci-release (#21778)
fix: restrict scenario deployments to only nightly (#21798)
fix(stdlib): zero-pad bufferFromFields when declared length exceeds
payload (#21802)
test(protocol-contracts): verify max-size bytecode fits in contract
class log (#21818)
fix: wire BOT_DA_GAS_LIMIT through helm/terraform for staging-public
(#21809)
fix: remove jest-mock-extended from worker processes + fix
parallelize_strict silent failures (#21821)
fix(archiver): throw on duplicate contract class or instance additions
(#21799)
chore: remove broadcasted function events (#21805)
fix: sync dateProvider from anvil stdout on every mined block (#21829)
fix(sequencer): use wall-clock time instead of L1 block timestamp for
slot estimation (#21769)
fix: use correct EthCheatCodes method name in epochs_missed_l1_slot test
(#21848)
feat(p2p): add tx validator for contract class id verification (#21788)
feat: publisher funding (#21631)
feat: batch chonk verifier TS integration (#21823)
fix(sequencer): remove l1 block timestamp check (#21853)
fix: use local IVC inputs for batch_verifier bench test (#21857)
fix(p2p): centralize gossipsub penalization and fix inconsistencies
(#21863)
chore: publish GitHub releases to AztecProtocol/barretenberg (#21775)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants