Skip to content

Adopt the two-field sysio.chains outpost struct - #105

Merged
valthon merged 2 commits into
masterfrom
cleanup/retired-opp-carriers
Oct 12, 2026
Merged

valthon merged 2 commits into
masterfrom
cleanup/retired-opp-carriers

Conversation

@valthon

@valthon valthon commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Adopts the contract ABIs from wire-sysio #674, which removes the underwriter outpost roles from the sysio.chains registry, the epochlog swap-fee audit fields, and the retired OPP attestation discriminants (the staking, processing-error and challenge values included).

Changes:

  • Regenerate SysioContractTypes.ts from the new ABIs (formatted with the repo's prettier, so the diff is only the ABI change).
  • Drop CHALLENGE_REQUEST and CHALLENGE_RESPONSE from the regenerated sysio.msgch attestation-type enum: no chain produces them, and the depot dropped both on dispatch.
  • sysio.chains client (Structs.ts, Types.ts, Actions.ts, Client.ts and the action test): outpost addresses are now opp_addr / opp_inbound_addr only.
  • Drop the unused root dependency on @wireio/opp-typescript-models. sdk-core's own import of ChainKind from that package is unchanged; it still conflicts with the cross-repo rule that no wire-libraries-ts package depend on it, which needs its own change.

Lands with wire-sysio #674 and the companion consumer PRs in wire-tools-ts, wire-ethereum and wire-solana (branch cleanup/retired-opp-carriers in each).

Validation (this commit, 213a77e):

Gate RC
pnpm build 0
pnpm run lint 0
pnpm run test:ci (83 suites, 947 tests) 0

All 12 flows passed locally with the companion heads (wire-sysio 4291581446; Solana before its rebase), each run with run-flow.mjs and the heartbeat monitor. Earlier, on the previous heads, the E2E Flow Tests workflow passed all 12 flows ("12 flows, 0 failed") on these exact heads: run 38051256122. wire-sysio 3597cc6f, wire-libraries-ts 53519742, wire-tools-ts 6fb42b7d, wire-ethereum 4c62fc22, wire-solana a5a15e4d, wire-cdt a00bd718.

🤖 Generated with Claude Code

https://claude.ai/code/session_01ARSeK35wwkiAy4et9mCkVm

wire-sysio PR #674 removes the underwriter outpost roles from the
sysio.chains registry (operator_registry_addr, source_deposit_addr),
the epochlog swap-fee audit fields, and the retired OPP attestation
discriminants, the staking and processing-error ones included. The generated contract declarations and the hand-written
chains client still carried them.

Regenerate SysioContractTypes from the new ABIs and update the chains
Structs, Types, Actions and Client, plus their action test, to the
opp_addr / opp_inbound_addr pair. Drop the unused root dependency on
@wireio/opp-typescript-models, which contradicted the generator/output
boundary; sdk-core's own live use of ChainKind is unchanged.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01ARSeK35wwkiAy4et9mCkVm
@valthon
valthon force-pushed the cleanup/retired-opp-carriers branch from 0c4e1c0 to 5351974 Compare October 10, 2026 12:14
wire-sysio removes CHALLENGE_REQUEST and CHALLENGE_RESPONSE: no chain
produces them, and the depot dropped both on dispatch. Regenerate
SysioContractTypes from the new sysio.msgch ABI, which no longer lists
the two values.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01ARSeK35wwkiAy4et9mCkVm
@valthon
valthon merged commit 539963d into master Oct 12, 2026
1 check passed
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