Skip to content

refactor(whatsrust): extract outbound message wrappers - #322

Closed
Andiveli wants to merge 2 commits into
refactor/whatsrust-message-payload-builders-v2from
refactor/whatsrust-send-wrappers
Closed

refactor(whatsrust): extract outbound message wrappers#322
Andiveli wants to merge 2 commits into
refactor/whatsrust-message-payload-builders-v2from
refactor/whatsrust-send-wrappers

Conversation

@Andiveli

Copy link
Copy Markdown
Owner

Summary

  • Move outbound send and forward wrappers into message_send.rs.
  • Preserve public signatures, FFI behavior, ownership, quote handling, and send policy.

Changes

  • Moved send_message, send_text_message, and forward_message from the facade.
  • Kept explicit facade availability through the existing module wiring.

Chain Context

Testing

  • cargo fmt --all -- --check
  • CARGO_TARGET_DIR=/home/samael/Escritorio/public/wptui-public/target cargo check --workspace --all-targets
  • CARGO_TARGET_DIR=/home/samael/Escritorio/public/wptui-public/target cargo test -p whatsrust message_send::tests -- --test-threads=1
  • git diff --check
  • Manual runtime testing: not applicable; behavior-preserving FFI extraction.

Review Budget and Rollback

  • Authored diff: 256 lines (128 additions + 128 deletions).
  • Rollback: revert commit c1a5264 to restore wrappers to lib.rs.

Out of Scope

  • No ABI, ownership, panic, send-policy, threading, or bug-fix changes.

Closes #320

@Andiveli

Copy link
Copy Markdown
Owner Author

Superseded by repaired replacement PR above; this branch was accidentally advanced with the next child and is closed without rewriting history.

@Andiveli Andiveli closed this Aug 26, 2026
@Andiveli
Andiveli deleted the refactor/whatsrust-send-wrappers branch August 31, 2026 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:refactor Code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant