docs(code-search): explain explicit AND queries (#1015)

Author: christian-byrne

comfy_cli/command/code_search.py

@@ -47,7 +47,8 @@
 # `timeout: 30` (a YAML-ish literal) are not rejected — a real Sourcegraph
 # filter value is never separated from its colon by whitespace.
 _INJECTED_FILTER_RE = re.compile(
-    r"(^|[\s(])-?(count|repo|r|timeout|context|rev|revision|fork|archived|select):(?=[^\s)])", re.IGNORECASE
+    r"(^|[\s(])-?(count|repo|r|timeout|context|rev|revision|fork|archived|select|patterntype):(?=[^\s)])",
+    re.IGNORECASE,
 )
 
 # C0 (0x00-0x1F) and C1 (0x7F-0x9F) control characters, including ESC (0x1B)
@@ -106,8 +107,8 @@ def _validate_query(query: str, repo: str | None) -> None:
         raise ValueError(f"query must be at most {MAX_QUERY_BYTES} bytes")
     if _INJECTED_FILTER_RE.search(query):
         raise ValueError(
-            "query must not contain count:, repo:, timeout:, context:, rev:, revision:, fork:, archived:, or "
-            "select: filters"
+            "query must not contain count:, repo:, timeout:, context:, rev:, revision:, fork:, archived:, "
+            "select:, or patterntype: filters"
         )
     if repo and re.search(r"\s", repo):
         raise ValueError("repo must not contain whitespace")
@@ -475,7 +476,12 @@ def code_search(
         typer.Argument(
             help=(
                 "Search query (supports Sourcegraph syntax). Defaults to file matches; "
-                "pass your own `type:` filter (e.g. `type:commit`) to override."
+                "pass your own `type:` filter (e.g. `type:commit`) to override. Whitespace is "
+                "not a reliable AND: whether `seedWidget serializeValue` matches both terms or "
+                "one literal phrase depends on the service's pattern-type default, which this "
+                "command does not pin. Join multiple concepts with explicit `AND` instead "
+                "(`seedWidget AND serializeValue`) to get the same result regardless of that "
+                "default."
             ),
         ),
     ],

tests/comfy_cli/command/test_code_search.py

@@ -184,6 +184,7 @@ def test_rejects_injected_filters_and_repo_whitespace(self):
             "fork:yes x",
             "archived:yes x",
             "select:repo x",
+            "patterntype:regexp x",
         ):
             with pytest.raises(ValueError):
                 _validate_query(query, None)
@@ -208,11 +209,13 @@ def test_rejects_grouped_and_aliased_repo_filters(self):
             "foo OR (fork:yes)",
             "foo OR (archived:yes)",
             "foo OR (select:repo)",
+            "foo OR (patterntype:regexp)",
             "foo -context:evil",
             "foo -rev:evil",
             "foo -fork:yes",
             "foo -archived:yes",
             "foo -select:repo",
+            "foo -patterntype:regexp",
         ):
             with pytest.raises(ValueError):
                 _validate_query(query, None)