@shbernal/ts-xlsx 1.1.0 → 1.2.0

package/dist/core/worksheet-model.js

@@ -1,11 +1,3 @@
-// The field table behind `Worksheet.model`'s round-trip.
-//
-// Both directions used to be hand-written statement lists, one in the getter and one in the setter,
-// each enumerating the same nineteen fields. Nothing but a comment asked the next editor to touch
-// both, and a field exported but not imported loses data silently — the merge-loss failure the
-// model contract exists to prevent. Here each field declares both directions in one place, and the
-// registry is proved exhaustive over `keyof WorksheetModel` at compile time, so adding a field
-// without wiring it is an error naming the field rather than a review catch.
 import { cellToModel, copyCellContent } from './cell.js';
 import { cloneConditionalFormatting } from './conditional-formatting.js';
 import { overwrite, replaceContents } from './containers.js';
@@ -14,11 +6,6 @@ import { INTERNAL } from './internal.js';
 function facet(key, read, write) {
     return { key, read, write: (sheet, model) => write(sheet, model[key]) };
 }
-/**
- * Every field of a {@link WorksheetModel}, in the order a model assignment applies them. Order is
- * load-bearing: cells are placed at their exact positions before any merge exists, so a covered
- * cell's value lands where the model says instead of being routed to a region master mid-load.
- */
 export const WORKSHEET_MODEL_FACETS = [
     facet('state', (sheet) => sheet.state, (sheet, value) => {
         sheet.state = value;
@@ -40,8 +27,6 @@ export const WORKSHEET_MODEL_FACETS = [
     }),
     facet('rows', (sheet) => {
         const rows = [];
-        // A row appears here only for its formatting; the cells it holds are the `cells` field's
-        // business, and a row carrying nothing but cells has no properties to round-trip.
         for (const { number, properties } of sheet.rows()) {
             if (properties !== undefined)
                 rows.push({ number, properties: { ...properties } });
@@ -82,16 +67,10 @@ export const WORKSHEET_MODEL_FACETS = [
         for (const options of value)
             sheet.addTable(options);
     }),
-    facet('autoFilter', (sheet) => sheet.autoFilter, 
-    // Through the public setter, which re-canonicalises the range and — on `undefined` — clears any
-    // autofilter the destination held. That clearing is why the field is applied even when absent.
-    (sheet, value) => {
+    facet('autoFilter', (sheet) => sheet.autoFilter, (sheet, value) => {
         sheet.autoFilter = value;
     }),
-    facet('protection', (sheet) => sheet.protection, 
-    // A loaded credential is already hashed, so it is reinstated verbatim rather than re-derived;
-    // `protect` cannot express that, which is why this goes through the internal channel.
-    (sheet, value) => {
+    facet('protection', (sheet) => sheet.protection, (sheet, value) => {
         if (value === undefined)
             sheet.unprotect();
         else