model-eol 0.2.3 → 0.3.0

package/lib/scanner.mjs

@@ -3,6 +3,7 @@ import path from 'node:path'
 import { spawnSync } from 'node:child_process'
 
 import { buildModelPattern } from './feeds.mjs'
+import { matchesAnyGlob, matchesGlob, normalizeRepoPath } from './glob.mjs'
 
 export const CODE_EXT = new Set([
   '.c', '.cc', '.cjs', '.cpp', '.cs', '.cxx', '.go', '.h', '.hpp', '.java', '.js',
@@ -122,6 +123,7 @@ const classifyRef = ({ file, line, lines, publisher }) => {
 
 const isScannable = (file, includeDocs) => {
   const basename = path.basename(file)
+  if (basename === '.model-eol.json') return false
   if (basename === '.env' || basename.startsWith('.env.')) return true
   const ext = path.extname(file)
   return CODE_EXT.has(ext) || (includeDocs && DOC_EXT.has(ext))
@@ -241,17 +243,58 @@ export const addedLinesForTargets = (targets, baseRef) => {
 export const filterFindingsToChanged = (findings, addedLines) =>
   findings.filter(finding => addedLines.has(changedKey(finding.file, finding.line)))
 
-const collectFilesDetailed = (targets, { includeDocs = false } = {}) => {
+const collectFilesDetailed = (targets, {
+  includeDocs = false,
+  ignorePaths = [],
+  ignoreRoots = [],
+  ignoredFiles = [],
+} = {}) => {
   const files = []
   const notes = []
   const validTargets = []
   const seen = new Set()
+  const canonicalPath = value => {
+    const resolved = path.resolve(value)
+    try {
+      return fs.realpathSync(resolved)
+    } catch {
+      return resolved
+    }
+  }
+  const excludedFiles = new Set(ignoredFiles.map(canonicalPath))
+  const roots = ignoreRoots.map(canonicalPath)
   let fileLimitWarned = false
   let stopped = false
 
   const note = value => notes.push(value)
+  const repoPathsFor = absolute => {
+    const resolved = canonicalPath(absolute)
+    const repoPaths = []
+    for (const root of roots) {
+      const relative = path.relative(root, resolved)
+      if (relative === '' || relative === '..' || relative.startsWith(`..${path.sep}`) || path.isAbsolute(relative)) continue
+      repoPaths.push(normalizeRepoPath(relative))
+    }
+    return repoPaths
+  }
+  const matchesIgnoredRepoPath = repoPath => matchesAnyGlob(repoPath, ignorePaths) ||
+    ignorePaths.some(pattern => {
+      const normalizedPattern = normalizeRepoPath(pattern).replace(/\/+$/, '')
+      return normalizedPattern && !/[*?]/.test(normalizedPattern) && repoPath.startsWith(`${normalizedPattern}/`)
+    })
+  const isIgnoredFile = absolute => {
+    const resolved = canonicalPath(absolute)
+    return excludedFiles.has(resolved) || repoPathsFor(resolved).some(matchesIgnoredRepoPath)
+  }
+  const isIgnoredDirectoryTree = absolute => repoPathsFor(absolute).some(repoPath =>
+    ignorePaths.some(pattern => {
+      const normalizedPattern = normalizeRepoPath(pattern)
+      const literalDirectory = !/[*?]/.test(normalizedPattern) && normalizedPattern.replace(/\/+$/, '') === repoPath
+      const subtreeGlob = normalizedPattern.endsWith('/**') && matchesGlob(repoPath, normalizedPattern.slice(0, -3))
+      return literalDirectory || subtreeGlob
+    }))
   const addFile = absolute => {
-    if (stopped || seen.has(absolute)) return
+    if (stopped || seen.has(absolute) || isIgnoredFile(absolute)) return
     seen.add(absolute)
 
     let st
@@ -288,7 +331,7 @@ const collectFilesDetailed = (targets, { includeDocs = false } = {}) => {
   }
 
   const walk = absolute => {
-    if (stopped) return
+    if (stopped || excludedFiles.has(canonicalPath(absolute))) return
     let st
     try {
       st = fs.lstatSync(absolute)
@@ -301,6 +344,7 @@ const collectFilesDetailed = (targets, { includeDocs = false } = {}) => {
       addFile(absolute)
       return
     }
+    if (isIgnoredDirectoryTree(absolute)) return
     if (SKIP_DIRS.has(path.basename(absolute))) return
     let names
     try {
@@ -373,11 +417,24 @@ const collectFilesDetailed = (targets, { includeDocs = false } = {}) => {
 
 export const collectFiles = (targets, options = {}) => collectFilesDetailed(targets, options).files
 
-export const scanTargets = ({ targets, entries, keys, includeDocs = false }) => {
-  const collected = collectFilesDetailed(targets, { includeDocs })
+export const scanTargets = ({
+  targets,
+  entries,
+  keys,
+  includeDocs = false,
+  ignoreModels = [],
+  ignorePaths = [],
+  ignoreRoots = [],
+  ignoredFiles = [],
+}) => {
+  const collected = collectFilesDetailed(targets, { includeDocs, ignorePaths, ignoreRoots, ignoredFiles })
   if (collected.validTargets.length === 0) throw new Error('no valid targets remain')
   const { files } = collected
   const pattern = buildModelPattern(keys)
+  const ignoredModelKeys = new Set(ignoreModels)
+  const ignoredCanonicalIds = new Set(ignoreModels.map(key => entries.get(key)?.entry.id ?? key))
+  const ignoresModel = (matched, id = null) =>
+    ignoredModelKeys.has(matched) || (id !== null && ignoredCanonicalIds.has(id))
   const modelRefs = []
   const candidateRefs = []
   const integrationHints = []
@@ -424,6 +481,7 @@ export const scanTargets = ({ targets, entries, keys, includeDocs = false }) =>
       let m
       while ((m = pattern.exec(lineText)) !== null) {
         const record = entries.get(m[1])
+        if (ignoresModel(m[1], record.entry.id)) continue
         const occurrence = [...lineText.matchAll(new RegExp(escapeRegExp(m[1]), 'g'))]
           .findIndex(match => match.index === m.index)
         const classification = classifyRef({
@@ -453,6 +511,7 @@ export const scanTargets = ({ targets, entries, keys, includeDocs = false }) =>
       }
 
       for (const candidate of findCandidates(lineText, entries)) {
+        if (ignoresModel(candidate)) continue
         const classification = classifyRef({
           file,
           line: i + 1,

package/package.json

@@ -1,13 +1,14 @@
 {
   "name": "model-eol",
-  "version": "0.2.3",
-  "description": "A machine-readable deprecation feed format for AI models, plus a zero-dependency checker and inventory CLI.",
+  "version": "0.3.0",
+  "description": "AI model deprecation feeds with a zero-dependency checker, inventory CLI, and migration bot.",
   "type": "module",
   "bin": {
-    "model-eol": "check.mjs"
+    "model-eol": "check.mjs",
+    "model-eol-bot": "bot/bot.mjs"
   },
   "scripts": {
-    "test": "node test/run.mjs && node refresh/test/run.mjs && node bot/test/run.mjs && node scripts/test-feed-changelog.mjs"
+    "test": "node test/run.mjs && node refresh/test/run.mjs && node bot/test/run.mjs && node scripts/test-action-contract.mjs && node bot/test/package.mjs && node scripts/test-feed-changelog.mjs"
   },
   "repository": {
     "type": "git",
@@ -31,7 +32,10 @@
   ],
   "files": [
     "check.mjs",
+    "bot.yml.example",
     "lib/",
+    "bot/bot.mjs",
+    "bot/lib/",
     "feeds/",
     "schema/",
     "SPEC.md",

package/schema/model-eol.bot-config.schema.json

@@ -1,8 +1,8 @@
 {
   "$schema": "http://json-schema.org/draft-07/schema#",
   "$id": "https://example.invalid/model-eol/0.1/bot-config.schema.json",
-  "title": "model-eol bot configuration",
-  "description": "Optional .model-eol.json configuration for the zero-dependency GitHub bot adapter.",
+  "title": "model-eol repository configuration",
+  "description": "Optional strict .model-eol.json configuration shared by the zero-dependency CLI, Action, and GitHub bot adapter.",
   "type": "object",
   "additionalProperties": false,
   "properties": {
@@ -15,10 +15,11 @@
     "scope": {
       "enum": ["direct", "all"],
       "default": "direct",
-      "description": "Use direct API references only, or include generic model references."
+      "description": "Use the direct-first scope (direct API plus generic model references, leaving cloud/gateway references as hints), or include all tracked references."
     },
     "via": {
       "type": ["string", "null"],
+      "minLength": 1,
       "default": null,
       "description": "Optional distributor lifecycle clock, such as azure-ai-foundry."
     },
@@ -42,13 +43,13 @@
           "type": "array",
           "items": { "type": "string", "minLength": 1 },
           "default": [],
-          "description": "Canonical IDs or aliases to ignore. A canonical ID ignores all of its aliases."
+          "description": "Canonical IDs or aliases to ignore across CLI, Action, and bot. Either form ignores every alias of the canonical feed entry."
         },
         "paths": {
           "type": "array",
           "items": { "type": "string", "minLength": 1 },
           "default": [],
-          "description": "Repo-relative simple globs supporting *, **, and ?."
+          "description": "Repo-relative simple globs supporting *, **, and ?. Matching paths are excluded before file coverage checks where possible."
         }
       },
       "default": { "models": [], "paths": [] }
@@ -71,17 +72,20 @@
       "properties": {
         "command": {
           "type": ["string", "null"],
+          "minLength": 1,
           "default": null,
-          "description": "Opt-in command run in the temporary clone for changed patches."
+          "description": "Opt-in evaluation command. Direct bot --eval runs it per changed patch in a temporary clone; the split example workflow runs it once as a read-only plan preflight."
         },
         "timeout_ms": {
           "type": "integer",
-          "minimum": 1,
+          "minimum": 1000,
+          "maximum": 3600000,
           "default": 600000
         },
         "max_report_bytes": {
           "type": "integer",
-          "minimum": 0,
+          "minimum": 1,
+          "maximum": 8388608,
           "default": 65536
         },
         "pass_env": {