@go-labs-sg/bb 2.18.0 → 2.19.0

package/dist/commands.js

@@ -35,16 +35,35 @@ const getAllNotifications = async () => {
     } while (cursor);
     return notifications;
 };
-const createApprovalEmailDeliveryReport = ({ sentCount, skippedCount, message, }) => ({
-    sent: sentCount > 0,
-    count: sentCount,
-    sentCount,
-    skippedCount,
-    message,
-});
+const createApprovalEmailDeliveryReport = (delivery) => {
+    if (!delivery) {
+        return {
+            queued: false,
+            sent: false,
+            count: 0,
+            queuedCount: 0,
+            sentCount: 0,
+            skippedCount: 0,
+            error: "No approval email delivery was queued",
+        };
+    }
+    const { queuedCount, sentCount, skippedCount, message, operationIds } = delivery;
+    return {
+        queued: queuedCount > 0,
+        sent: false,
+        count: queuedCount,
+        queuedCount,
+        sentCount,
+        skippedCount,
+        operationIds,
+        message,
+    };
+};
 const createApprovalEmailDeliveryFailure = (error) => ({
+    queued: false,
     sent: false,
     count: 0,
+    queuedCount: 0,
     sentCount: 0,
     skippedCount: 0,
     error,
@@ -533,16 +552,11 @@ export async function approveBill(billId) {
     }
     let email;
     try {
-        await api.email.sendReplyBillApprovalEmail.mutate({
-            id: result.updatedApproval.id,
-        });
-        email = { sent: true };
+        const delivery = result.emailDelivery;
+        email = createApprovalEmailDeliveryReport(delivery);
     }
     catch (error) {
-        email = {
-            sent: false,
-            error: error instanceof Error ? error.message : String(error),
-        };
+        email = createApprovalEmailDeliveryFailure(error instanceof Error ? error.message : String(error));
     }
     out({
         ...result,
@@ -629,7 +643,7 @@ export async function createBillApproval(billId) {
     }
     else {
         try {
-            const delivery = await api.email.sendBillApprovalRequestEmail.mutate(approvalIds);
+            const delivery = result.emailDelivery;
             email = createApprovalEmailDeliveryReport(delivery);
         }
         catch (error) {
@@ -1078,7 +1092,7 @@ export async function createBudgetApproval(budgetId, preflight) {
     }
     else {
         try {
-            const delivery = await api.email.sendApprovalRequestEmail.mutate(approvalIds);
+            const delivery = result.emailDelivery;
             email = createApprovalEmailDeliveryReport(delivery);
         }
         catch (error) {
@@ -1488,7 +1502,7 @@ export async function createSupplierFromPayload(raw) {
     }
     else {
         try {
-            const delivery = await api.email.sendSupplierApprovalRequestEmail.mutate(approvalIds);
+            const delivery = approval.emailDelivery;
             emailResult = createApprovalEmailDeliveryReport(delivery);
         }
         catch (error) {
@@ -1534,7 +1548,7 @@ export async function updateSupplierFromPayload(raw) {
         }
         else {
             try {
-                const delivery = await api.email.sendSupplierApprovalRequestEmail.mutate(approvalIds);
+                const delivery = approval.emailDelivery;
                 approvalResult = {
                     requested: true,
                     approvalCount: approvalIds.length,
@@ -1614,16 +1628,11 @@ export async function approveBudget(budgetId) {
     });
     let email;
     try {
-        await api.email.sendReplyApprovalEmail.mutate({
-            id: result.updatedApproval.id,
-        });
-        email = { sent: true };
+        const delivery = result.emailDelivery;
+        email = createApprovalEmailDeliveryReport(delivery);
     }
     catch (error) {
-        email = {
-            sent: false,
-            error: error instanceof Error ? error.message : String(error),
-        };
+        email = createApprovalEmailDeliveryFailure(error instanceof Error ? error.message : String(error));
     }
     out({
         ...result,
@@ -1653,16 +1662,11 @@ export async function rejectBudget(budgetId, reason) {
     });
     let email;
     try {
-        await api.email.sendReplyApprovalEmail.mutate({
-            id: result.updatedApproval.id,
-        });
-        email = { sent: true };
+        const delivery = result.emailDelivery;
+        email = createApprovalEmailDeliveryReport(delivery);
     }
     catch (error) {
-        email = {
-            sent: false,
-            error: error instanceof Error ? error.message : String(error),
-        };
+        email = createApprovalEmailDeliveryFailure(error instanceof Error ? error.message : String(error));
     }
     out({
         ...result,
@@ -1751,16 +1755,11 @@ export async function approveSupplier(supplierId) {
     });
     let email;
     try {
-        await api.email.sendReplySupplierApprovalEmail.mutate({
-            id: result.updatedApproval.id,
-        });
-        email = { sent: true };
+        const delivery = result.emailDelivery;
+        email = createApprovalEmailDeliveryReport(delivery);
     }
     catch (error) {
-        email = {
-            sent: false,
-            error: error instanceof Error ? error.message : String(error),
-        };
+        email = createApprovalEmailDeliveryFailure(error instanceof Error ? error.message : String(error));
     }
     out({
         ...result,
@@ -1789,16 +1788,11 @@ export async function rejectSupplier(supplierId, reason) {
     });
     let email;
     try {
-        await api.email.sendReplySupplierApprovalEmail.mutate({
-            id: result.updatedApproval.id,
-        });
-        email = { sent: true };
+        const delivery = result.emailDelivery;
+        email = createApprovalEmailDeliveryReport(delivery);
     }
     catch (error) {
-        email = {
-            sent: false,
-            error: error instanceof Error ? error.message : String(error),
-        };
+        email = createApprovalEmailDeliveryFailure(error instanceof Error ? error.message : String(error));
     }
     out({
         ...result,
@@ -1831,16 +1825,11 @@ export async function rejectBill(billId, reason) {
     }
     let email;
     try {
-        await api.email.sendReplyBillApprovalEmail.mutate({
-            id: result.updatedApproval.id,
-        });
-        email = { sent: true };
+        const delivery = result.emailDelivery;
+        email = createApprovalEmailDeliveryReport(delivery);
     }
     catch (error) {
-        email = {
-            sent: false,
-            error: error instanceof Error ? error.message : String(error),
-        };
+        email = createApprovalEmailDeliveryFailure(error instanceof Error ? error.message : String(error));
     }
     out({
         ...result,

package/dist/index.js

@@ -176,7 +176,7 @@ Budgets
   create-budget              --payload '<json>'  (budget.createBudget; when sourceBudgetId contains unavailable items, set unavailableItemReviewAcknowledged=true after explicit review)
   update-budget              --payload '<json>'  (budget.updateBudget; must include id)
   delete-budget <budgetId>
-  create-budget-approval <budgetId>   (also sends approval request emails)
+  create-budget-approval <budgetId>   (also queues approval request emails)
   create-estimate <budgetId>          (creates QuickBooks estimate; supports preflight updates)
   send-estimate-to-contact-person --payload '<json>'   Send using the web contact-person workflow; payload includes recipient fields plus HTML content/signature.
   add-budget-items           --budgetId --items '[{"itemId":"…","quantity":1,"markup":30},…]'
@@ -203,7 +203,7 @@ Bills
   list-claims                same flags as list-bills; only reimbursable claims (ignores --isClaimable)
     isClaimable differentiates the shared bill/claim records: false = bill, true = claim.
   Non-legacy supplier bills from 1 Jul 2026 00:00 SGT require approved quotation coverage linked to every selected line item, independent of the bill supplier, including already-paid bills.
-  create-bill-approval <billId>       (also sends approval request emails)
+  create-bill-approval <billId>       (also queues approval request emails)
   create-bill                --payload '<json>'  (bill.create; payload.isClaimable false = bill, true = claim)
   validate-bill-selection    --payload '<json>'  Check project and supplier validity separately, then verify approved quotation links for each budgetItemId; alreadyPaid never bypasses quotation checks.
   stage-bill-attachment <projectId> <filePath...> [--file <path>] [--files <csv>]   Upload files before create-bill; returns attachment JSON for attachments/paymentProofAttachments
@@ -251,7 +251,7 @@ Customer invoices
   void-customer-invoice <batchId>              Void the invoice batch in QuickBooks.
   approve-customer-invoice <batchId>           Admin approval; rejects voided invoices, notifies the creator, and closes the estimate at cumulative 100% coverage.
   reject-customer-invoice <batchId> --reason <text>   Admin rejection; voids QBO invoices and notifies the creator.
-  send-customer-invoice-to-contact-person --payload '<json>'   Same composer workflow as web; requires invoiceId, to, cc, replyTo, subject, HTML content, and HTML signature.
+  send-customer-invoice-to-contact-person --payload '<json>'   Queues same composer workflow as web; requires invoiceId, to, cc, replyTo, subject, HTML content, and HTML signature.
   download-customer-invoice-pdf <invoiceId> [--output <path>]
   sync-customer-invoice <invoiceId>             Refresh local status and balance from QuickBooks; paid invoices become PAID, reversals restore delivery/approval, and QBO voids can reopen an estimate.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@go-labs-sg/bb",
-  "version": "2.18.0",
+  "version": "2.19.0",
   "description": "Budget Builder CLI for AI agents — manage budgets, bills, claims, quotations, and customer invoices with explicit workflow previews for sensitive changes.",
   "type": "module",
   "main": "./dist/index.js",