@jsenv/lighthouse-impact 2.2.0 → 3.0.1

package/README.md

@@ -125,7 +125,7 @@ import {
 
 await reportLighthouseImpact({
   ...readGitHubWorkflowEnv(),
-  lighthouseReportPath: new URL(
+  lighthouseReportUrl: new URL(
     "./lighthouse.mjs#lighthouseReport",
     import.meta.url,
   ),
@@ -170,7 +170,7 @@ reportLighthouseImpact({
 +  repositoryName: process.env.TRAVIS_REPO_SLUG.split("/")[1],
 +  pullRequestNumber: process.env.TRAVIS_PULL_REQUEST,
 +  githubToken: process.env.GITHUB_TOKEN, // see next step
-   lighthouseReportPath: "./lighthouse.mjs#lighthouseReport",
+   lighthouseReportUrl: "./lighthouse.mjs#lighthouseReport",
 })
 ```
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/lighthouse-impact",
-  "version": "2.2.0",
+  "version": "3.0.1",
   "description": "Package description",
   "license": "MIT",
   "author": {
@@ -17,35 +17,30 @@
     "node": ">=16.13.0"
   },
   "publishConfig": {
-    "access": "public",
-    "registry": "https://registry.npmjs.org"
+    "access": "public"
   },
   "type": "module",
   "exports": {
     ".": {
-      "import": "./main.js"
+      "import": "./src/main.js"
     },
     "./*": "./*"
   },
-  "main": "./main.js",
+  "main": "./src/main.js",
   "files": [
-    "/src/",
-    "/main.js"
+    "/src/"
   ],
   "scripts": {
-    "eslint": "npx eslint . --ext=.js,.mjs",
-    "snapshot": "node ./test/comment/generate_comment_snapshot_file.mjs",
-    "test": "node ./script/test/test.mjs",
-    "test-with-coverage": "npm run test -- --coverage",
-    "prettier": "prettier --write ."
+    "snapshot": "node ./tests/comment/generate_comment_snapshot_file.mjs",
+    "test": "node ./scripts/test.mjs"
   },
   "dependencies": {
-    "@jsenv/abort": "4.1.2",
-    "@jsenv/dynamic-import-worker": "1.0.1",
-    "@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"
+    "@jsenv/abort": "4.2.4",
+    "@jsenv/dynamic-import-worker": "1.1.0",
+    "@jsenv/filesystem": "4.1.5",
+    "@jsenv/github-pull-request-impact": "1.7.0",
+    "@jsenv/log": "3.3.1",
+    "chrome-launcher": "0.15.1",
+    "lighthouse": "9.6.7"
   }
-}
+}

package/src/generateLighthouseReport.js

@@ -1,13 +1,8 @@
 // https://github.com/GoogleChrome/lighthouse/blob/5a14deb5c4e0ec4e8e58f50ff72b53851b021bcf/docs/readme.md#using-programmatically
 
 import { createRequire } from "node:module"
-
-import {
-  writeFile,
-  resolveUrl,
-  assertAndNormalizeDirectoryUrl,
-} from "@jsenv/filesystem"
-import { createLogger } from "@jsenv/logger"
+import { writeFile, assertAndNormalizeFileUrl } from "@jsenv/filesystem"
+import { createLogger } from "@jsenv/log"
 import { Abort, raceProcessTeardownEvents } from "@jsenv/abort"
 
 import { formatLighthouseReportForLog } from "./internal/formatLighthouseReportForLog.js"
@@ -31,13 +26,10 @@ export const generateLighthouseReport = async (
     runCount = 1,
     delayBetweenEachRunInSeconds = 1,
 
-    rootDirectoryUrl, // required only when jsonFile or htmlFile is passed
     log = false,
-    jsonFile = false,
-    jsonFileRelativeUrl = "./lighthouse/lighthouse_report.json",
+    jsonFileUrl,
     jsonFileLog = true,
-    htmlFile = false,
-    htmlFileRelativeUrl = "./lighthouse/lighthouse_report.html",
+    htmlFileUrl,
     htmlFileLog = true,
   } = {},
 ) => {
@@ -117,15 +109,11 @@ export const generateLighthouseReport = async (
 
     await chrome.kill()
 
-    if (jsonFile || htmlFile) {
-      rootDirectoryUrl = assertAndNormalizeDirectoryUrl(rootDirectoryUrl)
-    }
-
     const promises = []
-    if (jsonFile) {
+    if (jsonFileUrl) {
+      assertAndNormalizeFileUrl(jsonFileUrl)
       promises.push(
         (async () => {
-          const jsonFileUrl = resolveUrl(jsonFileRelativeUrl, rootDirectoryUrl)
           const json = JSON.stringify(lighthouseReport, null, "  ")
           await writeFile(jsonFileUrl, json)
           if (jsonFileLog) {
@@ -134,10 +122,10 @@ export const generateLighthouseReport = async (
         })(),
       )
     }
-    if (htmlFile) {
+    if (htmlFileUrl) {
+      assertAndNormalizeFileUrl(htmlFileUrl)
       promises.push(
         (async () => {
-          const htmlFileUrl = resolveUrl(htmlFileRelativeUrl, rootDirectoryUrl)
           const html = ReportGenerator.generateReportHtml(lighthouseReport)
           await writeFile(htmlFileUrl, html)
           if (htmlFileLog) {

package/src/internal/patchOrPostGists.js

@@ -1,5 +1,5 @@
 import * as githubRESTAPI from "@jsenv/github-pull-request-impact/src/internal/github_rest_api.js"
-import { createDetailedMessage } from "@jsenv/logger"
+import { createDetailedMessage } from "@jsenv/log"
 
 // https://developer.github.com/v3/gists/#create-a-gist
 

package/src/main.js

@@ -0,0 +1,4 @@
+export { readGitHubWorkflowEnv } from "@jsenv/github-pull-request-impact"
+
+export { generateLighthouseReport } from "./generateLighthouseReport.js"
+export { reportLighthouseImpact } from "./reportLighthouseImpact.js"

package/src/reportLighthouseImpact.js

@@ -16,7 +16,7 @@ export const reportLighthouseImpact = async ({
   repositoryName,
   pullRequestNumber,
   installCommand = "npm install",
-  lighthouseReportPath,
+  lighthouseReportUrl,
 
   runLink,
   commitInGeneratedByInfo,
@@ -24,14 +24,12 @@ export const reportLighthouseImpact = async ({
   skipGistWarning = false,
 }) => {
   rootDirectoryUrl = assertAndNormalizeDirectoryUrl(rootDirectoryUrl)
-  let lighthouseReportUrl
-  if (lighthouseReportPath === "string") {
-    lighthouseReportUrl = new URL(lighthouseReportPath, rootDirectoryUrl).href
-  } else if (lighthouseReportPath instanceof URL) {
-    lighthouseReportUrl = lighthouseReportPath.href
+  if (lighthouseReportUrl === "string") {
+    lighthouseReportUrl = new URL(lighthouseReportUrl, rootDirectoryUrl).href
+  } else if (lighthouseReportUrl instanceof URL) {
   } else {
     throw new TypeError(
-      `lighthouseReportPath must be a string or an url but received ${lighthouseReportPath}`,
+      `lighthouseReportUrl must be a string or an url but received ${lighthouseReportUrl}`,
     )
   }
 

package/main.js

@@ -1,4 +0,0 @@
-export { generateLighthouseReport } from "./src/generateLighthouseReport.js"
-
-export { readGitHubWorkflowEnv } from "@jsenv/github-pull-request-impact"
-export { reportLighthouseImpact } from "./src/reportLighthouseImpact.js"