Merge pull request #173 from backstage/rugvip/author

pr-automation: add actor override input and remove comment review scoring + bot check

Author: Rugvip

pr-automation/action.yml

@@ -29,7 +29,10 @@ inputs:
     description: Simulate a label being added (implies labeled action)
     required: false
   review-state:
-    description: Simulate a review submission with this state (approved, changes_requested, commented)
+    description: Simulate a review submission with this state (approved, changes_requested)
+    required: false
+  actor:
+    description: Override for the actor (user who triggered the event)
     required: false
 outputs: {}
 runs:

pr-automation/collectInput.ts

@@ -20,7 +20,6 @@ interface EventContext {
   repo: string;
   actor: string;
   labelAdded?: string;
-  commentAuthorLogin?: string;
   reviewState?: string;
 }
 
@@ -186,15 +185,6 @@ export async function collectInput(
   };
 
   const client = createAppClient();
-  const botLogin = await getBotLogin(client);
-
-  if (
-    botLogin &&
-    (event.actor === botLogin || event.commentAuthorLogin === botLogin)
-  ) {
-    core.info(`Skipping: triggered by bot (${botLogin})`);
-    return null;
-  }
 
   const data = await getPrAutomationData(client, {
     owner: ensuredEvent.owner,
@@ -232,9 +222,10 @@ export async function collectInput(
 }
 
 function getEventContext(): RawEventContext {
-  const prNumberInput = core.getInput('pr-number');
-  const labelAddedInput = core.getInput('label-added');
-  const reviewStateInput = core.getInput('review-state');
+  const prNumberInput = core.getInput('pr-number') || undefined;
+  const labelAddedInput = core.getInput('label-added') || undefined;
+  const reviewStateInput = core.getInput('review-state') || undefined;
+  const actorInput = core.getInput('actor') || undefined;
 
   // Infer event type from inputs
   let eventName = github.context.eventName;
@@ -253,15 +244,14 @@ function getEventContext(): RawEventContext {
     action,
     owner: github.context.repo.owner,
     repo: github.context.repo.repo,
-    actor: github.context.actor,
+    actor: actorInput ?? github.context.actor,
     labelAdded:
-      labelAddedInput ||
+      labelAddedInput ??
       (github.context.payload.action === 'labeled'
         ? github.context.payload.label?.name
         : undefined),
-    commentAuthorLogin: github.context.payload.comment?.user?.login,
     reviewState:
-      reviewStateInput ||
+      reviewStateInput ??
       (github.context.payload.review?.state as string | undefined),
   };
 }
@@ -276,17 +266,6 @@ function getPrNumber() {
   return undefined;
 }
 
-async function getBotLogin(
-  client: ReturnType<typeof github.getOctokit>,
-): Promise<string | null> {
-  try {
-    const app = await client.rest.apps.getAuthenticated();
-    return app.data.slug ? `${app.data.slug}[bot]` : null;
-  } catch {
-    return null;
-  }
-}
-
 async function listTeamMembers(
   client: ReturnType<typeof github.getOctokit>,
   org: string,

pr-automation/reviewerScoreLedger.ts

@@ -11,13 +11,11 @@ const STATUS_FIELD_NAME = 'Status';
 const REVIEW_SCORES: Record<string, number> = {
   APPROVED: 2,
   CHANGES_REQUESTED: 3,
-  COMMENTED: 1,
 };
 
 const REVIEW_STATUS_LABELS: Record<string, string> = {
   APPROVED: 'Approve',
   CHANGES_REQUESTED: 'Request changes',
-  COMMENTED: 'Comment',
 };
 
 interface ProjectData {