@forwardimpact/basecamp 2.9.2 → 2.9.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forwardimpact/basecamp",
-  "version": "2.9.2",
+  "version": "2.9.3",
   "description": "Claude Code-native personal knowledge system with autonomous agents",
   "license": "Apache-2.0",
   "repository": {
@@ -30,7 +30,7 @@
     "node": ">=18.0.0"
   },
   "dependencies": {
-    "xlsx": "^0.18.5"
+    "read-excel-file": "^7.0.3"
   },
   "publishConfig": {
     "access": "public"

package/template/.claude/skills/workday-requisition/SKILL.md

@@ -28,9 +28,9 @@ Run this skill:
 
 - A Workday requisition export file (`.xlsx`) accessible on the filesystem
   (typically in `~/Downloads/`)
-- The `xlsx` npm package installed in the KB root:
+- The `read-excel-file` npm package installed in the KB root:
   ```bash
-  npm install xlsx
+  npm install read-excel-file
   ```
 - User identity configured in `USER.md`
 
@@ -135,9 +135,9 @@ Names may include parenthetical annotations:
 
 1. Read `USER.md` to get the user's name, email, and domain.
 2. Confirm the XLSX file path with the user (or use the provided path).
-3. Ensure the `xlsx` package is installed:
+3. Ensure the `read-excel-file` package is installed:
    ```bash
-   npm list xlsx 2>/dev/null || npm install xlsx
+   npm list read-excel-file 2>/dev/null || npm install read-excel-file
    ```
 
 ## Step 1: Parse the Export

package/template/.claude/skills/workday-requisition/scripts/parse-workday.mjs

@@ -19,11 +19,9 @@
  *   node scripts/parse-workday.mjs <path-to-xlsx> --summary
  *   node scripts/parse-workday.mjs -h|--help
  *
- * Requires: npm install xlsx
+ * Requires: npm install read-excel-file
  */
 
-import { readFileSync } from "node:fs";
-
 if (
   process.argv.includes("-h") ||
   process.argv.includes("--help") ||
@@ -39,16 +37,16 @@ Usage:
 Output (JSON):
   { requisition: { id, title, ... }, candidates: [ { name, ... }, ... ] }
 
-Requires: npm install xlsx`);
+Requires: npm install read-excel-file`);
   process.exit(process.argv.length < 3 ? 1 : 0);
 }
 
-let XLSX;
+let readXlsxFile;
 try {
-  XLSX = await import("xlsx");
+  readXlsxFile = (await import("read-excel-file/node")).default;
 } catch {
   console.error(
-    "Error: xlsx package not found. Install it first:\n  npm install xlsx",
+    "Error: read-excel-file package not found. Install it first:\n  npm install read-excel-file",
   );
   process.exit(1);
 }
@@ -56,13 +54,20 @@ try {
 const filePath = process.argv[2];
 const summaryMode = process.argv.includes("--summary");
 
-const data = readFileSync(filePath);
-const wb = XLSX.read(data, { type: "buffer", cellDates: true });
+/** Read a sheet by number (1-indexed) or name, returning rows as arrays of strings. */
+async function readSheet(file, sheet) {
+  const rows = await readXlsxFile(file, { sheet });
+  // Normalise null cells to empty strings to match previous behaviour
+  return rows.map((row) => row.map((cell) => (cell == null ? "" : cell)));
+}
+
+// Get sheet names to find the candidates sheet
+const sheets = await readXlsxFile(filePath, { getSheets: true });
+const sheetNames = sheets.map((s) => s.name);
 
 // --- Sheet 1: Requisition metadata ---
 
-const ws1 = wb.Sheets[wb.SheetNames[0]];
-const sheet1Rows = XLSX.utils.sheet_to_json(ws1, { header: 1, defval: "" });
+const sheet1Rows = await readSheet(filePath, 1);
 
 /** Extract the requisition ID and title from the header row. */
 function parseReqHeader(headerText) {
@@ -113,10 +118,9 @@ const requisition = {
 //   - Old format: 3+ sheets, candidates on a sheet named "Candidates" or Sheet3
 //   - New format: 2 sheets, candidates on Sheet2
 const candSheetName =
-  wb.SheetNames.find((n) => n.toLowerCase() === "candidates") ||
-  wb.SheetNames[Math.min(2, wb.SheetNames.length - 1)];
-const ws3 = wb.Sheets[candSheetName];
-const candRows = XLSX.utils.sheet_to_json(ws3, { header: 1, defval: "" });
+  sheetNames.find((n) => n.toLowerCase() === "candidates") ||
+  sheetNames[Math.min(2, sheetNames.length - 1)];
+const candRows = await readSheet(filePath, candSheetName);
 
 // Find the header row dynamically — look for a row containing "Stage"
 // Old format: row 3 (index 2). New format: row 8 (index 7).
@@ -199,7 +203,7 @@ function parseName(raw) {
   const match = name.match(/^(.+?)\s*\(([^)]+)\)\s*$/);
   if (match) {
     const annotation = match[2].trim();
-    let ie = "";
+    let ie;
     if (/prior\s*worker/i.test(annotation)) ie = "External (Prior Worker)";
     else if (/internal/i.test(annotation)) ie = "Internal";
     else ie = annotation;