pagesight 0.14.3 → 0.15.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pagesight",
-  "version": "0.14.3",
+  "version": "0.15.1",
   "description": "See your site the way search engines and AI see it.",
   "keywords": [
     "seo",

package/src/tools/ai.ts

@@ -8,6 +8,9 @@ interface LlmsTxtResult {
   exists: boolean;
   size: number | null;
   firstLine: string | null;
+  lineCount: number | null;
+  sections: string[];
+  linkCount: number | null;
 }
 
 async function checkLlmsTxt(origin: string, path: string): Promise<LlmsTxtResult> {
@@ -20,24 +23,37 @@ async function checkLlmsTxt(origin: string, path: string): Promise<LlmsTxtResult
       signal: controller.signal,
     });
     clearTimeout(timeout);
-    if (!res.ok) return { exists: false, size: null, firstLine: null };
+    if (!res.ok) return { exists: false, size: null, firstLine: null, lineCount: null, sections: [], linkCount: null };
     // Cap at 1MB to avoid OOM on large responses
     const contentLength = Number(res.headers.get("content-length") ?? 0);
     if (contentLength > 1_048_576) {
-      return { exists: true, size: contentLength, firstLine: "(file too large to preview)" };
+      return {
+        exists: true,
+        size: contentLength,
+        firstLine: "(file too large to preview)",
+        lineCount: null,
+        sections: [],
+        linkCount: null,
+      };
     }
     const text = await res.text();
     if (text.length > 1_048_576) {
-      return { exists: true, size: text.length, firstLine: "(file too large to preview)" };
+      return {
+        exists: true,
+        size: text.length,
+        firstLine: "(file too large to preview)",
+        lineCount: null,
+        sections: [],
+        linkCount: null,
+      };
     }
-    const firstLine =
-      text
-        .split("\n")
-        .find((l) => l.trim().length > 0)
-        ?.trim() ?? null;
-    return { exists: true, size: text.length, firstLine };
+    const textLines = text.split("\n");
+    const firstLine = textLines.find((l) => l.trim().length > 0)?.trim() ?? null;
+    const sections = textLines.filter((l) => /^##\s/.test(l)).map((l) => l.replace(/^##\s+/, "").trim());
+    const linkCount = (text.match(/\[.*?\]\(https?:\/\/[^)]+\)/g) ?? []).length;
+    return { exists: true, size: text.length, firstLine, lineCount: textLines.length, sections, linkCount };
   } catch {
-    return { exists: false, size: null, firstLine: null };
+    return { exists: false, size: null, firstLine: null, lineCount: null, sections: [], linkCount: null };
   }
 }
 
@@ -46,8 +62,14 @@ function formatLlmsTxt(llmsTxt: LlmsTxtResult, llmsFullTxt: LlmsTxtResult): stri
 
   if (llmsTxt.exists) {
     const sizeKB = llmsTxt.size ? `${(llmsTxt.size / 1024).toFixed(1)} KB` : "unknown size";
-    lines.push(`llms.txt: FOUND (${sizeKB})`);
+    const details = [sizeKB];
+    if (llmsTxt.lineCount) details.push(`${llmsTxt.lineCount} lines`);
+    if (llmsTxt.linkCount) details.push(`${llmsTxt.linkCount} links`);
+    lines.push(`llms.txt: FOUND (${details.join(", ")})`);
     if (llmsTxt.firstLine) lines.push(`  ${llmsTxt.firstLine}`);
+    if (llmsTxt.sections.length > 0) {
+      lines.push(`  Sections: ${llmsTxt.sections.join(", ")}`);
+    }
   } else {
     lines.push("llms.txt: NOT FOUND — consider adding one for AI-friendly documentation");
   }

package/src/tools/audit.ts

@@ -382,6 +382,21 @@ export function registerAuditTool(server: McpServer): void {
       if (inspectResult.status === "fulfilled" && inspectResult.value) {
         const idx = inspectResult.value.indexStatusResult;
         addInspectFindings(idx.verdict, idx.coverageState, findings);
+
+        // Rich Results check
+        const richResults = inspectResult.value.richResultsResult;
+        if (richResults && richResults.verdict === "FAIL") {
+          const failingTypes = (richResults.detectedItems ?? [])
+            .filter((item) => item.items?.some((i) => (i.issues?.length ?? 0) > 0))
+            .map((item) => item.richResultType);
+          if (failingTypes.length > 0) {
+            findings.push({
+              severity: "MEDIUM",
+              message: `Rich Results failing for: ${failingTypes.join(", ")}`,
+              source: "inspect",
+            });
+          }
+        }
       } else if (inspectResult.status === "rejected") {
         errors.push(`Inspect: ${inspectResult.reason}`);
       }

package/src/tools/page.ts

@@ -924,6 +924,29 @@ export function registerPageTool(server: McpServer): void {
               })
               .filter(Boolean);
 
+            // Run structured data validation for batch summary
+            let jsonLdSummary = "none";
+            if (parsed.jsonLd.length > 0) {
+              const { issues: valIssues } = validateJsonLd(parsed.jsonLd);
+              const requiredMissing = valIssues.filter((i) => i.level === "required").length;
+              if (jsonLdTypes.length > 0) {
+                jsonLdSummary = jsonLdTypes.join(", ");
+                if (requiredMissing > 0) {
+                  jsonLdSummary += ` (${requiredMissing} required missing)`;
+                  issues.push(`${requiredMissing} missing required fields in JSON-LD`);
+                }
+              }
+            }
+
+            // Description with length
+            const descVal = getMeta(parsed.meta, "description");
+            let descSummary: string;
+            if (descVal) {
+              descSummary = descVal.length > 155 ? `yes (${descVal.length} chars ⚠)` : `yes (${descVal.length})`;
+            } else {
+              descSummary = "no";
+            }
+
             results.push({
               url: finalUrl,
               title: parsed.title
@@ -931,9 +954,9 @@ export function registerPageTool(server: McpServer): void {
                   ? `${parsed.title.slice(0, parsed.title.lastIndexOf(" ", 60) > 20 ? parsed.title.lastIndexOf(" ", 60) : 60)}...`
                   : parsed.title
                 : "(missing)",
-              description: getMeta(parsed.meta, "description") ? "yes" : "no",
+              description: descSummary,
               canonical: parsed.canonical ? "yes" : "no",
-              jsonLd: jsonLdTypes.length > 0 ? jsonLdTypes.join(", ") : "none",
+              jsonLd: jsonLdSummary,
               links: linkSummary,
               issues,
             });

package/src/tools/search.ts

@@ -302,6 +302,10 @@ function formatPerformance(
 
   const totals = computeTotals(rows);
 
+  if (totals.impressions < 10) {
+    lines.push("⚠ Low data volume — trends may not be meaningful.", "");
+  }
+
   lines.push(
     `--- Summary (${rows.length} rows returned) ---`,
     "",
@@ -351,15 +355,35 @@ function formatComparison(
     `Previous: ${previousStart} to ${previousEnd}`,
     `Dimensions: ${dimensions.join(", ")}`,
     "",
-    "--- Summary ---",
-    "",
-    "             Current     Previous    Change",
+  ];
+
+  if (cur.impressions < 10 && prev.impressions < 10) {
+    lines.push("⚠ Low data volume — trends may not be meaningful.", "");
+  }
+
+  lines.push("--- Summary ---", "");
+  lines.push("             Current     Previous    Change");
+  lines.push(
     `Clicks:      ${String(cur.clicks.toLocaleString()).padEnd(12)} ${String(prev.clicks.toLocaleString()).padEnd(12)} ${pctChange(cur.clicks, prev.clicks)}`,
+  );
+  lines.push(
     `Impressions: ${String(cur.impressions.toLocaleString()).padEnd(12)} ${String(prev.impressions.toLocaleString()).padEnd(12)} ${pctChange(cur.impressions, prev.impressions)}`,
-    `Avg CTR:     ${`${(cur.ctr * 100).toFixed(1)}%`.padEnd(12)} ${`${(prev.ctr * 100).toFixed(1)}%`.padEnd(12)} ${((cur.ctr - prev.ctr) * 100).toFixed(1)}pp`,
-    `Avg Position:${String(cur.position.toFixed(1)).padEnd(13)} ${String(prev.position.toFixed(1)).padEnd(12)} ${prev.position === 0 && cur.position > 0 ? "new" : cur.position < prev.position ? "improved" : cur.position > prev.position ? "regressed" : "stable"} (${(cur.position - prev.position).toFixed(1)})`,
-    "",
-  ];
+  );
+  const curCtr = `${(cur.ctr * 100).toFixed(1)}%`;
+  const prevCtr = `${(prev.ctr * 100).toFixed(1)}%`;
+  lines.push(`Avg CTR:     ${curCtr.padEnd(12)} ${prevCtr.padEnd(12)} ${((cur.ctr - prev.ctr) * 100).toFixed(1)}pp`);
+  const posStatus =
+    prev.position === 0 && cur.position > 0
+      ? "new"
+      : cur.position < prev.position
+        ? "improved"
+        : cur.position > prev.position
+          ? "regressed"
+          : "stable";
+  lines.push(
+    `Avg Position:${String(cur.position.toFixed(1)).padEnd(13)} ${String(prev.position.toFixed(1)).padEnd(12)} ${posStatus} (${(cur.position - prev.position).toFixed(1)})`,
+  );
+  lines.push("");
 
   // Build lookup for previous period rows
   const prevMap = new Map<string, (typeof prevRows)[0]>();

package/src/tools/speed.ts

@@ -438,14 +438,17 @@ function formatBatchTable(results: Array<{ url: string; result: PsiResult }>, st
   lines.push("");
 
   // Shared opportunities across pages
-  const oppCounts = new Map<string, { title: string; count: number }>();
+  const oppCounts = new Map<string, { title: string; count: number; maxSavings: string }>();
   for (const { result } of results) {
     for (const id of collectOpportunityIds(result.lighthouseResult.audits)) {
+      const audit = result.lighthouseResult.audits[id];
+      const savings = audit.displayValue ?? "";
       const existing = oppCounts.get(id);
       if (existing) {
         existing.count++;
+        if (savings && (!existing.maxSavings || savings > existing.maxSavings)) existing.maxSavings = savings;
       } else {
-        oppCounts.set(id, { title: result.lighthouseResult.audits[id].title, count: 1 });
+        oppCounts.set(id, { title: audit.title, count: 1, maxSavings: savings });
       }
     }
   }
@@ -453,14 +456,16 @@ function formatBatchTable(results: Array<{ url: string; result: PsiResult }>, st
   const sharedOpps = [...oppCounts.entries()].filter(([, v]) => v.count >= 2).sort((a, b) => b[1].count - a[1].count);
   if (sharedOpps.length > 0) {
     lines.push("--- Shared Opportunities ---", "");
-    for (const [, { title, count }] of sharedOpps.slice(0, 10)) {
-      lines.push(`  ${title} (${count}/${results.length} pages)`);
+    for (const [, { title, count, maxSavings }] of sharedOpps.slice(0, 10)) {
+      const cleaned = maxSavings.replace(/^Est savings of /i, "");
+      const savingsStr = cleaned ? `, up to ${cleaned}` : "";
+      lines.push(`  ${title} (${count}/${results.length} pages${savingsStr})`);
     }
     lines.push("");
   }
 
   // A11y failures deduplicated across pages
-  const a11yFailures = new Map<string, { title: string; count: number; pages: string[] }>();
+  const a11yFailures = new Map<string, { title: string; count: number; pages: string[]; selectors: string[] }>();
   for (const { url, result } of results) {
     const lhr = result.lighthouseResult;
     const a11yCat = lhr.categories.accessibility;
@@ -469,13 +474,20 @@ function formatBatchTable(results: Array<{ url: string; result: PsiResult }>, st
     for (const ref of a11yCat.auditRefs) {
       const audit = lhr.audits[ref.id];
       if (audit && audit.score !== null && audit.score < 1) {
-        const existing = a11yFailures.get(ref.id);
         const page = shortUrl(url, allUrls);
+        const selector = audit.details?.items?.[0]?.node?.selector ?? null;
+        const existing = a11yFailures.get(ref.id);
         if (existing) {
           existing.count++;
           existing.pages.push(page);
+          if (selector && !existing.selectors.includes(selector)) existing.selectors.push(selector);
         } else {
-          a11yFailures.set(ref.id, { title: audit.title, count: 1, pages: [page] });
+          a11yFailures.set(ref.id, {
+            title: audit.title,
+            count: 1,
+            pages: [page],
+            selectors: selector ? [selector] : [],
+          });
         }
       }
     }
@@ -486,8 +498,11 @@ function formatBatchTable(results: Array<{ url: string; result: PsiResult }>, st
     .sort((a, b) => b[1].count - a[1].count);
   if (sharedA11y.length > 0) {
     lines.push("--- Accessibility Issues (shared) ---", "");
-    for (const [, { title, count, pages }] of sharedA11y.slice(0, 10)) {
+    for (const [, { title, count, pages, selectors }] of sharedA11y.slice(0, 10)) {
       lines.push(`  ${title} (${count}/${results.length} pages: ${pages.join(", ")})`);
+      if (selectors.length > 0) {
+        lines.push(`    Elements: ${selectors.slice(0, 3).join(", ")}`);
+      }
     }
     lines.push("");
   }
@@ -601,6 +616,37 @@ function formatCrux(target: string, result: CruxResponse): string {
     }
   }
 
+  // Core Web Vitals assessment (Google ranking signal)
+  const cwvMetrics = r.metrics;
+  const lcp = cwvMetrics.largest_contentful_paint?.percentiles?.p75 as number | undefined;
+  const inp = cwvMetrics.interaction_to_next_paint?.percentiles?.p75 as number | undefined;
+  const cls = cwvMetrics.cumulative_layout_shift?.percentiles?.p75 as number | undefined;
+
+  if (lcp !== undefined || inp !== undefined || cls !== undefined) {
+    lines.push("", "--- Core Web Vitals Assessment ---", "");
+    const lcpPass = lcp !== undefined && lcp <= 2500;
+    const inpPass = inp !== undefined && inp <= 200;
+    const clsPass = cls !== undefined && cls <= 0.1;
+
+    if (lcp !== undefined)
+      lines.push(`LCP: ${lcp}ms ${lcpPass ? "GOOD" : lcp <= 4000 ? "NEEDS IMPROVEMENT" : "POOR"} (threshold: 2500ms)`);
+    if (inp !== undefined)
+      lines.push(`INP: ${inp}ms ${inpPass ? "GOOD" : inp <= 500 ? "NEEDS IMPROVEMENT" : "POOR"} (threshold: 200ms)`);
+    if (cls !== undefined)
+      lines.push(`CLS: ${cls} ${clsPass ? "GOOD" : cls <= 0.25 ? "NEEDS IMPROVEMENT" : "POOR"} (threshold: 0.1)`);
+
+    const allPresent = lcp !== undefined && inp !== undefined && cls !== undefined;
+    if (allPresent) {
+      const allPass = lcpPass && inpPass && clsPass;
+      lines.push(
+        "",
+        allPass
+          ? "Overall: PASS — all Core Web Vitals are good (positive ranking signal)"
+          : "Overall: FAIL — not all Core Web Vitals pass (may affect rankings)",
+      );
+    }
+  }
+
   return lines.join("\n");
 }