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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elementor/editor-audits",
-  "version": "4.4.0-1075",
+  "version": "4.4.0-1077",
   "private": false,
   "author": "Elementor Team",
   "homepage": "https://elementor.com/",
@@ -39,19 +39,19 @@
     "dev": "tsup --config=../../tsup.dev.ts"
   },
   "dependencies": {
-    "@elementor/editor": "4.4.0-1075",
-    "@elementor/editor-mcp": "4.4.0-1075",
-    "@elementor/editor-app-bar": "4.4.0-1075",
-    "@elementor/editor-elements": "4.4.0-1075",
-    "@elementor/editor-floating-panels": "4.4.0-1075",
-    "@elementor/editor-notifications": "4.4.0-1075",
-    "@elementor/editor-v1-adapters": "4.4.0-1075",
-    "@elementor/events": "4.4.0-1075",
-    "@elementor/http-client": "4.4.0-1075",
+    "@elementor/editor": "4.4.0-1077",
+    "@elementor/editor-mcp": "4.4.0-1077",
+    "@elementor/editor-app-bar": "4.4.0-1077",
+    "@elementor/editor-elements": "4.4.0-1077",
+    "@elementor/editor-floating-panels": "4.4.0-1077",
+    "@elementor/editor-notifications": "4.4.0-1077",
+    "@elementor/editor-v1-adapters": "4.4.0-1077",
+    "@elementor/events": "4.4.0-1077",
+    "@elementor/http-client": "4.4.0-1077",
     "@elementor/icons": "~1.77.0",
-    "@elementor/locations": "4.4.0-1075",
-    "@elementor/session": "4.4.0-1075",
-    "@elementor/store": "4.4.0-1075",
+    "@elementor/locations": "4.4.0-1077",
+    "@elementor/session": "4.4.0-1077",
+    "@elementor/store": "4.4.0-1077",
     "@elementor/ui": "1.37.5"
   },
   "peerDependencies": {

package/src/audits/privacy-policy.ts

@@ -28,6 +28,7 @@ export const audit: Audit = {
 					auditId: audit.id,
 					label: __( 'No privacy policy page is set.', 'elementor' ),
 					externalUrl: ctx.pageContext.privacy_settings_url,
+					ctaLabel: __( 'Create', 'elementor' ),
 				},
 			],
 		};

package/src/audits/scan-for-cookies.ts

@@ -0,0 +1,30 @@
+import { __ } from '@wordpress/i18n';
+
+import { type Audit } from '../types';
+
+export const audit: Audit = {
+	id: 'audits/scan-for-cookies',
+	title: __( 'Scan for cookies', 'elementor' ),
+	description: __(
+		"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: __( '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: audit.id,
+					label: __( 'This page has not been scanned for cookies yet.', 'elementor' ),
+					externalUrl: isReady ? ctx.pageContext.cookiez_scan_url : ctx.pageContext.cookiez_plugin_action_url,
+				},
+			],
+		};
+	},
+};

package/src/components/status-section.tsx

@@ -15,7 +15,7 @@ type Props = {
 export default function StatusSection( { label, count, color = 'default', defaultExpanded = false, children }: Props ) {
 	const [ expanded, setExpanded ] = useState( defaultExpanded );
 
-	if ( count === 0 ) {
+	if ( React.Children.count( children ) === 0 ) {
 		return null;
 	}
 

package/src/components/violation-cta-button.tsx

@@ -0,0 +1,22 @@
+import * as React from 'react';
+import { Button } from '@elementor/ui';
+
+type Props = {
+	ctaLabel: string;
+	externalUrl: string;
+};
+
+export default function ViolationCtaButton( { ctaLabel, externalUrl }: Props ) {
+	const handleClick = ( event: React.MouseEvent< HTMLButtonElement > ) => {
+		event.stopPropagation();
+		event.preventDefault();
+
+		window.open( externalUrl, '_blank', 'noopener' );
+	};
+
+	return (
+		<Button variant="outlined" color="secondary" size="small" onClick={ handleClick }>
+			{ ctaLabel }
+		</Button>
+	);
+}

package/src/components/violation-row.tsx

@@ -8,9 +8,11 @@ import { __ } from '@wordpress/i18n';
 import { focusViolation } from '../hooks/focus-violation';
 import { type AuditMeta, type AuditViolation } from '../types';
 import { buildAngiePrompt } from '../utils/build-angie-prompt';
+import { isScoredAudit } from '../utils/is-scored-audit';
 import { onKeyboardClick } from '../utils/keyboard-click';
 import FixViolationWithAngie from './fix-violation-with-angie';
 import SeverityIcon from './severity-icons';
+import ViolationCtaButton from './violation-cta-button';
 import ViolationIcon from './violation-icons';
 
 type Props = {
@@ -33,9 +35,11 @@ function StatusIndicator( { audit, violations }: Pick< Props, 'audit' | 'violati
 	if ( violations ) {
 		return (
 			<>
-				<Typography variant="caption" color="text.secondary" fontWeight="bold">
-					{ violations.length }
-				</Typography>
+				{ isScoredAudit( audit ) && (
+					<Typography variant="caption" color="text.secondary" fontWeight="bold">
+						{ violations.length }
+					</Typography>
+				) }
 				<SeverityIcon severity={ audit.severity } />
 			</>
 		);
@@ -157,7 +161,18 @@ export default function ViolationRow( { audit, skipReason, violations }: Props )
 									{ violation.angieFix && (
 										<FixViolationWithAngie prompt={ buildAngiePrompt( rowLabel ) } />
 									) }
-									<EyeIcon className="violation-hover-icon" fontSize="tiny" aria-hidden={ true } />
+									{ violation.ctaLabel && violation.externalUrl ? (
+										<ViolationCtaButton
+											ctaLabel={ violation.ctaLabel }
+											externalUrl={ violation.externalUrl }
+										/>
+									) : (
+										<EyeIcon
+											className="violation-hover-icon"
+											fontSize="tiny"
+											aria-hidden={ true }
+										/>
+									) }
 								</Box>
 							);
 						} ) }

package/src/register-audits.ts

@@ -15,6 +15,7 @@ import * as preferGlobalColors from './audits/prefer-global-colors';
 import * as preferGlobalFonts from './audits/prefer-global-fonts';
 import * as privacyPolicy from './audits/privacy-policy';
 import * as robotsNoindex from './audits/robots-noindex';
+import * as scanForCookies from './audits/scan-for-cookies';
 import * as sectionsAndColumns from './audits/sections-and-columns';
 import * as siteIdentity from './audits/site-identity';
 import * as tooManyWidgets from './audits/too-many-widgets';
@@ -47,6 +48,7 @@ const AUDITS: Audit[] = [
 	privacyPolicy.audit,
 	accessibilityPolicy.audit,
 	cookiePolicy.audit,
+	scanForCookies.audit,
 ];
 
 export function registerAllAudits(): void {

package/src/register-promotions.ts

@@ -49,6 +49,14 @@ export const PROMOTIONS: PromotionConfig[] = [
 			run.result.status === 'fail' ? __( 'Generate cookie policy', 'elementor' ) : null,
 		getCtaUrl: firstFailExternalUrl,
 	},
+	{
+		auditId: 'audits/scan-for-cookies',
+		icon: ElementorCookieIcon,
+		ctaLabel: __( 'Scan', 'elementor' ),
+		formatSubtitle: ( run ) =>
+			run.result.status === 'fail' ? __( 'Scan this page for cookies', 'elementor' ) : null,
+		getCtaUrl: firstFailExternalUrl,
+	},
 	{
 		auditId: 'audits/images-too-large',
 		icon: ShieldHalfFilledIcon,

package/src/types.ts

@@ -38,6 +38,7 @@ export type AuditViolation = {
 	label: string;
 	detail?: string;
 	angieFix?: boolean;
+	ctaLabel?: string;
 };
 
 export type PageContextResponse = {
@@ -65,6 +66,9 @@ export type PageContextResponse = {
 	ally_plugin_url: string;
 	cookiez_plugin_active: boolean;
 	cookiez_plugin_url: string;
+	cookiez_plugin_installed: boolean;
+	cookiez_plugin_action_url: string;
+	cookiez_scan_url: string;
 	image_optimization_plugin_active: boolean;
 	image_optimization_plugin_url: string;
 	site_identity: {

package/src/utils/audit-status-summary.ts

@@ -2,6 +2,7 @@ import { type ChipProps } from '@elementor/ui';
 import { __ } from '@wordpress/i18n';
 
 import { type AuditCategory, type AuditResult, type AuditRun, type PageAuditReport } from '../types';
+import { isScoredAudit } from './is-scored-audit';
 import { sortFailedAuditResults } from './sort-failed-audits';
 
 export type AuditStatusGroup = 'fail' | 'pass' | 'skipped';
@@ -37,7 +38,11 @@ export function partitionAuditResults(
 		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 } );
@@ -61,7 +66,11 @@ export function auditStatusDisplayCounts( report: PageAuditReport ): Record< Aud
 	let skipped = 0;
 	let totalViolations = 0;
 
-	for ( const { result } of report.auditResults ) {
+	for ( const { audit, result } of report.auditResults ) {
+		if ( ! isScoredAudit( audit ) ) {
+			continue;
+		}
+
 		switch ( result.status ) {
 			case 'fail':
 				totalViolations += result.violations.length;

package/src/utils/is-scored-audit.ts

@@ -0,0 +1,5 @@
+import { type AuditMeta } from '../types';
+
+export function isScoredAudit( audit: AuditMeta ): boolean {
+	return audit.weight > 0;
+}

package/src/utils/severity-counts.ts

@@ -1,6 +1,7 @@
 import { __, sprintf } from '@wordpress/i18n';
 
 import { type AuditCategory, type AuditSeverity, type PageAuditReport } from '../types';
+import { isScoredAudit } from './is-scored-audit';
 
 export type SeverityCounts = Record< AuditSeverity, number >;
 
@@ -18,6 +19,10 @@ export function countSeverities( report: PageAuditReport, category?: AuditCatego
 			continue;
 		}
 
+		if ( ! isScoredAudit( audit ) ) {
+			continue;
+		}
+
 		counts[ audit.severity ] += result.violations.length;
 	}