chore: switch from userLogin to botUsername

Signed-off-by: djamaile <rdjamaile@gmail.com>

Author: djamaile

changeset-feedback/index.ts

@@ -17,7 +17,7 @@ async function main() {
   const marker = core.getInput('marker', { required: true });
   const diffRef = core.getInput('diffRef', { required: true });
   const issueNumberStr = core.getInput('issue-number', { required: true });
-  const userLogin = core.getInput('userLogin', {required: true});
+  const botUsername = core.getInput('botUsername', {required: true});
   const changedFiles = await listChangedFiles(diffRef);
   const packages = await listPackages();
   const changesets = await loadChangesets(changedFiles);
@@ -32,7 +32,7 @@ async function main() {
     issueNumberStr,
     marker,
     feedback,
-    userLogin,
+    botUsername,
   });
 }
 

changeset-feedback/postFeedback.test.ts

@@ -52,7 +52,7 @@ const repoInfo = {
   owner: 'le-owner',
   repo: 'le-repo',
 };
-const userLogin = 'github-actions[bot]';
+const botUsername = 'github-actions[bot]';
 const log = jest.fn();
 const marker = 'changeset-feedback';
 const body = (feedback: string) =>
@@ -82,7 +82,7 @@ describe('changeset feedback', () => {
 
     await postFeedback(
       client,
-      { ...repoInfo, issueNumberStr: '1', marker, feedback, userLogin },
+      { ...repoInfo, issueNumberStr: '1', marker, feedback, botUsername },
       log,
     );
     expect(mockClient.rest.issues.createComment).toHaveBeenCalledWith({
@@ -97,7 +97,7 @@ describe('changeset feedback', () => {
     mockClient.paginate.mockResolvedValue(commentsWithFeedBack);
     await postFeedback(
       client,
-      { ...repoInfo, issueNumberStr: '1', marker, feedback, userLogin },
+      { ...repoInfo, issueNumberStr: '1', marker, feedback, botUsername },
       log,
     );
     expect(mockClient.rest.issues.updateComment).not.toHaveBeenCalled();
@@ -110,7 +110,7 @@ describe('changeset feedback', () => {
     mockClient.paginate.mockResolvedValue(commentsWithFeedBack);
     await postFeedback(
       client,
-      { ...repoInfo, issueNumberStr: '1', marker, feedback: feedbackUpdated, userLogin },
+      { ...repoInfo, issueNumberStr: '1', marker, feedback: feedbackUpdated, botUsername },
       log,
     );
     expect(log).toHaveBeenCalledWith('updating existing comment in #1');
@@ -125,7 +125,7 @@ describe('changeset feedback', () => {
     mockClient.paginate.mockResolvedValue(commentsWithFeedBack);
     await postFeedback(
       client,
-      { ...repoInfo, issueNumberStr: '1', marker: '', feedback: '', userLogin },
+      { ...repoInfo, issueNumberStr: '1', marker: '', feedback: '', botUsername },
       log,
     );
     expect(mockClient.rest.issues.deleteComment).toHaveBeenCalledWith({

changeset-feedback/postFeedback.ts

@@ -9,11 +9,11 @@ export async function postFeedback(
     issueNumberStr: string;
     marker: string;
     feedback: string;
-    userLogin: string;
+    botUsername: string;
   },
   log = core.info,
 ) {
-  const { owner, repo, issueNumberStr, marker, feedback, userLogin } = options;
+  const { owner, repo, issueNumberStr, marker, feedback, botUsername } = options;
   const issue_number = Number(issueNumberStr);
   const body = feedback.trim() ? feedback + marker : undefined;
 
@@ -26,11 +26,8 @@ export async function postFeedback(
     },
   );
 
-  const { login } = await client.rest.users.user();
-  log(`user: ${login}`);
-
   const existingComment = existingComments.find(
-    c => c.user?.login === userLogin && c.body?.includes(marker),
+    c => c.user?.login === botUsername && c.body?.includes(marker),
   );
 
   if (existingComment) {