bamboohr-cli 1.0.19 → 1.0.20-gc6d5f1a.1

package/dist/index.js

@@ -68,6 +68,58 @@ function positiveInt(raw) {
   return n;
 }
 
+// ../../cli-utils/dist/text-or-file.js
+import { readFileSync as readFileSync3 } from "fs";
+import { InvalidArgumentError as InvalidArgumentError2, Option } from "commander";
+var STDIN_REF = "-";
+function rejectStdinSentinel(value) {
+  if (value === "@-" || value === STDIN_REF) {
+    throw new InvalidArgumentError2(`"${value}" looks like a stdin redirect, which is not supported here. Pass the text directly, or read it from a file/stdin with the matching --\u2026-file <path|-> option.`);
+  }
+  return value;
+}
+function readFileOrStdin(ref) {
+  if (ref === STDIN_REF) {
+    if (process.stdin.isTTY) {
+      throw new InvalidArgumentError2('"--\u2026-file -" reads stdin, but stdin is a terminal (nothing piped).');
+    }
+    try {
+      return readFileSync3(0, "utf8");
+    } catch (err) {
+      throw new InvalidArgumentError2(`failed to read stdin for "--\u2026-file -": ${err.message}`);
+    }
+  }
+  try {
+    return readFileSync3(ref, "utf8");
+  } catch (err) {
+    const e = err;
+    if (e.code === "ENOENT") {
+      throw new InvalidArgumentError2(`file not found: "${ref}".`);
+    }
+    throw new InvalidArgumentError2(`failed to read file "${ref}": ${e.message}`);
+  }
+}
+function textOrFileOption(cmd, name, opts = {}) {
+  const label = `${name.charAt(0).toUpperCase()}${name.slice(1)} content`;
+  cmd.option(`--${name} <text>`, opts.description ?? label, rejectStdinSentinel);
+  cmd.addOption(new Option(`--${name}-file <path>`, `Read --${name} from a file, or "-" for stdin (mutually exclusive with --${name})`).conflicts(name));
+  return cmd;
+}
+function resolveTextOrFile(opts, name, { required = true } = {}) {
+  const literal = opts[name];
+  const ref = opts[`${name}File`];
+  if (literal !== void 0 && ref !== void 0) {
+    throw new InvalidArgumentError2(`--${name} and --${name}-file are mutually exclusive; provide only one.`);
+  }
+  if (ref !== void 0) {
+    return readFileOrStdin(ref);
+  }
+  if (literal === void 0 && required) {
+    throw new InvalidArgumentError2(`a ${name} is required: provide --${name} <text> or --${name}-file <path>.`);
+  }
+  return literal;
+}
+
 // ../../cli-utils/dist/builders.js
 var EXAMPLES = /* @__PURE__ */ new WeakSet();
 function commandPath(cmd) {
@@ -496,10 +548,10 @@ function mergeFields(existing, extra) {
 }
 
 // src/commands/employee/goals.ts
-import { Option } from "commander";
+import { Option as Option2 } from "commander";
 var GOAL_FILTERS = ["open", "closed", "all"];
 function goals(parent) {
-  const cmd = parent.command("goals <employeeId>").description("Get employee performance goals").addOption(new Option("--filter <filter>", "Filter goals by status").choices(GOAL_FILTERS).default("all"));
+  const cmd = parent.command("goals <employeeId>").description("Get employee performance goals").addOption(new Option2("--filter <filter>", "Filter goals by status").choices(GOAL_FILTERS).default("all"));
   examples(cmd, ["123", ["123 --filter open", "only open goals"], ["123 --filter closed", "only closed goals"]]);
   cmd.action(async (employeeId, opts) => {
     const client = getClient();
@@ -513,10 +565,10 @@ function goals(parent) {
 
 // src/commands/employee/photo.ts
 import { writeFileSync as writeFileSync2 } from "fs";
-import { Option as Option2 } from "commander";
+import { Option as Option3 } from "commander";
 var PHOTO_SIZES = ["original", "large", "medium", "small", "xs", "tiny"];
 function photo(parent) {
-  const cmd = parent.command("photo <employeeId>").description("Download employee photo").requiredOption("--output <path>", "File path to save photo").addOption(new Option2("--size <size>", "Photo size").choices(PHOTO_SIZES).default("medium"));
+  const cmd = parent.command("photo <employeeId>").description("Download employee photo").requiredOption("--output <path>", "File path to save photo").addOption(new Option3("--size <size>", "Photo size").choices(PHOTO_SIZES).default("medium"));
   examples(cmd, [["123 --output ./photo.jpg", "medium size (default)"], "123 --output ./photo.jpg --size large"]);
   cmd.action(async (employeeId, opts) => {
     const client = getClient();
@@ -704,15 +756,84 @@ function balance(parent) {
 }
 
 // src/commands/timeoff/create.ts
-import { Option as Option3 } from "commander";
+import { InvalidArgumentError as InvalidArgumentError4, Option as Option4 } from "commander";
+
+// src/utils/timeoff-dates.ts
+import { InvalidArgumentError as InvalidArgumentError3 } from "commander";
+var WEEKDAY_NAMES = ["Sunday", "Monday", "Tuesday", "Wednesday", "Thursday", "Friday", "Saturday"];
+var YMD = /^\d{4}-\d{2}-\d{2}$/;
+function parseYmd(value, label) {
+  if (!YMD.test(value)) {
+    throw new InvalidArgumentError3(`${label} must be a YYYY-MM-DD date, got "${value}".`);
+  }
+  const date = /* @__PURE__ */ new Date(`${value}T00:00:00Z`);
+  if (Number.isNaN(date.getTime())) {
+    throw new InvalidArgumentError3(`${label} is not a valid date: "${value}".`);
+  }
+  return date;
+}
+function toYmd(date) {
+  return date.toISOString().slice(0, 10);
+}
+function buildTimeOffDates(params) {
+  const { start, end, units, defaultHours, holidays = [] } = params;
+  const startDate = parseYmd(start, "--start-date");
+  const endDate = parseYmd(end, "--end-date");
+  if (endDate < startDate) {
+    throw new InvalidArgumentError3(`--end-date ${end} is before --start-date ${start}.`);
+  }
+  const hoursByName = new Map(defaultHours.map((entry) => [entry.name, Number(entry.amount)]));
+  const fallbackHours = hoursByName.get("default") ?? 8;
+  const dates = [];
+  for (const day = new Date(startDate); day <= endDate; day.setUTCDate(day.getUTCDate() + 1)) {
+    const ymd = toYmd(day);
+    const workingHours = hoursByName.get(WEEKDAY_NAMES[day.getUTCDay()]) ?? fallbackHours;
+    const isHoliday = holidays.some((h) => ymd >= h.start && ymd <= h.end);
+    const working = workingHours > 0 && !isHoliday;
+    dates.push({ ymd, amount: working ? units === "hours" ? workingHours : 1 : 0 });
+  }
+  return dates;
+}
+
+// src/commands/timeoff/create.ts
+function parseDatesJson(raw) {
+  let parsed;
+  try {
+    parsed = JSON.parse(raw);
+  } catch {
+    throw new InvalidArgumentError4(
+      `--dates is not valid JSON. Expected an array like '[{"ymd":"2026-03-21","amount":0.5}]'.`
+    );
+  }
+  if (!Array.isArray(parsed) || parsed.length === 0 || !parsed.every(
+    (d) => typeof d === "object" && d !== null && typeof d.ymd === "string"
+  )) {
+    throw new InvalidArgumentError4(
+      '--dates must be a non-empty JSON array of {"ymd":"YYYY-MM-DD","amount":<number>} objects.'
+    );
+  }
+  return parsed.map((d) => ({ ymd: d.ymd, amount: Number(d.amount) }));
+}
 function create(parent) {
-  const cmd = parent.command("create [employeeId]").description("Create a time off request (defaults to current user)").requiredOption("--start-date <date>", "Start date (YYYY-MM-DD)").requiredOption("--end-date <date>", "End date (YYYY-MM-DD)").requiredOption("--type-id <id>", 'Time off type ID (use "timeoff types" to find IDs)', positiveInt).requiredOption("--amount <amount>", "Total amount of time off (in days or hours depending on type)", parseFloat).addOption(
-    new Option3("--status <status>", "Request status").choices(["requested", "approved"]).default("requested")
-  ).option("--note <text>", "Note to include with the request").option("--dates <json>", `Per-day amounts as JSON array, e.g. '[{"ymd":"2026-03-21","amount":4}]' for half-day`);
+  const cmd = parent.command("create [employeeId]").description("Create a time off request (defaults to current user)").requiredOption("--start-date <date>", "Start date (YYYY-MM-DD)").requiredOption("--end-date <date>", "End date (YYYY-MM-DD)").requiredOption("--type-id <id>", 'Time off type ID (use "timeoff types" to find IDs)', positiveInt).option(
+    "--amount <amount>",
+    "Expected total (days or hours, per the type). Optional sanity check \u2014 the total is computed from the per-day breakdown",
+    parseFloat
+  ).addOption(
+    new Option4("--status <status>", "Request status").choices(["requested", "approved"]).default("requested")
+  ).option(
+    "--dates <json>",
+    `Per-day amounts as JSON array, e.g. '[{"ymd":"2026-03-21","amount":0.5}]' for a half day. When omitted, generated automatically: 1 day (or the weekday's working hours) per working day, 0 on weekends and company holidays`
+  );
+  textOrFileOption(cmd, "note", { description: "Note to include with the request" });
   examples(cmd, [
-    "--start-date 2026-04-01 --end-date 2026-04-01 --type-id 83 --amount 1",
+    ["--start-date 2026-04-06 --end-date 2026-04-10 --type-id 83", "a full week off, spread per day automatically"],
     [
-      '504 --start-date 2026-04-01 --end-date 2026-04-01 --type-id 83 --amount 1 --note "Doctor appointment"',
+      `--start-date 2026-04-01 --end-date 2026-04-01 --type-id 83 --dates '[{"ymd":"2026-04-01","amount":0.5}]'`,
+      "a half day"
+    ],
+    [
+      '504 --start-date 2026-04-01 --end-date 2026-04-01 --type-id 83 --note "Doctor appointment"',
       "for another employee"
     ]
   ]);
@@ -720,15 +841,48 @@ function create(parent) {
     async (employeeId, opts) => {
       const client = getClient();
       const id = await resolveEmployeeId(client, employeeId);
+      const note = resolveTextOrFile(opts, "note", { required: false });
+      let dates;
+      if (opts.dates) {
+        dates = parseDatesJson(opts.dates);
+      } else {
+        const [types2, whosOut2] = await Promise.all([
+          getTimeOffTypes(client),
+          client.timeOff.getWhosOut({ start: opts.startDate, end: opts.endDate })
+        ]);
+        const type = types2.timeOffTypes.find((t) => Number(t.id) === opts.typeId);
+        if (!type) {
+          throw new InvalidArgumentError4(`Unknown time off type ID ${opts.typeId}. Run "timeoff types" to list IDs.`);
+        }
+        const holidays = whosOut2.filter((e) => e.type === "holiday").map((e) => ({ start: e.start, end: e.end }));
+        dates = buildTimeOffDates({
+          start: opts.startDate,
+          end: opts.endDate,
+          units: type.units,
+          defaultHours: types2.defaultHours,
+          holidays
+        });
+      }
+      const total = dates.reduce((sum, d) => sum + d.amount, 0);
+      if (total <= 0) {
+        throw new InvalidArgumentError4(
+          "The requested range contains no working days (weekends/company holidays only). Adjust the dates or pass --dates explicitly."
+        );
+      }
+      if (opts.amount !== void 0 && Math.abs(opts.amount - total) > 1e-9) {
+        throw new InvalidArgumentError4(
+          `--amount ${opts.amount} does not match the per-day total ${total} (working days in range, minus weekends and company holidays). Omit --amount, or pass --dates to set per-day amounts explicitly (e.g. half days).`
+        );
+      }
       const result = await client.timeOff.createRequest({
         employeeId: id,
         status: opts.status,
         start: opts.startDate,
         end: opts.endDate,
         timeOffTypeId: opts.typeId,
-        amount: opts.amount,
-        notes: opts.note ? [{ from: "employee", note: opts.note }] : void 0,
-        dates: opts.dates ? JSON.parse(opts.dates) : void 0
+        amount: total,
+        notes: note ? [{ from: "employee", note }] : void 0,
+        dates
       });
       output(
         result ? stripNullish(result) : { created: true, employeeId: id, start: opts.startDate, end: opts.endDate }
@@ -738,10 +892,10 @@ function create(parent) {
 }
 
 // src/commands/timeoff/requests.ts
-import { Option as Option4 } from "commander";
+import { Option as Option5 } from "commander";
 function requests(parent) {
-  const cmd = parent.command("requests").description("Get time off requests (defaults to current user)").option("--request-id <id>", "Specific request ID", positiveInt).addOption(new Option4("--action <action>", "Access level filter").choices(["view", "approve"])).option("--employee-id <id>", "Filter by employee ID").option("--all", "Show all visible requests instead of just current user").option("--start-date <date>", "Start date (YYYY-MM-DD)").option("--end-date <date>", "End date (YYYY-MM-DD)").addOption(
-    new Option4("--status <status>", "Filter by request status").choices([
+  const cmd = parent.command("requests").description("Get time off requests (defaults to current user)").option("--request-id <id>", "Specific request ID", positiveInt).addOption(new Option5("--action <action>", "Access level filter").choices(["view", "approve"])).option("--employee-id <id>", "Filter by employee ID").option("--all", "Show all visible requests instead of just current user").option("--start-date <date>", "Start date (YYYY-MM-DD)").option("--end-date <date>", "End date (YYYY-MM-DD)").addOption(
+    new Option5("--status <status>", "Filter by request status").choices([
       "approved",
       "denied",
       "superceded",
@@ -791,25 +945,29 @@ function types(parent) {
 }
 
 // src/commands/timeoff/update-status.ts
-import { Option as Option5 } from "commander";
+import { Option as Option6 } from "commander";
 function updateStatus(parent) {
   const cmd = parent.command("update-status <requestId>").description("Update time off request status (approve, deny, cancel)").addOption(
-    new Option5("--status <status>", "New status").choices(["approved", "denied", "canceled"]).makeOptionMandatory()
-  ).option("--note <text>", "Note to attach to the status change");
+    new Option6("--status <status>", "New status").choices(["approved", "denied", "canceled"]).makeOptionMandatory()
+  );
+  textOrFileOption(cmd, "note", { description: "Note to attach to the status change" });
   examples(cmd, [
     "7890 --status approved",
     ['7890 --status denied --note "Team at capacity that week"', "deny with note"],
     "7890 --status canceled"
   ]);
-  cmd.action(async (requestId, opts) => {
-    const client = getClient();
-    const result = await client.timeOff.updateRequestStatus({
-      requestId,
-      status: opts.status,
-      note: opts.note
-    });
-    output(result ? stripNullish(result) : { updated: true, requestId, status: opts.status });
-  });
+  cmd.action(
+    async (requestId, opts) => {
+      const client = getClient();
+      const note = resolveTextOrFile(opts, "note", { required: false });
+      const result = await client.timeOff.updateRequestStatus({
+        requestId,
+        status: opts.status,
+        note
+      });
+      output(result ? stripNullish(result) : { updated: true, requestId, status: opts.status });
+    }
+  );
 }
 
 // src/commands/timeoff/whos-out.ts
@@ -831,7 +989,7 @@ function registerTimeoffCommands(program) {
   const timeoff = program.command("timeoff").description("Time off operations");
   examples(timeoff, [
     "types --requestable",
-    "create 504 --start-date 2026-04-01 --end-date 2026-04-01 --type-id 83 --amount 1",
+    "create 504 --start-date 2026-04-01 --end-date 2026-04-01 --type-id 83",
     ["requests --status requested --start-date 2026-01-01 --end-date 2026-12-31", "pending requests in date range"],
     "balance 504",
     "whos-out",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bamboohr-cli",
-  "version": "1.0.19",
+  "version": "1.0.20-gc6d5f1a.1",
   "publish": true,
   "type": "module",
   "main": "dist/index.js",
@@ -12,7 +12,7 @@
   ],
   "dependencies": {
     "commander": "^13.1.0",
-    "bamboohr-client": "1.0.24"
+    "bamboohr-client": "1.0.24-gc6d5f1a.1"
   },
   "devDependencies": {
     "@types/node": "24.10.4",