Skip to content

fix(voyager/state-gno): template hardcoded core realm in query_packet_by_hash#5476

Merged
cor merged 1 commit into
unionlabs:mainfrom
tbruyelle:fix-gno-state-module-hardcoded-realm
Jun 30, 2026
Merged

fix(voyager/state-gno): template hardcoded core realm in query_packet_by_hash#5476
cor merged 1 commit into
unionlabs:mainfrom
tbruyelle:fix-gno-state-module-hardcoded-realm

Conversation

@tbruyelle

Copy link
Copy Markdown
Contributor

Problem

query_packet_by_hash in the gno state module _ands two PacketSend event filters (one matched by source_channel_id, one by packet_hash). The first filter's pkg_path is templated from ibc_core_realm, but the second hardcodes gno.land/r/core/ibc/v1/core.

For any gno deployment whose IBC core realm differs from that default — e.g. the onbloc port at gno.land/r/onbloc/ibc/union/core — the second filter matches zero events, the _and yields nothing, and query_packet_by_hash returns no transaction.

Fix

Template both pkg_path filters from ibc_core_realm (bind it once as a format! captured identifier, like channel_id/packet_hash already are, and drop the now-unused positional arg). Behavioral change is one filter; diff is +4/-4.

Notes

  • Found while wiring Voyager's gno modules against the onbloc gno.land/r/onbloc/ibc/union/core deployment; with ibc_core_realm set to that realm, packet-by-hash lookups resolve correctly after this change.
  • I did not build voyager locally for this change — relying on CI; it's a focused string-template fix with no logic change.

🤖 Disclosure: this PR was prepared by Claude (an AI agent) on behalf of @tbruyelle.

…_by_hash

The second PacketSend filter in query_packet_by_hash hardcoded the realm
"gno.land/r/core/ibc/v1/core", while the sibling filter is templated from
ibc_core_realm. For any deployment whose IBC core realm differs from that
default (e.g. gno.land/r/onbloc/ibc/union/core), the _and over the two filters
matches zero events, so packet-by-hash lookups return nothing. Template both
pkg_path filters from ibc_core_realm.

Assisted-By: Claude (AI agent)
@tbruyelle tbruyelle requested a review from benluelo as a code owner June 30, 2026 08:18
@vercel

vercel Bot commented Jun 30, 2026

Copy link
Copy Markdown

@tbruyelle is attempting to deploy a commit to the unionbuild Team on Vercel.

A member of the Team first needs to authorize it.

@cor cor merged commit a934195 into unionlabs:main Jun 30, 2026
142 of 148 checks passed
@tbruyelle tbruyelle deleted the fix-gno-state-module-hardcoded-realm branch June 30, 2026 11:16
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.

3 participants