Repository navigation
Conversation
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
Collaborator
🟡 Heimdall Review Status
|
This branch was successfully deployed
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
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.
What changed? Why?
Adds a
Limitssection to the Validity Transaction RPC page. It documents the default 64-predicate-per-transaction limit as a version-qualified implementation default, links to the existing expiry rules, and states that the docs do not yet specify a storage-read cost forstoragepredicates or propagation to other nodes.Closes knowledge gap
gap-20261005-validity-transactions-predicate-limits-propagati: "For base_sendRawTransactionValidity, what are the limits on number of predicates per transaction and storage-read cost, do validity transactions stay sequencer-local or propagate to other nodes, and what are the stable JSON-RPC error codes?"Notes to reviewers
Primary sources:
DEFAULT_MAX_VALIDITY_PREDICATES = 64incrates/execution/txpool/src/validity.rs@ v1.4.2 (same value onmain); enforced viavalidate_batchincrates/execution/txpool-rpc/src/rpc.rs@ v1.4.2.block_numberexpiry: samerpc.rs(validate_block_expiry_bounds/validate_has_block_expiry); already documented onfees-ordering-and-lifecycle.mdx.rpc.rsforwards accepted transactions to builders, which does not establish peer propagation. The page therefore says the docs do not yet specify these.Needs owner confirmation before merge (Base protocol / node team owning
base_sendRawTransactionValidity):How has it been tested?
node scripts/lint-mdx.js --files-from=<changed files>: 0 errors (1 pre-existing warning on the request example code block).node scripts/validate-docs-structure.js: only pre-existing redirect errors for/upgrades/denim/eip-8130, unrelated to this change./specifications/build-transaction/fees-ordering-and-lifecycleexists indocs/docs.json; the GitHub link returns 200.Screenshots
N/A (text-only change)
Generated with Toshi