refactor: use searchQuery directly

Author: HerringtonDarkholme

src/extension/preview.ts

@@ -247,11 +247,7 @@ async function refreshSearchResult(
   fileUri: Uri,
   query: SearchQuery,
 ) {
-  const command = buildCommand({
-    pattern: query.pattern,
-    rewrite: query.rewrite,
-    includeFiles: [query.includeFile],
-  })
+  const command = buildCommand(query)
   const bytes = await workspace.fs.readFile(fileUri)
   const { receiveResult, conclude } = bufferMaker(bytes)
   const updatedResults: DisplayResult[] = []

src/extension/search.ts

@@ -106,15 +106,9 @@ async function uniqueCommand(
   }
 }
 
-interface CommandArgs {
-  pattern: string
-  rewrite?: string
-  includeFiles: string[]
-}
-
 // TODO: add unit test for commandBuilder
-export function buildCommand(query: CommandArgs) {
-  const { pattern, includeFiles, rewrite } = query
+export function buildCommand(query: SearchQuery) {
+  const { pattern, includeFile, rewrite } = query
   if (!pattern) {
     return
   }
@@ -124,7 +118,7 @@ export function buildCommand(query: CommandArgs) {
   if (rewrite) {
     args.push('--rewrite', rewrite)
   }
-  args.push(...includeFiles.filter(Boolean))
+  args.push(...includeFile.split(',').filter(Boolean))
   console.debug('running', query, command, args)
   // TODO: multi-workspaces support
   return spawn(command, args, {
@@ -139,11 +133,7 @@ interface Handlers {
 }
 
 function getPatternRes(query: SearchQuery, handlers: Handlers) {
-  const proc = buildCommand({
-    pattern: query.pattern,
-    includeFiles: [query.includeFile],
-    rewrite: query.rewrite,
-  })
+  const proc = buildCommand(query)
   if (proc) {
     proc.on('error', error => {
       console.debug('ast-grep CLI runs error')

src/webview/SearchSidebar/SearchWidgetContainer/PatternConfig.tsx

@@ -17,8 +17,8 @@ const NOOP = () => {}
 export default function PatternConfig() {
   const [strictness, setStrictness] = usePatternConfig('strictness', 'smart')
   const [selector, setSelector] = usePatternConfig('selector', '')
-  // biome-ignore lint/suspicious/noExplicitAny: onChange event has wrong type signature.
   const onStrictnessChange = useCallback(
+    // biome-ignore lint/suspicious/noExplicitAny: onChange event has wrong type signature.
     (e: any) => {
       const select = e.target as HTMLSelectElement
       setStrictness(select.value)