@go-labs-sg/bb 1.2.8 → 1.2.9

package/README.md

@@ -56,12 +56,12 @@ Global options and flags use `--key=value` or `--key value` (see `bb help`).
 
 | Area | Commands (non-exhaustive) |
 | --- | --- |
-| **Budgets** | `list-budgets` (full payload by default; `--summary` or `--includeDetails false` for slim list), `get-budget`, `get-budget-items`, `get-budget-details`, `get-budget-categories`, `get-budget-versions`, `update-budget-status`, `create-budget` / `update-budget` (`--payload`), `delete-budget`, `create-budget-approval`, `add-budget-items`, `update-budget-item`, `remove-budget-item`, `reorder-budget-items`, `update-budget-item-supplier`, `create-budget-category`, `update-budget-category`, `delete-budget-category`, `update-budget-commission`, `update-budget-discount` (`--payload` where noted), `upload-budget-attachment` (`<budgetId>` + local file path; uses `attachment.requestBudgetAttachmentUpload` + PUT + `attachment.confirmBudgetAttachment`) |
-| **Bills** | `list-bills`, `list-claims`, `create-bill` (`--payload`), `update-bill` (`--payload`), `delete-bill`, `create-bill-approval`, `update-bill-status`, `patch-bill-payment` (PAID bills: `--paymentTrackingUrl`, `--paymentReference`, `--quickbooksBillId`, `--paymentDate` ISO; clear with `--clearPaymentTrackingUrl` / `--clearPaymentReference` / `--clearQuickbooksBillId` / `--clearPaymentDate` `true`), `patch-bill-invoice-number`, `get-bill-attachments`, `upload-bill-attachment` (`<billId>` + local path), `get-bill-details` |
-| **Approvals** | `list-approvals` / `get-pending-approvals`, `approve-bill`, `reject-bill`, `approve-budget`, `reject-budget`, `approve-supplier`, `reject-supplier` |
+| **Budgets** | `list-budgets` (full payload by default; `--summary` or `--includeDetails false` for slim list), `get-budget`, `get-budget-items`, `get-budget-details`, `get-budget-categories`, `get-budget-versions`, `update-budget-status`, `create-budget` / `update-budget` (`--payload`), `delete-budget`, `create-budget-approval` (also sends approval request emails), `add-budget-items`, `update-budget-item`, `remove-budget-item`, `reorder-budget-items`, `update-budget-item-supplier`, `create-budget-category`, `update-budget-category`, `delete-budget-category`, `update-budget-commission`, `update-budget-discount` (`--payload` where noted), `upload-budget-attachment` (`<budgetId>` + local file path; uses `attachment.requestBudgetAttachmentUpload` + PUT + `attachment.confirmBudgetAttachment`) |
+| **Bills** | `list-bills`, `list-claims`, `create-bill` (`--payload`), `update-bill` (`--payload`), `delete-bill`, `create-bill-approval` (also sends approval request emails), `update-bill-status`, `patch-bill-payment` (PAID bills: `--paymentTrackingUrl`, `--paymentReference`, `--quickbooksBillId`, `--paymentDate` ISO; clear with `--clearPaymentTrackingUrl` / `--clearPaymentReference` / `--clearQuickbooksBillId` / `--clearPaymentDate` `true`), `patch-bill-invoice-number`, `get-bill-attachments`, `upload-bill-attachment` (`<billId>` + local path), `get-bill-details` |
+| **Approvals** | `list-approvals` / `get-pending-approvals`, `approve-bill` / `reject-bill` (send reply email), `approve-budget` / `reject-budget` (send reply email), `approve-supplier` / `reject-supplier` (send reply email) |
 | **Companies & projects** | `list-companies`, `get-company`, `create-company`, `update-company` (`--payload`), `delete-company`, `list-projects`, `get-project`, `create-project` (`--asanaTaskId`, `--slackChannelId`, `--slackChannelUrl`, `--slackChannelName`, …; optional `--requestQboAccountantNotification false` to skip QBO accountant emails), `update-project` (`--payload`, optional `requestQboAccountantNotification` in JSON), `delete-project`, `update-project-status` (`<id>` `<status>`: `PITCH` \| `EVENT` \| `COMPLETED` \| `LOST`) |
 | **Contacts** | `list-contacts`, `create-contact-person` (`--payload`), `update-contact-person` (`--payload`) |
-| **Suppliers & items** | `list-suppliers`, `create-supplier` (`--payload`; when supplier status is `PENDING_APPROVAL`, also runs `supplier.createSupplierApproval` and `email.sendSupplierApprovalRequestEmail`), `update-supplier` (`--payload`), `delete-suppliers` (`--ids` CSV; admin), `create-certification` / `create-payment-method` / `create-supplier-role` / `create-supplier-tag` (`--name`), `get-supplier-details`, `get-supplier-analytics`, `list-items`, `create-item` (`--payload`), `update-item` (`--payload`), `delete-item`, `get-item`, `list-item-categories`, `create-item-category`, `update-item-category`, `delete-item-categories` (`--ids` CSV; admin) |
+| **Suppliers & items** | `list-suppliers`, `create-supplier` (`--payload`; when supplier status is `PENDING_APPROVAL`, also runs `supplier.createSupplierApproval` and `email.sendSupplierApprovalRequestEmail`), `update-supplier` (`--payload`; when supplier status is `PENDING_APPROVAL`, also runs `supplier.createSupplierApproval` and `email.sendSupplierApprovalRequestEmail`), `delete-suppliers` (`--ids` CSV; admin), `create-certification` / `create-payment-method` / `create-supplier-role` / `create-supplier-tag` (`--name`), `get-supplier-details`, `get-supplier-analytics`, `list-items`, `create-item` (`--payload`), `update-item` (`--payload`), `delete-item`, `get-item`, `list-item-categories`, `create-item-category`, `update-item-category`, `delete-item-categories` (`--ids` CSV; admin) |
 | **Dashboard & users** | `list-users`, `get-user-performance`, `get-dashboard`, `get-monthly-metrics`, `get-system-overview`, `get-estimate-performance`, `get-financial-overview` |
 | **Errors** | `get-recent-errors`, `get-error-metrics` |
 | **Historical / benchmarks** | `get-approved-budgets`, `get-budget-category-benchmarks`, `get-item-pricing-history`, `get-supplier-pricing-history` |

package/dist/commands.js

@@ -81,7 +81,21 @@ export async function approveBill(billId) {
         billApprovalId: approval.id,
         status: "APPROVED",
     });
-    out(result);
+    let email;
+    try {
+        await api.email.sendReplyBillApprovalEmail.mutate(result.updatedApproval);
+        email = { sent: true };
+    }
+    catch (error) {
+        email = {
+            sent: false,
+            error: error instanceof Error ? error.message : String(error),
+        };
+    }
+    out({
+        ...result,
+        email,
+    });
 }
 export async function listApprovals(opts) {
     const notifications = await api.budget.getNotifications.query();
@@ -137,7 +151,28 @@ export async function listSuppliers(opts) {
 // --- Bills (MCP parity) ---
 export async function createBillApproval(billId) {
     const result = await api.bill.createApproval.mutate({ billId });
-    out(result);
+    const approvalIds = (result.results ?? []).map((item) => ({ id: item.id }));
+    let email;
+    if (approvalIds.length === 0) {
+        email = { sent: false, count: 0, error: "No approval ids returned" };
+    }
+    else {
+        try {
+            await api.email.sendBillApprovalRequestEmail.mutate(approvalIds);
+            email = { sent: true, count: approvalIds.length };
+        }
+        catch (error) {
+            email = {
+                sent: false,
+                count: approvalIds.length,
+                error: error instanceof Error ? error.message : String(error),
+            };
+        }
+    }
+    out({
+        ...result,
+        email,
+    });
 }
 export async function updateBillStatus(opts) {
     const result = await api.bill.updateStatus.mutate({
@@ -264,7 +299,28 @@ export async function getBudgetVersions(budgetId) {
 }
 export async function createBudgetApproval(budgetId) {
     const result = await api.budget.createBudgetApproval.mutate({ budgetId });
-    out(result);
+    const approvalIds = (result.results ?? []).map((item) => ({ id: item.id }));
+    let email;
+    if (approvalIds.length === 0) {
+        email = { sent: false, count: 0, error: "No approval ids returned" };
+    }
+    else {
+        try {
+            await api.email.sendApprovalRequestEmail.mutate(approvalIds);
+            email = { sent: true, count: approvalIds.length };
+        }
+        catch (error) {
+            email = {
+                sent: false,
+                count: approvalIds.length,
+                error: error instanceof Error ? error.message : String(error),
+            };
+        }
+    }
+    out({
+        ...result,
+        email,
+    });
 }
 export async function getBudgetDetails(budgetId) {
     const detail = await api.budget.getBudgetDetail.query({ id: budgetId });
@@ -412,8 +468,59 @@ export async function createSupplierFromPayload(raw) {
 }
 export async function updateSupplierFromPayload(raw) {
     const input = parseSupplierUpdatePayload(raw);
-    const result = await api.supplier.updateSupplier.mutate(input);
-    out(result);
+    const updated = await api.supplier.updateSupplier.mutate(input);
+    const supplier = updated.result;
+    if (supplier.status !== "PENDING_APPROVAL") {
+        out(updated);
+        return;
+    }
+    let approvalResult;
+    try {
+        const approval = await api.supplier.createSupplierApproval.mutate({
+            supplierId: supplier.id,
+        });
+        const approvalIds = (approval.results ?? []).map((item) => ({
+            id: item.id,
+        }));
+        if (approvalIds.length === 0) {
+            approvalResult = {
+                requested: true,
+                approvalCount: 0,
+                email: { sent: false, count: 0, error: "No approval ids returned" },
+            };
+        }
+        else {
+            try {
+                await api.email.sendSupplierApprovalRequestEmail.mutate(approvalIds);
+                approvalResult = {
+                    requested: true,
+                    approvalCount: approvalIds.length,
+                    email: { sent: true, count: approvalIds.length },
+                };
+            }
+            catch (error) {
+                approvalResult = {
+                    requested: true,
+                    approvalCount: approvalIds.length,
+                    email: {
+                        sent: false,
+                        count: approvalIds.length,
+                        error: error instanceof Error ? error.message : String(error),
+                    },
+                };
+            }
+        }
+    }
+    catch (error) {
+        approvalResult = {
+            requested: false,
+            error: error instanceof Error ? error.message : String(error),
+        };
+    }
+    out({
+        ...updated,
+        supplierApproval: approvalResult,
+    });
 }
 export async function createItemFromPayload(raw) {
     const input = parseItemCreatePayload(raw);
@@ -457,11 +564,25 @@ export async function approveBudget(budgetId) {
     if (!approval) {
         throw new Error("No pending budget approval found for this budget and approver.");
     }
-    await api.budget.updateBudgetApproval.mutate({
+    const result = await api.budget.updateBudgetApproval.mutate({
         budgetApprovalId: approval.id,
         status: ExtendedApprovalStatus.APPROVED,
     });
-    out({ success: true, message: "Budget approved successfully" });
+    let email;
+    try {
+        await api.email.sendReplyApprovalEmail.mutate(result.updatedApproval);
+        email = { sent: true };
+    }
+    catch (error) {
+        email = {
+            sent: false,
+            error: error instanceof Error ? error.message : String(error),
+        };
+    }
+    out({
+        ...result,
+        email,
+    });
 }
 export async function rejectBudget(budgetId, reason) {
     const notifications = await api.budget.getNotifications.query();
@@ -474,12 +595,26 @@ export async function rejectBudget(budgetId, reason) {
     if (!approval) {
         throw new Error("No pending budget approval found for this budget and approver.");
     }
-    await api.budget.updateBudgetApproval.mutate({
+    const result = await api.budget.updateBudgetApproval.mutate({
         budgetApprovalId: approval.id,
         status: ExtendedApprovalStatus.REJECTED,
         rejectionReason: reason,
     });
-    out({ success: true, message: "Budget rejected" });
+    let email;
+    try {
+        await api.email.sendReplyApprovalEmail.mutate(result.updatedApproval);
+        email = { sent: true };
+    }
+    catch (error) {
+        email = {
+            sent: false,
+            error: error instanceof Error ? error.message : String(error),
+        };
+    }
+    out({
+        ...result,
+        email,
+    });
 }
 export async function getSupplierDetails(supplierId) {
     const supplier = await api.supplier.getSupplierById.query({
@@ -497,11 +632,28 @@ export async function approveSupplier(supplierId) {
     if (!approval) {
         throw new Error("No pending supplier approval found.");
     }
-    await api.supplier.updateSupplierApproval.mutate({
+    const result = await api.supplier.updateSupplierApproval.mutate({
         supplierApprovalId: approval.id,
         status: ExtendedApprovalStatus.APPROVED,
     });
-    out({ success: true, message: "Supplier approved successfully" });
+    let email;
+    try {
+        await api.email.sendReplySupplierApprovalEmail.mutate({
+            ...result.updatedApproval,
+            rejectionReason: result.updatedApproval.rejectionReason ?? undefined,
+        });
+        email = { sent: true };
+    }
+    catch (error) {
+        email = {
+            sent: false,
+            error: error instanceof Error ? error.message : String(error),
+        };
+    }
+    out({
+        ...result,
+        email,
+    });
 }
 export async function rejectSupplier(supplierId, reason) {
     const notifications = await api.budget.getNotifications.query();
@@ -513,12 +665,29 @@ export async function rejectSupplier(supplierId, reason) {
     if (!approval) {
         throw new Error("No pending supplier approval found.");
     }
-    await api.supplier.updateSupplierApproval.mutate({
+    const result = await api.supplier.updateSupplierApproval.mutate({
         supplierApprovalId: approval.id,
         status: ExtendedApprovalStatus.REJECTED,
         rejectionReason: reason,
     });
-    out({ success: true, message: "Supplier rejected" });
+    let email;
+    try {
+        await api.email.sendReplySupplierApprovalEmail.mutate({
+            ...result.updatedApproval,
+            rejectionReason: result.updatedApproval.rejectionReason ?? undefined,
+        });
+        email = { sent: true };
+    }
+    catch (error) {
+        email = {
+            sent: false,
+            error: error instanceof Error ? error.message : String(error),
+        };
+    }
+    out({
+        ...result,
+        email,
+    });
 }
 export async function rejectBill(billId, reason) {
     const notifications = await api.budget.getNotifications.query();
@@ -530,12 +699,26 @@ export async function rejectBill(billId, reason) {
     if (!approval) {
         throw new Error("No pending bill approval found.");
     }
-    await api.bill.updateApproval.mutate({
+    const result = await api.bill.updateApproval.mutate({
         billApprovalId: approval.id,
         status: ExtendedApprovalStatus.REJECTED,
         rejectionReason: reason,
     });
-    out({ success: true, message: "Bill rejected" });
+    let email;
+    try {
+        await api.email.sendReplyBillApprovalEmail.mutate(result.updatedApproval);
+        email = { sent: true };
+    }
+    catch (error) {
+        email = {
+            sent: false,
+            error: error instanceof Error ? error.message : String(error),
+        };
+    }
+    out({
+        ...result,
+        email,
+    });
 }
 // --- Companies & projects ---
 export async function listCompanies(opts) {

package/dist/index.js

@@ -79,7 +79,7 @@ Budgets
   create-budget              --payload '<json>'  (budget.createBudget; ISO dates for date/startTime/endTime)
   update-budget              --payload '<json>'  (budget.updateBudget; must include id)
   delete-budget <budgetId>
-  create-budget-approval <budgetId>
+  create-budget-approval <budgetId>   (also sends approval request emails)
   add-budget-items           --budgetId --items '[{"itemId":"…","quantity":1,"markup":30},…]'
   update-budget-item         --id <budgetItemId> [--description] [--note] [--quantity] [--markup] [--cost] [--unitPrice] [--isFreeOfCharge true|false] [--gstInclusive true|false]
   remove-budget-item <budgetItemId>
@@ -95,7 +95,7 @@ Budgets
 Bills
   list-bills                 [--projectId] [--budgetId] [--status CSV] [--search <text>] [--isClaimable true|false] [--createdByIds <csv>] [--sortBy createdAt|amount|status] [--sortDir asc|desc] [--page] [--pageSize]
   list-claims                same flags as list-bills; only reimbursable claims (ignores --isClaimable)
-  create-bill-approval <billId>
+  create-bill-approval <billId>       (also sends approval request emails)
   create-bill                --payload '<json>'  (bill.create; use upload-bill-attachment after create to add files)
   update-bill                --payload '<json>'  (bill.update; must include id)
   delete-bill <billId>
@@ -110,12 +110,12 @@ Bills
 
 Approvals
   list-approvals | get-pending-approvals   [--type budget|supplier|bill|all]
-  approve-bill <billId>
-  reject-bill <billId> --reason <text>
-  approve-budget <budgetId>
-  reject-budget <budgetId> --reason <text>
-  approve-supplier <supplierId>
-  reject-supplier <supplierId> --reason <text>
+  approve-bill <billId>               (also sends requester reply email)
+  reject-bill <billId> --reason <text> (also sends requester reply email)
+  approve-budget <budgetId>           (also sends requester reply email)
+  reject-budget <budgetId> --reason <text> (also sends requester reply email)
+  approve-supplier <supplierId>       (also sends requester reply email)
+  reject-supplier <supplierId> --reason <text> (also sends requester reply email)
 
 Companies & projects
   list-companies             [--name] [--createdBy] [--page] [--perPage]
@@ -139,7 +139,7 @@ Contacts
 Suppliers & items
   list-suppliers             [--name] [--page] [--perPage]
   create-supplier            --payload '<json>'  (supplier.createSupplier; auto-requests approval + emails when status is PENDING_APPROVAL)
-  update-supplier            --payload '<json>'  (supplier.updateSupplier; must include id)
+  update-supplier            --payload '<json>'  (supplier.updateSupplier; must include id; auto-requests approval + emails when status is PENDING_APPROVAL)
   delete-suppliers           --ids <csv>  (supplier.deleteSuppliers; admin)
   create-certification       --name  (supplier.createCertification)
   create-payment-method      --name  (supplier.createPaymentMethod)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@go-labs-sg/bb",
-  "version": "1.2.8",
+  "version": "1.2.9",
   "description": "Budget Builder CLI — list budgets, bills, approvals, suppliers; approve bills; change status. For AI agents (e.g. Chuck/OpenClaw).",
   "type": "module",
   "main": "./dist/index.js",