@modelstatus/cli 0.1.75 → 0.1.76

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@modelstatus/cli",
-  "version": "0.1.75",
+  "version": "0.1.76",
   "description": "Track which AI models you use, where, and never get surprised by a retirement. Free offline model-health for any repo (mm status), browser sign-in for cloud inventory + alerts.",
   "keywords": [
     "llm",

package/src/index.js

@@ -373,9 +373,11 @@ async function ciReport(dir, flags, res) {
   const uniq = [...new Set(res.candidates.map((c) => c.model_string))];
   const resolved = uniq.length ? (await client.resolve(uniq)).data || [] : [];
   const byStr = new Map(resolved.map((r) => [r.input.toLowerCase(), r]));
+  const { dropResolvedFragments } = await import("./registry/local.js");
+  const uploadable = dropResolvedFragments(res.candidates, (c) => !!byStr.get(c.model_string.toLowerCase())?.model_id);
   const seen = new Set();
   const usages = [];
-  for (const c of res.candidates) {
+  for (const c of uploadable) {
     const r = byStr.get(c.model_string.toLowerCase());
     const k = `${r?.model_id ?? "custom:" + c.model_string}|${c.location_label}`;
     if (seen.has(k)) continue;
@@ -685,12 +687,14 @@ function cmdIntegrations(positional, flags) {
 async function cmdStatus(positional, flags) {
   const dir = path.resolve(positional[1] || flags.dir || ".");
   const { getRegistry } = await import("./registry/fetch.js");
-  const { resolveLocal, computeHealth } = await import("./registry/local.js");
+  const { resolveLocal, computeHealth, dropResolvedFragments } = await import("./registry/local.js");
   const snapshot = await getRegistry({ offline: !!flags.offline, log: (m) => process.stderr.write(`! ${m}\n`) });
 
-  const candidates = await collectFrom(parseSources(flags), scanOpts(flags, dir), snapshot.detection, explicitSources(flags));
+  let candidates = await collectFrom(parseSources(flags), scanOpts(flags, dir), snapshot.detection, explicitSources(flags));
   const resolved = resolveLocal(snapshot, [...new Set(candidates.map((c) => c.model_string))]);
   const byStr = new Map(resolved.map((r) => [r.input.toLowerCase(), r]));
+  // Suppress detector fragments of resolved aliases (see dropResolvedFragments).
+  candidates = dropResolvedFragments(candidates, (c) => !!byStr.get(c.model_string.toLowerCase())?.model_slug);
 
   // Aggregate locations per known model / per custom string.
   const known = new Map(); // slug -> { model, count }

package/src/registry/local.js

@@ -51,3 +51,34 @@ export function needsAttention(snapshot, retiringWindowDays = 90, today = new Da
     .filter((m) => m.health !== "ok")
     .sort((a, b) => String(a.retires_date || "9999-99-99").localeCompare(String(b.retires_date || "9999-99-99")));
 }
+
+/**
+ * Drop unresolved (custom) candidates that NEST with a resolved match at the
+ * same location (either direction of containment). The detector often emits
+ * overlapping spans for one id — e.g. from the Bedrock ARN
+ * "anthropic.claude-3-sonnet-20240229-v1:0" it emits both
+ * "claude-3-sonnet-20240229" (resolves → claude-3-sonnet) and the longer
+ * fragment "claude-3-sonnet-20240229-v1" (doesn't resolve) — which would
+ * double-count the line as a known model AND a phantom custom id. Both
+ * directions are intentional: for variants like "ft:gpt-4:acme", the resolved
+ * base model carries the lifecycle signal (a fine-tune dies with its base), so
+ * the custom row is noise there too. Customs that share a line with a resolved
+ * model WITHOUT string overlap (two different ids on one line) are kept.
+ */
+export function dropResolvedFragments(candidates, isResolved) {
+  const locOf = (c) => (c.source_path || c.location_label || "") + ":" + (c.source_line || "");
+  const resolvedByLoc = new Map(); // loc → [lowercased resolved strings]
+  for (const c of candidates) {
+    if (!isResolved(c)) continue;
+    const k = locOf(c);
+    if (!resolvedByLoc.has(k)) resolvedByLoc.set(k, []);
+    resolvedByLoc.get(k).push(c.model_string.toLowerCase());
+  }
+  return candidates.filter((c) => {
+    if (isResolved(c)) return true;
+    const here = resolvedByLoc.get(locOf(c));
+    if (!here) return true;
+    const s = c.model_string.toLowerCase();
+    return !here.some((rs) => rs !== s && (rs.includes(s) || s.includes(rs)));
+  });
+}

package/src/tui/scan-stream.js

@@ -11,7 +11,7 @@ import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
 import { getRegistry } from "../registry/fetch.js";
-import { resolveLocal, computeHealth } from "../registry/local.js";
+import { resolveLocal, computeHealth, dropResolvedFragments } from "../registry/local.js";
 import { scanFilesystemStreaming } from "../sources/filesystem.js";
 import { compilePatterns } from "../detect/core.js";
 
@@ -76,6 +76,9 @@ export function loadRegistry(log = () => {}) {
 export function summarize(snapshot, candidates) {
   const resolved = resolveLocal(snapshot, [...new Set(candidates.map((c) => c.model_string))]);
   const byStr = new Map(resolved.map((r) => [r.input.toLowerCase(), r]));
+  // A custom id that's a fragment of a resolved alias on the same line is a
+  // detector artifact, not a finding (e.g. the inner piece of a Bedrock ARN).
+  candidates = dropResolvedFragments(candidates, (c) => !!byStr.get(c.model_string.toLowerCase())?.model_slug);
   const known = new Map(); // slug → { model, count }
   const custom = new Map(); // string → count
   const refsBySlug = new Map(); // slug → [candidate]