@elementor/editor-audits 4.4.0-1076 → 4.4.0-1077

package/dist/index.mjs

@@ -103,8 +103,8 @@ async function requestPageContext(documentId, attachmentIds, allowNonceRetry) {
 
 // src/registry.ts
 var registry = /* @__PURE__ */ new Map();
-function registerAudit(audit21) {
-  registry.set(audit21.id, audit21);
+function registerAudit(audit22) {
+  registry.set(audit22.id, audit22);
 }
 function getRegisteredAudits() {
   return Array.from(registry.values());
@@ -140,6 +140,11 @@ var CATEGORY_LABELS = {
 // src/utils/audit-status-summary.ts
 import { __ as __2 } from "@wordpress/i18n";
 
+// src/utils/is-scored-audit.ts
+function isScoredAudit(audit22) {
+  return audit22.weight > 0;
+}
+
 // src/utils/sort-failed-audits.ts
 var SEVERITY_SORTING_RANK = {
   error: 0,
@@ -170,7 +175,9 @@ function partitionAuditResults(report, options = {}) {
     switch (run.result.status) {
       case "fail":
         failed.push({ ...run, result: run.result });
-        totalViolations += run.result.violations.length;
+        if (isScoredAudit(run.audit)) {
+          totalViolations += run.result.violations.length;
+        }
         break;
       case "pass":
         passed.push({ ...run, result: run.result });
@@ -191,7 +198,10 @@ function auditStatusDisplayCounts(report) {
   let pass = 0;
   let skipped = 0;
   let totalViolations = 0;
-  for (const { result } of report.auditResults) {
+  for (const { audit: audit22, result } of report.auditResults) {
+    if (!isScoredAudit(audit22)) {
+      continue;
+    }
     switch (result.status) {
       case "fail":
         totalViolations += result.violations.length;
@@ -239,16 +249,16 @@ function computeReport(documentId, results) {
   const accumulators = Object.fromEntries(
     ALL_CATEGORIES.map((c) => [c, { totalWeight: 0, passedWeight: 0, total: 0, failed: 0 }])
   );
-  for (const { audit: audit21, result } of results) {
+  for (const { audit: audit22, result } of results) {
     if (result.status === "skipped") {
       continue;
     }
-    for (const category of audit21.categories) {
+    for (const category of audit22.categories) {
       const acc = accumulators[category];
       acc.total++;
-      acc.totalWeight += audit21.weight;
+      acc.totalWeight += audit22.weight;
       if (result.status === "pass") {
-        acc.passedWeight += audit21.weight;
+        acc.passedWeight += audit22.weight;
       } else {
         acc.failed++;
       }
@@ -437,10 +447,10 @@ async function runPageAudit(documentId) {
   const ctx = { documentId, elements, pageContext, kit };
   const registered = getRegisteredAudits();
   const auditResults = await Promise.all(
-    registered.map(async (audit21) => {
-      const { evaluate: _evaluate, ...meta } = audit21;
+    registered.map(async (audit22) => {
+      const { evaluate: _evaluate, ...meta } = audit22;
       try {
-        const result = await audit21.evaluate(ctx);
+        const result = await audit22.evaluate(ctx);
         return { audit: meta, result };
       } catch (error) {
         const reason = error instanceof Error ? error.message : "unknown-error";
@@ -741,7 +751,7 @@ import { Box as Box4, Chip, Collapse, IconButton as IconButton2 } from "@element
 import { __ as __7 } from "@wordpress/i18n";
 function StatusSection({ label, count, color = "default", defaultExpanded = false, children }) {
   const [expanded, setExpanded] = useState2(defaultExpanded);
-  if (count === 0) {
+  if (React5.Children.count(children) === 0) {
     return null;
   }
   return /* @__PURE__ */ React5.createElement(Box4, { sx: { paddingBlock: 1 } }, /* @__PURE__ */ React5.createElement(
@@ -940,13 +950,13 @@ function ViolationIcon({ violation, widgetIcon }) {
 function SkipReasonTooltip({ reason }) {
   return /* @__PURE__ */ React11.createElement(Tooltip3, { title: reason, placement: "top" }, /* @__PURE__ */ React11.createElement(Box7, { "aria-label": reason, component: "span", sx: { display: "inline-flex", alignItems: "center" } }, /* @__PURE__ */ React11.createElement(HelpIcon, { fontSize: "small", color: "action" })));
 }
-function StatusIndicator({ audit: audit21, violations }) {
+function StatusIndicator({ audit: audit22, violations }) {
   if (violations) {
-    return /* @__PURE__ */ React11.createElement(React11.Fragment, null, /* @__PURE__ */ React11.createElement(Typography6, { variant: "caption", color: "text.secondary", fontWeight: "bold" }, violations.length), /* @__PURE__ */ React11.createElement(SeverityIcon, { severity: audit21.severity }));
+    return /* @__PURE__ */ React11.createElement(React11.Fragment, null, isScoredAudit(audit22) && /* @__PURE__ */ React11.createElement(Typography6, { variant: "caption", color: "text.secondary", fontWeight: "bold" }, violations.length), /* @__PURE__ */ React11.createElement(SeverityIcon, { severity: audit22.severity }));
   }
   return /* @__PURE__ */ React11.createElement(CheckIcon, { fontSize: "small", color: "success" });
 }
-function ViolationRow({ audit: audit21, skipReason, violations }) {
+function ViolationRow({ audit: audit22, skipReason, violations }) {
   const [expanded, setExpanded] = useState3(false);
   const toggleExpanded = () => setExpanded((value) => !value);
   return /* @__PURE__ */ React11.createElement(Box7, { sx: { borderBottom: 1, borderColor: "divider", paddingBlock: 0.5 } }, /* @__PURE__ */ React11.createElement(Box7, { sx: { display: "flex", alignItems: "center", gap: 0.5 } }, /* @__PURE__ */ React11.createElement(
@@ -962,8 +972,8 @@ function ViolationRow({ audit: audit21, skipReason, violations }) {
       },
       onClick: toggleExpanded
     },
-    /* @__PURE__ */ React11.createElement(Typography6, { variant: "body2", sx: { flex: 1 } }, audit21.title),
-    !skipReason && /* @__PURE__ */ React11.createElement(StatusIndicator, { audit: audit21, violations })
+    /* @__PURE__ */ React11.createElement(Typography6, { variant: "body2", sx: { flex: 1 } }, audit22.title),
+    !skipReason && /* @__PURE__ */ React11.createElement(StatusIndicator, { audit: audit22, violations })
   ), skipReason && /* @__PURE__ */ React11.createElement(SkipReasonTooltip, { reason: skipReason }), /* @__PURE__ */ React11.createElement(
     IconButton5,
     {
@@ -989,7 +999,7 @@ function ViolationRow({ audit: audit21, skipReason, violations }) {
       icon: /* @__PURE__ */ React11.createElement(AlertCircleIcon2, { fontSize: "small", color: "secondary", "aria-hidden": true })
     },
     /* @__PURE__ */ React11.createElement(AlertTitle, null, /* @__PURE__ */ React11.createElement(Typography6, { variant: "caption", component: "p", color: "text.primary", fontWeight: "bold" }, __11("What's the issue", "elementor"))),
-    /* @__PURE__ */ React11.createElement(Typography6, { variant: "caption", component: "p", color: "text.secondary" }, audit21.description)
+    /* @__PURE__ */ React11.createElement(Typography6, { variant: "caption", component: "p", color: "text.secondary" }, audit22.description)
   ), /* @__PURE__ */ React11.createElement(
     Alert,
     {
@@ -998,7 +1008,7 @@ function ViolationRow({ audit: audit21, skipReason, violations }) {
       icon: /* @__PURE__ */ React11.createElement(BulbIcon, { fontSize: "small", color: "info", "aria-hidden": true })
     },
     /* @__PURE__ */ React11.createElement(AlertTitle, null, /* @__PURE__ */ React11.createElement(Typography6, { variant: "caption", component: "p", color: "text.primary", fontWeight: "bold" }, __11("How to resolve", "elementor"))),
-    /* @__PURE__ */ React11.createElement(Typography6, { variant: "caption", component: "p", color: "text.secondary" }, audit21.fixHint)
+    /* @__PURE__ */ React11.createElement(Typography6, { variant: "caption", component: "p", color: "text.secondary" }, audit22.fixHint)
   )), violations && violations.length > 0 && /* @__PURE__ */ React11.createElement(Box7, { role: "list", sx: { paddingBlockEnd: 1, paddingInlineStart: 2 } }, violations.map((violation, idx) => {
     const widgetIcon = violation.elementId ? getElementIcon(violation.elementId) : null;
     const elementTitle = violation.elementId ? getElementTitle(violation.elementId) : null;
@@ -1138,14 +1148,17 @@ import { __ as __14, sprintf as sprintf2 } from "@wordpress/i18n";
 var ALL_SEVERITIES = ["error", "warning", "info"];
 function countSeverities(report, category) {
   const counts = { error: 0, warning: 0, info: 0 };
-  for (const { audit: audit21, result } of report.auditResults) {
+  for (const { audit: audit22, result } of report.auditResults) {
     if (result.status !== "fail") {
       continue;
     }
-    if (category && !audit21.categories.includes(category)) {
+    if (category && !audit22.categories.includes(category)) {
+      continue;
+    }
+    if (!isScoredAudit(audit22)) {
       continue;
     }
-    counts[audit21.severity] += result.violations.length;
+    counts[audit22.severity] += result.violations.length;
   }
   return counts;
 }
@@ -1257,6 +1270,13 @@ var PROMOTIONS = [
     formatSubtitle: (run) => run.result.status === "fail" ? __15("Generate cookie policy", "elementor") : null,
     getCtaUrl: firstFailExternalUrl
   },
+  {
+    auditId: "audits/scan-for-cookies",
+    icon: ElementorCookieIcon,
+    ctaLabel: __15("Scan", "elementor"),
+    formatSubtitle: (run) => run.result.status === "fail" ? __15("Scan this page for cookies", "elementor") : null,
+    getCtaUrl: firstFailExternalUrl
+  },
   {
     auditId: "audits/images-too-large",
     icon: ShieldHalfFilledIcon2,
@@ -2716,30 +2736,58 @@ var audit17 = {
   }
 };
 
-// src/audits/sections-and-columns.ts
+// src/audits/scan-for-cookies.ts
 import { __ as __40 } from "@wordpress/i18n";
 var audit18 = {
-  id: "audits/sections-and-columns",
-  title: __40("Sections and columns", "elementor"),
+  id: "audits/scan-for-cookies",
+  title: __40("Scan for cookies", "elementor"),
   description: __40(
+    "Your site may be setting cookies you haven't disclosed. Some countries require a banner even without a full consent policy. Scan to see what needs disclosure.",
+    "elementor"
+  ),
+  fixHint: __40("Run a Cookiez scan on this page to see which cookies need disclosure.", "elementor"),
+  categories: ["compliance"],
+  severity: "info",
+  weight: 0,
+  evaluate: (ctx) => {
+    const isReady = ctx.pageContext.cookiez_plugin_installed && ctx.pageContext.cookiez_plugin_active;
+    return {
+      status: "fail",
+      violations: [
+        {
+          auditId: audit18.id,
+          label: __40("This page has not been scanned for cookies yet.", "elementor"),
+          externalUrl: isReady ? ctx.pageContext.cookiez_scan_url : ctx.pageContext.cookiez_plugin_action_url
+        }
+      ]
+    };
+  }
+};
+
+// src/audits/sections-and-columns.ts
+import { __ as __41 } from "@wordpress/i18n";
+var audit19 = {
+  id: "audits/sections-and-columns",
+  title: __41("Sections and columns", "elementor"),
+  description: __41(
     "Sections and columns are legacy elements. Containers render fewer DOM nodes and are more flexible.",
     "elementor"
   ),
-  fixHint: __40("Use the Container Converter to replace each section/column with a container.", "elementor"),
+  fixHint: __41("Use the Container Converter to replace each section/column with a container.", "elementor"),
   categories: ["best-practices", "performance"],
   severity: "warning",
   weight: 7,
   evaluate: (ctx) => {
     if (ctx.elements.tree.length === 0) {
-      return { status: "skipped", reason: __40("No elements", "elementor") };
+      return { status: "skipped", reason: __41("No elements", "elementor") };
     }
     const violations = [];
     walkElements(ctx.elements.tree, (node) => {
       if (node.elType === "section" || node.elType === "column") {
         violations.push({
-          auditId: audit18.id,
+          auditId: audit19.id,
           elementId: node.id,
-          label: node.elType === "section" ? __40("Section element", "elementor") : __40("Column element", "elementor")
+          label: node.elType === "section" ? __41("Section element", "elementor") : __41("Column element", "elementor")
         });
       }
     });
@@ -2748,15 +2796,15 @@ var audit18 = {
 };
 
 // src/audits/site-identity.ts
-import { __ as __41 } from "@wordpress/i18n";
-var audit19 = {
+import { __ as __42 } from "@wordpress/i18n";
+var audit20 = {
   id: "audits/site-identity",
-  title: __41("Site identity", "elementor"),
-  description: __41(
+  title: __42("Site identity", "elementor"),
+  description: __42(
     "Site name, description, logo, and favicon establish your brand and appear in search results and browser tabs.",
     "elementor"
   ),
-  fixHint: __41("Open Site Settings \u2192 Site Identity and complete all the missing fields.", "elementor"),
+  fixHint: __42("Open Site Settings \u2192 Site Identity and complete all the missing fields.", "elementor"),
   categories: ["best-practices", "seo"],
   severity: "info",
   weight: 7,
@@ -2765,30 +2813,30 @@ var audit19 = {
     const violations = [];
     if (!identity.site_name_set) {
       violations.push({
-        auditId: audit19.id,
-        label: __41("Site name is missing or still uses the default.", "elementor"),
+        auditId: audit20.id,
+        label: __42("Site name is missing or still uses the default.", "elementor"),
         targetHint: "site-identity-settings"
       });
     }
     if (!identity.site_description_set) {
       violations.push({
-        auditId: audit19.id,
-        label: __41("Site description is missing or still uses the default.", "elementor"),
+        auditId: audit20.id,
+        label: __42("Site description is missing or still uses the default.", "elementor"),
         targetHint: "site-identity-settings",
         angieFix: true
       });
     }
     if (!identity.site_logo_set) {
       violations.push({
-        auditId: audit19.id,
-        label: __41("Site logo is not set.", "elementor"),
+        auditId: audit20.id,
+        label: __42("Site logo is not set.", "elementor"),
         targetHint: "site-identity-settings"
       });
     }
     if (!identity.site_favicon_set) {
       violations.push({
-        auditId: audit19.id,
-        label: __41("Site favicon is not set.", "elementor"),
+        auditId: audit20.id,
+        label: __42("Site favicon is not set.", "elementor"),
         targetHint: "site-identity-settings"
       });
     }
@@ -2803,19 +2851,19 @@ var audit19 = {
 };
 
 // src/audits/too-many-widgets.ts
-import { __ as __42 } from "@wordpress/i18n";
+import { __ as __43 } from "@wordpress/i18n";
 var WIDGET_COUNT_THRESHOLD = 100;
-var audit20 = {
+var audit21 = {
   id: "audits/too-many-widgets",
-  title: __42("Too many widgets", "elementor"),
-  description: __42("Excessive DOM size caused by too many widgets degrades rendering performance.", "elementor"),
-  fixHint: __42("Reduce the number of widgets on the page by removing or combining elements.", "elementor"),
+  title: __43("Too many widgets", "elementor"),
+  description: __43("Excessive DOM size caused by too many widgets degrades rendering performance.", "elementor"),
+  fixHint: __43("Reduce the number of widgets on the page by removing or combining elements.", "elementor"),
   categories: ["best-practices", "performance"],
   severity: "warning",
   weight: 5,
   evaluate: (ctx) => {
     if (ctx.elements.tree.length === 0) {
-      return { status: "skipped", reason: __42("No elements", "elementor") };
+      return { status: "skipped", reason: __43("No elements", "elementor") };
     }
     let widgetCount = 0;
     walkElements(ctx.elements.tree, (node) => {
@@ -2830,8 +2878,8 @@ var audit20 = {
       status: "fail",
       violations: [
         {
-          auditId: audit20.id,
-          label: __42("Page has too many widgets.", "elementor")
+          auditId: audit21.id,
+          label: __43("Page has too many widgets.", "elementor")
         }
       ]
     };
@@ -2845,12 +2893,12 @@ var AUDITS = [
   audit12,
   audit7,
   audit5,
-  audit20,
-  audit18,
+  audit21,
+  audit19,
   audit3,
   audit10,
   audit4,
-  audit19,
+  audit20,
   audit14,
   audit15,
   audit6,
@@ -2859,11 +2907,12 @@ var AUDITS = [
   audit17,
   audit16,
   audit,
-  audit2
+  audit2,
+  audit18
 ];
 function registerAllAudits() {
-  for (const audit21 of AUDITS) {
-    registerAudit(audit21);
+  for (const audit22 of AUDITS) {
+    registerAudit(audit22);
   }
 }