Skip to content

fix(gltest): widen genlayer-py to >=0.13.0,<0.17.0#76

Merged
MuncleUscles merged 1 commit intomainfrom
fix/widen-genlayer-py-pin-to-016
Apr 20, 2026
Merged

fix(gltest): widen genlayer-py to >=0.13.0,<0.17.0#76
MuncleUscles merged 1 commit intomainfrom
fix/widen-genlayer-py-pin-to-016

Conversation

@MuncleUscles
Copy link
Copy Markdown
Member

@MuncleUscles MuncleUscles commented Apr 20, 2026

Summary

Follow-up to #74/#75. Two Bradbury-specific fixes landed in genlayer-py v0.13.0 that downstream consumers hit immediately on first contact:

  1. Cloudflare bypass: genlayer-py@31412c9 ("feat: Expose tx_execution_result…") added "User-Agent": "genlayer-py" to the GenLayerProvider POST headers. Without it, every RPC call to rpc-bradbury.genlayer.com returns a Just a moment... challenge page — Cloudflare blocks the default python-requests/X.Y.Z UA. Verified live: curl with any non-default UA returns 200; requests with its default UA returns 403 + CF challenge.
  2. Bradbury V06 receipt decoding: wait_for_transaction_receipt raises a bytes-as-string error on Bradbury txs with genlayer-py==0.12.1. Same flow works cleanly on 0.16.3 (tested against a live deploy on Bradbury — tx 0xbfe6867f…).

Both fixes are gated behind the current <0.13.0 cap from #75. Bumping to >=0.13.0,<0.17.0 covers the whole current released line (0.16.3 is latest) with an explicit upper bound, as CodeRabbit suggested on #74.

Test plan

  • pytest tests/gltest_cli/config/ -q → 68/68 green locally
  • With genlayer-py 0.16.3, wait_for_transaction_receipt against a live Bradbury tx returns a structured receipt (status_name: ACCEPTED, full last_round + validator set)
  • Cloudflare no longer blocks Rally's gltest --network testnet_bradbury invocations
  • CI green

Summary by CodeRabbit

  • Chores
    • Updated genlayer-py dependency to support versions 0.13.0 through 0.16.x, expanding compatibility with the latest upstream releases.

Two Bradbury-related fixes landed in genlayer-py v0.13.0 that downstream
users hit immediately:

1. Cloudflare bypass: genlayerlabs/genlayer-py@31412c9 added a
   `User-Agent: genlayer-py` header. Without it, every RPC call to
   rpc-bradbury.genlayer.com returns a "Just a moment..." challenge
   page because Cloudflare blocks the default `python-requests/*` UA.

2. tx_execution_result decoding for Bradbury V06 ABI. Receipts from
   Bradbury txs couldn't be deserialised under 0.12.x.

Bumping the floor to 0.13.0 unblocks both. Expanding the upper bound
to <0.17.0 covers the current released line (0.16.3) — no breaking
changes between 0.13.x and 0.16.x based on release notes.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 20, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The genlayer-py dependency version constraint in pyproject.toml was updated from >=0.12.1,<0.13.0 to >=0.13.0,<0.17.0, expanding the range of compatible versions and raising the minimum required version.

Changes

Cohort / File(s) Summary
Dependency Version Update
pyproject.toml
Updated genlayer-py dependency constraint to allow versions >=0.13.0,<0.17.0 instead of >=0.12.1,<0.13.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

🐰 A version bump hops along the way,
Dependencies dance in a wider ballet,
From thirteen it starts, to seventeen it gleams,
New heights unlocked by genlayer's dreams! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: widening the genlayer-py version constraint from >=0.12.1,<0.13.0 to >=0.13.0,<0.17.0, which is the only modification in this PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/widen-genlayer-py-pin-to-016

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@MuncleUscles MuncleUscles merged commit 462df02 into main Apr 20, 2026
2 of 3 checks passed
@MuncleUscles MuncleUscles deleted the fix/widen-genlayer-py-pin-to-016 branch April 20, 2026 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant