Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds automatic sorting of forced transactions by their forcedTransactionNumber before they are added to the pool. This ensures transactions are processed in the correct order even when submitted out of order in a single RPC call.
Changes:
- Added sorting of forced transactions by
forcedTransactionNumberbefore adding them to the pool - Added comprehensive test coverage for the sorting behavior
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| besu-plugins/linea-sequencer/sequencer/src/main/java/net/consensys/linea/rpc/methods/LineaSendForcedRawTransaction.java | Added sorting logic to ensure forced transactions are processed in order by their forcedTransactionNumber |
| besu-plugins/linea-sequencer/sequencer/src/test/java/net/consensys/linea/rpc/methods/LineaSendForcedRawTransactionTest.java | Added test to verify transactions are sorted correctly before being added to the pool |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2350 +/- ##
=============================================
- Coverage 61.14% 10.18% -50.96%
=============================================
Files 436 436
Lines 17363 17363
Branches 1908 1908
=============================================
- Hits 10616 1769 -8847
- Misses 6088 15594 +9506
+ Partials 659 0 -659
*This pull request uses carry forward flags. Click here to find out more. 🚀 New features to boost your workflow:
|
fluentcrafter
approved these changes
Feb 16, 2026
thedarkjester
added a commit
that referenced
this pull request
Feb 18, 2026
* Forced Transactions - Acceptance tests - Added TooManyLogs test (#2296) * Forced Transactions - Acceptance tests - Added TooManyLogs test * Removed unused methods * Sequencer/ftx other rejection reason (#2312) * Added a test making sure a transaction with unknown inclusion status blocks the Ftx flow * Fixed the test and the implementation for Other rejection status reporting * Fixed a comment * deps(actions)(deps): bump advanced-security/ghas-to-csv from 2 to 3 (#2389) Bumps [advanced-security/ghas-to-csv](https://github.com/advanced-security/ghas-to-csv) from 2 to 3. - [Release notes](https://github.com/advanced-security/ghas-to-csv/releases) - [Commits](advanced-security/ghas-to-csv@v2...v3) --- updated-dependencies: - dependency-name: advanced-security/ghas-to-csv dependency-version: '3' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: simplify docker build process in ci (#2367) * fix: simplify docker build process in ci * fix: revert check and tag images changes * fix: only build changed servides or use develop tag * fix: error handling issue * docs: extract architecture documentation to separate file (#2378) Refactor README to be more concise and move detailed architecture documentation to docs/architecture.md. The README now provides a brief overview and references the architecture doc for design details. Also updates README with: - Improved section organization - Manual integration scripts documentation - Unit test instructions Co-authored-by: Cursor <cursoragent@cursor.com> * [ZkTracer] fix: 7702 in RlpTxn (#2391) * [ZkTracer] fix: 7702 in RlpTxn Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> * ras Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> * perf, ty cursor Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> --------- Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> * Sequencer - Added sorting by ftx number before adding to the pool (#2350) * [ZkTracer] chore: update corset to 1.2.0 (#2394) Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> * feat: add support of tracer and sequencer auto release if releases.ve… (#2219) * feat: add support of tracer and sequencer auto release if releases.versions.toml has updated * fix: add build_linea_besu_package_from_source input in publish-images-after-run-tests-success-on-main * fix: ASSEMBLE_LOCAL_ZIPS check in assemble action * chore: removed unnecessary steps * chore: removed commented out lines * feat: revise linea sequencer and tracer manual release workflows and linea-besu-package Makefile * feat: revise linea sequencer manual release workflows * test: linea sequencer and tracer auto release workflows * test: linea sequencer and tracer auto release workflows * test: linea sequencer and tracer auto release workflows * test: linea sequencer and tracer auto release workflows * feat: revert test changes and added and updated README docs * feat: renamed and updated README docs * feat: remove tracer and sequencer auto release and revise workflows * fix: check-lib-version-change action * test: arithmetization version update * fix: check-lib-version-change action * feat: added action to compute version tags * feat: add auto linea-besu-package release for build from source tracer and sequencer * fix: linea-besu-package release for fleet plugin * feat: update with tracer version comparison change and tag tracer plugin with commit hash suffix * fix: cursor comments * fix: parse-release-version action * fix: env injection to call make assemble-local * fix: env injection to call make assemble-local plus logs * fix: local tracer and sequencer artifact paths * fix: add releases.versions.toml change to trigger auto release when merge into main * test: to fix download and upload artifacts of linea-besu-package * feat: revert add path in download-artifact * test: to fix download and upload artifacts of linea-besu-package * fix: build_from_source always true issue and cursor comments * feat: added always in docker-build job * fix: using wrong besu version in release note when build from source * fix: readme typo * feat: removed the check of releases.versions.toml changes * feat: upgrade tracer release workflow to use med instance * test: update versions.env to test linea-besu-package manual release with download * feat: revert versions.env update for testing and update linea-besu-package image in local stack for trace-limits v5 * fix: spotless issue * feat: trigger linea-besu-package local build and auto release when relevant code changes * feat: set permission content as write on build-and-publish.yml * feat: revert permission content as read on build-and-publish.yml and trigger release workflow * fix: cursor comments * fix: cursor comments * fix: build-and-publish permissions * fix: build-and-publish permissions * fix: build-and-publish permissions * fix: build-and-publish branch reference * fix: build-and-publish branch reference * test: revise permissions on main.yml * test: revise permissions on main.yml * fix: revise permissions on main.yml * feat: revised README and auto release of linea-besu-package conditions * fix: typo for action descriptions * chore: revise for cursor comment --------- Co-authored-by: François Bojarski <54240434+letypequividelespoubelles@users.noreply.github.com> * feat: add support for blob compressor and decompressor v3 (#2321) * feat: add support for blob compressor and decompressor v3 * feat: use setup-gradle@v4 * feat: try update gradle version to 8.14.4 * feat: update org.jrelease version to 1.21.0 * feat: update org.jrelease version to 1.18.0 * feat: downgrade spotless and klint * feat: downgrade spotless and klint * feat: update coordinator to use blobCompressor v3 * feat: add support for blob shnarf calculator v3 * feat: add support for blob shnarf calculator v3 * feat: revise example release workflow * feat: added returning shnarf calculator version based on blob compressor version * fix: spotless issue * feat: revert coordinator config blob-compressor-version back to v2 on local stack * test: update jrelease to v1.22 and force org.eclipse.jgit version in jvm-libs gradle * test: update jrelease to v1.15 and force org.eclipse.jgit version in jvm-libs gradle * test: revert spotless to v8.1 and klint to 1.0.1 * Revert "test: revert spotless to v8.1 and klint to 1.0.1" This reverts commit 6a546fd. * feat: revert test changes * test: revert spotless to v8.1 and klint to 1.0.1 with force dependency on jgit 6.7 * test: revert spotless to v8.1 and klint to 1.0.1 with force dependency on jgit 6.7 * test: revert force dependency on jgit 6.7 and upgrade jreleaser to 1.22 * feat: upgrade bouncycastle * feat: revert jreleaser to 1.15 and bouncycastle to 1.79 * feat: set to use blob compression v3 in test * feat: parameterized tests * refactor: parameterize blob compression TOML config tests Convert three separate tests for blob compression config parsing (V1_2, V2, V3) into a single parameterized test covering all compressor versions and their expected shnarf calculator versions. Co-authored-by: Roman Vaseev <Filter94@users.noreply.github.com> --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Roman Vaseev <Filter94@users.noreply.github.com> * fix: add leaf length check in sparse merkle proof contrct (#2400) * fix: add leaf length check in sparse merkle proof contrct * fix: add new bytes length errors * fix: revert errors changes in sparse merkle proof * Coordinator forced tx part11 (#2348) * coordinator: add Disabled LongRunning Service placeholder for disabled services * coordinator: adds forced transactions config parsing * coordinator: allow easier debug on docker * coordinator: add ForcedTransactionsDao implementation and tests * coordinator: eth logs searcher support for state listener * coordinator: open DisabledService * coordinator: make forced transactions config optional * coordinator: forced transactions conflations safe block manager logic * coordinator: fix spotless * coordinator: contract client add Web3JContractVersionAwaiter * coordinator: add ConflationSafeBlockNumberProvider * coordinator: add ContractVersionAwaiter * coordinator: set default LINEA_COORDINATOR_DISABLED * coordinator: checking some changes left behind * coordinator: fix default forcedTransactionNumber=0 in FakeLineaRollupSmartContractClient * coordinator: forced transaction fix edge case before contract V8 * coordinator: forced transaction default configs * coordinator: use kotlin.time over kotlinx.datetime for instant handling * coordinator: forced transactions: implement processing delay * coordinator: forced transactions: fix error handling * coordinator: make ContractVersionProvider interface block aware * coordinator: ForcedTransactions default l1HighestBlockTag=FINALIZED * refactor: simplify FTX processing tests by removing unnecessary finalized state events * coordinator: local stack allow to skip coordinator with dev_LINEA_COORDINATOR_DISABLED * coordinator: add tests for ForcedTransactionsSafeBlockNumberManager and improve safe block number handling logic * Update jvm-libs/linea/clients/interfaces/src/main/kotlin/linea/contract/l1/LineaSmartContractClient.kt Co-authored-by: Roman Vaseev <4833306+Filter94@users.noreply.github.com> Signed-off-by: Fluent Crafter <205769460+fluentcrafter@users.noreply.github.com> * coordinator: refactor PostgresForcedTransactionsDaoTest to improve readability and consistency * linea: add missing braces to ContractVersionAwaiter interface * coordinator: fix type consistency issues in PostgresForcedTransactionsDaoTest and remove unused import in ForcedTransactionsL1EventsFetcher --------- Signed-off-by: Fluent Crafter <205769460+fluentcrafter@users.noreply.github.com> Co-authored-by: Roman Vaseev <4833306+Filter94@users.noreply.github.com> * [Fix] - merge all audit fixes (#2346) * Various informational fixes * use per pause type expiry * add back unchecked * allow other pause types to be paused during initial window * add docstring on role restriction * add public instead of external for better inheritence * emit genesis shnarf and initializer to latest version * missed saved file * allow SC pause extension without unpause * pause by type doc * PauseIndefinitely event for SC pausing * add state setting for SC * use correct event for pause indefinitely on YM * allow SC cooldown reset * add docstring * v7.1 ABI * updated doc * use live values * match name to version * add reentrancy checks and specific transient slot * add testing and correct slot * add extra new deploy check to initialize * tweaks * address wrong namespace * [ZkTracer] fix: constraints in SELFDESTRUCT (#2398) Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> Co-authored-by: Olivier Bégassat <olivier.begassat.cours@gmail.com> Co-authored-by: Olivier Bégassat <38285177+OlivierBBB@users.noreply.github.com> * fix: postman processing time metric issue (#2401) * feat: restructure BUGBOT rules and add API versioning guidelines (#2404) Move security-logging rules from BUGBOT.md into dedicated RULES.md alongside existing EXAMPLES.md. Add new versioning-guidelines rule set enforcing backwards compatibility for public APIs and versioned assets. Simplify .gitignore to include all of .cursor/rules/ instead of per-file allowlisting. * [Fix] - v7.1 coordinator (#2143) * emit version v7.1 on reinitializing * changes for the coordinator and make scripts * revert commit of # * Various informational fixes * use per pause type expiry * add back unchecked * allow other pause types to be paused during initial window * add docstring on role restriction * add public instead of external for better inheritence * emit genesis shnarf and initializer to latest version * missed saved file * allow SC pause extension without unpause * pause by type doc * PauseIndefinitely event for SC pausing * add state setting for SC * use correct event for pause indefinitely on YM * allow SC cooldown reset * update tag for coordinator * updated artifact * add docstring * v7.1 ABI * updated doc * abi patch * use working image and support dynamic chain config e2e * use live values * match name to version * add reentrancy checks and specific transient slot * add testing and correct slot * add extra new deploy check to initialize * add extra new deploy check to initialize * tweaks * address wrong namespace * deployment for v7.1 updates * artifacts * deploy 7_1 for 7 * force e2e * fix: make the docker build workflow succeed when all build are skipped * use v8 roles * use dynamic name for plonkverifier contract deploy --------- Co-authored-by: Victorien Gauch <85494462+VGau@users.noreply.github.com> Co-authored-by: VGau <victorien.gauch@consensys.net> * fix testing --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> Signed-off-by: Fluent Crafter <205769460+fluentcrafter@users.noreply.github.com> Co-authored-by: Roman Vaseev <4833306+Filter94@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Victorien Gauch <85494462+VGau@users.noreply.github.com> Co-authored-by: kyzooghost <73516204+kyzooghost@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com> Co-authored-by: François Bojarski <54240434+letypequividelespoubelles@users.noreply.github.com> Co-authored-by: jonesho <81145364+jonesho@users.noreply.github.com> Co-authored-by: Roman Vaseev <Filter94@users.noreply.github.com> Co-authored-by: Fluent Crafter <205769460+fluentcrafter@users.noreply.github.com> Co-authored-by: Olivier Bégassat <olivier.begassat.cours@gmail.com> Co-authored-by: Olivier Bégassat <38285177+OlivierBBB@users.noreply.github.com> Co-authored-by: VGau <victorien.gauch@consensys.net>
thedarkjester
added a commit
that referenced
this pull request
Feb 18, 2026
* use dynamic block time * fixes for informational and gas findings * fix: add l2 block duration seconds to forced transaction gateway constructor * rename for readability * cater for finalization scenario and same calculation * remove .only * use openzeppelin signer recovery * add files * export ABI and artifact * use OZ recover vs tryRecover * store L1 block number for checks * add sanity check for final forced tx number * add toml errors * use correct value for the fee * update ABI and artifacts * default to old timestamp if fork-timestamp file missing * [Fix] Add support for Validium (#2380) * validium v2 prep * validium deployment updates * use correct validium version * updated ABI * local-stack: fix genesis timestamp setup * local-stack: fix genesis timestamp setup * spacing * Chore/merging main (#2407) * Forced Transactions - Acceptance tests - Added TooManyLogs test (#2296) * Forced Transactions - Acceptance tests - Added TooManyLogs test * Removed unused methods * Sequencer/ftx other rejection reason (#2312) * Added a test making sure a transaction with unknown inclusion status blocks the Ftx flow * Fixed the test and the implementation for Other rejection status reporting * Fixed a comment * deps(actions)(deps): bump advanced-security/ghas-to-csv from 2 to 3 (#2389) Bumps [advanced-security/ghas-to-csv](https://github.com/advanced-security/ghas-to-csv) from 2 to 3. - [Release notes](https://github.com/advanced-security/ghas-to-csv/releases) - [Commits](advanced-security/ghas-to-csv@v2...v3) --- updated-dependencies: - dependency-name: advanced-security/ghas-to-csv dependency-version: '3' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: simplify docker build process in ci (#2367) * fix: simplify docker build process in ci * fix: revert check and tag images changes * fix: only build changed servides or use develop tag * fix: error handling issue * docs: extract architecture documentation to separate file (#2378) Refactor README to be more concise and move detailed architecture documentation to docs/architecture.md. The README now provides a brief overview and references the architecture doc for design details. Also updates README with: - Improved section organization - Manual integration scripts documentation - Unit test instructions Co-authored-by: Cursor <cursoragent@cursor.com> * [ZkTracer] fix: 7702 in RlpTxn (#2391) * [ZkTracer] fix: 7702 in RlpTxn Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> * ras Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> * perf, ty cursor Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> --------- Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> * Sequencer - Added sorting by ftx number before adding to the pool (#2350) * [ZkTracer] chore: update corset to 1.2.0 (#2394) Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> * feat: add support of tracer and sequencer auto release if releases.ve… (#2219) * feat: add support of tracer and sequencer auto release if releases.versions.toml has updated * fix: add build_linea_besu_package_from_source input in publish-images-after-run-tests-success-on-main * fix: ASSEMBLE_LOCAL_ZIPS check in assemble action * chore: removed unnecessary steps * chore: removed commented out lines * feat: revise linea sequencer and tracer manual release workflows and linea-besu-package Makefile * feat: revise linea sequencer manual release workflows * test: linea sequencer and tracer auto release workflows * test: linea sequencer and tracer auto release workflows * test: linea sequencer and tracer auto release workflows * test: linea sequencer and tracer auto release workflows * feat: revert test changes and added and updated README docs * feat: renamed and updated README docs * feat: remove tracer and sequencer auto release and revise workflows * fix: check-lib-version-change action * test: arithmetization version update * fix: check-lib-version-change action * feat: added action to compute version tags * feat: add auto linea-besu-package release for build from source tracer and sequencer * fix: linea-besu-package release for fleet plugin * feat: update with tracer version comparison change and tag tracer plugin with commit hash suffix * fix: cursor comments * fix: parse-release-version action * fix: env injection to call make assemble-local * fix: env injection to call make assemble-local plus logs * fix: local tracer and sequencer artifact paths * fix: add releases.versions.toml change to trigger auto release when merge into main * test: to fix download and upload artifacts of linea-besu-package * feat: revert add path in download-artifact * test: to fix download and upload artifacts of linea-besu-package * fix: build_from_source always true issue and cursor comments * feat: added always in docker-build job * fix: using wrong besu version in release note when build from source * fix: readme typo * feat: removed the check of releases.versions.toml changes * feat: upgrade tracer release workflow to use med instance * test: update versions.env to test linea-besu-package manual release with download * feat: revert versions.env update for testing and update linea-besu-package image in local stack for trace-limits v5 * fix: spotless issue * feat: trigger linea-besu-package local build and auto release when relevant code changes * feat: set permission content as write on build-and-publish.yml * feat: revert permission content as read on build-and-publish.yml and trigger release workflow * fix: cursor comments * fix: cursor comments * fix: build-and-publish permissions * fix: build-and-publish permissions * fix: build-and-publish permissions * fix: build-and-publish branch reference * fix: build-and-publish branch reference * test: revise permissions on main.yml * test: revise permissions on main.yml * fix: revise permissions on main.yml * feat: revised README and auto release of linea-besu-package conditions * fix: typo for action descriptions * chore: revise for cursor comment --------- Co-authored-by: François Bojarski <54240434+letypequividelespoubelles@users.noreply.github.com> * feat: add support for blob compressor and decompressor v3 (#2321) * feat: add support for blob compressor and decompressor v3 * feat: use setup-gradle@v4 * feat: try update gradle version to 8.14.4 * feat: update org.jrelease version to 1.21.0 * feat: update org.jrelease version to 1.18.0 * feat: downgrade spotless and klint * feat: downgrade spotless and klint * feat: update coordinator to use blobCompressor v3 * feat: add support for blob shnarf calculator v3 * feat: add support for blob shnarf calculator v3 * feat: revise example release workflow * feat: added returning shnarf calculator version based on blob compressor version * fix: spotless issue * feat: revert coordinator config blob-compressor-version back to v2 on local stack * test: update jrelease to v1.22 and force org.eclipse.jgit version in jvm-libs gradle * test: update jrelease to v1.15 and force org.eclipse.jgit version in jvm-libs gradle * test: revert spotless to v8.1 and klint to 1.0.1 * Revert "test: revert spotless to v8.1 and klint to 1.0.1" This reverts commit 6a546fd. * feat: revert test changes * test: revert spotless to v8.1 and klint to 1.0.1 with force dependency on jgit 6.7 * test: revert spotless to v8.1 and klint to 1.0.1 with force dependency on jgit 6.7 * test: revert force dependency on jgit 6.7 and upgrade jreleaser to 1.22 * feat: upgrade bouncycastle * feat: revert jreleaser to 1.15 and bouncycastle to 1.79 * feat: set to use blob compression v3 in test * feat: parameterized tests * refactor: parameterize blob compression TOML config tests Convert three separate tests for blob compression config parsing (V1_2, V2, V3) into a single parameterized test covering all compressor versions and their expected shnarf calculator versions. Co-authored-by: Roman Vaseev <Filter94@users.noreply.github.com> --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Roman Vaseev <Filter94@users.noreply.github.com> * fix: add leaf length check in sparse merkle proof contrct (#2400) * fix: add leaf length check in sparse merkle proof contrct * fix: add new bytes length errors * fix: revert errors changes in sparse merkle proof * Coordinator forced tx part11 (#2348) * coordinator: add Disabled LongRunning Service placeholder for disabled services * coordinator: adds forced transactions config parsing * coordinator: allow easier debug on docker * coordinator: add ForcedTransactionsDao implementation and tests * coordinator: eth logs searcher support for state listener * coordinator: open DisabledService * coordinator: make forced transactions config optional * coordinator: forced transactions conflations safe block manager logic * coordinator: fix spotless * coordinator: contract client add Web3JContractVersionAwaiter * coordinator: add ConflationSafeBlockNumberProvider * coordinator: add ContractVersionAwaiter * coordinator: set default LINEA_COORDINATOR_DISABLED * coordinator: checking some changes left behind * coordinator: fix default forcedTransactionNumber=0 in FakeLineaRollupSmartContractClient * coordinator: forced transaction fix edge case before contract V8 * coordinator: forced transaction default configs * coordinator: use kotlin.time over kotlinx.datetime for instant handling * coordinator: forced transactions: implement processing delay * coordinator: forced transactions: fix error handling * coordinator: make ContractVersionProvider interface block aware * coordinator: ForcedTransactions default l1HighestBlockTag=FINALIZED * refactor: simplify FTX processing tests by removing unnecessary finalized state events * coordinator: local stack allow to skip coordinator with dev_LINEA_COORDINATOR_DISABLED * coordinator: add tests for ForcedTransactionsSafeBlockNumberManager and improve safe block number handling logic * Update jvm-libs/linea/clients/interfaces/src/main/kotlin/linea/contract/l1/LineaSmartContractClient.kt Co-authored-by: Roman Vaseev <4833306+Filter94@users.noreply.github.com> Signed-off-by: Fluent Crafter <205769460+fluentcrafter@users.noreply.github.com> * coordinator: refactor PostgresForcedTransactionsDaoTest to improve readability and consistency * linea: add missing braces to ContractVersionAwaiter interface * coordinator: fix type consistency issues in PostgresForcedTransactionsDaoTest and remove unused import in ForcedTransactionsL1EventsFetcher --------- Signed-off-by: Fluent Crafter <205769460+fluentcrafter@users.noreply.github.com> Co-authored-by: Roman Vaseev <4833306+Filter94@users.noreply.github.com> * [Fix] - merge all audit fixes (#2346) * Various informational fixes * use per pause type expiry * add back unchecked * allow other pause types to be paused during initial window * add docstring on role restriction * add public instead of external for better inheritence * emit genesis shnarf and initializer to latest version * missed saved file * allow SC pause extension without unpause * pause by type doc * PauseIndefinitely event for SC pausing * add state setting for SC * use correct event for pause indefinitely on YM * allow SC cooldown reset * add docstring * v7.1 ABI * updated doc * use live values * match name to version * add reentrancy checks and specific transient slot * add testing and correct slot * add extra new deploy check to initialize * tweaks * address wrong namespace * [ZkTracer] fix: constraints in SELFDESTRUCT (#2398) Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> Co-authored-by: Olivier Bégassat <olivier.begassat.cours@gmail.com> Co-authored-by: Olivier Bégassat <38285177+OlivierBBB@users.noreply.github.com> * fix: postman processing time metric issue (#2401) * feat: restructure BUGBOT rules and add API versioning guidelines (#2404) Move security-logging rules from BUGBOT.md into dedicated RULES.md alongside existing EXAMPLES.md. Add new versioning-guidelines rule set enforcing backwards compatibility for public APIs and versioned assets. Simplify .gitignore to include all of .cursor/rules/ instead of per-file allowlisting. * [Fix] - v7.1 coordinator (#2143) * emit version v7.1 on reinitializing * changes for the coordinator and make scripts * revert commit of # * Various informational fixes * use per pause type expiry * add back unchecked * allow other pause types to be paused during initial window * add docstring on role restriction * add public instead of external for better inheritence * emit genesis shnarf and initializer to latest version * missed saved file * allow SC pause extension without unpause * pause by type doc * PauseIndefinitely event for SC pausing * add state setting for SC * use correct event for pause indefinitely on YM * allow SC cooldown reset * update tag for coordinator * updated artifact * add docstring * v7.1 ABI * updated doc * abi patch * use working image and support dynamic chain config e2e * use live values * match name to version * add reentrancy checks and specific transient slot * add testing and correct slot * add extra new deploy check to initialize * add extra new deploy check to initialize * tweaks * address wrong namespace * deployment for v7.1 updates * artifacts * deploy 7_1 for 7 * force e2e * fix: make the docker build workflow succeed when all build are skipped * use v8 roles * use dynamic name for plonkverifier contract deploy --------- Co-authored-by: Victorien Gauch <85494462+VGau@users.noreply.github.com> Co-authored-by: VGau <victorien.gauch@consensys.net> * fix testing --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> Signed-off-by: Fluent Crafter <205769460+fluentcrafter@users.noreply.github.com> Co-authored-by: Roman Vaseev <4833306+Filter94@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Victorien Gauch <85494462+VGau@users.noreply.github.com> Co-authored-by: kyzooghost <73516204+kyzooghost@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com> Co-authored-by: François Bojarski <54240434+letypequividelespoubelles@users.noreply.github.com> Co-authored-by: jonesho <81145364+jonesho@users.noreply.github.com> Co-authored-by: Roman Vaseev <Filter94@users.noreply.github.com> Co-authored-by: Fluent Crafter <205769460+fluentcrafter@users.noreply.github.com> Co-authored-by: Olivier Bégassat <olivier.begassat.cours@gmail.com> Co-authored-by: Olivier Bégassat <38285177+OlivierBBB@users.noreply.github.com> Co-authored-by: VGau <victorien.gauch@consensys.net> --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: F Bojarski <ceciestunepoubelle@protonmail.ch> Signed-off-by: Fluent Crafter <205769460+fluentcrafter@users.noreply.github.com> Co-authored-by: Victorien Gauch <85494462+VGau@users.noreply.github.com> Co-authored-by: Fluent Crafter <205769460+fluentcrafter@users.noreply.github.com> Co-authored-by: Roman Vaseev <4833306+Filter94@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: kyzooghost <73516204+kyzooghost@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com> Co-authored-by: François Bojarski <54240434+letypequividelespoubelles@users.noreply.github.com> Co-authored-by: jonesho <81145364+jonesho@users.noreply.github.com> Co-authored-by: Roman Vaseev <Filter94@users.noreply.github.com> Co-authored-by: Olivier Bégassat <olivier.begassat.cours@gmail.com> Co-authored-by: Olivier Bégassat <38285177+OlivierBBB@users.noreply.github.com> Co-authored-by: VGau <victorien.gauch@consensys.net>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR implements issue(s) #
Checklist