Skip to content

Deprecate legacy OPP envelope metadata#334

Open
huangminghuang wants to merge 1 commit into
feature/opp-part3-operator-managementfrom
feature/remove-legacy-fields
Open

Deprecate legacy OPP envelope metadata#334
huangminghuang wants to merge 1 commit into
feature/opp-part3-operator-managementfrom
feature/remove-legacy-fields

Conversation

@huangminghuang
Copy link
Copy Markdown
Contributor

Summary

  • remove legacy outbound envelope table fields from sysio.msgch ABI/types
  • keep protobuf legacy envelope fields documented as deprecated while omitting them from WIRE DataStream serialization
  • update outbound envelope ABI shape to rely on raw_envelope/envelope_hash instead of inline Merkle/message-id metadata

Verification

  • regenerated OPP attestation protobuf outputs to fix stale generated CDT headers
  • ninja -C /home/ssm-user/code/wire/wire-sysio/build/debug/contracts sysio.msgch
  • recreated and ran local OPP cluster from /home/ssm-user/code/wire/.clusters/opp-legacy-envelope
  • verified sysio.msgch::outenvelopes rows no longer expose merkle_root, start_message_id, or end_message_id
  • decoded emitted OPP debugging envelopes and confirmed legacy protobuf fields remain zero-length

Notes

  • Full cmake --build build/debug -j2 reached late native dispatch generation but failed in unrelated sysio.opreg_native dispatch code with missing sysio::opreg::stake; cluster-critical WIRE binaries/contracts and sysio.msgch rebuilt and deployed successfully.

@huangminghuang huangminghuang requested review from heifner and jglanz May 13, 2026 21:10
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