recker 1.0.43 → 1.0.44

package/dist/seo/rules/pwa.js

@@ -7,8 +7,6 @@ export const pwaRules = [
         severity: 'info',
         description: 'Pages should link to a web app manifest for PWA support',
         check: (ctx) => {
-            if (ctx.hasManifest === undefined)
-                return null;
             if (!ctx.hasManifest) {
                 return createResult({ id: 'pwa-manifest-link', name: 'Web App Manifest', category: 'technical', severity: 'info' }, 'info', 'No web app manifest linked', {
                     recommendation: 'Add a manifest.json for PWA installability',
@@ -29,8 +27,6 @@ export const pwaRules = [
         severity: 'info',
         description: 'Pages should define a theme color for browser UI',
         check: (ctx) => {
-            if (ctx.themeColor === undefined)
-                return null;
             if (!ctx.themeColor) {
                 return createResult({ id: 'pwa-theme-color', name: 'Theme Color', category: 'mobile', severity: 'info' }, 'info', 'No theme-color meta tag', {
                     recommendation: 'Add theme-color for browser UI customization',
@@ -50,8 +46,6 @@ export const pwaRules = [
         severity: 'info',
         description: 'iOS devices need apple-touch-icon for home screen',
         check: (ctx) => {
-            if (ctx.hasAppleTouchIcon === undefined)
-                return null;
             if (!ctx.hasAppleTouchIcon) {
                 return createResult({ id: 'pwa-apple-touch-icon', name: 'Apple Touch Icon', category: 'mobile', severity: 'info' }, 'info', 'No apple-touch-icon found', {
                     recommendation: 'Add apple-touch-icon for iOS home screen',
@@ -72,8 +66,6 @@ export const pwaRules = [
         severity: 'info',
         description: 'Enable standalone mode on iOS devices',
         check: (ctx) => {
-            if (ctx.hasAppleMobileWebAppCapable === undefined)
-                return null;
             if (!ctx.hasAppleMobileWebAppCapable) {
                 return createResult({ id: 'pwa-apple-mobile-capable', name: 'Apple Mobile Web App', category: 'mobile', severity: 'info' }, 'info', 'No apple-mobile-web-app-capable meta tag', {
                     recommendation: 'Add for full-screen iOS experience',
@@ -93,10 +85,11 @@ export const pwaRules = [
         severity: 'info',
         description: 'Configure iOS status bar appearance',
         check: (ctx) => {
-            if (!ctx.hasAppleMobileWebAppCapable)
-                return null;
-            if (ctx.appleStatusBarStyle === undefined)
-                return null;
+            if (!ctx.hasAppleMobileWebAppCapable) {
+                return createResult({ id: 'pwa-apple-status-bar', name: 'Apple Status Bar Style', category: 'mobile', severity: 'info' }, 'info', 'Not applicable (apple-mobile-web-app-capable not set)', {
+                    recommendation: 'First enable apple-mobile-web-app-capable, then configure status bar',
+                });
+            }
             if (!ctx.appleStatusBarStyle) {
                 return createResult({ id: 'pwa-apple-status-bar', name: 'Apple Status Bar Style', category: 'mobile', severity: 'info' }, 'info', 'No apple-mobile-web-app-status-bar-style defined', {
                     recommendation: 'Define status bar style for iOS',
@@ -116,8 +109,6 @@ export const pwaRules = [
         severity: 'info',
         description: 'Manifest should include maskable icons for Android',
         check: (ctx) => {
-            if (ctx.hasMaskableIcon === undefined)
-                return null;
             if (!ctx.hasMaskableIcon) {
                 return createResult({ id: 'pwa-maskable-icon', name: 'Maskable Icon', category: 'mobile', severity: 'info' }, 'info', 'No maskable icon in manifest', {
                     recommendation: 'Add maskable icon for adaptive icons on Android',
@@ -145,10 +136,11 @@ export const pwaRules = [
         severity: 'info',
         description: 'Manifest should define a start_url',
         check: (ctx) => {
-            if (!ctx.hasManifest)
-                return null;
-            if (ctx.manifestStartUrl === undefined)
-                return null;
+            if (!ctx.hasManifest) {
+                return createResult({ id: 'pwa-start-url', name: 'Start URL', category: 'technical', severity: 'info' }, 'info', 'Not applicable (no manifest)', {
+                    recommendation: 'First add a web app manifest, then define start_url',
+                });
+            }
             if (!ctx.manifestStartUrl) {
                 return createResult({ id: 'pwa-start-url', name: 'Start URL', category: 'technical', severity: 'info' }, 'info', 'Manifest missing start_url', {
                     recommendation: 'Define start_url in manifest for PWA launch',
@@ -168,10 +160,11 @@ export const pwaRules = [
         severity: 'info',
         description: 'Manifest should define display mode',
         check: (ctx) => {
-            if (!ctx.hasManifest)
-                return null;
-            if (ctx.manifestDisplay === undefined)
-                return null;
+            if (!ctx.hasManifest) {
+                return createResult({ id: 'pwa-display-mode', name: 'Display Mode', category: 'technical', severity: 'info' }, 'info', 'Not applicable (no manifest)', {
+                    recommendation: 'First add a web app manifest, then define display mode',
+                });
+            }
             const display = ctx.manifestDisplay;
             if (!display) {
                 return createResult({ id: 'pwa-display-mode', name: 'Display Mode', category: 'technical', severity: 'info' }, 'info', 'Manifest missing display mode', {
@@ -198,10 +191,11 @@ export const pwaRules = [
         severity: 'info',
         description: 'Manifest should define navigation scope',
         check: (ctx) => {
-            if (!ctx.hasManifest)
-                return null;
-            if (ctx.manifestScope === undefined)
-                return null;
+            if (!ctx.hasManifest) {
+                return createResult({ id: 'pwa-scope', name: 'Navigation Scope', category: 'technical', severity: 'info' }, 'info', 'Not applicable (no manifest)', {
+                    recommendation: 'First add a web app manifest, then define scope',
+                });
+            }
             if (!ctx.manifestScope) {
                 return createResult({ id: 'pwa-scope', name: 'Navigation Scope', category: 'technical', severity: 'info' }, 'info', 'Manifest missing scope', {
                     recommendation: 'Define scope to control PWA navigation boundaries',
@@ -221,11 +215,12 @@ export const pwaRules = [
         severity: 'info',
         description: 'Manifest should include multiple icon sizes',
         check: (ctx) => {
-            if (!ctx.hasManifest)
-                return null;
-            if (ctx.manifestIconSizes === undefined)
-                return null;
-            const sizes = ctx.manifestIconSizes;
+            if (!ctx.hasManifest) {
+                return createResult({ id: 'pwa-icons-sizes', name: 'Icon Sizes', category: 'mobile', severity: 'info' }, 'info', 'Not applicable (no manifest)', {
+                    recommendation: 'First add a web app manifest with icons',
+                });
+            }
+            const sizes = ctx.manifestIconSizes || [];
             const requiredSizes = [192, 512];
             const missingSizes = requiredSizes.filter(s => !sizes.includes(s));
             if (missingSizes.length > 0) {
@@ -248,10 +243,11 @@ export const pwaRules = [
         severity: 'info',
         description: 'Manifest should have a short_name for home screen',
         check: (ctx) => {
-            if (!ctx.hasManifest)
-                return null;
-            if (ctx.manifestShortName === undefined && ctx.manifestName === undefined)
-                return null;
+            if (!ctx.hasManifest) {
+                return createResult({ id: 'pwa-short-name', name: 'Short Name', category: 'technical', severity: 'info' }, 'info', 'Not applicable (no manifest)', {
+                    recommendation: 'First add a web app manifest with name/short_name',
+                });
+            }
             const shortName = ctx.manifestShortName;
             const name = ctx.manifestName;
             if (!shortName && !name) {
@@ -283,10 +279,11 @@ export const pwaRules = [
         severity: 'info',
         description: 'Manifest should define background_color for splash screen',
         check: (ctx) => {
-            if (!ctx.hasManifest)
-                return null;
-            if (ctx.manifestBackgroundColor === undefined)
-                return null;
+            if (!ctx.hasManifest) {
+                return createResult({ id: 'pwa-background-color', name: 'Background Color', category: 'mobile', severity: 'info' }, 'info', 'Not applicable (no manifest)', {
+                    recommendation: 'First add a web app manifest, then define background_color',
+                });
+            }
             if (!ctx.manifestBackgroundColor) {
                 return createResult({ id: 'pwa-background-color', name: 'Background Color', category: 'mobile', severity: 'info' }, 'info', 'Manifest missing background_color', {
                     recommendation: 'Define background_color for splash screen',

package/dist/seo/rules/readability.js

@@ -7,8 +7,9 @@ export const readabilityRules = [
         severity: 'info',
         description: 'Content should be easy to read for the target audience',
         check: (ctx) => {
-            if (ctx.fleschReadingEase === undefined)
-                return null;
+            if (ctx.fleschReadingEase === undefined) {
+                return createResult({ id: 'readability-flesch-score', name: 'Flesch Reading Ease', category: 'content', severity: 'info' }, 'info', 'Not applicable (Flesch Reading Ease data unavailable)', { recommendation: 'This rule analyzes content readability using the Flesch Reading Ease formula' });
+            }
             const score = ctx.fleschReadingEase;
             let level;
             let status;
@@ -49,8 +50,9 @@ export const readabilityRules = [
         severity: 'info',
         description: 'Sentences should be concise for better readability',
         check: (ctx) => {
-            if (ctx.avgSentenceLength === undefined)
-                return null;
+            if (ctx.avgSentenceLength === undefined) {
+                return createResult({ id: 'readability-sentence-length', name: 'Sentence Length', category: 'content', severity: 'info' }, 'info', 'Not applicable (sentence length data unavailable)', { recommendation: 'This rule checks average sentence length for optimal readability' });
+            }
             const avgLength = ctx.avgSentenceLength;
             if (avgLength > 30) {
                 return createResult({ id: 'readability-sentence-length', name: 'Sentence Length', category: 'content', severity: 'info' }, 'warn', `Average sentence length: ${Math.round(avgLength)} words (too long)`, {
@@ -81,8 +83,9 @@ export const readabilityRules = [
         severity: 'info',
         description: 'Paragraphs should be short for web readability',
         check: (ctx) => {
-            if (ctx.avgParagraphLength === undefined)
-                return null;
+            if (ctx.avgParagraphLength === undefined) {
+                return createResult({ id: 'readability-paragraph-length', name: 'Paragraph Length', category: 'content', severity: 'info' }, 'info', 'Not applicable (paragraph length data unavailable)', { recommendation: 'This rule checks average paragraph length for web readability' });
+            }
             const avgLength = ctx.avgParagraphLength;
             if (avgLength > 150) {
                 return createResult({ id: 'readability-paragraph-length', name: 'Paragraph Length', category: 'content', severity: 'info' }, 'warn', `Average paragraph length: ${Math.round(avgLength)} words (too long)`, {
@@ -109,8 +112,9 @@ export const readabilityRules = [
         severity: 'info',
         description: 'Limit passive voice for clearer writing',
         check: (ctx) => {
-            if (ctx.passiveVoicePercentage === undefined)
-                return null;
+            if (ctx.passiveVoicePercentage === undefined) {
+                return createResult({ id: 'readability-passive-voice', name: 'Passive Voice', category: 'content', severity: 'info' }, 'info', 'Not applicable (passive voice data unavailable)', { recommendation: 'This rule checks passive voice usage for clearer writing' });
+            }
             const percentage = ctx.passiveVoicePercentage;
             if (percentage > 20) {
                 return createResult({ id: 'readability-passive-voice', name: 'Passive Voice', category: 'content', severity: 'info' }, 'warn', `Passive voice: ${Math.round(percentage)}% of sentences`, {
@@ -127,7 +131,7 @@ export const readabilityRules = [
                     recommendation: 'Consider reducing passive voice usage',
                 });
             }
-            return null;
+            return createResult({ id: 'readability-passive-voice', name: 'Passive Voice', category: 'content', severity: 'info' }, 'pass', `Passive voice usage acceptable (${Math.round(percentage)}%)`);
         },
     },
     {
@@ -137,8 +141,9 @@ export const readabilityRules = [
         severity: 'info',
         description: 'Use transition words for better flow',
         check: (ctx) => {
-            if (ctx.transitionWordPercentage === undefined)
-                return null;
+            if (ctx.transitionWordPercentage === undefined) {
+                return createResult({ id: 'readability-transition-words', name: 'Transition Words', category: 'content', severity: 'info' }, 'info', 'Not applicable (transition word data unavailable)', { recommendation: 'This rule checks for transition words that improve content flow' });
+            }
             const percentage = ctx.transitionWordPercentage;
             if (percentage < 20) {
                 return createResult({ id: 'readability-transition-words', name: 'Transition Words', category: 'content', severity: 'info' }, 'info', `Transition words: ${Math.round(percentage)}% of sentences`, {
@@ -161,8 +166,9 @@ export const readabilityRules = [
         severity: 'info',
         description: 'Break content with subheadings every 300 words',
         check: (ctx) => {
-            if (!ctx.wordCount || !ctx.h2Count)
-                return null;
+            if (!ctx.wordCount || !ctx.h2Count) {
+                return createResult({ id: 'readability-subheading-distribution', name: 'Subheading Distribution', category: 'content', severity: 'info' }, 'info', 'Not applicable (word count or heading data unavailable)', { recommendation: 'This rule checks for proper subheading distribution to break up content' });
+            }
             const wordsPerSubheading = ctx.wordCount / (ctx.h2Count + 1);
             if (ctx.wordCount > 500 && wordsPerSubheading > 400) {
                 return createResult({ id: 'readability-subheading-distribution', name: 'Subheading Distribution', category: 'content', severity: 'info' }, 'warn', `${Math.round(wordsPerSubheading)} words between subheadings (too many)`, {
@@ -189,8 +195,9 @@ export const readabilityRules = [
         severity: 'info',
         description: 'Use varied sentence structures and word choices',
         check: (ctx) => {
-            if (ctx.consecutiveSentenceStarts === undefined)
-                return null;
+            if (ctx.consecutiveSentenceStarts === undefined) {
+                return createResult({ id: 'readability-text-variety', name: 'Text Variety', category: 'content', severity: 'info' }, 'info', 'Not applicable (sentence variety data unavailable)', { recommendation: 'This rule checks for varied sentence structures and beginnings' });
+            }
             if (ctx.consecutiveSentenceStarts > 3) {
                 return createResult({ id: 'readability-text-variety', name: 'Text Variety', category: 'content', severity: 'info' }, 'info', `${ctx.consecutiveSentenceStarts} consecutive sentences start similarly`, {
                     recommendation: 'Vary sentence beginnings for better flow',
@@ -199,7 +206,7 @@ export const readabilityRules = [
                     },
                 });
             }
-            return null;
+            return createResult({ id: 'readability-text-variety', name: 'Text Variety', category: 'content', severity: 'info' }, 'pass', 'Good sentence variety');
         },
     },
     {
@@ -209,8 +216,9 @@ export const readabilityRules = [
         severity: 'info',
         description: 'Avoid overly complex vocabulary',
         check: (ctx) => {
-            if (ctx.complexWordPercentage === undefined)
-                return null;
+            if (ctx.complexWordPercentage === undefined) {
+                return createResult({ id: 'readability-word-complexity', name: 'Word Complexity', category: 'content', severity: 'info' }, 'info', 'Not applicable (word complexity data unavailable)', { recommendation: 'This rule checks for complex vocabulary that may reduce readability' });
+            }
             if (ctx.complexWordPercentage > 15) {
                 return createResult({ id: 'readability-word-complexity', name: 'Word Complexity', category: 'content', severity: 'info' }, 'warn', `Complex words: ${Math.round(ctx.complexWordPercentage)}%`, {
                     recommendation: 'Use simpler vocabulary where possible',
@@ -224,7 +232,7 @@ export const readabilityRules = [
             if (ctx.complexWordPercentage > 10) {
                 return createResult({ id: 'readability-word-complexity', name: 'Word Complexity', category: 'content', severity: 'info' }, 'info', `Complex words: ${Math.round(ctx.complexWordPercentage)}%`);
             }
-            return null;
+            return createResult({ id: 'readability-word-complexity', name: 'Word Complexity', category: 'content', severity: 'info' }, 'pass', `Word complexity acceptable (${Math.round(ctx.complexWordPercentage)}%)`);
         },
     },
     {
@@ -234,8 +242,9 @@ export const readabilityRules = [
         severity: 'info',
         description: 'Use lists to improve scannability',
         check: (ctx) => {
-            if (!ctx.wordCount || ctx.listCount === undefined)
-                return null;
+            if (!ctx.wordCount || ctx.listCount === undefined) {
+                return createResult({ id: 'readability-list-usage', name: 'List Usage', category: 'content', severity: 'info' }, 'info', 'Not applicable (word count or list data unavailable)', { recommendation: 'This rule checks for proper use of lists to improve scannability' });
+            }
             if (ctx.wordCount > 500 && ctx.listCount === 0) {
                 return createResult({ id: 'readability-list-usage', name: 'List Usage', category: 'content', severity: 'info' }, 'info', 'No lists found in long-form content', {
                     recommendation: 'Consider using bullet points or numbered lists',
@@ -249,7 +258,7 @@ export const readabilityRules = [
             if (ctx.listCount > 0) {
                 return createResult({ id: 'readability-list-usage', name: 'List Usage', category: 'content', severity: 'info' }, 'pass', `${ctx.listCount} list(s) found`);
             }
-            return null;
+            return createResult({ id: 'readability-list-usage', name: 'List Usage', category: 'content', severity: 'info' }, 'pass', 'Content length does not require lists');
         },
     },
 ];

package/dist/seo/rules/redirects.js

@@ -7,8 +7,9 @@ export const redirectRules = [
         severity: 'warning',
         description: 'Redirect chains should not exceed 3 hops',
         check: (ctx) => {
-            if (ctx.redirectChain === undefined)
-                return null;
+            if (ctx.redirectChain === undefined) {
+                return createResult({ id: 'redirect-chain-length', name: 'Redirect Chain Length', category: 'technical', severity: 'warning' }, 'info', 'Unable to check redirect chain (data unavailable)', { recommendation: 'Ensure redirect analysis completed' });
+            }
             const chainLength = ctx.redirectChain.length;
             if (chainLength === 0) {
                 return createResult({ id: 'redirect-chain-length', name: 'Redirect Chain Length', category: 'technical', severity: 'warning' }, 'pass', 'No redirects detected');
@@ -50,8 +51,9 @@ export const redirectRules = [
         severity: 'error',
         description: 'Pages should not create redirect loops',
         check: (ctx) => {
-            if (!ctx.redirectChain || ctx.redirectChain.length === 0)
-                return null;
+            if (!ctx.redirectChain || ctx.redirectChain.length === 0) {
+                return createResult({ id: 'redirect-loop', name: 'Redirect Loop', category: 'technical', severity: 'error' }, 'pass', 'No redirect chain to check for loops');
+            }
             const urls = ctx.redirectChain.map(r => r.from);
             urls.push(ctx.redirectChain[ctx.redirectChain.length - 1].to);
             const seen = new Set();
@@ -74,7 +76,7 @@ export const redirectRules = [
                     }
                 });
             }
-            return null;
+            return createResult({ id: 'redirect-loop', name: 'Redirect Loop', category: 'technical', severity: 'error' }, 'pass', 'No redirect loop detected');
         },
     },
     {
@@ -84,8 +86,9 @@ export const redirectRules = [
         severity: 'info',
         description: 'Permanent content moves should use 301 redirects',
         check: (ctx) => {
-            if (!ctx.redirectChain || ctx.redirectChain.length === 0)
-                return null;
+            if (!ctx.redirectChain || ctx.redirectChain.length === 0) {
+                return createResult({ id: 'redirect-type', name: 'Redirect Type', category: 'technical', severity: 'info' }, 'info', 'No redirects to analyze', { recommendation: 'Page loaded without redirects' });
+            }
             const temporary = ctx.redirectChain.filter(r => r.status === 302 || r.status === 307);
             const permanent = ctx.redirectChain.filter(r => r.status === 301 || r.status === 308);
             if (temporary.length > 0 && permanent.length === 0) {
@@ -101,7 +104,7 @@ export const redirectRules = [
             if (permanent.length > 0) {
                 return createResult({ id: 'redirect-type', name: 'Redirect Type', category: 'technical', severity: 'info' }, 'pass', 'Using permanent redirect (301)');
             }
-            return null;
+            return createResult({ id: 'redirect-type', name: 'Redirect Type', category: 'technical', severity: 'info' }, 'info', 'Unknown redirect type', { recommendation: 'Check redirect status codes' });
         },
     },
     {
@@ -111,8 +114,9 @@ export const redirectRules = [
         severity: 'warning',
         description: 'Site should redirect consistently to either WWW or non-WWW version',
         check: (ctx) => {
-            if (ctx.wwwConsistency === undefined)
-                return null;
+            if (ctx.wwwConsistency === undefined) {
+                return createResult({ id: 'www-consistency', name: 'WWW vs Non-WWW', category: 'canonicalization', severity: 'warning' }, 'info', 'Unable to check WWW consistency (data unavailable)', { recommendation: 'Ensure WWW/non-WWW analysis completed' });
+            }
             const { canonicalHasWww, urlHasWww, redirectsToCanonical } = ctx.wwwConsistency;
             if (canonicalHasWww !== urlHasWww && !redirectsToCanonical) {
                 return createResult({ id: 'www-consistency', name: 'WWW vs Non-WWW', category: 'canonicalization', severity: 'warning' }, 'warn', 'WWW/non-WWW version accessible without redirect', {
@@ -134,8 +138,9 @@ export const redirectRules = [
         severity: 'warning',
         description: 'HTTP version should redirect to HTTPS',
         check: (ctx) => {
-            if (ctx.httpRedirectsToHttps === undefined)
-                return null;
+            if (ctx.httpRedirectsToHttps === undefined) {
+                return createResult({ id: 'http-to-https-redirect', name: 'HTTP to HTTPS Redirect', category: 'security', severity: 'warning' }, 'info', 'Unable to check HTTP to HTTPS redirect (data unavailable)', { recommendation: 'Ensure HTTPS redirect analysis completed' });
+            }
             if (!ctx.httpRedirectsToHttps) {
                 return createResult({ id: 'http-to-https-redirect', name: 'HTTP to HTTPS Redirect', category: 'security', severity: 'warning' }, 'warn', 'HTTP version does not redirect to HTTPS', {
                     recommendation: 'Configure server to redirect HTTP traffic to HTTPS',
@@ -157,8 +162,9 @@ export const redirectRules = [
         severity: 'info',
         description: 'Detects redirects to different domains',
         check: (ctx) => {
-            if (!ctx.redirectChain || ctx.redirectChain.length === 0)
-                return null;
+            if (!ctx.redirectChain || ctx.redirectChain.length === 0) {
+                return createResult({ id: 'cross-domain-redirect', name: 'Cross-Domain Redirect', category: 'technical', severity: 'info' }, 'info', 'No redirect chain to check', { recommendation: 'Page loaded without redirects' });
+            }
             const crossDomainRedirects = ctx.redirectChain.filter(r => {
                 try {
                     const fromHost = new URL(r.from).hostname;
@@ -178,7 +184,7 @@ export const redirectRules = [
                     }
                 });
             }
-            return null;
+            return createResult({ id: 'cross-domain-redirect', name: 'Cross-Domain Redirect', category: 'technical', severity: 'info' }, 'pass', 'No cross-domain redirects');
         },
     },
 ];