Skip to content

SEC-140: Generate unified Ethereum client configuration - #36

Open
huangminghuang wants to merge 18 commits into
masterfrom
fix/sec-140
Open

SEC-140: Generate unified Ethereum client configuration#36
huangminghuang wants to merge 18 commits into
masterfrom
fix/sec-140

Conversation

@huangminghuang

@huangminghuang huangminghuang commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Updated this branch through current master (ad346700) and adopted the
    final host-side configuration shape delivered by
    wire-sysio#531.
  • Generate the versioned data/ethereum-client.json ProtoJSON artifact for
    batch-operator and underwriter daemons, passed through
    --outpost-ethereum-client-config-file. Artifact preparation is shared by
    create, run, restart, and flow paths; bios and producer-only nodes receive
    neither an Ethereum signing client nor that option.
  • Keep this host-only Ethereum client configuration local to wire-tools-ts.
    SEC-131 intentionally keeps it outside the shared OPP model bundles used by
    Solidity and Solana, so this PR neither changes wire-sysio nor publishes a
    generated model package.
  • Local Anvil artifacts contain a finite nested transaction_policy: 2 gwei
    priority fee, 100 gwei maximum fee, 6,000,000 final gas limit, and 0.7 ETH
    total native cost. The total covers the 0.6 ETH maximum gas spend and leaves
    0.1 ETH for transaction value; regression coverage checks the 5,270,438-gas
    buffered remote epochIn high-water mark.
  • External-outpost artifacts deliberately omit transaction_policy; the
    operator of that endpoint selects production limits outside wire-tools-ts.
    The obsolete standalone configuration helpers and legacy
    --outpost-ethereum-client path are removed.
  • Include the current typed challenge_id default in the shared underwriting
    request test fixture, restoring compatibility with the released SDK contract
    types and resolving SEC-151.

Validation

  • Focused policy/configuration/artifact coverage: 4 Jest suites, 34 tests
    passed with Node 24.15.0.
  • After synchronizing and rebuilding the linked @wireio/sdk-core@1.0.84, the
    complete pnpm test gate passed: build plus 221 Jest suites / 1,708 tests.
    pnpm lint and git diff --check also passed. SEC-151 is Done.
  • The historical remote pass for superseded SHA 29287522 is not used as
    final evidence. A replacement remote E2E run will target this PR's final
    pushed head and be linked here when it passes.

Dependency

Requires merged wire-sysio#531,
which provides the nodeop nested-client loader and enforcement. No companion
wire-sysio PR is required.

Jira: SEC-140

@huangminghuang huangminghuang changed the title SEC-140: Provision Anvil Ethereum transaction policy SEC-140: Generate unified Ethereum client configuration Jul 21, 2026
@bearcubsvet
bearcubsvet requested a review from jglanz August 5, 2026 18:54
Change-Id: If148dd96875d0562bd9948e064f4400519643dab

# Conflicts:
#	packages/cluster-tool/tests/orchestration/steps/processes/NodeopProcessSteps.test.ts
#	packages/cluster-tool/tests/tools/wire/OperatorDaemonTool.test.ts
Change-Id: Ib1dbe9091f9f9f6a24d1eeed37e346fdc8399031
Change-Id: I6ba337a4f4de0fcdc354cfb4e109e72f81bf812b
Change-Id: I2d3d70e03825afae8badc3786b988aa78bbca100
Change-Id: I87ed7df1ebdfe4a7b4558dbe1b42ae9c889f63cf
Change-Id: I7bfd31f466d6253843c3e9590271a0763e40f315
Change-Id: Ic61af7fcfe8b96b04234e13b5d2d86d19d2e2540
Change-Id: Ie75b460a2aaef7ea168895d26092ce49e3fd4610
Change-Id: Ib05444cea40e2cfe6519d2e7b1fe8f872922a1a0
Change-Id: Ie971bf1021416c2aecd3f1767f4decc0e391a636
Change-Id: Iafd1d3e9fd52de270915ce758f4009e95b8fb8d1
Change-Id: I3110bb452dd855af919aa7e0582522644e8107fd
Change-Id: Idff438957348683cbcf1a468fc8003ba44843e80
Change-Id: I9497e406cc600081d571c50fc3e95674b128f899
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.

1 participant