Skip to content

fix: pin privy dependency versions#751

Merged
CarsonRoscoe merged 1 commit into
mainfrom
fix/privy-dependency-bug
Jun 9, 2025
Merged

fix: pin privy dependency versions#751
CarsonRoscoe merged 1 commit into
mainfrom
fix/privy-dependency-bug

Conversation

@CarsonRoscoe

@CarsonRoscoe CarsonRoscoe commented Jun 9, 2025

Copy link
Copy Markdown
Contributor

Description

Privy, specifically @privyio/server-auth, had a bunch of recent back-to-back updates that changed the underlying dependencies of the package. In doing so, a number of changes were made recently that break installing AgentKit.

I attempted to find the last version of @privyio/server-auth (and a corresponding @privyio/public-api released the same day) that satisfies this issue. However, in doing so, I discovered that there have been interface-breaking Privy updates. We cannot trust the minor versions or even patch versions to not introducing breaking changes to either dependencies or interfaces.

Therefore, this PR simply goes back to the last guaranteed trusted version, and pins it by removing the ^ on the dependency.

We can fast-follow with a more robust solution, however this hotfix is needed to fix new installations of AgentKit.

Tests

Validate the issue by running npm run create onchain-agent@latest, and run through the setup with Langchain. Run npm install, try to chat with the bot, and you will see a series of dependency issues in the console, all stemming from privyio server.

Doing the same locally with this new version produces an agent example which works out of the box

@CarsonRoscoe
CarsonRoscoe requested a review from 0xRAG June 9, 2025 21:22
@cb-heimdall

cb-heimdall commented Jun 9, 2025

Copy link
Copy Markdown

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@CarsonRoscoe
CarsonRoscoe marked this pull request as ready for review June 9, 2025 21:22
@CarsonRoscoe
CarsonRoscoe requested a review from murrlincoln as a code owner June 9, 2025 21:22
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jun 9, 2025
@CarsonRoscoe
CarsonRoscoe force-pushed the fix/privy-dependency-bug branch 3 times, most recently from 7f4639e to b96b49e Compare June 9, 2025 21:44
@CarsonRoscoe
CarsonRoscoe force-pushed the fix/privy-dependency-bug branch from b96b49e to cfc0ad9 Compare June 9, 2025 21:52
@CarsonRoscoe
CarsonRoscoe merged commit 97408c1 into main Jun 9, 2025
28 checks passed
@CarsonRoscoe
CarsonRoscoe deleted the fix/privy-dependency-bug branch June 9, 2025 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Development

Successfully merging this pull request may close these issues.

3 participants