@rankcli/agent-runtime 0.0.29 → 0.0.30

package/dist/index.js

@@ -3352,7 +3352,9 @@ __export(index_exports, {
   analyzeUrlSafety: () => analyzeUrlSafety,
   analyzers: () => analyzers_exports,
   applyFixes: () => applyFixes,
+  assessReplacement: () => assessReplacement,
   buildDeterministicFixes: () => buildDeterministicFixes,
+  buildFrameworkGuidance: () => buildFrameworkGuidance,
   buildGSCApiRequest: () => buildGSCApiRequest,
   buildGSCRequest: () => buildGSCRequest,
   calculateAIVisibilityScore: () => calculateAIVisibilityScore,
@@ -3393,6 +3395,7 @@ __export(index_exports, {
   createPullRequest: () => createPullRequest,
   createSEOCommit: () => createSEOCommit,
   createSEOCommits: () => createSEOCommits,
+  describeConcerns: () => describeConcerns,
   detectDuplicates: () => detectDuplicates,
   detectFramework: () => detectFramework4,
   detectGitHubPages: () => detectGitHubPages,
@@ -12740,20 +12743,20 @@ function findMissingPreconnects(thirdPartyDomains, existingHints) {
       }
     })
   );
-  const missing = [];
+  const missing2 = [];
   for (const domain of thirdPartyDomains) {
     try {
       const hostname = new URL(domain).hostname;
       const recommended = CRITICAL_THIRD_PARTIES[hostname] || [domain];
       for (const rec of recommended) {
         if (!preconnected.has(rec)) {
-          missing.push(rec);
+          missing2.push(rec);
         }
       }
     } catch {
     }
   }
-  return [...new Set(missing)];
+  return [...new Set(missing2)];
 }
 function analyzeResourceHints(html, url) {
   const issues = [];
@@ -33049,6 +33052,167 @@ function findAppRoot(files) {
   return `${nonRoot[0]}/`;
 }
 
+// src/fixer/safety.ts
+var HTML_EXT = /\.html?$/i;
+var SOURCE_EXT = /\.(ts|tsx|js|jsx|mjs|cjs|vue|svelte|astro)$/i;
+function attributeValues(html, pattern) {
+  const found = /* @__PURE__ */ new Set();
+  for (const match of html.matchAll(pattern)) {
+    const value = match[1]?.trim();
+    if (value) found.add(value);
+  }
+  return found;
+}
+function scriptSources(html) {
+  return attributeValues(html, /<script[^>]+src=["']([^"']+)["']/gi);
+}
+function stylesheetHrefs(html) {
+  return attributeValues(html, /<link[^>]+href=["']([^"']+)["'][^>]*>/gi);
+}
+function verificationTokens(html) {
+  const found = /* @__PURE__ */ new Set();
+  for (const match of html.matchAll(/<meta[^>]+name=["']([^"']*(?:site-verification|verify|verification)[^"']*)["'][^>]*>/gi)) {
+    found.add(match[1].toLowerCase());
+  }
+  return found;
+}
+function importSpecifiers(source) {
+  const found = /* @__PURE__ */ new Set();
+  for (const match of source.matchAll(/\bfrom\s+["']([^"']+)["']/g)) found.add(match[1]);
+  for (const match of source.matchAll(/\brequire\(\s*["']([^"']+)["']\s*\)/g)) found.add(match[1]);
+  return found;
+}
+function missing(before, after) {
+  return [...before].filter((value) => !after.has(value));
+}
+function xmlConcern(xml) {
+  const stack = [];
+  for (const match of xml.matchAll(/<\/?([A-Za-z_][\w:.-]*)[^>]*?(\/?)>/g)) {
+    const [tag, name, selfClosing] = match;
+    if (tag.startsWith("<?") || tag.startsWith("<!")) continue;
+    if (tag.startsWith("</")) {
+      if (stack.pop() !== name) return `unbalanced </${name}>`;
+    } else if (!selfClosing) {
+      stack.push(name);
+    }
+  }
+  return stack.length ? `unclosed <${stack[stack.length - 1]}>` : null;
+}
+function sitemapUrlCount(xml) {
+  return [...xml.matchAll(/<loc>/gi)].length;
+}
+function newestLastmod(xml) {
+  const dates = [...xml.matchAll(/<lastmod>([^<]+)<\/lastmod>/gi)].map((m) => Date.parse(m[1].trim())).filter((t) => !Number.isNaN(t));
+  return dates.length ? Math.max(...dates) : null;
+}
+function jsonLdConcerns(html) {
+  const bad = [];
+  for (const match of html.matchAll(/<script[^>]+type=["']application\/ld\+json["'][^>]*>([\s\S]*?)<\/script>/gi)) {
+    try {
+      JSON.parse(match[1]);
+    } catch (error) {
+      bad.push(error.message);
+    }
+  }
+  return bad;
+}
+function assessReplacement(path3, original, replacement) {
+  const concerns = [];
+  const add = (code, detail) => concerns.push({ code, detail });
+  if (/\.json$/i.test(path3) || /\.webmanifest$/i.test(path3)) {
+    try {
+      JSON.parse(replacement);
+    } catch (error) {
+      add("invalid_json", error.message);
+    }
+  }
+  if (/\.xml$/i.test(path3)) {
+    const problem = xmlConcern(replacement);
+    if (problem) add("invalid_xml", problem);
+  }
+  if (HTML_EXT.test(path3)) {
+    for (const problem of jsonLdConcerns(replacement)) add("invalid_json_ld", problem);
+  }
+  if (original === null) return concerns;
+  if (replacement.length < original.length * 0.6) {
+    add("shrunk", `${Math.round(replacement.length / original.length * 100)}% of the original length`);
+  }
+  if (HTML_EXT.test(path3)) {
+    for (const tag of ["</head>", "</body>", "</html>"]) {
+      if (original.includes(tag) && !replacement.includes(tag)) add("lost_closing_tag", tag);
+    }
+    const lostScripts = missing(scriptSources(original), scriptSources(replacement));
+    if (lostScripts.length) add("lost_script", lostScripts.join(", "));
+    const lostStyles = missing(stylesheetHrefs(original), stylesheetHrefs(replacement));
+    if (lostStyles.length) add("lost_stylesheet", lostStyles.join(", "));
+    const lostVerification = missing(verificationTokens(original), verificationTokens(replacement));
+    if (lostVerification.length) add("lost_verification", lostVerification.join(", "));
+  }
+  if (SOURCE_EXT.test(path3)) {
+    const lostImports = missing(importSpecifiers(original), importSpecifiers(replacement));
+    if (lostImports.length) add("lost_import", lostImports.join(", "));
+  }
+  if (/sitemap[^/]*\.xml$/i.test(path3)) {
+    const before = sitemapUrlCount(original);
+    const after = sitemapUrlCount(replacement);
+    if (after < before) add("fewer_sitemap_urls", `${before} -> ${after}`);
+    const newestBefore = newestLastmod(original);
+    const newestAfter = newestLastmod(replacement);
+    if (newestBefore !== null && newestAfter !== null && newestAfter < newestBefore) {
+      add(
+        "lastmod_moved_backwards",
+        `${new Date(newestBefore).toISOString().slice(0, 10)} -> ${new Date(newestAfter).toISOString().slice(0, 10)}`
+      );
+    }
+  }
+  return concerns;
+}
+function describeConcerns(concerns) {
+  return concerns.map((c) => `${c.code} (${c.detail})`).join("; ");
+}
+
+// src/fixer/framework-guidance.ts
+var DEFAULT_MAX_CHARS = 4e3;
+function checkBlock(check) {
+  const fix = check.howToFix?.trim();
+  return [
+    `### ${check.title}${check.code ? ` (${check.code})` : ""}`,
+    check.description?.trim(),
+    fix ? `How this framework wants it fixed:
+${fix}` : null
+  ].filter(Boolean).join("\n");
+}
+function buildFrameworkGuidance(recipe, options = {}) {
+  if (!recipe) return "";
+  const budget = options.maxChars ?? DEFAULT_MAX_CHARS;
+  const reported = new Set(options.issueCodes ?? []);
+  const header = [
+    `## ${recipe.display_name} conventions`,
+    `Renders on the server by default: ${recipe.ssr_default ? "yes" : "no"}.`,
+    recipe.seo_challenges?.length ? `Known traps in this framework:
+${recipe.seo_challenges.map((c) => `- ${c}`).join("\n")}` : null
+  ].filter(Boolean).join("\n");
+  const checks = recipe.checks ?? [];
+  const relevant = checks.filter((c) => c.code && reported.has(c.code));
+  const rest = checks.filter((c) => !relevant.includes(c));
+  let out = header;
+  let included = 0;
+  for (const check of [...relevant, ...rest]) {
+    const block = checkBlock(check);
+    if (out.length + block.length + 2 > budget) break;
+    out += `
+
+${block}`;
+    included++;
+  }
+  if (included < checks.length) {
+    out += `
+
+_(${checks.length - included} more ${recipe.display_name} checks omitted.)_`;
+  }
+  return out;
+}
+
 // src/git/commit-helper.ts
 var import_child_process2 = require("child_process");
 var import_util2 = require("util");
@@ -35594,19 +35758,16 @@ __export(frameworks_exports, {
   loadRecipe: () => loadRecipe
 });
 var SUPPORTED_FRAMEWORKS = [
-  "react",
-  "nextjs",
-  "vue",
-  "nuxt",
   "angular",
-  "svelte",
-  "sveltekit",
   "astro",
-  "remix",
-  "gatsby",
-  "rails",
-  "laravel",
   "django",
+  "laravel",
+  "nextjs",
+  "nuxt",
+  "rails",
+  "react",
+  "sveltekit",
+  "vue",
   "wordpress"
 ];
 // Annotate the CommonJS export names for ESM import in node:
@@ -35656,7 +35817,9 @@ var SUPPORTED_FRAMEWORKS = [
   analyzeUrlSafety,
   analyzers,
   applyFixes,
+  assessReplacement,
   buildDeterministicFixes,
+  buildFrameworkGuidance,
   buildGSCApiRequest,
   buildGSCRequest,
   calculateAIVisibilityScore,
@@ -35697,6 +35860,7 @@ var SUPPORTED_FRAMEWORKS = [
   createPullRequest,
   createSEOCommit,
   createSEOCommits,
+  describeConcerns,
   detectDuplicates,
   detectFramework,
   detectGitHubPages,

package/dist/index.mjs

@@ -9199,20 +9199,20 @@ function findMissingPreconnects(thirdPartyDomains, existingHints) {
       }
     })
   );
-  const missing = [];
+  const missing2 = [];
   for (const domain of thirdPartyDomains) {
     try {
       const hostname = new URL(domain).hostname;
       const recommended = CRITICAL_THIRD_PARTIES[hostname] || [domain];
       for (const rec of recommended) {
         if (!preconnected.has(rec)) {
-          missing.push(rec);
+          missing2.push(rec);
         }
       }
     } catch {
     }
   }
-  return [...new Set(missing)];
+  return [...new Set(missing2)];
 }
 function analyzeResourceHints(html, url) {
   const issues = [];
@@ -29481,6 +29481,167 @@ function findAppRoot(files) {
   return `${nonRoot[0]}/`;
 }
 
+// src/fixer/safety.ts
+var HTML_EXT = /\.html?$/i;
+var SOURCE_EXT = /\.(ts|tsx|js|jsx|mjs|cjs|vue|svelte|astro)$/i;
+function attributeValues(html, pattern) {
+  const found = /* @__PURE__ */ new Set();
+  for (const match of html.matchAll(pattern)) {
+    const value = match[1]?.trim();
+    if (value) found.add(value);
+  }
+  return found;
+}
+function scriptSources(html) {
+  return attributeValues(html, /<script[^>]+src=["']([^"']+)["']/gi);
+}
+function stylesheetHrefs(html) {
+  return attributeValues(html, /<link[^>]+href=["']([^"']+)["'][^>]*>/gi);
+}
+function verificationTokens(html) {
+  const found = /* @__PURE__ */ new Set();
+  for (const match of html.matchAll(/<meta[^>]+name=["']([^"']*(?:site-verification|verify|verification)[^"']*)["'][^>]*>/gi)) {
+    found.add(match[1].toLowerCase());
+  }
+  return found;
+}
+function importSpecifiers(source) {
+  const found = /* @__PURE__ */ new Set();
+  for (const match of source.matchAll(/\bfrom\s+["']([^"']+)["']/g)) found.add(match[1]);
+  for (const match of source.matchAll(/\brequire\(\s*["']([^"']+)["']\s*\)/g)) found.add(match[1]);
+  return found;
+}
+function missing(before, after) {
+  return [...before].filter((value) => !after.has(value));
+}
+function xmlConcern(xml) {
+  const stack = [];
+  for (const match of xml.matchAll(/<\/?([A-Za-z_][\w:.-]*)[^>]*?(\/?)>/g)) {
+    const [tag, name, selfClosing] = match;
+    if (tag.startsWith("<?") || tag.startsWith("<!")) continue;
+    if (tag.startsWith("</")) {
+      if (stack.pop() !== name) return `unbalanced </${name}>`;
+    } else if (!selfClosing) {
+      stack.push(name);
+    }
+  }
+  return stack.length ? `unclosed <${stack[stack.length - 1]}>` : null;
+}
+function sitemapUrlCount(xml) {
+  return [...xml.matchAll(/<loc>/gi)].length;
+}
+function newestLastmod(xml) {
+  const dates = [...xml.matchAll(/<lastmod>([^<]+)<\/lastmod>/gi)].map((m) => Date.parse(m[1].trim())).filter((t) => !Number.isNaN(t));
+  return dates.length ? Math.max(...dates) : null;
+}
+function jsonLdConcerns(html) {
+  const bad = [];
+  for (const match of html.matchAll(/<script[^>]+type=["']application\/ld\+json["'][^>]*>([\s\S]*?)<\/script>/gi)) {
+    try {
+      JSON.parse(match[1]);
+    } catch (error) {
+      bad.push(error.message);
+    }
+  }
+  return bad;
+}
+function assessReplacement(path3, original, replacement) {
+  const concerns = [];
+  const add = (code, detail) => concerns.push({ code, detail });
+  if (/\.json$/i.test(path3) || /\.webmanifest$/i.test(path3)) {
+    try {
+      JSON.parse(replacement);
+    } catch (error) {
+      add("invalid_json", error.message);
+    }
+  }
+  if (/\.xml$/i.test(path3)) {
+    const problem = xmlConcern(replacement);
+    if (problem) add("invalid_xml", problem);
+  }
+  if (HTML_EXT.test(path3)) {
+    for (const problem of jsonLdConcerns(replacement)) add("invalid_json_ld", problem);
+  }
+  if (original === null) return concerns;
+  if (replacement.length < original.length * 0.6) {
+    add("shrunk", `${Math.round(replacement.length / original.length * 100)}% of the original length`);
+  }
+  if (HTML_EXT.test(path3)) {
+    for (const tag of ["</head>", "</body>", "</html>"]) {
+      if (original.includes(tag) && !replacement.includes(tag)) add("lost_closing_tag", tag);
+    }
+    const lostScripts = missing(scriptSources(original), scriptSources(replacement));
+    if (lostScripts.length) add("lost_script", lostScripts.join(", "));
+    const lostStyles = missing(stylesheetHrefs(original), stylesheetHrefs(replacement));
+    if (lostStyles.length) add("lost_stylesheet", lostStyles.join(", "));
+    const lostVerification = missing(verificationTokens(original), verificationTokens(replacement));
+    if (lostVerification.length) add("lost_verification", lostVerification.join(", "));
+  }
+  if (SOURCE_EXT.test(path3)) {
+    const lostImports = missing(importSpecifiers(original), importSpecifiers(replacement));
+    if (lostImports.length) add("lost_import", lostImports.join(", "));
+  }
+  if (/sitemap[^/]*\.xml$/i.test(path3)) {
+    const before = sitemapUrlCount(original);
+    const after = sitemapUrlCount(replacement);
+    if (after < before) add("fewer_sitemap_urls", `${before} -> ${after}`);
+    const newestBefore = newestLastmod(original);
+    const newestAfter = newestLastmod(replacement);
+    if (newestBefore !== null && newestAfter !== null && newestAfter < newestBefore) {
+      add(
+        "lastmod_moved_backwards",
+        `${new Date(newestBefore).toISOString().slice(0, 10)} -> ${new Date(newestAfter).toISOString().slice(0, 10)}`
+      );
+    }
+  }
+  return concerns;
+}
+function describeConcerns(concerns) {
+  return concerns.map((c) => `${c.code} (${c.detail})`).join("; ");
+}
+
+// src/fixer/framework-guidance.ts
+var DEFAULT_MAX_CHARS = 4e3;
+function checkBlock(check) {
+  const fix = check.howToFix?.trim();
+  return [
+    `### ${check.title}${check.code ? ` (${check.code})` : ""}`,
+    check.description?.trim(),
+    fix ? `How this framework wants it fixed:
+${fix}` : null
+  ].filter(Boolean).join("\n");
+}
+function buildFrameworkGuidance(recipe, options = {}) {
+  if (!recipe) return "";
+  const budget = options.maxChars ?? DEFAULT_MAX_CHARS;
+  const reported = new Set(options.issueCodes ?? []);
+  const header = [
+    `## ${recipe.display_name} conventions`,
+    `Renders on the server by default: ${recipe.ssr_default ? "yes" : "no"}.`,
+    recipe.seo_challenges?.length ? `Known traps in this framework:
+${recipe.seo_challenges.map((c) => `- ${c}`).join("\n")}` : null
+  ].filter(Boolean).join("\n");
+  const checks = recipe.checks ?? [];
+  const relevant = checks.filter((c) => c.code && reported.has(c.code));
+  const rest = checks.filter((c) => !relevant.includes(c));
+  let out = header;
+  let included = 0;
+  for (const check of [...relevant, ...rest]) {
+    const block = checkBlock(check);
+    if (out.length + block.length + 2 > budget) break;
+    out += `
+
+${block}`;
+    included++;
+  }
+  if (included < checks.length) {
+    out += `
+
+_(${checks.length - included} more ${recipe.display_name} checks omitted.)_`;
+  }
+  return out;
+}
+
 // src/git/commit-helper.ts
 import { execSync, exec as exec2 } from "child_process";
 import { promisify as promisify2 } from "util";
@@ -32019,19 +32180,16 @@ __export(frameworks_exports, {
   loadRecipe: () => loadRecipe
 });
 var SUPPORTED_FRAMEWORKS = [
-  "react",
-  "nextjs",
-  "vue",
-  "nuxt",
   "angular",
-  "svelte",
-  "sveltekit",
   "astro",
-  "remix",
-  "gatsby",
-  "rails",
-  "laravel",
   "django",
+  "laravel",
+  "nextjs",
+  "nuxt",
+  "rails",
+  "react",
+  "sveltekit",
+  "vue",
   "wordpress"
 ];
 export {
@@ -32080,7 +32238,9 @@ export {
   analyzeUrlSafety,
   analyzers_exports as analyzers,
   applyFixes,
+  assessReplacement,
   buildDeterministicFixes,
+  buildFrameworkGuidance,
   buildGSCApiRequest,
   buildGSCRequest,
   calculateAIVisibilityScore,
@@ -32121,6 +32281,7 @@ export {
   createPullRequest,
   createSEOCommit,
   createSEOCommits,
+  describeConcerns,
   detectDuplicates,
   detectFramework4 as detectFramework,
   detectGitHubPages,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rankcli/agent-runtime",
-  "version": "0.0.29",
+  "version": "0.0.30",
   "description": "RankCLI agent runtime - executes SEO audits and fixes with AI",
   "homepage": "https://rankcli.dev",
   "main": "dist/index.js",