style(ast-grep): fix lint errors

- Add process import from node:process in constants.ts and downloader.ts
- Convert capturing groups to non-capturing in utils.ts regex
- Add YAML document separators (---) in ast-grep.md examples
- Apply auto-fixes for import ordering and code style

Author: amondnet

agents/ast-grep.md

@@ -110,8 +110,8 @@ For complex queries, use YAML rules with operators.
 id: rule-name
 language: javascript
 rule:
-  # Rule definition here
-message: "Explanation of what was found"
+# Rule definition here
+message: Explanation of what was found
 ```
 
 ### Atomic Rules
@@ -161,7 +161,7 @@ rule:
     has:
       kind: try_statement
       stopBy: end
-
+---
 # Find React useState without dependency
 id: usestate-in-component
 language: tsx
@@ -170,15 +170,15 @@ rule:
   inside:
     kind: function_declaration
     stopBy: end
-
+---
 # Find console.log in production code
 id: no-console-log
 language: javascript
 rule:
   pattern: console.log($$$)
   not:
     inside:
-      regex: "test|spec|__tests__"
+      regex: test|spec|__tests__
       kind: string
 ```
 
@@ -197,7 +197,7 @@ rule:
         kind: string
       nthChild: 1
       stopBy: end
-
+---
 # Find bare except clauses
 id: bare-except
 language: python

packages/dora/src/providers/ast-grep/cli.ts

@@ -4,16 +4,16 @@
  * Executes sg CLI commands and parses results
  */
 
+import type { CliMatch, RunSgOptions, SgResult } from './types'
+import { existsSync } from 'node:fs'
 import { spawn } from 'bun'
-import { existsSync } from 'fs'
 import {
   CLI_LANGUAGES,
   DEFAULT_MAX_MATCHES,
   DEFAULT_MAX_OUTPUT_BYTES,
   DEFAULT_TIMEOUT_MS,
 } from './constants'
 import { ensureAstGrepBinary, getInstallInstructions } from './downloader'
-import type { CliMatch, RunSgOptions, SgResult } from './types'
 
 // Cached binary path
 let resolvedCliPath: string | null = null

packages/dora/src/providers/ast-grep/constants.ts

@@ -2,9 +2,10 @@
  * Constants for ast-grep provider
  */
 
-import * as os from 'os'
-import * as path from 'path'
 import type { PlatformConfig, PlatformId } from './types'
+import * as os from 'node:os'
+import * as path from 'node:path'
+import process from 'node:process'
 
 // CLI supported languages (25 total)
 export const CLI_LANGUAGES = [

packages/dora/src/providers/ast-grep/downloader.ts

@@ -5,17 +5,18 @@
  * Caches to ~/.cache/dora/ast-grep/
  */
 
-import { existsSync, mkdirSync, chmodSync, unlinkSync, writeFileSync, readFileSync } from 'fs'
+import type { PlatformId } from './types'
+import { chmodSync, existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from 'node:fs'
+import process from 'node:process'
 import { spawn } from 'bun'
 import {
   AST_GREP_VERSION,
-  PLATFORM_CONFIGS,
   getAstGrepCacheDir,
   getCachedBinaryPath,
   getPlatformId,
   getVersionMarkerPath,
+  PLATFORM_CONFIGS,
 } from './constants'
-import type { PlatformId } from './types'
 
 /**
  * Verify a binary is actually ast-grep by checking --version output
@@ -95,8 +96,8 @@ async function extractZip(archivePath: string, destDir: string): Promise<void> {
   let proc
   if (process.platform === 'win32') {
     // Escape single quotes for PowerShell by doubling them
-    const escapedArchive = archivePath.replace(/'/g, "''")
-    const escapedDest = destDir.replace(/'/g, "''")
+    const escapedArchive = archivePath.replace(/'/g, '\'\'')
+    const escapedDest = destDir.replace(/'/g, '\'\'')
     proc = spawn(
       [
         'powershell',

packages/dora/src/providers/ast-grep/index.ts

@@ -4,14 +4,14 @@
  * Provides AST-aware code search and transformation tools
  */
 
-import { z, ZodError } from 'zod'
 import type { Provider, ToolDefinition, ToolResult } from '../provider'
 import type { RegistryConfig } from '../registry'
-import { CLI_LANGUAGES, NAPI_LANGUAGES } from './constants'
-import { runSg, isCliAvailable } from './cli'
-import { isNapiAvailable, getNapiError, analyzeCode, transformCode } from './napi'
-import { formatSearchResult, formatReplaceResult, formatAnalyzeResult, formatTransformResult, getEmptyResultHint } from './utils'
 import type { NapiLanguage } from './types'
+import { z, ZodError } from 'zod'
+import { isCliAvailable, runSg } from './cli'
+import { CLI_LANGUAGES, NAPI_LANGUAGES } from './constants'
+import { analyzeCode, getNapiError, isNapiAvailable, transformCode } from './napi'
+import { formatAnalyzeResult, formatReplaceResult, formatSearchResult, formatTransformResult, getEmptyResultHint } from './utils'
 
 /**
  * Format error for tool result, with special handling for Zod validation errors
@@ -157,7 +157,9 @@ export class AstGrepProvider implements Provider {
     const napiAvailable = isNapiAvailable()
     if (!napiAvailable) {
       const error = getNapiError()
+
       console.log(`[ast-grep] NAPI not available: ${error ?? 'unknown'}`)
+
       console.log('[ast-grep] In-memory tools (analyze/transform) disabled')
     }
 

packages/dora/src/providers/ast-grep/napi.ts

@@ -5,12 +5,11 @@
  * Provides faster in-memory analysis and transformation without spawning CLI
  */
 
-import { NAPI_LANGUAGES } from './constants'
 import type { AnalyzeResult, MetaVariable, NapiLanguage, Range } from './types'
+import { NAPI_LANGUAGES } from './constants'
 
-// eslint-disable-next-line ts/no-explicit-any -- Runtime-loaded optional dependency
 type AstGrepNapiModule = any
-// eslint-disable-next-line ts/no-explicit-any -- AST node from NAPI module
+
 type SgNode = any
 
 // Dynamic import for @ast-grep/napi (optional dependency)
@@ -21,8 +20,10 @@ let napiLoadError: Error | null = null
  * Check if NAPI is available
  */
 export function isNapiAvailable(): boolean {
-  if (napiModule !== null) return true
-  if (napiLoadError !== null) return false
+  if (napiModule !== null)
+    return true
+  if (napiLoadError !== null)
+    return false
 
   try {
     // eslint-disable-next-line ts/no-require-imports
@@ -67,20 +68,21 @@ function getLangEnum(lang: NapiLanguage): unknown {
 /**
  * Parse code using NAPI
  */
+// eslint-disable-next-line ts/explicit-function-return-type
 export function parseCode(code: string, lang: NapiLanguage) {
   if (!isNapiAvailable()) {
     throw new Error(
       `@ast-grep/napi not available: ${napiLoadError?.message ?? 'unknown error'}\n`
-        + 'Install with: bun add -D @ast-grep/napi',
+      + 'Install with: bun add -D @ast-grep/napi',
     )
   }
 
   if (!NAPI_LANGUAGES.includes(lang)) {
     const supportedLangs = NAPI_LANGUAGES.join(', ')
     throw new Error(
       `Unsupported language for NAPI: "${lang}"\n`
-        + `Supported languages: ${supportedLangs}\n\n`
-        + `Use ast_grep_search for other languages (25 supported via CLI).`,
+      + `Supported languages: ${supportedLangs}\n\n`
+      + `Use ast_grep_search for other languages (25 supported via CLI).`,
     )
   }
 
@@ -91,6 +93,7 @@ export function parseCode(code: string, lang: NapiLanguage) {
 /**
  * Find pattern matches in parsed tree
  */
+// eslint-disable-next-line ts/explicit-function-return-type
 export function findPattern(root: ReturnType<typeof parseCode>, pattern: string) {
   return root.root().findAll(pattern)
 }
@@ -166,7 +169,7 @@ export function transformCode(
   lang: NapiLanguage,
   pattern: string,
   rewrite: string,
-): { transformed: string; editCount: number } {
+): { transformed: string, editCount: number } {
   const root = parseCode(code, lang)
   const matches = findPattern(root, pattern)
 
@@ -192,7 +195,7 @@ export function transformCode(
 /**
  * Get root node info for debugging
  */
-export function getRootInfo(code: string, lang: NapiLanguage): { kind: string; childCount: number } {
+export function getRootInfo(code: string, lang: NapiLanguage): { kind: string, childCount: number } {
   const root = parseCode(code, lang)
   const rootNode = root.root()
   return {

packages/dora/src/providers/ast-grep/types.ts

@@ -26,13 +26,13 @@ export interface Range {
 export interface CliMatch {
   text: string
   range: {
-    byteOffset: { start: number; end: number }
+    byteOffset: { start: number, end: number }
     start: Position
     end: Position
   }
   file: string
   lines: string
-  charCount: { leading: number; trailing: number }
+  charCount: { leading: number, trailing: number }
   language: string
 }
 
@@ -96,12 +96,12 @@ export interface RunSgOptions {
 }
 
 /** Platform identifier */
-export type PlatformId =
-  | 'win-x64'
-  | 'linux-x64'
-  | 'linux-arm64'
-  | 'osx-x64'
-  | 'osx-arm64'
+export type PlatformId
+  = | 'win-x64'
+    | 'linux-x64'
+    | 'linux-arm64'
+    | 'osx-x64'
+    | 'osx-arm64'
 
 /** Platform-specific binary configuration */
 export interface PlatformConfig {

packages/dora/src/providers/ast-grep/utils.ts

@@ -139,7 +139,7 @@ export function getEmptyResultHint(pattern: string, lang: CliLanguage): string |
   }
 
   if (['javascript', 'typescript', 'tsx'].includes(lang)) {
-    if (/^(export\s+)?(async\s+)?function\s+\$[A-Z_]+\s*$/i.test(src)) {
+    if (/^(?:export\s+)?(?:async\s+)?function\s+\$[A-Z_]+\s*$/i.test(src)) {
       return `Hint: Function patterns need params and body. Try "function $NAME($$$) { $$$ }"`
     }
   }