@soulbatical/tetra-dev-toolkit 1.8.1 → 1.8.2

package/lib/checks/health/naming-conventions.js

@@ -42,6 +42,39 @@ function scanDatabaseNaming(projectPath, options = {}) {
 
   if (sqlFiles.length === 0) return { totalFields: 0, compliant: 0, violations: [], compliancePercent: 100 }
 
+  // Sort migration files chronologically so we can track renames
+  sqlFiles.sort()
+
+  // First pass: collect all renames and late-added columns from ALL migration files.
+  // These are used to suppress violations from earlier migrations.
+  const renamedColumns = new Map()  // "table.old_col" → "new_col"
+  const renamedIndexes = new Map()  // "old_index" → "new_index"
+  const lateAddedColumns = new Map() // "table.col" → true (columns added via ALTER TABLE ADD COLUMN)
+
+  for (const filePath of sqlFiles) {
+    let content
+    try { content = readFileSync(filePath, 'utf-8') } catch { continue }
+
+    // Track RENAME COLUMN: ALTER TABLE x RENAME COLUMN old TO new
+    const renameColRegex = /ALTER\s+TABLE\s+(?:IF\s+EXISTS\s+)?(?:public\.)?["']?(\w+)["']?\s+RENAME\s+COLUMN\s+["']?(\w+)["']?\s+TO\s+["']?(\w+)["']?/gi
+    let m
+    while ((m = renameColRegex.exec(content)) !== null) {
+      renamedColumns.set(`${m[1]}.${m[2]}`, m[3])
+    }
+
+    // Track RENAME INDEX: ALTER INDEX old RENAME TO new
+    const renameIdxRegex = /ALTER\s+INDEX\s+(?:IF\s+EXISTS\s+)?["']?(\w+)["']?\s+RENAME\s+TO\s+["']?(\w+)["']?/gi
+    while ((m = renameIdxRegex.exec(content)) !== null) {
+      renamedIndexes.set(m[1], m[2])
+    }
+
+    // Track ADD COLUMN (timestamps added later fix "missing timestamps" violations)
+    const addColRegex = /ALTER\s+TABLE\s+(?:IF\s+EXISTS\s+)?(?:public\.)?["']?(\w+)["']?\s+ADD\s+(?:COLUMN\s+)?(?:IF\s+NOT\s+EXISTS\s+)?["']?(\w+)["']?\s+\w+/gi
+    while ((m = addColRegex.exec(content)) !== null) {
+      lateAddedColumns.set(`${m[1]}.${m[2]}`, true)
+    }
+  }
+
   const globalColumns = {}
 
   const checkColumn = (tableName, colName, colType, body, fileName) => {
@@ -140,6 +173,59 @@ function scanDatabaseNaming(projectPath, options = {}) {
     }
   }
 
+  // ── Filter violations resolved by later migrations ──
+
+  // Helper: check if a column was renamed to a compliant name in a later migration
+  const isColumnRenamed = (table, col) => {
+    const key = `${table}.${col}`
+    return renamedColumns.has(key)
+  }
+
+  // Helper: check if an index was renamed to a compliant name in a later migration
+  const isIndexRenamed = (indexName) => {
+    return renamedIndexes.has(indexName)
+  }
+
+  // Helper: check if a missing timestamp was added in a later migration
+  const isTimestampAddedLater = (table, col) => {
+    return lateAddedColumns.has(`${table}.${col}`)
+  }
+
+  // Filter out resolved violations
+  const unresolvedViolations = violations.filter(v => {
+    // Column violations: "Boolean/JSON/Column "table.col" ..."
+    const colMatch = v.match(/(?:Boolean|JSON|Column|PK|FK)\s+"(\w+)\.(\w+)"/)
+    if (colMatch && isColumnRenamed(colMatch[1], colMatch[2])) {
+      compliant++ // was a violation, now fixed
+      return false
+    }
+
+    // Index violations: 'Index "name" should use idx_...'
+    const idxMatch = v.match(/Index\s+"(\w+)"\s+should/)
+    if (idxMatch && isIndexRenamed(idxMatch[1])) {
+      compliant++
+      return false
+    }
+
+    // Missing timestamp violations: 'Table "x" missing created_at, updated_at'
+    const tsMatch = v.match(/Table\s+"(\w+)"\s+missing\s+(.+?)(?:\s+\()/)
+    if (tsMatch) {
+      const table = tsMatch[1]
+      const missingCols = tsMatch[2].split(/,\s*/)
+      const allFixed = missingCols.every(col => isTimestampAddedLater(table, col.trim()))
+      if (allFixed) {
+        compliant++
+        return false
+      }
+    }
+
+    return true
+  })
+
+  // Replace violations array with filtered version
+  violations.length = 0
+  violations.push(...unresolvedViolations)
+
   // Consistency checks
   const consistencyGroups = [
     { concept: 'creation timestamp', preferred: 'created_at', alternatives: ['created_on', 'creation_date', 'inserted_at', 'date_created'] },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@soulbatical/tetra-dev-toolkit",
-  "version": "1.8.1",
+  "version": "1.8.2",
   "publishConfig": {
     "access": "restricted"
   },