From ef1e22261dbfd33366a54216b9d20c1ce65ad242 Mon Sep 17 00:00:00 2001 From: Tim Stranske Date: Thu, 3 Sep 2026 21:23:38 -0500 Subject: [PATCH 1/3] chore(deps): align shared ruff pin to 0.16.6 --- .github/workflows/autofix-versions.env | 2 +- pyproject.toml | 2 +- requirements.lock | 2 +- templates/consumer-repo/.github/workflows/autofix-versions.env | 2 +- .../integration-repo/.github/workflows/autofix-versions.env | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/workflows/autofix-versions.env b/.github/workflows/autofix-versions.env index 504812faa..da0fe0a04 100644 --- a/.github/workflows/autofix-versions.env +++ b/.github/workflows/autofix-versions.env @@ -5,7 +5,7 @@ # Runtime dependencies (PyYAML, Pydantic, Hypothesis) should be managed via Dependabot # in each consumer repo's pyproject.toml directly, NOT synced from this file. BLACK_VERSION=26.5.1 -RUFF_VERSION=0.16.5 +RUFF_VERSION=0.16.6 ISORT_VERSION=8.0.1 DOCFORMATTER_VERSION=1.7.8 MYPY_VERSION=2.3.1 diff --git a/pyproject.toml b/pyproject.toml index c209156b6..73e6a764a 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -31,7 +31,7 @@ app = [] dev = [ "pre-commit==4.6.1", "black==26.5.1", - "ruff==0.16.5", + "ruff==0.16.6", "isort==8.0.1", "docformatter==1.7.8", "mypy==2.3.1", diff --git a/requirements.lock b/requirements.lock index 51558d8f8..71384b983 100644 --- a/requirements.lock +++ b/requirements.lock @@ -298,7 +298,7 @@ rpds-py==2026.6.3 # via # jsonschema # referencing -ruff==0.16.5 +ruff==0.16.6 # via workflows (pyproject.toml) six==1.17.0 # via python-dateutil diff --git a/templates/consumer-repo/.github/workflows/autofix-versions.env b/templates/consumer-repo/.github/workflows/autofix-versions.env index 504812faa..da0fe0a04 100644 --- a/templates/consumer-repo/.github/workflows/autofix-versions.env +++ b/templates/consumer-repo/.github/workflows/autofix-versions.env @@ -5,7 +5,7 @@ # Runtime dependencies (PyYAML, Pydantic, Hypothesis) should be managed via Dependabot # in each consumer repo's pyproject.toml directly, NOT synced from this file. BLACK_VERSION=26.5.1 -RUFF_VERSION=0.16.5 +RUFF_VERSION=0.16.6 ISORT_VERSION=8.0.1 DOCFORMATTER_VERSION=1.7.8 MYPY_VERSION=2.3.1 diff --git a/templates/integration-repo/.github/workflows/autofix-versions.env b/templates/integration-repo/.github/workflows/autofix-versions.env index 504812faa..da0fe0a04 100644 --- a/templates/integration-repo/.github/workflows/autofix-versions.env +++ b/templates/integration-repo/.github/workflows/autofix-versions.env @@ -5,7 +5,7 @@ # Runtime dependencies (PyYAML, Pydantic, Hypothesis) should be managed via Dependabot # in each consumer repo's pyproject.toml directly, NOT synced from this file. BLACK_VERSION=26.5.1 -RUFF_VERSION=0.16.5 +RUFF_VERSION=0.16.6 ISORT_VERSION=8.0.1 DOCFORMATTER_VERSION=1.7.8 MYPY_VERSION=2.3.1 From 6010ee452a311aef7c218b51cd25f41fddae46c9 Mon Sep 17 00:00:00 2001 From: closer-lane Date: Sat, 5 Sep 2026 01:27:43 -0500 Subject: [PATCH 2/3] fix(agents): recognize bare coderabbit login --- .github/scripts/__tests__/bot-comment-handler.test.js | 1 + .github/scripts/bot-comment-handler.js | 1 + .github/workflows/reusable-bot-comment-handler.yml | 4 ++-- tests/workflows/test_bot_comment_handler.py | 1 + 4 files changed, 5 insertions(+), 2 deletions(-) diff --git a/.github/scripts/__tests__/bot-comment-handler.test.js b/.github/scripts/__tests__/bot-comment-handler.test.js index c3c14cb89..5a2986447 100644 --- a/.github/scripts/__tests__/bot-comment-handler.test.js +++ b/.github/scripts/__tests__/bot-comment-handler.test.js @@ -28,6 +28,7 @@ test('default bot author allowlist recognizes canonical review bots', () => { for (const login of [ 'copilot[bot]', 'github-actions[bot]', + 'coderabbitai', 'coderabbitai[bot]', 'chatgpt-codex-connector', 'chatgpt-codex-connector[bot]', diff --git a/.github/scripts/bot-comment-handler.js b/.github/scripts/bot-comment-handler.js index 0331515a0..26bb3f3e9 100644 --- a/.github/scripts/bot-comment-handler.js +++ b/.github/scripts/bot-comment-handler.js @@ -7,6 +7,7 @@ const MAX_COLLECTED_COMMENT_OUTPUT_LENGTH = 450000; const DEFAULT_BOT_AUTHORS = Object.freeze([ 'copilot[bot]', 'github-actions[bot]', + 'coderabbitai', 'coderabbitai[bot]', 'chatgpt-codex-connector', 'chatgpt-codex-connector[bot]', diff --git a/.github/workflows/reusable-bot-comment-handler.yml b/.github/workflows/reusable-bot-comment-handler.yml index 199e80cce..121cdf955 100644 --- a/.github/workflows/reusable-bot-comment-handler.yml +++ b/.github/workflows/reusable-bot-comment-handler.yml @@ -38,7 +38,7 @@ on: required: false type: string default: >- - Copilot,copilot[bot],github-actions[bot],coderabbitai[bot], + Copilot,copilot[bot],github-actions[bot],coderabbitai,coderabbitai[bot], chatgpt-codex-connector,chatgpt-codex-connector[bot] skip_if_human_replied: description: 'Legacy option: skip active threads after any non-bot reply' @@ -68,7 +68,7 @@ on: required: false type: string default: >- - Copilot,copilot[bot],github-actions[bot],coderabbitai[bot], + Copilot,copilot[bot],github-actions[bot],coderabbitai,coderabbitai[bot], chatgpt-codex-connector,chatgpt-codex-connector[bot] skip_if_human_replied: description: 'Legacy option: skip active threads after any non-bot reply' diff --git a/tests/workflows/test_bot_comment_handler.py b/tests/workflows/test_bot_comment_handler.py index 95711335e..e6ed3c6c6 100644 --- a/tests/workflows/test_bot_comment_handler.py +++ b/tests/workflows/test_bot_comment_handler.py @@ -19,6 +19,7 @@ def test_reusable_bot_comment_handler_ignores_agents_paths() -> None: assert ".agents/" in ignored_paths.split(",") bot_authors = inputs.get("bot_authors", {}).get("default", "") + assert "coderabbitai" in bot_authors.split(",") assert "chatgpt-codex-connector[bot]" in bot_authors From 3b01466f0747d761305a153b1e5accdcb35efb2f Mon Sep 17 00:00:00 2001 From: closer-lane Date: Sat, 5 Sep 2026 03:26:58 -0500 Subject: [PATCH 3/3] test(agents): keep bot author defaults aligned --- docs/bot-comment-handler.md | 2 +- tests/workflows/test_bot_comment_handler.py | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/docs/bot-comment-handler.md b/docs/bot-comment-handler.md index b43663fbb..4cd1165e7 100644 --- a/docs/bot-comment-handler.md +++ b/docs/bot-comment-handler.md @@ -45,7 +45,7 @@ The workflow reads the PR's labels to determine which agent to use: By default, the workflow processes comments from: - `copilot[bot]` - GitHub Copilot code review - `github-actions[bot]` - GitHub Actions (lint, type check suggestions) -- `coderabbitai[bot]` - CodeRabbit AI review +- `coderabbitai` and `coderabbitai[bot]` - CodeRabbit AI review identities - `chatgpt-codex-connector` - Codex connector review - `chatgpt-codex-connector[bot]` - Codex connector bot identity diff --git a/tests/workflows/test_bot_comment_handler.py b/tests/workflows/test_bot_comment_handler.py index e6ed3c6c6..fa728e2a4 100644 --- a/tests/workflows/test_bot_comment_handler.py +++ b/tests/workflows/test_bot_comment_handler.py @@ -63,6 +63,9 @@ def test_reusable_bot_comment_handler_has_manual_terminal_probe() -> None: assert dispatch_inputs["pr_number"].get("required") is True assert dispatch_inputs["dry_run"].get("default") is True + assert dispatch_inputs["bot_authors"].get("default") == call_inputs["bot_authors"].get( + "default" + ) assert dispatch_inputs["ignored_paths"].get("default") == call_inputs["ignored_paths"].get( "default" )