src/core/services/PaymentService.ts

@@ -5,15 +5,7 @@ import { toast } from 'vue-sonner'
 
 export interface PaymentResult {
   payment_hash: string
-  /**
+  fee_msat: number
-   * Routing fee in millisats, always positive.
-   *
-   * LNbits returns this as `fee`, not `fee_msat`, and signs it negative for
-   * outgoing payments (like `amount`). This interface previously declared
-   * `fee_msat: number`, a field the API never sends, so every consumer read
-   * `undefined` while the compiler vouched for a number.
-   */
-  feeMsat: number
   preimage: string
 }
 
@@ -238,16 +230,7 @@ export class PaymentService extends BaseService {
       throw new Error(errorMessage)
     }
 
-    // Normalize at the boundary rather than handing the raw LNbits payload
+    return await response.json()
-    // to callers: `fee` is signed millisats, and consumers all want a
-    // positive magnitude. Doing it here means no call site can pick the
-    // wrong field name or forget the sign.
-    const raw = await response.json()
-    return {
-      payment_hash: raw.payment_hash,
-      feeMsat: Math.abs(Number(raw.fee ?? 0)),
-      preimage: raw.preimage ?? ''
-    }
   }
 
   /**
@@ -283,7 +266,7 @@ export class PaymentService extends BaseService {
       
       this.debug('Payment successful', {
         paymentHash: paymentResult.payment_hash,
-        feeMsat: paymentResult.feeMsat
+        feeMsat: paymentResult.fee_msat
       })
 
       // Show success notification if enabled

src/modules/events/services/LnbitsPaymentProvider.ts

@@ -84,9 +84,7 @@ export class LnbitsPaymentProvider implements PaymentProvider {
     const data = await response.json()
     return {
       paymentHash: data.payment_hash,
-      // LNbits sends `fee` (signed millisats), never `fee_msat`. Reading the
+      feeMsat: data.fee_msat ?? 0,
-      // latter meant every payment recorded a zero fee.
-      feeMsat: Math.abs(Number(data.fee ?? 0)),
       preimage: data.preimage ?? '',
     }
   }

src/modules/market/composables/useLightningPayment.ts

@@ -42,7 +42,7 @@ export function useLightningPayment() {
             status: 'paid' as const,
             paidAt: Math.floor(Date.now() / 1000),
             paymentHash: paymentResult.payment_hash,
-            feeMsat: paymentResult.feeMsat,
+            feeMsat: paymentResult.fee_msat,
             items: [...order.items], // Convert readonly to mutable
             shippingZone: order.shippingZone ? {
               ...order.shippingZone,
@@ -88,7 +88,7 @@ export function useLightningPayment() {
             status: 'paid' as const,
             paidAt: Math.floor(Date.now() / 1000),
             paymentHash: result.payment_hash,
-            feeMsat: result.feeMsat,
+            feeMsat: result.fee_msat,
             items: [...order.items], // Convert readonly to mutable
             shippingZone: order.shippingZone ? {
               ...order.shippingZone,

src/modules/market/types/market.ts

@@ -129,7 +129,6 @@ export interface Order {
   originalOrderId?: string
   lightningInvoice?: any
   paymentHash?: string
-  feeMsat?: number  // Routing fee paid on the Lightning payment, in millisats
   paidAt?: number
   paymentStatus?: 'pending' | 'paid' | 'expired'
   paid?: boolean  // Direct boolean field matching nostrmarket reference