feat: add support for basic vyper project using forge init --vyper by startup-dreamer · Pull Request #9930 · foundry-rs/foundry

startup-dreamer · 2025-02-21T05:10:28Z

Closes: #8340

This PR aims to add forge init --vyper command which instantiate a basic vyper project like this project as suggested here #8340 (comment).

~~This command mimics forge init --template https://github.com/Patronum-Labs/foundry-vyper currently the standard way of initializing a vyper project.~~

PR Checklist

Added Tests
Added Documentation
Breaking changes

zerosnacks

Hi @startup-dreamer, thanks for your PR

I would prefer if the Vyper template used local files, see: https://github.com/foundry-rs/foundry/tree/master/crates/forge/assets

We will have to move the Solidity ones in their respective subfolder in the assets directory and create a Vyper equivalent

Thanks!

startup-dreamer · 2025-02-21T10:26:06Z

Makes sense, I'll do the changes.

zerosnacks · 2025-03-27T16:54:31Z

Hey @startup-dreamer, if time permits - would be great to get this in! Thanks

startup-dreamer · 2025-03-27T22:50:21Z

Hey @startup-dreamer, if time permits - would be great to get this in! Thanks

I will try to ship this by the end of weekend.

… flag

zerosnacks · 2025-03-31T08:34:10Z

Thanks! This is looking great

startup-dreamer · 2025-04-18T19:25:22Z

Hey @zerosnacks! I can continue working on this — I'd also love for this feature to go out ASAP. Just let me know if my approach and semantics look good to you; if not, I’m happy to make changes.

zerosnacks · 2025-06-05T11:31:31Z

crates/forge/assets/vyper/workflowTemplate.yml

+        with:
+          version: nightly


Suggested change

with:

version: nightly

We prefer using stable by default

zerosnacks · 2025-06-05T11:32:23Z

crates/forge/assets/vyper/workflowTemplate.yml

+      - name: Install pnpm
+        uses: pnpm/action-setup@v3
+        with:
+          version: latest
+          run_install: false
+
+      - name: Get pnpm cache directory path
+        id: pnpm-cache-dir-path
+        run: echo "dir=$(pnpm store path --silent)" >> $GITHUB_OUTPUT
+
+      - name: Restore pnpm cache
+        uses: actions/cache@v4
+        id: pnpm-cache
+        with:
+          path: ${{ steps.pnpm-cache-dir-path.outputs.dir }}
+          key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
+          restore-keys: ${{ runner.os }}-pnpm-store-
+
+      - name: Use Node.js ${{ matrix.node_version }}
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node_version }}
+
+      - name: Install pnpm project with a clean slate
+        run: pnpm install --prefer-offline --frozen-lockfile


Suggested change

- name: Install pnpm

uses: pnpm/action-setup@v3

with:

version: latest

run_install: false

- name: Get pnpm cache directory path

id: pnpm-cache-dir-path

run: echo "dir=$(pnpm store path --silent)" >> $GITHUB_OUTPUT

- name: Restore pnpm cache

uses: actions/cache@v4

id: pnpm-cache

with:

path: ${{ steps.pnpm-cache-dir-path.outputs.dir }}

key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}

restore-keys: ${{ runner.os }}-pnpm-store-

- name: Use Node.js ${{ matrix.node_version }}

uses: actions/setup-node@v4

with:

node-version: ${{ matrix.node_version }}

- name: Install pnpm project with a clean slate

run: pnpm install --prefer-offline --frozen-lockfile

- name: Install pnpm

uses: pnpm/action-setup@v3

with:

version: latest

run_install: false

- name: Get pnpm cache directory path

id: pnpm-cache-dir-path

run: echo "dir=$(pnpm store path --silent)" >> $GITHUB_OUTPUT

- name: Restore pnpm cache

uses: actions/cache@v4

id: pnpm-cache

with:

path: ${{ steps.pnpm-cache-dir-path.outputs.dir }}

key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}

restore-keys: ${{ runner.os }}-pnpm-store-

- name: Use Node.js ${{ matrix.node_version }}

uses: actions/setup-node@v4

with:

node-version: ${{ matrix.node_version }}

- name: Install pnpm project with a clean slate

run: pnpm install --prefer-offline --frozen-lockfile

I would prefer keeping this workflow template focused on Vyper + Foundry alone

zerosnacks · 2025-06-05T11:39:45Z

crates/forge/assets/vyper/VyperDeployerTemplate.sol

+
+import {Vm} from "forge-std/Vm.sol";
+
+contract VyperDeployer {


I believe it is possible, per #8181, to deployCode directly like this https://github.com/mpeyfuss/vyper-template/blob/08a0eb515b2f0493d595de3ebf5a048442ea9a0f/script/Counter.s.sol#L10

eliminating the need for VyperDeployer and FFI should also not be required anymore

zerosnacks · 2025-06-05T11:40:08Z

crates/forge/src/cmd/init.rs

+                // Write the full config with FFI enabled to foundry.toml
+                if !dest.exists() {
+                    let toml_content = "[profile.default]\nsrc = \"src\"\nout = \"out\"\nlibs = [\"lib\"]\nffi = true\n\n# See more config options https://github.com/foundry-rs/foundry/blob/master/crates/config/README.md#all-options".to_string();
+                    fs::write(dest, toml_content)?;


See above, I don't think we actually need FFI anymore

zerosnacks · 2025-06-05T11:41:44Z

Hey @startup-dreamer!

My apologies for my late reply! This got lost in all my notifications

Directionally this is looking good but it includes a few functions / files that are not required anymore

zerosnacks · 2025-07-02T10:10:32Z

Hi @startup-dreamer, would you be up for making the proposed modifications or would you like me to take this over? Thanks!

zerosnacks · 2025-08-21T11:03:03Z

Closing in favor of #11383

Added you as a co-author

Thanks!

startup-dreamer requested review from DaniPopes, grandizzy, klkvr, mattsse, yash-atreya and zerosnacks as code owners February 21, 2025 05:10

zerosnacks reviewed Feb 21, 2025

View reviewed changes

startup-dreamer marked this pull request as draft February 25, 2025 08:28

feat: add support for basic vyper project using forge init --vyper

b7bf60a

startup-dreamer force-pushed the feat/add_forge_vyper_init_flag branch from 5528dc5 to 985132e Compare March 29, 2025 20:07

This comment was marked as outdated.

Sign in to view

startup-dreamer force-pushed the feat/add_forge_vyper_init_flag branch from 985132e to ec2d2f1 Compare March 30, 2025 15:15

feat: moved assets to their respective subdirectories and add --vyper…

5b06b8e

… flag

startup-dreamer force-pushed the feat/add_forge_vyper_init_flag branch from ec2d2f1 to 5b06b8e Compare March 30, 2025 16:05

startup-dreamer marked this pull request as ready for review March 30, 2025 16:18

startup-dreamer requested a review from zerosnacks March 30, 2025 16:18

jenpaff moved this to Ready For Review in Foundry Apr 15, 2025

zerosnacks self-assigned this Jun 5, 2025

zerosnacks reviewed Jun 5, 2025

View reviewed changes

jenpaff mentioned this pull request Jun 4, 2025

meta(compilers): Vyper support #8113

Open

7 tasks

zerosnacks moved this from Ready For Review to In Progress in Foundry Jul 8, 2025

zerosnacks added this to the v1.4.0 milestone Aug 18, 2025

zerosnacks mentioned this pull request Aug 21, 2025

feat(forge): add --vyper to forge init to start with a Vyper project #11383

Merged

3 tasks

zerosnacks removed their assignment Aug 21, 2025

zerosnacks removed this from the v1.4.0 milestone Aug 21, 2025

zerosnacks removed this from Foundry Aug 21, 2025

zerosnacks closed this Aug 21, 2025


		import {Vm} from "forge-std/Vm.sol";

		contract VyperDeployer {

Conversation

startup-dreamer commented Feb 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Checklist

Uh oh!

zerosnacks left a comment

Choose a reason for hiding this comment

Uh oh!

startup-dreamer commented Feb 21, 2025

Uh oh!

zerosnacks commented Mar 27, 2025

Uh oh!

startup-dreamer commented Mar 27, 2025

Uh oh!

This comment was marked as outdated.

zerosnacks commented Mar 31, 2025

Uh oh!

startup-dreamer commented Apr 18, 2025

Uh oh!

zerosnacks Jun 5, 2025

Choose a reason for hiding this comment

Uh oh!

zerosnacks Jun 5, 2025

Choose a reason for hiding this comment

Uh oh!

zerosnacks Jun 5, 2025

Choose a reason for hiding this comment

Uh oh!

zerosnacks Jun 5, 2025

Choose a reason for hiding this comment

Uh oh!

zerosnacks commented Jun 5, 2025

Uh oh!

zerosnacks commented Jul 2, 2025

Uh oh!

zerosnacks commented Aug 21, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

startup-dreamer commented Feb 21, 2025 •

edited

Loading