From 9fd5d7b212363a41f87f3facb66d4b0a5f293de1 Mon Sep 17 00:00:00 2001 From: Adam Moussa Date: Wed, 8 Jul 2026 17:17:20 -0400 Subject: [PATCH] fix(e2e): drop duplicate _ATTRIBUTION_RE from cherry-pick The reviewer-misc pick re-added _ATTRIBUTION_RE next to _latest_attribution, but the constant was already defined at module top (line 57, alongside _PLAN_URL_RE) via the earlier #81 sync. Remove the redundant redefinition; _latest_attribution resolves the surviving top-level constant. --- tests/e2e/fake_llm.py | 1 - 1 file changed, 1 deletion(-) diff --git a/tests/e2e/fake_llm.py b/tests/e2e/fake_llm.py index e2a6e6f4..99b7f85b 100644 --- a/tests/e2e/fake_llm.py +++ b/tests/e2e/fake_llm.py @@ -256,7 +256,6 @@ def _plan_complete_step(messages: list[BaseMessage]) -> AIMessage: FOLLOW_UP_REPLY = "Thanks! The PR is ready for review — anything else you'd like changed?" -_ATTRIBUTION_RE = re.compile(r"@([A-Za-z0-9-]+):") def _latest_attribution(messages: list[BaseMessage]) -> str | None: