Repository navigation
Align bootstrap and flows with Sysio #670 - #116
Merged
Merged
Conversation
valthon
marked this pull request as ready for review
October 9, 2026 00:26
valthon
force-pushed
the
fix/sysio-670-consumers
branch
from
October 9, 2026 10:01
a514572 to
d38ae47
Compare
valthon
force-pushed
the
fix/sysio-670-consumers
branch
from
October 9, 2026 17:21
d38ae47 to
9271300
Compare
This was referenced Oct 10, 2026
Remove retired reward consumers and capital-action callers, use the shared gross return minimum, and align native Andon permissions and retained flows with the current SDK. Verify raw fallback for removed reward and reserve-creation wire values without blocking active attestation decoding.
Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01SUTuRushtiEZgUXLRaoEiv
valthon
force-pushed
the
fix/sysio-670-consumers
branch
from
October 10, 2026 22:44
f258cc5 to
e3d76da
Compare
wire-sysio PR #674 removes the retired reserve, swap, underwriting, pretoken, staking and processing-error OPP carriers, and the underwriter outpost roles on the sysio.chains registry. The harness still wrote those registry fields, decoded the removed payloads for display, and scanned for SwapRevert. Stop writing operator_registry_addr / source_deposit_addr in RegistrySteps and drop the fixtures that carried OperatorRegistry and ReserveManager addresses for the deleted underwriter daemon. Remove the retired display decoders (StakeUpdate, StakeResult and AttestationProcessingError among them) and the swap-revert scanner; the codec test now pins that removed wire values still render raw beside decoded entries, and the envelope-scan tests use live attestation types. Native sysio.swap tooling, the Solana OperatorRegistry PDA, and the node-owner and pretoken purchase history surfaces stay: they are live. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01ARSeK35wwkiAy4et9mCkVm
The syndication-underwriting flow's later-intake-held step failed in e2e run 38009710729 with "missing decoded syndication". The step polled until the depot held the intake envelope, then read the relayed SyndicateLIQ from the OPP debugging artifacts once. The debugging plugin writes that artifact after the depot accepts the envelope; in that run it landed 85 ms after the read. Read the decoded attestation inside the poll, so the step waits for both the held row and the persisted artifact before it records the epoch and bond request. The custody check against the attestation's own total is unchanged. readDecodedSyndications is exported so the read can be tested on its own. Move the artifact-writing test helpers out of the envelope-scan tests into a shared fixture, and cover the filter, the wait and the custody refusal. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01ARSeK35wwkiAy4et9mCkVm
wire-sysio removes CHALLENGE_REQUEST, CHALLENGE_RESPONSE and their ChallengeRequest / ChallengeOperatorHash payloads: no chain produces them, and the depot dropped both on dispatch. The OPP display decoder was their last consumer here. Remove the two decoders. The codec test now pins that both removed wire values (60932, 60945) render raw beside a decoded entry. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01ARSeK35wwkiAy4et9mCkVm
Drop retired OPP carriers from the harness
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Companion to wire-sysio #670. Bootstrap and retained flows still invoke retired capital-reward actions, omit the required minimum, and read obsolete reward fields, preventing the harness from matching the current depot contracts.
setkickerand emissions-soak capital-shortfall/claim-window assumptions.min_desyndicatein bootstrap and retained flow configurations. Both shadow pairs use the same 1,000,000-unit gross floor, equivalent to 0.001 SOL at nine decimals: an intentionally arbitrary shared launch minimum, with no per-chain policy.master(39a4a480, which adds fix: recognize tier-cap node-owner rejection reason #109's tier-cap node-owner rejection fix on top of the native Andon implementation from Tools Fix native Andon permissions in cluster bootstrap #115 and the published no-expiry SDK 1.0.96). Preserve those prerequisites and update phase-order tests and documentation for the retired LIQ configuration phase.Commits:
e8065451Align launch bootstrap and flows with contracts;e3d76da1Require published SDK 1.0.97 for the min_desyndicate config.Validation after rebasing onto
39a4a480: per-commit gate from a clean checkout of each commit (CI's install/build sequence,pnpm run lint, thenpnpm run test:ci). Both commits returned RC 0, each with 249/249 Jest suites and 2,651/2,651 tests plus the 7 runner tests and the flow unit checks. No live multi-chain flow was rerun.Validation used local sibling artifacts:
@wireio/sdk-corelinked from a wire-libraries-ts checkout ata01a40d(Libraries #102 merged, plus #106'ssysio.kickertypes, which this PR does not use) and the OPP TypeScript models built from wire-sysio. The registry's latest@wireio/sdk-coreis still 1.0.96, so registry-only CI cannot install the^1.0.97pin until 1.0.97 is published; coordinate the fresh OPP model release with #670 before relying on published consumer dependencies. Tools #115 is merged; its implementation is now inherited frommaster.