chore(blockifier): access self.execution_flags instead of passing as args

[avivg-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[avivg-starkware]

• chore(blockifier): access self.execution_flags instead of passing as args #2491 👈 (View in Graphite)
• chore(blockifier): rename fn account_invoke_tx to invoke_tx_with_default_flags #2479
• chore(blockifier): remove declare.rs deploy_account.rs invoke.rs from blockifier #2478
• chore(blockifier): remove struct ExecutionFlags and replace w concurrency_mode bool #2470
• chore(blockifier): remove only_qury from IvokeTxArgs #2437
• chore(blockifier): add new constructor to AccountTransaction #2431
• chore(blockifier): add struct ExecutionFlags to AccTx & use instead of transaction::ExecutionFlags #2429
• chore(blockifier): use test_utils::invoke_tx() instead of trans::test_utils::account_invoke_tx() #2428 : 1 other dependent PR (#2354 )
• chore(blockifier): invoke() declare() deploy_account() change ret val to api_tx #2412
• chore(blockifier): enforce_fee() impl by api::executable_transaction::AccountTransaction #2377
• chore(blockifier): explicit creation of AccountTransaction #2331
• main

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

[github-actions]

Artifacts upload workflows:

• Started at 2024-12-05T14:19:39Z
• Started at 2024-12-05T14:31:28Z
• Started at 2024-12-05T15:46:52Z
• Started at 2024-12-05T19:51:42Z
• Started at 2024-12-05T20:23:05Z
• Started at 2024-12-05T21:24:30Z
• Started at 2024-12-05T21:57:16Z
• Started at 2024-12-06T08:17:06Z
• Started at 2024-12-06T09:29:32Z
• Started at 2024-12-06T09:30:20Z
• Started at 2024-12-06T15:52:42Z
• Started at 2024-12-06T15:54:17Z
• Started at 2024-12-06T17:09:16Z
• Started at 2024-12-06T17:10:34Z
• Started at 2024-12-06T21:37:26Z
• Started at 2024-12-06T21:37:35Z
• Started at 2024-12-06T21:38:36Z

[codecov]

Codecov Report

Attention: Patch coverage is 94.44444% with 2 lines in your changes missing coverage. Please review.

Project coverage is 71.96%. Comparing base (e3165c4) to head (4dbb97b).
Report is 733 commits behind head on main.

Files with missing lines	Patch %	Lines
.../blockifier/src/transaction/account_transaction.rs	94.44%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #2491       +/-   ##
===========================================
+ Coverage   40.10%   71.96%   +31.86%     
===========================================
  Files          26       98       +72     
  Lines        1895    13488    +11593     
  Branches     1895    13488    +11593     
===========================================
+ Hits          760     9707     +8947     
- Misses       1100     3366     +2266     
- Partials       35      415      +380     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Yoni-Starkware]

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @avivg-starkware)

[avivg-starkware]

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @avivg-starkware)

[avivg-starkware]

Merge activity

• Dec 6, 4:37 PM EST: A user started a stack merge that includes this pull request via Graphite.
• Dec 6, 4:39 PM EST: Graphite rebased this pull request as part of a merge.
• Dec 6, 4:57 PM EST: A user merged this pull request with Graphite.