fix: label_command doesn't trigger when workflow also has slash_command

actions/setup/js/check_command_position.cjs

@@ -41,6 +41,18 @@ async function main() {
   const eventName = context.eventName;
 
   try {
+    // For labeled events (label-command triggers), skip the command position check.
+    // The label name itself is the trigger, not a slash command in the body.
+    // Label name matching is enforced by the workflow-level `if:` condition
+    // (e.g. github.event.label.name == 'cloclo'), so no additional filtering
+    // is needed here regardless of which labels are configured.
+    if (context.payload?.action === "labeled") {
+      core.info(`Event ${eventName} with action 'labeled' does not require command position check`);
+      core.setOutput("command_position_ok", "true");
+      core.setOutput("matched_command", "");
+      return;
+    }
+
     if (eventName === "issues") {
       text = context.payload.issue?.body || "";
     } else if (eventName === "pull_request") {

actions/setup/js/check_command_position.test.cjs

@@ -114,5 +114,29 @@ const mockCore = {
           (mockContext.payload = { comment: { body: "/discuss-bot analyze this" } }),
           await eval(`(async () => { ${checkCommandPositionScript}; await main(); })()`),
           expect(mockCore.setOutput).toHaveBeenCalledWith("command_position_ok", "true"));
+      }),
+      it("should pass for labeled issues event (label-command trigger)", async () => {
+        process.env.GH_AW_COMMANDS = JSON.stringify(["cloclo"]);
+        mockContext.eventName = "issues";
+        mockContext.payload = { action: "labeled", label: { name: "cloclo" }, issue: { body: "This is a regular issue body" } };
+        await eval(`(async () => { ${checkCommandPositionScript}; await main(); })()`);
+        expect(mockCore.setOutput).toHaveBeenCalledWith("command_position_ok", "true");
+        expect(mockCore.info).toHaveBeenCalledWith(expect.stringContaining("does not require command position check"));
+      }),
+      it("should pass for labeled pull_request event (label-command trigger)", async () => {
+        process.env.GH_AW_COMMANDS = JSON.stringify(["cloclo"]);
+        mockContext.eventName = "pull_request";
+        mockContext.payload = { action: "labeled", label: { name: "cloclo" }, pull_request: { body: "PR body without command" } };
+        await eval(`(async () => { ${checkCommandPositionScript}; await main(); })()`);
+        expect(mockCore.setOutput).toHaveBeenCalledWith("command_position_ok", "true");
+        expect(mockCore.info).toHaveBeenCalledWith(expect.stringContaining("does not require command position check"));
+      }),
+      it("should pass for labeled discussion event (label-command trigger)", async () => {
+        process.env.GH_AW_COMMANDS = JSON.stringify(["cloclo"]);
+        mockContext.eventName = "discussion";
+        mockContext.payload = { action: "labeled", label: { name: "cloclo" }, discussion: { body: "Discussion body without command" } };
+        await eval(`(async () => { ${checkCommandPositionScript}; await main(); })()`);
+        expect(mockCore.setOutput).toHaveBeenCalledWith("command_position_ok", "true");
+        expect(mockCore.info).toHaveBeenCalledWith(expect.stringContaining("does not require command position check"));
       }));
   }));