@jsenv/file-size-impact 13.0.2 → 14.1.0

package/bin/filesize.mjs

@@ -0,0 +1,19 @@
+#!/usr/bin/env node
+
+import { pathToFileURL } from "node:url"
+import { generateFileSizeReport } from "@jsenv/file-size-impact"
+
+const cwdUrl = `${pathToFileURL(process.cwd())}/`
+const argv = process.argv.slice(2)
+const dirname = argv[0] || "dist"
+
+await generateFileSizeReport({
+  log: true,
+  rootDirectoryUrl: cwdUrl,
+  trackingConfig: {
+    [dirname]: {
+      [`${dirname}/**/*`]: true,
+      [`${dirname}/**/*.map`]: false,
+    },
+  },
+})

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/file-size-impact",
-  "version": "13.0.2",
+  "version": "14.1.0",
   "description": "Add files size impact into pull requests",
   "license": "MIT",
   "repository": {
@@ -21,6 +21,9 @@
     },
     "./*": "./*"
   },
+  "bin": {
+    "filesize": "./bin/filesize.mjs"
+  },
   "main": "./src/main.js",
   "files": [
     "/src/"
@@ -32,9 +35,9 @@
   },
   "dependencies": {
     "@jsenv/dynamic-import-worker": "1.1.0",
-    "@jsenv/filesystem": "4.1.6",
+    "@jsenv/filesystem": "4.2.3",
     "@jsenv/github-pull-request-impact": "1.7.1",
-    "@jsenv/log": "3.3.1",
-    "@jsenv/urls": "1.2.8"
+    "@jsenv/log": "3.3.4",
+    "@jsenv/urls": "2.0.0"
   }
 }

package/readme.md

@@ -69,7 +69,7 @@ _.github/workflows/file_size_impact.yml_
 
 name: file size impact
 
-on: pull_request_target
+on: pull_request
 
 jobs:
   file_size_impact:
@@ -77,11 +77,11 @@ jobs:
     name: file size impact
     steps:
       - name: Setup git
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
       - name: Setup node
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v3
         with:
-          node-version: "16.6.1"
+          node-version: "18.3.0"
       - name: Setup npm
         run: npm install
       - name: Report file size impact
@@ -101,11 +101,11 @@ _report_file_size_impact.mjs_
  */
 
 import {
-  reportFileSizeImpact,
+  reportFileSizeImpactInGitHubPullRequest,
   readGitHubWorkflowEnv,
 } from "@jsenv/file-size-impact"
 
-await reportFileSizeImpact({
+await reportFileSizeImpactInGitHubPullRequest({
   ...readGitHubWorkflowEnv(),
   buildCommand: "npm run dist",
   fileSizeReportUrl: new URL("./file_size.mjs#fileSizeReport", import.meta.url),
@@ -139,13 +139,13 @@ node ./report_file_size_impact.mjs
 
 ### 2. Adjust _report_file_size_impact.mjs_
 
-When outside a GitHub workflow, you cannot use _readGitHubWorkflowEnv()_. It means you must pass several parameters to _reportFileSizeImpact_. The example below assume code is executed by Travis.
+When outside a GitHub workflow, you cannot use _readGitHubWorkflowEnv()_. It means you must pass several parameters to _reportFileSizeImpactInGitHubPullRequest_. The example below assume code is executed by Travis.
 
 ```diff
-- import { reportFileSizeImpact, readGitHubWorkflowEnv } from "@jsenv/file-size-impact"
-+ import { reportFileSizeImpact } from "@jsenv/file-size-impact"
+- import { reportFileSizeImpactInGitHubPullRequest, readGitHubWorkflowEnv } from "@jsenv/file-size-impact"
++ import { reportFileSizeImpactInGitHubPullRequest } from "@jsenv/file-size-impact"
 
-reportFileSizeImpact({
+reportFileSizeImpactInGitHubPullRequest({
 - ...readGitHubWorkflowEnv(),
 + rootDirectoryUrl: process.env.TRAVIS_BUILD_DIR,
 + repositoryOwner: process.env.TRAVIS_REPO_SLUG.split("/")[0],
@@ -284,9 +284,9 @@ _manifestConfig_ parameter is an object used to configure the location of an opt
 This parameter reuses the shape of [trackingConfig](#trackingConfig) (associating pattern + value).
 
 ```js
-import { reportFileSizeImpact } from "@jsenv/file-size-impact"
+import { reportFileSizeImpactInGitHubPullRequest } from "@jsenv/file-size-impact"
 
-await reportFileSizeImpact({
+await reportFileSizeImpactInGitHubPullRequest({
   manifestConfig: {
     "./dist/**/manifest.json": true,
   },
@@ -296,9 +296,9 @@ await reportFileSizeImpact({
 You can disable manifest files handling by passing `null`.
 
 ```js
-import { reportFileSizeImpact } from "@jsenv/file-size-impact"
+import { reportFileSizeImpactInGitHubPullRequest } from "@jsenv/file-size-impact"
 
-await reportFileSizeImpact({
+await reportFileSizeImpactInGitHubPullRequest({
   manifestConfig: {
     "./dist/**/manifest.json": null,
   },
@@ -307,14 +307,17 @@ await reportFileSizeImpact({
 
 In that case _manifest.json_ will be handled as a regular file.
 
-# reportFileSizeImpact
+# reportFileSizeImpactInGitHubPullRequest
 
-_reportFileSizeImpact_ is an async function that will analyse a pull request file size impact and post a comment with the result of this analysis.
+_reportFileSizeImpactInGitHubPullRequest_ is an async function that will analyse a pull request file size impact and post a comment with the result of this analysis.
 
 ```js
-import { reportFileSizeImpact, raw } from "@jsenv/file-size-impact"
+import {
+  reportFileSizeImpactInGitHubPullRequest,
+  raw,
+} from "@jsenv/file-size-impact"
 
-await reportFileSizeImpact({
+await reportFileSizeImpactInGitHubPullRequest({
   logLevel: "info",
 
   rootDirectoryUrl: "file:///directory",
@@ -372,9 +375,9 @@ This parameter is returned by [readGitHubWorkflowEnv](#readGitHubWorkflowEnv) me
 Inside an other workflow, you can pass your own _runLink_. As in the example below where it is assumed that script is runned by jenkins.
 
 ```js
-import { reportFileSizeImpact } from "@jsenv/file-size-impact"
+import { reportFileSizeImpactInGitHubPullRequest } from "@jsenv/file-size-impact"
 
-await reportFileSizeImpact({
+await reportFileSizeImpactInGitHubPullRequest({
   runLink: {
     url: process.env.BUILD_URL,
     text: `${process.env.JOB_NAME}#${process.env.BUILD_ID}`,
@@ -390,17 +393,17 @@ _commitInGeneratedByInfo_ parameter is a boolean controlling if a link to the co
 
 # readGitHubWorkflowEnv
 
-_readGitHubWorkflowEnv_ is a function meant to be runned inside a GitHub workflow. It returns an object meant to be forwarded to [reportFileSizeImpact](#reportFileSizeImpact).
+_readGitHubWorkflowEnv_ is a function meant to be runned inside a GitHub workflow. It returns an object meant to be forwarded to [reportFileSizeImpactInGitHubPullRequest](#reportFileSizeImpactInGitHubPullRequest).
 
 ```js
 import {
-  reportFileSizeImpact,
+  reportFileSizeImpactInGitHubPullRequest,
   readGitHubWorkflowEnv,
 } from "@jsenv/file-size-impact"
 
 const gitHubWorkflowEnv = readGitHubWorkflowEnv()
 
-await reportFileSizeImpact({
+await reportFileSizeImpactInGitHubPullRequest({
   ...gitHubWorkflowEnv,
 })
 ```

package/src/main.js

@@ -5,4 +5,4 @@ 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 { reportFileSizeImpact } from "./reportFileSizeImpact.js"
+export { reportFileSizeImpactInGitHubPullRequest } from "./report_file_size_impact_in_github_pr.js"

package/src/{reportFileSizeImpact.js → report_file_size_impact_in_github_pr.js}

@@ -5,7 +5,7 @@ import { importOneExportFromFile } from "@jsenv/dynamic-import-worker"
 import { formatComment } from "./internal/formatComment.js"
 import { jsenvCommentParameters } from "./internal/jsenvCommentParameters.js"
 
-export const reportFileSizeImpact = async ({
+export const reportFileSizeImpactInGitHubPullRequest = async ({
   logLevel,
   commandLogs,
   cancellationToken,