pagesight 0.12.0 → 0.12.1

package/package.json

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

package/src/lib/psi.ts

@@ -79,6 +79,10 @@ export interface PsiResult {
 export type PsiStrategy = "mobile" | "desktop";
 export type PsiCategoryType = "performance" | "accessibility" | "best-practices" | "seo";
 
+export function hasApiKey(): boolean {
+  return !!process.env.GOOGLE_API_KEY;
+}
+
 export async function runPagespeed(
   url: string,
   options?: {

package/src/tools/audit.ts

@@ -252,26 +252,28 @@ function addInspectFindings(verdict: string, coverageState: string, findings: Fi
 function formatAudit(url: string, findings: Finding[], errors: string[]): string {
   const lines: string[] = [`=== Site Audit: ${url} ===`, ""];
 
+  // Lead with errors so partial results are obvious
+  if (errors.length > 0) {
+    lines.push(`${errors.length} check${errors.length > 1 ? "s" : ""} failed — results below are partial:`, "");
+    for (const e of errors) {
+      lines.push(`  FAIL  ${e}`);
+    }
+    lines.push("");
+  }
+
   // Sort: HIGH first, then MEDIUM, then LOW
   const order: Record<string, number> = { HIGH: 0, MEDIUM: 1, LOW: 2 };
   findings.sort((a, b) => order[a.severity] - order[b.severity]);
 
-  if (findings.length === 0) {
+  if (findings.length === 0 && errors.length === 0) {
     lines.push("No issues found.");
-  } else {
-    lines.push(`${findings.length} findings:`, "");
+  } else if (findings.length > 0) {
+    lines.push(`${findings.length} finding${findings.length > 1 ? "s" : ""}:`, "");
     for (const f of findings) {
       lines.push(`${f.severity.padEnd(6)}  ${f.message}`);
     }
   }
 
-  if (errors.length > 0) {
-    lines.push("", "--- Errors (partial audit) ---", "");
-    for (const e of errors) {
-      lines.push(`- ${e}`);
-    }
-  }
-
   return lines.join("\n");
 }
 

package/src/tools/pagespeed.ts

@@ -1,6 +1,7 @@
 import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { z } from "zod";
 import {
+  hasApiKey,
   type PsiAudit,
   type PsiAuditDetailItem,
   type PsiCategoryType,
@@ -8,6 +9,9 @@ import {
   runPagespeed,
 } from "../lib/psi.js";
 
+const QUOTA_NOTE =
+  "\n\nNote: No GOOGLE_API_KEY configured — using shared quota (400 req/day). Set your own key to avoid rate limits.";
+
 function scoreLabel(score: number | null): string {
   if (score === null) return "N/A";
   const pct = Math.round(score * 100);
@@ -536,7 +540,8 @@ export function registerPagespeedTool(server: McpServer): void {
       if (url) {
         try {
           const result = await runPagespeed(url, opts);
-          return { content: [{ type: "text", text: formatPagespeed(url, result) }] };
+          const text = formatPagespeed(url, result) + (hasApiKey() ? "" : QUOTA_NOTE);
+          return { content: [{ type: "text", text }] };
         } catch (err) {
           const msg = err instanceof Error ? err.message : String(err);
           return { content: [{ type: "text", text: `Error running PageSpeed analysis: ${msg}` }] };
@@ -569,6 +574,8 @@ export function registerPagespeedTool(server: McpServer): void {
         output += `\n\n--- Errors ---\n${errorLines.join("\n")}`;
       }
 
+      if (!hasApiKey()) output += QUOTA_NOTE;
+
       return { content: [{ type: "text", text: output }] };
     },
   );