@aaronshaf/ger 2.0.3 → 2.0.5

package/src/cli/commands/mine.ts

@@ -5,6 +5,7 @@ import { colors } from '@/utils/formatters'
 
 interface MineOptions {
   xml?: boolean
+  json?: boolean
 }
 
 // ANSI color codes
@@ -17,7 +18,24 @@ export const mineCommand = (
 
     const changes = yield* gerritApi.listChanges('owner:self status:open')
 
-    if (options.xml) {
+    if (options.json) {
+      const jsonOutput = {
+        status: 'success',
+        count: changes.length,
+        changes: changes.map((change) => ({
+          number: change._number,
+          subject: change.subject,
+          project: change.project,
+          branch: change.branch,
+          status: change.status,
+          change_id: change.change_id,
+          ...(change.updated ? { updated: change.updated } : {}),
+          ...(change.owner?.name ? { owner: change.owner.name } : {}),
+          ...(change.labels ? { labels: change.labels } : {}),
+        })),
+      }
+      console.log(JSON.stringify(jsonOutput, null, 2))
+    } else if (options.xml) {
       console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
       console.log(`<changes count="${changes.length}">`)
 

package/src/cli/commands/projects.ts

@@ -4,6 +4,7 @@ import { type ApiError, GerritApiService } from '@/api/gerrit'
 interface ProjectsOptions {
   pattern?: string
   xml?: boolean
+  json?: boolean
 }
 
 /**
@@ -27,7 +28,9 @@ export const projectsCommand = (
 
     // Handle empty results
     if (projects.length === 0) {
-      if (options.xml) {
+      if (options.json) {
+        console.log(JSON.stringify({ status: 'success', projects: [] }, null, 2))
+      } else if (options.xml) {
         console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
         console.log(`<projects_result>`)
         console.log(`  <status>success</status>`)
@@ -40,7 +43,24 @@ export const projectsCommand = (
     }
 
     // Output results
-    if (options.xml) {
+    if (options.json) {
+      console.log(
+        JSON.stringify(
+          {
+            status: 'success',
+            count: projects.length,
+            projects: projects.map((project) => ({
+              id: project.id,
+              name: project.name,
+              ...(project.parent ? { parent: project.parent } : {}),
+              ...(project.state ? { state: project.state } : {}),
+            })),
+          },
+          null,
+          2,
+        ),
+      )
+    } else if (options.xml) {
       console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
       console.log(`<projects_result>`)
       console.log(`  <status>success</status>`)

package/src/cli/commands/rebase.ts

@@ -6,6 +6,7 @@ import { escapeXML, sanitizeCDATA } from '@/utils/shell-safety'
 interface RebaseOptions {
   base?: string
   xml?: boolean
+  json?: boolean
 }
 
 /**
@@ -30,7 +31,21 @@ export const rebaseCommand = (
     // Perform the rebase - this returns the rebased change info
     const change = yield* gerritApi.rebaseChange(resolvedChangeId, { base: options.base })
 
-    if (options.xml) {
+    if (options.json) {
+      console.log(
+        JSON.stringify(
+          {
+            status: 'success',
+            change_number: change._number,
+            subject: change.subject,
+            branch: change.branch,
+            ...(options.base ? { base: options.base } : {}),
+          },
+          null,
+          2,
+        ),
+      )
+    } else if (options.xml) {
       console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
       console.log(`<rebase_result>`)
       console.log(`  <status>success</status>`)
@@ -57,7 +72,9 @@ export const rebaseCommand = (
             ? error.message
             : String(error)
 
-        if (options.xml) {
+        if (options.json) {
+          console.log(JSON.stringify({ status: 'error', error: errorMessage }, null, 2))
+        } else if (options.xml) {
           console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
           console.log(`<rebase_result>`)
           console.log(`  <status>error</status>`)

package/src/cli/commands/remove-reviewer.ts

@@ -6,6 +6,7 @@ interface RemoveReviewerOptions {
   change?: string
   notify?: string
   xml?: boolean
+  json?: boolean
 }
 
 type NotifyLevel = 'NONE' | 'OWNER' | 'OWNER_REVIEWERS' | 'ALL'
@@ -23,6 +24,10 @@ const outputXmlError = (message: string): void => {
   console.log(`</remove_reviewer_result>`)
 }
 
+const outputJsonError = (message: string): void => {
+  console.log(JSON.stringify({ status: 'error', error: message }, null, 2))
+}
+
 class ValidationError extends Error {
   readonly _tag = 'ValidationError'
 }
@@ -39,7 +44,9 @@ export const removeReviewerCommand = (
     if (!changeId) {
       const message =
         'Change ID is required. Use -c <change-id> or run from a branch with an active change.'
-      if (options.xml) {
+      if (options.json) {
+        outputJsonError(message)
+      } else if (options.xml) {
         outputXmlError(message)
       } else {
         console.error(`✗ ${message}`)
@@ -49,7 +56,9 @@ export const removeReviewerCommand = (
 
     if (reviewers.length === 0) {
       const message = 'At least one reviewer is required.'
-      if (options.xml) {
+      if (options.json) {
+        outputJsonError(message)
+      } else if (options.xml) {
         outputXmlError(message)
       } else {
         console.error(`✗ ${message}`)
@@ -62,7 +71,9 @@ export const removeReviewerCommand = (
       const upperNotify = options.notify.toUpperCase()
       if (!isValidNotifyLevel(upperNotify)) {
         const message = `Invalid notify level: ${options.notify}. Valid values: none, owner, owner_reviewers, all`
-        if (options.xml) {
+        if (options.json) {
+          outputJsonError(message)
+        } else if (options.xml) {
           outputXmlError(message)
         } else {
           console.error(`✗ ${message}`)
@@ -90,7 +101,24 @@ export const removeReviewerCommand = (
       results.push({ reviewer, success: true })
     }
 
-    if (options.xml) {
+    if (options.json) {
+      const allSuccess = results.every((r) => r.success)
+      console.log(
+        JSON.stringify(
+          {
+            status: allSuccess ? 'success' : 'partial_failure',
+            change_id: changeId,
+            reviewers: results.map((r) =>
+              r.success
+                ? { input: r.reviewer, status: 'removed' }
+                : { input: r.reviewer, error: r.error, status: 'failed' },
+            ),
+          },
+          null,
+          2,
+        ),
+      )
+    } else if (options.xml) {
       console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
       console.log(`<remove_reviewer_result>`)
       console.log(`  <change_id>${escapeXML(changeId)}</change_id>`)

package/src/cli/commands/restore.ts

@@ -4,6 +4,7 @@ import { type ApiError, GerritApiService } from '@/api/gerrit'
 interface RestoreOptions {
   message?: string
   xml?: boolean
+  json?: boolean
 }
 
 /**
@@ -31,7 +32,20 @@ export const restoreCommand = (
     // Perform the restore - this returns the restored change info
     const change = yield* gerritApi.restoreChange(changeId, options.message)
 
-    if (options.xml) {
+    if (options.json) {
+      console.log(
+        JSON.stringify(
+          {
+            status: 'success',
+            change_number: change._number,
+            subject: change.subject,
+            ...(options.message ? { message: options.message } : {}),
+          },
+          null,
+          2,
+        ),
+      )
+    } else if (options.xml) {
       console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
       console.log(`<restore_result>`)
       console.log(`  <status>success</status>`)

package/src/cli/commands/search.ts

@@ -46,6 +46,7 @@ Full query syntax: https://gerrit-review.googlesource.com/Documentation/user-sea
 
 interface SearchOptions {
   xml?: boolean
+  json?: boolean
   limit?: string
 }
 
@@ -93,7 +94,27 @@ export const searchCommand = (
     // Group changes by project (used by both output formats)
     const groupedChanges = changes.length > 0 ? groupChangesByProject(changes) : []
 
-    if (options.xml) {
+    if (options.json) {
+      // JSON output
+      const jsonOutput = {
+        status: 'success',
+        query: finalQuery,
+        count: changes.length,
+        changes: groupedChanges.flatMap(({ project, changes: projectChanges }) =>
+          projectChanges.map((change) => ({
+            number: change._number,
+            subject: change.subject,
+            status: change.status,
+            project,
+            branch: change.branch,
+            owner: change.owner?.name ?? 'Unknown',
+            ...(change.owner?.email ? { owner_email: change.owner.email } : {}),
+            ...(change.updated ? { updated: change.updated } : {}),
+          })),
+        ),
+      }
+      console.log(JSON.stringify(jsonOutput, null, 2))
+    } else if (options.xml) {
       // XML output
       const xmlOutput = [
         '<?xml version="1.0" encoding="UTF-8"?>',

package/src/cli/commands/status.ts

@@ -3,6 +3,7 @@ import { GerritApiService } from '@/api/gerrit'
 
 interface StatusOptions {
   xml?: boolean
+  json?: boolean
 }
 
 export const statusCommand = (
@@ -13,7 +14,16 @@ export const statusCommand = (
 
     const isConnected = yield* apiService.testConnection
 
-    if (options.xml) {
+    if (options.json) {
+      // JSON output
+      console.log(
+        JSON.stringify(
+          { status: isConnected ? 'success' : 'error', connected: isConnected },
+          null,
+          2,
+        ),
+      )
+    } else if (options.xml) {
       // XML output for LLM consumption
       console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
       console.log(`<status_result>`)

package/src/cli/commands/submit.ts

@@ -3,6 +3,7 @@ import { type ApiError, GerritApiService } from '@/api/gerrit'
 
 interface SubmitOptions {
   xml?: boolean
+  json?: boolean
 }
 
 /**
@@ -65,7 +66,21 @@ export const submitCommand = (
         reasons.push('Change does not meet submit requirements')
       }
 
-      if (options.xml) {
+      if (options.json) {
+        console.log(
+          JSON.stringify(
+            {
+              status: 'error',
+              change_number: change._number,
+              subject: change.subject,
+              submittable: false,
+              reasons,
+            },
+            null,
+            2,
+          ),
+        )
+      } else if (options.xml) {
         console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
         console.log(`<submit_result>`)
         console.log(`  <status>error</status>`)
@@ -93,7 +108,20 @@ export const submitCommand = (
     // Change is submittable, proceed with submission
     const result = yield* gerritApi.submitChange(changeId)
 
-    if (options.xml) {
+    if (options.json) {
+      console.log(
+        JSON.stringify(
+          {
+            status: 'success',
+            change_number: change._number,
+            subject: change.subject,
+            submit_status: result.status,
+          },
+          null,
+          2,
+        ),
+      )
+    } else if (options.xml) {
       console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
       console.log(`<submit_result>`)
       console.log(`  <status>success</status>`)

package/src/cli/commands/topic.ts

@@ -22,6 +22,7 @@ Note: When no change-id is provided, it will be auto-detected from the HEAD comm
 
 interface TopicOptions {
   xml?: boolean
+  json?: boolean
   delete?: boolean
 }
 
@@ -52,7 +53,15 @@ export const topicCommand = (
     if (options.delete) {
       yield* gerritApi.deleteTopic(resolvedChangeId)
 
-      if (options.xml) {
+      if (options.json) {
+        console.log(
+          JSON.stringify(
+            { status: 'success', action: 'deleted', change_id: resolvedChangeId },
+            null,
+            2,
+          ),
+        )
+      } else if (options.xml) {
         console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
         console.log(`<topic_result>`)
         console.log(`  <status>success</status>`)
@@ -69,7 +78,15 @@ export const topicCommand = (
     if (topic !== undefined && topic.trim() !== '') {
       const newTopic = yield* gerritApi.setTopic(resolvedChangeId, topic)
 
-      if (options.xml) {
+      if (options.json) {
+        console.log(
+          JSON.stringify(
+            { status: 'success', action: 'set', change_id: resolvedChangeId, topic: newTopic },
+            null,
+            2,
+          ),
+        )
+      } else if (options.xml) {
         console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
         console.log(`<topic_result>`)
         console.log(`  <status>success</status>`)
@@ -86,7 +103,20 @@ export const topicCommand = (
     // Handle get operation (default)
     const currentTopic = yield* gerritApi.getTopic(resolvedChangeId)
 
-    if (options.xml) {
+    if (options.json) {
+      console.log(
+        JSON.stringify(
+          {
+            status: 'success',
+            action: 'get',
+            change_id: resolvedChangeId,
+            topic: currentTopic || null,
+          },
+          null,
+          2,
+        ),
+      )
+    } else if (options.xml) {
       console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
       console.log(`<topic_result>`)
       console.log(`  <status>success</status>`)

package/src/cli/commands/vote.ts

@@ -8,6 +8,7 @@ interface VoteOptions {
   label?: string[]
   message?: string
   xml?: boolean
+  json?: boolean
 }
 
 /**
@@ -92,7 +93,20 @@ export const voteCommand = (
     yield* gerritApi.postReview(changeId, reviewInput)
 
     // Output success
-    if (options.xml) {
+    if (options.json) {
+      console.log(
+        JSON.stringify(
+          {
+            status: 'success',
+            change_id: changeId,
+            labels,
+            ...(options.message ? { message: options.message } : {}),
+          },
+          null,
+          2,
+        ),
+      )
+    } else if (options.xml) {
       console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
       console.log(`<vote_result>`)
       console.log(`  <status>success</status>`)

package/src/cli/commands/workspace.ts

@@ -7,6 +7,7 @@ import { type ConfigError, ConfigService } from '@/services/config'
 
 interface WorkspaceOptions {
   xml?: boolean
+  json?: boolean
 }
 
 const parseChangeSpec = (changeSpec: string): { changeId: string; patchset?: string } => {
@@ -129,7 +130,11 @@ export const workspaceCommand = (
 
     // Check if worktree already exists
     if (fs.existsSync(workspaceDir)) {
-      if (options.xml) {
+      if (options.json) {
+        console.log(
+          JSON.stringify({ status: 'success', path: workspaceDir, exists: true }, null, 2),
+        )
+      } else if (options.xml) {
         console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
         console.log(`<workspace>`)
         console.log(`  <path>${workspaceDir}</path>`)
@@ -150,7 +155,7 @@ export const workspaceCommand = (
 
     // Fetch the change ref
     const changeRef = revision.ref
-    if (!options.xml) {
+    if (!options.xml && !options.json) {
       console.log(`Fetching change ${change._number}: ${change.subject}`)
     }
 
@@ -168,7 +173,7 @@ export const workspaceCommand = (
     }
 
     // Create worktree
-    if (!options.xml) {
+    if (!options.xml && !options.json) {
       console.log(`Creating worktree at: ${workspaceDir}`)
     }
 
@@ -185,7 +190,21 @@ export const workspaceCommand = (
       throw new Error(`Failed to create worktree: ${error}`)
     }
 
-    if (options.xml) {
+    if (options.json) {
+      console.log(
+        JSON.stringify(
+          {
+            status: 'success',
+            path: workspaceDir,
+            change_number: change._number,
+            subject: change.subject,
+            created: true,
+          },
+          null,
+          2,
+        ),
+      )
+    } else if (options.xml) {
       console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
       console.log(`<workspace>`)
       console.log(`  <path>${workspaceDir}</path>`)