Sync upstream v11.2.7 - #195
Conversation
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…lockscout#14674) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
Code Review
This pull request upgrades Blockscout to version 11.2.7, introduces a lightweight transaction preview endpoint, optimizes ENS and metadata preloading using concurrent requests, and implements a pooled HTTP client using Finch to reuse connections. It also adds caching for contract methods and fixes state changes for Eden sponsored transactions. The review feedback highlights critical issues, including unresolved merge conflicts in common-blockscout.env and mix.lock, a bug in HttpClient where Finch.Error is not properly rescued, and several potential runtime crashes due to missing safety checks for nil values or unconfigured environment variables.
d3c3751 to
4f8c2b8
Compare
3d005dd to
e05174d
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_c58c6bc5-00c5-48d5-b950-420a8f18a9d6) |
e05174d to
07eb854
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_a9588607-baa6-4066-9ba7-06a0d0e9538b) |
07eb854 to
1425d31
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_9b37a20c-d99e-4527-a316-89a90d6aa259) |
1425d31 to
87dfa5a
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_7167e7cd-1b4a-4c4d-8a5a-4fd646969274) |
87dfa5a to
31d8c92
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_34248327-cebd-4ff2-b17f-e1a1bfb2641d) |
31d8c92 to
a43292f
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_f64b5c6a-97bb-45b3-a636-bce966f6b9d8) |
a43292f to
15003bc
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_965d4803-1105-4910-a95d-b1a6c5175c23) |
15003bc to
ae88280
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_97553bac-5c98-498b-a8a5-0132cf28f7e9) |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_acad6b6e-2ec3-4ca7-8a70-829fd14bed05) |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_9309d0f9-a5e0-4473-8073-f5a022acab6e) |
7a4a0a8 to
139e324
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_66b3c76c-67ed-4592-962f-eb1323c78669) |
Upstream Sync - v11.2.7
Auto-merge with upstream
v11.2.7failed. Version/workflow conflicts were auto-resolved,but the following files have code conflicts that need manual resolution:
To resolve:
v11.2.7to trigger Docker buildUpstream release notes
Note
Medium Risk
Touches high-traffic API preload, HTTP pooling, proxy-implementation caching, and transaction participant loading. Behavior is mostly additive/perf, but mis-tuned pools or cache TTLs can change latency and proxy-detection freshness.
Overview
v11.2.7 adds
GET /api/v2/transactions/{hash}/previewfor OG/social embeds (opt-in ENS/metadata/input decode) and a set of API latency cuts.ENS and metadata preloads now run concurrently through
maybe_preload_ens_and_metadata, over a new Finch keep-alive client with separate pools for short preloads vs long proxy calls (MICROSERVICE_HTTP_POOL_SIZE/MICROSERVICE_HTTP_POOL_COUNT). Transaction detail loads participants in one deduped pass, skips ABIs where unused, batches address tags, and folds address existence checks into a single query. Empty “not a proxy” results for self-verified contracts use a longer TTL (CONTRACT_PROXY_EMPTY_IMPLEMENTATION_DATA_CACHE_TTL); method-id lookups are cached.Eden: sponsored txs attribute fees to the fee payer, credit batched call recipients (including unindexed addresses), and include fee payer/calls in interpretation requests. Indexer address/coin-balance extraction follows the same call/fee-payer set. ZkSync address-transform doctests no longer treat internal-tx
created_contract_codeas deployed bytecode.Reviewed by Cursor Bugbot for commit 139e324. Bugbot is set up for automated code reviews on this repo. Configure here.