@jsenv/github-pull-request-impact 1.6.11 → 1.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/github-pull-request-impact",
-  "version": "1.6.11",
+  "version": "1.7.0",
   "description": "Report pull request impact on something",
   "license": "MIT",
   "author": {
@@ -23,24 +23,20 @@
   "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",
-    "test": "node ./script/test/test.mjs",
-    "test-with-coverage": "npm run test -- --coverage",
-    "prettier": "prettier --write ."
+    "test": "node ./scripts/test.mjs"
   },
   "dependencies": {
-    "@jsenv/filesystem": "3.1.0",
-    "@jsenv/logger": "4.0.1",
-    "node-fetch": "2.6.7"
+    "@jsenv/fetch": "1.1.2",
+    "@jsenv/filesystem": "4.1.0",
+    "@jsenv/log": "2.1.0"
   }
 }

package/src/commentGitHubPullRequestImpact.js

@@ -4,11 +4,9 @@
  * @jsenv/file-size-impact, @jsenv/performance-impact and @jsenv/lighthouse-score-impact
  */
 
-import {
-  assertAndNormalizeDirectoryUrl,
-  urlToFileSystemPath,
-} from "@jsenv/filesystem"
-import { createLogger } from "@jsenv/logger"
+import { fileURLToPath } from "node:url"
+import { assertAndNormalizeDirectoryUrl } from "@jsenv/filesystem"
+import { createLogger } from "@jsenv/log"
 
 import * as githubRESTAPI from "./internal/github_rest_api.js"
 import { exec } from "./internal/exec.js"
@@ -133,7 +131,7 @@ const commentPrImpact = async ({
   const execCommandInRootDirectory = (command) => {
     logger.info(`> ${command}`)
     return exec(command, {
-      cwd: urlToFileSystemPath(rootDirectoryUrl),
+      cwd: fileURLToPath(rootDirectoryUrl),
       onLog: (string) => {
         if (commandLogs) {
           logger.info(string)
@@ -370,9 +368,7 @@ const ensureGitConfig = async (
     await execCommandInRootDirectory(`git config ${name}`)
     return () => {}
   } catch (e) {
-    await execCommandInRootDirectory(
-      `git config ${name} "${valueIfMissing}"`,
-    )
+    await execCommandInRootDirectory(`git config ${name} "${valueIfMissing}"`)
     return async () => {
       await execCommandInRootDirectory(`git config --unset ${name}`)
     }

package/src/internal/createGitHubPullRequestCommentText.js

@@ -36,7 +36,14 @@ export const createGitHubPullRequestCommentText = ({
   })
 }
 
-const truncateComment = ({ commentLength, maxLength, header, warnings, body, footer }) => {
+const truncateComment = ({
+  commentLength,
+  maxLength,
+  header,
+  warnings,
+  body,
+  footer,
+}) => {
   const warningsWithTruncateWarning = [
     ...warnings,
     `**Warning:** The comment body was truncated to fit GitHub limit on comment length.
@@ -69,7 +76,9 @@ For the record the full comment length was ${commentLength} bytes.`,
 
 const composeCommentText = ({ header, warnings, body, footer }) => {
   const warningsAsText = composeWarningsText(warnings)
-  const parts = [header, warningsAsText, body, footer].filter((string) => string.length > 0)
+  const parts = [header, warningsAsText, body, footer].filter(
+    (string) => string.length > 0,
+  )
 
   return parts.join(`
 

package/src/internal/github_rest_api.js

@@ -1,4 +1,4 @@
-import nodeFetch from "node-fetch"
+import { fetchUrl } from "@jsenv/fetch"
 
 export const GET = async ({ url, githubToken, headers }) => {
   return sendHttpRequest({
@@ -72,7 +72,7 @@ const sendHttpRequest = async ({
 }) => {
   let response
   try {
-    response = await nodeFetch(url, {
+    response = await fetchUrl(url, {
       method,
       headers: {
         ...(typeof body === "undefined"

package/src/internal/renderGeneratedByText.js

@@ -1,4 +1,8 @@
-export const renderGeneratedByText = ({ generatedByLink, runLink, commitLink }) => {
+export const renderGeneratedByText = ({
+  generatedByLink,
+  runLink,
+  commitLink,
+}) => {
   if (runLink && typeof runLink !== "object") {
     console.warn(
       `runLink ignored because it should be an object with {url, text}, received ${runLink}`,
@@ -9,7 +13,9 @@ export const renderGeneratedByText = ({ generatedByLink, runLink, commitLink })
   const parts = [
     `Generated by <a href="${generatedByLink.url}">${generatedByLink.text}</a>`,
     ...(runLink ? [`during <a href="${runLink.url}">${runLink.text}</a>`] : []),
-    ...(commitLink ? [`on <a href="${commitLink.url}">${commitLink.text}</a>`] : []),
+    ...(commitLink
+      ? [`on <a href="${commitLink.url}">${commitLink.text}</a>`]
+      : []),
   ]
 
   return `<sub>

package/src/main.js

@@ -0,0 +1,6 @@
+export { readGitHubWorkflowEnv } from "./readGitHubWorkflowEnv.js"
+export { commentGitHubPullRequestImpact } from "./commentGitHubPullRequestImpact.js"
+
+// not planned to be documented but can be useful
+// to get a github comment from a { warnings, body } object
+export { createGitHubPullRequestCommentText } from "./internal/createGitHubPullRequestCommentText.js"

package/main.js

@@ -1,6 +0,0 @@
-export { readGitHubWorkflowEnv } from "./src/readGitHubWorkflowEnv.js"
-export { commentGitHubPullRequestImpact } from "./src/commentGitHubPullRequestImpact.js"
-
-// not planned to be documented but can be useful
-// to get a github comment from a { warnings, body } object
-export { createGitHubPullRequestCommentText } from "./src/internal/createGitHubPullRequestCommentText.js"