Fix

Author: fisker

src/logger.ts

@@ -4,12 +4,9 @@ import { getOctokit } from "./octokit.ts";
 import { codeBlock } from "./utilities.ts";
 import { inspect } from "node:util";
 
-type Comment = Exclude<Awaited<ReturnType<typeof createComment>>, undefined>;
+type Comment = Awaited<ReturnType<typeof createComment>>;
 
 async function createComment(body: string) {
-  if (!IS_GITHUB_ACTION) {
-    return;
-  }
   const octokit = getOctokit();
 
   return await octokit.rest.issues.createComment({
@@ -20,9 +17,6 @@ async function createComment(body: string) {
 }
 
 async function updateComment(body: string, comment: Comment) {
-  if (!IS_GITHUB_ACTION) {
-    return;
-  }
   const octokit = getOctokit();
 
   return await octokit.rest.issues.updateComment({
@@ -34,39 +28,55 @@ async function updateComment(body: string, comment: Comment) {
 
 let briefCommentRequest: ReturnType<typeof createComment> | undefined;
 export async function brief(body: string) {
-  body += `\n__[details](${GITHUB_ACTION_JOB_URL})__`;
+  console.log(body);
 
-  if (!briefCommentRequest) {
-    briefCommentRequest = createComment(body);
-    await briefCommentRequest;
+  if (!IS_GITHUB_ACTION) {
     return;
   }
 
-  const comment = await briefCommentRequest;
+  body += `\n__[details](${GITHUB_ACTION_JOB_URL})__`;
+
+  if (briefCommentRequest) {
+    const comment = await briefCommentRequest;
+    try {
+      return await updateComment(body, comment);
+    } catch {
+      // No op
+    }
+  }
 
-  await updateComment(body, comment!);
+  briefCommentRequest = createComment(body);
+  return await briefCommentRequest;
 }
 
-export async function error(error: Error): Promise<void> {
+export async function error(error: Error) {
+  console.error(error);
+  if (!IS_GITHUB_ACTION) {
+    return;
+  }
+
   const text = `
   ### [${error.name}]
 
   ${codeBlock(inspect(error))}
   `;
-  console.error(error);
-  await brief(text);
+  return await brief(text);
 }
 
 export async function report(body: string) {
   console.log(body);
 
   if (briefCommentRequest) {
+    const request = briefCommentRequest;
     briefCommentRequest = undefined;
 
-    const comment = await briefCommentRequest;
-    await updateComment(body, comment!);
-    return;
+    try {
+      const comment = await request;
+      return await updateComment(body, comment);
+    } catch {
+      // No op
+    }
   }
 
-  await createComment(body);
+  return await createComment(body);
 }

src/report.ts

@@ -38,7 +38,8 @@ export const stringifyReport = ({
 }: {
   title: string;
   reports: Report[];
-}) => [title, ...reports.map(({ head, body }) => [head, body])].join("\n\n");
+}) =>
+  [title, ...reports.flatMap(({ head, body }) => [head, body])].join("\n\n");
 
 export function getReport({
   alternative,

src/utilities.ts

@@ -50,7 +50,7 @@ export const commitChanges = async (directory: string, message: string) => {
     { cwd: directory },
   );
 
-  const match = stdout.match(/^\[(?<commitHash>[a-f0-9]{40}) [a-f0-9]{7}]/);
+  const match = stdout.match(/^\[(?<commitHash>[a-f0-9]{40}) [a-f0-9]{8}] /);
   if (!match?.groups!.commitHash) {
     throw new Error(`Unexpected commit hash '${stdout}'`);
   }