recker 1.0.43 → 1.0.44

package/dist/seo/rules/internal-linking.js

@@ -7,8 +7,9 @@ export const internalLinkingRules = [
         severity: 'warning',
         description: 'Pages should have a healthy number of internal links',
         check: (ctx) => {
-            if (ctx.internalLinks === undefined)
-                return null;
+            if (ctx.internalLinks === undefined) {
+                return createResult({ id: 'linking-internal-count', name: 'Internal Link Count', category: 'links', severity: 'warning' }, 'info', 'Not applicable (internal links data unavailable)', { recommendation: 'This rule checks internal link count to ensure proper site navigation and link equity distribution' });
+            }
             const count = ctx.internalLinks;
             if (count === 0) {
                 return createResult({ id: 'linking-internal-count', name: 'Internal Link Count', category: 'links', severity: 'warning' }, 'warn', 'No internal links found', {
@@ -39,10 +40,12 @@ export const internalLinkingRules = [
         severity: 'info',
         description: 'Pages should have more internal than external links',
         check: (ctx) => {
-            if (ctx.internalLinks === undefined || ctx.externalLinks === undefined)
-                return null;
-            if (ctx.totalLinks === undefined || ctx.totalLinks === 0)
-                return null;
+            if (ctx.internalLinks === undefined || ctx.externalLinks === undefined) {
+                return createResult({ id: 'linking-internal-ratio', name: 'Internal/External Link Ratio', category: 'links', severity: 'info' }, 'info', 'Not applicable (link data unavailable)', { recommendation: 'This rule checks the balance between internal and external links' });
+            }
+            if (ctx.totalLinks === undefined || ctx.totalLinks === 0) {
+                return createResult({ id: 'linking-internal-ratio', name: 'Internal/External Link Ratio', category: 'links', severity: 'info' }, 'info', 'Not applicable (no links detected)', { recommendation: 'This rule requires at least some links to analyze the internal/external ratio' });
+            }
             const internal = ctx.internalLinks;
             const external = ctx.externalLinks;
             const ratio = internal / (external || 1);
@@ -66,11 +69,13 @@ export const internalLinkingRules = [
         severity: 'info',
         description: 'Internal links should use diverse, descriptive anchor text',
         check: (ctx) => {
-            if (!ctx.allLinks || ctx.allLinks.length === 0)
-                return null;
+            if (!ctx.allLinks || ctx.allLinks.length === 0) {
+                return createResult({ id: 'linking-anchor-diversity', name: 'Anchor Text Diversity', category: 'links', severity: 'info' }, 'info', 'Not applicable (no links detected)', { recommendation: 'This rule checks anchor text diversity to prevent over-optimization' });
+            }
             const internalLinks = ctx.allLinks.filter(l => l.type === 'internal');
-            if (internalLinks.length < 3)
-                return null;
+            if (internalLinks.length < 3) {
+                return createResult({ id: 'linking-anchor-diversity', name: 'Anchor Text Diversity', category: 'links', severity: 'info' }, 'info', 'Not applicable (insufficient internal links)', { recommendation: 'This rule requires at least 3 internal links to analyze anchor text diversity' });
+            }
             const anchorCounts = {};
             for (const link of internalLinks) {
                 const anchor = (link.text || '').toLowerCase().trim();
@@ -100,11 +105,13 @@ export const internalLinkingRules = [
         severity: 'info',
         description: 'Pages should link to deep content, not just homepage',
         check: (ctx) => {
-            if (!ctx.allLinks || ctx.allLinks.length === 0)
-                return null;
+            if (!ctx.allLinks || ctx.allLinks.length === 0) {
+                return createResult({ id: 'linking-deep-links', name: 'Deep Linking', category: 'links', severity: 'info' }, 'info', 'Not applicable (no links detected)', { recommendation: 'This rule checks for deep linking patterns to ensure inner pages are properly linked' });
+            }
             const internalLinks = ctx.allLinks.filter(l => l.type === 'internal');
-            if (internalLinks.length === 0)
-                return null;
+            if (internalLinks.length === 0) {
+                return createResult({ id: 'linking-deep-links', name: 'Deep Linking', category: 'links', severity: 'info' }, 'info', 'Not applicable (no internal links detected)', { recommendation: 'This rule requires internal links to analyze deep linking patterns' });
+            }
             let rootLinks = 0;
             let deepLinks = 0;
             for (const link of internalLinks) {
@@ -142,10 +149,12 @@ export const internalLinkingRules = [
         severity: 'info',
         description: 'Check for proper navigation link structure',
         check: (ctx) => {
-            if (!ctx.hasNav)
-                return null;
-            if (ctx.navLinkCount === undefined)
-                return null;
+            if (!ctx.hasNav) {
+                return createResult({ id: 'linking-nav-links', name: 'Navigation Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (no navigation element detected)', { recommendation: 'This rule checks navigation link structure when a <nav> element is present' });
+            }
+            if (ctx.navLinkCount === undefined) {
+                return createResult({ id: 'linking-nav-links', name: 'Navigation Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (navigation link count unavailable)', { recommendation: 'This rule checks the number of links in navigation elements' });
+            }
             if (ctx.navLinkCount === 0) {
                 return createResult({ id: 'linking-nav-links', name: 'Navigation Links', category: 'links', severity: 'info' }, 'warn', 'Navigation element has no links', {
                     recommendation: 'Add links to navigation for user experience and SEO',
@@ -175,10 +184,12 @@ export const internalLinkingRules = [
         severity: 'info',
         description: 'Footer should contain important site-wide links',
         check: (ctx) => {
-            if (!ctx.hasFooter)
-                return null;
-            if (ctx.footerLinkCount === undefined)
-                return null;
+            if (!ctx.hasFooter) {
+                return createResult({ id: 'linking-footer-links', name: 'Footer Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (no footer element detected)', { recommendation: 'This rule checks footer link structure when a <footer> element is present' });
+            }
+            if (ctx.footerLinkCount === undefined) {
+                return createResult({ id: 'linking-footer-links', name: 'Footer Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (footer link count unavailable)', { recommendation: 'This rule checks the number of links in footer elements' });
+            }
             if (ctx.footerLinkCount === 0) {
                 return createResult({ id: 'linking-footer-links', name: 'Footer Links', category: 'links', severity: 'info' }, 'info', 'Footer has no links', {
                     recommendation: 'Add important links to footer',
@@ -206,10 +217,12 @@ export const internalLinkingRules = [
         severity: 'info',
         description: 'Check for in-content contextual links',
         check: (ctx) => {
-            if (ctx.contextualLinkCount === undefined)
-                return null;
-            if (!ctx.wordCount || ctx.wordCount < 300)
-                return null;
+            if (ctx.contextualLinkCount === undefined) {
+                return createResult({ id: 'linking-contextual', name: 'Contextual Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (contextual link count unavailable)', { recommendation: 'This rule checks for in-content contextual links that pass more link equity' });
+            }
+            if (!ctx.wordCount || ctx.wordCount < 300) {
+                return createResult({ id: 'linking-contextual', name: 'Contextual Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (insufficient content for contextual link analysis)', { recommendation: 'This rule requires at least 300 words to analyze contextual link patterns' });
+            }
             const count = ctx.contextualLinkCount;
             if (count === 0) {
                 return createResult({ id: 'linking-contextual', name: 'Contextual Links', category: 'links', severity: 'info' }, 'info', 'No contextual links in content', {
@@ -240,8 +253,9 @@ export const internalLinkingRules = [
         severity: 'warning',
         description: 'Pages should be linked from other pages on the site',
         check: (ctx) => {
-            if (ctx.incomingInternalLinks === undefined)
-                return null;
+            if (ctx.incomingInternalLinks === undefined) {
+                return createResult({ id: 'linking-orphan-page', name: 'Orphan Page Detection', category: 'links', severity: 'warning' }, 'info', 'Not applicable (incoming internal links data unavailable)', { recommendation: 'This rule checks if pages receive incoming internal links to prevent orphan pages' });
+            }
             if (ctx.incomingInternalLinks === 0) {
                 return createResult({ id: 'linking-orphan-page', name: 'Orphan Page Detection', category: 'links', severity: 'warning' }, 'warn', 'Page may be an orphan (no incoming internal links)', {
                     recommendation: 'Link to this page from other pages on your site',
@@ -262,8 +276,9 @@ export const internalLinkingRules = [
         severity: 'info',
         description: 'Avoid excessive self-referencing links',
         check: (ctx) => {
-            if (ctx.selfReferencingLinks === undefined)
-                return null;
+            if (ctx.selfReferencingLinks === undefined) {
+                return createResult({ id: 'linking-self-referencing', name: 'Self-Referencing Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (self-referencing links data unavailable)', { recommendation: 'This rule checks for excessive self-referencing links that waste crawl budget' });
+            }
             if (ctx.selfReferencingLinks > 3) {
                 return createResult({ id: 'linking-self-referencing', name: 'Self-Referencing Links', category: 'links', severity: 'info' }, 'info', `${ctx.selfReferencingLinks} self-referencing links`, {
                     recommendation: 'Reduce links that point to the current page',
@@ -274,7 +289,7 @@ export const internalLinkingRules = [
                     },
                 });
             }
-            return null;
+            return createResult({ id: 'linking-self-referencing', name: 'Self-Referencing Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (acceptable self-referencing link count)', { recommendation: 'Self-referencing links are acceptable at current level (0-3 links)' });
         },
     },
     {
@@ -284,8 +299,9 @@ export const internalLinkingRules = [
         severity: 'error',
         description: 'Internal links should not be broken',
         check: (ctx) => {
-            if (ctx.brokenInternalLinks === undefined)
-                return null;
+            if (ctx.brokenInternalLinks === undefined) {
+                return createResult({ id: 'linking-broken-internal', name: 'Broken Internal Links', category: 'links', severity: 'error' }, 'info', 'Not applicable (broken internal links data unavailable)', { recommendation: 'This rule checks for broken internal links that harm user experience and waste crawl budget' });
+            }
             if (ctx.brokenInternalLinks.length > 0) {
                 return createResult({ id: 'linking-broken-internal', name: 'Broken Internal Links', category: 'links', severity: 'error' }, 'fail', `${ctx.brokenInternalLinks.length} broken internal link(s)`, {
                     recommendation: 'Fix or remove broken internal links',
@@ -306,8 +322,9 @@ export const internalLinkingRules = [
         severity: 'warning',
         description: 'Internal links should not go through redirect chains',
         check: (ctx) => {
-            if (ctx.redirectChainLinks === undefined)
-                return null;
+            if (ctx.redirectChainLinks === undefined) {
+                return createResult({ id: 'linking-redirect-chains', name: 'Redirect Chains', category: 'links', severity: 'warning' }, 'info', 'Not applicable (redirect chain data unavailable)', { recommendation: 'This rule checks for redirect chains that slow crawling and lose link equity' });
+            }
             if (ctx.redirectChainLinks.length > 0) {
                 return createResult({ id: 'linking-redirect-chains', name: 'Redirect Chains', category: 'links', severity: 'warning' }, 'warn', `${ctx.redirectChainLinks.length} link(s) go through redirects`, {
                     recommendation: 'Update links to point to final destination URLs',
@@ -328,8 +345,9 @@ export const internalLinkingRules = [
         severity: 'warning',
         description: 'Internal links should not use nofollow',
         check: (ctx) => {
-            if (!ctx.allLinks)
-                return null;
+            if (!ctx.allLinks) {
+                return createResult({ id: 'linking-nofollow-internal', name: 'Nofollow Internal Links', category: 'links', severity: 'warning' }, 'info', 'Not applicable (links data unavailable)', { recommendation: 'This rule checks for nofollow attributes on internal links which waste PageRank' });
+            }
             const nofollowInternal = ctx.allLinks.filter(l => l.type === 'internal' && l.rel?.includes('nofollow'));
             if (nofollowInternal.length > 0) {
                 return createResult({ id: 'linking-nofollow-internal', name: 'Nofollow Internal Links', category: 'links', severity: 'warning' }, 'warn', `${nofollowInternal.length} internal link(s) have nofollow`, {
@@ -351,8 +369,9 @@ export const internalLinkingRules = [
         severity: 'info',
         description: 'Important pages should be reachable in few clicks',
         check: (ctx) => {
-            if (ctx.pageClickDepth === undefined)
-                return null;
+            if (ctx.pageClickDepth === undefined) {
+                return createResult({ id: 'linking-click-depth', name: 'Click Depth', category: 'links', severity: 'info' }, 'info', 'Not applicable (click depth data unavailable)', { recommendation: 'This rule checks how many clicks from homepage, affecting crawl priority and link equity' });
+            }
             const depth = ctx.pageClickDepth;
             if (depth > 4) {
                 return createResult({ id: 'linking-click-depth', name: 'Click Depth', category: 'links', severity: 'info' }, 'warn', `Page is ${depth} clicks from homepage`, {

package/dist/seo/rules/links.js

@@ -36,8 +36,9 @@ export const linkRules = [
         severity: 'warning',
         description: 'Links should have descriptive anchor text',
         check: (ctx) => {
-            if (ctx.totalLinks === undefined || ctx.totalLinks === 0)
-                return null;
+            if (ctx.totalLinks === undefined || ctx.totalLinks === 0) {
+                return createResult({ id: 'links-descriptive-text', name: 'Link Text', category: 'links', severity: 'warning' }, 'info', 'Not applicable (no links detected on page)', { recommendation: 'This rule checks for descriptive anchor text on all links when links are present' });
+            }
             const withoutText = ctx.problematicLinks?.withoutText ?? [];
             if (withoutText.length > 0) {
                 const examples = withoutText.slice(0, 3).map((l) => l.href).join(', ');
@@ -73,7 +74,7 @@ export const linkRules = [
                     },
                 });
             }
-            return null;
+            return createResult({ id: 'links-generic-text', name: 'Generic Link Text', category: 'links', severity: 'warning' }, 'pass', 'All links have descriptive anchor text');
         },
     },
     {
@@ -83,8 +84,9 @@ export const linkRules = [
         severity: 'info',
         description: 'Page should have at least 3 internal links',
         check: (ctx) => {
-            if (ctx.internalLinks === undefined)
-                return null;
+            if (ctx.internalLinks === undefined) {
+                return createResult({ id: 'links-internal-count', name: 'Internal Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (internal links data unavailable)', { recommendation: 'This rule checks for internal linking structure when link data is available' });
+            }
             const min = SEO_THRESHOLDS.links.minInternal;
             if (ctx.internalLinks < min) {
                 return createResult({ id: 'links-internal-count', name: 'Internal Links', category: 'links', severity: 'info' }, 'info', `Few internal links (${ctx.internalLinks})`, { value: ctx.internalLinks, recommendation: `Add at least ${min} internal links for better navigation` });
@@ -99,13 +101,22 @@ export const linkRules = [
         severity: 'info',
         description: 'Page should not have too many external links',
         check: (ctx) => {
-            if (ctx.externalLinks === undefined)
-                return null;
+            if (ctx.externalLinks === undefined) {
+                return createResult({ id: 'links-external-count', name: 'External Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (external links data unavailable)', { recommendation: 'This rule checks external link count when link data is available' });
+            }
             const max = SEO_THRESHOLDS.links.maxExternal;
             if (ctx.externalLinks > max) {
-                return createResult({ id: 'links-external-count', name: 'External Links', category: 'links', severity: 'info' }, 'warn', `Too many external links (${ctx.externalLinks})`, { value: ctx.externalLinks, recommendation: `Reduce external links to under ${max}` });
+                return createResult({ id: 'links-external-count', name: 'External Links', category: 'links', severity: 'info' }, 'warn', `Too many external links (${ctx.externalLinks})`, {
+                    value: ctx.externalLinks,
+                    recommendation: `Reduce external links to under ${max}`,
+                    evidence: {
+                        found: `${ctx.externalLinks} external links`,
+                        expected: `Less than ${max} external links`,
+                        impact: 'Too many external links can dilute page authority and may appear spammy',
+                    },
+                });
             }
-            return null;
+            return createResult({ id: 'links-external-count', name: 'External Links', category: 'links', severity: 'info' }, 'pass', `External links count is acceptable (${ctx.externalLinks} links)`, { value: ctx.externalLinks });
         },
     },
     {
@@ -128,7 +139,7 @@ export const linkRules = [
                     },
                 });
             }
-            return null;
+            return createResult({ id: 'links-external-noopener', name: 'External Links Noopener', category: 'security', severity: 'warning' }, 'pass', 'All external links with target="_blank" have proper security attributes');
         },
     },
     {
@@ -150,7 +161,7 @@ export const linkRules = [
                     },
                 });
             }
-            return null;
+            return createResult({ id: 'links-external-noreferrer', name: 'External Links Noreferrer', category: 'security', severity: 'info' }, 'pass', 'External links have appropriate privacy attributes');
         },
     },
     {
@@ -160,8 +171,9 @@ export const linkRules = [
         severity: 'info',
         description: 'Rel attributes `sponsored` and `ugc` should be used for paid or user-generated content links.',
         check: (ctx) => {
-            if (!ctx.totalLinks)
-                return null;
+            if (!ctx.totalLinks) {
+                return createResult({ id: 'links-sponsored-ugc-directives', name: 'Sponsored/UGC Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (no links detected on page)', { recommendation: 'This rule checks for proper rel="sponsored" and rel="ugc" attributes when links are present' });
+            }
             let messages = [];
             if (ctx.sponsoredLinks && ctx.sponsoredLinks > 0) {
                 messages.push(`${ctx.sponsoredLinks} link(s) with rel="sponsored".`);
@@ -172,7 +184,7 @@ export const linkRules = [
             if (messages.length > 0) {
                 return createResult({ id: 'links-sponsored-ugc-directives', name: 'Sponsored/UGC Links', category: 'links', severity: 'info' }, 'info', messages.join(' '), { recommendation: 'Ensure rel="sponsored" is used for paid links and rel="ugc" for user-generated content.' });
             }
-            return null;
+            return createResult({ id: 'links-sponsored-ugc-directives', name: 'Sponsored/UGC Links', category: 'links', severity: 'info' }, 'pass', 'No sponsored or UGC links detected');
         },
     },
     {
@@ -182,8 +194,9 @@ export const linkRules = [
         severity: 'warning',
         description: 'Anchor text should describe the link destination',
         check: (ctx) => {
-            if (!ctx.allLinks || ctx.allLinks.length === 0)
-                return null;
+            if (!ctx.allLinks || ctx.allLinks.length === 0) {
+                return createResult({ id: 'links-anchor-text-non-descriptive', name: 'Non-Descriptive Anchor Text', category: 'links', severity: 'warning' }, 'info', 'Not applicable (no links data available)', { recommendation: 'This rule checks for descriptive anchor text on all links when link data is available' });
+            }
             const nonDescriptive = ctx.allLinks.filter((link) => link.text && isGenericAnchorText(link.text));
             if (nonDescriptive.length > 0) {
                 return createResult({ id: 'links-anchor-text-non-descriptive', name: 'Non-Descriptive Anchor Text', category: 'links', severity: 'warning' }, 'warn', `${nonDescriptive.length} link(s) with non-descriptive anchor text`, {
@@ -196,7 +209,7 @@ export const linkRules = [
                     },
                 });
             }
-            return null;
+            return createResult({ id: 'links-anchor-text-non-descriptive', name: 'Non-Descriptive Anchor Text', category: 'links', severity: 'warning' }, 'pass', 'All links have descriptive anchor text');
         },
     },
     {
@@ -206,8 +219,9 @@ export const linkRules = [
         severity: 'info',
         description: 'Pages should not link to themselves excessively',
         check: (ctx) => {
-            if (!ctx.selfReferencingLinks)
-                return null;
+            if (!ctx.selfReferencingLinks) {
+                return createResult({ id: 'links-self-referencing', name: 'Self-Referencing Links', category: 'links', severity: 'info' }, 'info', 'Not applicable (self-referencing links data unavailable)', { recommendation: 'This rule checks for excessive self-referencing links when link data is available' });
+            }
             if (ctx.selfReferencingLinks > 3) {
                 return createResult({ id: 'links-self-referencing', name: 'Self-Referencing Links', category: 'links', severity: 'info' }, 'info', `${ctx.selfReferencingLinks} self-referencing link(s)`, {
                     value: ctx.selfReferencingLinks,
@@ -217,7 +231,7 @@ export const linkRules = [
                     },
                 });
             }
-            return null;
+            return createResult({ id: 'links-self-referencing', name: 'Self-Referencing Links', category: 'links', severity: 'info' }, 'pass', `Self-referencing links are acceptable (${ctx.selfReferencingLinks} found)`, { value: ctx.selfReferencingLinks });
         },
     },
     {
@@ -227,8 +241,9 @@ export const linkRules = [
         severity: 'error',
         description: 'Internal links should not return 4xx errors',
         check: (ctx) => {
-            if (!ctx.brokenInternalLinks || ctx.brokenInternalLinks.length === 0)
-                return null;
+            if (!ctx.brokenInternalLinks || ctx.brokenInternalLinks.length === 0) {
+                return createResult({ id: 'links-broken-internal', name: 'Broken Internal Links', category: 'links', severity: 'error' }, 'pass', 'No broken internal links detected');
+            }
             return createResult({ id: 'links-broken-internal', name: 'Broken Internal Links', category: 'links', severity: 'error' }, 'fail', `${ctx.brokenInternalLinks.length} broken internal link(s)`, {
                 value: ctx.brokenInternalLinks.length,
                 recommendation: 'Fix or remove broken internal links',
@@ -246,8 +261,9 @@ export const linkRules = [
         severity: 'warning',
         description: 'External links should not return 4xx errors',
         check: (ctx) => {
-            if (!ctx.brokenExternalLinks || ctx.brokenExternalLinks.length === 0)
-                return null;
+            if (!ctx.brokenExternalLinks || ctx.brokenExternalLinks.length === 0) {
+                return createResult({ id: 'links-broken-external', name: 'Broken External Links', category: 'links', severity: 'warning' }, 'pass', 'No broken external links detected');
+            }
             return createResult({ id: 'links-broken-external', name: 'Broken External Links', category: 'links', severity: 'warning' }, 'warn', `${ctx.brokenExternalLinks.length} broken external link(s)`, {
                 value: ctx.brokenExternalLinks.length,
                 recommendation: 'Update or remove broken external links',
@@ -265,8 +281,9 @@ export const linkRules = [
         severity: 'warning',
         description: 'Internal links should not point to redirect chains',
         check: (ctx) => {
-            if (!ctx.redirectChainLinks || ctx.redirectChainLinks.length === 0)
-                return null;
+            if (!ctx.redirectChainLinks || ctx.redirectChainLinks.length === 0) {
+                return createResult({ id: 'links-redirect-chains', name: 'Redirect Chain Links', category: 'links', severity: 'warning' }, 'pass', 'No redirect chain links detected');
+            }
             return createResult({ id: 'links-redirect-chains', name: 'Redirect Chain Links', category: 'links', severity: 'warning' }, 'warn', `${ctx.redirectChainLinks.length} link(s) with redirect chains`, {
                 value: ctx.redirectChainLinks.length,
                 recommendation: 'Update links to point directly to final destination',
@@ -284,8 +301,9 @@ export const linkRules = [
         severity: 'warning',
         description: 'Pages should have at least 3 internal links',
         check: (ctx) => {
-            if (ctx.internalLinks === undefined)
-                return null;
+            if (ctx.internalLinks === undefined) {
+                return createResult({ id: 'links-few-internal', name: 'Pages with Few Internal Links', category: 'links', severity: 'warning' }, 'info', 'Not applicable (internal links data unavailable)', { recommendation: 'This rule checks for sufficient internal linking when link data is available' });
+            }
             if (ctx.internalLinks < 3) {
                 return createResult({ id: 'links-few-internal', name: 'Pages with Few Internal Links', category: 'links', severity: 'warning' }, 'warn', `Only ${ctx.internalLinks} internal link(s)`, {
                     value: ctx.internalLinks,
@@ -297,7 +315,7 @@ export const linkRules = [
                     },
                 });
             }
-            return null;
+            return createResult({ id: 'links-few-internal', name: 'Pages with Few Internal Links', category: 'links', severity: 'warning' }, 'pass', `Good internal linking (${ctx.internalLinks} internal links)`, { value: ctx.internalLinks });
         },
     },
     {
@@ -307,10 +325,12 @@ export const linkRules = [
         severity: 'warning',
         description: 'Pages should have incoming internal links',
         check: (ctx) => {
-            if (ctx.incomingInternalLinks === undefined)
-                return null;
-            if (ctx.isStartPage)
-                return null;
+            if (ctx.incomingInternalLinks === undefined) {
+                return createResult({ id: 'links-orphan-page', name: 'Orphan Page Detection', category: 'links', severity: 'warning' }, 'info', 'Not applicable (incoming links data unavailable)', { recommendation: 'This rule checks for orphan pages when incoming link data is available' });
+            }
+            if (ctx.isStartPage) {
+                return createResult({ id: 'links-orphan-page', name: 'Orphan Page Detection', category: 'links', severity: 'warning' }, 'info', 'Not applicable (start page does not require incoming links)', { recommendation: 'This rule checks non-homepage pages for incoming internal links' });
+            }
             if (ctx.incomingInternalLinks === 0) {
                 return createResult({ id: 'links-orphan-page', name: 'Orphan Page Detection', category: 'links', severity: 'warning' }, 'warn', 'Page has no incoming internal links (orphan)', {
                     value: 0,
@@ -320,7 +340,7 @@ export const linkRules = [
                     },
                 });
             }
-            return null;
+            return createResult({ id: 'links-orphan-page', name: 'Orphan Page Detection', category: 'links', severity: 'warning' }, 'pass', `Page has incoming internal links (${ctx.incomingInternalLinks} links)`, { value: ctx.incomingInternalLinks });
         },
     },
     {
@@ -330,8 +350,9 @@ export const linkRules = [
         severity: 'warning',
         description: 'Important pages should be within 3 clicks from homepage',
         check: (ctx) => {
-            if (ctx.clickDepth === undefined)
-                return null;
+            if (ctx.clickDepth === undefined) {
+                return createResult({ id: 'links-click-depth', name: 'Page Click Depth', category: 'links', severity: 'warning' }, 'info', 'Not applicable (click depth data unavailable)', { recommendation: 'This rule checks page depth from homepage when crawl data is available' });
+            }
             if (ctx.clickDepth > 3) {
                 return createResult({ id: 'links-click-depth', name: 'Page Click Depth', category: 'links', severity: 'warning' }, 'warn', `Page is ${ctx.clickDepth} clicks from homepage`, {
                     value: ctx.clickDepth,
@@ -353,10 +374,12 @@ export const linkRules = [
         severity: 'info',
         description: 'Pages should have more internal than external links',
         check: (ctx) => {
-            if (ctx.internalLinks === undefined || ctx.externalLinks === undefined)
-                return null;
-            if (ctx.totalLinks === undefined || ctx.totalLinks === 0)
-                return null;
+            if (ctx.internalLinks === undefined || ctx.externalLinks === undefined) {
+                return createResult({ id: 'links-external-ratio', name: 'External to Internal Link Ratio', category: 'links', severity: 'info' }, 'info', 'Not applicable (link ratio data unavailable)', { recommendation: 'This rule checks external to internal link ratio when link data is available' });
+            }
+            if (ctx.totalLinks === undefined || ctx.totalLinks === 0) {
+                return createResult({ id: 'links-external-ratio', name: 'External to Internal Link Ratio', category: 'links', severity: 'info' }, 'info', 'Not applicable (no links detected on page)', { recommendation: 'This rule checks external to internal link ratio when links are present' });
+            }
             const externalRatio = (ctx.externalLinks / ctx.totalLinks) * 100;
             if (externalRatio > 70) {
                 return createResult({ id: 'links-external-ratio', name: 'External to Internal Link Ratio', category: 'links', severity: 'info' }, 'info', `High external link ratio (${Math.round(externalRatio)}%)`, {
@@ -369,7 +392,7 @@ export const linkRules = [
                     },
                 });
             }
-            return null;
+            return createResult({ id: 'links-external-ratio', name: 'External to Internal Link Ratio', category: 'links', severity: 'info' }, 'pass', `Good link distribution (${Math.round(externalRatio)}% external)`, { value: Math.round(externalRatio) });
         },
     },
     {
@@ -379,8 +402,9 @@ export const linkRules = [
         severity: 'info',
         description: 'Internal links with nofollow prevent PageRank flow',
         check: (ctx) => {
-            if (!ctx.nofollowInternalLinks || ctx.nofollowInternalLinks === 0)
-                return null;
+            if (!ctx.nofollowInternalLinks || ctx.nofollowInternalLinks === 0) {
+                return createResult({ id: 'links-nofollow-internal', name: 'Internal Nofollow Links', category: 'links', severity: 'info' }, 'pass', 'No internal nofollow links detected');
+            }
             return createResult({ id: 'links-nofollow-internal', name: 'Internal Nofollow Links', category: 'links', severity: 'info' }, 'info', `${ctx.nofollowInternalLinks} internal link(s) with nofollow`, {
                 value: ctx.nofollowInternalLinks,
                 recommendation: 'Remove nofollow from internal links unless intentional',
@@ -397,8 +421,9 @@ export const linkRules = [
         severity: 'warning',
         description: 'Pages should not have more than 3,000 links',
         check: (ctx) => {
-            if (ctx.totalLinks === undefined)
-                return null;
+            if (ctx.totalLinks === undefined) {
+                return createResult({ id: 'excessive-links', name: 'Excessive Links on Page', category: 'links', severity: 'warning' }, 'info', 'Not applicable (total links data unavailable)', { recommendation: 'This rule checks for excessive links (>3,000) when link data is available' });
+            }
             if (ctx.totalLinks > 3000) {
                 return createResult({ id: 'excessive-links', name: 'Excessive Links on Page', category: 'links', severity: 'warning' }, 'fail', `Page has ${ctx.totalLinks} links (exceeds 3,000 limit)`, {
                     value: ctx.totalLinks,
@@ -421,7 +446,7 @@ export const linkRules = [
                     }
                 });
             }
-            return null;
+            return createResult({ id: 'excessive-links', name: 'Excessive Links on Page', category: 'links', severity: 'warning' }, 'pass', `Links count is acceptable (${ctx.totalLinks} links)`, { value: ctx.totalLinks });
         },
     },
     {
@@ -431,8 +456,9 @@ export const linkRules = [
         severity: 'info',
         description: 'Links should point to pages, not raw resources like images',
         check: (ctx) => {
-            if (ctx.linksToResources === undefined)
-                return null;
+            if (ctx.linksToResources === undefined) {
+                return createResult({ id: 'links-to-resources', name: 'Links to Resources', category: 'links', severity: 'info' }, 'info', 'Not applicable (resource links data unavailable)', { recommendation: 'This rule checks for links pointing to raw resources (images, PDFs) when link data is available' });
+            }
             if (ctx.linksToResources > 0) {
                 return createResult({ id: 'links-to-resources', name: 'Links to Resources', category: 'links', severity: 'info' }, 'info', `${ctx.linksToResources} links point directly to resources (images, PDFs)`, {
                     value: ctx.linksToResources,
@@ -443,7 +469,7 @@ export const linkRules = [
                     }
                 });
             }
-            return null;
+            return createResult({ id: 'links-to-resources', name: 'Links to Resources', category: 'links', severity: 'info' }, 'pass', 'No direct links to raw resources detected');
         },
     },
     {
@@ -453,8 +479,9 @@ export const linkRules = [
         severity: 'warning',
         description: 'External links should not return 403 Forbidden',
         check: (ctx) => {
-            if (ctx.forbidden403Links === undefined)
-                return null;
+            if (ctx.forbidden403Links === undefined) {
+                return createResult({ id: 'links-403-forbidden', name: '403 Forbidden Links', category: 'links', severity: 'warning' }, 'info', 'Not applicable (forbidden links data unavailable)', { recommendation: 'This rule checks for external links returning 403 Forbidden when link validation data is available' });
+            }
             if (ctx.forbidden403Links > 0) {
                 return createResult({ id: 'links-403-forbidden', name: '403 Forbidden Links', category: 'links', severity: 'warning' }, 'warn', `${ctx.forbidden403Links} external links return 403 Forbidden`, {
                     value: ctx.forbidden403Links,
@@ -465,7 +492,7 @@ export const linkRules = [
                     }
                 });
             }
-            return null;
+            return createResult({ id: 'links-403-forbidden', name: '403 Forbidden Links', category: 'links', severity: 'warning' }, 'pass', 'No forbidden (403) links detected');
         },
     },
 ];