Skip to content

Ignore silent option for anvil instances#81

Open
chrisx9z wants to merge 1 commit into
wevm:mainfrom
chrisx9z:fix/anvil-silent-parameter
Open

Ignore silent option for anvil instances#81
chrisx9z wants to merge 1 commit into
wevm:mainfrom
chrisx9z:fix/anvil-silent-parameter

Conversation

@chrisx9z

@chrisx9z chrisx9z commented Jun 6, 2026

Copy link
Copy Markdown

Summary

  • stop forwarding the Anvil silent option so startup logs remain available to the readiness resolver
  • remove silent from the public Anvil parameter type
  • add a focused regression test that confirms silent is stripped from internal args

Fixes #80.

Verification

  • pnpm exec vitest src/instances/anvil.test.ts --run -t "ignores silent option"
  • pnpm exec tsc --noEmit
  • pnpm exec biome check src/instances/anvil.ts src/instances/anvil.test.ts
  • git diff --check

Note: the full anvil integration test file cannot run on this Windows machine because the anvil binary is not installed in PATH; the focused regression test does not require the binary.

Comment: Prevents Anvil startup from suppressing the log line that prool uses to resolve readiness.
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.

anvil: silent footgun

1 participant