feature-296-bugbot-autofix: Implement getTokenUserDetails method to retrieve GitHub user details for commit author configuration, enhancing bugbot autofix commit process with proper user attribution.

efraespada · efraespada · commit 18284987b937 · 2026-02-12T00:59:20.000+01:00
diff --git a/.gitignore b/.gitignore
@@ -131,6 +131,9 @@ web_modules/
 
 *.lock
 
+# OpenCode runtime (created/removed by action or CLI)
+opencode.json
+
 .idea
 
 .env
diff --git a/build/cli/index.js b/build/cli/index.js
@@ -51379,6 +51379,16 @@ class ProjectRepository {
             const { data: user } = await octokit.rest.users.getAuthenticated();
             return user.login;
         };
+        /** Name and email of the token user, for git commit author (e.g. bugbot autofix). */
+        this.getTokenUserDetails = async (token) => {
+            const octokit = github.getOctokit(token);
+            const { data: user } = await octokit.rest.users.getAuthenticated();
+            const name = (user.name ?? user.login ?? "GitHub Action").trim() || "GitHub Action";
+            const email = (typeof user.email === "string" && user.email.trim().length > 0)
+                ? user.email.trim()
+                : `${user.login}@users.noreply.github.com`;
+            return { name, email };
+        };
         this.findTag = async (owner, repo, tag, token) => {
             const octokit = github.getOctokit(token);
             try {
@@ -53743,6 +53753,7 @@ exports.SingleActionUseCase = SingleActionUseCase;
 /**
  * Runs verify commands and then git add/commit/push for bugbot autofix.
  * Uses @actions/exec; intended to run in the GitHub Action runner where the repo is checked out.
+ * Configures git user.name and user.email from the token user so the commit has a valid author.
  */
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
     if (k2 === undefined) k2 = k;
@@ -53780,6 +53791,7 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.runBugbotAutofixCommitAndPush = runBugbotAutofixCommitAndPush;
 const exec = __importStar(__nccwpck_require__(1514));
+const project_repository_1 = __nccwpck_require__(7917);
 const logger_1 = __nccwpck_require__(8836);
 /**
  * Optionally check out the branch (when event is issue_comment and we resolved the branch from an open PR).
@@ -53867,6 +53879,11 @@ async function runBugbotAutofixCommitAndPush(execution, options) {
         return { success: true, committed: false };
     }
     try {
+        const projectRepository = new project_repository_1.ProjectRepository();
+        const { name, email } = await projectRepository.getTokenUserDetails(execution.tokens.token);
+        await exec.exec("git", ["config", "user.name", name]);
+        await exec.exec("git", ["config", "user.email", email]);
+        (0, logger_1.logDebugInfo)(`Git author set to ${name} <${email}>.`);
         await exec.exec("git", ["add", "-A"]);
         const commitMessage = "fix: bugbot autofix - resolve reported findings";
         await exec.exec("git", ["commit", "-m", commitMessage]);
diff --git a/build/cli/src/data/repository/project_repository.d.ts b/build/cli/src/data/repository/project_repository.d.ts
@@ -21,6 +21,11 @@ export declare class ProjectRepository {
     getRandomMembers: (organization: string, membersToAdd: number, currentMembers: string[], token: string) => Promise<string[]>;
     getAllMembers: (organization: string, token: string) => Promise<string[]>;
     getUserFromToken: (token: string) => Promise<string>;
+    /** Name and email of the token user, for git commit author (e.g. bugbot autofix). */
+    getTokenUserDetails: (token: string) => Promise<{
+        name: string;
+        email: string;
+    }>;
     private findTag;
     private getTagSHA;
     updateTag: (owner: string, repo: string, sourceTag: string, targetTag: string, token: string) => Promise<void>;
diff --git a/build/cli/src/usecase/steps/commit/bugbot/__tests__/bugbot_autofix_commit.test.d.ts b/build/cli/src/usecase/steps/commit/bugbot/__tests__/bugbot_autofix_commit.test.d.ts
@@ -1,4 +1,4 @@
 /**
- * Unit tests for runBugbotAutofixCommitAndPush: no branch, branchOverride checkout, verify commands, no changes, commit/push.
+ * Unit tests for runBugbotAutofixCommitAndPush: no branch, branchOverride checkout, verify commands, no changes, commit/push, git author.
  */
 export {};
diff --git a/build/cli/src/usecase/steps/commit/bugbot/bugbot_autofix_commit.d.ts b/build/cli/src/usecase/steps/commit/bugbot/bugbot_autofix_commit.d.ts
@@ -1,6 +1,7 @@
 /**
  * Runs verify commands and then git add/commit/push for bugbot autofix.
  * Uses @actions/exec; intended to run in the GitHub Action runner where the repo is checked out.
+ * Configures git user.name and user.email from the token user so the commit has a valid author.
  */
 import type { Execution } from "../../../../data/model/execution";
 export interface BugbotAutofixCommitResult {
diff --git a/build/github_action/index.js b/build/github_action/index.js
@@ -46470,6 +46470,16 @@ class ProjectRepository {
             const { data: user } = await octokit.rest.users.getAuthenticated();
             return user.login;
         };
+        /** Name and email of the token user, for git commit author (e.g. bugbot autofix). */
+        this.getTokenUserDetails = async (token) => {
+            const octokit = github.getOctokit(token);
+            const { data: user } = await octokit.rest.users.getAuthenticated();
+            const name = (user.name ?? user.login ?? "GitHub Action").trim() || "GitHub Action";
+            const email = (typeof user.email === "string" && user.email.trim().length > 0)
+                ? user.email.trim()
+                : `${user.login}@users.noreply.github.com`;
+            return { name, email };
+        };
         this.findTag = async (owner, repo, tag, token) => {
             const octokit = github.getOctokit(token);
             try {
@@ -48834,6 +48844,7 @@ exports.SingleActionUseCase = SingleActionUseCase;
 /**
  * Runs verify commands and then git add/commit/push for bugbot autofix.
  * Uses @actions/exec; intended to run in the GitHub Action runner where the repo is checked out.
+ * Configures git user.name and user.email from the token user so the commit has a valid author.
  */
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
     if (k2 === undefined) k2 = k;
@@ -48871,6 +48882,7 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.runBugbotAutofixCommitAndPush = runBugbotAutofixCommitAndPush;
 const exec = __importStar(__nccwpck_require__(1514));
+const project_repository_1 = __nccwpck_require__(7917);
 const logger_1 = __nccwpck_require__(8836);
 /**
  * Optionally check out the branch (when event is issue_comment and we resolved the branch from an open PR).
@@ -48958,6 +48970,11 @@ async function runBugbotAutofixCommitAndPush(execution, options) {
         return { success: true, committed: false };
     }
     try {
+        const projectRepository = new project_repository_1.ProjectRepository();
+        const { name, email } = await projectRepository.getTokenUserDetails(execution.tokens.token);
+        await exec.exec("git", ["config", "user.name", name]);
+        await exec.exec("git", ["config", "user.email", email]);
+        (0, logger_1.logDebugInfo)(`Git author set to ${name} <${email}>.`);
         await exec.exec("git", ["add", "-A"]);
         const commitMessage = "fix: bugbot autofix - resolve reported findings";
         await exec.exec("git", ["commit", "-m", commitMessage]);
diff --git a/build/github_action/src/data/repository/project_repository.d.ts b/build/github_action/src/data/repository/project_repository.d.ts
@@ -21,6 +21,11 @@ export declare class ProjectRepository {
     getRandomMembers: (organization: string, membersToAdd: number, currentMembers: string[], token: string) => Promise<string[]>;
     getAllMembers: (organization: string, token: string) => Promise<string[]>;
     getUserFromToken: (token: string) => Promise<string>;
+    /** Name and email of the token user, for git commit author (e.g. bugbot autofix). */
+    getTokenUserDetails: (token: string) => Promise<{
+        name: string;
+        email: string;
+    }>;
     private findTag;
     private getTagSHA;
     updateTag: (owner: string, repo: string, sourceTag: string, targetTag: string, token: string) => Promise<void>;
diff --git a/build/github_action/src/usecase/steps/commit/bugbot/__tests__/bugbot_autofix_commit.test.d.ts b/build/github_action/src/usecase/steps/commit/bugbot/__tests__/bugbot_autofix_commit.test.d.ts
@@ -1,4 +1,4 @@
 /**
- * Unit tests for runBugbotAutofixCommitAndPush: no branch, branchOverride checkout, verify commands, no changes, commit/push.
+ * Unit tests for runBugbotAutofixCommitAndPush: no branch, branchOverride checkout, verify commands, no changes, commit/push, git author.
  */
 export {};
diff --git a/build/github_action/src/usecase/steps/commit/bugbot/bugbot_autofix_commit.d.ts b/build/github_action/src/usecase/steps/commit/bugbot/bugbot_autofix_commit.d.ts
@@ -1,6 +1,7 @@
 /**
  * Runs verify commands and then git add/commit/push for bugbot autofix.
  * Uses @actions/exec; intended to run in the GitHub Action runner where the repo is checked out.
+ * Configures git user.name and user.email from the token user so the commit has a valid author.
  */
 import type { Execution } from "../../../../data/model/execution";
 export interface BugbotAutofixCommitResult {
diff --git a/src/data/repository/project_repository.ts b/src/data/repository/project_repository.ts
@@ -558,7 +558,19 @@ export class ProjectRepository {
         const octokit = github.getOctokit(token);
         const {data: user} = await octokit.rest.users.getAuthenticated();
         return user.login;
-    }
+    };
+
+    /** Name and email of the token user, for git commit author (e.g. bugbot autofix). */
+    getTokenUserDetails = async (token: string): Promise<{ name: string; email: string }> => {
+        const octokit = github.getOctokit(token);
+        const { data: user } = await octokit.rest.users.getAuthenticated();
+        const name = (user.name ?? user.login ?? "GitHub Action").trim() || "GitHub Action";
+        const email =
+            (typeof user.email === "string" && user.email.trim().length > 0)
+                ? user.email.trim()
+                : `${user.login}@users.noreply.github.com`;
+        return { name, email };
+    };
 
     private findTag = async (owner: string, repo: string, tag: string, token: string): Promise<{ object: { sha: string } } | undefined> => {
       const octokit = github.getOctokit(token);
diff --git a/src/usecase/steps/commit/bugbot/__tests__/bugbot_autofix_commit.test.ts b/src/usecase/steps/commit/bugbot/__tests__/bugbot_autofix_commit.test.ts
@@ -1,5 +1,5 @@
 /**
- * Unit tests for runBugbotAutofixCommitAndPush: no branch, branchOverride checkout, verify commands, no changes, commit/push.
+ * Unit tests for runBugbotAutofixCommitAndPush: no branch, branchOverride checkout, verify commands, no changes, commit/push, git author.
  */
 
 import * as exec from "@actions/exec";
@@ -12,6 +12,13 @@ jest.mock("../../../../../utils/logger", () => ({
     logError: jest.fn(),
 }));
 
+const mockGetTokenUserDetails = jest.fn();
+jest.mock("../../../../../data/repository/project_repository", () => ({
+    ProjectRepository: jest.fn().mockImplementation(() => ({
+        getTokenUserDetails: mockGetTokenUserDetails,
+    })),
+}));
+
 const mockExec = jest.spyOn(exec, "exec") as jest.Mock;
 
 type ExecCallback = (
@@ -37,6 +44,10 @@ function baseExecution(overrides: Partial<Execution> = {}): Execution {
 describe("runBugbotAutofixCommitAndPush", () => {
     beforeEach(() => {
         mockExec.mockReset();
+        mockGetTokenUserDetails.mockResolvedValue({
+            name: "Test User",
+            email: "test@users.noreply.github.com",
+        });
     });
 
     it("returns success false and committed false when branch is empty", async () => {
@@ -113,7 +124,7 @@ describe("runBugbotAutofixCommitAndPush", () => {
         expect(result.committed).toBe(false);
     });
 
-    it("runs git add, commit, push when there are changes", async () => {
+    it("runs git config (user.name, user.email), add, commit, push when there are changes", async () => {
         (mockExec.mockImplementation as (fn: ExecCallback) => void)((_cmd, args, opts) => {
             const a = args ?? [];
             if (a[0] === "status" && opts?.listeners?.stdout) {
@@ -126,6 +137,9 @@ describe("runBugbotAutofixCommitAndPush", () => {
 
         expect(result.success).toBe(true);
         expect(result.committed).toBe(true);
+        expect(mockGetTokenUserDetails).toHaveBeenCalledWith("t");
+        expect(mockExec).toHaveBeenCalledWith("git", ["config", "user.name", "Test User"]);
+        expect(mockExec).toHaveBeenCalledWith("git", ["config", "user.email", "test@users.noreply.github.com"]);
         expect(mockExec).toHaveBeenCalledWith("git", ["add", "-A"]);
         expect(mockExec).toHaveBeenCalledWith("git", [
             "commit",
@@ -144,6 +158,7 @@ describe("runBugbotAutofixCommitAndPush", () => {
             if (a[0] === "commit") return Promise.reject(new Error("commit failed"));
             return Promise.resolve(0);
         });
+        mockGetTokenUserDetails.mockResolvedValue({ name: "U", email: "u@x.com" });
 
         const result = await runBugbotAutofixCommitAndPush(baseExecution());
 
diff --git a/src/usecase/steps/commit/bugbot/bugbot_autofix_commit.ts b/src/usecase/steps/commit/bugbot/bugbot_autofix_commit.ts
@@ -1,9 +1,11 @@
 /**
  * Runs verify commands and then git add/commit/push for bugbot autofix.
  * Uses @actions/exec; intended to run in the GitHub Action runner where the repo is checked out.
+ * Configures git user.name and user.email from the token user so the commit has a valid author.
  */
 
 import * as exec from "@actions/exec";
+import { ProjectRepository } from "../../../../data/repository/project_repository";
 import { logDebugInfo, logError, logInfo } from "../../../../utils/logger";
 import type { Execution } from "../../../../data/model/execution";
 
@@ -108,6 +110,12 @@ export async function runBugbotAutofixCommitAndPush(
     }
 
     try {
+        const projectRepository = new ProjectRepository();
+        const { name, email } = await projectRepository.getTokenUserDetails(execution.tokens.token);
+        await exec.exec("git", ["config", "user.name", name]);
+        await exec.exec("git", ["config", "user.email", email]);
+        logDebugInfo(`Git author set to ${name} <${email}>.`);
+
         await exec.exec("git", ["add", "-A"]);
         const commitMessage = "fix: bugbot autofix - resolve reported findings";
         await exec.exec("git", ["commit", "-m", commitMessage]);
