recker 1.0.43 → 1.0.44

package/dist/seo/rules/local.js

@@ -7,8 +7,6 @@ export const localRules = [
         severity: 'info',
         description: 'Local businesses should have LocalBusiness schema',
         check: (ctx) => {
-            if (!ctx.jsonLdTypes)
-                return null;
             const localBusinessTypes = [
                 'LocalBusiness', 'Restaurant', 'Store', 'MedicalBusiness',
                 'FinancialService', 'FoodEstablishment', 'HealthAndBeautyBusiness',
@@ -20,8 +18,12 @@ export const localRules = [
                 'LodgingBusiness', 'MovingCompany', 'Pharmacy', 'Physician',
                 'PlaceOfWorship', 'Plumber', 'RealEstateAgent', 'ShoppingCenter',
             ];
-            const hasLocalBusiness = ctx.jsonLdTypes.some(t => localBusinessTypes.includes(t));
-            if (!hasLocalBusiness && ctx.hasLocalBusinessSignals) {
+            const hasLocalBusiness = ctx.jsonLdTypes?.some(t => localBusinessTypes.includes(t));
+            if (hasLocalBusiness) {
+                const foundTypes = ctx.jsonLdTypes.filter(t => localBusinessTypes.includes(t));
+                return createResult({ id: 'local-business-schema', name: 'LocalBusiness Schema', category: 'structured-data', severity: 'info' }, 'pass', `LocalBusiness schema found: ${foundTypes.join(', ')}`);
+            }
+            if (ctx.hasLocalBusinessSignals) {
                 return createResult({ id: 'local-business-schema', name: 'LocalBusiness Schema', category: 'structured-data', severity: 'info' }, 'info', 'Page may benefit from LocalBusiness schema', {
                     recommendation: 'Add LocalBusiness structured data for local search visibility',
                     evidence: {
@@ -42,11 +44,9 @@ export const localRules = [
                     },
                 });
             }
-            if (hasLocalBusiness) {
-                const foundTypes = ctx.jsonLdTypes.filter(t => localBusinessTypes.includes(t));
-                return createResult({ id: 'local-business-schema', name: 'LocalBusiness Schema', category: 'structured-data', severity: 'info' }, 'pass', `LocalBusiness schema found: ${foundTypes.join(', ')}`);
-            }
-            return null;
+            return createResult({ id: 'local-business-schema', name: 'LocalBusiness Schema', category: 'structured-data', severity: 'info' }, 'info', 'No LocalBusiness schema detected', {
+                recommendation: 'Add LocalBusiness schema if this is a local business page',
+            });
         },
     },
     {
@@ -56,8 +56,11 @@ export const localRules = [
         severity: 'warning',
         description: 'LocalBusiness schema should include complete address',
         check: (ctx) => {
-            if (!ctx.localBusinessSchema)
-                return null;
+            if (!ctx.localBusinessSchema) {
+                return createResult({ id: 'local-business-address', name: 'Business Address', category: 'structured-data', severity: 'warning' }, 'info', 'Not applicable (no LocalBusiness schema)', {
+                    recommendation: 'Add LocalBusiness schema first, then include address',
+                });
+            }
             const address = ctx.localBusinessSchema.address;
             if (!address) {
                 return createResult({ id: 'local-business-address', name: 'Business Address', category: 'structured-data', severity: 'warning' }, 'warn', 'LocalBusiness schema missing address', {
@@ -96,8 +99,11 @@ export const localRules = [
         severity: 'warning',
         description: 'LocalBusiness schema should include phone number',
         check: (ctx) => {
-            if (!ctx.localBusinessSchema)
-                return null;
+            if (!ctx.localBusinessSchema) {
+                return createResult({ id: 'local-business-phone', name: 'Business Phone', category: 'structured-data', severity: 'warning' }, 'info', 'Not applicable (no LocalBusiness schema)', {
+                    recommendation: 'Add LocalBusiness schema first, then include phone',
+                });
+            }
             const phone = ctx.localBusinessSchema.telephone;
             if (!phone) {
                 return createResult({ id: 'local-business-phone', name: 'Business Phone', category: 'structured-data', severity: 'warning' }, 'warn', 'LocalBusiness schema missing phone number', {
@@ -128,8 +134,11 @@ export const localRules = [
         severity: 'info',
         description: 'LocalBusiness schema should include opening hours',
         check: (ctx) => {
-            if (!ctx.localBusinessSchema)
-                return null;
+            if (!ctx.localBusinessSchema) {
+                return createResult({ id: 'local-business-hours', name: 'Business Hours', category: 'structured-data', severity: 'info' }, 'info', 'Not applicable (no LocalBusiness schema)', {
+                    recommendation: 'Add LocalBusiness schema first, then include hours',
+                });
+            }
             const hours = ctx.localBusinessSchema.openingHoursSpecification ||
                 ctx.localBusinessSchema.openingHours;
             if (!hours) {
@@ -156,8 +165,11 @@ export const localRules = [
         severity: 'info',
         description: 'LocalBusiness schema should include geo coordinates',
         check: (ctx) => {
-            if (!ctx.localBusinessSchema)
-                return null;
+            if (!ctx.localBusinessSchema) {
+                return createResult({ id: 'local-business-geo', name: 'Business Coordinates', category: 'structured-data', severity: 'info' }, 'info', 'Not applicable (no LocalBusiness schema)', {
+                    recommendation: 'Add LocalBusiness schema first, then include geo coordinates',
+                });
+            }
             const geo = ctx.localBusinessSchema.geo;
             if (!geo) {
                 return createResult({ id: 'local-business-geo', name: 'Business Coordinates', category: 'structured-data', severity: 'info' }, 'info', 'LocalBusiness schema missing geo coordinates', {
@@ -192,8 +204,11 @@ export const localRules = [
         severity: 'info',
         description: 'Local business pages should display Name, Address, Phone (NAP)',
         check: (ctx) => {
-            if (!ctx.hasLocalBusinessSignals)
-                return null;
+            if (!ctx.hasLocalBusinessSignals) {
+                return createResult({ id: 'local-nap-presence', name: 'NAP Presence', category: 'content', severity: 'info' }, 'info', 'No local business signals detected', {
+                    recommendation: 'Add NAP information if this is a local business page',
+                });
+            }
             const missing = [];
             if (!ctx.hasPhoneOnPage)
                 missing.push('Phone');
@@ -218,12 +233,18 @@ export const localRules = [
         severity: 'info',
         description: 'Service area businesses should specify their coverage',
         check: (ctx) => {
-            if (!ctx.localBusinessSchema)
-                return null;
+            if (!ctx.localBusinessSchema) {
+                return createResult({ id: 'local-service-area', name: 'Service Area', category: 'structured-data', severity: 'info' }, 'info', 'Not applicable (no LocalBusiness schema)', {
+                    recommendation: 'Add LocalBusiness schema first, then include service area if applicable',
+                });
+            }
             const isServiceArea = ctx.localBusinessSchema['@type'] === 'ServiceAreaBusiness' ||
                 ctx.localBusinessSchema.areaServed !== undefined;
-            if (!isServiceArea)
-                return null;
+            if (!isServiceArea) {
+                return createResult({ id: 'local-service-area', name: 'Service Area', category: 'structured-data', severity: 'info' }, 'info', 'Not a service area business', {
+                    recommendation: 'Add areaServed if you serve customers at their location',
+                });
+            }
             const areaServed = ctx.localBusinessSchema.areaServed;
             if (!areaServed) {
                 return createResult({ id: 'local-service-area', name: 'Service Area', category: 'structured-data', severity: 'info' }, 'info', 'Service area business missing areaServed', {
@@ -247,8 +268,11 @@ export const localRules = [
         severity: 'info',
         description: 'LocalBusiness can include price range for user expectations',
         check: (ctx) => {
-            if (!ctx.localBusinessSchema)
-                return null;
+            if (!ctx.localBusinessSchema) {
+                return createResult({ id: 'local-pricerange', name: 'Price Range', category: 'structured-data', severity: 'info' }, 'info', 'Not applicable (no LocalBusiness schema)', {
+                    recommendation: 'Add LocalBusiness schema first, then include price range',
+                });
+            }
             const priceRange = ctx.localBusinessSchema.priceRange;
             if (!priceRange) {
                 return createResult({ id: 'local-pricerange', name: 'Price Range', category: 'structured-data', severity: 'info' }, 'info', 'LocalBusiness missing price range', {