From e1d6f2e36cd70998f7452e7c17f181307220bea0 Mon Sep 17 00:00:00 2001
From: luciferlive112116 <291889058+luciferlive112116@users.noreply.github.com>
Date: Thu, 2 Jul 2026 04:25:35 +0800
Subject: [PATCH] fix(signals): match hyphenated refactor-only no-issue
 rationale

Co-authored-by: Cursor <cursoragent@cursor.com>
---
 src/signals/engine.ts        |  3 ++-
 test/unit/signals-v2.test.ts | 13 +++++++++++++
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/signals/engine.ts b/src/signals/engine.ts
index ff4d12627a..235cbee3f9 100644
--- a/src/signals/engine.ts
+++ b/src/signals/engine.ts
@@ -4980,7 +4980,8 @@ export function hasClearNoIssueRationale(pr: Pick<PullRequestRecord, "title" | "
   // `tests?[\s-]+only` extends the same rule to test-only PRs (regression/coverage-only diffs) — parallel
   // to the docs-only hyphenation fix merged in #1905 and the test-only follow-up in #1993.
   // `ci[\s-]+only` covers CI/workflow-only PRs using the same Conventional Commits spelling.
-  return /\b(?:no issue\s*(?:because\b|:)|no linked issue\s*(?:because\b|:)|no ticket\s*(?:because\b|:)|(?:maintenance|docs?[\s-]+only|tests?[\s-]+only|ci[\s-]+only|typo|chore|cleanup)\b)/i.test([pr.title, pr.body ?? ""].join(" "));
+  // `refactor[\s-]+only` covers internal refactors with no behavior change using the same spelling.
+  return /\b(?:no issue\s*(?:because\b|:)|no linked issue\s*(?:because\b|:)|no ticket\s*(?:because\b|:)|(?:maintenance|docs?[\s-]+only|tests?[\s-]+only|ci[\s-]+only|refactor[\s-]+only|typo|chore|cleanup)\b)/i.test([pr.title, pr.body ?? ""].join(" "));
 }
 
 function hasValidationNote(value: string): boolean {
diff --git a/test/unit/signals-v2.test.ts b/test/unit/signals-v2.test.ts
index 0a96a86ada..3d16be51ad 100644
--- a/test/unit/signals-v2.test.ts
+++ b/test/unit/signals-v2.test.ts
@@ -1787,6 +1787,19 @@ describe("hasClearNoIssueRationale ci-only spelling", () => {
   });
 });
 
+describe("hasClearNoIssueRationale refactor-only spelling", () => {
+  it("recognizes hyphenated and spaced refactor-only rationales", () => {
+    expect(hasClearNoIssueRationale({ title: "refactor only: split helper", body: "" })).toBe(true);
+    expect(hasClearNoIssueRationale({ title: "refactor-only: split helper", body: "" })).toBe(true);
+    expect(hasClearNoIssueRationale({ title: "Rename queue module", body: "This is a refactor only rename." })).toBe(true);
+  });
+
+  it("still rejects unrelated PR text that mentions refactors without a rationale", () => {
+    expect(hasClearNoIssueRationale({ title: "Refactor queue processor", body: "Extracts shared helper." })).toBe(false);
+    expect(hasClearNoIssueRationale({ title: "Improve signal engine structure", body: "" })).toBe(false);
+  });
+});
+
 function snapshot(
   id: string,
   repositories: Array<{