chore(genvm): migrate to GenVM v0.3.0-rc7 SDK APIs ⬆️ - #106
Open
kp2pml30 wants to merge 3 commits into
Open
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
kp2pml30
force-pushed
the
chore/update-genvm-runner-hashes
branch
from
August 19, 2026 06:59
89a9efc to
fb5ce97
Compare
kp2pml30
marked this pull request as ready for review
August 19, 2026 13:01
MuncleUscles
approved these changes
Aug 20, 2026
Squashed: the v0.3.0-rc7 SDK API migration and the follow-up rename of the raw gl_call request tags to their v1 names.
kp2pml30
force-pushed
the
chore/update-genvm-runner-hashes
branch
from
August 21, 2026 11:33
467e815 to
df6c68a
Compare
The pinned embeddings runner's `VecDB` takes the distance metric as a fourth type parameter. Three arguments now fail at contract import, which leaves the deploy without state rather than reporting a type error.
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
2 participants
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.
Description
Migrates gltest to the GenVM v0.3.0-rc7 SDK. No runner-hash bump — this repo pins zero hashes, every fixture uses
py-genlayer:latest.on="accepted"→on="decided"in 4 contract fixturesgltest/direct/sdk_loader.pyreads both.zipand.tarrunner archives — upstream changed the built tree from<id>/<aa>/<rest>.tarto.zip, while the v0.2legacy-runners/tree stays.tarand both are resolved by the same globDeployContract/PostMessagerequest names and rc7'sEmitInternalDeployMessage/EmitInternalMessage, keeping native CI compatible while the multi-repo cut landsUpstream: genvm-manager #24.
Depends-On: genlayerlabs/genvm-manager#24
Motivation and Context
Direct mode extracts runners straight out of the GenVM tree, so the archive-format change breaks it outright. The
on=rename has no back-compat alias.How Has This Been Tested?
Python 3.13 native-CI environment (
uv sync --extra sim):tests/glsimsuite: 72 passedExtraction logic additionally verified standalone against a synthetic tree: zip-by-hash, tar-by-hash and
latestall resolve, and path-sort preference still putsrunners/(v0.3) ahead oflegacy-runners/for:latest.Decisions Made
gltest/assertions.pyACCEPTED_STATUSES/_has_accepted_statusis a consensus transaction status, not the SDKon=value — untouched.gltest/contracts/*already spokedecided/finalized.tar.xznames insdk_loader.pyare genvm-manager release bundles, a different artifact from runner archives — untouchedRisks
The
.zippath usesZipFile.extractallwith no equivalent of tar'sfilter="data"(stdlib offers none). Inputs are our own release artifacts, the same trust level as the existing tar path, but it is a slightly weaker guard against crafted entry names.Changeset
v0.6-devv0.6-devv0.123-devv0.19-devv0.30-devmainmainNo PR for genlayer-js or genlayer-consensus — neither needed a change.