Skip to content

fix: bump genlayer-js for staking wizard min stake#282

Merged
MuncleUscles merged 1 commit intomainfrom
fix/bump-genlayer-js
Mar 17, 2026
Merged

fix: bump genlayer-js for staking wizard min stake#282
MuncleUscles merged 1 commit intomainfrom
fix/bump-genlayer-js

Conversation

@MuncleUscles
Copy link
Member

@MuncleUscles MuncleUscles commented Mar 17, 2026

Summary

  • Bump genlayer-js from ^0.21.0 to ^0.21.2
  • Picks up fix where getEpochInfo() now returns validatorMinStakeRaw and validatorMinStake
  • Fixes staking wizard crash: "Minimum stake required: undefined" → "Cannot mix BigInt and other types"

Test plan

  • npm run build succeeds
  • genlayer-js 0.21.2 published and includes the fix

Summary by CodeRabbit

  • Chores
    • Updated dependencies to latest versions.

getEpochInfo now returns validatorMinStakeRaw and validatorMinStake,
fixing the "Cannot mix BigInt and other types" crash in the staking
wizard.
@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0058e66d-9d69-4578-bd06-697a752afd24

📥 Commits

Reviewing files that changed from the base of the PR and between 5758705 and f8a02cf.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json

📝 Walkthrough

Walkthrough

The pull request updates the genlayer-js dependency from version ^0.21.0 to ^0.21.2 in the project's package configuration file, incorporating a patch-level version bump.

Changes

Cohort / File(s) Summary
Dependency Update
package.json
Bumped genlayer-js from ^0.21.0 to ^0.21.2 (patch version update).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tiny bump, so small and neat,
Version two-one brings a treat,
genlayer-js climbs higher still,
Dependencies dance to our will! ✨

🚥 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 summarizes the main change: bumping genlayer-js dependency to fix a staking wizard issue related to minimum stake validation.
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/bump-genlayer-js
📝 Coding Plan
  • Generate coding plan for human review comments

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 73d8809 into main Mar 17, 2026
3 checks passed
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