@pagefront/lint-commerce 0.1.0

package/dist/rules/pdp-w-005.d.ts

@@ -0,0 +1,18 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * PDP-W-005 — Material composition percentages don't sum to 100.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * The catalogue targets both `pagefront:materialComposition` and its
+ * textile alias `pagefront:fabricComposition`; a Rule carries one
+ * JSONPath, expressed here as an RFC 9535 multi-selector bracket. Each
+ * match is one composition array. Per the catalogue, the rule does not
+ * fire if any entry omits `percentage`; an empty array is read as having
+ * no percentage-declaring entries and does not fire (the catalogue's
+ * "all entries declare" would be vacuously true on it, but a sum of 0
+ * on zero materials is not the transcription-error signal this rule
+ * exists for). Per the catalogue, `{sum}` is rendered with at most two
+ * decimal places, so floating-point summation noise never reaches the
+ * report.
+ */
+export declare const pdpW005: Rule;

package/dist/rules/pdp-w-005.js

@@ -0,0 +1,44 @@
+/**
+ * PDP-W-005 — Material composition percentages don't sum to 100.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * The catalogue targets both `pagefront:materialComposition` and its
+ * textile alias `pagefront:fabricComposition`; a Rule carries one
+ * JSONPath, expressed here as an RFC 9535 multi-selector bracket. Each
+ * match is one composition array. Per the catalogue, the rule does not
+ * fire if any entry omits `percentage`; an empty array is read as having
+ * no percentage-declaring entries and does not fire (the catalogue's
+ * "all entries declare" would be vacuously true on it, but a sum of 0
+ * on zero materials is not the transcription-error signal this rule
+ * exists for). Per the catalogue, `{sum}` is rendered with at most two
+ * decimal places, so floating-point summation noise never reaches the
+ * report.
+ */
+export const pdpW005 = {
+    id: "PDP-W-005",
+    title: "Material composition percentages don't sum to 100",
+    severity: "warning",
+    target: "$.data['pagefront:materialComposition','pagefront:fabricComposition']",
+    check: (match) => {
+        const entries = match.value;
+        if (!Array.isArray(entries) || entries.length === 0)
+            return null;
+        const percentages = [];
+        for (const entry of entries) {
+            const p = typeof entry === "object" && entry !== null
+                ? entry["percentage"]
+                : undefined;
+            if (typeof p !== "number")
+                return null;
+            percentages.push(p);
+        }
+        const sum = percentages.reduce((a, b) => a + b, 0);
+        return sum >= 99.5 && sum <= 100.5
+            ? null
+            : { values: { sum: Math.round(sum * 100) / 100 } };
+    },
+    messageTemplate: "Material composition percentages at `{path}` sum to `{sum}`, not approximately 100.",
+    remediation: "Verify the composition data. If a material is missing, add it. If the values are correct but the publisher's source data uses percentages for a different denominator, omit `percentage` rather than use a value that doesn't represent a fraction of total composition.",
+    introduced: "v0.6",
+    specReference: "product.md#materials-material-and-pagefrontmaterialcomposition",
+};

package/dist/rules/pdp-w-006.d.ts

@@ -0,0 +1,10 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * PDP-W-006 — Certification missing scope.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * Bespoke check: fires when the certification's `additionalProperty`
+ * array has no entry with `name: "scope"`. `{name}` is the
+ * certification's `name`, which the schema requires on every entry.
+ */
+export declare const pdpW006: Rule;

package/dist/rules/pdp-w-006.js

@@ -0,0 +1,31 @@
+/**
+ * PDP-W-006 — Certification missing scope.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * Bespoke check: fires when the certification's `additionalProperty`
+ * array has no entry with `name: "scope"`. `{name}` is the
+ * certification's `name`, which the schema requires on every entry.
+ */
+export const pdpW006 = {
+    id: "PDP-W-006",
+    title: "Certification missing scope",
+    severity: "warning",
+    target: "$.data.hasCertification[*]",
+    check: (match) => {
+        const cert = match.value;
+        if (typeof cert !== "object" || cert === null)
+            return null;
+        const c = cert;
+        const props = Array.isArray(c["additionalProperty"])
+            ? c["additionalProperty"]
+            : [];
+        const hasScope = props.some((p) => typeof p === "object" &&
+            p !== null &&
+            p["name"] === "scope");
+        return hasScope ? null : { values: { name: String(c["name"]) } };
+    },
+    messageTemplate: "Certification `{name}` at `{path}` has no scope declared.",
+    remediation: "Add a `PropertyValue` to `additionalProperty` with name `scope` and a value drawn from the conventional vocabulary (`yarn`, `fabric`, `finished_product`, `diamond`, `gemstone`, etc.).",
+    introduced: "v0.6",
+    specReference: "vocabulary.md#certification-scope",
+};

package/dist/rules/pdp-w-007.d.ts

@@ -0,0 +1,9 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * PDP-W-007 — ProductGroup with single variant.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * Bespoke cross-field check: `@type` is `ProductGroup` and `hasVariant`
+ * has exactly one entry.
+ */
+export declare const pdpW007: Rule;

package/dist/rules/pdp-w-007.js

@@ -0,0 +1,29 @@
+/**
+ * PDP-W-007 — ProductGroup with single variant.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * Bespoke cross-field check: `@type` is `ProductGroup` and `hasVariant`
+ * has exactly one entry.
+ */
+export const pdpW007 = {
+    id: "PDP-W-007",
+    title: "ProductGroup with single variant",
+    severity: "warning",
+    target: "$.data",
+    check: (match) => {
+        const data = match.value;
+        if (typeof data !== "object" || data === null)
+            return null;
+        const d = data;
+        const variants = d["hasVariant"];
+        return d["@type"] === "ProductGroup" &&
+            Array.isArray(variants) &&
+            variants.length === 1
+            ? {}
+            : null;
+    },
+    messageTemplate: "ProductGroup at `{path}` has only one variant.",
+    remediation: "Either add the missing variants to `hasVariant`, or change `@type` to `Product` and model the single variant directly. The `productGroupID` field should be replaced with `productID` or `sku` if switching to Product.",
+    introduced: "v0.6",
+    specReference: "product.md#configurator-vs-productgroup-guidance",
+};

package/dist/rules/pdp-w-008.d.ts

@@ -0,0 +1,19 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * PDP-W-008 — Sheet expires_at is past or trivially close to generated_at.
+ * Transcribed from commerce/spec/rules.md (normative), with two
+ * reductions the catalogue text leaves open:
+ *
+ * - The catalogue names both `$.expires_at` and `$.generated_at` as the
+ *   target; a Rule carries one JSONPath. The target here is
+ *   `$.expires_at` — the rule cannot fire without it (the field is
+ *   optional) — and `generated_at` is read via RuleContext.
+ * - The catalogue message's `{past | trivially close to generated_at}`
+ *   prose alternation is realized as the `{reason}` placeholder carrying
+ *   one of those two literal texts; when both conditions hold, `past`
+ *   wins (the catalogue's order).
+ *
+ * "The time of linting" is the run's `checked_at` (ctx.checkedAt), not
+ * the wall clock, so fixture output stays deterministic.
+ */
+export declare const pdpW008: Rule;

package/dist/rules/pdp-w-008.js

@@ -0,0 +1,43 @@
+/**
+ * PDP-W-008 — Sheet expires_at is past or trivially close to generated_at.
+ * Transcribed from commerce/spec/rules.md (normative), with two
+ * reductions the catalogue text leaves open:
+ *
+ * - The catalogue names both `$.expires_at` and `$.generated_at` as the
+ *   target; a Rule carries one JSONPath. The target here is
+ *   `$.expires_at` — the rule cannot fire without it (the field is
+ *   optional) — and `generated_at` is read via RuleContext.
+ * - The catalogue message's `{past | trivially close to generated_at}`
+ *   prose alternation is realized as the `{reason}` placeholder carrying
+ *   one of those two literal texts; when both conditions hold, `past`
+ *   wins (the catalogue's order).
+ *
+ * "The time of linting" is the run's `checked_at` (ctx.checkedAt), not
+ * the wall clock, so fixture output stays deterministic.
+ */
+export const pdpW008 = {
+    id: "PDP-W-008",
+    title: "Sheet expires_at is past or trivially close to generated_at",
+    severity: "warning",
+    target: "$.expires_at",
+    check: (match, ctx) => {
+        const expiresAt = Date.parse(String(match.value));
+        if (Number.isNaN(expiresAt))
+            return null;
+        const generatedAt = Date.parse(String(ctx.sheet.generated_at));
+        const past = expiresAt < Date.parse(ctx.checkedAt);
+        const close = !Number.isNaN(generatedAt) && expiresAt - generatedAt < 60_000;
+        if (!past && !close)
+            return null;
+        return {
+            values: {
+                expires_at: String(match.value),
+                reason: past ? "past" : "trivially close to generated_at",
+            },
+        };
+    },
+    messageTemplate: "Sheet `expires_at` is `{expires_at}`, which is {reason}.",
+    remediation: "Set `expires_at` to a sensible window after `generated_at`. For static product catalogs, hours to days; for sheets with live pricing or inventory signals, minutes.",
+    introduced: "v0.6",
+    specReference: "product.md#the-envelope",
+};

package/dist/rules/pdp-w-009.d.ts

@@ -0,0 +1,9 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * PDP-W-009 — Empty hasVariant.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * Bespoke cross-field check: `@type` is `ProductGroup` and `hasVariant`
+ * is absent or an empty array.
+ */
+export declare const pdpW009: Rule;

package/dist/rules/pdp-w-009.js

@@ -0,0 +1,29 @@
+/**
+ * PDP-W-009 — Empty hasVariant.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * Bespoke cross-field check: `@type` is `ProductGroup` and `hasVariant`
+ * is absent or an empty array.
+ */
+export const pdpW009 = {
+    id: "PDP-W-009",
+    title: "Empty hasVariant",
+    severity: "warning",
+    target: "$.data",
+    check: (match) => {
+        const data = match.value;
+        if (typeof data !== "object" || data === null)
+            return null;
+        const d = data;
+        const variants = d["hasVariant"];
+        return d["@type"] === "ProductGroup" &&
+            (variants === undefined ||
+                (Array.isArray(variants) && variants.length === 0))
+            ? {}
+            : null;
+    },
+    messageTemplate: "ProductGroup at `{path}` has no variants.",
+    remediation: "Add variant entries to `hasVariant`. If the product genuinely has no variants, change `@type` to `Product`.",
+    introduced: "v0.6",
+    specReference: "product.md#configurator-vs-productgroup-guidance",
+};

package/dist/rules/pdp-w-010.d.ts

@@ -0,0 +1,24 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * PDP-W-010 — Configurator and isVariantOf both populated for same dimension.
+ * Transcribed from commerce/spec/rules.md (normative), with two
+ * reductions:
+ *
+ * - The catalogue's target is `$.data`, but its message is per colliding
+ *   dimension (`{dimension}`, `{configurator_path}`, `{variant_path}`),
+ *   and a check yields at most one finding per match. The target here is
+ *   each configurator dimension, so every colliding dimension produces
+ *   its own finding — the per-dimension reading the message implies.
+ * - "A property declared on a variant" is a direct property of the
+ *   variant entry, or an `additionalProperty` entry whose `propertyID`
+ *   matches the dimension — or, when the entry carries no `propertyID`,
+ *   whose `name` matches (lenient fallback for content predating the
+ *   convention; `propertyID` is the language-neutral coordination key,
+ *   `name` the possibly-localized display label). An entry whose
+ *   `propertyID` names a *different* dimension never falls back to
+ *   `name`.
+ *
+ * `{variant_path}` is the first matching variant entry's normalized
+ * path, built in the engine's path style.
+ */
+export declare const pdpW010: Rule;

package/dist/rules/pdp-w-010.js

@@ -0,0 +1,85 @@
+/**
+ * PDP-W-010 — Configurator and isVariantOf both populated for same dimension.
+ * Transcribed from commerce/spec/rules.md (normative), with two
+ * reductions:
+ *
+ * - The catalogue's target is `$.data`, but its message is per colliding
+ *   dimension (`{dimension}`, `{configurator_path}`, `{variant_path}`),
+ *   and a check yields at most one finding per match. The target here is
+ *   each configurator dimension, so every colliding dimension produces
+ *   its own finding — the per-dimension reading the message implies.
+ * - "A property declared on a variant" is a direct property of the
+ *   variant entry, or an `additionalProperty` entry whose `propertyID`
+ *   matches the dimension — or, when the entry carries no `propertyID`,
+ *   whose `name` matches (lenient fallback for content predating the
+ *   convention; `propertyID` is the language-neutral coordination key,
+ *   `name` the possibly-localized display label). An entry whose
+ *   `propertyID` names a *different* dimension never falls back to
+ *   `name`.
+ *
+ * `{variant_path}` is the first matching variant entry's normalized
+ * path, built in the engine's path style.
+ */
+export const pdpW010 = {
+    id: "PDP-W-010",
+    title: "Configurator and isVariantOf both populated for same dimension",
+    severity: "warning",
+    target: "$.data['pagefront:configurator'].dimensions[*]",
+    check: (match, ctx) => {
+        const dim = match.value;
+        if (typeof dim !== "object" || dim === null)
+            return null;
+        const name = dim["name"];
+        if (typeof name !== "string")
+            return null;
+        const data = typeof ctx.sheet.data === "object" && ctx.sheet.data !== null
+            ? ctx.sheet.data
+            : {};
+        let variants;
+        let variantBase;
+        if (data["@type"] === "ProductGroup") {
+            variants = data["hasVariant"];
+            variantBase = "$['data']['hasVariant']";
+        }
+        else {
+            const parent = data["isVariantOf"];
+            variants =
+                typeof parent === "object" && parent !== null
+                    ? parent["hasVariant"]
+                    : undefined;
+            variantBase = "$['data']['isVariantOf']['hasVariant']";
+        }
+        if (!Array.isArray(variants) || variants.length === 0)
+            return null;
+        for (let i = 0; i < variants.length; i++) {
+            const variant = variants[i];
+            if (typeof variant !== "object" || variant === null)
+                continue;
+            const v = variant;
+            const direct = v[name] !== undefined;
+            const viaProps = Array.isArray(v["additionalProperty"]) &&
+                v["additionalProperty"].some((p) => {
+                    if (typeof p !== "object" || p === null)
+                        return false;
+                    const prop = p;
+                    return prop["propertyID"] !== undefined
+                        ? prop["propertyID"] === name
+                        : prop["name"] === name;
+                });
+            if (direct || viaProps) {
+                return {
+                    values: {
+                        dimension: name,
+                        configurator_path: match.path,
+                        variant_path: `${variantBase}[${i}]`,
+                    },
+                };
+            }
+        }
+        return null;
+    },
+    messageTemplate: "Dimension `{dimension}` is declared in both `pagefront:configurator` at `{configurator_path}` and as a variant property at `{variant_path}`.",
+    remediation: "Choose one representation. Use the configurator for combinatorial spaces, made-to-order products, or continuous dimensions. Use `ProductGroup` + `hasVariant` for discrete SKU families that ship from inventory.",
+    introduced: "v0.6",
+    specReference: "product.md#configurator-vs-productgroup-guidance",
+};

package/dist/rules/pdp-w-011.d.ts

@@ -0,0 +1,2 @@
+import type { Rule } from "@pagefront/lint-core";
+export declare const pdpW011: Rule;

package/dist/rules/pdp-w-011.js

@@ -0,0 +1,44 @@
+/**
+ * PDP-W-011 — Multi-language reference without availableLanguages.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * The catalogue's "recognizable language segment (`/fr/`, `/en/`,
+ * `.fr.`, `.en.`, etc.)" is realized as: a path segment that is a
+ * two-letter lowercase code optionally followed by a region subtag
+ * (`/fr/`, `/en-GB/`), or a two-letter lowercase dotted label
+ * (`.fr.`). Both the canonical URL and the Sheet's `sheet_url` are
+ * inspected — the dotted-label form covers language-suffixed Sheet
+ * names such as `….en.json` (pass #6, from the case-study run). The
+ * catalogue declares the check heuristic and false positives
+ * acceptable. "Does not declare" covers both an absent
+ * `pagefront:availableLanguages` and an empty array (an empty
+ * declaration declares nothing).
+ */
+const LANGUAGE_SEGMENT = /\/[a-z]{2}(?:-[A-Za-z]{2,4})?\/|\.[a-z]{2}\./;
+export const pdpW011 = {
+    id: "PDP-W-011",
+    title: "Multi-language reference without availableLanguages",
+    severity: "warning",
+    target: "$.data",
+    check: (match, ctx) => {
+        const canonicalUrl = ctx.sheet.canonical_url;
+        if (typeof canonicalUrl !== "string")
+            return null;
+        const sheetUrl = ctx.sheet.sheet_url;
+        const signal = LANGUAGE_SEGMENT.test(canonicalUrl) ||
+            (typeof sheetUrl === "string" && LANGUAGE_SEGMENT.test(sheetUrl));
+        if (!signal)
+            return null;
+        const data = match.value;
+        const declared = typeof data === "object" && data !== null
+            ? data["pagefront:availableLanguages"]
+            : undefined;
+        if (Array.isArray(declared) && declared.length > 0)
+            return null;
+        return { values: { canonical_url: canonicalUrl } };
+    },
+    messageTemplate: "Sheet at `{canonical_url}` appears to be part of a multi-language site but does not declare `pagefront:availableLanguages`.",
+    remediation: "Add a `pagefront:availableLanguages` array listing the URLs of sibling sheets in other languages, with their BCP 47 language tags.",
+    introduced: "v0.6",
+    specReference: "product.md#internationalization",
+};

package/dist/rules/pdp-w-012.d.ts

@@ -0,0 +1,18 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * PDP-W-012 — Video missing accessibility metadata.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * Severity is context-dependent: warning for videos with role
+ * `overview` or `tutorial`, info for other roles — realized as the
+ * CheckHit severity override, with the rule's declared severity staying
+ * the catalogue's stronger default. Two readings:
+ *
+ * - The check is the literal "has neither `caption` nor `transcript`"
+ *   (key presence); a present-but-empty caption does not fire.
+ * - The video's role lives in `additionalProperty` (`name: "role"`),
+ *   mirroring the image-role pattern. A video with no declared role is
+ *   not overview/tutorial, so it downgrades to info, and `{role}`
+ *   renders as `none`.
+ */
+export declare const pdpW012: Rule;

package/dist/rules/pdp-w-012.js

@@ -0,0 +1,51 @@
+/**
+ * PDP-W-012 — Video missing accessibility metadata.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * Severity is context-dependent: warning for videos with role
+ * `overview` or `tutorial`, info for other roles — realized as the
+ * CheckHit severity override, with the rule's declared severity staying
+ * the catalogue's stronger default. Two readings:
+ *
+ * - The check is the literal "has neither `caption` nor `transcript`"
+ *   (key presence); a present-but-empty caption does not fire.
+ * - The video's role lives in `additionalProperty` (`name: "role"`),
+ *   mirroring the image-role pattern. A video with no declared role is
+ *   not overview/tutorial, so it downgrades to info, and `{role}`
+ *   renders as `none`.
+ */
+export const pdpW012 = {
+    id: "PDP-W-012",
+    title: "Video missing accessibility metadata",
+    severity: "warning",
+    target: "$.data.video[*]",
+    check: (match) => {
+        const video = match.value;
+        if (typeof video !== "object" || video === null)
+            return null;
+        const v = video;
+        if (v["caption"] !== undefined || v["transcript"] !== undefined)
+            return null;
+        let role = "none";
+        if (Array.isArray(v["additionalProperty"])) {
+            for (const p of v["additionalProperty"]) {
+                if (typeof p === "object" &&
+                    p !== null &&
+                    p["name"] === "role" &&
+                    typeof p["value"] === "string") {
+                    role = p["value"];
+                    break;
+                }
+            }
+        }
+        const isPrimaryRole = role === "overview" || role === "tutorial";
+        return {
+            values: { role },
+            ...(isPrimaryRole ? {} : { severity: "info" }),
+        };
+    },
+    messageTemplate: "Video at `{path}` with role `{role}` has no caption or transcript.",
+    remediation: "Add a `caption` field with a 1–3 sentence summary of the video's content, or a `transcript` field with the full spoken content. For overview and tutorial videos, at least one is strongly recommended.",
+    introduced: "v0.6",
+    specReference: "product.md#accessibility",
+};

package/dist/rules/pdp-w-013.d.ts

@@ -0,0 +1,13 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * PDP-W-013 — Legacy HTTP Schema.org URI.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * The catalogue targets "`$..availability`, `$..itemCondition`, and any
+ * other Schema.org URI fields" — an open-ended set. Reduced here to
+ * `$..*` with a string check: any string value anywhere in the sheet
+ * matching `http://schema.org/...` fires, which covers every
+ * "Schema.org URI field" by definition of carrying such a value, and
+ * cannot fire on anything else.
+ */
+export declare const pdpW013: Rule;

package/dist/rules/pdp-w-013.js

@@ -0,0 +1,25 @@
+/**
+ * PDP-W-013 — Legacy HTTP Schema.org URI.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * The catalogue targets "`$..availability`, `$..itemCondition`, and any
+ * other Schema.org URI fields" — an open-ended set. Reduced here to
+ * `$..*` with a string check: any string value anywhere in the sheet
+ * matching `http://schema.org/...` fires, which covers every
+ * "Schema.org URI field" by definition of carrying such a value, and
+ * cannot fire on anything else.
+ */
+export const pdpW013 = {
+    id: "PDP-W-013",
+    title: "Legacy HTTP Schema.org URI",
+    severity: "warning",
+    target: "$..*",
+    check: (match) => typeof match.value === "string" &&
+        match.value.startsWith("http://schema.org/")
+        ? { values: { value: match.value } }
+        : null,
+    messageTemplate: "Legacy HTTP Schema.org URI `{value}` at `{path}`.",
+    remediation: "Change `http://schema.org/...` to `https://schema.org/...`. The value is otherwise unchanged.",
+    introduced: "v0.6",
+    specReference: "product.md#offers",
+};

package/dist/rules/pdp-w-014.d.ts

@@ -0,0 +1,26 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * PDP-W-014 — ProductGroup variant URL absent on distinctly-addressable
+ * variants. Transcribed from commerce/spec/rules.md (normative), with
+ * two reductions:
+ *
+ * - The catalogue's target is `$.data.hasVariant[*]`, but the check and
+ *   message are group-level ("ProductGroup at `{path}` has `{count}`
+ *   variants…", firing when *none* of the entries declare a `url`). The
+ *   target here is `$.data`, producing the one group-level finding the
+ *   message describes.
+ * - A variant identifier is contained in the group's `url` when a
+ *   variant's `sku` or bare `@id` token (`#` stripped) appears in it as
+ *   a whole token — delimited on both sides by one of `/ - _ . ? = & #`
+ *   or the string's start/end. An identifier appearing only as part of
+ *   a longer alphanumeric run (sku `12` inside `product-12345`) does
+ *   not count, so coincidental substrings no longer suppress the
+ *   warning.
+ *
+ * Per the catalogue: no fire without a top-level `url` (page-less
+ * parent), and the partial-population case belongs to PDP-W-018. Per the
+ * pass-#6 editor decision, same-document fragment `@id`s on variants
+ * identify but do not address — they neither suppress the rule nor
+ * change when it fires; only the message wording acknowledges them.
+ */
+export declare const pdpW014: Rule;

package/dist/rules/pdp-w-014.js

@@ -0,0 +1,69 @@
+/**
+ * PDP-W-014 — ProductGroup variant URL absent on distinctly-addressable
+ * variants. Transcribed from commerce/spec/rules.md (normative), with
+ * two reductions:
+ *
+ * - The catalogue's target is `$.data.hasVariant[*]`, but the check and
+ *   message are group-level ("ProductGroup at `{path}` has `{count}`
+ *   variants…", firing when *none* of the entries declare a `url`). The
+ *   target here is `$.data`, producing the one group-level finding the
+ *   message describes.
+ * - A variant identifier is contained in the group's `url` when a
+ *   variant's `sku` or bare `@id` token (`#` stripped) appears in it as
+ *   a whole token — delimited on both sides by one of `/ - _ . ? = & #`
+ *   or the string's start/end. An identifier appearing only as part of
+ *   a longer alphanumeric run (sku `12` inside `product-12345`) does
+ *   not count, so coincidental substrings no longer suppress the
+ *   warning.
+ *
+ * Per the catalogue: no fire without a top-level `url` (page-less
+ * parent), and the partial-population case belongs to PDP-W-018. Per the
+ * pass-#6 editor decision, same-document fragment `@id`s on variants
+ * identify but do not address — they neither suppress the rule nor
+ * change when it fires; only the message wording acknowledges them.
+ */
+export const pdpW014 = {
+    id: "PDP-W-014",
+    title: "ProductGroup variant URL absent on distinctly-addressable variants",
+    severity: "warning",
+    target: "$.data",
+    check: (match) => {
+        const data = match.value;
+        if (typeof data !== "object" || data === null)
+            return null;
+        const d = data;
+        if (d["@type"] !== "ProductGroup")
+            return null;
+        const variants = d["hasVariant"];
+        if (!Array.isArray(variants) || variants.length < 2)
+            return null;
+        const groupUrl = d["url"];
+        if (typeof groupUrl !== "string" || groupUrl.trim() === "")
+            return null;
+        const entries = variants.filter((v) => typeof v === "object" && v !== null);
+        const anyUrl = entries.some((v) => typeof v["url"] === "string" && v["url"].trim() !== "");
+        if (anyUrl)
+            return null;
+        const DELIMITER = "[/\\-_.?=&#]";
+        const containsAsToken = (url, id) => {
+            const escaped = id.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+            return new RegExp(`(?:^|${DELIMITER})${escaped}(?:${DELIMITER}|$)`).test(url);
+        };
+        const urlContainsVariantId = entries.some((v) => {
+            const ids = [
+                typeof v["sku"] === "string" ? v["sku"] : undefined,
+                typeof v["@id"] === "string"
+                    ? v["@id"].replace(/^#/, "")
+                    : undefined,
+            ];
+            return ids.some((id) => id !== undefined && id !== "" && containsAsToken(groupUrl, id));
+        });
+        if (urlContainsVariantId)
+            return null;
+        return { values: { count: variants.length } };
+    },
+    messageTemplate: "ProductGroup at `{path}` has `{count}` variants with no declared per-variant `url`s. If each variant is genuinely reachable at its own address — its own page or a variant-selecting parameter — declare it in the variant's `url` field; a fragment `@id` identifies the variant but does not promise it is addressable.",
+    remediation: "For each variant entry in `hasVariant`, add a `url` field with the variant's addressable URL — its own page (case 3) or the shared page with its variant-selecting query parameter (case 2). If all variants share the exact same URL with no parameter-based differentiation (case 1), the per-variant `url` field can be omitted.",
+    introduced: "v0.6",
+    specReference: "product.md#per-variant-urls",
+};

package/dist/rules/pdp-w-015.d.ts

@@ -0,0 +1,10 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * PDP-W-015 — Price-on-request without contact channel.
+ * Transcribed from commerce/spec/rules.md (normative).
+ *
+ * Bespoke check: `method` is anything other than `unspecified` and none
+ * of `contactUrl`, `contactEmail`, `contactPhone` is declared (key
+ * presence — the catalogue's "declares none of").
+ */
+export declare const pdpW015: Rule;