chore(swift-sdk): remove not planned to use tx module in swift-sdk

[ZocoLini]

Removed Tx module in swift-sdk since we are not using it and transactions are being built with a single function call, not using a builder

Summary by CodeRabbit

Release Notes

• Refactor
  • Removed unimplemented transaction builder functionality and related type definitions from the Swift SDK, including transaction builder class, input/output handling structures, and associated error types. This cleanup removes non-functional code.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: dfba555a-91bc-456b-9db6-2aea6ed3df05

📥 Commits

Reviewing files that changed from the base of the PR and between 5381e78 and 051d126.

📒 Files selected for processing (2)

• packages/swift-sdk/Sources/SwiftDashSDK/Tx/TransactionBuilder.swift
• packages/swift-sdk/Sources/SwiftDashSDK/Tx/TransactionTypes.swift

💤 Files with no reviewable changes (2)

• packages/swift-sdk/Sources/SwiftDashSDK/Tx/TransactionTypes.swift
• packages/swift-sdk/Sources/SwiftDashSDK/Tx/TransactionBuilder.swift

---

📝 Walkthrough

Walkthrough

Removed incomplete transaction-related functionality from the Swift SDK, specifically the SDKTransactionBuilder class with its nested structs and the SDKBuiltTransaction struct along with SDKTxError enum from the Tx module.

Changes

Cohort / File(s)	Summary
Removed Transaction Types packages/swift-sdk/Sources/SwiftDashSDK/Tx/TransactionBuilder.swift, packages/swift-sdk/Sources/SwiftDashSDK/Tx/TransactionTypes.swift	Deleted unimplemented transaction builder class and associated types, including SDKTransactionBuilder (with nested Input and Output structs), SDKBuiltTransaction, and SDKTxError enum. Total 74 lines removed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Incomplete builders fade away,
Unimplemented code won't stay,
Transaction dreams once built so tall,
Now removed—no trace at all!
Cleaner code hops on with glee, ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title accurately describes the main change: removal of the tx module from swift-sdk due to it not being planned for use.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/remove-tx-module

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[thepastaclaw]

✅ Review complete (commit 051d126)

[github-actions]

✅ DashSDKFFI.xcframework built for this PR.

• Workflow run: https://github.com/dashpay/platform/actions/runs/23922991715
• Artifacts: DashSDKFFI.xcframework (folder), DashSDKFFI.xcframework.zip, xc_checksum.txt

SwiftPM (host the zip at a stable URL, then use):

.binaryTarget(
  name: "DashSDKFFI",
  url: "https://your.cdn.example/DashSDKFFI.xcframework.zip",
  checksum: "29312c05a39d4a3eef295a894fb94ad0f5d14bbe2733dabfd307705ac42d4a45"
)

Xcode manual integration:

• Download 'DashSDKFFI.xcframework' artifact from the run link above.
• Drag it into your app target (Frameworks, Libraries & Embedded Content) and set Embed & Sign.
• If using the Swift wrapper package, point its binaryTarget to the xcframework location or add the package and place the xcframework at the expected path.

[thepastaclaw]

Code Review

Clean removal of unused stub types. SDKTransactionBuilder and SDKBuiltTransaction were placeholder stubs — build() threw notImplemented. The actual send transaction implementation landed in PR #3130 via a different path (KeyWallet/Transaction). No FFI contracts affected since these types never crossed the Swift/Rust boundary. No issues.

Reviewed commit: 051d126