foundry-rs / foundry-rs/foundry

feat(forge): More general `MultiContractRunnerBuilder`

Open
#9,036 0 comments 0 reactions 0 assignees View on GitHub
A-compiler A-internals A-library T-feature T-post-V1
Dominant language
Rust
Stars
10.6k
Forks
2.6k
Avg merge
16h 38m
Merged PRs (30d)
511

Description

### Component

Forge

### Describe the feature you would like

Tracked by #9012, `MultiContractRunnerBuilder` currently requires the entire output from `foundry_compilers` as input, however it only ever uses a single field on that type.

The problem with this is library users may be constructing their artifacts by different means and do not want to construct the full type.

I propose changing the input to just take in an `ArtifactsContracts<_>`

### Additional context

_No response_

Contributor guide

Open the contributing guide

Research direction

Start by locating MultiContractRunnerBuilder in the Forge code and inspect how it currently consumes foundry_compilers output. Trace the ArtifactsContracts type and the existing builder call sites, then update the input boundary so it accepts ArtifactsContracts directly. Done means callers no longer need to construct the full compiler output type.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
tooling
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.