fix(linear-sync): support variable-length team keys in issue regex (#3469) (#3472)

DEVOPS-471

Hardcoded \w{3}-\d{4} regex only matched 3-letter team keys like
ENG or OPS. Linear renamed OPS to DEVOPS (6 chars), breaking issue
detection in PR bodies and branch names.

New regex \w{2,10}-\d{1,5} supports:
- Team keys from 2-10 characters (QA, ENG, DEVOPS, etc.)
- Issue numbers from 1-5 digits (realistic for any team)

Added test cases for DEVOPS, QA, mixed team keys, and edge cases.

(cherry picked from commit 3aa6f71)

# Conflicts:
#	hack/linear-sync/linear_test.go

Co-authored-by: Piotr <[email protected]>

Author: loft-bot

hack/linear-sync/linear_test.go

@@ -243,10 +243,15 @@ func TestIssueIDsExtraction(t *testing.T) {
 	defer func() {
 		issuesInBodyREs = originalRegex
 	}()
+<<<<<<< HEAD
 
 	// For testing, use a regex that matches any 3-letter prefix format
+=======
+
+	// For testing, use a regex that matches team keys of 2-10 chars and issue numbers 1-5 digits
+>>>>>>> 3aa6f7157 (fix(linear-sync): support variable-length team keys in issue regex (#3469))
 	issuesInBodyREs = []*regexp.Regexp{
-		regexp.MustCompile(`(?P<issue>\w{3}-\d{4})`),
+		regexp.MustCompile(`(?P<issue>\w{2,10}-\d{1,5})`),
 	}
 
 	testCases := []struct {
@@ -285,6 +290,36 @@ func TestIssueIDsExtraction(t *testing.T) {
 			headRefName: "security/fix",
 			expected:    []string{},
 		},
+		{
+			name:        "Long team key (DEVOPS)",
+			body:        "This PR fixes DEVOPS-471",
+			headRefName: "feature/infra-update",
+			expected:    []string{"devops-471"},
+		},
+		{
+			name:        "Short team key (QA)",
+			body:        "This PR fixes QA-42",
+			headRefName: "feature/test-fix",
+			expected:    []string{"qa-42"},
+		},
+		{
+			name:        "Mixed team keys",
+			body:        "This PR fixes ENG-1234 and DEVOPS-471",
+			headRefName: "feature/QA-99-cross-team",
+			expected:    []string{"eng-1234", "devops-471", "qa-99"},
+		},
+		{
+			name:        "Issue with short number",
+			body:        "This PR fixes ENG-1",
+			headRefName: "feature/quick-fix",
+			expected:    []string{"eng-1"},
+		},
+		{
+			name:        "Issue with long number",
+			body:        "This PR fixes ENG-12345",
+			headRefName: "feature/big-project",
+			expected:    []string{"eng-12345"},
+		},
 	}
 
 	for _, tc := range testCases {

hack/linear-sync/pr.go

@@ -8,7 +8,7 @@ import (
 )
 
 var issuesInBodyREs = []*regexp.Regexp{
-	regexp.MustCompile(`(?P<issue>\w{3}-\d{4})`),
+	regexp.MustCompile(`(?i)(?P<issue>[A-Z]{2,10}-\d{1,5})`),
 }
 
 const PageSize = 100
@@ -64,4 +64,4 @@ func (p LinearPullRequest) IssueIDs() []string {
 	}
 
 	return issueIDs
-}
+}