@aaronshaf/ger 0.1.0

package/src/cli/commands/incoming.ts

@@ -0,0 +1,226 @@
+import { Effect } from 'effect'
+import { select } from '@inquirer/prompts'
+import { exec } from 'node:child_process'
+import { promisify } from 'node:util'
+import { type ApiError, GerritApiService } from '@/api/gerrit'
+import { type ConfigError, ConfigService } from '@/services/config'
+import { colors } from '@/utils/formatters'
+import { getStatusIndicators } from '@/utils/status-indicators'
+import type { ChangeInfo } from '@/schemas/gerrit'
+import { sanitizeUrlSync, getOpenCommand } from '@/utils/shell-safety'
+
+const execAsync = promisify(exec)
+
+interface IncomingOptions {
+  xml?: boolean
+  interactive?: boolean
+}
+
+// Group changes by project for better organization
+const groupChangesByProject = (changes: readonly ChangeInfo[]) => {
+  const grouped = new Map<string, ChangeInfo[]>()
+
+  for (const change of changes) {
+    const project = change.project
+    if (!grouped.has(project)) {
+      grouped.set(project, [])
+    }
+    grouped.get(project)!.push(change)
+  }
+
+  // Sort projects alphabetically and changes by updated date
+  return Array.from(grouped.entries())
+    .sort(([a], [b]) => a.localeCompare(b))
+    .map(([project, projectChanges]) => ({
+      project,
+      changes: projectChanges.sort((a, b) => {
+        const dateA = a.updated ? new Date(a.updated).getTime() : 0
+        const dateB = b.updated ? new Date(b.updated).getTime() : 0
+        return dateB - dateA
+      }),
+    }))
+}
+
+// Format change for display in inquirer
+const formatChangeChoice = (change: ChangeInfo) => {
+  const indicators = getStatusIndicators(change)
+  const statusPart = indicators ? `${indicators} ` : ''
+  const subject =
+    change.subject.length > 60 ? `${change.subject.substring(0, 57)}...` : change.subject
+
+  return {
+    name: `${statusPart}${subject} (${change._number})`,
+    value: change,
+    description: `By ${change.owner?.name || 'Unknown'} • ${change.status}`,
+  }
+}
+
+// Open change in browser
+const openInBrowser = async (gerritHost: string, changeNumber: number) => {
+  const url = `${gerritHost}/c/${changeNumber}`
+  const sanitizedUrl = sanitizeUrlSync(url)
+
+  if (!sanitizedUrl) {
+    console.error(`${colors.red}✗ Invalid URL: ${url}${colors.reset}`)
+    return
+  }
+
+  const openCmd = getOpenCommand()
+  try {
+    await execAsync(`${openCmd} "${sanitizedUrl}"`)
+    console.log(`${colors.green}✓ Opened ${changeNumber} in browser${colors.reset}`)
+  } catch (error) {
+    console.error(`${colors.red}✗ Failed to open browser: ${error}${colors.reset}`)
+  }
+}
+
+export const incomingCommand = (
+  options: IncomingOptions,
+): Effect.Effect<void, ApiError | ConfigError, GerritApiService | ConfigService> =>
+  Effect.gen(function* () {
+    const gerritApi = yield* GerritApiService
+
+    // Query for changes where user is a reviewer but not the owner
+    const changes = yield* gerritApi.listChanges(
+      'is:open -owner:self -is:wip -is:ignored reviewer:self',
+    )
+
+    if (options.interactive) {
+      if (changes.length === 0) {
+        console.log(`${colors.yellow}No incoming reviews found${colors.reset}`)
+        return
+      }
+
+      // Get Gerrit host for opening changes in browser
+      const configService = yield* ConfigService
+      const credentials = yield* configService.getCredentials
+
+      // Group changes by project
+      const groupedChanges = groupChangesByProject(changes)
+
+      // Create choices for inquirer with project sections
+      const choices: Array<{ name: string; value: ChangeInfo | string }> = []
+
+      for (const { project, changes: projectChanges } of groupedChanges) {
+        // Add project header as separator
+        choices.push({
+          name: `\n${colors.blue}━━━ ${project} ━━━${colors.reset}`,
+          value: 'separator',
+        })
+
+        // Add changes for this project
+        for (const change of projectChanges) {
+          const formatted = formatChangeChoice(change)
+          choices.push({
+            name: formatted.name,
+            value: change,
+          })
+        }
+      }
+
+      // Add exit option
+      choices.push({
+        name: `\n${colors.gray}Exit${colors.reset}`,
+        value: 'exit',
+      })
+
+      // Interactive selection loop
+      let continueSelecting = true
+      while (continueSelecting) {
+        const selected = yield* Effect.promise(async () => {
+          return await select({
+            message: 'Select a change to open in browser:',
+            choices: choices.filter((c) => c.value !== 'separator'),
+            pageSize: 15,
+          })
+        })
+
+        if (selected === 'exit' || !selected) {
+          continueSelecting = false
+        } else if (typeof selected !== 'string') {
+          // Open the selected change
+          yield* Effect.promise(() => openInBrowser(credentials.host, selected._number))
+
+          // Ask if user wants to continue
+          const continueChoice = yield* Effect.promise(async () => {
+            return await select({
+              message: 'Continue?',
+              choices: [
+                { name: 'Select another change', value: 'continue' },
+                { name: 'Exit', value: 'exit' },
+              ],
+            })
+          })
+
+          if (continueChoice === 'exit') {
+            continueSelecting = false
+          }
+        }
+      }
+
+      return
+    }
+
+    if (options.xml) {
+      // XML output
+      const xmlOutput = [
+        '<?xml version="1.0" encoding="UTF-8"?>',
+        '<incoming_reviews>',
+        `  <count>${changes.length}</count>`,
+      ]
+
+      if (changes.length > 0) {
+        xmlOutput.push('  <changes>')
+
+        // Group by project for XML output too
+        const groupedChanges = groupChangesByProject(changes)
+
+        for (const { project, changes: projectChanges } of groupedChanges) {
+          xmlOutput.push(`    <project name="${project}">`)
+          for (const change of projectChanges) {
+            xmlOutput.push('      <change>')
+            xmlOutput.push(`        <number>${change._number}</number>`)
+            xmlOutput.push(`        <subject><![CDATA[${change.subject}]]></subject>`)
+            xmlOutput.push(`        <status>${change.status}</status>`)
+            xmlOutput.push(`        <owner>${change.owner?.name || 'Unknown'}</owner>`)
+            xmlOutput.push(`        <updated>${change.updated}</updated>`)
+            xmlOutput.push('      </change>')
+          }
+          xmlOutput.push('    </project>')
+        }
+
+        xmlOutput.push('  </changes>')
+      }
+
+      xmlOutput.push('</incoming_reviews>')
+      console.log(xmlOutput.join('\n'))
+    } else {
+      // Pretty output (default)
+      if (changes.length === 0) {
+        console.log(`${colors.green}✓ No incoming reviews${colors.reset}`)
+        return
+      }
+
+      console.log(`${colors.blue}Incoming Reviews (${changes.length})${colors.reset}\n`)
+
+      // Group by project for display
+      const groupedChanges = groupChangesByProject(changes)
+
+      for (const { project, changes: projectChanges } of groupedChanges) {
+        console.log(`${colors.gray}${project}${colors.reset}`)
+
+        for (const change of projectChanges) {
+          const indicators = getStatusIndicators(change)
+          const statusPart = indicators ? `${indicators} ` : ''
+
+          console.log(
+            `  ${statusPart}${colors.yellow}#${change._number}${colors.reset} ${change.subject}`,
+          )
+          console.log(
+            `    ${colors.gray}by ${change.owner?.name || 'Unknown'} • ${change.status}${colors.reset}`,
+          )
+        }
+        console.log() // Empty line between projects
+      }
+    }
+  })

package/src/cli/commands/init.ts

@@ -0,0 +1,164 @@
+import * as fs from 'node:fs'
+import * as os from 'node:os'
+import * as path from 'node:path'
+import * as readline from 'node:readline/promises'
+import { Effect } from 'effect'
+import type { GerritCredentials } from '@/schemas/gerrit'
+import { ConfigService } from '@/services/config'
+
+const CONFIG_FILE = path.join(os.homedir(), '.ger', 'auth.json')
+
+const obscureToken = (token: string): string => {
+  if (token.length <= 8) return '****'
+  return `${token.substring(0, 4)}****${token.substring(token.length - 4)}`
+}
+
+// Hidden password input using manual stdin manipulation
+const readPassword = async (prompt: string, fallbackPrompt?: string): Promise<string> => {
+  const stdin = process.stdin
+  const stdout = process.stdout
+
+  // Check if we can use raw mode (TTY environment)
+  if (!stdin.isTTY || !stdin.setRawMode) {
+    // Fallback to regular readline with warning
+    if (fallbackPrompt) {
+      console.log('⚠️  Note: Password will be visible while typing (non-TTY environment)')
+    }
+    const rl = readline.createInterface({
+      input: stdin,
+      output: stdout,
+    })
+    const answer = await rl.question(fallbackPrompt || prompt)
+    rl.close()
+    return answer
+  }
+
+  return new Promise((resolve) => {
+    stdout.write(prompt)
+
+    stdin.setRawMode(true)
+    stdin.resume()
+    stdin.setEncoding('utf8')
+
+    let password = ''
+
+    const onData = (char: string) => {
+      const code = char.charCodeAt(0)
+
+      if (code === 3) {
+        // Ctrl+C
+        stdout.write('\n')
+        process.exit(0)
+      } else if (code === 13 || code === 10) {
+        // Enter
+        stdin.setRawMode(false)
+        stdin.pause()
+        stdin.removeListener('data', onData)
+        stdout.write('\n')
+        resolve(password)
+      } else if (code === 127 || code === 8) {
+        // Backspace
+        if (password.length > 0) {
+          password = password.slice(0, -1)
+          stdout.write('\b \b') // Move back, write space, move back again
+        }
+      } else if (code >= 32 && code <= 126) {
+        // Printable characters
+        password += char
+        stdout.write('*')
+      }
+    }
+
+    stdin.on('data', onData)
+  })
+}
+
+const readExistingConfig = (): GerritCredentials | null => {
+  try {
+    if (fs.existsSync(CONFIG_FILE)) {
+      const content = fs.readFileSync(CONFIG_FILE, 'utf8')
+      return JSON.parse(content)
+    }
+  } catch {
+    // Ignore errors
+  }
+  return null
+}
+
+export const initCommand = (): Effect.Effect<void, Error, ConfigService> =>
+  Effect.gen(function* () {
+    const configService = yield* ConfigService
+
+    const rl = readline.createInterface({
+      input: process.stdin,
+      output: process.stdout,
+    })
+
+    console.log('Gerrit CLI Setup')
+    console.log('================')
+    console.log('')
+
+    // Load existing config if it exists
+    const existing = readExistingConfig()
+
+    if (existing) {
+      console.log('Found existing configuration:')
+      console.log(`  Host: ${existing.host}`)
+      console.log(`  Username: ${existing.username}`)
+      console.log(`  Password: ${obscureToken(existing.password)}`)
+      console.log('')
+      console.log('Press Enter to keep existing values, or type new ones.')
+      console.log('')
+    }
+
+    // Prompt for host
+    const hostPrompt = existing?.host
+      ? `Gerrit host [${existing.host}]: `
+      : 'Gerrit host (e.g., https://gerrit.example.com): '
+    const host = yield* Effect.tryPromise(() => rl.question(hostPrompt)).pipe(
+      Effect.map((answer) => answer || existing?.host),
+      Effect.flatMap((value) =>
+        value ? Effect.succeed(value) : Effect.fail(new Error('Host is required')),
+      ),
+    )
+
+    // Prompt for username
+    const usernamePrompt = existing?.username ? `Username [${existing.username}]: ` : 'Username: '
+    const username = yield* Effect.tryPromise(() => rl.question(usernamePrompt)).pipe(
+      Effect.map((answer) => answer || existing?.username),
+      Effect.flatMap((value) =>
+        value ? Effect.succeed(value) : Effect.fail(new Error('Username is required')),
+      ),
+    )
+
+    // Close readline interface before password prompt (we'll use raw mode)
+    rl.close()
+
+    // Prompt for password (with hidden input)
+    const passwordPrompt = existing?.password
+      ? `HTTP Password [${obscureToken(existing.password)}]: `
+      : 'HTTP Password (from Gerrit Settings → HTTP Password): '
+
+    const password = yield* Effect.tryPromise(() => readPassword(passwordPrompt)).pipe(
+      Effect.map((answer) => answer || existing?.password),
+      Effect.flatMap((value) =>
+        value ? Effect.succeed(value) : Effect.fail(new Error('Password is required')),
+      ),
+    )
+
+    const credentials: GerritCredentials = {
+      host: host.replace(/\/$/, ''), // Remove trailing slash if present
+      username,
+      password,
+    }
+
+    yield* configService.saveCredentials(credentials)
+
+    console.log('')
+    console.log('✓ Credentials saved to ~/.ger/auth.json')
+    console.log('')
+    console.log('You can now use commands like:')
+    console.log('  ger status')
+    console.log('  ger mine --pretty')
+    console.log('  ger workspace <change-id>')
+  })

package/src/cli/commands/mine.ts

@@ -0,0 +1,115 @@
+import { Effect } from 'effect'
+import { type ApiError, GerritApiService } from '@/api/gerrit'
+import type { ChangeInfo } from '@/schemas/gerrit'
+import { colors } from '@/utils/formatters'
+
+interface MineOptions {
+  xml?: boolean
+}
+
+// ANSI color codes
+
+export const mineCommand = (
+  options: MineOptions,
+): Effect.Effect<void, ApiError, GerritApiService> =>
+  Effect.gen(function* () {
+    const gerritApi = yield* GerritApiService
+
+    const changes = yield* gerritApi.listChanges('owner:self status:open')
+
+    if (options.xml) {
+      console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
+      console.log(`<changes count="${changes.length}">`)
+
+      for (const change of changes) {
+        console.log(`  <change>`)
+        console.log(`    <number>${change._number}</number>`)
+        console.log(`    <subject><![CDATA[${change.subject}]]></subject>`)
+        console.log(`    <project>${change.project}</project>`)
+        console.log(`    <branch>${change.branch}</branch>`)
+        console.log(`    <status>${change.status}</status>`)
+        console.log(`    <change_id>${change.change_id}</change_id>`)
+        if (change.updated) {
+          console.log(`    <updated>${change.updated}</updated>`)
+        }
+        if (change.owner?.name) {
+          console.log(`    <owner>${change.owner.name}</owner>`)
+        }
+        console.log(`  </change>`)
+      }
+
+      console.log(`</changes>`)
+    } else {
+      // Pretty output by default
+      if (changes.length === 0) {
+        return
+      }
+
+      // Group changes by project
+      const changesByProject = changes.reduce(
+        (acc, change) => {
+          if (!acc[change.project]) {
+            acc[change.project] = []
+          }
+          acc[change.project] = [...acc[change.project], change]
+          return acc
+        },
+        {} as Record<string, ChangeInfo[]>,
+      )
+
+      // Sort projects alphabetically
+      const sortedProjects = Object.keys(changesByProject).sort()
+
+      for (const [index, project] of sortedProjects.entries()) {
+        if (index > 0) {
+          console.log('') // Add blank line between projects
+        }
+        console.log(`${colors.blue}${project}${colors.reset}`)
+
+        const projectChanges = changesByProject[project]
+        for (const change of projectChanges) {
+          // Build status indicators
+          const indicators: string[] = []
+          const indicatorChars: string[] = [] // Track visual characters for padding
+
+          if (change.labels?.['Code-Review']) {
+            const cr = change.labels['Code-Review']
+            if (cr.approved || cr.value === 2) {
+              indicators.push(`${colors.green}✓${colors.reset}`)
+              indicatorChars.push('✓')
+            } else if (cr.rejected || cr.value === -2) {
+              indicators.push(`${colors.red}✗${colors.reset}`)
+              indicatorChars.push('✗')
+            } else if (cr.recommended || cr.value === 1) {
+              indicators.push(`${colors.cyan}↑${colors.reset}`)
+              indicatorChars.push('↑')
+            } else if (cr.disliked || cr.value === -1) {
+              indicators.push(`${colors.yellow}↓${colors.reset}`)
+              indicatorChars.push('↓')
+            }
+          }
+
+          // Check for Verified label as well
+          if (change.labels?.['Verified']) {
+            const v = change.labels.Verified
+            if (v.approved || v.value === 1) {
+              if (!indicatorChars.includes('✓')) {
+                indicators.push(`${colors.green}✓${colors.reset}`)
+                indicatorChars.push('✓')
+              }
+            } else if (v.rejected || v.value === -1) {
+              indicators.push(`${colors.red}✗${colors.reset}`)
+              indicatorChars.push('✗')
+            }
+          }
+
+          // Calculate padding based on visual characters, not color codes
+          const visualWidth = indicatorChars.join('  ').length
+          const padding = ' '.repeat(Math.max(0, 8 - visualWidth))
+          const statusStr = indicators.length > 0 ? indicators.join('  ') + padding : '        '
+
+          console.log(`${statusStr} ${change._number}  ${change.subject}`)
+        }
+      }
+    }
+  })

package/src/cli/commands/open.ts

@@ -0,0 +1,57 @@
+import { Effect } from 'effect'
+import { type ApiError, GerritApiService } from '@/api/gerrit'
+import { type ConfigError, ConfigService } from '@/services/config'
+import { extractChangeNumber, isValidChangeId } from '@/utils/url-parser'
+import { sanitizeUrl, getOpenCommand } from '@/utils/shell-safety'
+import { exec } from 'node:child_process'
+
+interface OpenOptions {
+  // No options for now, but keeping the structure for future extensibility
+}
+
+export const openCommand = (
+  changeId: string,
+  _options: OpenOptions = {},
+): Effect.Effect<void, ApiError | ConfigError | Error, GerritApiService | ConfigService> =>
+  Effect.gen(function* () {
+    const gerritApi = yield* GerritApiService
+    const configService = yield* ConfigService
+
+    // Extract change number if a URL was provided
+    const cleanChangeId = extractChangeNumber(changeId)
+
+    // Validate the change ID
+    if (!isValidChangeId(cleanChangeId)) {
+      yield* Effect.fail(new Error(`Invalid change ID: ${cleanChangeId}`))
+    }
+
+    // Fetch change details to get the project name
+    const change = yield* gerritApi.getChange(cleanChangeId)
+
+    // Get the Gerrit host from config
+    const credentials = yield* configService.getCredentials
+    const gerritHost = credentials.host
+
+    const changeUrl = `${gerritHost}/c/${change.project}/+/${change._number}`
+
+    // Sanitize URL for shell safety
+    const safeUrl = yield* sanitizeUrl(changeUrl)
+
+    // Open the URL in the default browser
+    const openCmd = getOpenCommand()
+
+    yield* Effect.promise(
+      () =>
+        new Promise<void>((resolve, reject) => {
+          exec(`${openCmd} "${safeUrl}"`, (error) => {
+            if (error) {
+              reject(new Error(`Failed to open URL: ${error.message}`))
+            } else {
+              resolve()
+            }
+          })
+        }),
+    )
+
+    console.log(`Opened: ${changeUrl}`)
+  })