fix: add workaround for missing duplicated BIP21 check

Author: jvsena42

Bitkit/Utilities/Bip21Utils.swift

@@ -0,0 +1,18 @@
+import Foundation
+
+/// Utility methods for BIP21 URI handling
+enum Bip21Utils {
+    private static let bip21Prefix = "bitcoin:"
+
+    /// Checks if a BIP21 URI is duplicated (contains multiple bitcoin: prefixes).
+    /// Workaround for https://github.com/synonymdev/bitkit-core/issues/63
+    /// - Parameter input: The string to check
+    /// - Returns: true if the input contains duplicated BIP21 URIs, false otherwise
+    static func isDuplicatedBip21(_ input: String) -> Bool {
+        let lowercased = input.lowercased()
+        guard let firstIndex = lowercased.range(of: bip21Prefix)?.upperBound else {
+            return false
+        }
+        return lowercased[firstIndex...].contains(bip21Prefix)
+    }
+}

Bitkit/ViewModels/AppViewModel.swift

@@ -230,6 +230,17 @@ extension AppViewModel {
         // Reset send state before handling new data
         resetSendState()
 
+        // Workaround for duplicated BIP21 URIs (bitkit-core#63)
+        if Bip21Utils.isDuplicatedBip21(uri) {
+            toast(
+                type: .error,
+                title: t("other__scan_err_decoding"),
+                description: t("other__scan__error__generic"),
+                accessibilityIdentifier: "InvalidAddressToast"
+            )
+            return
+        }
+
         let data = try await decode(invoice: uri)
 
         switch data {
@@ -551,6 +562,15 @@ extension AppViewModel {
             return
         }
 
+        // Workaround for duplicated BIP21 URIs (bitkit-core#63)
+        if Bip21Utils.isDuplicatedBip21(normalized) {
+            guard currentSequence == manualEntryValidationSequence else { return }
+            manualEntryValidationResult = .invalid
+            isManualEntryInputValid = false
+            showValidationErrorToast(for: .invalid)
+            return
+        }
+
         // Try to decode the invoice
         guard let decodedData = try? await decode(invoice: normalized) else {
             guard currentSequence == manualEntryValidationSequence else { return }

BitkitTests/Bip21UtilsTests.swift

@@ -0,0 +1,51 @@
+@testable import Bitkit
+import XCTest
+
+final class Bip21UtilsTests: XCTestCase {
+    // MARK: - isDuplicatedBip21 Tests
+
+    func testIsDuplicatedBip21_ReturnsFalseForSingleValidBIP21URI() {
+        let input = "bitcoin:bc1qar0srrr7xfkvy5l643lydnw9re59gtzzwf5mdq?amount=0.001&message=Bitkit"
+        XCTAssertFalse(Bip21Utils.isDuplicatedBip21(input))
+    }
+
+    func testIsDuplicatedBip21_ReturnsTrueWhenBIP21URIIsDuplicated() {
+        let first = "bitcoin:bcrt1qr289x0fhg62672e8urudfnxnsr8tcax64xk2vk?amount=0.0000002&message=Bitkit"
+        let second = "bitcoin:bcrt1qr289x0fhg62672e8urudfnxnsr8tcax64xk2vk?amount=0.0000003&message=Bitkit"
+        let input = first + second
+        XCTAssertTrue(Bip21Utils.isDuplicatedBip21(input))
+    }
+
+    func testIsDuplicatedBip21_HandlesCaseInsensitiveBitcoinPrefix() {
+        let first = "BITCOIN:bc1qar0srrr7xfkvy5l643lydnw9re59gtzzwf5mdq?amount=0.001"
+        let second = "bitcoin:bc1qar0srrr7xfkvy5l643lydnw9re59gtzzwf5mdq?amount=0.002"
+        let input = first + second
+        XCTAssertTrue(Bip21Utils.isDuplicatedBip21(input))
+    }
+
+    func testIsDuplicatedBip21_ReturnsFalseForNonBitcoinURIs() {
+        let input = "lnbc500n1p3k9v3pp5kzmj..."
+        XCTAssertFalse(Bip21Utils.isDuplicatedBip21(input))
+    }
+
+    func testIsDuplicatedBip21_ReturnsFalseForEmptyString() {
+        XCTAssertFalse(Bip21Utils.isDuplicatedBip21(""))
+    }
+
+    func testIsDuplicatedBip21_HandlesMixedCaseDuplicatedURIs() {
+        let first = "Bitcoin:bc1qaddr1?amount=0.001"
+        let second = "BITCOIN:bc1qaddr2?amount=0.002"
+        let input = first + second
+        XCTAssertTrue(Bip21Utils.isDuplicatedBip21(input))
+    }
+
+    func testIsDuplicatedBip21_ReturnsFalseForPlainBitcoinAddress() {
+        let input = "bc1qar0srrr7xfkvy5l643lydnw9re59gtzzwf5mdq"
+        XCTAssertFalse(Bip21Utils.isDuplicatedBip21(input))
+    }
+
+    func testIsDuplicatedBip21_ReturnsFalseForSingleBIP21WithLightningParam() {
+        let input = "bitcoin:bc1qar0srrr7xfkvy5l643lydnw9re59gtzzwf5mdq?lightning=lnbc500n1p3k9v3pp5kzmj"
+        XCTAssertFalse(Bip21Utils.isDuplicatedBip21(input))
+    }
+}