@atxp/client 0.8.1 → 0.8.3

package/dist/index.js

@@ -53,7 +53,8 @@ class OAuthClient extends OAuthResourceClient {
             strict,
             allowInsecureRequests,
             clientName,
-            logger
+            logger,
+            registrationType: 'client'
         });
         this.extractResourceUrl = (response) => {
             if (response.status !== 401) {
@@ -348,16 +349,16 @@ class ATXPFetcher {
                 this.logger.info(`PAYMENT FAILED: ${error.message}`);
             }
         };
-        this.handleMultiDestinationPayment = async (paymentRequestData, paymentRequestUrl, paymentRequestId) => {
-            if (!paymentRequestData.destinations || paymentRequestData.destinations.length === 0) {
+        this.handleMultiDestinationPayment = async (paymentRequest, paymentRequestUrl, paymentRequestId) => {
+            if (!paymentRequest.options || paymentRequest.options.length === 0) {
                 return false;
             }
             // Get sources from the account
             const sources = await this.account.getSources();
-            // Apply destination mappers to transform destinations
-            // Convert PaymentRequestDestination[] to Destination[] for mapper compatibility
+            // Apply destination mappers to transform options
+            // Convert PaymentRequestOption[] to Destination[] for mapper compatibility
             const mappedDestinations = [];
-            for (const option of paymentRequestData.destinations) {
+            for (const option of paymentRequest.options) {
                 const destinationMaker = this.destinationMakers.get(option.network);
                 if (!destinationMaker) {
                     this.logger.debug(`ATXP: destination maker for network '${option.network}' not available, trying next destination`);
@@ -379,11 +380,11 @@ class ATXPFetcher {
             const firstDest = mappedDestinations[0];
             const prospectivePayment = {
                 accountId: this.account.accountId,
-                resourceUrl: paymentRequestData.resource?.toString() ?? '',
-                resourceName: paymentRequestData.resourceName ?? '',
+                resourceUrl: paymentRequest.resource?.toString() ?? '',
+                resourceName: paymentRequest.payeeName ?? '',
                 currency: firstDest.currency,
                 amount: firstDest.amount,
-                iss: paymentRequestData.iss ?? '',
+                iss: paymentRequest.payeeName ?? '',
             };
             // Ask for approval once for all payment attempts
             if (!await this.approvePayment(prospectivePayment)) {
@@ -396,7 +397,7 @@ class ATXPFetcher {
             for (const paymentMaker of this.account.paymentMakers) {
                 try {
                     // Pass all destinations to payment maker - it will filter and pick the one it can handle
-                    const result = await paymentMaker.makePayment(mappedDestinations, paymentRequestData.iss, paymentRequestId);
+                    const result = await paymentMaker.makePayment(mappedDestinations, paymentRequest.payeeName ?? '', paymentRequestId);
                     if (result === null) {
                         this.logger.debug(`ATXP: payment maker cannot handle these destinations, trying next`);
                         continue; // Try next payment maker
@@ -460,28 +461,28 @@ class ATXPFetcher {
                 this.logger.info(`ATXP: payment requirement is not allowed on this server`);
                 return false;
             }
-            const paymentRequestData = await this.getPaymentRequestData(paymentRequestUrl);
-            if (!paymentRequestData) {
+            const paymentRequest = await this.getPaymentRequest(paymentRequestUrl);
+            if (!paymentRequest) {
                 throw new Error(`ATXP: payment request ${paymentRequestId} not found on server ${paymentRequestUrl}`);
             }
-            // Handle multi-destination format
-            if (paymentRequestData.destinations && paymentRequestData.destinations.length > 0) {
-                return this.handleMultiDestinationPayment(paymentRequestData, paymentRequestUrl, paymentRequestId);
+            // Handle multi-option format
+            if (paymentRequest.options && paymentRequest.options.length > 0) {
+                return this.handleMultiDestinationPayment(paymentRequest, paymentRequestUrl, paymentRequestId);
             }
-            // Payment request doesn't have destinations - this shouldn't happen with new SDK
-            throw new Error(`ATXP: payment request does not contain destinations array`);
+            // Payment request doesn't have options - this shouldn't happen with new SDK
+            throw new Error(`ATXP: payment request does not contain options array`);
         };
-        this.getPaymentRequestData = async (paymentRequestUrl) => {
+        this.getPaymentRequest = async (paymentRequestUrl) => {
             const prRequest = await this.sideChannelFetch(paymentRequestUrl);
             if (!prRequest.ok) {
                 throw new Error(`ATXP: GET ${paymentRequestUrl} failed: ${prRequest.status} ${prRequest.statusText}`);
             }
             const paymentRequest = await prRequest.json();
             // Parse amount strings to BigNumber objects
-            if (paymentRequest.destinations) {
-                for (const dest of paymentRequest.destinations) {
-                    if (typeof dest.amount === 'string' || typeof dest.amount === 'number') {
-                        dest.amount = new BigNumber(dest.amount);
+            if (paymentRequest.options) {
+                for (const option of paymentRequest.options) {
+                    if (typeof option.amount === 'string' || typeof option.amount === 'number') {
+                        option.amount = new BigNumber(option.amount);
                     }
                 }
             }