Skip to content

Conversation

@soinclined
Copy link
Contributor

Remove bigint-buffer dependency by upgrading @solana/web3.js to 2.0.0

Summary

This PR removes the bigint-buffer dependency from the worldstore-agent repository by upgrading @solana/web3.js from version 1.x to 2.0.0. The bigint-buffer dependency was only present as a transitive dependency through the chain: @solana/[email protected]@solana/buffer-layout-utilsbigint-buffer.

The solution uses pnpm overrides to force @solana/web3.js to version 2.0.0, which no longer depends on bigint-buffer as it now uses native BigInt support instead.

Key changes:

  • Added pnpm override for @solana/web3.js: "2.0.0" in package.json
  • Updated pnpm-lock.yaml with new dependency tree
  • Verified bigint-buffer is completely removed from dependency tree
  • Build and type checking still pass

Review & Testing Checklist for Human

Risk Level: 🟡 Medium (2 critical items to verify)

  • End-to-end functionality verification: Test core application flows that involve Solana operations to ensure the major version upgrade of @solana/web3.js doesn't break existing functionality
  • Runtime error checking: Start the application and verify no runtime errors occur during Solana-related operations, especially around BigInt/buffer conversions

Notes

  • This is a major version upgrade (1.x → 2.x) of @solana/web3.js, which could introduce breaking API changes
  • The upgrade uses pnpm overrides to force dependency versions, which may cause peer dependency warnings (visible in install output)
  • No direct code changes were needed since the repository doesn't directly import bigint-buffer or @solana/web3.js APIs
  • @solana/[email protected] is marked as deprecated in favor of @solana/kit, but this approach still achieves the goal of removing bigint-buffer

Link to Devin run: https://app.devin.ai/sessions/c0085a46aa9a43f6b63a1b6b44cfab6b
Requested by: @soinclined

@devin-ai-integration
Copy link

Original prompt from Penelope
Please go to each of the following repositories and see if you can you remove the dependency to bigint-buffer and replace it with an alternative such as BigInt without breaking anything?

crossmint/worldstore-agent


crossmint/office-snacks-slack-bot

@devin-ai-integration
Copy link

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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