@jsenv/lighthouse-impact 2.1.7 → 2.1.10

package/README.md

@@ -115,7 +115,7 @@ _report_lighthouse_impact.mjs_
  * This file is executed by lighthouse_impact.yml GitHub workflow.
  * - it generates lighthouse report before and after merging a pull request
  * - Then, it creates or updates a comment in the pull request
- * See https://github.com/jsenv/lighthouse-impact#how-it-works
+ * See https://github.com/jsenv/workflow/tree/main/packages/jsenv-performance-impact
  */
 
 import {
@@ -162,7 +162,7 @@ The example below assume code is executed by Travis.
 
 reportLighthouseImpact({
 -  ...readGitHubWorkflowEnv(),
-+  projectDirectoryUrl: process.env.TRAVIS_BUILD_DIR,
++  rootDirectoryUrl: process.env.TRAVIS_BUILD_DIR,
 +  repositoryOwner: process.env.TRAVIS_REPO_SLUG.split("/")[0],
 +  repositoryName: process.env.TRAVIS_REPO_SLUG.split("/")[1],
 +  pullRequestNumber: process.env.TRAVIS_PULL_REQUEST,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/lighthouse-impact",
-  "version": "2.1.7",
+  "version": "2.1.10",
   "description": "Package description",
   "license": "MIT",
   "author": {
@@ -34,7 +34,6 @@
   ],
   "scripts": {
     "eslint": "npx eslint . --ext=.js,.mjs",
-    "importmap": "node ./script/importmap/importmap.mjs",
     "snapshot": "node ./test/comment/generate_comment_snapshot_file.mjs",
     "test": "node ./script/test/test.mjs",
     "test-with-coverage": "npm run test -- --coverage",
@@ -43,20 +42,10 @@
   "dependencies": {
     "@jsenv/abort": "4.1.2",
     "@jsenv/dynamic-import-worker": "1.0.1",
-    "@jsenv/filesystem": "2.7.1",
-    "@jsenv/github-pull-request-impact": "1.6.9",
+    "@jsenv/filesystem": "3.1.0",
+    "@jsenv/github-pull-request-impact": "1.6.11",
     "@jsenv/logger": "4.0.1",
     "chrome-launcher": "0.15.0",
     "lighthouse": "9.2.0"
-  },
-  "devDependencies": {
-    "@jsenv/assert": "2.4.1",
-    "@jsenv/core": "25.3.0",
-    "@jsenv/eslint-config": "16.0.9",
-    "@jsenv/importmap-eslint-resolver": "5.2.5",
-    "@jsenv/importmap-node-module": "5.1.3",
-    "eslint": "8.7.0",
-    "eslint-plugin-import": "2.25.4",
-    "prettier": "2.5.1"
   }
 }

package/src/generateLighthouseReport.js

@@ -31,7 +31,7 @@ export const generateLighthouseReport = async (
     runCount = 1,
     delayBetweenEachRunInSeconds = 1,
 
-    projectDirectoryUrl, // required only when jsonFile or htmlFile is passed
+    rootDirectoryUrl, // required only when jsonFile or htmlFile is passed
     log = false,
     jsonFile = false,
     jsonFileRelativeUrl = "./lighthouse/lighthouse_report.json",
@@ -118,17 +118,14 @@ export const generateLighthouseReport = async (
     await chrome.kill()
 
     if (jsonFile || htmlFile) {
-      projectDirectoryUrl = assertAndNormalizeDirectoryUrl(projectDirectoryUrl)
+      rootDirectoryUrl = assertAndNormalizeDirectoryUrl(rootDirectoryUrl)
     }
 
     const promises = []
     if (jsonFile) {
       promises.push(
         (async () => {
-          const jsonFileUrl = resolveUrl(
-            jsonFileRelativeUrl,
-            projectDirectoryUrl,
-          )
+          const jsonFileUrl = resolveUrl(jsonFileRelativeUrl, rootDirectoryUrl)
           const json = JSON.stringify(lighthouseReport, null, "  ")
           await writeFile(jsonFileUrl, json)
           if (jsonFileLog) {
@@ -140,10 +137,7 @@ export const generateLighthouseReport = async (
     if (htmlFile) {
       promises.push(
         (async () => {
-          const htmlFileUrl = resolveUrl(
-            htmlFileRelativeUrl,
-            projectDirectoryUrl,
-          )
+          const htmlFileUrl = resolveUrl(htmlFileRelativeUrl, rootDirectoryUrl)
           const html = ReportGenerator.generateReportHtml(lighthouseReport)
           await writeFile(htmlFileUrl, html)
           if (htmlFileLog) {

package/src/reportLighthouseImpact.js

@@ -9,7 +9,7 @@ export const reportLighthouseImpact = async ({
   logLevel,
   commandLogs = false,
   cancelOnSIGINT,
-  projectDirectoryUrl,
+  rootDirectoryUrl,
 
   githubToken,
   repositoryOwner,
@@ -23,17 +23,14 @@ export const reportLighthouseImpact = async ({
   catchError,
   skipGistWarning = false,
 }) => {
-  projectDirectoryUrl = assertAndNormalizeDirectoryUrl(projectDirectoryUrl)
+  rootDirectoryUrl = assertAndNormalizeDirectoryUrl(rootDirectoryUrl)
   if (typeof lighthouseReportPath !== "string") {
     throw new TypeError(
       `lighthouseReportPath must be a string but received ${lighthouseReportPath}`,
     )
   }
-  projectDirectoryUrl = assertAndNormalizeDirectoryUrl(projectDirectoryUrl)
-  const lighthouseReportUrl = resolveUrl(
-    lighthouseReportPath,
-    projectDirectoryUrl,
-  )
+  rootDirectoryUrl = assertAndNormalizeDirectoryUrl(rootDirectoryUrl)
+  const lighthouseReportUrl = resolveUrl(lighthouseReportPath, rootDirectoryUrl)
 
   return commentGitHubPullRequestImpact({
     logLevel,
@@ -41,7 +38,7 @@ export const reportLighthouseImpact = async ({
     // lighthouse report are super verbose, do not log them
     infoLogs: false,
     cancelOnSIGINT,
-    projectDirectoryUrl,
+    rootDirectoryUrl,
 
     githubToken,
     repositoryOwner,
@@ -111,7 +108,7 @@ export const reportLighthouseImpact = async ({
       }
     },
     generatedByLink: {
-      url: "https://github.com/jsenv/lighthouse-impact",
+      url: "https://github.com/jsenv/workflow/tree/main/packages/jsenv-lighthouse-impact",
       text: "@jsenv/lighthouse-impact",
     },
     runLink,