Skip to content

chore: rename simulateUtility -> executeUtility#20572

Merged
benesjan merged 1 commit intomerge-train/fairiesfrom
jan/rename-simulate-utility-to-execute-utility
Feb 17, 2026
Merged

chore: rename simulateUtility -> executeUtility#20572
benesjan merged 1 commit intomerge-train/fairiesfrom
jan/rename-simulate-utility-to-execute-utility

Conversation

@benesjan
Copy link
Contributor

@benesjan benesjan commented Feb 17, 2026

Fixes https://linear.app/aztec-labs/issue/F-285/renames-simulateutility-executeutility

Summary

  • Renames simulateUtility to executeUtility across the entire codebase (TypeScript, Noir, TXE oracle layers)
  • Renames UtilitySimulationResult to UtilityExecutionResult
  • Renames associated types: SimulateUtilityOptsExecuteUtilityOpts, SimulateUtilityOptionsExecuteUtilityOptions
  • Renames Noir functions: simulate_utility_functionexecute_utility_function, env.simulate_utility()env.execute_utility()
  • Renames TXE oracle: txeSimulateUtilityFunctiontxeExecuteUtilityFunction
  • Updates all jsdoc comments and log messages to reflect the new terminology

Utility functions are executed, not simulated — this rename makes the terminology consistent: simulateTx simulates transactions, while executeUtility executes utility (unconstrained) functions.

Test plan

  • Existing unit tests pass (renamed test names and assertions updated)
  • TypeScript compilation succeeds across all affected packages
  • Noir contract tests compile and pass with renamed execute_utility calls

🤖 Generated with Claude Code

@benesjan benesjan requested review from a team, LeilaWang and nventuro as code owners February 17, 2026 07:55
@benesjan benesjan changed the base branch from next to merge-train/fairies February 17, 2026 08:00
@AztecBot
Copy link
Collaborator

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/e44b9c50b2b293ed�e44b9c50b2b293ed8;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_epochs/epochs_proof_fails.parallel.test.ts "does not allow submitting proof after epoch end" (231s) (code: 0) group:e2e-p2p-epoch-flakes

@benesjan benesjan requested review from Thunkar and mverzilli and removed request for LeilaWang and nventuro February 17, 2026 08:25
Copy link
Contributor

@mverzilli mverzilli left a comment

Choose a reason for hiding this comment

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

When I see we found happier names 😁
When I think of the slop accusations 😭

Fixes https://linear.app/aztec-labs/issue/F-285/renames-simulateutility-executeutility

## Summary
- Renames `simulateUtility` to `executeUtility` across the entire codebase (TypeScript, Noir, TXE oracle layers)
- Renames `UtilitySimulationResult` to `UtilityExecutionResult`
- Renames associated types: `SimulateUtilityOpts` → `ExecuteUtilityOpts`, `SimulateUtilityOptions` → `ExecuteUtilityOptions`
- Renames Noir functions: `simulate_utility_function` → `execute_utility_function`, `env.simulate_utility()` → `env.execute_utility()`
- Renames TXE oracle: `txeSimulateUtilityFunction` → `txeExecuteUtilityFunction`
- Updates all jsdoc comments and log messages to reflect the new terminology

Utility functions are executed, not simulated — this rename makes the terminology consistent: `simulateTx` simulates transactions, while `executeUtility` executes utility (unconstrained) functions.

## Test plan
- [ ] Existing unit tests pass (renamed test names and assertions updated)
- [ ] TypeScript compilation succeeds across all affected packages
- [ ] Noir contract tests compile and pass with renamed `execute_utility` calls

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@AztecBot AztecBot force-pushed the jan/rename-simulate-utility-to-execute-utility branch from 1e8a4b5 to 1f5b442 Compare February 17, 2026 12:23
@benesjan benesjan merged commit 7b7dc2d into merge-train/fairies Feb 17, 2026
10 checks passed
@benesjan benesjan deleted the jan/rename-simulate-utility-to-execute-utility branch February 17, 2026 12:23
github-merge-queue bot pushed a commit that referenced this pull request Feb 18, 2026
BEGIN_COMMIT_OVERRIDE
chore: reverting accidental de-macroifycation of AuthRegistry (#20532)
docs: add delayedpublicmutable apiref, fix misc docs (#20512)
chore: rename simulateUtility -> executeUtility (#20572)
fix: enable contract debug logs in `aztec test` (#20580)
chore: merging next to merge-train/fairies (#20622)
END_COMMIT_OVERRIDE
@AztecBot
Copy link
Collaborator

❌ Failed to cherry-pick to v4 due to conflicts. Please backport manually.

AztecBot added a commit that referenced this pull request Feb 26, 2026
The cherry-pick of PR #20572 (simulateUtility -> executeUtility rename)
conflicted in private_voting_contract/src/test/first.nr because v4 had
already renamed simulate_utility to view_public for public view functions.

get_tally is a #[external("public")] #[view] function, so the correct
method to call it on v4 is view_public (for public static calls), not
execute_utility (for utility/unconstrained functions). This restores the
correct v4 behavior.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants