@svelte-vitals/core 0.31.0 → 0.31.1

package/dist/index.d.ts

@@ -946,9 +946,9 @@ interface Rule {
     severity: Severity;
     /** 'route' = evaluated per route, 'project' = site-wide (design §10, §12). */
     scope: Scope;
-    /** Why this rule matters — one or two sentences, surfaced by explain_rule (issue #24). */
+    /** Why this rule matters — one or two sentences, surfaced by `svelte-vitals explain` (issue #24). */
     rationale: string;
-    /** Canonical remediation template, shared by findings and explain_rule (issue #24). */
+    /** Canonical remediation template, shared by findings and `svelte-vitals explain` (issue #24). */
     fix?: Fix;
     /** Configurable options for this rule; absent means the rule takes none. */
     options?: RuleOptionsSpec;
@@ -1290,7 +1290,7 @@ declare const performanceStateRaw: Rule;
 
 declare const allRules: Rule[];
 
-/** One configurable option of a rule, flattened for explain_rule's consumers. */
+/** One configurable option of a rule, flattened for `svelte-vitals explain`'s output. */
 interface RuleOptionInfo {
     name: string;
     /** `integer` replaces the default; `string-list`/`string-map` are ADDED to it. */
@@ -1314,7 +1314,7 @@ interface RuleInfo {
      */
     options?: RuleOptionInfo[];
 }
-/** Look up a rule's static metadata for the MCP explain_rule tool (issue #24). Rule ids are matched exactly (case-sensitive, e.g. "seo/ssr-disabled"). */
+/** Look up a rule's static metadata, as `svelte-vitals explain` renders it (issue #24). Rule ids are matched exactly (case-sensitive, e.g. "seo/ssr-disabled"). */
 declare function explainRule(id: string): RuleInfo | undefined;
 
 interface HeadTagRuleOptions {
@@ -1326,7 +1326,7 @@ interface HeadTagRuleOptions {
     /** Short human label, e.g. 'description'. */
     label: string;
     recommendation: string;
-    /** Why this rule matters — surfaced by explain_rule (issue #24). */
+    /** Why this rule matters — surfaced by `svelte-vitals explain` (issue #24). */
     rationale: string;
     /** Agent-actionable remediation attached to every finding (issue #18). */
     fix?: Fix;
@@ -1432,7 +1432,14 @@ interface ScoreModel {
     criticalCap: number | null;
 }
 interface ScoreResult {
+    /** The score as displayed: `Math.floor(rawScore)`, so 100 means the deduction was exactly zero. */
     score: number;
+    /**
+     * The same score before flooring, after `sitePenalty` and the cap, clamped to `[0, 100]`. Exposed so
+     * `computeHealth` can average unrounded values and floor once — averaging the displayed scores would
+     * compose two roundings and move Health by up to two points.
+     */
+    rawScore: number;
     scoreModel: ScoreModel;
 }
 interface ScoreOptions {
@@ -1482,7 +1489,7 @@ interface JsonReport {
     }>;
     siteIssues: JsonIssue[];
 }
-/** Build the structured JSON report object (design §7). Shared by the json reporter and the MCP `analyze` tool (issue #24). */
+/** Build the structured JSON report object (design §7). The shape the `json` reporter emits (issue #24). */
 declare function buildJsonReport(results: Result[], config: Config, meta: {
     version: string;
 }): JsonReport;

package/dist/index.js

@@ -4938,7 +4938,7 @@ var architectureUnitEntryFile = {
           category: "architecture",
           severity: "info",
           detection: { presence: "own", value: "static" },
-          route: expected,
+          location: expected,
           message: "Unit entry file",
           recommendation: recommendation8,
           docsUrl: docsUrl8
@@ -5641,7 +5641,8 @@ function computeScore(results, config, options = {}) {
     routeScores.set(route, routeScores.get(route) - deduction);
   }
   const scores = [...routeScores.values()].map(clamp);
-  const routeAverage = scores.length ? Math.round(scores.reduce((a, b) => a + b, 0) / scores.length) : 100;
+  const rawRouteAverage = scores.length ? scores.reduce((a, b) => a + b, 0) / scores.length : 100;
+  const routeAverage = Math.floor(rawRouteAverage);
   const projectRuleMax = /* @__PURE__ */ new Map();
   for (const r of projectResults) {
     if (!isPenalized(r.detection, config.treatDynamicAs)) continue;
@@ -5653,11 +5654,11 @@ function computeScore(results, config, options = {}) {
   let sitePenalty = 0;
   for (const deduction of projectRuleMax.values()) sitePenalty += deduction;
   const applyCap = options.applyCriticalCap ?? true;
-  const uncapped = routeAverage - sitePenalty;
-  const capBinds = applyCap && anyCritical && uncapped > CRITICAL_CAP;
+  const rawUncapped = rawRouteAverage - sitePenalty;
+  const capBinds = applyCap && anyCritical && rawUncapped > CRITICAL_CAP;
   const criticalCap = capBinds ? CRITICAL_CAP : null;
-  const score = capBinds ? CRITICAL_CAP : uncapped;
-  return { score: clamp(score), scoreModel: { routeAverage, sitePenalty, criticalCap } };
+  const rawScore = clamp(capBinds ? CRITICAL_CAP : rawUncapped);
+  return { score: Math.floor(rawScore), rawScore, scoreModel: { routeAverage, sitePenalty, criticalCap } };
 }
 function scoresByCategory(results, config) {
   const byCat = /* @__PURE__ */ new Map();
@@ -5674,7 +5675,7 @@ function scoresByCategory(results, config) {
 function computeHealth(results, config) {
   const categories = scoresByCategory(results, config);
   const weights = {};
-  let weighted = 0;
+  let weightedDeficit = 0;
   let total = 0;
   for (const cat of Object.keys(categories)) {
     const w = config.weights?.[cat] ?? 1;
@@ -5682,14 +5683,15 @@ function computeHealth(results, config) {
       throw new RangeError(`invalid weight for '${cat}'; expected a finite number >= 0.`);
     }
     weights[cat] = w;
-    weighted += categories[cat].score * w;
+    weightedDeficit += (100 - categories[cat].rawScore) * w;
     total += w;
   }
   if (Object.keys(weights).length === 0) return { health: 100, categories, weights };
   if (total === 0) {
     throw new RangeError("Health weights sum to 0; at least one present category must have a positive weight.");
   }
-  const health = Math.round(weighted / total);
+  const averageDeficit = weightedDeficit / total;
+  const health = averageDeficit === 0 ? 100 : Math.min(99, Math.floor(100 - averageDeficit));
   return { health, categories, weights };
 }
 
@@ -5763,7 +5765,7 @@ function byRouteTree(p, results, config, verbose) {
   }
   if (!verbose && scored.length > MAX_ROUTES_BY_ROUTE) {
     const remaining = scored.slice(MAX_ROUTES_BY_ROUTE);
-    const avgScore = Math.round(remaining.reduce((sum, r) => sum + r.score, 0) / remaining.length);
+    const avgScore = Math.floor(remaining.reduce((sum, r) => sum + r.score, 0) / remaining.length);
     lines.push(
       p.dim(
         `\u2026and ${remaining.length} more route${remaining.length > 1 ? "s" : ""} (avg score ${avgScore}) \u2014 run with --verbose to see all`
@@ -5833,8 +5835,9 @@ function formatConsoleReport(results, config, options = {}) {
     if (options.verbose) {
       for (const r of passed) {
         const marker = classify(r, config) === "dynamic" ? p.cyan("  \u21AF dynamic") : "";
-        const route = r.route ? `  ${r.route}` : "";
-        lines.push(`${p.green("\u2713")} ${r.id}  ${r.message}${marker}${route}`);
+        const where = r.location ?? r.route;
+        const suffix = where ? `  ${where}` : "";
+        lines.push(`${p.green("\u2713")} ${r.id}  ${r.message}${marker}${suffix}`);
       }
     }
     lines.push("");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@svelte-vitals/core",
-  "version": "0.31.0",
+  "version": "0.31.1",
   "description": "Shared, runtime-agnostic core for svelte-vitals (types, rule engine, scorer, reporter).",
   "type": "module",
   "license": "MIT",