@jsenv/performance-impact 4.2.1 → 4.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/performance-impact",
-  "version": "4.2.1",
+  "version": "4.3.1",
   "description": "Report pull request impacts on performance metrics",
   "license": "MIT",
   "author": {
@@ -36,9 +36,9 @@
   },
   "dependencies": {
     "@jsenv/dynamic-import-worker": "1.2.1",
-    "@jsenv/filesystem": "4.6.6",
-    "@jsenv/github-pull-request-impact": "1.7.5",
-    "@jsenv/humanize": "1.1.0",
+    "@jsenv/filesystem": "4.10.2",
+    "@jsenv/github-pull-request-impact": "1.7.7",
+    "@jsenv/humanize": "1.2.8",
     "@jsenv/log": "3.5.2"
   }
 }

package/readme.md

@@ -24,7 +24,7 @@ With time you'll be capable to recognize unusual variation in your performance m
 
 ## How to catch small performance impacts?
 
-Catching (very) small performance impacts with confidence requires repetition and time. You need to: 
+Catching (very) small performance impacts with confidence requires repetition and time. You need to:
 
 1. Let your code be used a lot of times in a lot of scenarios and see the results. This could be scripts, real users or both.
 

package/src/internal/comment/formatImpact.js

@@ -29,7 +29,7 @@ export const formatImpactAsPercentage = ({
     metricValueBeforeMerge === 0
       ? 1
       : metricValueAfterMerge === 0
-      ? -1
-      : diff / metricValueBeforeMerge;
+        ? -1
+        : diff / metricValueBeforeMerge;
   return formatRatioAsPercentage(diffRatio);
 };

package/src/internal/comment/jsenvFormatGroupSummary.js

@@ -52,10 +52,10 @@ const getAllImpactRatios = ({ afterMergeMetrics, beforeMergeMetrics }) => {
       metricDiff === 0
         ? 0
         : metricValueBeforeMerge === 0
-        ? 1
-        : metricValueAfterMerge === 0
-        ? -1
-        : metricDiff / metricValueBeforeMerge;
+          ? 1
+          : metricValueAfterMerge === 0
+            ? -1
+            : metricDiff / metricValueBeforeMerge;
     allRatios.push(metricDiffRatio);
   });
   return allRatios;

package/src/startMeasures.js

@@ -29,13 +29,52 @@ export const startMeasures = ({
     });
   }
   if (filesystem) {
+    const _internalFs = process.binding("fs");
+    const restoreCallbackSet = new Set();
+    let fsOpenCall = 0;
+    let fsStatCall = 0;
+    const methodEffects = {
+      open: () => {
+        fsOpenCall++;
+      },
+      openFileHandle: () => {
+        fsOpenCall++;
+      },
+      stat: () => {
+        fsStatCall++;
+      },
+      lstat: () => {
+        fsStatCall++;
+      },
+      access: () => {
+        fsStatCall++;
+      },
+    };
+    for (const method of Object.keys(methodEffects)) {
+      const previous = _internalFs[method];
+      if (typeof previous !== "function") {
+        continue;
+      }
+      _internalFs[method] = function (...args) {
+        methodEffects[method](...args);
+        return previous.call(this, ...args);
+      };
+      restoreCallbackSet.add(() => {
+        _internalFs[method] = previous;
+      });
+    }
     const beforeRessourceUsage = resourceUsage();
     measures.push(() => {
+      for (const restoreCallback of restoreCallbackSet) {
+        restoreCallback();
+      }
       const afterRessourceUsage = resourceUsage();
       const fsRead = afterRessourceUsage.fsRead - beforeRessourceUsage.fsRead;
       const fsWrite =
         afterRessourceUsage.fsWrite - beforeRessourceUsage.fsWrite;
       return {
+        fsOpenCall,
+        fsStatCall,
         fsRead,
         fsWrite,
       };