@go-labs-sg/bb 1.0.3 → 1.0.4

package/dist/commands.js

@@ -67,6 +67,9 @@ export async function listApprovals(opts) {
     const notifications = await api.budget.getNotifications.query();
     const pending = notifications.filter(isPendingRequest);
     const raw = opts.type?.toUpperCase();
+    if (raw !== undefined && raw !== "ALL" && !isApprovalType(raw)) {
+        throw new Error("Invalid --type. Use one of: budget, supplier, bill, all.");
+    }
     const typeFilter = !opts.type || opts.type.toLowerCase() === "all"
         ? (_n) => true
         : (n) => raw !== undefined && isApprovalType(raw) && n.type === raw;

package/dist/index.js

@@ -14,6 +14,15 @@ const BUDGET_STATUSES = [
     "ESTIMATE_REJECTED",
     "ESTIMATE_CLOSED",
 ];
+function parsePositiveIntFlag(value, flagName) {
+    if (value === undefined)
+        return undefined;
+    const parsed = Number.parseInt(String(value), 10);
+    if (!Number.isFinite(parsed) || parsed <= 0) {
+        throw new Error(`${flagName} must be a positive integer.`);
+    }
+    return parsed;
+}
 function printHelp() {
     const help = `
 bb — Budget Builder CLI (for AI agents / shell)
@@ -57,16 +66,18 @@ async function main() {
         switch (command) {
             case "list-budgets": {
                 const statusStr = getFlag(flags, "status");
+                const statuses = statusStr !== undefined
+                    ? String(statusStr)
+                        .split(",")
+                        .map((s) => s.trim())
+                        .filter((s) => s.length > 0)
+                    : undefined;
                 await listBudgets({
                     projectId: getFlag(flags, "projectId"),
                     name: getFlag(flags, "name"),
-                    statuses: statusStr !== undefined
-                        ? String(statusStr)
-                            .split(",")
-                            .map((s) => s.trim())
-                        : undefined,
-                    page: getFlag(flags, "page", (s) => Number.parseInt(s, 10)),
-                    perPage: getFlag(flags, "perPage", (s) => Number.parseInt(s, 10)),
+                    statuses,
+                    page: parsePositiveIntFlag(getFlag(flags, "page"), "--page"),
+                    perPage: parsePositiveIntFlag(getFlag(flags, "perPage"), "--perPage"),
                 });
                 break;
             }
@@ -101,8 +112,8 @@ async function main() {
                     projectId: getFlag(flags, "projectId"),
                     budgetId: getFlag(flags, "budgetId"),
                     status: getFlag(flags, "status"),
-                    page: getFlag(flags, "page", (s) => Number.parseInt(s, 10)),
-                    pageSize: getFlag(flags, "pageSize", (s) => Number.parseInt(s, 10)),
+                    page: parsePositiveIntFlag(getFlag(flags, "page"), "--page"),
+                    pageSize: parsePositiveIntFlag(getFlag(flags, "pageSize"), "--pageSize"),
                 });
                 break;
             }
@@ -124,8 +135,8 @@ async function main() {
             case "list-suppliers": {
                 await listSuppliers({
                     name: getFlag(flags, "name"),
-                    page: getFlag(flags, "page", (s) => Number.parseInt(s, 10)),
-                    perPage: getFlag(flags, "perPage", (s) => Number.parseInt(s, 10)),
+                    page: parsePositiveIntFlag(getFlag(flags, "page"), "--page"),
+                    perPage: parsePositiveIntFlag(getFlag(flags, "perPage"), "--perPage"),
                 });
                 break;
             }

package/dist/parse-args.js

@@ -8,8 +8,15 @@ export function parseArgs(argv) {
     const flags = {};
     for (let i = 0; i < args.length; i++) {
         const arg = args[i];
-        if (arg === undefined || arg === "--" || arg === "")
+        if (arg === undefined || arg === "")
             continue;
+        if (arg === "--") {
+            for (const rest of args.slice(i + 1)) {
+                if (rest !== "")
+                    positional.push(rest);
+            }
+            break;
+        }
         if (arg.startsWith("--")) {
             const eq = arg.indexOf("=");
             if (eq > 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@go-labs-sg/bb",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "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",