sot-validator 0.1.0 → 0.1.1

package/index.js

@@ -12,7 +12,7 @@
  * @license CC-BY-4.0
  */
 
-const VERSION = '0.1.0';
+const VERSION = '0.1.1';
 
 /**
  * Validate a Source of Truth (.sot) file
@@ -23,8 +23,11 @@ function validate(content) {
   const errors = [];
   const warnings = [];
   
-  // Required: Header block
-  if (!content.includes('-- Source of Truth')) {
+  // Strip BOM if present
+  const cleanContent = content.replace(/^\uFEFF/, '');
+  
+  // Required: Header block (case-insensitive)
+  if (!/--\s*source\s+of\s+truth/i.test(cleanContent)) {
     errors.push({
       code: 'MISSING_HEADER',
       message: 'Document must contain "-- Source of Truth" header',
@@ -32,17 +35,32 @@ function validate(content) {
     });
   }
   
-  // Required: Last Updated
-  if (!content.match(/\*\*Last Updated:\*\*/)) {
+  // Required: Last Updated (case-insensitive)
+  if (!/\*\*last\s*updated:\*\*/i.test(cleanContent)) {
     errors.push({
       code: 'MISSING_DATE',
       message: 'Document must contain "**Last Updated:**" field',
       line: null
     });
+  } else {
+    // Validate date is not in future
+    const dateMatch = cleanContent.match(/\*\*last\s*updated:\*\*\s*(\d{4}-\d{2}-\d{2})/i);
+    if (dateMatch) {
+      const docDate = new Date(dateMatch[1]);
+      const today = new Date();
+      today.setHours(0, 0, 0, 0);
+      if (docDate > today) {
+        errors.push({
+          code: 'FUTURE_DATE',
+          message: `Last Updated date (${dateMatch[1]}) is in the future`,
+          line: null
+        });
+      }
+    }
   }
   
-  // Required: Owner
-  if (!content.match(/\*\*Owner:\*\*/)) {
+  // Required: Owner (case-insensitive)
+  if (!/\*\*owner:\*\*/i.test(cleanContent)) {
     errors.push({
       code: 'MISSING_OWNER',
       message: 'Document must contain "**Owner:**" field',
@@ -50,8 +68,8 @@ function validate(content) {
     });
   }
   
-  // Required: Status
-  if (!content.match(/\*\*Status:\*\*/)) {
+  // Required: Status (case-insensitive)
+  if (!/\*\*status:\*\*/i.test(cleanContent)) {
     errors.push({
       code: 'MISSING_STATUS',
       message: 'Document must contain "**Status:**" field',
@@ -59,17 +77,28 @@ function validate(content) {
     });
   }
   
-  // Required: Verification Status table
-  if (!content.includes('## Verification Status')) {
+  // Required: Verification Status section (case-insensitive)
+  const hasVerificationSection = /##\s*verification\s+status/i.test(cleanContent);
+  if (!hasVerificationSection) {
     errors.push({
       code: 'MISSING_VERIFICATION_TABLE',
       message: 'Document must contain "## Verification Status" section',
       line: null
     });
+  } else {
+    // BUG FIX #4: Check that section contains a table
+    const verificationSection = cleanContent.match(/##\s*verification\s+status[\s\S]*?(?=##|$)/i);
+    if (verificationSection && !verificationSection[0].includes('|')) {
+      warnings.push({
+        code: 'NO_TABLE_IN_VERIFICATION',
+        message: 'Verification Status section should contain a markdown table',
+        line: null
+      });
+    }
   }
   
-  // Warning: Status says VERIFIED without qualification
-  if (content.match(/Status:\*\*\s*VERIFIED\s*$/m)) {
+  // Warning: Status says VERIFIED without qualification (case-insensitive)
+  if (/status:\*\*\s*verified\s*$/im.test(cleanContent)) {
     warnings.push({
       code: 'UNQUALIFIED_VERIFIED',
       message: 'Status "VERIFIED" should be qualified (e.g., "with noted exceptions")',
@@ -77,23 +106,37 @@ function validate(content) {
     });
   }
   
-  // Warning: Estimates in Verified Data section
-  const verifiedDataSection = content.match(/## Verified Data[\s\S]*?(?=##|$)/);
-  if (verifiedDataSection && verifiedDataSection[0].match(/~\d|estimated|approximately/i)) {
-    warnings.push({
-      code: 'ESTIMATES_IN_VERIFIED',
-      message: 'Estimates found in Verified Data section - move to Estimates section',
-      line: null
-    });
-  }
+  // Extract Verified Data section for scoped checks
+  const verifiedDataMatch = cleanContent.match(/##\s*verified\s+data[\s\S]*?(?=##|$)/i);
+  const verifiedDataSection = verifiedDataMatch ? verifiedDataMatch[0] : null;
   
-  // Warning: Missing staleness markers
-  if (content.includes('## Verified Data') && !content.match(/\[STABLE\]|\[VOLATILE\]|\[CHECK BEFORE CITING\]|\[SNAPSHOT\]/)) {
-    warnings.push({
-      code: 'MISSING_STALENESS',
-      message: 'Verified Data should include staleness markers ([STABLE], [VOLATILE], etc.)',
-      line: null
-    });
+  if (verifiedDataSection) {
+    // BUG FIX #4: Check that Verified Data contains a table
+    if (!verifiedDataSection.includes('|')) {
+      warnings.push({
+        code: 'NO_TABLE_IN_VERIFIED_DATA',
+        message: 'Verified Data section should contain a markdown table',
+        line: null
+      });
+    }
+    
+    // Warning: Estimates in Verified Data section
+    if (/~\d|estimated|approximately/i.test(verifiedDataSection)) {
+      warnings.push({
+        code: 'ESTIMATES_IN_VERIFIED',
+        message: 'Estimates found in Verified Data section - move to Estimates section',
+        line: null
+      });
+    }
+    
+    // BUG FIX #1: Check staleness markers WITHIN Verified Data section only
+    if (!/\[STABLE\]|\[VOLATILE\]|\[CHECK BEFORE CITING\]|\[SNAPSHOT\]/i.test(verifiedDataSection)) {
+      warnings.push({
+        code: 'MISSING_STALENESS',
+        message: 'Verified Data should include staleness markers ([STABLE], [VOLATILE], etc.)',
+        line: null
+      });
+    }
   }
   
   return {
@@ -119,8 +162,9 @@ function isValid(content) {
  * @returns {boolean} True if appears to be .sot format
  */
 function detect(content) {
-  return content.includes('-- Source of Truth') && 
-         content.includes('## Verification Status');
+  const cleanContent = content.replace(/^\uFEFF/, '');
+  return /--\s*source\s+of\s+truth/i.test(cleanContent) && 
+         /##\s*verification\s+status/i.test(cleanContent);
 }
 
 module.exports = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sot-validator",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Validator for Source of Truth (.sot) files - epistemic quality verification for AI-safe documentation",
   "main": "index.js",
   "types": "index.d.ts",