pagesight 0.3.0 → 0.4.0

package/package.json

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

package/src/tools/crux.ts

@@ -193,14 +193,15 @@ export function registerCruxTool(server: McpServer): void {
       } catch (err) {
         const msg = err instanceof Error ? err.message : String(err);
         if (msg.includes("404")) {
-          return {
-            content: [
-              {
-                type: "text",
-                text: `No CrUX data available for ${url ?? origin}. The page may not have enough Chrome user traffic.`,
-              },
-            ],
-          };
+          const target = url ?? origin ?? "";
+          const lines = [`No CrUX data for ${target}.`, ""];
+          lines.push("CrUX requires sufficient Chrome user traffic (roughly 1,000+ monthly visits).");
+          if (url) {
+            const originUrl = new URL(url).origin;
+            lines.push(`Try origin-level data instead: origin "${originUrl}"`);
+          }
+          lines.push("For lab metrics without traffic requirements, use the pagespeed tool.");
+          return { content: [{ type: "text", text: lines.join("\n") }] };
         }
         if (msg.includes("SERVICE_DISABLED") || msg.includes("API_KEY_SERVICE_BLOCKED")) {
           return {
@@ -268,14 +269,15 @@ export function registerCruxTool(server: McpServer): void {
       } catch (err) {
         const msg = err instanceof Error ? err.message : String(err);
         if (msg.includes("404")) {
-          return {
-            content: [
-              {
-                type: "text",
-                text: `No CrUX history data for ${url ?? origin}. The page may not have enough Chrome user traffic.`,
-              },
-            ],
-          };
+          const target = url ?? origin ?? "";
+          const lines = [`No CrUX history data for ${target}.`, ""];
+          lines.push("CrUX requires sufficient Chrome user traffic (roughly 1,000+ monthly visits).");
+          if (url) {
+            const originUrl = new URL(url).origin;
+            lines.push(`Try origin-level data instead: origin "${originUrl}"`);
+          }
+          lines.push("For lab metrics without traffic requirements, use the pagespeed tool.");
+          return { content: [{ type: "text", text: lines.join("\n") }] };
         }
         if (msg.includes("SERVICE_DISABLED")) {
           return {

package/src/tools/metatags.ts

@@ -134,6 +134,77 @@ function formatJsonLd(data: unknown, indent = 0): string[] {
   return lines;
 }
 
+interface RedirectHop {
+  url: string;
+  status: number;
+}
+
+interface ImageCheck {
+  url: string;
+  tag: string;
+  status: number | null;
+  contentType: string | null;
+  contentLength: number | null;
+  error: string | null;
+}
+
+async function followRedirects(
+  url: string,
+  ua: string,
+  maxHops = 10,
+): Promise<{ chain: RedirectHop[]; response: Response }> {
+  const chain: RedirectHop[] = [];
+  let current = url;
+
+  for (let i = 0; i < maxHops; i++) {
+    const res = await fetch(current, {
+      headers: { "User-Agent": ua, Accept: "text/html" },
+      redirect: "manual",
+    });
+
+    chain.push({ url: current, status: res.status });
+
+    if (res.status >= 300 && res.status < 400) {
+      const location = res.headers.get("location");
+      if (!location) break;
+      current = new URL(location, current).href;
+      continue;
+    }
+
+    return { chain, response: res };
+  }
+
+  // If we exhausted hops, do a final follow-redirect fetch
+  const res = await fetch(current, {
+    headers: { "User-Agent": ua, Accept: "text/html" },
+    redirect: "follow",
+  });
+  return { chain, response: res };
+}
+
+async function checkImage(imageUrl: string, tag: string): Promise<ImageCheck> {
+  try {
+    const res = await fetch(imageUrl, { method: "HEAD", redirect: "follow" });
+    return {
+      url: imageUrl,
+      tag,
+      status: res.status,
+      contentType: res.headers.get("content-type"),
+      contentLength: res.headers.has("content-length") ? Number(res.headers.get("content-length")) : null,
+      error: null,
+    };
+  } catch (err) {
+    return {
+      url: imageUrl,
+      tag,
+      status: null,
+      contentType: null,
+      contentLength: null,
+      error: err instanceof Error ? err.message : String(err),
+    };
+  }
+}
+
 function formatMetatags(url: string, parsed: ParsedHead): string {
   const lines: string[] = [`=== Meta Tags: ${url} ===`, ""];
 
@@ -231,13 +302,7 @@ export function registerMetatagsTool(server: McpServer): void {
     async ({ url, user_agent }) => {
       try {
         const ua = user_agent ?? "Mozilla/5.0 (compatible; Googlebot/2.1; +http://www.google.com/bot.html)";
-        const res = await fetch(url, {
-          headers: {
-            "User-Agent": ua,
-            Accept: "text/html",
-          },
-          redirect: "follow",
-        });
+        const { chain, response: res } = await followRedirects(url, ua);
 
         if (!res.ok) {
           return {
@@ -252,15 +317,50 @@ export function registerMetatagsTool(server: McpServer): void {
 
         const html = await res.text();
         const parsed = parseHead(html);
-        const finalUrl = res.url !== url ? `(redirected to ${res.url})\n\n` : "";
+        const output: string[] = [];
+
+        // Redirect chain (only if there were redirects)
+        if (chain.length > 1) {
+          output.push("--- Redirect Chain ---", "");
+          for (let i = 0; i < chain.length; i++) {
+            const hop = chain[i];
+            const prefix = i === chain.length - 1 ? "" : `${hop.status} → `;
+            output.push(`${i + 1}. ${prefix}${hop.url}`);
+          }
+          output.push("");
+        }
+
+        // Main meta tag report
+        const finalUrl = chain.length > 1 ? chain[chain.length - 1].url : url;
+        output.push(formatMetatags(finalUrl, parsed));
+
+        // OG/Twitter image validation
+        const ogImage = getMeta(parsed.meta, "og:image");
+        const twitterImage = getMeta(parsed.meta, "twitter:image");
+        const imagesToCheck: Array<{ url: string; tag: string }> = [];
+        if (ogImage) imagesToCheck.push({ url: ogImage, tag: "og:image" });
+        if (twitterImage && twitterImage !== ogImage) imagesToCheck.push({ url: twitterImage, tag: "twitter:image" });
+
+        if (imagesToCheck.length > 0) {
+          const checks = await Promise.all(imagesToCheck.map((img) => checkImage(img.url, img.tag)));
+          output.push("", "--- Image Validation ---", "");
+          for (const check of checks) {
+            if (check.error) {
+              output.push(`${check.tag}: FAILED — ${check.error}`);
+              output.push(`  URL: ${check.url}`);
+            } else if (check.status && check.status >= 400) {
+              output.push(`${check.tag}: BROKEN — HTTP ${check.status}`);
+              output.push(`  URL: ${check.url}`);
+            } else {
+              const size = check.contentLength ? ` (${Math.round(check.contentLength / 1024)} KB)` : "";
+              const type = check.contentType ? ` ${check.contentType}` : "";
+              output.push(`${check.tag}: OK —${type}${size}`);
+            }
+          }
+        }
 
         return {
-          content: [
-            {
-              type: "text",
-              text: `${finalUrl}${formatMetatags(res.url, parsed)}`,
-            },
-          ],
+          content: [{ type: "text", text: output.join("\n") }],
         };
       } catch (err) {
         const msg = err instanceof Error ? err.message : String(err);

package/src/tools/pagespeed.ts

@@ -52,8 +52,12 @@ function formatOpportunities(audits: Record<string, PsiAudit>): string[] {
   const opportunities: PsiAudit[] = [];
 
   for (const audit of Object.values(audits)) {
-    if (audit.score !== null && audit.score < 1 && audit.numericValue && audit.numericValue > 0) {
-      if (audit.scoreDisplayMode === "numeric" || audit.scoreDisplayMode === "binary") {
+    if (audit.score === null || audit.score >= 1) continue;
+    const mode = audit.scoreDisplayMode;
+    const hasItems = (audit.details?.items?.length ?? 0) > 0;
+    const hasNumeric = audit.numericValue && audit.numericValue > 0;
+    if (mode === "metricSavings" || ((mode === "numeric" || mode === "binary") && hasNumeric)) {
+      if (hasNumeric || hasItems) {
         opportunities.push(audit);
       }
     }
@@ -90,7 +94,12 @@ function formatDiagnostics(audits: Record<string, PsiAudit>): string[] {
   const failing: Array<{ title: string; displayValue: string }> = [];
 
   for (const audit of Object.values(audits)) {
-    if (audit.score !== null && audit.score < 0.5 && audit.scoreDisplayMode === "numeric" && audit.displayValue) {
+    if (
+      audit.score !== null &&
+      audit.score < 0.5 &&
+      (audit.scoreDisplayMode === "numeric" || audit.scoreDisplayMode === "metricSavings") &&
+      audit.displayValue
+    ) {
       failing.push({ title: audit.title, displayValue: audit.displayValue });
     }
   }