@particle-academy/holy-sheet 2.4.3 → 2.5.0

package/README.md

@@ -1,12 +1,12 @@
 # @particle-academy/holy-sheet
 
-[![Fancy UI suite](art/fancy-ui.svg)](https://particle.academy)
+[![Fancified](art/fancified.svg)](https://particle.academy)
 
 Zero-dependency, **isomorphic** (browser + Node) `.xlsx` writer + reader +
 formula linter for agentic document creation. The Node/TypeScript mirror of the
 PHP [`particle-academy/holy-sheet`](https://github.com/Particle-Academy/holy-sheet)
-— same schema in, same `.xlsx` out (feature-parity with PHP 1.2.0; the PHP
-package has since shipped 1.3.0, whose additions are not yet mirrored here).
+— same schema in, same `.xlsx` out (the PHP package's `dumpJson` and `Toolkit`
+are not mirrored here; `CHANGELOG.md` records which PHP releases each fix mirrors).
 
 ```ts
 import { Agent } from "@particle-academy/holy-sheet";

package/dist/index.cjs

@@ -615,7 +615,7 @@ var Normalizer = class {
       name,
       cells,
       this.normalizeMerges(sheet.mergedRegions ?? []),
-      this.normalizeColumnWidths(sheet.columnWidths ?? {}),
+      this.normalizeWidths(sheet),
       toInt(sheet.frozenRows ?? 0),
       toInt(sheet.frozenCols ?? 0)
     );
@@ -661,6 +661,36 @@ var Normalizer = class {
     }
     return out;
   }
+  /**
+   * The two ways a width can be stated, folded into one map.
+   *
+   * `columns[].width` widens the column at that POSITION; the sheet-level
+   * `columnWidths` map keys the same columns by 0-based index. Both have been
+   * described in the shared `skills/holy-sheet.schema.json` ("Same as
+   * columnWidths but per-column") for as long as the field has existed, and only
+   * the map was ever read — so a width written the way the schema documents it
+   * emitted no `<cols>` element at all, with no error and no repair note
+   * (holy-sheet#8, fixed in PHP 2.4.0; this is the twin).
+   *
+   * The MAP IS APPLIED LAST and wins, matching PHP. It is the mechanism that
+   * already worked, so anyone who reached for it to work around this bug must not
+   * now find a leftover `width` quietly overriding it.
+   */
+  normalizeWidths(sheet) {
+    const out = /* @__PURE__ */ new Map();
+    for (const [colIdx, columnDef] of (sheet.columns ?? []).entries()) {
+      if (columnDef === null || typeof columnDef !== "object") continue;
+      if (columnDef.width === void 0) continue;
+      const index = ColumnWidths.index(colIdx);
+      const width = ColumnWidths.width(columnDef.width);
+      if (index === null || width === null) continue;
+      out.set(index, width);
+    }
+    for (const [index, width] of this.normalizeColumnWidths(sheet.columnWidths ?? {})) {
+      out.set(index, width);
+    }
+    return new Map([...out.entries()].sort((a, b) => a[0] - b[0]));
+  }
   normalizeColumnWidths(widths) {
     const out = /* @__PURE__ */ new Map();
     for (const [key, px] of Object.entries(widths)) {
@@ -1662,6 +1692,17 @@ var Validator = class {
         )
       );
     }
+    if (col.width !== void 0 && ColumnWidths.width(col.width) === null) {
+      errors.push(
+        error(
+          `${path}.width`,
+          "a non-negative number of pixels",
+          typeOf(col.width),
+          col.width,
+          "A column width is a number of pixels, like 120. The sheet-level columnWidths map takes precedence over it."
+        )
+      );
+    }
     return errors;
   }
 };
@@ -5138,7 +5179,7 @@ var holy_sheet_schema_default = {
 };
 
 // src/agent.ts
-var VERSION = "2.4.3";
+var VERSION = "2.5.0";
 function toU8(input) {
   return input instanceof Uint8Array ? input : new Uint8Array(input);
 }