From 5fe66cbb6bd46ea4b2f4ea22f957583892e41f4b Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 23 Jan 2026 02:04:54 +0000 Subject: [PATCH 1/2] Initial plan From d9319f7cc36ede7e6c3ffbce92eee7867b2343db Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 23 Jan 2026 02:14:29 +0000 Subject: [PATCH 2/2] Make label removal instructions explicit in smoke tests - Update smoke-copilot.md to explicitly mention using remove_labels safe-output tool - Update smoke-codex.md to consistently use remove_labels tool - Recompile workflows to apply changes The issue was that the prompt said "Remove the label `smoke`" but didn't explicitly tell the agent to use the `remove_labels` safe-output tool. Making this explicit ensures the agent knows which tool to call. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- .github/workflows/smoke-codex.lock.yml | 4 +++- .github/workflows/smoke-codex.md | 4 +++- .github/workflows/smoke-copilot.lock.yml | 4 ++-- .github/workflows/smoke-copilot.md | 4 ++-- 4 files changed, 10 insertions(+), 6 deletions(-) diff --git a/.github/workflows/smoke-codex.lock.yml b/.github/workflows/smoke-codex.lock.yml index a3c8f2f16e..1347ff9a40 100644 --- a/.github/workflows/smoke-codex.lock.yml +++ b/.github/workflows/smoke-codex.lock.yml @@ -1047,7 +1047,9 @@ jobs: - ✅ or ❌ for each test result - Overall status: PASS or FAIL - If all tests pass, add the label `smoke-codex` to the pull request. + If all tests pass: + - Use the `add_labels` safe-output tool to add the label `smoke-codex` to the pull request + - Use the `remove_labels` safe-output tool to remove the label `smoke` from the pull request PROMPT_EOF - name: Substitute placeholders diff --git a/.github/workflows/smoke-codex.md b/.github/workflows/smoke-codex.md index b6c21ab088..2a48c21da4 100644 --- a/.github/workflows/smoke-codex.md +++ b/.github/workflows/smoke-codex.md @@ -81,4 +81,6 @@ Add a **very brief** comment (max 5-10 lines) to the current pull request with: - ✅ or ❌ for each test result - Overall status: PASS or FAIL -If all tests pass, add the label `smoke-codex` to the pull request. +If all tests pass: +- Use the `add_labels` safe-output tool to add the label `smoke-codex` to the pull request +- Use the `remove_labels` safe-output tool to remove the label `smoke` from the pull request diff --git a/.github/workflows/smoke-copilot.lock.yml b/.github/workflows/smoke-copilot.lock.yml index 69356f38d0..21b5868d02 100644 --- a/.github/workflows/smoke-copilot.lock.yml +++ b/.github/workflows/smoke-copilot.lock.yml @@ -966,8 +966,8 @@ jobs: - Mention the pull request author and any assignees If all tests pass: - - Add the label `smoke-copilot` to the pull request - - Remove the label `smoke` from the pull request + - Use the `add_labels` safe-output tool to add the label `smoke-copilot` to the pull request + - Use the `remove_labels` safe-output tool to remove the label `smoke` from the pull request PROMPT_EOF - name: Substitute placeholders diff --git a/.github/workflows/smoke-copilot.md b/.github/workflows/smoke-copilot.md index 07a2d35b82..34b82c9d6f 100644 --- a/.github/workflows/smoke-copilot.md +++ b/.github/workflows/smoke-copilot.md @@ -95,5 +95,5 @@ strict: true - Mention the pull request author and any assignees If all tests pass: -- Add the label `smoke-copilot` to the pull request -- Remove the label `smoke` from the pull request +- Use the `add_labels` safe-output tool to add the label `smoke-copilot` to the pull request +- Use the `remove_labels` safe-output tool to remove the label `smoke` from the pull request