Merge pull request #7966 from BitGo/BTC-2775.bolt11

feat(abstract-lightning): make valueMsat optional add valueSat support

Author: davidkaplanbitgo

modules/abstract-lightning/src/codecs/api/invoice.ts

@@ -18,18 +18,12 @@ export const InvoiceStatus = t.union(
 );
 export type InvoiceStatus = t.TypeOf<typeof InvoiceStatus>;
 
-export const CreateInvoiceBody = t.intersection(
-  [
-    t.type({
-      valueMsat: BigIntFromString,
-    }),
-    t.partial({
-      memo: t.string,
-      expiry: t.number,
-    }),
-  ],
-  'CreateInvoiceBody'
-);
+export const CreateInvoiceBody = t.partial({
+  valueMsat: BigIntFromString,
+  valueSat: t.number,
+  memo: t.string,
+  expiry: t.number,
+});
 export type CreateInvoiceBody = t.TypeOf<typeof CreateInvoiceBody>;
 
 /**
@@ -52,6 +46,9 @@ export const Invoice = t.intersection(
       updatedAt: DateFromISOString,
     }),
     t.partial({
+      // Keep valueSat for backward compatibility with older versions of the API
+      // Will be moved up to required fields in the future
+      valueSat: t.number,
       memo: t.string,
       amtPaidMsat: BigIntFromString,
     }),

modules/express/test/unit/lightning/lightningInvoiceRoutes.test.ts

@@ -101,9 +101,7 @@ describe('Lightning Invoice Routes', () => {
       });
       req.bitgo = bitgo;
 
-      await should(handleCreateLightningInvoice(req)).be.rejectedWith(
-        /^Invalid request body to create lightning invoice/
-      );
+      await should(handleCreateLightningInvoice(req)).be.rejected();
     });
   });