Skip to content

fix(sdk-core): align Andon interfaces with Sysio #662 - #100

Merged
huangminghuang merged 1 commit into
masterfrom
fix/sysio-662-contract-types
Oct 8, 2026
Merged

huangminghuang merged 1 commit into
masterfrom
fix/sysio-662-contract-types

Conversation

@huangminghuang

Copy link
Copy Markdown
Contributor

Summary

  • Match Andon's native-permission API: pull({ reason }), clear({ note }), and the cord row with pulled, when, and reason.
  • Remove retired Andon actions/configuration and syndication bucket frozen_mark from generated declarations and the contract registry.
  • Add producer-ABI regression tests for action encoding, explicit authorization, cord serialization, and rejection of retired members.

Why

wire-sysio #662 changed these interfaces, but SDK 1.0.94 still publishes their earlier forms. Consumers following the merged contract cannot compile against those stale declarations, and the generated facade still exposes actions the contract no longer supports.

Regenerated all 20 system ABIs using the generator and checked-in ABIs from #662's exact merge commit, a16738c488e8a821f06e1fa7d3188b98f4e35207. The resulting changes are confined to Andon and the syndication bucket field. A new SDK release is required after this source change merges.

Validation

  • Generated declarations match the producer generation plus repository formatting exactly; the test ABI fixture matches the checked-in producer ABI.
  • Workspace build, SDK strict-null typecheck, and changed-file lint pass. Native Andon and generic system-contract client tests: 17 pass.
  • Tools chore(release): patch bump v1.0.101 (stable) #115 builds successfully against this SDK candidate; 82 focused bootstrap/action/helper tests and 50 emergency-stop/rate-limit unit tests pass. These checks use the local candidate, not a published SDK; no live E2E result is claimed.

Companion PRs

  • wire-tools-ts #115: native Andon bootstrap and flow compatibility. Merge this SDK source fix, publish and verify a compatible SDK, then update and validate the Tools dependency.

Change-Id: I3d0e005116ff385fe9cad5fd55dafd3d23d9e9e3
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.

2 participants