refactor(@ngtools/webpack): reduce webpack fs decorator system calls

Author: clydin

packages/ngtools/webpack/src/compiler_host.ts

@@ -113,20 +113,16 @@ export class WebpackCompilerHost implements ts.CompilerHost {
     }
   }
 
-  readFileBuffer(fileName: string): Buffer | undefined {
+  readFileBuffer(fileName: string): Buffer {
     const filePath = this.resolve(fileName);
 
-    try {
-      return Buffer.from(this._syncHost.read(filePath));
-    } catch {
-      return undefined;
-    }
+    return Buffer.from(this._syncHost.read(filePath));
   }
 
   stat(path: string): Stats | null {
     const p = this.resolve(path);
 
-    let stats;
+    let stats: virtualFs.Stats<Partial<Stats>> | null = null;
     try {
       stats = this._syncHost.stat(p);
     } catch { }
@@ -136,25 +132,31 @@ export class WebpackCompilerHost implements ts.CompilerHost {
     }
 
     return {
-      isBlockDevice: () => false,
-      isCharacterDevice: () => false,
-      isFIFO: () => false,
-      isSymbolicLink: () => false,
-      isSocket: () => false,
-      dev,
-      ino: Math.floor(Math.random() * 100000),
+      isFile: stats.isFile,
+      isDirectory: stats.isDirectory,
+      isBlockDevice: stats.isBlockDevice || (() => false),
+      isCharacterDevice: stats.isCharacterDevice || (() => false),
+      isFIFO: stats.isFIFO || (() => false),
+      isSymbolicLink: stats.isSymbolicLink || (() => false),
+      isSocket: stats.isSocket || (() => false),
+      dev: stats.dev || dev,
+      ino: stats.ino || Math.floor(Math.random() * 100000),
       mode: parseInt('777', 8),
       nlink: 1,
       uid: 0,
       gid: 0,
       rdev: 0,
+      size: stats.size,
       blksize: 512,
       blocks: Math.ceil(stats.size / 512),
+      atime: stats.atime,
       atimeMs: stats.atime.getTime(),
+      mtime: stats.mtime,
       mtimeMs: stats.mtime.getTime(),
+      ctime: stats.ctime,
       ctimeMs: stats.ctime.getTime(),
+      birthtime: stats.birthtime,
       birthtimeMs: stats.birthtime.getTime(),
-      ...stats,
     };
   }
 

packages/ngtools/webpack/src/virtual_file_system_decorator.ts

@@ -5,7 +5,7 @@
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://angular.io/license
  */
-import { Path, getSystemPath, normalize } from '@angular-devkit/core';
+import { FileDoesNotExistException, Path, getSystemPath, normalize } from '@angular-devkit/core';
 import { Stats } from 'fs';
 import { InputFileSystem } from 'webpack';
 import { WebpackCompilerHost } from './compiler_host';
@@ -21,33 +21,17 @@ export class VirtualFileSystemDecorator implements InputFileSystem {
     private _webpackCompilerHost: WebpackCompilerHost,
   ) { }
 
-  private _readFileSync(path: string): Buffer | null {
-    if (this._webpackCompilerHost.fileExists(path)) {
-      return this._webpackCompilerHost.readFileBuffer(path) || null;
-    }
-
-    return null;
-  }
-
-  private _statSync(path: string): Stats | null {
-    if (this._webpackCompilerHost.fileExists(path)) {
-      return this._webpackCompilerHost.stat(path);
-    }
-
-    return null;
-  }
-
   getVirtualFilesPaths() {
     return this._webpackCompilerHost.getNgFactoryPaths();
   }
 
   stat(path: string, callback: (err: Error, stats: Stats) => void): void {
-    const result = this._statSync(path);
-    if (result) {
+    try {
+      // tslint:disable-next-line:no-any
+      callback(null as any, this._webpackCompilerHost.stat(path) as any);
+    } catch (e) {
       // tslint:disable-next-line:no-any
-      callback(null as any, result);
-    } else {
-      this._inputFileSystem.stat(path, callback);
+      callback(e, undefined as any);
     }
   }
 
@@ -57,12 +41,12 @@ export class VirtualFileSystemDecorator implements InputFileSystem {
   }
 
   readFile(path: string, callback: (err: Error, contents: Buffer) => void): void {
-    const result = this._readFileSync(path);
-    if (result) {
+    try {
       // tslint:disable-next-line:no-any
-      callback(null as any, result);
-    } else {
-      this._inputFileSystem.readFile(path, callback);
+      callback(null as any, this._webpackCompilerHost.readFileBuffer(path));
+    } catch (e) {
+      // tslint:disable-next-line:no-any
+      callback(e, undefined as any);
     }
   }
 
@@ -76,9 +60,12 @@ export class VirtualFileSystemDecorator implements InputFileSystem {
   }
 
   statSync(path: string): Stats {
-    const result = this._statSync(path);
+    const stats = this._webpackCompilerHost.stat(path);
+    if (stats === null) {
+      throw new FileDoesNotExistException(path);
+    }
 
-    return result || this._inputFileSystem.statSync(path);
+    return stats;
   }
 
   readdirSync(path: string): string[] {
@@ -87,9 +74,7 @@ export class VirtualFileSystemDecorator implements InputFileSystem {
   }
 
   readFileSync(path: string): Buffer {
-    const result = this._readFileSync(path);
-
-    return result || this._inputFileSystem.readFileSync(path);
+    return this._webpackCompilerHost.readFileBuffer(path);
   }
 
   readJsonSync(path: string): string {