yini-cli 1.2.0-beta → 1.2.1-beta

package/dist/commands/validateCommand.js

@@ -1,7 +1,6 @@
-import assert from 'node:assert';
 import { exit } from 'node:process';
 import YINI from 'yini-parser';
-const IS_DEBUG = true; // For local debugging purposes, etc.
+const IS_DEBUG = false; // For local debugging purposes, etc.
 /*
     TODO / SHOULD-DO:
 
@@ -140,18 +139,8 @@ const IS_DEBUG = true; // For local debugging purposes, etc.
 export const validateFile = (file, commandOptions = {}) => {
     let parsedResult = undefined;
     let isCatchedError = true;
-    // let failLevel: PreferredFailLevel = 'auto'
-    // if (commandOptions.failFast) {
-    //     failLevel = 'warnings-and-errors'
-    // }
-    // if (commandOptions.bestEffort) {
-    //     failLevel = 'ignore-errors'
-    // }
     const parseOptions = {
         strictMode: commandOptions.strict ?? false,
-        // failLevel: 'errors',
-        // failLevel: commandOptions.force ? 'ignore-errors' : 'errors',
-        // failLevel: 'ignore-errors',
         failLevel: 'ignore-errors',
         includeMetadata: true,
         includeDiagnostics: true,
@@ -163,6 +152,10 @@ export const validateFile = (file, commandOptions = {}) => {
     }
     catch (err) {
         isCatchedError = true;
+        if (!commandOptions.silent) {
+            const message = err instanceof Error ? err.message : String(err);
+            console.error(`Error: ${message}`);
+        }
     }
     let metadata = null;
     let errors = 0;
@@ -171,9 +164,13 @@ export const validateFile = (file, commandOptions = {}) => {
     let infos = 0;
     if (!isCatchedError && parsedResult?.meta) {
         metadata = parsedResult?.meta;
-        assert(metadata); // Make sure there is metadata!
+        // assert(metadata) // Make sure there is metadata!
         // printObject(metadata, true)
-        assert(metadata.diagnostics);
+        // assert(metadata.diagnostics)
+        if (!metadata?.diagnostics) {
+            console.error('Internal error: Missing diagnostics metadata');
+            exit(1);
+        }
         const diag = metadata.diagnostics;
         errors = diag.errors.errorCount;
         warnings = diag.warnings.warningCount;
@@ -183,11 +180,10 @@ export const validateFile = (file, commandOptions = {}) => {
     IS_DEBUG && console.log();
     IS_DEBUG && console.log('isCatchedError = ' + isCatchedError);
     IS_DEBUG && console.log('TEMP OUTPUT');
-    IS_DEBUG && console.log('isCatchedError = ' + isCatchedError);
     IS_DEBUG && console.log('  errors = ' + errors);
     IS_DEBUG && console.log('warnings = ' + warnings);
     IS_DEBUG && console.log(' notices = ' + notices);
-    IS_DEBUG && console.log('   infor = ' + infos);
+    IS_DEBUG && console.log('   infos = ' + infos);
     IS_DEBUG && console.log('metadata = ' + metadata);
     IS_DEBUG &&
         console.log('includeMetadata = ' +
@@ -244,17 +240,18 @@ export const validateFile = (file, commandOptions = {}) => {
         // if (commandOptions.details) {
         if (errors || warnings) {
             if (!metadata) {
-                console.error('Internal Error: No meta data found');
+                console.error('Internal error: No metadata available');
+                exit(1);
             }
-            assert(metadata); // Make sure there is metadata!
+            // assert(metadata) // Make sure there is metadata!
             console.log();
             printIssuesFound(file, metadata);
         }
         if (commandOptions.stats) {
             if (!metadata) {
-                console.error('Internal Error: No meta data found');
+                console.error('Internal error: No metadata available');
+                exit(1);
             }
-            assert(metadata); // Make sure there is metadata!
             console.log();
             console.log(formatToStatsReport(file, metadata).trim());
         }
@@ -278,7 +275,7 @@ mode, errors, warnings, notices, infos) => {
             break;
         case 'Passed-with-Warnings':
             // result = '⚠️ Validation finished'
-            result = '✔  Validation successful';
+            result = '✔  Validation successful (with warnings)';
             break;
         case 'Failed':
             // result = '✖  Validation failed'
@@ -355,7 +352,10 @@ const formatToStatsReport = (fileWithPath, metadata) => {
     // console.log('formatToStatsReport(..)')
     // printObject(metadata)
     // console.log()
-    assert(metadata.diagnostics);
+    if (!metadata?.diagnostics) {
+        console.error('Internal error: Missing diagnostics');
+        exit(1);
+    }
     const diag = metadata.diagnostics;
     const issuesCount = diag.errors.errorCount +
         diag.warnings.warningCount +
@@ -408,7 +408,10 @@ const printIssuesFound = (fileWithPath, metadata) => {
     // printObject(metadata)
     // console.log(toPrettyJSON(metadata))
     // console.log()
-    assert(metadata.diagnostics);
+    if (!metadata?.diagnostics) {
+        console.error('Internal error: Missing diagnostics metadata');
+        exit(1);
+    }
     const diag = metadata.diagnostics;
     IS_DEBUG && console.log('*** Issues Found');
     IS_DEBUG && console.log('*** ------------');

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "yini-cli",
-    "version": "1.2.0-beta",
+    "version": "1.2.1-beta",
     "description": "CLI tool for YINI text-based format - an INI-like config format with real structure, comments, nested sections, no YAML foot-guns, and less noise than JSON.",
     "keywords": [
         "yini",