@cj-tech-master/excelts 9.4.2 → 9.5.0

package/dist/browser/modules/excel/utils/ooxml-validator/check-chart-sidecar.d.ts

@@ -0,0 +1,35 @@
+/**
+ * ChartEx sidecar completeness check.
+ *
+ * Every ChartEx `xl/charts/chartExN.xml` MUST ship with a chartStyle
+ * sidecar (`xl/charts/styleN.xml` or `xl/charts/styleExN.xml`) and a
+ * chartColorStyle sidecar (`xl/charts/colorsN.xml` or
+ * `xl/charts/colorsExN.xml`), linked from `chartExN.xml.rels`. Without
+ * them Excel 2016+ discards the chartEx on load ("Removed Part:
+ * /xl/drawings/drawingN.xml (Drawing shape)").
+ *
+ * Both sidecars are STUB-REJECTED **for chartEx** — an id-only
+ * skeleton such as `<cs:chartStyle id="395"/>` (no child elements)
+ * makes Excel treat the sidecar as malformed and drop the parent
+ * chartEx + drawing. Excel authoring tools emit the FULL content: ~40
+ * child elements for chartStyle (axisTitle, dataPoint, dataLabel, …)
+ * and 6 `schemeClr` entries each with 9 `variation` siblings for
+ * chartColorStyle.
+ *
+ * CLASSIC charts (`xl/charts/chartN.xml`) can share the chartStyle
+ * namespace but Excel is lenient there — classic chart style stubs
+ * are accepted. We therefore only enforce the no-stub rule on
+ * sidecars **referenced by a `chartExN.xml` part** via its rels.
+ *
+ * The precise content is too intricate to validate without baking
+ * ~1500 lines of template XML into this module. Instead we detect the
+ * two high-signal stub patterns:
+ *
+ *   - chartStyle root element has zero element children (pure stub).
+ *   - chartColorStyle root element has zero element children (pure stub).
+ *
+ * A rich sidecar trivially has dozens of children so the zero-child
+ * case is a strong writer-bug signature.
+ */
+import { type ValidationContext } from "./context.js";
+export declare function checkChartSidecars(ctx: ValidationContext): void;

package/dist/browser/modules/excel/utils/ooxml-validator/check-chart-sidecar.js

@@ -0,0 +1,101 @@
+/**
+ * ChartEx sidecar completeness check.
+ *
+ * Every ChartEx `xl/charts/chartExN.xml` MUST ship with a chartStyle
+ * sidecar (`xl/charts/styleN.xml` or `xl/charts/styleExN.xml`) and a
+ * chartColorStyle sidecar (`xl/charts/colorsN.xml` or
+ * `xl/charts/colorsExN.xml`), linked from `chartExN.xml.rels`. Without
+ * them Excel 2016+ discards the chartEx on load ("Removed Part:
+ * /xl/drawings/drawingN.xml (Drawing shape)").
+ *
+ * Both sidecars are STUB-REJECTED **for chartEx** — an id-only
+ * skeleton such as `<cs:chartStyle id="395"/>` (no child elements)
+ * makes Excel treat the sidecar as malformed and drop the parent
+ * chartEx + drawing. Excel authoring tools emit the FULL content: ~40
+ * child elements for chartStyle (axisTitle, dataPoint, dataLabel, …)
+ * and 6 `schemeClr` entries each with 9 `variation` siblings for
+ * chartColorStyle.
+ *
+ * CLASSIC charts (`xl/charts/chartN.xml`) can share the chartStyle
+ * namespace but Excel is lenient there — classic chart style stubs
+ * are accepted. We therefore only enforce the no-stub rule on
+ * sidecars **referenced by a `chartExN.xml` part** via its rels.
+ *
+ * The precise content is too intricate to validate without baking
+ * ~1500 lines of template XML into this module. Instead we detect the
+ * two high-signal stub patterns:
+ *
+ *   - chartStyle root element has zero element children (pure stub).
+ *   - chartColorStyle root element has zero element children (pure stub).
+ *
+ * A rich sidecar trivially has dozens of children so the zero-child
+ * case is a strong writer-bug signature.
+ */
+import {} from "./context.js";
+import { resolveRelTarget } from "./path-utils.js";
+const CHARTEX_PART_RE = /^xl\/charts\/chartEx\d+\.xml$/;
+const CHARTSTYLE_REL_TYPE = "http://schemas.microsoft.com/office/2011/relationships/chartStyle";
+const CHARTCOLORS_REL_TYPE = "http://schemas.microsoft.com/office/2011/relationships/chartColorStyle";
+export function checkChartSidecars(ctx) {
+    // Phase 1: collect all sidecar paths that are referenced from a
+    // chartEx part. Only these warrant the "stub is rejection" rule.
+    const chartExStylePaths = new Set();
+    const chartExColorsPaths = new Set();
+    for (const [path, entry] of ctx.files()) {
+        if (entry.type === "directory" || !CHARTEX_PART_RE.test(path)) {
+            continue;
+        }
+        const relsPath = chartRelsPath(path);
+        if (!ctx.has(relsPath)) {
+            continue;
+        }
+        const rels = ctx.readRels(relsPath);
+        for (const rel of rels.rels) {
+            if (rel.targetMode === "External") {
+                continue;
+            }
+            const resolved = resolveRelTarget(relsPath, rel.target);
+            if (rel.type === CHARTSTYLE_REL_TYPE) {
+                chartExStylePaths.add(resolved);
+            }
+            else if (rel.type === CHARTCOLORS_REL_TYPE) {
+                chartExColorsPaths.add(resolved);
+            }
+        }
+    }
+    // Phase 2: for every sidecar referenced by a chartEx, flag stub form.
+    for (const path of chartExStylePaths) {
+        if (ctx.reporter.capped) {
+            return;
+        }
+        checkStubRoot(ctx, path, "chartEx-chartStyle-stub-form", "chartStyle");
+    }
+    for (const path of chartExColorsPaths) {
+        if (ctx.reporter.capped) {
+            return;
+        }
+        checkStubRoot(ctx, path, "chartEx-chartColorStyle-stub-form", "chartColorStyle");
+    }
+}
+function chartRelsPath(chartPath) {
+    const slash = chartPath.lastIndexOf("/");
+    const dir = slash >= 0 ? chartPath.slice(0, slash) : "";
+    const name = slash >= 0 ? chartPath.slice(slash + 1) : chartPath;
+    return dir ? `${dir}/_rels/${name}.rels` : `_rels/${name}.rels`;
+}
+function checkStubRoot(ctx, path, kind, label) {
+    if (!ctx.has(path)) {
+        return; // missing-target already reported by the rels checker.
+    }
+    const dom = ctx.readDom(path);
+    if (!dom) {
+        return;
+    }
+    const root = dom.root;
+    const hasElementChildren = root.children.some(c => c.type === "element");
+    if (!hasElementChildren) {
+        ctx.reporter.error(kind, `${path}: ${label} sidecar referenced by a chartEx part is an id-only stub ` +
+            `(root element has no children). Excel 2016+ rejects chartEx stub sidecars and ` +
+            `drops the parent chartEx + drawing. Emit the full Microsoft-authored default content.`, path);
+    }
+}

package/dist/browser/modules/excel/utils/ooxml-validator/check-chart.d.ts

@@ -0,0 +1,32 @@
+/**
+ * Chart / chartEx internal structure check.
+ *
+ * Classic charts (`xl/charts/chartN.xml`) must contain both `c:chart` and
+ * `c:plotArea`. ChartEx charts must contain `cx:chart`, `cx:plotArea`
+ * and at least one `cx:series`; series must carry `layoutId` and their
+ * `dataId`/`axisId` back-references must resolve inside the chart.
+ * `cx:externalData` nodes must refer to a declared relationship.
+ *
+ * In addition we catch five chartEx schema-violation patterns that
+ * cause Excel 2016+ to drop the whole chartEx part with "Removed Part:
+ * /xl/drawings/drawingN.xml (Drawing shape)":
+ *
+ *   - `<cx:series>` with more than one `<cx:dataId>` child
+ *     (`CT_Series/dataId` has `maxOccurs="1"`; multi-dim series must
+ *     use a single `<cx:data>` wrapper with multiple strDim/numDim).
+ *   - `<cx:axisId>N</cx:axisId>`, `<cx:dataId>N</cx:dataId>`,
+ *     `<cx:binCount>N</cx:binCount>`, `<cx:binSize>N</cx:binSize>`
+ *     emitted as text content instead of `val="N"` attribute. The
+ *     underlying types are `CT_UnsignedInteger`/`CT_Double`, which
+ *     Excel's strict loader only accepts via the attribute form.
+ *   - `<cx:auto/>` element anywhere in the chartEx. Auto binning is
+ *     expressed by the ABSENCE of `binSize`/`binCount`, not a
+ *     dedicated `<cx:auto/>` tag. The tag is schema-invalid.
+ *   - `<cx:paretoLine>` child of `<cx:layoutPr>`. Not in the
+ *     CT_SeriesLayoutProperties schema. A real pareto chart adds a
+ *     second series with `layoutId="paretoLine"`.
+ *   - `<cx:title>` with a direct `<cx:layout>` child. Title layout
+ *     belongs in `extLst`-based extensions.
+ */
+import type { ValidationContext } from "./context.js";
+export declare function checkChart(ctx: ValidationContext): void;