@aaronshaf/ger 2.0.1 → 2.0.3

package/src/api/gerrit.ts

@@ -90,26 +90,25 @@ export interface GerritApiServiceImpl {
     accountId: string,
     options?: { notify?: 'NONE' | 'OWNER' | 'OWNER_REVIEWERS' | 'ALL' },
   ) => Effect.Effect<void, ApiError>
+  readonly getTopic: (changeId: string) => Effect.Effect<string | null, ApiError>
+  readonly setTopic: (changeId: string, topic: string) => Effect.Effect<string, ApiError>
+  readonly deleteTopic: (changeId: string) => Effect.Effect<void, ApiError>
 }
 
-// Export both the tag value and the type for use in Effect requirements
 export const GerritApiService: Context.Tag<GerritApiServiceImpl, GerritApiServiceImpl> =
   Context.GenericTag<GerritApiServiceImpl>('GerritApiService')
 export type GerritApiService = Context.Tag.Identifier<typeof GerritApiService>
 
-// Export ApiError fields interface explicitly
 export interface ApiErrorFields {
   readonly message: string
   readonly status?: number
 }
 
-// Define error schema (not exported, so type can be implicit)
 const ApiErrorSchema = Schema.TaggedError<ApiErrorFields>()('ApiError', {
   message: Schema.String,
   status: Schema.optional(Schema.Number),
 } as const) as unknown
 
-// Export the error class with explicit constructor signature for isolatedDeclarations
 export class ApiError
   extends (ApiErrorSchema as new (
     args: ApiErrorFields,
@@ -127,7 +126,7 @@ const createAuthHeader = (credentials: GerritCredentials): string => {
 const makeRequest = <T = unknown>(
   url: string,
   authHeader: string,
-  method: 'GET' | 'POST' = 'GET',
+  method: 'GET' | 'POST' | 'PUT' | 'DELETE' = 'GET',
   body?: unknown,
   schema?: Schema.Schema<T>,
 ): Effect.Effect<T, ApiError> =>
@@ -155,12 +154,7 @@ const makeRequest = <T = unknown>(
         try: () => response.text(),
         catch: () => 'Unknown error',
       }).pipe(Effect.orElseSucceed(() => 'Unknown error'))
-      yield* Effect.fail(
-        new ApiError({
-          message: errorText,
-          status: response.status,
-        }),
-      )
+      yield* Effect.fail(new ApiError({ message: errorText, status: response.status }))
     }
 
     const text = yield* Effect.tryPromise({
@@ -168,13 +162,8 @@ const makeRequest = <T = unknown>(
       catch: () => new ApiError({ message: 'Failed to read response data' }),
     })
 
-    // Gerrit returns JSON with )]}' prefix for security
     const cleanJson = text.replace(/^\)\]\}'\n?/, '')
-
-    if (!cleanJson.trim()) {
-      // Empty response - return empty object for endpoints that expect void
-      return {} as T
-    }
+    if (!cleanJson.trim()) return {} as T
 
     const parsed = yield* Effect.try({
       try: () => JSON.parse(cleanJson),
@@ -186,8 +175,6 @@ const makeRequest = <T = unknown>(
         Effect.mapError(() => new ApiError({ message: 'Invalid response format from server' })),
       )
     }
-
-    // When no schema is provided, the caller expects void or doesn't care about the response
     return parsed
   })
 
@@ -201,16 +188,13 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
         const credentials = yield* configService.getCredentials.pipe(
           Effect.mapError(() => new ApiError({ message: 'Failed to get credentials' })),
         )
-        // Ensure host doesn't have trailing slash
-        const normalizedCredentials = {
-          ...credentials,
-          host: credentials.host.replace(/\/$/, ''),
+        const normalizedCredentials = { ...credentials, host: credentials.host.replace(/\/$/, '') }
+        return {
+          credentials: normalizedCredentials,
+          authHeader: createAuthHeader(normalizedCredentials),
         }
-        const authHeader = createAuthHeader(normalizedCredentials)
-        return { credentials: normalizedCredentials, authHeader }
       })
 
-      // Helper to normalize and validate change identifier
       const normalizeAndValidate = (changeId: string): Effect.Effect<string, ApiError> =>
         Effect.try({
           try: () => normalizeChangeIdentifier(changeId),
@@ -231,9 +215,7 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
       const listChanges = (query = 'is:open') =>
         Effect.gen(function* () {
           const { credentials, authHeader } = yield* getCredentialsAndAuth
-          const encodedQuery = encodeURIComponent(query)
-          // Add additional options to get detailed information
-          const url = `${credentials.host}/a/changes/?q=${encodedQuery}&o=LABELS&o=DETAILED_LABELS&o=DETAILED_ACCOUNTS&o=SUBMITTABLE`
+          const url = `${credentials.host}/a/changes/?q=${encodeURIComponent(query)}&o=LABELS&o=DETAILED_LABELS&o=DETAILED_ACCOUNTS&o=SUBMITTABLE`
           return yield* makeRequest(url, authHeader, 'GET', undefined, Schema.Array(ChangeInfo))
         })
 
@@ -241,10 +223,7 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
         Effect.gen(function* () {
           const { credentials, authHeader } = yield* getCredentialsAndAuth
           let url = `${credentials.host}/a/projects/`
-          if (options?.pattern) {
-            url += `?p=${encodeURIComponent(options.pattern)}`
-          }
-          // Gerrit returns projects as a Record, need to convert to array
+          if (options?.pattern) url += `?p=${encodeURIComponent(options.pattern)}`
           const projectsRecord = yield* makeRequest(
             url,
             authHeader,
@@ -252,7 +231,6 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
             undefined,
             Schema.Record({ key: Schema.String, value: ProjectInfo }),
           )
-          // Convert Record to Array and sort alphabetically by name
           return Object.values(projectsRecord).sort((a, b) => a.name.localeCompare(b.name))
         })
 
@@ -306,7 +284,6 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
         return true
       }).pipe(
         Effect.catchAll((error) => {
-          // Log the actual error for debugging
           if (process.env.DEBUG) {
             console.error('Connection error:', error)
           }
@@ -373,15 +350,8 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
               try: () => response.text(),
               catch: () => 'Unknown error',
             }).pipe(Effect.orElseSucceed(() => 'Unknown error'))
-
-            yield* Effect.fail(
-              new ApiError({
-                message: errorText,
-                status: response.status,
-              }),
-            )
+            yield* Effect.fail(new ApiError({ message: errorText, status: response.status }))
           }
-
           const base64Content = yield* Effect.tryPromise({
             try: () => response.text(),
             catch: () => new ApiError({ message: 'Failed to read response data' }),
@@ -414,15 +384,8 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
               try: () => response.text(),
               catch: () => 'Unknown error',
             }).pipe(Effect.orElseSucceed(() => 'Unknown error'))
-
-            yield* Effect.fail(
-              new ApiError({
-                message: errorText,
-                status: response.status,
-              }),
-            )
+            yield* Effect.fail(new ApiError({ message: errorText, status: response.status }))
           }
-
           const base64Patch = yield* Effect.tryPromise({
             try: () => response.text(),
             catch: () => new ApiError({ message: 'Failed to read response data' }),
@@ -546,7 +509,6 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
           const url = `${credentials.host}/a/changes/${encodeURIComponent(normalized)}?o=MESSAGES`
           const response = yield* makeRequest(url, authHeader, 'GET')
 
-          // Extract messages from the change response with runtime validation
           const changeResponse = yield* Schema.decodeUnknown(
             Schema.Struct({
               messages: Schema.optional(Schema.Array(MessageInfo)),
@@ -616,7 +578,6 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
             url += `?${params.join('&')}`
           }
 
-          // Gerrit returns groups as a Record, need to convert to array
           const groupsRecord = yield* makeRequest(
             url,
             authHeader,
@@ -624,12 +585,9 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
             undefined,
             Schema.Record({ key: Schema.String, value: GroupInfo }),
           )
-          // Convert Record to Array and sort alphabetically by name
-          return Object.values(groupsRecord).sort((a, b) => {
-            const aName = a.name || a.id
-            const bName = b.name || b.id
-            return aName.localeCompare(bName)
-          })
+          return Object.values(groupsRecord).sort((a, b) =>
+            (a.name || a.id).localeCompare(b.name || b.id),
+          )
         })
 
       const getGroup = (groupId: string) =>
@@ -661,12 +619,54 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
         Effect.gen(function* () {
           const { credentials, authHeader } = yield* getCredentialsAndAuth
           const normalized = yield* normalizeAndValidate(changeId)
-          // Use POST to /delete endpoint to support request body with notify option
           const url = `${credentials.host}/a/changes/${encodeURIComponent(normalized)}/reviewers/${encodeURIComponent(accountId)}/delete`
           const body = options?.notify ? { notify: options.notify } : {}
           yield* makeRequest(url, authHeader, 'POST', body)
         })
 
+      const getTopicUrl = (host: string, changeId: string): string =>
+        `${host}/a/changes/${encodeURIComponent(changeId)}/topic`
+
+      const getTopic = (changeId: string) =>
+        Effect.gen(function* () {
+          const { credentials, authHeader } = yield* getCredentialsAndAuth
+          const normalized = yield* normalizeAndValidate(changeId)
+          return yield* makeRequest(
+            getTopicUrl(credentials.host, normalized),
+            authHeader,
+            'GET',
+            undefined,
+            Schema.String,
+          ).pipe(
+            Effect.map((t) => t.replace(/^"|"$/g, '') || null),
+            Effect.catchIf(
+              (e) => e instanceof ApiError && e.status === 404,
+              () => Effect.succeed(null),
+            ),
+          )
+        })
+
+      const setTopic = (changeId: string, topic: string) =>
+        Effect.gen(function* () {
+          const { credentials, authHeader } = yield* getCredentialsAndAuth
+          const normalized = yield* normalizeAndValidate(changeId)
+          const result = yield* makeRequest(
+            getTopicUrl(credentials.host, normalized),
+            authHeader,
+            'PUT',
+            { topic },
+            Schema.String,
+          )
+          return result.replace(/^"|"$/g, '')
+        })
+
+      const deleteTopic = (changeId: string) =>
+        Effect.gen(function* () {
+          const { credentials, authHeader } = yield* getCredentialsAndAuth
+          const normalized = yield* normalizeAndValidate(changeId)
+          yield* makeRequest(getTopicUrl(credentials.host, normalized), authHeader, 'DELETE')
+        })
+
       return {
         getChange,
         listChanges,
@@ -691,6 +691,9 @@ export const GerritApiServiceLive: Layer.Layer<GerritApiService, never, ConfigSe
         getGroupDetail,
         getGroupMembers,
         removeReviewer,
+        getTopic,
+        setTopic,
+        deleteTopic,
       }
     }),
   )

package/src/cli/commands/comment.ts

@@ -3,6 +3,24 @@ import { Effect, pipe } from 'effect'
 import { type ApiError, GerritApiService } from '@/api/gerrit'
 import type { ChangeInfo, ReviewInput } from '@/schemas/gerrit'
 
+export const COMMENT_HELP_TEXT = `
+Examples:
+  # Post a general comment on a change
+  $ ger comment 12345 -m "Looks good to me!"
+
+  # Post a comment using piped input
+  $ echo "This is a comment from stdin!" | ger comment 12345
+
+  # Post a line-specific comment
+  $ ger comment 12345 --file src/main.js --line 42 -m "Consider using const here"
+
+  # Post multiple comments using batch mode
+  $ echo '{"message": "Review complete", "comments": [
+      {"file": "src/main.js", "line": 10, "message": "Good refactor"}
+    ]}' | ger comment 12345 --batch
+
+Note: Line numbers refer to the NEW version of the file, not diff line numbers.`
+
 interface CommentOptions {
   message?: string
   xml?: boolean

package/src/cli/commands/install-hook.ts

@@ -0,0 +1,59 @@
+import { Effect, Console } from 'effect'
+import chalk from 'chalk'
+import {
+  CommitHookService,
+  type CommitHookServiceImpl,
+  type HookInstallError,
+  type NotGitRepoError,
+} from '@/services/commit-hook'
+import { type ConfigError, type ConfigServiceImpl } from '@/services/config'
+
+export interface InstallHookOptions {
+  force?: boolean
+  xml?: boolean
+}
+
+export type InstallHookErrors = ConfigError | HookInstallError | NotGitRepoError
+
+export const installHookCommand = (
+  options: InstallHookOptions,
+): Effect.Effect<void, InstallHookErrors, CommitHookServiceImpl | ConfigServiceImpl> =>
+  Effect.gen(function* () {
+    const commitHookService = yield* CommitHookService
+
+    // Check if hook already exists using service method
+    const hookExists = yield* commitHookService.hasHook()
+
+    if (hookExists && !options.force) {
+      if (options.xml) {
+        yield* Console.log('<?xml version="1.0" encoding="UTF-8"?>')
+        yield* Console.log('<install_hook_result>')
+        yield* Console.log('  <status>skipped</status>')
+        yield* Console.log('  <message><![CDATA[commit-msg hook already installed]]></message>')
+        yield* Console.log('  <hint><![CDATA[Use --force to overwrite]]></hint>')
+        yield* Console.log('</install_hook_result>')
+      } else {
+        yield* Console.log(chalk.yellow('commit-msg hook already installed'))
+        yield* Console.log(chalk.dim('Use --force to overwrite'))
+      }
+      return
+    }
+
+    if (hookExists && options.force) {
+      if (!options.xml) {
+        yield* Console.log(chalk.yellow('Overwriting existing commit-msg hook...'))
+      }
+    }
+
+    // Install the hook (service logs progress messages in non-XML mode)
+    yield* commitHookService.installHook()
+
+    // Only output XML here - service already logs success message for non-XML mode
+    if (options.xml) {
+      yield* Console.log('<?xml version="1.0" encoding="UTF-8"?>')
+      yield* Console.log('<install_hook_result>')
+      yield* Console.log('  <status>success</status>')
+      yield* Console.log('  <message><![CDATA[commit-msg hook installed successfully]]></message>')
+      yield* Console.log('</install_hook_result>')
+    }
+  })

package/src/cli/commands/show.ts

@@ -7,7 +7,6 @@ import { formatDiffPretty } from '@/utils/diff-formatters'
 import { sanitizeCDATA, escapeXML } from '@/utils/shell-safety'
 import { formatDate } from '@/utils/formatters'
 import { getChangeIdFromHead, GitError, NoChangeIdError } from '@/utils/git-commit'
-import { writeFileSync } from 'node:fs'
 
 export const SHOW_HELP_TEXT = `
 Examples:
@@ -34,6 +33,13 @@ interface ShowOptions {
   json?: boolean
 }
 
+interface ReviewerIdentity {
+  accountId?: number
+  name?: string
+  email?: string
+  username?: string
+}
+
 interface ChangeDetails {
   id: string
   number: number
@@ -48,6 +54,25 @@ interface ChangeDetails {
   created?: string
   updated?: string
   commitMessage: string
+  topic?: string
+  reviewers: ReviewerIdentity[]
+  ccs: ReviewerIdentity[]
+}
+
+const formatReviewerLabel = (reviewer: ReviewerIdentity): string => {
+  const preferredIdentity = reviewer.name || reviewer.email || reviewer.username
+  if (!preferredIdentity) {
+    if (reviewer.accountId !== undefined) {
+      return `Account ${reviewer.accountId}`
+    }
+    return 'Unknown Reviewer'
+  }
+
+  if (reviewer.email && reviewer.name && reviewer.name !== reviewer.email) {
+    return `${reviewer.name} <${reviewer.email}>`
+  }
+
+  return preferredIdentity
 }
 
 const getChangeDetails = (
@@ -57,6 +82,21 @@ const getChangeDetails = (
     const gerritApi = yield* GerritApiService
     const change = yield* gerritApi.getChange(changeId)
 
+    let reviewerMap = change.reviewers
+    const shouldFetchReviewerFallback =
+      reviewerMap === undefined ||
+      (reviewerMap.REVIEWER === undefined && reviewerMap.CC === undefined)
+
+    if (shouldFetchReviewerFallback) {
+      const detailedChanges = yield* gerritApi
+        .listChanges(`change:${change._number}`)
+        .pipe(Effect.catchAll(() => Effect.succeed([])))
+      const detailedChange =
+        detailedChanges.find((candidate) => candidate._number === change._number) ||
+        detailedChanges[0]
+      reviewerMap = detailedChange?.reviewers
+    }
+
     return {
       id: change.change_id,
       number: change._number,
@@ -71,6 +111,19 @@ const getChangeDetails = (
       created: change.created,
       updated: change.updated,
       commitMessage: change.subject, // For now, using subject as commit message
+      topic: change.topic,
+      reviewers: (reviewerMap?.REVIEWER ?? []).map((reviewer) => ({
+        accountId: reviewer._account_id,
+        name: reviewer.name,
+        email: reviewer.email,
+        username: reviewer.username,
+      })),
+      ccs: (reviewerMap?.CC ?? []).map((cc) => ({
+        accountId: cc._account_id,
+        name: cc.name,
+        email: cc.email,
+        username: cc.username,
+      })),
     }
   })
 
@@ -142,6 +195,9 @@ const formatShowPretty = (
   console.log(`   Project: ${changeDetails.project}`)
   console.log(`   Branch: ${changeDetails.branch}`)
   console.log(`   Status: ${changeDetails.status}`)
+  if (changeDetails.topic) {
+    console.log(`   Topic: ${changeDetails.topic}`)
+  }
   console.log(`   Owner: ${changeDetails.owner.name || changeDetails.owner.email || 'Unknown'}`)
   console.log(
     `   Created: ${changeDetails.created ? formatDate(changeDetails.created) : 'Unknown'}`,
@@ -149,6 +205,14 @@ const formatShowPretty = (
   console.log(
     `   Updated: ${changeDetails.updated ? formatDate(changeDetails.updated) : 'Unknown'}`,
   )
+  if (changeDetails.reviewers.length > 0) {
+    console.log(
+      `   Reviewers: ${changeDetails.reviewers.map((reviewer) => formatReviewerLabel(reviewer)).join(', ')}`,
+    )
+  }
+  if (changeDetails.ccs.length > 0) {
+    console.log(`   CCs: ${changeDetails.ccs.map((cc) => formatReviewerLabel(cc)).join(', ')}`)
+  }
   console.log(`   Change-Id: ${changeDetails.id}`)
   console.log()
 
@@ -220,7 +284,24 @@ const formatShowJson = async (
       status: changeDetails.status,
       project: changeDetails.project,
       branch: changeDetails.branch,
+      topic: changeDetails.topic,
       owner: removeUndefined(changeDetails.owner),
+      reviewers: changeDetails.reviewers.map((reviewer) =>
+        removeUndefined({
+          account_id: reviewer.accountId,
+          name: reviewer.name,
+          email: reviewer.email,
+          username: reviewer.username,
+        }),
+      ),
+      ccs: changeDetails.ccs.map((cc) =>
+        removeUndefined({
+          account_id: cc.accountId,
+          name: cc.name,
+          email: cc.email,
+          username: cc.username,
+        }),
+      ),
       created: changeDetails.created,
       updated: changeDetails.updated,
     }),
@@ -301,6 +382,9 @@ const formatShowXml = async (
   xmlParts.push(`    <status>${escapeXML(changeDetails.status)}</status>`)
   xmlParts.push(`    <project>${escapeXML(changeDetails.project)}</project>`)
   xmlParts.push(`    <branch>${escapeXML(changeDetails.branch)}</branch>`)
+  if (changeDetails.topic) {
+    xmlParts.push(`    <topic><![CDATA[${sanitizeCDATA(changeDetails.topic)}]]></topic>`)
+  }
   xmlParts.push(`    <owner>`)
   if (changeDetails.owner.name) {
     xmlParts.push(`      <name><![CDATA[${sanitizeCDATA(changeDetails.owner.name)}]]></name>`)
@@ -309,6 +393,44 @@ const formatShowXml = async (
     xmlParts.push(`      <email>${escapeXML(changeDetails.owner.email)}</email>`)
   }
   xmlParts.push(`    </owner>`)
+  xmlParts.push(`    <reviewers>`)
+  xmlParts.push(`      <count>${changeDetails.reviewers.length}</count>`)
+  for (const reviewer of changeDetails.reviewers) {
+    xmlParts.push(`      <reviewer>`)
+    if (reviewer.accountId !== undefined) {
+      xmlParts.push(`        <account_id>${reviewer.accountId}</account_id>`)
+    }
+    if (reviewer.name) {
+      xmlParts.push(`        <name><![CDATA[${sanitizeCDATA(reviewer.name)}]]></name>`)
+    }
+    if (reviewer.email) {
+      xmlParts.push(`        <email>${escapeXML(reviewer.email)}</email>`)
+    }
+    if (reviewer.username) {
+      xmlParts.push(`        <username>${escapeXML(reviewer.username)}</username>`)
+    }
+    xmlParts.push(`      </reviewer>`)
+  }
+  xmlParts.push(`    </reviewers>`)
+  xmlParts.push(`    <ccs>`)
+  xmlParts.push(`      <count>${changeDetails.ccs.length}</count>`)
+  for (const cc of changeDetails.ccs) {
+    xmlParts.push(`      <cc>`)
+    if (cc.accountId !== undefined) {
+      xmlParts.push(`        <account_id>${cc.accountId}</account_id>`)
+    }
+    if (cc.name) {
+      xmlParts.push(`        <name><![CDATA[${sanitizeCDATA(cc.name)}]]></name>`)
+    }
+    if (cc.email) {
+      xmlParts.push(`        <email>${escapeXML(cc.email)}</email>`)
+    }
+    if (cc.username) {
+      xmlParts.push(`        <username>${escapeXML(cc.username)}</username>`)
+    }
+    xmlParts.push(`      </cc>`)
+  }
+  xmlParts.push(`    </ccs>`)
   xmlParts.push(`    <created>${escapeXML(changeDetails.created || '')}</created>`)
   xmlParts.push(`    <updated>${escapeXML(changeDetails.updated || '')}</updated>`)
   xmlParts.push(`  </change>`)

package/src/cli/commands/topic.ts

@@ -0,0 +1,108 @@
+import { Effect } from 'effect'
+import { type ApiError, GerritApiService } from '@/api/gerrit'
+import { sanitizeCDATA } from '@/utils/shell-safety'
+import { getChangeIdFromHead, GitError, NoChangeIdError } from '@/utils/git-commit'
+
+export const TOPIC_HELP_TEXT = `
+Examples:
+  # View current topic (auto-detect from HEAD)
+  $ ger topic
+
+  # View topic for specific change
+  $ ger topic 12345
+
+  # Set topic on a change
+  $ ger topic 12345 my-feature
+
+  # Remove topic from a change
+  $ ger topic 12345 --delete
+  $ ger topic --delete  # auto-detect from HEAD
+
+Note: When no change-id is provided, it will be auto-detected from the HEAD commit.`
+
+interface TopicOptions {
+  xml?: boolean
+  delete?: boolean
+}
+
+/**
+ * Manages topic for a Gerrit change.
+ *
+ * - No topic argument: get current topic
+ * - With topic argument: set topic
+ * - With --delete flag: remove topic
+ *
+ * @param changeId - Change number or Change-ID (auto-detects from HEAD if not provided)
+ * @param topic - Optional topic to set
+ * @param options - Configuration options
+ * @returns Effect that completes when the operation finishes
+ */
+export const topicCommand = (
+  changeId: string | undefined,
+  topic: string | undefined,
+  options: TopicOptions = {},
+): Effect.Effect<void, ApiError | GitError | NoChangeIdError, GerritApiService> =>
+  Effect.gen(function* () {
+    const gerritApi = yield* GerritApiService
+
+    // Auto-detect Change-ID from HEAD commit if not provided
+    const resolvedChangeId = changeId?.trim() || (yield* getChangeIdFromHead())
+
+    // Handle delete operation
+    if (options.delete) {
+      yield* gerritApi.deleteTopic(resolvedChangeId)
+
+      if (options.xml) {
+        console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
+        console.log(`<topic_result>`)
+        console.log(`  <status>success</status>`)
+        console.log(`  <action>deleted</action>`)
+        console.log(`  <change_id><![CDATA[${sanitizeCDATA(resolvedChangeId)}]]></change_id>`)
+        console.log(`</topic_result>`)
+      } else {
+        console.log(`✓ Removed topic from change ${resolvedChangeId}`)
+      }
+      return
+    }
+
+    // Handle set operation
+    if (topic !== undefined && topic.trim() !== '') {
+      const newTopic = yield* gerritApi.setTopic(resolvedChangeId, topic)
+
+      if (options.xml) {
+        console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
+        console.log(`<topic_result>`)
+        console.log(`  <status>success</status>`)
+        console.log(`  <action>set</action>`)
+        console.log(`  <change_id><![CDATA[${sanitizeCDATA(resolvedChangeId)}]]></change_id>`)
+        console.log(`  <topic><![CDATA[${sanitizeCDATA(newTopic)}]]></topic>`)
+        console.log(`</topic_result>`)
+      } else {
+        console.log(`✓ Set topic on change ${resolvedChangeId}: ${newTopic}`)
+      }
+      return
+    }
+
+    // Handle get operation (default)
+    const currentTopic = yield* gerritApi.getTopic(resolvedChangeId)
+
+    if (options.xml) {
+      console.log(`<?xml version="1.0" encoding="UTF-8"?>`)
+      console.log(`<topic_result>`)
+      console.log(`  <status>success</status>`)
+      console.log(`  <action>get</action>`)
+      console.log(`  <change_id><![CDATA[${sanitizeCDATA(resolvedChangeId)}]]></change_id>`)
+      if (currentTopic) {
+        console.log(`  <topic><![CDATA[${sanitizeCDATA(currentTopic)}]]></topic>`)
+      } else {
+        console.log(`  <topic />`)
+      }
+      console.log(`</topic_result>`)
+    } else {
+      if (currentTopic) {
+        console.log(currentTopic)
+      } else {
+        console.log(`No topic set for change ${resolvedChangeId}`)
+      }
+    }
+  })