Skip to content

Cover critical cross-chain integration journeys - #136

Merged
valthon merged 2 commits into
masterfrom
test/critical-integration-journeys
Nov 10, 2026
Merged

valthon merged 2 commits into
masterfrom
test/critical-integration-journeys

Conversation

@valthon

@valthon valthon commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Add three integration journeys for settlement and recovery outcomes, preserving every existing flow test and assertion:

  • flow-ethereum-round-trip: acquire liqETH with new ETH, syndicate real custody, deliver exact shadow to the authenticated wallet, redeem externally for principal net of the live desyndication fee, and verify another accepted inbound epoch without duplicate payment.
  • flow-batch-relay-restart: stop and reap the sole relay while an external payout is outstanding, relaunch the same data and keys, and verify exact settlement plus continued destination progress.
  • flow-batch-roster-replacement: admit and run a replacement, verify both outposts consume its roster, and require its recorded delivery to match accepted envelope checksums on both chains.

The Ethereum journey uses explicit governance custody validation. Autonomous underwriting remains covered by the plugin underwriting flows.

Commit series on master at 95f051a2 (includes the merged Solana AuthEx signature fix, #181):

  1. e726b17b — keep the no-prefix startup-script test independent of Bash's handling of cd "", exposing only dirname on its restricted PATH.
  2. 0d40796d — add the three journeys, their supporting assertions, and focused regressions.

Validation at 0d40796d:

  • Full CI sequence at each rebased commit: pnpm install, pnpm build, pnpm install, pnpm run build, pnpm run test:ci, with Node 24.20.0 and Bash 5.3.9.
  • Per-commit results: e726b17b RC 0; 0d40796d RC 0. History and script checks also returned RC 0.
  • GitHub CI run 42856744409 passed on the rebased head, including Build and test and Jest results.
  • Both commits are patch-equivalent to the previously published series; git range-diff reports = for both.
  • Hosted full-flow run 42856110914 is pinned to 0d40796d3f0c599a94170da3ec75589d842e7b1e, with both FLOW_INCLUDE and FLOW_EXCLUDE empty so every discovered flow runs. Launched after the local gates and push; see the run page for its result.
  • Live flows were not run locally.

Earlier-head evidence: E2E run 42692193099 used dc5febf7 and failed all 19 flows. Its bootstrap logs report Solana AuthEx links failing with offset is out of bounds. The rebased head inherits #181's fix; the new hosted run will verify it with the complete flow suite.

@valthon
valthon requested a review from qhool October 16, 2026 18:57
@valthon
valthon force-pushed the test/critical-integration-journeys branch from 192339e to 941eb1d Compare October 20, 2026 14:53
@valthon
valthon force-pushed the test/critical-integration-journeys branch from 941eb1d to dc5febf Compare November 9, 2026 19:45
valthon and others added 2 commits November 10, 2026 11:04
The test ran the rendered start.sh under an empty PATH, which also hides
dirname. Bash 5.3 refuses the resulting `cd ""`, so the script exits at
its first line before the prefix guard the test checks. Expose only
dirname on the PATH; nodeop is still absent, so every branch misses.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Exercise fresh Ethereum settlement, relay restart with outstanding
work, and roster replacement consumed by both outposts. Verify exact
custody, payout net of the desyndication fee, identity, and accepted
delivery.

Keep existing flows and assertions. Add focused failure regressions for
the settlement oracles, real registry provisioning, and runner exits.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@valthon
valthon force-pushed the test/critical-integration-journeys branch from dc5febf to 0d40796 Compare November 10, 2026 16:35
@valthon
valthon merged commit c258371 into master Nov 10, 2026
2 checks 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