react-native-iap 14.4.34 → 14.4.35-rc.1

package/android/build.gradle

@@ -194,6 +194,11 @@ dependencies {
     // Use standard Google Play Billing
     implementation "io.github.hyochan.openiap:openiap-google:${googleVersionString}"
   }
+
+  // Test dependencies
+  testImplementation 'junit:junit:4.13.2'
+  testImplementation 'org.jetbrains.kotlin:kotlin-test'
+  testImplementation 'org.jetbrains.kotlin:kotlin-test-junit'
 }
 
 configurations.all {

package/android/src/main/java/com/margelo/nitro/iap/HybridRnIap.kt

@@ -37,6 +37,22 @@ import org.json.JSONArray
 import org.json.JSONObject
 import java.util.Locale
 
+/**
+ * Custom exception for OpenIAP errors that only includes the error JSON without stack traces.
+ * This ensures clean error messages are passed to JavaScript without Java/Kotlin stack traces.
+ */
+class OpenIapException(private val errorJson: String) : Exception() {
+    override val message: String
+        get() = errorJson
+
+    override fun toString(): String = errorJson
+
+    override fun fillInStackTrace(): Throwable {
+        // Don't fill in stack trace to avoid it being serialized
+        return this
+    }
+}
+
 class HybridRnIap : HybridRnIapSpec() {
     
     // Get ReactApplicationContext lazily from NitroModules
@@ -167,7 +183,7 @@ class HybridRnIap : HybridRnIapSpec() {
                 } catch (err: Throwable) {
                     val error = OpenIAPError.InitConnection
                     RnIapLog.failure("initConnection.native", err)
-                    throw Exception(
+                    throw OpenIapException(
                         toErrorJson(
                             error = error,
                             debugMessage = err.message,
@@ -178,7 +194,7 @@ class HybridRnIap : HybridRnIapSpec() {
                 if (!ok) {
                     val error = OpenIAPError.InitConnection
                     RnIapLog.failure("initConnection.native", Exception(error.message))
-                    throw Exception(
+                    throw OpenIapException(
                         toErrorJson(
                             error = error,
                             messageOverride = "Failed to initialize connection"
@@ -225,7 +241,7 @@ class HybridRnIap : HybridRnIapSpec() {
             )
 
             if (skus.isEmpty()) {
-                throw Exception(toErrorJson(OpenIAPError.EmptySkuList))
+                throw OpenIapException(toErrorJson(OpenIAPError.EmptySkuList))
             }
 
             initConnection(null).await()
@@ -528,7 +544,7 @@ class HybridRnIap : HybridRnIapSpec() {
             } catch (e: Exception) {
                 RnIapLog.failure("getActiveSubscriptions", e)
                 val error = OpenIAPError.ServiceUnavailable
-                throw Exception(
+                throw OpenIapException(
                     toErrorJson(
                         error = error,
                         debugMessage = e.message,
@@ -937,14 +953,14 @@ class HybridRnIap : HybridRnIapSpec() {
     // iOS-specific method - not supported on Android
     override fun getStorefrontIOS(): Promise<String> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
 
     // iOS-specific method - not supported on Android
     override fun getAppTransactionIOS(): Promise<String?> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
 
@@ -1031,7 +1047,7 @@ class HybridRnIap : HybridRnIapSpec() {
             try {
                 // For Android, we need the androidOptions to be provided
                 val androidOptions = params.androidOptions
-                    ?: throw Exception(toErrorJson(OpenIAPError.DeveloperError))
+                    ?: throw OpenIapException(toErrorJson(OpenIAPError.DeveloperError))
 
                 // Android receipt validation would typically involve server-side validation
                 // using Google Play Developer API. Here we provide a simplified implementation
@@ -1070,7 +1086,7 @@ class HybridRnIap : HybridRnIapSpec() {
             } catch (e: Exception) {
                 val debugMessage = e.message
                 val error = OpenIAPError.InvalidReceipt
-                throw Exception(
+                throw OpenIapException(
                     toErrorJson(
                         error = error,
                         debugMessage = debugMessage,
@@ -1084,31 +1100,31 @@ class HybridRnIap : HybridRnIapSpec() {
     // iOS-specific methods - Not applicable on Android, return appropriate defaults
     override fun subscriptionStatusIOS(sku: String): Promise<Array<NitroSubscriptionStatus>?> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
     
     override fun currentEntitlementIOS(sku: String): Promise<NitroPurchase?> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
     
     override fun latestTransactionIOS(sku: String): Promise<NitroPurchase?> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
     
     override fun getPendingTransactionsIOS(): Promise<Array<NitroPurchase>> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
     
     override fun syncIOS(): Promise<Boolean> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
     
@@ -1116,37 +1132,37 @@ class HybridRnIap : HybridRnIapSpec() {
     
     override fun isEligibleForIntroOfferIOS(groupID: String): Promise<Boolean> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
     
     override fun getReceiptDataIOS(): Promise<String> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
 
     override fun getReceiptIOS(): Promise<String> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
 
     override fun requestReceiptRefreshIOS(): Promise<String> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
 
     override fun isTransactionVerifiedIOS(sku: String): Promise<Boolean> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
     
     override fun getTransactionJwsIOS(sku: String): Promise<String?> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
 
@@ -1166,7 +1182,7 @@ class HybridRnIap : HybridRnIapSpec() {
             } catch (err: Throwable) {
                 RnIapLog.failure("checkAlternativeBillingAvailabilityAndroid", err)
                 val errorType = parseOpenIapError(err)
-                throw Exception(toErrorJson(errorType, debugMessage = err.message))
+                throw OpenIapException(toErrorJson(errorType, debugMessage = err.message))
             }
         }
     }
@@ -1176,7 +1192,7 @@ class HybridRnIap : HybridRnIapSpec() {
             RnIapLog.payload("showAlternativeBillingDialogAndroid", null)
             try {
                 val activity = context.currentActivity
-                    ?: throw Exception(toErrorJson(OpenIAPError.DeveloperError, debugMessage = "Activity not available"))
+                    ?: throw OpenIapException(toErrorJson(OpenIAPError.DeveloperError, debugMessage = "Activity not available"))
 
                 val userAccepted = withContext(Dispatchers.Main) {
                     openIap.setActivity(activity)
@@ -1187,7 +1203,7 @@ class HybridRnIap : HybridRnIapSpec() {
             } catch (err: Throwable) {
                 RnIapLog.failure("showAlternativeBillingDialogAndroid", err)
                 val errorType = parseOpenIapError(err)
-                throw Exception(toErrorJson(errorType, debugMessage = err.message))
+                throw OpenIapException(toErrorJson(errorType, debugMessage = err.message))
             }
         }
     }
@@ -1206,7 +1222,7 @@ class HybridRnIap : HybridRnIapSpec() {
             } catch (err: Throwable) {
                 RnIapLog.failure("createAlternativeBillingTokenAndroid", err)
                 val errorType = parseOpenIapError(err)
-                throw Exception(toErrorJson(errorType, debugMessage = err.message))
+                throw OpenIapException(toErrorJson(errorType, debugMessage = err.message))
             }
         }
     }
@@ -1236,19 +1252,19 @@ class HybridRnIap : HybridRnIapSpec() {
 
     override fun canPresentExternalPurchaseNoticeIOS(): Promise<Boolean> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
 
     override fun presentExternalPurchaseNoticeSheetIOS(): Promise<ExternalPurchaseNoticeResultIOS> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
 
     override fun presentExternalPurchaseLinkIOS(url: String): Promise<ExternalPurchaseLinkResultIOS> {
         return Promise.async {
-            throw Exception(toErrorJson(OpenIAPError.FeatureNotSupported))
+            throw OpenIapException(toErrorJson(OpenIAPError.FeatureNotSupported))
         }
     }
 

package/android/src/test/java/com/margelo/nitro/iap/OpenIapExceptionTest.kt

@@ -0,0 +1,93 @@
+package com.margelo.nitro.iap
+
+import org.junit.Test
+import org.junit.Assert.*
+
+/**
+ * Unit tests for OpenIapException to verify that error messages
+ * are returned without Java/Kotlin stack traces.
+ *
+ * This addresses Issue #3075 where error.code was undefined because
+ * stack traces were breaking JSON parsing on the JavaScript side.
+ */
+class OpenIapExceptionTest {
+
+    @Test
+    fun `test OpenIapException message returns clean JSON`() {
+        val errorJson = """{"code":"init-connection","message":"Failed to initialize connection"}"""
+        val exception = OpenIapException(errorJson)
+
+        // Verify message is exactly the JSON string
+        assertEquals(errorJson, exception.message)
+    }
+
+    @Test
+    fun `test OpenIapException toString returns clean JSON without stack trace`() {
+        val errorJson = """{"code":"user-cancelled","message":"User cancelled"}"""
+        val exception = OpenIapException(errorJson)
+
+        // toString() should return only the JSON, not "java.lang.Exception: ..."
+        val result = exception.toString()
+        assertEquals(errorJson, result)
+        assertFalse(result.startsWith("java.lang.Exception:"))
+        assertFalse(result.contains("\tat "))
+    }
+
+    @Test
+    fun `test thrown OpenIapException message is clean`() {
+        val errorJson = """{"code":"network-error","message":"Network error occurred","responseCode":-1}"""
+
+        try {
+            throw OpenIapException(errorJson)
+        } catch (e: Exception) {
+            // Verify the caught exception message is clean
+            assertEquals(errorJson, e.message)
+            assertEquals(errorJson, e.toString())
+        }
+    }
+
+    @Test
+    fun `test OpenIapException with complex JSON structure`() {
+        val errorJson = """{"code":"purchase-error","message":"Purchase failed","responseCode":3,"debugMessage":"Item unavailable","productId":"com.test.product"}"""
+        val exception = OpenIapException(errorJson)
+
+        assertEquals(errorJson, exception.message)
+        assertEquals(errorJson, exception.toString())
+    }
+
+    @Test
+    fun `test multiple OpenIapException instances are independent`() {
+        val error1 = """{"code":"error-1","message":"First error"}"""
+        val error2 = """{"code":"error-2","message":"Second error"}"""
+
+        val exception1 = OpenIapException(error1)
+        val exception2 = OpenIapException(error2)
+
+        assertEquals(error1, exception1.message)
+        assertEquals(error2, exception2.message)
+        assertNotEquals(exception1.message, exception2.message)
+    }
+
+    @Test
+    fun `test OpenIapException with empty JSON`() {
+        val errorJson = """{}"""
+        val exception = OpenIapException(errorJson)
+
+        assertEquals(errorJson, exception.message)
+        assertEquals(errorJson, exception.toString())
+    }
+
+    @Test
+    fun `test OpenIapException message does not contain stack trace keywords`() {
+        val errorJson = """{"code":"test-error","message":"Test message"}"""
+        val exception = OpenIapException(errorJson)
+
+        val result = exception.toString()
+
+        // Verify no stack trace keywords are present
+        assertFalse("Should not contain 'at ' (stack trace)", result.contains("\tat "))
+        assertFalse("Should not contain 'java.lang.Exception:'", result.contains("java.lang.Exception:"))
+        assertFalse("Should not contain '.kt:' (Kotlin file reference)", result.contains(".kt:"))
+        assertFalse("Should not contain '.java:' (Java file reference)", result.contains(".java:"))
+    }
+}

package/ios/reactnativeiap.xcodeproj/project.xcworkspace/contents.xcworkspacedata

@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Workspace
+   version = "1.0">
+   <FileRef
+      location = "self:">
+   </FileRef>
+</Workspace>

package/ios/reactnativeiap.xcodeproj/project.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist

@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<dict>
+	<key>IDEDidComputeMac32BitWarning</key>
+	<true/>
+</dict>
+</plist>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-iap",
-  "version": "14.4.34",
+  "version": "14.4.35-rc.1",
   "description": "React Native In-App Purchases module for iOS and Android using Nitro",
   "main": "./lib/module/index.js",
   "types": "./lib/typescript/src/index.d.ts",

package/plugin/build/src/withIAP.d.ts

@@ -0,0 +1,3 @@
+import type { ConfigPlugin } from 'expo/config-plugins';
+declare const _default: ConfigPlugin<void>;
+export default _default;

package/plugin/build/src/withIAP.js

@@ -0,0 +1,81 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const config_plugins_1 = require("expo/config-plugins");
+const pkg = require('../../package.json');
+// Global flag to prevent duplicate logs
+let hasLoggedPluginExecution = false;
+const addLineToGradle = (content, anchor, lineToAdd, offset = 1) => {
+    const lines = content.split('\n');
+    const index = lines.findIndex((line) => line.match(anchor));
+    if (index === -1) {
+        console.warn(`Anchor "${anchor}" not found in build.gradle. Appending to end.`);
+        lines.push(lineToAdd);
+    }
+    else {
+        lines.splice(index + offset, 0, lineToAdd);
+    }
+    return lines.join('\n');
+};
+const modifyAppBuildGradle = (gradle) => {
+    let modified = gradle;
+    // Add billing library dependencies to app-level build.gradle
+    const billingDep = `    implementation "com.android.billingclient:billing-ktx:8.0.0"`;
+    const gmsDep = `    implementation "com.google.android.gms:play-services-base:18.1.0"`;
+    let hasAddedDependency = false;
+    if (!modified.includes(billingDep)) {
+        modified = addLineToGradle(modified, /dependencies\s*{/, billingDep);
+        hasAddedDependency = true;
+    }
+    if (!modified.includes(gmsDep)) {
+        modified = addLineToGradle(modified, /dependencies\s*{/, gmsDep, 1);
+        hasAddedDependency = true;
+    }
+    // Log only once and only if we actually added dependencies
+    if (hasAddedDependency && !hasLoggedPluginExecution) {
+        console.log('🛠️ react-native-iap: Added billing dependencies to build.gradle');
+    }
+    return modified;
+};
+const withIapAndroid = (config) => {
+    // Add IAP dependencies to app build.gradle
+    config = (0, config_plugins_1.withAppBuildGradle)(config, (config) => {
+        config.modResults.contents = modifyAppBuildGradle(config.modResults.contents);
+        return config;
+    });
+    config = (0, config_plugins_1.withAndroidManifest)(config, (config) => {
+        const manifest = config.modResults;
+        if (!manifest.manifest['uses-permission']) {
+            manifest.manifest['uses-permission'] = [];
+        }
+        const permissions = manifest.manifest['uses-permission'];
+        const billingPerm = { $: { 'android:name': 'com.android.vending.BILLING' } };
+        const alreadyExists = permissions.some((p) => p.$['android:name'] === 'com.android.vending.BILLING');
+        if (!alreadyExists) {
+            permissions.push(billingPerm);
+            if (!hasLoggedPluginExecution) {
+                console.log('✅ Added com.android.vending.BILLING to AndroidManifest.xml');
+            }
+        }
+        else {
+            if (!hasLoggedPluginExecution) {
+                console.log('ℹ️ com.android.vending.BILLING already exists in AndroidManifest.xml');
+            }
+        }
+        return config;
+    });
+    return config;
+};
+const withIAP = (config, _props) => {
+    try {
+        const result = withIapAndroid(config);
+        // Set flag after first execution to prevent duplicate logs
+        hasLoggedPluginExecution = true;
+        return result;
+    }
+    catch (error) {
+        config_plugins_1.WarningAggregator.addWarningAndroid('react-native-iap', `react-native-iap plugin encountered an error: ${error}`);
+        console.error('react-native-iap plugin error:', error);
+        return config;
+    }
+};
+exports.default = (0, config_plugins_1.createRunOncePlugin)(withIAP, pkg.name, pkg.version);

package/plugin/build/tsconfig.tsbuildinfo

@@ -0,0 +1 @@
+{"root":["../src/withiap.ts"],"version":"5.9.2"}

package/plugin/tsconfig.tsbuildinfo

@@ -1 +1 @@
-{"root":["./src/withIAP.ts"],"version":"5.9.2"}
+{"root":["./src/withiap.ts"],"version":"5.9.2"}