fix(codeql): wire _FLEX_TEST into _TEST_ID_PATTERN — resolves unused-variable alert #132

tbitcs · oz-agent · tbitcs · commit 3c42e061b9c2 · 2026-05-30T09:03:37.000-04:00
_TEST_ID_PATTERN was duplicating the TEST- regex inline instead of referencing
the module-level _FLEX_TEST constant, leaving _FLEX_TEST unused.

Co-Authored-By: Oz &lt;oz-agent@warp.dev&gt;
diff --git a/src/specsmith/requirements.py b/src/specsmith/requirements.py
@@ -20,7 +20,7 @@
     r"(?:Covers|\*\*Requirement(?:\s+ID)?:?\*\*|Requirement(?:\s+ID)?):?\s*"
     r"(" + _FLEX_REQ + r"(?:\s*,\s*" + _FLEX_REQ + r")*)"
 )
-_TEST_ID_PATTERN = re.compile(r"\b(TEST-(?:[A-Z][A-Z0-9_]*-)?\d+[a-z]*)\b")
+_TEST_ID_PATTERN = re.compile(r"\b(" + _FLEX_TEST + r")\b")
 
 # Heading detectors for REQUIREMENTS.md (two styles supported):
 #   Style A: ## REQ-001 or ## REQ-CLI-001

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@`
`20`	`20`	`r"(?:Covers\|\\Requirement(?:\s+ID)?:?\\\|Requirement(?:\s+ID)?):?\s*"`
`21`	`21`	`r"(" + _FLEX_REQ + r"(?:\s,\s" + _FLEX_REQ + r")*)"`
`22`	`22`	`)`
`23`		`-_TEST_ID_PATTERN = re.compile(r"\b(TEST-(?:[A-Z][A-Z0-9_]-)?\d+[a-z])\b")`
	`23`	`+_TEST_ID_PATTERN = re.compile(r"\b(" + _FLEX_TEST + r")\b")`
`24`	`24`
`25`	`25`	`# Heading detectors for REQUIREMENTS.md (two styles supported):`
`26`	`26`	`# Style A: ## REQ-001 or ## REQ-CLI-001`