feat: create comment with discussion instead of note (#27)

close #16

Author: JounQin

.changeset/eight-buttons-argue.md

@@ -0,0 +1,5 @@
+---
+"changesets-gitlab": minor
+---
+
+feat: use discussion instead of comment

src/comment.ts

@@ -99,17 +99,31 @@ ${changedPackages.map(x => `"${x}": patch`).join('\n')}
 ${title}
 `)
 
-const getCommentId = (api: Gitlab, mrIid: number | string) =>
-  api.MergeRequestNotes.all(context.projectId, mrIid).then(comments => {
-    const changesetBotComment = comments.find(
-      comment =>
-        comment.author.username === process.env.GITLAB_CI_USER_NAME &&
-        // We need to ensure the comment is generated by us but we don't have a app bot like GitHub
-        // @see https://github.com/apps/changeset-bot
-        comment.body.includes('Generated By Changesets GitLab Bot'),
-    )
-    return changesetBotComment ? changesetBotComment.id : null
-  })
+const getNoteInfo = (api: Gitlab, mrIid: number | string) =>
+  api.MergeRequestDiscussions.all(context.projectId, mrIid).then(
+    discussions => {
+      for (const discussion of discussions) {
+        if (!discussion.notes) {
+          continue
+        }
+
+        const changesetBotNote = discussion.notes.find(
+          note =>
+            note.author.username === process.env.GITLAB_CI_USER_NAME &&
+            // We need to ensure the note is generated by us but we don't have a app bot like GitHub
+            // @see https://github.com/apps/changeset-bot
+            note.body.includes('Generated By Changesets GitLab Bot'),
+        )
+
+        if (changesetBotNote) {
+          return {
+            discussionId: discussion.id,
+            noteId: changesetBotNote.id,
+          }
+        }
+      }
+    },
+  )
 
 const hasChangesetBeenAdded = (
   changedFilesPromise: ReturnType<MergeRequests['changes']>,
@@ -154,9 +168,9 @@ export const comment = async () => {
       mrIid,
     )
 
-    const [commentId, hasChangeset, { changedPackages, releasePlan }] =
+    const [noteInfo, hasChangeset, { changedPackages, releasePlan }] =
       await Promise.all([
-        getCommentId(api, mrIid),
+        getNoteInfo(api, mrIid),
         hasChangesetBeenAdded(changedFilesPromise),
         getChangedPackages({
           changedFiles: changedFilesPromise.then(x =>
@@ -193,15 +207,23 @@ export const comment = async () => {
         : getAbsentMessage(latestCommitSha, addChangesetUrl, releasePlan)) +
       errFromFetchingChangedFiles
 
-    if (commentId != null) {
-      return api.MergeRequestNotes.edit(
+    if (noteInfo != null) {
+      return api.MergeRequestDiscussions.editNote(
         context.projectId,
         mrIid,
-        commentId,
-        prComment,
+        // @ts-expect-error - https://github.com/jdalrymple/gitbeaker/pull/523#issuecomment-975276068
+        noteInfo.discussionId,
+        noteInfo.noteId,
+        {
+          body: prComment,
+        },
       )
     }
-    return api.MergeRequestNotes.create(context.projectId, mrIid, prComment)
+    return api.MergeRequestDiscussions.create(
+      context.projectId,
+      mrIid,
+      prComment,
+    )
   } catch (err: unknown) {
     console.error(err)
     throw err