npm - svelte-vitals - Versions diffs - 0.30.0 → 0.31.1 - Mend

svelte-vitals 0.30.0 → 0.31.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (6) hide show

package/README.md +3 -3
package/dist/bin.js +18 -9
package/dist/{chunk-QBXO46PU.js → chunk-3YO2A3YF.js} +80 -54
package/dist/index.d.ts +3 -3
package/dist/index.js +1 -1
package/package.json +4 -3

package/README.md CHANGED Viewed

@@ -33,16 +33,16 @@ Architecture Score: 100/100   (route avg 100)
 Critical (2)
 ────────────────────────
-✗ SEO001  Missing <title>
+✗ seo/title-presence  Missing <title>
             /
             src/routes/+page.svelte
-✗ SEO002  Missing <meta name="description">
+✗ seo/description-presence  Missing <meta name="description">
             /
             src/routes/+page.svelte
 Warnings (10)
 ────────────────────────
-✗ SEO003  Missing <link rel="canonical">
+✗ seo/canonical-url  Missing <link rel="canonical">
             /
             src/routes/+page.svelte
             …and 1 more

package/dist/bin.js CHANGED Viewed

@@ -9,7 +9,7 @@ import {
   readCoreVersion,
   readPackageVersion,
   run
-} from "./chunk-QBXO46PU.js";
+} from "./chunk-3YO2A3YF.js";
 // src/bin.ts
 import mri3 from "mri";
@@ -341,6 +341,12 @@ function isConfigTargetId(id) {
   return CONFIG_TARGETS.some((t) => t.id === id);
 }
+// src/ci/action-pin-comment.ts
+function actionPinComment(version) {
+  return `action-v${version}`;
+}
+var ACTION_PIN_COMMENT_RE = /#\s*(?:action-v|@svelte-vitals\/action@)(\S+)/;
 // src/ci/workflow.ts
 var WORKFLOW_PATH = ".github/workflows/svelte-vitals.yml";
 var CHECKOUT_SHA = "9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0";
@@ -371,7 +377,7 @@ function buildWorkflowYaml(opts) {
     `      - uses: actions/checkout@${CHECKOUT_SHA} # ${CHECKOUT_VERSION}`,
     "        with:",
     "          fetch-depth: 0",
-    `      - uses: oekazuma/svelte-vitals/packages/action@${actionSha} # @svelte-vitals/action@${actionVersion}`,
+    `      - uses: oekazuma/svelte-vitals/packages/action@${actionSha} # ${actionPinComment(actionVersion)}`,
     "        with:",
     "          diff: origin/${{ github.base_ref }}",
     "          baseline: origin/${{ github.base_ref }}",
@@ -505,7 +511,7 @@ without a judgment call of its own.
    none today, by design), no code edits, no commits, no formatters, no
    dependency installs. Run svelte-vitals read-only, for evidence only.
 3. **Plans must be fully self-contained.** The executor has zero context
-   from this conversation. Never write "fix it like SEO001 above" \u2014 inline
+   from this conversation. Never write "fix it like seo/title-presence above" \u2014 inline
    the exact file, line, current code, and the exact fix (svelte-vitals'
    \`fix.snippet\`/\`fix.description\` for the rule, quoted verbatim \u2014 see
    below).
@@ -789,7 +795,7 @@ function buildConfigFileTemplate(opts = {}) {
   const header = "// svelte-vitals config file \u2014 https://oekazuma.github.io/svelte-vitals/guides/configuration/\n";
   const options = `  // treatDynamicAs: 'pass', // 'pass' | 'warn' | 'fail' \u2014 how {data.title}-style dynamic values are scored
   // metaComponents: ['Seo'], // component names that resolve SEO tags into <head>
-  // rules: {}, // e.g. { SEO001: 'off' } to disable a rule
+  // rules: {}, // e.g. { 'seo/title-presence': 'off' } to disable a rule
   // failOn: 'critical', // 'critical' | 'warning' | 'info'
   // weights: {} // e.g. { seo: 2 } \u2014 per-category weight for the combined Health score`;
   if (opts.useDefineConfig) {
@@ -998,8 +1004,8 @@ function readInstalledViteVersion(io) {
 }
 // src/ci/action-pin.generated.ts
-var ACTION_SHA = "81d49a11b71a16ea1294212d43520a0137b4bd5e";
-var ACTION_VERSION = "0.3.5";
+var ACTION_SHA = "7ae39edda8bfbaba16ddb6c24871619a9ebe0a5d";
+var ACTION_VERSION = "0.3.7";
 // src/install/index.ts
 function detectPackageManagerNear(io, appDir) {
@@ -1572,6 +1578,7 @@ import mri2 from "mri";
 // src/ci/upgrade.ts
 var ACTION_USES_LINE = /^(?<indent>\s*-\s*uses:\s*(?:&\S+\s+)?oekazuma\/svelte-vitals\/packages\/action@)(?<ref>[^\s#]+)(?<comment>\s*#.*)?$/;
+var LEGACY_COMMENT = /#\s*@svelte-vitals\/action@/;
 function upgradeActionPin(content, sha, version) {
   const lines = content.split("\n");
   let replaced = 0;
@@ -1583,13 +1590,15 @@ function upgradeActionPin(content, sha, version) {
     if (!match || !match.groups) return line;
     const { indent: indent2, ref } = match.groups;
     if (indent2 === void 0 || ref === void 0) return line;
-    if (ref === sha) return line;
+    const comment = match.groups.comment ?? "";
+    const commentMatch = ACTION_PIN_COMMENT_RE.exec(comment);
+    const isCanonicalComment = commentMatch !== null && !LEGACY_COMMENT.test(comment);
+    if (ref === sha && isCanonicalComment) return line;
     if (from === void 0) {
-      const commentMatch = /#\s*@svelte-vitals\/action@(\S+)/.exec(match.groups.comment ?? "");
       from = commentMatch ? commentMatch[1] : ref.slice(0, 7);
     }
     replaced += 1;
-    return `${indent2}${sha} # @svelte-vitals/action@${version}${eol}`;
+    return `${indent2}${sha} # ${actionPinComment(version)}${eol}`;
   });
   if (replaced === 0) {
     const hasAnyReference = lines.some((line) => ACTION_USES_LINE.test(line.endsWith("\r") ? line.slice(0, -1) : line));

package/dist/{chunk-QBXO46PU.js → chunk-3YO2A3YF.js} RENAMED Viewed

@@ -53,7 +53,8 @@ import { defaultConfig } from "@svelte-vitals/core";
 // src/providers/source/project.ts
 import {
   ROBOTS_SOURCE_PATHS,
-  SITEMAP_SOURCE_PATHS
+  SITEMAP_SOURCE_PATHS,
+  findMinifyDisabled
 } from "@svelte-vitals/core";
 var ProjectError = class extends Error {
   constructor(message) {
@@ -110,18 +111,39 @@ async function robotsRefsSitemap(rt, cwd) {
     return void 0;
   }
 }
+var VITE_CONFIG_FILES = [
+  "vite.config.js",
+  "vite.config.mjs",
+  "vite.config.ts",
+  "vite.config.cjs",
+  "vite.config.mts",
+  "vite.config.cts"
+];
+async function detectViteMinifyDisabled(rt, cwd) {
+  const exists = await Promise.all(VITE_CONFIG_FILES.map((f) => rt.exists(rt.join(cwd, f))));
+  const file = VITE_CONFIG_FILES[exists.indexOf(true)];
+  if (!file) return void 0;
+  try {
+    const hit = findMinifyDisabled(await rt.readFile(rt.join(cwd, file)));
+    return hit ? { file, line: hit.line } : void 0;
+  } catch {
+    return void 0;
+  }
+}
 async function collectProjectFacts(rt, cwd) {
-  const [hasRobotsTxt, hasSitemap, htmlLang] = await Promise.all([
+  const [hasRobotsTxt, hasSitemap, htmlLang, viteMinifyDisabled] = await Promise.all([
     existsAny(rt, cwd, ROBOTS_SOURCE_PATHS),
     existsAny(rt, cwd, SITEMAP_SOURCE_PATHS),
-    detectAppHtmlLang(rt, cwd)
+    detectAppHtmlLang(rt, cwd),
+    detectViteMinifyDisabled(rt, cwd)
   ]);
   const robotsReferencesSitemap = await robotsRefsSitemap(rt, cwd);
   return {
     hasRobotsTxt,
     hasSitemap,
     htmlLang,
-    ...robotsReferencesSitemap !== void 0 ? { robotsReferencesSitemap } : {}
+    ...robotsReferencesSitemap !== void 0 ? { robotsReferencesSitemap } : {},
+    ...viteMinifyDisabled ? { viteMinifyDisabled } : {}
   };
 }
@@ -139,16 +161,16 @@ function exprValue(node) {
 }
 function resolveMetaObject(attr, keyMap) {
   if (!attr) return { tags: [], opaque: false };
-  const expr = attr.value?.type === "ExpressionTag" ? attr.value.expression : void 0;
+  const expr = attr.value !== true && !Array.isArray(attr.value) ? attr.value.expression : void 0;
   if (!expr || expr.type !== "ObjectExpression") return { tags: [], opaque: true };
   const tags = [];
   let opaque = false;
-  for (const prop of expr.properties ?? []) {
-    if (prop?.type !== "Property" || prop.computed) {
+  for (const prop of expr.properties) {
+    if (prop.type !== "Property" || prop.computed) {
       opaque = true;
       continue;
     }
-    const key = prop.key?.name ?? prop.key?.value;
+    const key = prop.key.type === "Identifier" ? prop.key.name : prop.key.type === "Literal" ? prop.key.value : void 0;
     const make = typeof key === "string" ? keyMap[key] : void 0;
     if (make) tags.push(make(exprValue(prop.value)));
   }
@@ -168,7 +190,7 @@ var TWITTER_KEYS = {
 // src/providers/source/adapters/svelte-meta-tags.ts
 function findAttr(attributes, name) {
-  return attributes.find((a) => a?.type === "Attribute" && a.name === name);
+  return attributes.find((a) => a.type === "Attribute" && a.name === name);
 }
 var svelteMetaTagsAdapter = {
   match(info) {
@@ -221,7 +243,7 @@ var svelteMetaTagsJsonLdAdapter = {
 // src/providers/source/adapters/svelte-seo.ts
 import { attrValueOf as attrValueOf2, attrTextOf as attrTextOf2 } from "@svelte-vitals/core";
 function findAttr2(attributes, name) {
-  return attributes.find((a) => a?.type === "Attribute" && a.name === name);
+  return attributes.find((a) => a.type === "Attribute" && a.name === name);
 }
 var svelteSeoAdapter = {
   match(info) {
@@ -273,28 +295,31 @@ import {
 // src/providers/source/imports.ts
 function addImportsFromProgram(program, map) {
   for (const node of program?.body ?? []) {
-    if (node?.type !== "ImportDeclaration") continue;
-    const source = String(node.source?.value ?? "");
-    for (const spec of node.specifiers ?? []) {
-      const local = spec?.local?.name;
+    if (node.type !== "ImportDeclaration") continue;
+    const source = String(node.source.value ?? "");
+    for (const spec of node.specifiers) {
+      const local = spec.local?.name;
       if (!local) continue;
       if (spec.type === "ImportDefaultSpecifier") {
         map.set(local, { source, imported: "default" });
       } else if (spec.type === "ImportSpecifier") {
-        map.set(local, { source, imported: spec.imported?.name ?? spec.imported?.value ?? local });
+        const imported = spec.imported.type === "Identifier" ? spec.imported.name : spec.imported.value;
+        map.set(local, { source, imported: typeof imported === "string" ? imported : local });
       }
     }
   }
 }
 function collectImports(ast) {
-  const root = ast;
   const map = /* @__PURE__ */ new Map();
-  addImportsFromProgram(root?.instance?.content, map);
-  addImportsFromProgram(root?.module?.content, map);
+  addImportsFromProgram(ast.instance?.content, map);
+  addImportsFromProgram(ast.module?.content, map);
   return map;
 }
 // src/providers/source/parse.ts
+function childOf(node, key) {
+  return node[key];
+}
 function collectSvelteHeads(node, acc) {
   if (Array.isArray(node)) {
     for (const child of node) collectSvelteHeads(child, acc);
@@ -303,32 +328,33 @@ function collectSvelteHeads(node, acc) {
   if (!node || typeof node !== "object") return;
   if (node.type === "SvelteHead") acc.push(node);
   for (const key of CHILD_NODE_KEYS) {
-    if (key in node) collectSvelteHeads(node[key], acc);
+    if (key in node) collectSvelteHeads(childOf(node, key), acc);
   }
 }
 function tagsFromHead(head) {
   const tags = [];
-  const children = head?.fragment?.nodes ?? [];
+  const children = head.fragment.nodes;
   for (const node of children) {
-    if (node?.type === "TitleElement") {
-      const titleNodes = node.fragment?.nodes ?? [];
+    if (node.type === "TitleElement") {
+      const titleNodes = node.fragment.nodes;
       const text = textFromNodes(titleNodes);
       tags.push({ kind: "title", value: valueFromNodes(titleNodes), ...text !== void 0 ? { text } : {} });
       continue;
     }
-    if (node?.type !== "RegularElement") continue;
+    if (node.type !== "RegularElement") continue;
+    const attributes = node.attributes;
     if (node.name === "meta") {
-      const charset = attrValue(node.attributes, "charset");
+      const charset = attrValue(attributes, "charset");
       if (charset !== "absent") {
         tags.push({ kind: "meta", name: "charset", value: charset });
         continue;
       }
-      const name = attrText(node.attributes, "name");
-      const property = attrText(node.attributes, "property");
-      const content = name === "robots" ? attrText(node.attributes, "content") : void 0;
+      const name = attrText(attributes, "name");
+      const property = attrText(attributes, "property");
+      const content = name === "robots" ? attrText(attributes, "content") : void 0;
       const noindex = content !== void 0 && /(^|[\s,])(noindex|none)([\s,]|$)/i.test(content);
-      const contentValue = attrValue(node.attributes, "content");
-      const descText = name === "description" && contentValue === "static" ? attrText(node.attributes, "content") : void 0;
+      const contentValue = attrValue(attributes, "content");
+      const descText = name === "description" && contentValue === "static" ? attrText(attributes, "content") : void 0;
       tags.push({
         kind: "meta",
         ...name ? { name } : {},
@@ -338,33 +364,33 @@ function tagsFromHead(head) {
         ...descText !== void 0 ? { text: descText } : {}
       });
     } else if (node.name === "link") {
-      const rel = attrText(node.attributes, "rel");
-      const hasAs = findAttr3(node.attributes, "as") !== void 0;
-      const asLiteral = attrText(node.attributes, "as");
-      const hasCrossorigin = findAttr3(node.attributes, "crossorigin") !== void 0;
-      const hreflang = attrText(node.attributes, "hreflang");
-      const href = attrText(node.attributes, "href");
+      const rel = attrText(attributes, "rel");
+      const hasAs = findAttr3(attributes, "as") !== void 0;
+      const asLiteral = attrText(attributes, "as");
+      const hasCrossorigin = findAttr3(attributes, "crossorigin") !== void 0;
+      const hreflang = attrText(attributes, "hreflang");
+      const href = attrText(attributes, "href");
       tags.push({
         kind: "link",
         ...rel ? { rel } : {},
-        value: attrValue(node.attributes, "href"),
+        value: attrValue(attributes, "href"),
         ...hasAs ? { hasAs: true } : {},
         ...asLiteral ? { as: asLiteral } : {},
         ...hasCrossorigin ? { hasCrossorigin: true } : {},
-        // Keep a literal empty hreflang="" (present-but-invalid) so SEO026 can flag it.
+        // Keep a literal empty hreflang="" (present-but-invalid) so seo/hreflang can flag it.
         ...hreflang !== void 0 ? { hreflang } : {},
         ...href ? { href } : {}
       });
     } else if (node.name === "script") {
-      const type = attrText(node.attributes, "type");
+      const type = attrText(attributes, "type");
       if (type === "application/ld+json") {
-        const nodes = node.fragment?.nodes ?? [];
+        const nodes = node.fragment.nodes;
         const raw = textFromNodes(nodes);
         tags.push({ kind: "jsonld", value: valueFromNodes(nodes), ...raw !== void 0 ? { jsonld: raw } : {} });
       } else {
-        const src = attrText(node.attributes, "src");
+        const src = attrText(attributes, "src");
         if (src) {
-          const blocking = findAttr3(node.attributes, "defer") === void 0 && findAttr3(node.attributes, "async") === void 0 && type !== "module";
+          const blocking = findAttr3(attributes, "defer") === void 0 && findAttr3(attributes, "async") === void 0 && type !== "module";
           tags.push({ kind: "script", value: "static", href: src, ...blocking ? { blocking: true } : {} });
         }
       }
@@ -378,16 +404,16 @@ function collectComponents(node, acc) {
     return;
   }
   if (!node || typeof node !== "object") return;
-  if (node.type === "Component" && typeof node.name === "string") {
-    const attributes = node.attributes ?? [];
+  if (node.type === "Component") {
+    const attributes = node.attributes;
     acc.push({
       name: node.name,
       attributes,
-      hasSpread: attributes.some((a) => a?.type === "SpreadAttribute")
+      hasSpread: attributes.some((a) => a.type === "SpreadAttribute")
     });
   }
   for (const key of CHILD_NODE_KEYS) {
-    if (key in node) collectComponents(node[key], acc);
+    if (key in node) collectComponents(childOf(node, key), acc);
   }
 }
 function collectImages(node, source, acc) {
@@ -397,8 +423,8 @@ function collectImages(node, source, acc) {
   }
   if (!node || typeof node !== "object") return;
   if (node.type === "RegularElement" && node.name === "img") {
-    const attrs = node.attributes ?? [];
-    const hasSpread = attrs.some((a) => a?.type === "SpreadAttribute");
+    const attrs = node.attributes;
+    const hasSpread = node.attributes.some((a) => a.type === "SpreadAttribute");
     acc.push({
       hasWidth: hasSpread || Boolean(findAttr3(attrs, "width")),
       hasHeight: hasSpread || Boolean(findAttr3(attrs, "height")),
@@ -411,7 +437,7 @@ function collectImages(node, source, acc) {
     });
   }
   for (const key of CHILD_NODE_KEYS) {
-    if (key in node) collectImages(node[key], source, acc);
+    if (key in node) collectImages(childOf(node, key), source, acc);
   }
 }
 function collectHeadings(node, source, acc) {
@@ -421,23 +447,23 @@ function collectHeadings(node, source, acc) {
   }
   if (!node || typeof node !== "object") return;
   if (node.type === "SvelteHead") return;
-  if (node.type === "RegularElement" && typeof node.name === "string" && /^h[1-6]$/.test(node.name)) {
+  if (node.type === "RegularElement" && /^h[1-6]$/.test(node.name)) {
     acc.push({ level: Number(node.name[1]), line: lineOf(source, node.start) });
   }
   for (const key of CHILD_NODE_KEYS) {
-    if (key in node) collectHeadings(node[key], source, acc);
+    if (key in node) collectHeadings(childOf(node, key), source, acc);
   }
 }
 function parseFile(source, filename) {
   const ast = parse(source, { modern: true, filename });
   const heads = [];
-  collectSvelteHeads(ast.fragment ?? ast, heads);
+  collectSvelteHeads(ast.fragment, heads);
   const components = [];
-  collectComponents(ast.fragment ?? ast, components);
+  collectComponents(ast.fragment, components);
   const images = [];
-  collectImages(ast.fragment ?? ast, source, images);
+  collectImages(ast.fragment, source, images);
   const headings = [];
-  collectHeadings(ast.fragment ?? ast, source, headings);
+  collectHeadings(ast.fragment, source, headings);
   return {
     headTags: heads.flatMap(tagsFromHead),
     components,

package/dist/index.d.ts CHANGED Viewed

@@ -1,5 +1,6 @@
 import { HeadTag, RuleSetting, Config, Severity, Category, Result } from '@svelte-vitals/core';
 export { defineConfig } from '@svelte-vitals/core';
+import { AST } from 'svelte/compiler';
 /** A resolved import binding: which module, and which export ('default' for default imports). */
 interface ImportInfo {
@@ -11,10 +12,9 @@ type ImportMap = Map<string, ImportInfo>;
 /** A head tag parsed from one file, before layout-chain presence is assigned. */
 type ParsedTag = Omit<HeadTag, 'presence' | 'file'>;
-type Node = any;
 interface ComponentUse {
     name: string;
-    attributes: Node[];
+    attributes: AST.Component['attributes'];
     hasSpread: boolean;
 }
 interface ParsedImage {
@@ -27,7 +27,7 @@ interface ParsedImage {
     /** 1-based source line, or 0 if unknown. */
     line: number;
 }
-/** A page-body heading (<h1>–<h6>) parsed from one file (SEO027). */
+/** A page-body heading (<h1>–<h6>) parsed from one file (seo/single-h1). */
 interface ParsedHeading {
     /** Heading level 1–6. */
     level: number;

package/dist/index.js CHANGED Viewed

@@ -10,7 +10,7 @@ import {
   routeMatcher,
   run,
   spinnerEnabled
-} from "./chunk-QBXO46PU.js";
+} from "./chunk-3YO2A3YF.js";
 export {
   ProjectError,
   analyzeProject,

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "svelte-vitals",
-  "version": "0.30.0",
+  "version": "0.31.1",
   "description": "A SvelteKit SEO checker — not a runtime Web Vitals reporter. Static analysis of your routes' head metadata.",
   "type": "module",
   "license": "MIT",
@@ -44,11 +44,12 @@
     "magicast": "^0.5.3",
     "mri": "^1.2.0",
     "smol-toml": "^1.7.0",
-    "svelte": "^5.56.4",
+    "svelte": "^5.56.6",
     "tinyglobby": "^0.2.17",
-    "@svelte-vitals/core": "0.27.0"
+    "@svelte-vitals/core": "0.28.0"
   },
   "devDependencies": {
+    "@types/estree": "^1.0.9",
     "@types/node": "^24.13.3"
   },
   "scripts": {