@pagefront/lint-commerce 0.3.0 → 0.4.0

package/dist/catalogue.js

@@ -39,6 +39,6 @@ import { pdpI001 } from "./rules/pdp-i-001.js";
 export const commerceCatalogue = {
     vertical: "commerce",
     formatVersion: "0.9",
-    catalogueVersion: "0.6.0",
+    catalogueVersion: "0.7.0",
     rules: [pdpE001, pdpE002, pdpE003, pdpW001, pdpW002, pdpW003, pdpW004, pdpW005, pdpW006, pdpW007, pdpW008, pdpW009, pdpW010, pdpW011, pdpW012, pdpW013, pdpW014, pdpW015, pdpW016, pdpW017, pdpW018, pdpW019, pdpW020, pdpW021, pdpW022, pdpW023, pdpW024, pdpW025, pdpW026, pdpW027, pdpI001],
 };

package/dist/organization-catalogue.js

@@ -1,3 +1,4 @@
+import { orgE001 } from "./rules/org-e-001.js";
 import { orgW001 } from "./rules/org-w-001.js";
 import { orgW002 } from "./rules/org-w-002.js";
 import { orgW003 } from "./rules/org-w-003.js";
@@ -16,6 +17,6 @@ import { orgW006 } from "./rules/org-w-006.js";
 export const organizationCatalogue = {
     vertical: "commerce",
     formatVersion: "0.9",
-    catalogueVersion: "0.6.0",
-    rules: [orgW001, orgW002, orgW003, orgW004, orgW005, orgW006],
+    catalogueVersion: "0.7.0",
+    rules: [orgE001, orgW001, orgW002, orgW003, orgW004, orgW005, orgW006],
 };

package/dist/rules/context-prefix.d.ts

@@ -0,0 +1,16 @@
+/**
+ * Shared by PDP-E-003 and ORG-E-001: does the data block declare the
+ * `pagefront` prefix, and which `pagefront:` term does it use first?
+ * A term is a property name or an `@type` value beginning with
+ * `pagefront:`, anywhere in the block; the walk is depth-first in
+ * document order, so the named term is the first one a reader meets.
+ */
+export declare const VOCABULARY_IRI = "https://themachineweb.org/pagefront/vocabulary/";
+export declare const declaresPagefrontPrefix: (context: unknown) => boolean;
+export declare const firstPagefrontTerm: (value: unknown) => string | undefined;
+/** The check both rules share: null when the prefix is declared or no term is used. */
+export declare const missingPrefixTerm: (data: unknown) => {
+    values: {
+        term: string;
+    };
+} | null;

package/dist/rules/context-prefix.js

@@ -0,0 +1,44 @@
+/**
+ * Shared by PDP-E-003 and ORG-E-001: does the data block declare the
+ * `pagefront` prefix, and which `pagefront:` term does it use first?
+ * A term is a property name or an `@type` value beginning with
+ * `pagefront:`, anywhere in the block; the walk is depth-first in
+ * document order, so the named term is the first one a reader meets.
+ */
+export const VOCABULARY_IRI = "https://themachineweb.org/pagefront/vocabulary/";
+export const declaresPagefrontPrefix = (context) => Array.isArray(context) &&
+    context.some((entry) => typeof entry === "object" &&
+        entry !== null &&
+        !Array.isArray(entry) &&
+        entry["pagefront"] === VOCABULARY_IRI);
+export const firstPagefrontTerm = (value) => {
+    if (Array.isArray(value)) {
+        for (const item of value) {
+            const found = firstPagefrontTerm(item);
+            if (found !== undefined)
+                return found;
+        }
+        return undefined;
+    }
+    if (typeof value !== "object" || value === null)
+        return undefined;
+    for (const [key, child] of Object.entries(value)) {
+        if (key.startsWith("pagefront:"))
+            return key;
+        if (key === "@type" && typeof child === "string" && child.startsWith("pagefront:"))
+            return child;
+        const found = firstPagefrontTerm(child);
+        if (found !== undefined)
+            return found;
+    }
+    return undefined;
+};
+/** The check both rules share: null when the prefix is declared or no term is used. */
+export const missingPrefixTerm = (data) => {
+    if (typeof data !== "object" || data === null || Array.isArray(data))
+        return null;
+    if (declaresPagefrontPrefix(data["@context"]))
+        return null;
+    const term = firstPagefrontTerm(data);
+    return term === undefined ? null : { values: { term } };
+};

package/dist/rules/org-e-001.d.ts

@@ -0,0 +1,7 @@
+import type { Rule } from "@pagefront/lint-core";
+/**
+ * ORG-E-001 — `pagefront:` term without the vocabulary prefix in `@context`.
+ * Transcribed from commerce/spec/rules.md (normative). The organization
+ * counterpart of PDP-E-003; the check is shared (context-prefix.ts).
+ */
+export declare const orgE001: Rule;

package/dist/rules/org-e-001.js

@@ -0,0 +1,17 @@
+import { missingPrefixTerm } from "./context-prefix.js";
+/**
+ * ORG-E-001 — `pagefront:` term without the vocabulary prefix in `@context`.
+ * Transcribed from commerce/spec/rules.md (normative). The organization
+ * counterpart of PDP-E-003; the check is shared (context-prefix.ts).
+ */
+export const orgE001 = {
+    id: "ORG-E-001",
+    title: "pagefront: term without the vocabulary prefix",
+    severity: "error",
+    target: "$.data",
+    check: (match) => missingPrefixTerm(match.value),
+    messageTemplate: "The Sheet uses `{term}` but `@context` at `{path}` does not declare the `pagefront` prefix, so the term does not resolve to the Pagefront vocabulary.",
+    remediation: 'Declare the prefix: `"@context": ["https://schema.org", {"pagefront": "https://themachineweb.org/pagefront/vocabulary/"}]`.',
+    introduced: "v0.9",
+    specReference: "vocabulary.md#how-to-use-this-vocabulary",
+};

package/dist/rules/pdp-e-003.d.ts

@@ -1,11 +1,7 @@
 import type { Rule } from "@pagefront/lint-core";
 /**
  * PDP-E-003 — `pagefront:` term without the vocabulary prefix in `@context`.
- * Transcribed from commerce/spec/rules.md (normative).
- *
- * A term is a property name or an `@type` value beginning with
- * `pagefront:`, anywhere in the data block. The walk is depth-first in
- * document order, so the named term is the first one a reader meets.
+ * Transcribed from commerce/spec/rules.md (normative). The check is
+ * shared with ORG-E-001 (context-prefix.ts).
  */
-export declare const VOCABULARY_IRI = "https://themachineweb.org/pagefront/vocabulary/";
 export declare const pdpE003: Rule;

package/dist/rules/pdp-e-003.js

@@ -1,53 +1,15 @@
+import { missingPrefixTerm } from "./context-prefix.js";
 /**
  * PDP-E-003 — `pagefront:` term without the vocabulary prefix in `@context`.
- * Transcribed from commerce/spec/rules.md (normative).
- *
- * A term is a property name or an `@type` value beginning with
- * `pagefront:`, anywhere in the data block. The walk is depth-first in
- * document order, so the named term is the first one a reader meets.
+ * Transcribed from commerce/spec/rules.md (normative). The check is
+ * shared with ORG-E-001 (context-prefix.ts).
  */
-export const VOCABULARY_IRI = "https://themachineweb.org/pagefront/vocabulary/";
-const declaresPrefix = (context) => Array.isArray(context) &&
-    context.some((entry) => typeof entry === "object" &&
-        entry !== null &&
-        !Array.isArray(entry) &&
-        entry["pagefront"] === VOCABULARY_IRI);
-const firstTerm = (value) => {
-    if (Array.isArray(value)) {
-        for (const item of value) {
-            const found = firstTerm(item);
-            if (found !== undefined)
-                return found;
-        }
-        return undefined;
-    }
-    if (typeof value !== "object" || value === null)
-        return undefined;
-    for (const [key, child] of Object.entries(value)) {
-        if (key.startsWith("pagefront:"))
-            return key;
-        if (key === "@type" && typeof child === "string" && child.startsWith("pagefront:"))
-            return child;
-        const found = firstTerm(child);
-        if (found !== undefined)
-            return found;
-    }
-    return undefined;
-};
 export const pdpE003 = {
     id: "PDP-E-003",
     title: "pagefront: term without the vocabulary prefix",
     severity: "error",
     target: "$.data",
-    check: (match) => {
-        const data = match.value;
-        if (typeof data !== "object" || data === null || Array.isArray(data))
-            return null;
-        if (declaresPrefix(data["@context"]))
-            return null;
-        const term = firstTerm(data);
-        return term === undefined ? null : { values: { term } };
-    },
+    check: (match) => missingPrefixTerm(match.value),
     messageTemplate: "The Sheet uses `{term}` but `@context` at `{path}` does not declare the `pagefront` prefix, so the term does not resolve to the Pagefront vocabulary.",
     remediation: 'Declare the prefix: `"@context": ["https://schema.org", {"pagefront": "https://themachineweb.org/pagefront/vocabulary/"}]`.',
     introduced: "v0.9",

package/dist/version.d.ts

@@ -5,4 +5,4 @@
  * (test/linter-version.test.mjs) fails when they diverge. Regenerate
  * on version bump.
  */
-export declare const PACKAGE_VERSION = "0.3.0";
+export declare const PACKAGE_VERSION = "0.4.0";

package/dist/version.js

@@ -5,4 +5,4 @@
  * (test/linter-version.test.mjs) fails when they diverge. Regenerate
  * on version bump.
  */
-export const PACKAGE_VERSION = "0.3.0";
+export const PACKAGE_VERSION = "0.4.0";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pagefront/lint-commerce",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "Pagefront commerce linter: the commerce rule catalogue and CLI, on the @pagefront/lint-core engine.",
   "license": "Apache-2.0",
   "repository": {