fix: slice logging count not fairly representing the actual annotatio…

…n collection slices
Codex- · Oct 31, 2023 · cbe5de1 · cbe5de1
1 parent e3e8fb9
commit cbe5de1
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/dist/index.mjs b/dist/index.mjs
diff --git a/src/tasks/check.ts b/src/tasks/check.ts
@@ -24,7 +24,9 @@ export async function updateCheckAnnotations(
 
   let i = 0;
   while (i < minimalAnnotations.length) {
-    core.debug(`[updateCheckAnnotations]: Slicing ${i}...${i + CHECK_ANNOTATIONS_UPDATE_LIMIT}`);
+    core.debug(
+      `[updateCheckAnnotations]: Slicing ${i}...${i + (CHECK_ANNOTATIONS_UPDATE_LIMIT - 1)}`,
+    );
 
     const slice = minimalAnnotations
       .slice(i, i + CHECK_ANNOTATIONS_UPDATE_LIMIT)

diff --git a/src/tasks/comment.ts b/src/tasks/comment.ts
@@ -90,7 +90,7 @@ async function createOrUpdateComments(
 ): Promise<number[]> {
   let existingIdsIndex = 0;
   for (const comment of commentsToPost) {
-    if (existingCommentIds && existingCommentIds[existingIdsIndex] !== undefined) {
+    if (existingCommentIds?.[existingIdsIndex] !== undefined) {
       const commentId = existingCommentIds[existingIdsIndex]!;
       await updateComment(commentId, comment);
       core.debug(`[createOrUpdateComments]: updated comment (${commentId})`);