v4.2.2

changelog.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## 2026-03-05 - 4.2.2 - fix(compiler)
+force global filesystem sync to flush XFS delayed logging and add diagnostics comparing Node's readdirSync with system ls to detect directory entry inconsistencies
+
+- Replace per-directory fs.fsyncSync loop with execSync('sync') to ensure parent B+tree metadata is flushed on XFS
+- Import execSync from child_process
+- Add diagnostic comparison: run ls -1 and compare its entries to fs.readdirSync; log mismatches and full entry lists for debugging Node.js caching/readdir inconsistencies
+
+## 2026-03-05 - 4.2.1 - fix(compiler)
+use TypeScript sys hooks instead of fs monkeypatching to detect writes/deletes in previous output directories
+
+- Replace direct fs.* monkeypatching with interception of typescript.sys.writeFile, typescript.sys.deleteFile and typescript.sys.createDirectory
+- Add guards for optional sys.deleteFile before overriding it and preserve original sys methods to restore after compilation
+- Update diagnostic messages to reference TypeScript sys ops and add an informational message when no ops are observed
+- Reduce surface area of changes by avoiding global fs changes and focusing on TypeScript's sys API for safer interception
+
 ## 2026-03-05 - 4.2.0 - feat(mod_compiler)
 add diagnostic interception of fs operations to detect and report unexpected file system changes in previously compiled output directories during compilation
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@git.zone/tsbuild",
-  "version": "4.2.0",
+  "version": "4.2.2",
   "private": false,
   "description": "A tool for compiling TypeScript files using the latest nightly features, offering flexible APIs and a CLI for streamlined development.",
   "main": "dist_ts/index.js",

ts/00_commitinfo_data.ts

@@ -3,6 +3,6 @@
  */
 export const commitinfo = {
   name: '@git.zone/tsbuild',
-  version: '4.2.0',
+  version: '4.2.2',
   description: 'A tool for compiling TypeScript files using the latest nightly features, offering flexible APIs and a CLI for streamlined development.'
 }

ts/mod_compiler/classes.tscompiler.ts

@@ -1,6 +1,7 @@
 import type { CompilerOptions, Diagnostic, Program } from 'typescript';
 import typescript from 'typescript';
 import * as fs from 'fs';
+import { execSync } from 'child_process';
 import * as smartdelay from '@push.rocks/smartdelay';
 import * as smartpromise from '@push.rocks/smartpromise';
 import * as smartpath from '@push.rocks/smartpath';
@@ -380,45 +381,33 @@ export class TsCompiler {
         fileCount: absoluteFiles.length,
       };
 
-      // Diagnostic: intercept fs operations during compilation to detect
-      // any unexpected deletions in previously compiled output directories
+      // Diagnostic: intercept TypeScript sys operations during compilation to detect
+      // any unexpected writes/deletes in previously compiled output directories
       const watchedDirs = successfulOutputDirs.filter(d => d !== destDir);
-      const origUnlink = fs.unlinkSync;
-      const origRm = fs.rmSync;
-      const origRmdir = fs.rmdirSync;
-      const origRename = fs.renameSync;
-      const origWriteFile = fs.writeFileSync;
       let interceptedOps: string[] = [];
+      const origSysWriteFile = typescript.sys.writeFile;
+      const origSysDeleteFile = typescript.sys.deleteFile;
+      const origSysCreateDir = typescript.sys.createDirectory;
       if (watchedDirs.length > 0 && !isQuiet && !isJson) {
-        (fs as any).unlinkSync = (p: string, ...args: any[]) => {
-          if (watchedDirs.some(d => String(p).startsWith(d + '/'))) {
-            interceptedOps.push(`unlink: ${p}`);
+        typescript.sys.writeFile = (p: string, data: string, writeBom?: boolean) => {
+          if (watchedDirs.some(d => p.startsWith(d + '/'))) {
+            interceptedOps.push(`sys.writeFile: ${p}`);
           }
-          return origUnlink.call(fs, p, ...args);
+          return origSysWriteFile.call(typescript.sys, p, data, writeBom);
         };
-        (fs as any).rmSync = (p: string, ...args: any[]) => {
-          if (watchedDirs.some(d => String(p).startsWith(d + '/'))) {
-            interceptedOps.push(`rm: ${p}`);
+        if (origSysDeleteFile) {
+          typescript.sys.deleteFile = (p: string) => {
+            if (watchedDirs.some(d => p.startsWith(d + '/'))) {
+              interceptedOps.push(`sys.deleteFile: ${p}`);
+            }
+            return origSysDeleteFile!.call(typescript.sys, p);
+          };
+        }
+        typescript.sys.createDirectory = (p: string) => {
+          if (watchedDirs.some(d => p.startsWith(d + '/'))) {
+            interceptedOps.push(`sys.createDirectory: ${p}`);
           }
-          return origRm.call(fs, p, ...args);
-        };
-        (fs as any).rmdirSync = (p: string, ...args: any[]) => {
-          if (watchedDirs.some(d => String(p).startsWith(d + '/'))) {
-            interceptedOps.push(`rmdir: ${p}`);
-          }
-          return origRmdir.call(fs, p, ...args);
-        };
-        (fs as any).renameSync = (src: string, dest: string, ...args: any[]) => {
-          if (watchedDirs.some(d => String(src).startsWith(d + '/') || String(dest).startsWith(d + '/'))) {
-            interceptedOps.push(`rename: ${src} → ${dest}`);
-          }
-          return origRename.call(fs, src, dest, ...args);
-        };
-        (fs as any).writeFileSync = (p: string, ...args: any[]) => {
-          if (watchedDirs.some(d => String(p).startsWith(d + '/'))) {
-            interceptedOps.push(`write: ${p}`);
-          }
-          return origWriteFile.call(fs, p, ...args);
+          return origSysCreateDir.call(typescript.sys, p);
         };
       }
 
@@ -426,21 +415,21 @@ export class TsCompiler {
       emittedFiles.push(...result.emittedFiles);
       errorSummaries.push(result.errorSummary);
 
-      // Restore original fs methods and report any intercepted operations
+      // Restore original sys methods and report any intercepted operations
       if (watchedDirs.length > 0 && !isQuiet && !isJson) {
-        (fs as any).unlinkSync = origUnlink;
-        (fs as any).rmSync = origRm;
-        (fs as any).rmdirSync = origRmdir;
-        (fs as any).renameSync = origRename;
-        (fs as any).writeFileSync = origWriteFile;
+        typescript.sys.writeFile = origSysWriteFile;
+        typescript.sys.deleteFile = origSysDeleteFile;
+        typescript.sys.createDirectory = origSysCreateDir;
         if (interceptedOps.length > 0) {
-          console.log(`   ⚠️ [diag] ${interceptedOps.length} ops on previous output dirs during compilation:`);
+          console.log(`   ⚠️ [diag] ${interceptedOps.length} TypeScript sys ops on previous output dirs:`);
           for (const op of interceptedOps.slice(0, 30)) {
             console.log(`      ${op.replace(this.cwd + '/', '')}`);
           }
           if (interceptedOps.length > 30) {
             console.log(`      ... and ${interceptedOps.length - 30} more`);
           }
+        } else {
+          console.log(`   ℹ️ [diag] No TypeScript sys ops on previous output dirs during this compilation`);
         }
       }
       diagSnap('post-compile');
@@ -465,29 +454,28 @@ export class TsCompiler {
         successfulOutputDirs.push(destDir);
       }
 
-      // Fsync all output directories to force XFS metadata commit
-      // before the next compilation step. Without this, XFS delayed logging
-      // can cause directory entries from previous compilations to become
-      // invisible or corrupted during subsequent TypeScript emit operations.
-      for (const dir of successfulOutputDirs) {
-        try {
-          const fd = fs.openSync(dir, 'r');
-          fs.fsyncSync(fd);
-          fs.closeSync(fd);
-        } catch {
-          // Directory might not exist yet
-        }
-      }
+      // Force global filesystem sync to flush XFS delayed logging (CIL)
+      // before the next compilation step. Per-directory fsync was insufficient;
+      // XFS defers parent B+tree metadata commits which can make sibling
+      // directory entries invisible to readdirSync after heavy write activity.
+      execSync('sync');
 
-      // Diagnostic: log all output directory states after each compilation
+      // Diagnostic: compare readdirSync vs external ls to detect Node.js caching
       if (!isQuiet && !isJson) {
         for (const prevDir of successfulOutputDirs) {
           try {
-            const entries = fs.readdirSync(prevDir);
-            const dirs = entries.filter(e => {
-              try { return fs.statSync(prevDir + '/' + e).isDirectory(); } catch { return false; }
-            });
-            console.log(`   📋 [diag] ${prevDir.replace(this.cwd + '/', '')}: ${entries.length} entries, ${dirs.length} dirs`);
+            const nodeEntries = fs.readdirSync(prevDir);
+            const nodeDirs = nodeEntries.filter(e => { try { return fs.statSync(prevDir + '/' + e).isDirectory(); } catch { return false; } });
+            const lsOutput = execSync(`ls -1 "${prevDir}"`, { encoding: 'utf8' }).trim();
+            const lsEntries = lsOutput ? lsOutput.split('\n') : [];
+            const shortDir = prevDir.replace(this.cwd + '/', '');
+            if (nodeEntries.length !== lsEntries.length) {
+              console.log(`   ⚠️ [diag] ${shortDir}: readdirSync=${nodeEntries.length}, ls=${lsEntries.length} (MISMATCH!)`);
+              console.log(`      readdirSync: [${nodeEntries.sort().join(', ')}]`);
+              console.log(`      ls:          [${lsEntries.sort().join(', ')}]`);
+            } else {
+              console.log(`   📋 [diag] ${shortDir}: ${nodeEntries.length} entries, ${nodeDirs.length} dirs ✓`);
+            }
           } catch {
             console.log(`   📋 [diag] ${prevDir.replace(this.cwd + '/', '')}: MISSING!`);
           }