feat: refactor npm to pnpm by CarsonRoscoe · Pull Request #613 · coinbase/agentkit

CarsonRoscoe · 2025-03-26T19:58:24Z

Description

pnpm is a mostly-drop in replacement for npm, focused on performance.

Some benefits include:

Faster install via content-addressable storage and caching
Stricter dependency resolution, prevents hidden hoisting issues
Smaller node_modules size through symlinks
Better monorepo support, as it natively aligns well with Turbo's workspace model

This PR refactors our turbo monorepo to use pnpm instead of npm. This requires:

Migrating the node repos package management system
Updating the CI/CD workflows to use pnpm
Fixing any dependency issues due to the stricter environment
Updating the README's and documentation
Linking dependencies between workspaces via workspace:* syntax, allowing pnpm to resolve versions for us during pnpm publish, removing bump step during releases

Checklist

A couple of things to include in your PR for completeness:

Added documentation to all relevant README.md files

cb-heimdall · 2025-03-26T19:58:33Z

✅ 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

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 · 2025-03-26T19:58:54Z

 node --version
-npm list @coinbase/agentkit
-npm list @coinbase/agentkit-langchain # if using LangChain extension
+pnpm -r list @coinbase/agentkit


npm list is recursive by default, while pnpm list is not. Hence the added -r flag to recurse

CarsonRoscoe · 2025-03-26T20:42:01Z

 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v4
+ - name: Setup pnpm


pnpm docs recommend using pnpm/action-setup@v4 to setup pnpm before running actions/setup-node@v4
sha pin to v4.1.0

CarsonRoscoe · 2025-03-26T20:50:59Z

+packages:
+ - "agentkit"
+ - "create-onchain-agent"
+ - "create-onchain-agent/templates/*"


I figured a minor devx improvement could be made here, so contributors do not need to remember to add to this list. However, I am open to reversing this back to individually listing each package if we want to be explicit

(For reference, this list used to live in package.json)

CarsonRoscoe · 2025-03-26T20:52:10Z

+ "outputs": [
+ "dist/**",
+ ".next/**",
+ "!.next/cache/**",


Fixes warnings from templates that get added to the workspace, but whose build process differs from AgentKit traditionally builds to /dist.
Specifically, the CLI templates for Next and Mcp Server require this to get rid of the cli warnings on build

Should we even be building the templates as part of our normal build process? If/when the templates incorporate nunjucks, the builds will stop working

(not blocking, can be a follow-up / still up for debate)

True, right now we're just building the projects that don't use nunjucks. Next was not affected by that PR, I just used it for the new generate commands.
I am fine with straight up ignoring them in a follow

0xRAG

🚀

CarsonRoscoe · 2025-03-26T19:58:24Z

Description

pnpm is a mostly-drop in replacement for npm, focused on performance.

Some benefits include:

Faster install via content-addressable storage and caching
Stricter dependency resolution, prevents hidden hoisting issues
Smaller node_modules size through symlinks
Better monorepo support, as it natively aligns well with Turbo's workspace model

This PR refactors our turbo monorepo to use pnpm instead of npm. This requires:

Migrating the node repos package management system
Updating the CI/CD workflows to use pnpm
Fixing any dependency issues due to the stricter environment
Updating the README's and documentation
Linking dependencies between workspaces via workspace:* syntax, allowing pnpm to resolve versions for us during pnpm publish, removing bump step during releases

Checklist

A couple of things to include in your PR for completeness:

Added documentation to all relevant README.md files

cb-heimdall · 2025-03-26T19:58:33Z

✅ 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

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 · 2025-03-26T19:58:54Z

 node --version
-npm list @coinbase/agentkit
-npm list @coinbase/agentkit-langchain # if using LangChain extension
+pnpm -r list @coinbase/agentkit


npm list is recursive by default, while pnpm list is not. Hence the added -r flag to recurse

CarsonRoscoe · 2025-03-26T20:42:01Z

 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v4
+ - name: Setup pnpm


pnpm docs recommend using pnpm/action-setup@v4 to setup pnpm before running actions/setup-node@v4
sha pin to v4.1.0

CarsonRoscoe · 2025-03-26T20:50:59Z

+packages:
+ - "agentkit"
+ - "create-onchain-agent"
+ - "create-onchain-agent/templates/*"


I figured a minor devx improvement could be made here, so contributors do not need to remember to add to this list. However, I am open to reversing this back to individually listing each package if we want to be explicit

(For reference, this list used to live in package.json)

CarsonRoscoe · 2025-03-26T20:52:10Z

+ "outputs": [
+ "dist/**",
+ ".next/**",
+ "!.next/cache/**",


Fixes warnings from templates that get added to the workspace, but whose build process differs from AgentKit traditionally builds to /dist.
Specifically, the CLI templates for Next and Mcp Server require this to get rid of the cli warnings on build

Should we even be building the templates as part of our normal build process? If/when the templates incorporate nunjucks, the builds will stop working

(not blocking, can be a follow-up / still up for debate)

True, right now we're just building the projects that don't use nunjucks. Next was not affected by that PR, I just used it for the new generate commands.
I am fine with straight up ignoring them in a follow

0xRAG

🚀

CarsonRoscoe · 2025-03-26T19:58:24Z

Description

pnpm is a mostly-drop in replacement for npm, focused on performance.

Some benefits include:

Faster install via content-addressable storage and caching
Stricter dependency resolution, prevents hidden hoisting issues
Smaller node_modules size through symlinks
Better monorepo support, as it natively aligns well with Turbo's workspace model

This PR refactors our turbo monorepo to use pnpm instead of npm. This requires:

Migrating the node repos package management system
Updating the CI/CD workflows to use pnpm
Fixing any dependency issues due to the stricter environment
Updating the README's and documentation
Linking dependencies between workspaces via workspace:* syntax, allowing pnpm to resolve versions for us during pnpm publish, removing bump step during releases

Checklist

A couple of things to include in your PR for completeness:

Added documentation to all relevant README.md files

cb-heimdall · 2025-03-26T19:58:33Z

✅ 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

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 · 2025-03-26T19:58:54Z

 node --version
-npm list @coinbase/agentkit
-npm list @coinbase/agentkit-langchain # if using LangChain extension
+pnpm -r list @coinbase/agentkit


npm list is recursive by default, while pnpm list is not. Hence the added -r flag to recurse

CarsonRoscoe · 2025-03-26T20:42:01Z

 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v4
+ - name: Setup pnpm


pnpm docs recommend using pnpm/action-setup@v4 to setup pnpm before running actions/setup-node@v4
sha pin to v4.1.0

CarsonRoscoe · 2025-03-26T20:50:59Z

+packages:
+ - "agentkit"
+ - "create-onchain-agent"
+ - "create-onchain-agent/templates/*"


I figured a minor devx improvement could be made here, so contributors do not need to remember to add to this list. However, I am open to reversing this back to individually listing each package if we want to be explicit

(For reference, this list used to live in package.json)

CarsonRoscoe · 2025-03-26T20:52:10Z

+ "outputs": [
+ "dist/**",
+ ".next/**",
+ "!.next/cache/**",


Fixes warnings from templates that get added to the workspace, but whose build process differs from AgentKit traditionally builds to /dist.
Specifically, the CLI templates for Next and Mcp Server require this to get rid of the cli warnings on build

Should we even be building the templates as part of our normal build process? If/when the templates incorporate nunjucks, the builds will stop working

(not blocking, can be a follow-up / still up for debate)

True, right now we're just building the projects that don't use nunjucks. Next was not affected by that PR, I just used it for the new generate commands.
I am fine with straight up ignoring them in a follow

0xRAG

🚀

CarsonRoscoe · 2025-03-26T19:58:24Z

Description

pnpm is a mostly-drop in replacement for npm, focused on performance.

Some benefits include:

Faster install via content-addressable storage and caching
Stricter dependency resolution, prevents hidden hoisting issues
Smaller node_modules size through symlinks
Better monorepo support, as it natively aligns well with Turbo's workspace model

This PR refactors our turbo monorepo to use pnpm instead of npm. This requires:

Migrating the node repos package management system
Updating the CI/CD workflows to use pnpm
Fixing any dependency issues due to the stricter environment
Updating the README's and documentation
Linking dependencies between workspaces via workspace:* syntax, allowing pnpm to resolve versions for us during pnpm publish, removing bump step during releases

Checklist

A couple of things to include in your PR for completeness:

Added documentation to all relevant README.md files

cb-heimdall · 2025-03-26T19:58:33Z

✅ 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

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 · 2025-03-26T19:58:54Z

 node --version
-npm list @coinbase/agentkit
-npm list @coinbase/agentkit-langchain # if using LangChain extension
+pnpm -r list @coinbase/agentkit


npm list is recursive by default, while pnpm list is not. Hence the added -r flag to recurse

CarsonRoscoe · 2025-03-26T20:42:01Z

 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v4
+ - name: Setup pnpm


pnpm docs recommend using pnpm/action-setup@v4 to setup pnpm before running actions/setup-node@v4
sha pin to v4.1.0

CarsonRoscoe · 2025-03-26T20:50:59Z

+packages:
+ - "agentkit"
+ - "create-onchain-agent"
+ - "create-onchain-agent/templates/*"


I figured a minor devx improvement could be made here, so contributors do not need to remember to add to this list. However, I am open to reversing this back to individually listing each package if we want to be explicit

(For reference, this list used to live in package.json)

CarsonRoscoe · 2025-03-26T20:52:10Z

+ "outputs": [
+ "dist/**",
+ ".next/**",
+ "!.next/cache/**",


Fixes warnings from templates that get added to the workspace, but whose build process differs from AgentKit traditionally builds to /dist.
Specifically, the CLI templates for Next and Mcp Server require this to get rid of the cli warnings on build

Should we even be building the templates as part of our normal build process? If/when the templates incorporate nunjucks, the builds will stop working

(not blocking, can be a follow-up / still up for debate)

True, right now we're just building the projects that don't use nunjucks. Next was not affected by that PR, I just used it for the new generate commands.
I am fine with straight up ignoring them in a follow

0xRAG

🚀

CarsonRoscoe · 2025-03-26T19:58:24Z

Description

pnpm is a mostly-drop in replacement for npm, focused on performance.

Some benefits include:

Faster install via content-addressable storage and caching
Stricter dependency resolution, prevents hidden hoisting issues
Smaller node_modules size through symlinks
Better monorepo support, as it natively aligns well with Turbo's workspace model

This PR refactors our turbo monorepo to use pnpm instead of npm. This requires:

Migrating the node repos package management system
Updating the CI/CD workflows to use pnpm
Fixing any dependency issues due to the stricter environment
Updating the README's and documentation
Linking dependencies between workspaces via workspace:* syntax, allowing pnpm to resolve versions for us during pnpm publish, removing bump step during releases

Checklist

A couple of things to include in your PR for completeness:

Added documentation to all relevant README.md files

cb-heimdall · 2025-03-26T19:58:33Z

✅ 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

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 · 2025-03-26T19:58:54Z

 node --version
-npm list @coinbase/agentkit
-npm list @coinbase/agentkit-langchain # if using LangChain extension
+pnpm -r list @coinbase/agentkit


npm list is recursive by default, while pnpm list is not. Hence the added -r flag to recurse

CarsonRoscoe · 2025-03-26T20:42:01Z

 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v4
+ - name: Setup pnpm


pnpm docs recommend using pnpm/action-setup@v4 to setup pnpm before running actions/setup-node@v4
sha pin to v4.1.0

CarsonRoscoe · 2025-03-26T20:50:59Z

+packages:
+ - "agentkit"
+ - "create-onchain-agent"
+ - "create-onchain-agent/templates/*"


I figured a minor devx improvement could be made here, so contributors do not need to remember to add to this list. However, I am open to reversing this back to individually listing each package if we want to be explicit

(For reference, this list used to live in package.json)

CarsonRoscoe · 2025-03-26T20:52:10Z

+ "outputs": [
+ "dist/**",
+ ".next/**",
+ "!.next/cache/**",


Fixes warnings from templates that get added to the workspace, but whose build process differs from AgentKit traditionally builds to /dist.
Specifically, the CLI templates for Next and Mcp Server require this to get rid of the cli warnings on build

Should we even be building the templates as part of our normal build process? If/when the templates incorporate nunjucks, the builds will stop working

(not blocking, can be a follow-up / still up for debate)

True, right now we're just building the projects that don't use nunjucks. Next was not affected by that PR, I just used it for the new generate commands.
I am fine with straight up ignoring them in a follow

0xRAG

🚀

CarsonRoscoe · 2025-03-26T19:58:24Z

Description

pnpm is a mostly-drop in replacement for npm, focused on performance.

Some benefits include:

Faster install via content-addressable storage and caching
Stricter dependency resolution, prevents hidden hoisting issues
Smaller node_modules size through symlinks
Better monorepo support, as it natively aligns well with Turbo's workspace model

This PR refactors our turbo monorepo to use pnpm instead of npm. This requires:

Migrating the node repos package management system
Updating the CI/CD workflows to use pnpm
Fixing any dependency issues due to the stricter environment
Updating the README's and documentation
Linking dependencies between workspaces via workspace:* syntax, allowing pnpm to resolve versions for us during pnpm publish, removing bump step during releases

Checklist

A couple of things to include in your PR for completeness:

Added documentation to all relevant README.md files

cb-heimdall · 2025-03-26T19:58:33Z

✅ 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

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 · 2025-03-26T19:58:54Z

 node --version
-npm list @coinbase/agentkit
-npm list @coinbase/agentkit-langchain # if using LangChain extension
+pnpm -r list @coinbase/agentkit


npm list is recursive by default, while pnpm list is not. Hence the added -r flag to recurse

CarsonRoscoe · 2025-03-26T20:42:01Z

 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v4
+ - name: Setup pnpm


pnpm docs recommend using pnpm/action-setup@v4 to setup pnpm before running actions/setup-node@v4
sha pin to v4.1.0

CarsonRoscoe · 2025-03-26T20:50:59Z

+packages:
+ - "agentkit"
+ - "create-onchain-agent"
+ - "create-onchain-agent/templates/*"


I figured a minor devx improvement could be made here, so contributors do not need to remember to add to this list. However, I am open to reversing this back to individually listing each package if we want to be explicit

(For reference, this list used to live in package.json)

CarsonRoscoe · 2025-03-26T20:52:10Z

+ "outputs": [
+ "dist/**",
+ ".next/**",
+ "!.next/cache/**",


Fixes warnings from templates that get added to the workspace, but whose build process differs from AgentKit traditionally builds to /dist.
Specifically, the CLI templates for Next and Mcp Server require this to get rid of the cli warnings on build

Should we even be building the templates as part of our normal build process? If/when the templates incorporate nunjucks, the builds will stop working

(not blocking, can be a follow-up / still up for debate)

True, right now we're just building the projects that don't use nunjucks. Next was not affected by that PR, I just used it for the new generate commands.
I am fine with straight up ignoring them in a follow

0xRAG

🚀

CarsonRoscoe · 2025-03-26T19:58:24Z

Description

pnpm is a mostly-drop in replacement for npm, focused on performance.

Some benefits include:

Faster install via content-addressable storage and caching
Stricter dependency resolution, prevents hidden hoisting issues
Smaller node_modules size through symlinks
Better monorepo support, as it natively aligns well with Turbo's workspace model

This PR refactors our turbo monorepo to use pnpm instead of npm. This requires:

Migrating the node repos package management system
Updating the CI/CD workflows to use pnpm
Fixing any dependency issues due to the stricter environment
Updating the README's and documentation
Linking dependencies between workspaces via workspace:* syntax, allowing pnpm to resolve versions for us during pnpm publish, removing bump step during releases

Checklist

A couple of things to include in your PR for completeness:

Added documentation to all relevant README.md files

cb-heimdall · 2025-03-26T19:58:33Z

✅ 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

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 · 2025-03-26T19:58:54Z

 node --version
-npm list @coinbase/agentkit
-npm list @coinbase/agentkit-langchain # if using LangChain extension
+pnpm -r list @coinbase/agentkit


npm list is recursive by default, while pnpm list is not. Hence the added -r flag to recurse

CarsonRoscoe · 2025-03-26T20:42:01Z

 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v4
+ - name: Setup pnpm


pnpm docs recommend using pnpm/action-setup@v4 to setup pnpm before running actions/setup-node@v4
sha pin to v4.1.0

CarsonRoscoe · 2025-03-26T20:50:59Z

+packages:
+ - "agentkit"
+ - "create-onchain-agent"
+ - "create-onchain-agent/templates/*"


I figured a minor devx improvement could be made here, so contributors do not need to remember to add to this list. However, I am open to reversing this back to individually listing each package if we want to be explicit

(For reference, this list used to live in package.json)

CarsonRoscoe · 2025-03-26T20:52:10Z

+ "outputs": [
+ "dist/**",
+ ".next/**",
+ "!.next/cache/**",


Fixes warnings from templates that get added to the workspace, but whose build process differs from AgentKit traditionally builds to /dist.
Specifically, the CLI templates for Next and Mcp Server require this to get rid of the cli warnings on build

Should we even be building the templates as part of our normal build process? If/when the templates incorporate nunjucks, the builds will stop working

(not blocking, can be a follow-up / still up for debate)

True, right now we're just building the projects that don't use nunjucks. Next was not affected by that PR, I just used it for the new generate commands.
I am fine with straight up ignoring them in a follow

0xRAG

🚀

CarsonRoscoe · 2025-03-26T19:58:24Z

Description

pnpm is a mostly-drop in replacement for npm, focused on performance.

Some benefits include:

Faster install via content-addressable storage and caching
Stricter dependency resolution, prevents hidden hoisting issues
Smaller node_modules size through symlinks
Better monorepo support, as it natively aligns well with Turbo's workspace model

This PR refactors our turbo monorepo to use pnpm instead of npm. This requires:

Migrating the node repos package management system
Updating the CI/CD workflows to use pnpm
Fixing any dependency issues due to the stricter environment
Updating the README's and documentation
Linking dependencies between workspaces via workspace:* syntax, allowing pnpm to resolve versions for us during pnpm publish, removing bump step during releases

Checklist

A couple of things to include in your PR for completeness:

Added documentation to all relevant README.md files

cb-heimdall · 2025-03-26T19:58:33Z

✅ 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

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 · 2025-03-26T19:58:54Z

 node --version
-npm list @coinbase/agentkit
-npm list @coinbase/agentkit-langchain # if using LangChain extension
+pnpm -r list @coinbase/agentkit


npm list is recursive by default, while pnpm list is not. Hence the added -r flag to recurse

CarsonRoscoe · 2025-03-26T20:42:01Z

 runs-on: ubuntu-latest
 steps:
 - uses: actions/checkout@v4
+ - name: Setup pnpm


pnpm docs recommend using pnpm/action-setup@v4 to setup pnpm before running actions/setup-node@v4
sha pin to v4.1.0

CarsonRoscoe · 2025-03-26T20:50:59Z

+packages:
+ - "agentkit"
+ - "create-onchain-agent"
+ - "create-onchain-agent/templates/*"


I figured a minor devx improvement could be made here, so contributors do not need to remember to add to this list. However, I am open to reversing this back to individually listing each package if we want to be explicit

(For reference, this list used to live in package.json)

CarsonRoscoe · 2025-03-26T20:52:10Z

+ "outputs": [
+ "dist/**",
+ ".next/**",
+ "!.next/cache/**",


Fixes warnings from templates that get added to the workspace, but whose build process differs from AgentKit traditionally builds to /dist.
Specifically, the CLI templates for Next and Mcp Server require this to get rid of the cli warnings on build

Should we even be building the templates as part of our normal build process? If/when the templates incorporate nunjucks, the builds will stop working

(not blocking, can be a follow-up / still up for debate)

True, right now we're just building the projects that don't use nunjucks. Next was not affected by that PR, I just used it for the new generate commands.
I am fine with straight up ignoring them in a follow

0xRAG

🚀

github-actionsBot added documentation Improvements or additions to documentation wallet provider New wallet provider example New example agent framework extension New framework extension typescript labels Mar 26, 2025

CarsonRoscoe commented Mar 26, 2025
View reviewed changes

Comment thread.github/workflows/format.yml

CarsonRoscoe commented Mar 26, 2025
View reviewed changes

Comment threadtypescript/agentkit/src/wallet-providers/solanaKeypairWalletProvider.ts Outdated

CarsonRoscoe commented Mar 26, 2025
View reviewed changes

Comment threadtypescript/framework-extensions/langchain/README.md Outdated

CarsonRoscoe commented Mar 26, 2025
View reviewed changes

CarsonRoscoe marked this pull request as ready for review March 26, 2025 20:52

CarsonRoscoe requested a review from murrlincoln as a code owner March 26, 2025 20:52

CarsonRoscoe requested review from 0xRAG and John-peterson-coinbase March 26, 2025 20:55

0xRAG reviewed Mar 26, 2025
View reviewed changes

Comment thread.github/workflows/publish_nightly.yml Outdated

0xRAG reviewed Mar 26, 2025
View reviewed changes

Comment threadCONTRIBUTING-TYPESCRIPT.md Outdated

0xRAG reviewed Mar 26, 2025
View reviewed changes

Comment threadtypescript/.changeset/quick-ravens-act.md Outdated

0xRAG reviewed Mar 26, 2025
View reviewed changes

Comment threadtypescript/agentkit/package.json Outdated

0xRAG reviewed Mar 26, 2025
View reviewed changes

Comment threadtypescript/framework-extensions/vercel-ai-sdk/README.md Outdated

github-actionsBot removed the example New example agent label Mar 26, 2025

0xRAG reviewed Mar 26, 2025
View reviewed changes

Comment threadCONTRIBUTING-TYPESCRIPT.md Outdated

github-actionsBot added the example New example agent label Mar 26, 2025

CarsonRoscoe force-pushed the feat/pnpm branch 2 times, most recently from 6b86cef to c0d4607Compare March 27, 2025 15:46

CarsonRoscoe requested a review from 0xRAG March 27, 2025 21:18

0xRAG reviewed Mar 27, 2025
View reviewed changes

Comment threadtypescript/agentkit/package.json Outdated

CarsonRoscoe force-pushed the feat/pnpm branch from cd63bfa to b276b27Compare March 27, 2025 21:45

CarsonRoscoe added 10 commits March 27, 2025 17:45

chore: updated readme to include pnpm installation instructions
ac77ce9

chore: removed test:dry-run commands
1cb991c

chore: reverted user-facing comments to reference npm rather than pnpm
8b2d786

chore: removed changelog entry
0e06970

feat: link subprojects dependening on each other via workspace:* pattern
64920f2

chore: changed recommended node manager from nvm to fnm
501b90a

chore: removed .npmrc and added missing dependencies
3e4fb70

chore: refactored npx jest to pnpm jest
4379b47

fix: dependencies in examples
862b506

fix: bs58 dependency issue
9144952

CarsonRoscoe force-pushed the feat/pnpm branch from b276b27 to 9144952Compare March 27, 2025 21:47

feat: updated bs58 dependency to ^4.0.1
446408f

CarsonRoscoe force-pushed the feat/pnpm branch from be5d15a to 446408fCompare March 27, 2025 21:59

fix: jest hoisting
c6b4464

CarsonRoscoe force-pushed the feat/pnpm branch 2 times, most recently from e403380 to c6b4464Compare March 27, 2025 22:06

CarsonRoscoe added 2 commits March 27, 2025 18:53

fix: unit tests
d77b7ea

chore: npm to pnpm migration note for contributors
113cfb6

0xRAG reviewed Mar 27, 2025
View reviewed changes

Comment threadCONTRIBUTING-TYPESCRIPT.md Outdated

0xRAG reviewed Mar 27, 2025
View reviewed changes

Comment threadtypescript/agentkit/src/wallet-providers/solanaKeypairWalletProvider.ts Outdated

0xRAG reviewed Mar 27, 2025
View reviewed changes

Comment threadtypescript/agentkit/src/wallet-providers/solanaKeypairWalletProvider.ts Outdated

0xRAG reviewed Mar 27, 2025
View reviewed changes

Comment threadtypescript/agentkit/src/wallet-providers/solanaKeypairWalletProvider.ts Outdated

0xRAG reviewed Mar 27, 2025
View reviewed changes

Comment threadtypescript/examples/langchain-solana-chatbot/chatbot.ts Outdated

0xRAG reviewed Mar 27, 2025
View reviewed changes

Comment threadtypescript/examples/langchain-solana-chatbot/package.json Outdated

fix: pr feedback
5df686a

CarsonRoscoe requested a review from 0xRAG March 27, 2025 23:19

chore: format
f07451a

0xRAG approved these changes Mar 27, 2025
View reviewed changes

CarsonRoscoe merged commit 6938b5d into mainMar 27, 2025

CarsonRoscoe deleted the feat/pnpm branch March 27, 2025 23:35