From 28cc70261979d03399c8e7ae007ed125a7dc0358 Mon Sep 17 00:00:00 2001
From: Yuku Kotani <yukukotani@gmail.com>
Date: Wed, 6 Aug 2025 14:19:37 +0900
Subject: [PATCH] fix: update bot name format to include [bot] suffix in tests
 and docs
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Update test cases to use correct bot actor names with [bot] suffix
- Update documentation example to show correct bot name format
- Align with GitHub's actual bot naming convention

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 docs/usage.md      |  2 +-
 test/actor.test.ts | 16 ++++++++--------
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/docs/usage.md b/docs/usage.md
index 3f8ff73..7e77080 100644
--- a/docs/usage.md
+++ b/docs/usage.md
@@ -43,7 +43,7 @@ jobs:
           # additional_permissions: |
           #   actions: read
           # Optional: allow bot users to trigger the action
-          # allowed_bots: "dependabot,renovate"
+          # allowed_bots: "dependabot[bot],renovate[bot]"
 ```
 
 ## Inputs
diff --git a/test/actor.test.ts b/test/actor.test.ts
index 677689d..7d51e29 100644
--- a/test/actor.test.ts
+++ b/test/actor.test.ts
@@ -31,18 +31,18 @@ describe("checkHumanActor", () => {
   test("should throw error for bot actor when not allowed", async () => {
     const mockOctokit = createMockOctokit("Bot");
     const context = createMockContext();
-    context.actor = "test-bot";
+    context.actor = "test-bot[bot]";
     context.inputs.allowedBots = "";
 
     await expect(checkHumanActor(mockOctokit, context)).rejects.toThrow(
-      "Workflow initiated by non-human actor: test-bot (type: Bot). Add bot to allowed_bots list or use '*' to allow all bots.",
+      "Workflow initiated by non-human actor: test-bot[bot] (type: Bot). Add bot to allowed_bots list or use '*' to allow all bots.",
     );
   });
 
   test("should pass for bot actor when all bots allowed", async () => {
     const mockOctokit = createMockOctokit("Bot");
     const context = createMockContext();
-    context.actor = "test-bot";
+    context.actor = "test-bot[bot]";
     context.inputs.allowedBots = "*";
 
     await expect(
@@ -53,8 +53,8 @@ describe("checkHumanActor", () => {
   test("should pass for specific bot when in allowed list", async () => {
     const mockOctokit = createMockOctokit("Bot");
     const context = createMockContext();
-    context.actor = "dependabot";
-    context.inputs.allowedBots = "dependabot,renovate";
+    context.actor = "dependabot[bot]";
+    context.inputs.allowedBots = "dependabot[bot],renovate[bot]";
 
     await expect(
       checkHumanActor(mockOctokit, context),
@@ -64,11 +64,11 @@ describe("checkHumanActor", () => {
   test("should throw error for bot not in allowed list", async () => {
     const mockOctokit = createMockOctokit("Bot");
     const context = createMockContext();
-    context.actor = "other-bot";
-    context.inputs.allowedBots = "dependabot,renovate";
+    context.actor = "other-bot[bot]";
+    context.inputs.allowedBots = "dependabot[bot],renovate[bot]";
 
     await expect(checkHumanActor(mockOctokit, context)).rejects.toThrow(
-      "Workflow initiated by non-human actor: other-bot (type: Bot). Add bot to allowed_bots list or use '*' to allow all bots.",
+      "Workflow initiated by non-human actor: other-bot[bot] (type: Bot). Add bot to allowed_bots list or use '*' to allow all bots.",
     );
   });
 });