yini-parser 1.5.0 → 1.6.0

package/dist/core/errorDataHandler.d.ts

@@ -8,6 +8,7 @@ export declare class ErrorDataHandler {
     private readonly subjectType;
     private readonly fileName;
     private readonly persistThreshold;
+    private readonly isDiagnosticOutputEnabled;
     private readonly isQuiet;
     private readonly isSilent;
     private readonly isThrowOnError;
@@ -30,7 +31,8 @@ export declare class ErrorDataHandler {
             - Level 1 = abort on errors only
             - Level 2 = abort even on warnings
      */
-    constructor(subjectType: TSubjectType, fileName?: string | undefined, bailSensitivityLevel?: TBailSensitivityLevel, isQuiet?: boolean, // Reduce output (show only errors, does not effect warnings and etc. in meta data).
+    constructor(subjectType: TSubjectType, fileName?: string | undefined, bailSensitivityLevel?: TBailSensitivityLevel, isDiagnosticOutputEnabled?: boolean, // Opt in to writing diagnostics to stderr.
+    isQuiet?: boolean, // When diagnostic output is enabled, show errors only.
     isSilent?: boolean, // Suppress all output (even errors, exit code only).
     isThrowOnError?: boolean);
     private makeIssue;
@@ -50,6 +52,9 @@ export declare class ErrorDataHandler {
      */
     pushOrBail(locInput: IErrorLocationInput | undefined, type: TIssueType, msgWhat: string, msgWhy?: string, msgHint?: string): void;
     private formatSignificantMessageLine;
+    private shouldEmitDiagnostics;
+    private shouldEmitNonErrors;
+    private emitDiagnosticToStderr;
     private emitFatalError;
     private emitInternalError;
     private emitSyntaxError;

package/dist/core/errorDataHandler.js

@@ -42,7 +42,8 @@ class ErrorDataHandler {
             - Level 1 = abort on errors only
             - Level 2 = abort even on warnings
      */
-    constructor(subjectType, fileName = undefined, bailSensitivityLevel = '1-Abort-on-Errors', isQuiet = false, // Reduce output (show only errors, does not effect warnings and etc. in meta data).
+    constructor(subjectType, fileName = undefined, bailSensitivityLevel = '1-Abort-on-Errors', isDiagnosticOutputEnabled = false, // Opt in to writing diagnostics to stderr.
+    isQuiet = false, // When diagnostic output is enabled, show errors only.
     isSilent = false, // Suppress all output (even errors, exit code only).
     isThrowOnError = false) {
         this.errors = [];
@@ -58,6 +59,7 @@ class ErrorDataHandler {
         this.subjectType = subjectType;
         this.fileName = fileName;
         this.persistThreshold = bailSensitivityLevel;
+        this.isDiagnosticOutputEnabled = isDiagnosticOutputEnabled;
         this.isQuiet = isQuiet;
         this.isSilent = isSilent;
         this.isThrowOnError = isThrowOnError;
@@ -118,8 +120,8 @@ class ErrorDataHandler {
             lineNum: lineNum || 0, // 1-based, if n/a use 0.
             colNum: colNum || 0, // 1-based, if n/a use 0.
         };
-        if (!this.isSilent) {
-            console.log(); // Print an empty line before outputting message.
+        if (this.shouldEmitDiagnostics()) {
+            console.error(); // Print an empty line before outputting message.
         }
         switch (type) {
             case 'Internal-Error':
@@ -229,68 +231,53 @@ class ErrorDataHandler {
             }
         }
     }
-    /*
-     * - error/warning → console.error / console.warn
-     * - notice/info → console.log / console.info
-     */
+    shouldEmitDiagnostics() {
+        return this.isDiagnosticOutputEnabled && !this.isSilent;
+    }
+    shouldEmitNonErrors() {
+        return this.shouldEmitDiagnostics() && !this.isQuiet;
+    }
+    emitDiagnosticToStderr(write, messageHeader, msgWhat = '', msgWhy = '', msgHint = '') {
+        write(messageHeader);
+        msgWhat && write(msgWhat);
+        msgWhy && write(msgWhy);
+        msgHint && write(msgHint);
+        write();
+    }
     emitFatalError(loc, msgWhat = 'Something went wrong!', msgWhy = '', msgHint = '') {
         const messageHeader = this.formatSignificantMessageLine(loc, issueTitle[0]);
-        if (!this.isSilent) {
-            console.error(messageHeader); // Print the issue title.
-            msgWhat && console.log(msgWhat);
-            msgWhy && console.log(msgWhy);
-            msgHint && console.log(msgHint);
-            console.log(); // Emit an empty line before outputting message.
+        if (this.shouldEmitDiagnostics()) {
+            this.emitDiagnosticToStderr(console.error.bind(console), messageHeader, msgWhat, msgWhy, msgHint);
         }
     }
     emitInternalError(loc, msgWhat = 'Something went wrong!', msgWhy = '', msgHint = '') {
         const messageHeader = this.formatSignificantMessageLine(loc, issueTitle[1]);
-        if (!this.isSilent) {
-            console.error(messageHeader); // Print the issue title.
-            msgWhat && console.log(msgWhat);
-            msgWhy && console.log(msgWhy);
-            msgHint && console.log(msgHint);
-            console.log(); // Emit an empty line before outputting message.
+        if (this.shouldEmitDiagnostics()) {
+            this.emitDiagnosticToStderr(console.error.bind(console), messageHeader, msgWhat, msgWhy, msgHint);
         }
     }
     emitSyntaxError(loc, msgWhat, msgWhy = '', msgHint = '') {
         const messageHeader = this.formatSignificantMessageLine(loc, issueTitle[2]);
-        if (!this.isSilent) {
-            console.error(messageHeader); // Print the issue title.
-            msgWhat && console.log(msgWhat);
-            msgWhy && console.log(msgWhy);
-            msgHint && console.log(msgHint);
-            console.log(); // Emit an empty line before outputting message.
+        if (this.shouldEmitDiagnostics()) {
+            this.emitDiagnosticToStderr(console.error.bind(console), messageHeader, msgWhat, msgWhy, msgHint);
         }
     }
     emitSyntaxWarning(loc, msgWhat, msgWhy = '', msgHint = '') {
         const messageHeader = this.formatSignificantMessageLine(loc, issueTitle[3]);
-        if (!this.isQuiet && !this.isSilent) {
-            console.warn(messageHeader); // Print the issue title.
-            msgWhat && console.log(msgWhat);
-            msgWhy && console.log(msgWhy);
-            msgHint && console.log(msgHint);
-            console.log(); // Emit an empty line before outputting message.
+        if (this.shouldEmitNonErrors()) {
+            this.emitDiagnosticToStderr(console.warn.bind(console), messageHeader, msgWhat, msgWhy, msgHint);
         }
     }
     emitNotice(loc, msgWhat, msgWhy = '', msgHint = '') {
         const messageHeader = this.formatSignificantMessageLine(loc, issueTitle[4]);
-        if (!this.isQuiet && !this.isSilent) {
-            console.log(messageHeader); // Print the issue title.
-            msgWhat && console.log(msgWhat);
-            msgWhy && console.log(msgWhy);
-            msgHint && console.log(msgHint);
-            console.log(); // Emit an empty line before outputting message.
+        if (this.shouldEmitNonErrors()) {
+            this.emitDiagnosticToStderr(console.error.bind(console), messageHeader, msgWhat, msgWhy, msgHint);
         }
     }
     emitInfo(loc, msgWhat, msgWhy = '', msgHint = '') {
         const messageHeader = this.formatSignificantMessageLine(loc, issueTitle[5]);
-        if (!this.isQuiet && !this.isSilent) {
-            console.info(messageHeader); // Print the issue title.
-            msgWhat && console.log(msgWhat);
-            msgWhy && console.log(msgWhy);
-            msgHint && console.log(msgHint);
-            console.log(); // Emit an empty line before outputting message.
+        if (this.shouldEmitNonErrors()) {
+            this.emitDiagnosticToStderr(console.error.bind(console), messageHeader, msgWhat, msgWhy, msgHint);
         }
     }
     getNumOfAllMessages() {

package/dist/core/internalTypes.d.ts

@@ -70,7 +70,7 @@ export type TObjectValue = {
     entries: Readonly<Record<string, TValueLiteral>>;
     tag: string | undefined;
 };
-type TStringKind = 'raw' | 'classic' | 'hyper' | 'triple-raw' | 'triple-classic';
+type TStringKind = 'raw' | 'classic' | 'triple-raw' | 'triple-classic';
 export interface IParsedStringInput {
     strKind: TStringKind;
     value: string;
@@ -82,6 +82,13 @@ export interface IErrorLocationInput {
     endColumn?: number;
 }
 export type TIssueType = 'Fatal-Error' | 'Internal-Error' | 'Syntax-Error' | 'Syntax-Warning' | 'Notice' | 'Info';
+export interface IPreflightIssue {
+    locInput: IErrorLocationInput | undefined;
+    type: TIssueType;
+    msgWhat: string;
+    msgWhy?: string;
+    msgHint?: string;
+}
 interface IMetaBaseInfo {
     sourceType: TSourceType;
     fileName: string | undefined;
@@ -96,6 +103,7 @@ export interface IRuntimeInfo extends IMetaBaseInfo {
     timeIoMs: number | null;
     preferredBailSensitivity: null | PreferredFailLevel;
     sha256: string | null;
+    preflightIssues: IPreflightIssue[];
 }
 export interface IParseCoreOptions {
     rules: IParseRuleOptions;
@@ -104,6 +112,7 @@ export interface IParseCoreOptions {
     isWithDiagnostics: boolean;
     isWithTiming: boolean;
     isKeepUndefinedInMeta: boolean;
+    isDiagnosticOutputEnabled: boolean;
     isQuiet: boolean;
     isSilent: boolean;
     isThrowOnError: boolean;

package/dist/core/objectBuilder.js

@@ -17,6 +17,7 @@ const print_1 = require("../utils/print");
  *
  * @note All `tag` fields MUST be ignored.
  */
+// src/core/objectBuilder.ts
 const astToObject = (ast, errorHandler) => {
     (0, print_1.debugPrint)('-> constructFinalObject(..)');
     const out = {};
@@ -28,12 +29,12 @@ const astToObject = (ast, errorHandler) => {
     }
     // 2) Mount explicit top-level sections directly onto result.
     for (const child of ast.root.children) {
-        // Collision: a lenient orphan member already used this name.
         if (Object.prototype.hasOwnProperty.call(out, child.sectionName)) {
             errorHandler.pushOrBail(undefined, 'Syntax-Error', `Name collision between orphan member and section '${child.sectionName}'`, `The name '${child.sectionName}' is already used by a top-level member outside any explicit section and cannot also be used as a top-level section name.`, `Rename either the orphan member or the section to avoid the collision.`);
             continue;
         }
-        define(out, child.sectionName, sectionToObject(child));
+        // define(out, child.sectionName, sectionToObject(child))
+        define(out, child.sectionName, sectionToObject(child, errorHandler));
     }
     return out;
 };
@@ -42,15 +43,29 @@ exports.astToObject = astToObject;
  * Convert a section (members + nested sections) to a plain object,
  * preserving insertion order.
  */
-const sectionToObject = (node) => {
+// const sectionToObject = (node: IYiniSection): Record<string, unknown> => {
+//     const obj: Record<string, unknown> = {}
+//     // 1) Members (Map preserves insertion order).
+//     for (const [key, val] of node.members) {
+//         define(obj, key, literalToJS(val))
+//     }
+//     // 2) Nested sections (array order preserved).
+//     for (const child of node.children) {
+//         define(obj, child.sectionName, sectionToObject(child))
+//     }
+//     return obj
+// }
+const sectionToObject = (node, errorHandler) => {
     const obj = {};
-    // 1) Members (Map preserves insertion order).
     for (const [key, val] of node.members) {
         define(obj, key, literalToJS(val));
     }
-    // 2) Nested sections (array order preserved).
     for (const child of node.children) {
-        define(obj, child.sectionName, sectionToObject(child));
+        if (Object.prototype.hasOwnProperty.call(obj, child.sectionName)) {
+            errorHandler.pushOrBail(undefined, 'Syntax-Error', `Name collision between member and subsection '${child.sectionName}'`, `The name '${child.sectionName}' is already used by a member in section '${node.sectionName}' and cannot also be used as a subsection name.`, 'Rename either the member or the subsection.');
+            continue;
+        }
+        define(obj, child.sectionName, sectionToObject(child, errorHandler));
     }
     return obj;
 };

package/dist/core/options/defaultParserOptions.d.ts

@@ -7,11 +7,12 @@ export declare const getDefaultUserOptions: (mode: TParserMode) => {
     onDuplicateKey: import("../../types").OnDuplicateKey;
     requireDocTerminator: import("../../types").DocumentTerminatorRule;
     treatEmptyValueAsNull: import("../../types").EmptyValueRule;
+    logDiagnostics: boolean;
     quiet: boolean;
     silent: boolean;
     throwOnError: boolean;
-    strictMode: boolean;
     failLevel: import("../../types").PreferredFailLevel;
     includeMetadata: boolean;
+    strictMode: boolean;
 };
-export type TNormalizedUserOptions = Required<Pick<ParseOptions, 'strictMode' | 'failLevel' | 'includeMetadata' | 'includeDiagnostics' | 'includeTiming' | 'preserveUndefinedInMeta' | 'onDuplicateKey' | 'requireDocTerminator' | 'treatEmptyValueAsNull' | 'quiet' | 'silent' | 'throwOnError'>>;
+export type TNormalizedUserOptions = Required<Pick<ParseOptions, 'strictMode' | 'failLevel' | 'includeMetadata' | 'includeDiagnostics' | 'includeTiming' | 'preserveUndefinedInMeta' | 'onDuplicateKey' | 'requireDocTerminator' | 'treatEmptyValueAsNull' | 'logDiagnostics' | 'quiet' | 'silent' | 'throwOnError'>>;

package/dist/core/options/defaultParserOptions.js

@@ -14,7 +14,8 @@ const BASE_DEFAULTS = {
     onDuplicateKey: 'error',
     requireDocTerminator: 'optional',
     treatEmptyValueAsNull: 'allow-with-warning',
-    quiet: false, // Suppress warnings in console (does not affect warnings in meta data).
+    logDiagnostics: false,
+    quiet: false, // When logDiagnostics is true, suppress warnings in stderr output.
     silent: false,
     //@todo: Change default throwOnError to false
     throwOnError: true, // Will throw on first parse error encountered.

package/dist/core/options/optionsFunctions.js

@@ -15,13 +15,16 @@ const toCoreOptions = (userOpts, bailLevel) => {
             treatEmptyValueAsNull: userOpts.treatEmptyValueAsNull,
         },
         bailSensitivity: level,
-        isIncludeMeta: userOpts.includeMetadata,
+        isIncludeMeta: userOpts.includeMetadata ||
+            userOpts.includeDiagnostics ||
+            userOpts.includeTiming,
         // isWithDiagnostics: isDev() || isDebug() || userOpts.includeDiagnostics,
         isWithDiagnostics: userOpts.includeDiagnostics,
         // isWithTiming: isDev() || isDebug() || userOpts.includeTiming,
         isWithTiming: userOpts.includeTiming,
         // isKeepUndefinedInMeta: isDebug() || userOpts.preserveUndefinedInMeta,
         isKeepUndefinedInMeta: userOpts.preserveUndefinedInMeta,
+        isDiagnosticOutputEnabled: userOpts.logDiagnostics,
         isQuiet: userOpts.quiet, // Suppress warnings, etc.
         isSilent: userOpts.silent,
         isThrowOnError: userOpts.throwOnError,
@@ -41,6 +44,7 @@ const isOptionsObjectForm = (v) => {
             'onDuplicateKey' in v ||
             'requireDocTerminator' in v ||
             'treatEmptyValueAsNull' in v ||
+            'logDiagnostics' in v ||
             'quiet' in v ||
             'silent' in v ||
             'throwOnError' in v));

package/dist/core/pipeline/pipeline.js

@@ -63,12 +63,21 @@ const runPipeline = (yiniContent, coreOptions, runtimeInfo, _meta_userOpts) => {
     //     default:
     //         persistThreshold = '2-Abort-Even-on-Warnings'
     // }
-    const errorHandler = new errorDataHandler_1.ErrorDataHandler(runtimeInfo.sourceType, runtimeInfo.fileName, coreOptions.bailSensitivity, coreOptions.isQuiet, coreOptions.isSilent, coreOptions.isThrowOnError);
-    if (yiniContent.trim() === '') {
-        const isFileSourceType = runtimeInfo?.sourceType === 'File';
-        // Note, after pushing processing may continue or exit, depending on the error and/or the bail threshold.
-        errorHandler.pushOrBail(undefined, 'Syntax-Error', 'Empty YINI document.', `The input is blank or contains only whitespace in the ${isFileSourceType ? 'YINI file' : 'YINI inline content'}.`, `Tip: Add at least one section '^ SectionName' or a key-value pair 'key = value' to make it a valid YINI file.`);
+    const errorHandler = new errorDataHandler_1.ErrorDataHandler(runtimeInfo.sourceType, runtimeInfo.fileName, coreOptions.bailSensitivity, coreOptions.isDiagnosticOutputEnabled, coreOptions.isQuiet, coreOptions.isSilent, coreOptions.isThrowOnError);
+    for (const issue of runtimeInfo.preflightIssues) {
+        errorHandler.pushOrBail(issue.locInput, issue.type, issue.msgWhat, issue.msgWhy, issue.msgHint);
     }
+    // if (yiniContent.trim() === '') {
+    //     const isFileSourceType: boolean = runtimeInfo?.sourceType === 'File'
+    //     // Note, after pushing processing may continue or exit, depending on the error and/or the bail threshold.
+    //     errorHandler.pushOrBail(
+    //         undefined,
+    //         'Syntax-Error',
+    //         'Empty YINI document.',
+    //         `The input is blank or contains only whitespace in the ${isFileSourceType ? 'YINI file' : 'YINI inline content'}.`,
+    //         `Tip: Add at least one section '^ SectionName' or a key-value pair 'key = value' to make it a valid YINI file.`,
+    //     )
+    // }
     //---------------------------------------------
     // Note: Only computed when isWithTiming.
     let timeStartMs = 0;
@@ -140,8 +149,18 @@ const runPipeline = (yiniContent, coreOptions, runtimeInfo, _meta_userOpts) => {
     const builder = new astBuilder_1.default(errorHandler, coreOptions, runtimeInfo.sourceType, runtimeInfo.fileName || null);
     const ast = builder.buildAST(parseTree);
     if (ast.numOfMembers === 0 && ast.numOfSections === 0) {
-        // Note, after pushing processing may continue or exit, depending on the error and/or the bail threshold.
-        errorHandler.pushOrBail(undefined, 'Syntax-Error', 'No meaningful content.', `No sections or members found in the ${ast.sourceType === 'File' ? 'YINI file' : 'YINI inline content'}.`, `${ast.sourceType === 'File' ? 'A valid YINI file' : 'Any valid YINI inline content'} must contain at least one section '^ SectionName' or a key–value pair 'key = value' to make it a valid YINI file.`);
+        // Lenient mode: empty document is allowed, but warning.
+        // Strict mode: empty document is invalid, error.
+        const sourceLabel = ast.sourceType === 'File' ? 'YINI file' : 'YINI inline content';
+        const sourceRequirement = ast.sourceType === 'File'
+            ? 'A valid YINI file must contain at least one section, for example: ^ SectionName, or a key-value pair, for example: key = value.'
+            : 'Valid YINI inline content must contain at least one section, for example: ^ SectionName, or a key-value pair, for example: key = value.';
+        if (ast.isStrict === true) {
+            errorHandler.pushOrBail(undefined, 'Syntax-Error', 'Empty YINI document.', `The ${sourceLabel} contains no meaningful content.`, sourceRequirement);
+        }
+        else {
+            errorHandler.pushOrBail(undefined, 'Syntax-Warning', 'Empty YINI document.', `The ${sourceLabel} contains no meaningful content.`, sourceRequirement);
+        }
     }
     if ((0, env_1.isDebug)()) {
         console.log();
@@ -180,8 +199,9 @@ const runPipeline = (yiniContent, coreOptions, runtimeInfo, _meta_userOpts) => {
         // Note, after pushing processing may continue or exit, depending on the error and/or the bail threshold.
         errorHandler.pushOrBail(undefined, 'Syntax-Warning', 'Warning: Strict initialMode is not yet fully implemented.', 'Some validation rules may still be missing or incomplete.');
         if (coreOptions.bailSensitivity === '0-Ignore-Errors') {
-            // IMPORTANT: If "silent" option is set, do not log anything to console!
-            if (!coreOptions.isQuiet && !coreOptions.isSilent) {
+            if (coreOptions.isDiagnosticOutputEnabled &&
+                !coreOptions.isQuiet &&
+                !coreOptions.isSilent) {
                 console.warn(`Warning: The initial mode was set to strict mode, but fail level is set to 'ignore-errors'. This combination is contradictory and might be a mistake.`);
             }
         }
@@ -208,7 +228,8 @@ const runPipeline = (yiniContent, coreOptions, runtimeInfo, _meta_userOpts) => {
     const constructedMetadata = (0, resultMetadataBuilder_1.buildResultMetadata)(params);
     (0, print_1.debugPrint)('getNumOfErrors(): ' + errorHandler.getNumOfErrors());
     // Print a summary line at the end if any errors or warnings.
-    if (!coreOptions.isQuiet && !coreOptions.isSilent) {
+    if (coreOptions.isDiagnosticOutputEnabled &&
+        !coreOptions.isSilent) {
         const errors = errorHandler.getNumOfErrors();
         const warnings = errorHandler.getNumOfWarnings();
         // Notes:

package/dist/core/runtime.js

@@ -16,8 +16,10 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 var _YiniRuntime_runtime;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.YiniRuntime = void 0;
+// src/core/runtime.ts
 const fs_1 = __importDefault(require("fs"));
 const env_1 = require("../config/env");
+const validateShebangPlacement_1 = require("../parsers/validateShebangPlacement");
 const pathAndFileName_1 = require("../utils/pathAndFileName");
 const print_1 = require("../utils/print");
 const string_1 = require("../utils/string");
@@ -54,6 +56,7 @@ class YiniRuntime {
             timeIoMs: null,
             preferredBailSensitivity: null,
             sha256: null,
+            preflightIssues: [],
         };
     }
     // --- Single implementation --------------------------------------------
@@ -102,6 +105,19 @@ class YiniRuntime {
                 includeMetadata,
             };
         }
+        const shebangIssue = (0, validateShebangPlacement_1.getShebangPlacementIssue)(yiniContent, userOpts.strictMode);
+        if (shebangIssue) {
+            __classPrivateFieldGet(this, _YiniRuntime_runtime, "f").preflightIssues.push(shebangIssue);
+        }
+        const originalContent = yiniContent;
+        yiniContent = (0, validateShebangPlacement_1.stripBomAndValidShebang)(yiniContent);
+        if (originalContent.startsWith('\uFEFF')) {
+            (0, print_1.devPrint)('runParse(..): BOM was detected and stripped from UTF-8 content.');
+        }
+        if (originalContent.startsWith('#!') ||
+            originalContent.startsWith('\uFEFF#!')) {
+            (0, print_1.devPrint)('runParse(..): Shebang detected on first line and ignored.');
+        }
         if (userOpts.includeMetadata && __classPrivateFieldGet(this, _YiniRuntime_runtime, "f").sourceType === 'Inline') {
             const lineCount = yiniContent.split(/\r?\n/).length; // Counts the lines.
             const sha256 = (0, string_1.computeSha256)(yiniContent); // NOTE: Compute BEFORE any possible tampering of content.
@@ -111,9 +127,13 @@ class YiniRuntime {
         }
         // NOTE: Important: Do not trim or mutate the yiniContent here, due
         // to it will mess up the line numbers in error reporting.
-        if (!yiniContent) {
-            throw new Error('Syntax-Error: Unexpected blank YINI input');
+        // if (!yiniContent) {
+        //     throw new Error('Syntax-Error: Unexpected blank YINI input')
+        // }
+        if (yiniContent === null || yiniContent === undefined) {
+            throw new Error('Syntax-Error: Missing YINI input');
         }
+        // IMPORTANT: Makes sure input ends with an empty NL!
         if (!yiniContent.endsWith('\n')) {
             yiniContent += '\n';
         }
@@ -166,13 +186,6 @@ class YiniRuntime {
             };
         }
         if ((0, pathAndFileName_1.getFileNameExtension)(filePath).toLowerCase() !== '.yini') {
-            // IMPORTANT: If "silent" option is set, do not log anything to console!
-            if (!userOpts.silent) {
-                // In quiet-mode we still show errors (these are fine).
-                console.error('Invalid file extension for YINI file:');
-                console.error(`"${filePath}"`);
-                console.error('File does not have a valid ".yini" extension (case-insensitive).');
-            }
             throw new Error('Error: Unexpected file extension for YINI file');
         }
         // ---- Phase 0: I/O ----
@@ -192,22 +205,18 @@ class YiniRuntime {
             __classPrivateFieldGet(this, _YiniRuntime_runtime, "f").preferredBailSensitivity = userOpts.failLevel;
             __classPrivateFieldGet(this, _YiniRuntime_runtime, "f").sha256 = (0, string_1.computeSha256)(content); // NOTE: Compute BEFORE any possible tampering of content.
         }
-        let hasNoNewlineAtEOF = false;
         if (!content.endsWith('\n')) {
+            __classPrivateFieldGet(this, _YiniRuntime_runtime, "f").preflightIssues.push({
+                locInput: undefined,
+                type: 'Syntax-Warning',
+                msgWhat: 'No newline at end of file.',
+                msgWhy: `It's recommended to end a YINI file with a newline. File: "${filePath}"`,
+            });
             content += '\n';
-            hasNoNewlineAtEOF = true;
         }
         const result = this.runParse(content, {
             ...userOpts,
         });
-        // if (hasNoNewlineAtEOF && !userOpts.quiet && !userOpts.silent) {
-        if (hasNoNewlineAtEOF && !userOpts.quiet) {
-            // IMPORTANT: If "silent" option is set, do not log anything to console!
-            if (!userOpts.silent) {
-                //@todo: (or maybe not, 20250917) Maybe let errorHandler emit message
-                console.warn(`No newline at end of file, it's recommended to end a file with a newline. File:\n"${filePath}"`);
-            }
-        }
         return result;
     }
 }

package/dist/grammar/generated/YiniLexer.d.ts

@@ -1,4 +1,4 @@
-import { ATN, CharStream, DFA, Lexer, RuleContext } from "antlr4";
+import { ATN, CharStream, DFA, Lexer } from "antlr4";
 export default class YiniLexer extends Lexer {
     static readonly SHEBANG = 1;
     static readonly YINI_TOKEN = 2;
@@ -13,44 +13,39 @@ export default class YiniLexer extends Lexer {
     static readonly EMPTY_OBJECT = 11;
     static readonly EMPTY_LIST = 12;
     static readonly STRING = 13;
-    static readonly TRIPLE_QUOTED_STRING = 14;
-    static readonly SINGLE_OR_DOUBLE = 15;
-    static readonly R_AND_C_STRING = 16;
-    static readonly HYPER_STRING = 17;
-    static readonly NUMBER = 18;
-    static readonly SS = 19;
-    static readonly CARET = 20;
-    static readonly GT = 21;
-    static readonly LT = 22;
-    static readonly EQ = 23;
-    static readonly HASH = 24;
-    static readonly COMMA = 25;
-    static readonly COLON = 26;
-    static readonly OB = 27;
-    static readonly CB = 28;
-    static readonly OC = 29;
-    static readonly CC = 30;
-    static readonly PLUS = 31;
-    static readonly DOLLAR = 32;
-    static readonly PC = 33;
-    static readonly AT = 34;
-    static readonly SEMICOLON = 35;
-    static readonly NL = 36;
-    static readonly WS = 37;
-    static readonly BLOCK_COMMENT = 38;
-    static readonly LINE_COMMENT = 39;
-    static readonly INLINE_COMMENT = 40;
-    static readonly KEY = 41;
-    static readonly IDENT_INVALID = 42;
-    static readonly REST = 43;
-    static readonly META_INVALID = 44;
+    static readonly NUMBER = 14;
+    static readonly CARET = 15;
+    static readonly SS = 16;
+    static readonly GT = 17;
+    static readonly LT = 18;
+    static readonly EQ = 19;
+    static readonly COMMA = 20;
+    static readonly COLON = 21;
+    static readonly OB = 22;
+    static readonly CB = 23;
+    static readonly OC = 24;
+    static readonly CC = 25;
+    static readonly PLUS = 26;
+    static readonly DOLLAR = 27;
+    static readonly PC = 28;
+    static readonly AT = 29;
+    static readonly SEMICOLON = 30;
+    static readonly NL = 31;
+    static readonly WS = 32;
+    static readonly BLOCK_COMMENT = 33;
+    static readonly DISABLED_LINE = 34;
+    static readonly FULL_LINE_COMMENT = 35;
+    static readonly INLINE_COMMENT = 36;
+    static readonly KEY = 37;
+    static readonly IDENT_INVALID = 38;
+    static readonly REST = 39;
+    static readonly META_INVALID = 40;
     static readonly EOF: number;
     static readonly channelNames: string[];
     static readonly literalNames: (string | null)[];
     static readonly symbolicNames: (string | null)[];
     static readonly modeNames: string[];
     static readonly ruleNames: string[];
-    atLineStart(): boolean;
     constructor(input: CharStream);
     get grammarFileName(): string;
     get literalNames(): (string | null)[];
@@ -59,8 +54,6 @@ export default class YiniLexer extends Lexer {
     get serializedATN(): number[];
     get channelNames(): string[];
     get modeNames(): string[];
-    sempred(localctx: RuleContext, ruleIndex: number, predIndex: number): boolean;
-    private LINE_COMMENT_sempred;
     static readonly _serializedATN: number[];
     private static __ATN;
     static get _ATN(): ATN;