Skip to content

Comments

fix(ci, signer-aws): increase recursion limit to fix error: queries overflow the depth limit!#3633

Merged
zerosnacks merged 4 commits intomainfrom
zerosnacks/fix-clippy-issue
Feb 2, 2026
Merged

fix(ci, signer-aws): increase recursion limit to fix error: queries overflow the depth limit!#3633
zerosnacks merged 4 commits intomainfrom
zerosnacks/fix-clippy-issue

Conversation

@zerosnacks
Copy link
Member

@zerosnacks zerosnacks commented Feb 2, 2026

Motivation

Fixes CI issue:

error: queries overflow the depth limit!
  |
  = help: consider increasing the recursion limit by adding a `#![recursion_limit = "256"]` attribute to your crate (`alloy_signer_aws`)
  = note: query depth increased by 130 when computing layout of `{async block@crates/signer-aws/src/signer.rs:120:5: 120:23}`

This is breaking downstream CI's as aws-* is pinned to major. I have not fully looked into why this is happening, I can't find reports on the respective repo.

To repro:

cargo update
cargo +nightly clippy --all-features

Solution

Bumps to #![recursion_limit = "256"]

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

error: queries overflow the depth limit!
  |
  = help: consider increasing the recursion limit by adding a `#![recursion_limit = "256"]` attribute to your crate (`alloy_signer_aws`)
  = note: query depth increased by 130 when computing layout of `{async block@crates/signer-aws/src/signer.rs:120:5: 120:23}`
@zerosnacks zerosnacks changed the title chore(ci): increase recursion limit to fix error: queries overflow the depth limit! fix(ci): increase recursion limit to fix error: queries overflow the depth limit! Feb 2, 2026
@zerosnacks zerosnacks changed the title fix(ci): increase recursion limit to fix error: queries overflow the depth limit! fix(ci, signer-aws): increase recursion limit to fix error: queries overflow the depth limit! Feb 2, 2026
@github-project-automation github-project-automation bot moved this to Reviewed in Alloy Feb 2, 2026
@zerosnacks zerosnacks enabled auto-merge (squash) February 2, 2026 11:17
@zerosnacks zerosnacks merged commit 782eed4 into main Feb 2, 2026
30 checks passed
@zerosnacks zerosnacks deleted the zerosnacks/fix-clippy-issue branch February 2, 2026 11:23
@github-project-automation github-project-automation bot moved this from Reviewed to Done in Alloy Feb 2, 2026
zerosnacks added a commit that referenced this pull request Feb 2, 2026
…ng regular 128 limit, undo #3633 (#3634)

remove instrumentation to prevent deep recursion violating regular 128 limit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants