@particle-academy/holy-sheet 2.4.3 → 2.5.0

package/dist/index.d.cts

@@ -223,7 +223,7 @@ interface SetMetaOp {
 }
 
 /** This package's own version, pinned to package.json by `version.test.ts`. */
-declare const VERSION = "2.4.3";
+declare const VERSION = "2.5.0";
 type Any$6 = any;
 /**
  * Agent — the structured-tool surface for Holy Sheet. Mirrors PHP `Agent`.
@@ -295,7 +295,7 @@ type Any$5 = any;
  * over `Agent`. Provided for parity / DI ergonomics; most callers use `Agent`.
  */
 declare class HolySheet {
-    static readonly VERSION = "2.4.3";
+    static readonly VERSION = "2.5.0";
     validate(schema: Any$5): ValidationError[];
     toBytes(schema: Any$5): Uint8Array;
     write(schema: Any$5, path: string): Promise<WriteResult>;
@@ -460,6 +460,22 @@ declare class Normalizer {
     private normalizeSheet;
     private normalizeCellMap;
     private normalizeMerges;
+    /**
+     * 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.
+     */
+    private normalizeWidths;
     private normalizeColumnWidths;
     private columnFormat;
     private buildCell;

package/dist/index.d.ts

@@ -223,7 +223,7 @@ interface SetMetaOp {
 }
 
 /** This package's own version, pinned to package.json by `version.test.ts`. */
-declare const VERSION = "2.4.3";
+declare const VERSION = "2.5.0";
 type Any$6 = any;
 /**
  * Agent — the structured-tool surface for Holy Sheet. Mirrors PHP `Agent`.
@@ -295,7 +295,7 @@ type Any$5 = any;
  * over `Agent`. Provided for parity / DI ergonomics; most callers use `Agent`.
  */
 declare class HolySheet {
-    static readonly VERSION = "2.4.3";
+    static readonly VERSION = "2.5.0";
     validate(schema: Any$5): ValidationError[];
     toBytes(schema: Any$5): Uint8Array;
     write(schema: Any$5, path: string): Promise<WriteResult>;
@@ -460,6 +460,22 @@ declare class Normalizer {
     private normalizeSheet;
     private normalizeCellMap;
     private normalizeMerges;
+    /**
+     * 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.
+     */
+    private normalizeWidths;
     private normalizeColumnWidths;
     private columnFormat;
     private buildCell;

package/dist/index.js

@@ -613,7 +613,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)
     );
@@ -659,6 +659,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)) {
@@ -1660,6 +1690,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;
   }
 };
@@ -5136,7 +5177,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);
 }