@jsenv/file-size-impact 14.1.1 → 14.1.2

package/src/internal/jsenvCommentParameters.js

@@ -1,5 +1,5 @@
-import { formatPercentage } from "./format_percentage.js"
-import { formatSize } from "./formatSize.js"
+import { formatPercentage } from "./format_percentage.js";
+import { formatSize } from "./formatSize.js";
 
 const jsenvFormatGroupSummary = ({
   groupName,
@@ -7,30 +7,30 @@ const jsenvFormatGroupSummary = ({
   groupSizeMapAfterMerge,
   sizeNames,
 }) => {
-  const firstSizeName = sizeNames[0]
-  const groupSizeBeforeMerge = groupSizeMapBeforeMerge[firstSizeName]
-  const groupSizeAfterMerge = groupSizeMapAfterMerge[firstSizeName]
+  const firstSizeName = sizeNames[0];
+  const groupSizeBeforeMerge = groupSizeMapBeforeMerge[firstSizeName];
+  const groupSizeAfterMerge = groupSizeMapAfterMerge[firstSizeName];
 
   if (groupSizeBeforeMerge === groupSizeAfterMerge) {
-    return `${groupName} (no impact)`
+    return `${groupName} (no impact)`;
   }
 
   return `${groupName} (${formatSizeImpactAsPercentage({
     sizeBeforeMerge: groupSizeBeforeMerge,
     sizeAfterMerge: groupSizeAfterMerge,
-  })})`
-}
+  })})`;
+};
 
 const jsenvFormatFileRelativeUrl = (fileRelativeUrl) => {
-  return fileRelativeUrl
-}
+  return fileRelativeUrl;
+};
 
 const jsenvFormatFileCell = ({ fileRelativeUrlDisplayed, sizeAfterMerge }) => {
   if (sizeAfterMerge === undefined) {
-    return `<del>${fileRelativeUrlDisplayed}</del>`
+    return `<del>${fileRelativeUrlDisplayed}</del>`;
   }
-  return fileRelativeUrlDisplayed
-}
+  return fileRelativeUrlDisplayed;
+};
 
 /**
  * - added
@@ -47,8 +47,8 @@ const jsenvFormatFileSizeImpactCell = ({ sizeBeforeMerge, sizeAfterMerge }) => {
   // but it would make no sense to compare with something that does not exists like
   // "100 B (+100B / +100%)
   if (sizeBeforeMerge === undefined) {
-    const sizeAfterMergeFormatted = formatSize(sizeAfterMerge)
-    return sizeAfterMergeFormatted
+    const sizeAfterMergeFormatted = formatSize(sizeAfterMerge);
+    return sizeAfterMergeFormatted;
   }
 
   // The file is deleted, it makes sense to display
@@ -57,41 +57,41 @@ const jsenvFormatFileSizeImpactCell = ({ sizeBeforeMerge, sizeAfterMerge }) => {
   // but it would be redundant to add the percentage
   // "deleted (-100 B / -100%)"
   if (sizeAfterMerge === undefined) {
-    const sizeDiff = -sizeBeforeMerge
-    const sizeDiffFormatted = formatSize(sizeDiff, { diff: true })
-    return `deleted (${sizeDiffFormatted})`
+    const sizeDiff = -sizeBeforeMerge;
+    const sizeDiffFormatted = formatSize(sizeDiff, { diff: true });
+    return `deleted (${sizeDiffFormatted})`;
   }
 
-  const sizeAfterMergeFormatted = formatSize(sizeAfterMerge)
-  const sizeDiff = sizeAfterMerge - sizeBeforeMerge
-  const sizeDiffFormatted = formatSize(sizeDiff, { diff: true })
+  const sizeAfterMergeFormatted = formatSize(sizeAfterMerge);
+  const sizeDiff = sizeAfterMerge - sizeBeforeMerge;
+  const sizeDiffFormatted = formatSize(sizeDiff, { diff: true });
   const sizeDiffAsPercentageFormatted = formatSizeImpactAsPercentage({
     sizeBeforeMerge,
     sizeAfterMerge,
-  })
-  return `${sizeAfterMergeFormatted} (${sizeDiffFormatted} / ${sizeDiffAsPercentageFormatted})`
-}
+  });
+  return `${sizeAfterMergeFormatted} (${sizeDiffFormatted} / ${sizeDiffAsPercentageFormatted})`;
+};
 
 export const jsenvFormatEmojiCell = ({ sizeBeforeMerge, sizeAfterMerge }) => {
   if (sizeBeforeMerge === undefined) {
-    return ":baby:"
+    return ":baby:";
   }
 
   if (sizeAfterMerge === undefined) {
-    return ""
+    return "";
   }
 
-  const delta = sizeAfterMerge - sizeBeforeMerge
+  const delta = sizeAfterMerge - sizeBeforeMerge;
   if (delta === 0) {
-    return ":ghost:"
+    return ":ghost:";
   }
 
   if (delta > 0) {
-    return ":arrow_upper_right:"
+    return ":arrow_upper_right:";
   }
 
-  return ":arrow_lower_right:"
-}
+  return ":arrow_lower_right:";
+};
 
 export const jsenvCommentParameters = {
   filesOrdering: "size_impact",
@@ -104,10 +104,10 @@ export const jsenvCommentParameters = {
   formatFileSizeImpactCell: jsenvFormatFileSizeImpactCell,
   formatEmojiCell: jsenvFormatEmojiCell,
   shouldOpenGroupByDefault: () => false,
-}
+};
 
 const formatSizeImpactAsPercentage = ({ sizeBeforeMerge, sizeAfterMerge }) => {
-  const sizeDiff = sizeAfterMerge - sizeBeforeMerge
+  const sizeDiff = sizeAfterMerge - sizeBeforeMerge;
   const sizeDiffRatio =
     sizeDiff === 0
       ? 0
@@ -115,8 +115,8 @@ const formatSizeImpactAsPercentage = ({ sizeBeforeMerge, sizeAfterMerge }) => {
       ? 1
       : sizeAfterMerge === 0
       ? -1
-      : sizeDiff / sizeBeforeMerge
-  const sizeDiffAsPercentage = sizeDiffRatio * 100
-  const sizeDiffAsPercentageFormatted = formatPercentage(sizeDiffAsPercentage)
-  return sizeDiffAsPercentageFormatted
-}
+      : sizeDiff / sizeBeforeMerge;
+  const sizeDiffAsPercentage = sizeDiffRatio * 100;
+  const sizeDiffAsPercentageFormatted = formatPercentage(sizeDiffAsPercentage);
+  return sizeDiffAsPercentageFormatted;
+};

package/src/internal/manifest.js

@@ -1,39 +1,39 @@
-import { urlToRelativeUrl } from "@jsenv/urls"
+import { urlToRelativeUrl } from "@jsenv/urls";
 
 export const manifestToMappings = (manifestMap) => {
-  const mappings = {}
+  const mappings = {};
   Object.keys(manifestMap).forEach((manifestRelativeUrl) => {
-    const manifest = manifestMap[manifestRelativeUrl]
+    const manifest = manifestMap[manifestRelativeUrl];
     const manifestAbstractUrl = new URL(
       manifestRelativeUrl,
       ABSTRACT_DIRECTORY_URL,
-    ).href
+    ).href;
     Object.keys(manifest).forEach((manifestKey) => {
-      const manifestValue = manifest[manifestKey]
+      const manifestValue = manifest[manifestKey];
       const manifestKeyAsAbstractUrl = new URL(manifestKey, manifestAbstractUrl)
-        .href
+        .href;
       const manifestValueAsAbstractUrl = new URL(
         manifestValue,
         manifestAbstractUrl,
-      ).href
+      ).href;
       const manifestKeyAsRelativeUrl = urlToRelativeUrl(
         manifestKeyAsAbstractUrl,
         ABSTRACT_DIRECTORY_URL,
-      )
+      );
       const manifestValueAsRelativeUrl = urlToRelativeUrl(
         manifestValueAsAbstractUrl,
         ABSTRACT_DIRECTORY_URL,
-      )
-      mappings[manifestKeyAsRelativeUrl] = manifestValueAsRelativeUrl
-    })
-  })
-  return mappings
-}
+      );
+      mappings[manifestKeyAsRelativeUrl] = manifestValueAsRelativeUrl;
+    });
+  });
+  return mappings;
+};
 
-const ABSTRACT_DIRECTORY_URL = "file:///directory/"
+const ABSTRACT_DIRECTORY_URL = "file:///directory/";
 
 export const manifestKeyFromRelativeUrl = (relativeUrl, mappings) => {
   return Object.keys(mappings).find((keyCandidate) => {
-    return mappings[keyCandidate] === relativeUrl
-  })
-}
+    return mappings[keyCandidate] === relativeUrl;
+  });
+};

package/src/internal/orderBySizeImpact.js

@@ -1,28 +1,28 @@
 export const orderBySizeImpact = (fileByFileImpact, sizeNames) => {
-  const impactOrderedBySizeImpact = {}
-  const files = Object.keys(fileByFileImpact)
-  const firstSizeName = sizeNames[0]
+  const impactOrderedBySizeImpact = {};
+  const files = Object.keys(fileByFileImpact);
+  const firstSizeName = sizeNames[0];
   files.sort((leftFile, rightFile) => {
     const leftFileSizeImpact =
-      fileByFileImpact[leftFile].sizeImpactMap[firstSizeName]
+      fileByFileImpact[leftFile].sizeImpactMap[firstSizeName];
     const rightFileSizeImpact =
-      fileByFileImpact[rightFile].sizeImpactMap[firstSizeName]
+      fileByFileImpact[rightFile].sizeImpactMap[firstSizeName];
     if (leftFileSizeImpact === 0) {
-      return 1
+      return 1;
     }
     if (rightFileSizeImpact === 0) {
-      return -1
+      return -1;
     }
     if (leftFileSizeImpact < rightFileSizeImpact) {
-      return 1
+      return 1;
     }
     if (leftFileSizeImpact > rightFileSizeImpact) {
-      return -1
+      return -1;
     }
-    return 0
-  })
+    return 0;
+  });
   files.forEach((file) => {
-    impactOrderedBySizeImpact[file] = fileByFileImpact[file]
-  })
-  return impactOrderedBySizeImpact
-}
+    impactOrderedBySizeImpact[file] = fileByFileImpact[file];
+  });
+  return impactOrderedBySizeImpact;
+};

package/src/internal/renderImpactTable.js

@@ -1,4 +1,4 @@
-import { getSizeMapsForManyFiles } from "./size_map.js"
+import { getSizeMapsForManyFiles } from "./size_map.js";
 
 export const renderImpactTable = ({
   fileByFileImpact,
@@ -38,13 +38,13 @@ export const renderImpactTable = ({
         groupSizeMapAfterMerge,
       })}
     </tfoot>
-  </table>`
+  </table>`;
 
-  return table
-}
+  return table;
+};
 
 const renderSizeImpactTableHeader = (transformationKeys) => {
-  const lines = []
+  const lines = [];
   const headerLine = [
     `<th nowrap>Files</th>`,
     ...transformationKeys.map(
@@ -54,11 +54,11 @@ const renderSizeImpactTableHeader = (transformationKeys) => {
         }</th>`,
     ),
     `<th></th>`,
-  ]
-  lines.push(headerLine)
+  ];
+  lines.push(headerLine);
 
-  return renderTableLines(lines)
-}
+  return renderTableLines(lines);
+};
 
 const renderSizeImpactTableBody = ({
   fileByFileImpact,
@@ -70,44 +70,44 @@ const renderSizeImpactTableBody = ({
   formatFileSizeImpactCell,
   formatEmojiCell,
 }) => {
-  const lines = []
-  const sizeNames = transformationKeys
-  const firstSizeName = sizeNames[0]
+  const lines = [];
+  const sizeNames = transformationKeys;
+  const firstSizeName = sizeNames[0];
 
-  let remainingFilesToDisplay = maxFilesPerGroup
-  const truncateds = []
-  const unmodifieds = []
+  let remainingFilesToDisplay = maxFilesPerGroup;
+  const truncateds = [];
+  const unmodifieds = [];
   Object.keys(fileByFileImpact).forEach((fileRelativeUrl) => {
-    const fileImpact = fileByFileImpact[fileRelativeUrl]
+    const fileImpact = fileByFileImpact[fileRelativeUrl];
 
     if (fileImpact.event === "none") {
-      unmodifieds.push(fileRelativeUrl)
-      return
+      unmodifieds.push(fileRelativeUrl);
+      return;
     }
 
     if (remainingFilesToDisplay === 0) {
-      truncateds.push(fileRelativeUrl)
-      return
+      truncateds.push(fileRelativeUrl);
+      return;
     }
 
-    remainingFilesToDisplay--
+    remainingFilesToDisplay--;
     const fileAbstractRelativeUrl =
-      fileAbstractRelativeUrlFromFileImpact(fileImpact)
+      fileAbstractRelativeUrlFromFileImpact(fileImpact);
     const fileRelativeUrlFormatted = (
       fileImpact.formatFileRelativeUrl || formatFileRelativeUrl
-    )(fileAbstractRelativeUrl)
+    )(fileAbstractRelativeUrl);
     const fileRelativeUrlDisplayed = truncateFileRelativeUrl(
       fileRelativeUrlFormatted,
       fileRelativeUrlMaxLength,
-    )
+    );
     const { sizeMapBeforeMerge, sizeMapAfterMerge } =
-      fileByFileImpact[fileRelativeUrl]
+      fileByFileImpact[fileRelativeUrl];
     const fileCellFormatted = formatFileCell({
       fileRelativeUrl,
       fileRelativeUrlDisplayed,
       sizeBeforeMerge: sizeMapBeforeMerge[firstSizeName],
       sizeAfterMerge: sizeMapAfterMerge[firstSizeName],
-    })
+    });
 
     const line = [
       `<td nowrap>${fileCellFormatted}</td>`,
@@ -116,7 +116,7 @@ const renderSizeImpactTableBody = ({
           sizeBeforeMerge: sizeMapBeforeMerge[sizeName],
           sizeAfterMerge: sizeMapAfterMerge[sizeName],
           sizeName,
-        })}</td>`
+        })}</td>`;
       }),
       ...(formatEmojiCell
         ? [
@@ -126,17 +126,17 @@ const renderSizeImpactTableBody = ({
             })}</td>`,
           ]
         : []),
-    ]
-    lines.push(line)
-  })
+    ];
+    lines.push(line);
+  });
 
-  const truncatedCount = truncateds.length
+  const truncatedCount = truncateds.length;
   if (truncatedCount > 0) {
     const { sizeMapBeforeMerge, sizeMapAfterMerge } = getSizeMapsForManyFiles({
       sizeNames,
       fileByFileImpact,
       files: truncateds,
-    })
+    });
     const lineForTruncatedFiles = [
       `<td nowrap><i>Truncated (${truncatedCount})</i></td>`,
       ...sizeNames.map((sizeName) => {
@@ -144,7 +144,7 @@ const renderSizeImpactTableBody = ({
           sizeBeforeMerge: sizeMapBeforeMerge[sizeName],
           sizeAfterMerge: sizeMapAfterMerge[sizeName],
           sizeName,
-        })}</td>`
+        })}</td>`;
       }),
       ...(formatEmojiCell
         ? [
@@ -154,17 +154,17 @@ const renderSizeImpactTableBody = ({
             })}</td>`,
           ]
         : []),
-    ]
-    lines.push(lineForTruncatedFiles)
+    ];
+    lines.push(lineForTruncatedFiles);
   }
 
-  const unmodifiedCount = unmodifieds.length
+  const unmodifiedCount = unmodifieds.length;
   if (unmodifiedCount > 0) {
     const { sizeMapBeforeMerge, sizeMapAfterMerge } = getSizeMapsForManyFiles({
       sizeNames,
       fileByFileImpact,
       files: unmodifieds,
-    })
+    });
     const lineForUnmodifiedFiles = [
       `<td nowrap><i>Unmodified (${unmodifiedCount})</i></td>`,
       ...sizeNames.map((sizeName) => {
@@ -172,7 +172,7 @@ const renderSizeImpactTableBody = ({
           sizeBeforeMerge: sizeMapBeforeMerge[sizeName],
           sizeAfterMerge: sizeMapAfterMerge[sizeName],
           sizeName,
-        })}</td>`
+        })}</td>`;
       }),
       ...(formatEmojiCell
         ? [
@@ -182,12 +182,12 @@ const renderSizeImpactTableBody = ({
             })}</td>`,
           ]
         : []),
-    ]
-    lines.push(lineForUnmodifiedFiles)
+    ];
+    lines.push(lineForUnmodifiedFiles);
   }
 
-  return renderTableLines(lines)
-}
+  return renderTableLines(lines);
+};
 
 const fileAbstractRelativeUrlFromFileImpact = ({
   manifestKeyBeforeMerge,
@@ -200,8 +200,8 @@ const fileAbstractRelativeUrlFromFileImpact = ({
     relativeUrlAfterMerge ||
     manifestKeyBeforeMerge ||
     relativeUrlBeforeMerge
-  )
-}
+  );
+};
 
 const renderSizeImpactTableFooter = ({
   fileByFileImpact,
@@ -211,11 +211,11 @@ const renderSizeImpactTableFooter = ({
   groupSizeMapBeforeMerge,
   groupSizeMapAfterMerge,
 }) => {
-  const footerLines = []
+  const footerLines = [];
 
-  const fileCount = Object.keys(fileByFileImpact).length
-  const sizeNames = transformationKeys
-  const firstSizeName = sizeNames[0]
+  const fileCount = Object.keys(fileByFileImpact).length;
+  const sizeNames = transformationKeys;
+  const firstSizeName = sizeNames[0];
 
   const groupSizeImpactLine = [
     `<td nowrap><strong>Total (${fileCount})</strong></td>`,
@@ -235,28 +235,28 @@ const renderSizeImpactTableFooter = ({
           })}</td>`,
         ]
       : []),
-  ]
-  footerLines.push(groupSizeImpactLine)
+  ];
+  footerLines.push(groupSizeImpactLine);
 
-  return renderTableLines(footerLines)
-}
+  return renderTableLines(footerLines);
+};
 
 const truncateFileRelativeUrl = (fileRelativeUrl, fileRelativeUrlMaxLength) => {
-  const length = fileRelativeUrl.length
-  const extraLength = length - fileRelativeUrlMaxLength
+  const length = fileRelativeUrl.length;
+  const extraLength = length - fileRelativeUrlMaxLength;
   if (extraLength > 0) {
-    return `…${fileRelativeUrl.slice(extraLength)}`
+    return `…${fileRelativeUrl.slice(extraLength)}`;
   }
-  return fileRelativeUrl
-}
+  return fileRelativeUrl;
+};
 
 const renderTableLines = (lines, { indentCount = 3, indentSize = 2 } = {}) => {
   if (lines.length === 0) {
-    return ""
+    return "";
   }
 
-  const cellLeftSpacing = indent(indentCount + 1, indentSize)
-  const lineLeftSpacing = indent(indentCount, indentSize)
+  const cellLeftSpacing = indent(indentCount + 1, indentSize);
+  const lineLeftSpacing = indent(indentCount, indentSize);
 
   return `<tr>${lines.map(
     (cells) => `
@@ -265,7 +265,7 @@ ${cellLeftSpacing}`)}`,
   ).join(`
 ${lineLeftSpacing}</tr>
 ${lineLeftSpacing}<tr>`)}
-${lineLeftSpacing}</tr>`
-}
+${lineLeftSpacing}</tr>`;
+};
 
-const indent = (count, size) => ` `.repeat(size * count)
+const indent = (count, size) => ` `.repeat(size * count);

package/src/internal/size_map.js

@@ -1,55 +1,55 @@
 export const getSizeMapsOneFile = ({ sizeNames, beforeMerge, afterMerge }) => {
-  const sizeMapBeforeMerge = {}
-  const sizeMapAfterMerge = {}
-  const sizeImpactMap = {}
+  const sizeMapBeforeMerge = {};
+  const sizeMapAfterMerge = {};
+  const sizeImpactMap = {};
 
   sizeNames.forEach((sizeName) => {
     if (beforeMerge === null) {
-      sizeMapBeforeMerge[sizeName] = undefined
-      sizeMapAfterMerge[sizeName] = afterMerge.sizeMap[sizeName]
-      sizeImpactMap[sizeName] = afterMerge.sizeMap[sizeName]
+      sizeMapBeforeMerge[sizeName] = undefined;
+      sizeMapAfterMerge[sizeName] = afterMerge.sizeMap[sizeName];
+      sizeImpactMap[sizeName] = afterMerge.sizeMap[sizeName];
     } else if (afterMerge === null) {
-      sizeMapBeforeMerge[sizeName] = beforeMerge.sizeMap[sizeName]
-      sizeMapAfterMerge[sizeName] = undefined
-      sizeImpactMap[sizeName] = -beforeMerge.sizeMap[sizeName]
+      sizeMapBeforeMerge[sizeName] = beforeMerge.sizeMap[sizeName];
+      sizeMapAfterMerge[sizeName] = undefined;
+      sizeImpactMap[sizeName] = -beforeMerge.sizeMap[sizeName];
     } else {
-      sizeMapBeforeMerge[sizeName] = beforeMerge.sizeMap[sizeName]
-      sizeMapAfterMerge[sizeName] = afterMerge.sizeMap[sizeName]
+      sizeMapBeforeMerge[sizeName] = beforeMerge.sizeMap[sizeName];
+      sizeMapAfterMerge[sizeName] = afterMerge.sizeMap[sizeName];
       sizeImpactMap[sizeName] =
-        afterMerge.sizeMap[sizeName] - beforeMerge.sizeMap[sizeName]
+        afterMerge.sizeMap[sizeName] - beforeMerge.sizeMap[sizeName];
     }
-  })
+  });
 
   return {
     sizeMapBeforeMerge,
     sizeMapAfterMerge,
     sizeImpactMap,
-  }
-}
+  };
+};
 
 export const getSizeMapsForManyFiles = ({
   sizeNames,
   fileByFileImpact,
   files,
 }) => {
-  const sizeMapBeforeMerge = {}
-  const sizeMapAfterMerge = {}
+  const sizeMapBeforeMerge = {};
+  const sizeMapAfterMerge = {};
 
   sizeNames.forEach((sizeName) => {
-    let sizeBeforeMerge = 0
-    let sizeAfterMerge = 0
+    let sizeBeforeMerge = 0;
+    let sizeAfterMerge = 0;
     files.forEach((fileRelativeUrl) => {
       const { sizeMapBeforeMerge, sizeMapAfterMerge } =
-        fileByFileImpact[fileRelativeUrl]
-      sizeBeforeMerge += sizeMapBeforeMerge[sizeName] || 0
-      sizeAfterMerge += sizeMapAfterMerge[sizeName] || 0
-    })
-    sizeMapBeforeMerge[sizeName] = sizeBeforeMerge
-    sizeMapAfterMerge[sizeName] = sizeAfterMerge
-  })
+        fileByFileImpact[fileRelativeUrl];
+      sizeBeforeMerge += sizeMapBeforeMerge[sizeName] || 0;
+      sizeAfterMerge += sizeMapAfterMerge[sizeName] || 0;
+    });
+    sizeMapBeforeMerge[sizeName] = sizeBeforeMerge;
+    sizeMapAfterMerge[sizeName] = sizeAfterMerge;
+  });
 
   return {
     sizeMapBeforeMerge,
     sizeMapAfterMerge,
-  }
-}
+  };
+};

package/src/jsenvTrackingConfig.js

@@ -3,4 +3,4 @@ export const jsenvTrackingConfig = {
     "./dist/**/*": true,
     "./dist/**/*.map": false,
   },
-}
+};

package/src/main.js

@@ -1,8 +1,8 @@
-export { readGitHubWorkflowEnv } from "@jsenv/github-pull-request-impact"
+export { readGitHubWorkflowEnv } from "@jsenv/github-pull-request-impact";
 
-export { jsenvTrackingConfig } from "./jsenvTrackingConfig.js"
-export { transform as raw } from "./rawTransformation.js"
-export { transform as gzip } from "./gzipTransformation.js"
-export { transform as brotli } from "./brotliTransformation.js"
-export { generateFileSizeReport } from "./generateFileSizeReport.js"
-export { reportFileSizeImpactInGitHubPullRequest } from "./report_file_size_impact_in_github_pr.js"
+export { jsenvTrackingConfig } from "./jsenvTrackingConfig.js";
+export { transform as raw } from "./rawTransformation.js";
+export { transform as gzip } from "./gzipTransformation.js";
+export { transform as brotli } from "./brotliTransformation.js";
+export { generateFileSizeReport } from "./generateFileSizeReport.js";
+export { reportFileSizeImpactInGitHubPullRequest } from "./report_file_size_impact_in_github_pr.js";

package/src/rawTransformation.js

@@ -1,3 +1,3 @@
-export const name = "raw"
+export const name = "raw";
 
-export const transform = (value) => value
+export const transform = (value) => value;