@aaronshaf/ger 0.2.0 → 0.2.2

package/EXAMPLES.md

@@ -0,0 +1,409 @@
+# Programmatic Usage Examples
+
+This package can be used both as a CLI tool and as a library. Below are examples of using `@aaronshaf/ger` programmatically with Effect-TS.
+
+## Installation
+
+```bash
+bun add @aaronshaf/ger
+# or
+npm install @aaronshaf/ger
+```
+
+## Basic Setup
+
+All services in this package are built with Effect-TS, providing type-safe, composable operations.
+
+### Import the services
+
+```typescript
+import { Effect, pipe } from 'effect'
+import {
+  GerritApiService,
+  GerritApiServiceLive,
+  ConfigServiceLive,
+  type ChangeInfo,
+} from '@aaronshaf/ger'
+```
+
+## Configuration
+
+### Using Environment Variables
+
+Set these environment variables before running your program:
+
+```bash
+export GERRIT_HOST="https://gerrit.example.com"
+export GERRIT_USERNAME="your-username"
+export GERRIT_PASSWORD="your-http-password"
+```
+
+### Using File-Based Config
+
+Or run the CLI once to set up configuration:
+
+```bash
+ger setup
+```
+
+This stores credentials in `~/.ger/config.json`.
+
+## Examples
+
+### 1. Get Change Information
+
+```typescript
+import { Effect, pipe } from 'effect'
+import {
+  GerritApiService,
+  GerritApiServiceLive,
+  ConfigServiceLive,
+} from '@aaronshaf/ger'
+
+const getChangeDetails = (changeId: string) =>
+  Effect.gen(function* () {
+    const api = yield* GerritApiService
+    const change = yield* api.getChange(changeId)
+
+    console.log(`Change: ${change.subject}`)
+    console.log(`Status: ${change.status}`)
+    console.log(`Owner: ${change.owner?.name || 'Unknown'}`)
+
+    return change
+  })
+
+// Run the program
+const program = pipe(
+  getChangeDetails('12345'),
+  Effect.provide(GerritApiServiceLive),
+  Effect.provide(ConfigServiceLive)
+)
+
+Effect.runPromise(program)
+  .then(() => console.log('Done!'))
+  .catch(console.error)
+```
+
+### 2. List Open Changes
+
+```typescript
+import { Effect, pipe } from 'effect'
+import {
+  GerritApiService,
+  GerritApiServiceLive,
+  ConfigServiceLive,
+} from '@aaronshaf/ger'
+
+const listMyChanges = Effect.gen(function* () {
+  const api = yield* GerritApiService
+
+  // Query for your open changes
+  const changes = yield* api.listChanges('is:open owner:self')
+
+  console.log(`You have ${changes.length} open changes:`)
+  for (const change of changes) {
+    console.log(`  - #${change._number}: ${change.subject}`)
+  }
+
+  return changes
+})
+
+const program = pipe(
+  listMyChanges,
+  Effect.provide(GerritApiServiceLive),
+  Effect.provide(ConfigServiceLive)
+)
+
+Effect.runPromise(program).catch(console.error)
+```
+
+### 3. Post a Comment
+
+```typescript
+import { Effect, pipe } from 'effect'
+import {
+  GerritApiService,
+  GerritApiServiceLive,
+  ConfigServiceLive,
+  type ReviewInput,
+} from '@aaronshaf/ger'
+
+const postComment = (changeId: string) =>
+  Effect.gen(function* () {
+    const api = yield* GerritApiService
+
+    const review: ReviewInput = {
+      message: 'Looks good to me!',
+      labels: {
+        'Code-Review': 1,
+      },
+    }
+
+    yield* api.postReview(changeId, review)
+    console.log('Comment posted successfully!')
+  })
+
+const program = pipe(
+  postComment('12345'),
+  Effect.provide(GerritApiServiceLive),
+  Effect.provide(ConfigServiceLive)
+)
+
+Effect.runPromise(program).catch(console.error)
+```
+
+### 4. Post Inline Comments
+
+```typescript
+import { Effect, pipe } from 'effect'
+import {
+  GerritApiService,
+  GerritApiServiceLive,
+  ConfigServiceLive,
+  type ReviewInput,
+} from '@aaronshaf/ger'
+
+const postInlineComments = (changeId: string) =>
+  Effect.gen(function* () {
+    const api = yield* GerritApiService
+
+    const review: ReviewInput = {
+      message: 'Review complete',
+      comments: {
+        'src/api.ts': [
+          {
+            line: 42,
+            message: 'Consider using const here for immutability',
+            unresolved: false,
+          },
+          {
+            line: 55,
+            message: 'This could cause a security issue',
+            unresolved: true,
+          },
+        ],
+        'src/utils.ts': [
+          {
+            line: 10,
+            message: 'Nice refactor!',
+          },
+        ],
+      },
+    }
+
+    yield* api.postReview(changeId, review)
+    console.log('Inline comments posted!')
+  })
+
+const program = pipe(
+  postInlineComments('12345'),
+  Effect.provide(GerritApiServiceLive),
+  Effect.provide(ConfigServiceLive)
+)
+
+Effect.runPromise(program).catch(console.error)
+```
+
+### 5. Get Diff for a Change
+
+```typescript
+import { Effect, pipe } from 'effect'
+import {
+  GerritApiService,
+  GerritApiServiceLive,
+  ConfigServiceLive,
+  type DiffOptions,
+} from '@aaronshaf/ger'
+
+const getDiff = (changeId: string) =>
+  Effect.gen(function* () {
+    const api = yield* GerritApiService
+
+    // Get unified diff format (default)
+    const diff = yield* api.getDiff(changeId, { format: 'unified' })
+    console.log('Diff:', diff)
+
+    // Or get list of changed files
+    const files = yield* api.getDiff(changeId, { format: 'files' })
+    console.log('Changed files:', files)
+
+    return diff
+  })
+
+const program = pipe(
+  getDiff('12345'),
+  Effect.provide(GerritApiServiceLive),
+  Effect.provide(ConfigServiceLive)
+)
+
+Effect.runPromise(program).catch(console.error)
+```
+
+### 6. Test Connection
+
+```typescript
+import { Effect, pipe } from 'effect'
+import {
+  GerritApiService,
+  GerritApiServiceLive,
+  ConfigServiceLive,
+} from '@aaronshaf/ger'
+
+const testConnection = Effect.gen(function* () {
+  const api = yield* GerritApiService
+  const isConnected = yield* api.testConnection
+
+  if (isConnected) {
+    console.log('✓ Connected to Gerrit!')
+  } else {
+    console.log('✗ Connection failed')
+  }
+
+  return isConnected
+})
+
+const program = pipe(
+  testConnection,
+  Effect.provide(GerritApiServiceLive),
+  Effect.provide(ConfigServiceLive)
+)
+
+Effect.runPromise(program).catch(console.error)
+```
+
+### 7. Error Handling with Effect
+
+```typescript
+import { Effect, pipe, Console } from 'effect'
+import {
+  GerritApiService,
+  GerritApiServiceLive,
+  ConfigServiceLive,
+  ApiError,
+  ConfigError,
+} from '@aaronshaf/ger'
+
+const safeGetChange = (changeId: string) =>
+  Effect.gen(function* () {
+    const api = yield* GerritApiService
+    const change = yield* api.getChange(changeId)
+    return change
+  }).pipe(
+    Effect.catchTag('ApiError', (error) =>
+      Console.error(`API Error: ${error.message}`).pipe(
+        Effect.map(() => null)
+      )
+    ),
+    Effect.catchTag('ConfigError', (error) =>
+      Console.error(`Config Error: ${error.message}`).pipe(
+        Effect.map(() => null)
+      )
+    )
+  )
+
+const program = pipe(
+  safeGetChange('invalid-change'),
+  Effect.provide(GerritApiServiceLive),
+  Effect.provide(ConfigServiceLive)
+)
+
+Effect.runPromise(program)
+```
+
+### 8. Using Utilities
+
+```typescript
+import {
+  normalizeChangeIdentifier,
+  extractChangeIdFromCommitMessage,
+  extractChangeNumber,
+  normalizeGerritHost,
+} from '@aaronshaf/ger'
+
+// Normalize change identifiers
+const normalized = normalizeChangeIdentifier('12345')
+// or
+const normalizedId = normalizeChangeIdentifier('If5a3ae8cb5a107e187447802358417f311d0c4b1')
+
+// Extract change ID from commit message
+const commitMsg = `feat: add feature
+
+Change-Id: If5a3ae8cb5a107e187447802358417f311d0c4b1`
+
+const changeId = extractChangeIdFromCommitMessage(commitMsg)
+console.log(changeId) // "If5a3ae8cb5a107e187447802358417f311d0c4b1"
+
+// Extract change number from Gerrit URL
+const url = 'https://gerrit.example.com/c/project/+/12345'
+const changeNumber = extractChangeNumber(url)
+console.log(changeNumber) // "12345"
+
+// Normalize Gerrit host
+const host = normalizeGerritHost('gerrit.example.com')
+console.log(host) // "https://gerrit.example.com"
+```
+
+### 9. Working with Schemas
+
+```typescript
+import { Schema } from '@effect/schema'
+import { Effect } from 'effect'
+import { ChangeInfo, ReviewInput } from '@aaronshaf/ger'
+
+// Validate and decode API responses
+const validateChange = (data: unknown) =>
+  Schema.decodeUnknown(ChangeInfo)(data)
+
+// Validate review input before sending
+const validateReview = (review: unknown) =>
+  Schema.decodeUnknown(ReviewInput)(review)
+
+// Use in an Effect program
+const safeReview = Effect.gen(function* () {
+  const review = {
+    message: 'LGTM',
+    labels: { 'Code-Review': 2 },
+  }
+
+  const validated = yield* validateReview(review)
+  console.log('Review is valid:', validated)
+
+  return validated
+})
+```
+
+## Direct Module Access
+
+You can also import directly from specific modules:
+
+```typescript
+// Import from specific services
+import { GerritApiService, GerritApiServiceLive } from '@aaronshaf/ger/api'
+import { ConfigService, ConfigServiceLive } from '@aaronshaf/ger/services/config'
+
+// Import from specific schemas
+import { ChangeInfo, ReviewInput } from '@aaronshaf/ger/schemas/gerrit'
+
+// Import utilities
+import { normalizeChangeIdentifier, extractChangeNumber } from '@aaronshaf/ger/utils'
+```
+
+## TypeScript Configuration
+
+Make sure your `tsconfig.json` includes:
+
+```json
+{
+  "compilerOptions": {
+    "moduleResolution": "bundler",
+    "allowImportingTsExtensions": true,
+    "strict": true
+  }
+}
+```
+
+## More Information
+
+- See the [main README](./README.md) for CLI usage
+- Check out the [Effect documentation](https://effect.website/) to learn more about Effect-TS
+- View the type definitions in your IDE for detailed API documentation

package/README.md

@@ -199,7 +199,7 @@ ger comments 12345 --pretty
 Extract URLs from change messages and comments for automation and scripting:
 
 ```bash
-# Extract all Jenkins build-summary-report URLs
+# Extract URLs from current HEAD commit's change (auto-detect)
 ger extract-url "build-summary-report"
 
 # Get the latest build URL (using tail)
@@ -208,9 +208,15 @@ ger extract-url "build-summary-report" | tail -1
 # Get the first/oldest build URL (using head)
 ger extract-url "jenkins" | head -1
 
-# For a specific change
+# For a specific change (using change number)
 ger extract-url "build-summary" 12345
 
+# For a specific change (using Change-ID)
+ger extract-url "build-summary" If5a3ae8cb5a107e187447802358417f311d0c4b1
+
+# Chain with other tools for specific change
+ger extract-url "build-summary-report" 12345 | tail -1 | jk failures --smart --xml
+
 # Use regex for precise matching
 ger extract-url "job/Canvas/job/main/\d+/" --regex
 
@@ -225,6 +231,7 @@ ger extract-url "jenkins" --xml
 ```
 
 #### How it works:
+- **Change detection**: Auto-detects Change-ID from HEAD commit if not specified, or accepts explicit change number/Change-ID
 - **Pattern matching**: Substring match by default, regex with `--regex`
 - **Sources**: Searches messages by default, add `--include-comments` to include inline comments
 - **Ordering**: URLs are output in chronological order (oldest first)

package/index.ts

@@ -0,0 +1,219 @@
+/**
+ * @aaronshaf/ger - Gerrit CLI and SDK
+ *
+ * This package provides both a CLI tool and a programmatic API for interacting with Gerrit Code Review.
+ * Built with Effect-TS for type-safe, composable operations.
+ *
+ * @module
+ *
+ * @example Basic usage with Effect
+ * ```typescript
+ * import { Effect, pipe } from 'effect'
+ * import {
+ *   GerritApiService,
+ *   GerritApiServiceLive,
+ *   ConfigServiceLive,
+ * } from '@aaronshaf/ger'
+ *
+ * const program = Effect.gen(function* () {
+ *   const api = yield* GerritApiService
+ *   const change = yield* api.getChange('12345')
+ *   console.log(change.subject)
+ * })
+ *
+ * const runnable = pipe(
+ *   program,
+ *   Effect.provide(GerritApiServiceLive),
+ *   Effect.provide(ConfigServiceLive)
+ * )
+ *
+ * Effect.runPromise(runnable)
+ * ```
+ */
+
+// ============================================================================
+// Core API Service
+// ============================================================================
+
+export {
+  // Service tag and implementation
+  GerritApiService,
+  GerritApiServiceLive,
+  // Types
+  type GerritApiServiceImpl,
+  // Errors
+  ApiError,
+  type ApiErrorFields,
+} from './src/api/gerrit'
+
+// ============================================================================
+// Configuration Service
+// ============================================================================
+
+export {
+  // Service tag and implementation
+  ConfigService,
+  ConfigServiceLive,
+  // Types
+  type ConfigServiceImpl,
+  // Errors
+  ConfigError,
+  type ConfigErrorFields,
+} from './src/services/config'
+
+// ============================================================================
+// Review Strategy Service
+// ============================================================================
+
+export {
+  // Strategy types
+  type ReviewStrategy,
+  // Built-in strategies
+  claudeCliStrategy,
+  geminiCliStrategy,
+  openCodeCliStrategy,
+  // Service
+  ReviewStrategyService,
+  ReviewStrategyServiceLive,
+  type ReviewStrategyServiceImpl,
+  // Errors
+  ReviewStrategyError,
+  type ReviewStrategyErrorFields,
+} from './src/services/review-strategy'
+
+// ============================================================================
+// Git Worktree Service
+// ============================================================================
+
+export {
+  // Service tag and implementation
+  GitWorktreeService,
+  GitWorktreeServiceLive,
+  type GitWorktreeServiceImpl,
+  // Types
+  type WorktreeInfo,
+  // Errors
+  WorktreeCreationError,
+  type WorktreeCreationErrorFields,
+  PatchsetFetchError,
+  type PatchsetFetchErrorFields,
+  DirtyRepoError,
+  type DirtyRepoErrorFields,
+  NotGitRepoError,
+  type NotGitRepoErrorFields,
+  type GitWorktreeError,
+} from './src/services/git-worktree'
+
+// ============================================================================
+// Schemas and Types
+// ============================================================================
+
+export {
+  // Authentication
+  GerritCredentials,
+  type GerritCredentials as GerritCredentialsType,
+  // Changes
+  ChangeInfo,
+  type ChangeInfo as ChangeInfoType,
+  // Comments
+  CommentInput,
+  type CommentInput as CommentInputType,
+  CommentInfo,
+  type CommentInfo as CommentInfoType,
+  // Messages
+  MessageInfo,
+  type MessageInfo as MessageInfoType,
+  // Reviews
+  ReviewInput,
+  type ReviewInput as ReviewInputType,
+  // Files and Diffs
+  FileInfo,
+  type FileInfo as FileInfoType,
+  FileDiffContent,
+  type FileDiffContent as FileDiffContentType,
+  RevisionInfo,
+  type RevisionInfo as RevisionInfoType,
+  // Diff Options
+  DiffFormat,
+  type DiffFormat as DiffFormatType,
+  DiffOptions,
+  type DiffOptions as DiffOptionsType,
+  DiffCommandOptions,
+  type DiffCommandOptions as DiffCommandOptionsType,
+  // Errors
+  GerritError,
+  type GerritError as GerritErrorType,
+} from './src/schemas/gerrit'
+
+export {
+  // Config schemas
+  AppConfig,
+  type AppConfig as AppConfigType,
+  AiConfig,
+  type AiConfig as AiConfigType,
+  // Utilities
+  aiConfigFromFlat,
+  migrateFromNestedConfig,
+} from './src/schemas/config'
+
+// ============================================================================
+// Utilities
+// ============================================================================
+
+export {
+  // Change ID handling
+  normalizeChangeIdentifier,
+  isChangeId,
+  isChangeNumber,
+  isValidChangeIdentifier,
+  getIdentifierType,
+} from './src/utils/change-id'
+
+export {
+  // Git commit utilities
+  extractChangeIdFromCommitMessage,
+  getLastCommitMessage,
+  getChangeIdFromHead,
+  GitError,
+  NoChangeIdError,
+} from './src/utils/git-commit'
+
+export {
+  // URL parsing
+  extractChangeNumber,
+  normalizeGerritHost,
+  isValidChangeId,
+} from './src/utils/url-parser'
+
+export {
+  // Message filtering
+  filterMeaningfulMessages,
+  sortMessagesByDate,
+} from './src/utils/message-filters'
+
+export {
+  // Shell safety
+  sanitizeCDATA,
+} from './src/utils/shell-safety'
+
+export {
+  // Formatters
+  formatDate,
+  getStatusIndicator,
+  colors,
+} from './src/utils/formatters'
+
+export {
+  // Comment formatters
+  formatCommentsPretty,
+  formatCommentsXml,
+  type CommentWithContext,
+} from './src/utils/comment-formatters'
+
+export {
+  // Diff formatters
+  formatDiffPretty,
+  formatDiffSummary,
+  formatFilesList,
+  extractDiffStats,
+} from './src/utils/diff-formatters'

package/package.json

@@ -1,11 +1,56 @@
 {
   "name": "@aaronshaf/ger",
-  "version": "0.2.0",
+  "version": "0.2.2",
+  "description": "Gerrit CLI and SDK - A modern CLI tool and TypeScript SDK for Gerrit Code Review, built with Effect-TS",
+  "keywords": [
+    "gerrit",
+    "code-review",
+    "cli",
+    "sdk",
+    "effect",
+    "effect-ts",
+    "typescript",
+    "api-client"
+  ],
   "module": "index.ts",
   "type": "module",
   "bin": {
     "ger": "./bin/ger"
   },
+  "exports": {
+    ".": {
+      "import": "./index.ts",
+      "types": "./index.ts"
+    },
+    "./api": {
+      "import": "./src/api/gerrit.ts",
+      "types": "./src/api/gerrit.ts"
+    },
+    "./services/config": {
+      "import": "./src/services/config.ts",
+      "types": "./src/services/config.ts"
+    },
+    "./services/review-strategy": {
+      "import": "./src/services/review-strategy.ts",
+      "types": "./src/services/review-strategy.ts"
+    },
+    "./services/git-worktree": {
+      "import": "./src/services/git-worktree.ts",
+      "types": "./src/services/git-worktree.ts"
+    },
+    "./schemas/gerrit": {
+      "import": "./src/schemas/gerrit.ts",
+      "types": "./src/schemas/gerrit.ts"
+    },
+    "./schemas/config": {
+      "import": "./src/schemas/config.ts",
+      "types": "./src/schemas/config.ts"
+    },
+    "./utils": {
+      "import": "./src/utils/index.ts",
+      "types": "./src/utils/index.ts"
+    }
+  },
   "repository": {
     "type": "git",
     "url": "git+https://github.com/aaronshaf/ger.git"

package/src/cli/commands/review.ts

@@ -1,22 +1,8 @@
-import { Effect, pipe, Schema, Layer } from 'effect'
-import {
-  ReviewStrategyService,
-  type ReviewStrategy,
-  ReviewStrategyError,
-} from '@/services/review-strategy'
+import { Effect, pipe, Schema } from 'effect'
+import { ReviewStrategyService, ReviewStrategyError } from '@/services/review-strategy'
 import { commentCommandWithInput } from './comment'
 import { Console } from 'effect'
-import { type ApiError, GerritApiService } from '@/api/gerrit'
-import type { CommentInfo } from '@/schemas/gerrit'
-import { sanitizeCDATA, escapeXML } from '@/utils/shell-safety'
-import { formatDiffPretty } from '@/utils/diff-formatters'
-import { formatDate } from '@/utils/formatters'
-import {
-  formatChangeAsXML,
-  formatCommentsAsXML,
-  formatMessagesAsXML,
-  flattenComments,
-} from '@/utils/review-formatters'
+import { GerritApiService } from '@/api/gerrit'
 import { buildEnhancedPrompt } from '@/utils/review-prompt-builder'
 import * as fs from 'node:fs/promises'
 import * as fsSync from 'node:fs'
@@ -25,7 +11,7 @@ import * as path from 'node:path'
 import { fileURLToPath } from 'node:url'
 import { dirname } from 'node:path'
 import * as readline from 'node:readline'
-import { GitWorktreeService, GitWorktreeServiceLive } from '@/services/git-worktree'
+import { GitWorktreeService } from '@/services/git-worktree'
 
 // Get the directory of this module
 const __filename = fileURLToPath(import.meta.url)
@@ -126,7 +112,7 @@ const validateAndFixInlineComments = (
     for (const rawComment of rawComments) {
       // Validate comment structure using Effect Schema
       const parseResult = yield* Schema.decodeUnknown(InlineCommentSchema)(rawComment).pipe(
-        Effect.catchTag('ParseError', (parseError) =>
+        Effect.catchTag('ParseError', (_parseError) =>
           Effect.gen(function* () {
             yield* Console.warn('Skipping comment with invalid structure')
             return yield* Effect.succeed(null)
@@ -192,118 +178,6 @@ const validateAndFixInlineComments = (
     return validComments
   })
 
-// Legacy helper for backward compatibility (will be removed)
-const getChangeDataAsXml = (changeId: string): Effect.Effect<string, ApiError, GerritApiService> =>
-  Effect.gen(function* () {
-    const gerritApi = yield* GerritApiService
-
-    // Fetch all data
-    const change = yield* gerritApi.getChange(changeId)
-    const diffResult = yield* gerritApi.getDiff(changeId)
-    const diff = typeof diffResult === 'string' ? diffResult : JSON.stringify(diffResult)
-    const commentsMap = yield* gerritApi.getComments(changeId)
-    const messages = yield* gerritApi.getMessages(changeId)
-
-    const comments = flattenComments(commentsMap)
-
-    // Build XML string using helper functions
-    const xmlLines: string[] = []
-    xmlLines.push(`<?xml version="1.0" encoding="UTF-8"?>`)
-    xmlLines.push(`<show_result>`)
-    xmlLines.push(`  <status>success</status>`)
-    xmlLines.push(...formatChangeAsXML(change))
-    xmlLines.push(`  <diff><![CDATA[${sanitizeCDATA(diff)}]]></diff>`)
-    xmlLines.push(...formatCommentsAsXML(comments))
-    xmlLines.push(...formatMessagesAsXML(messages))
-    xmlLines.push(`</show_result>`)
-
-    return xmlLines.join('\n')
-  })
-
-// Helper to get change data and format as pretty string
-const getChangeDataAsPretty = (
-  changeId: string,
-): Effect.Effect<string, ApiError, GerritApiService> =>
-  Effect.gen(function* () {
-    const gerritApi = yield* GerritApiService
-
-    // Fetch all data
-    const change = yield* gerritApi.getChange(changeId)
-    const diffResult = yield* gerritApi.getDiff(changeId)
-    const diff = typeof diffResult === 'string' ? diffResult : JSON.stringify(diffResult)
-    const commentsMap = yield* gerritApi.getComments(changeId)
-    const messages = yield* gerritApi.getMessages(changeId)
-
-    const comments = flattenComments(commentsMap)
-
-    // Build pretty string
-    const lines: string[] = []
-
-    // Change details header
-    lines.push('━'.repeat(80))
-    lines.push(`📋 Change ${change._number}: ${change.subject}`)
-    lines.push('━'.repeat(80))
-    lines.push('')
-
-    // Metadata
-    lines.push('📝 Details:')
-    lines.push(`   Project: ${change.project}`)
-    lines.push(`   Branch: ${change.branch}`)
-    lines.push(`   Status: ${change.status}`)
-    lines.push(`   Owner: ${change.owner?.name || change.owner?.email || 'Unknown'}`)
-    lines.push(`   Created: ${change.created ? formatDate(change.created) : 'Unknown'}`)
-    lines.push(`   Updated: ${change.updated ? formatDate(change.updated) : 'Unknown'}`)
-    lines.push(`   Change-Id: ${change.change_id}`)
-    lines.push('')
-
-    // Diff section
-    lines.push('🔍 Diff:')
-    lines.push('─'.repeat(40))
-    lines.push(formatDiffPretty(diff))
-    lines.push('')
-
-    // Comments section
-    if (comments.length > 0) {
-      lines.push('💬 Inline Comments:')
-      lines.push('─'.repeat(40))
-      for (const comment of comments) {
-        const author = comment.author?.name || 'Unknown'
-        const date = comment.updated ? formatDate(comment.updated) : 'Unknown'
-        lines.push(`📅 ${date} - ${author}`)
-        if (comment.path) lines.push(`   File: ${comment.path}`)
-        if (comment.line) lines.push(`   Line: ${comment.line}`)
-        lines.push(`   ${comment.message}`)
-        if (comment.unresolved) lines.push(`   ⚠️ Unresolved`)
-        lines.push('')
-      }
-    }
-
-    // Messages section
-    if (messages.length > 0) {
-      lines.push('📝 Review Activity:')
-      lines.push('─'.repeat(40))
-      for (const message of messages) {
-        const author = message.author?.name || 'Unknown'
-        const date = formatDate(message.date)
-        const cleanMessage = message.message.trim()
-
-        // Skip very short automated messages
-        if (
-          cleanMessage.length < 10 &&
-          (cleanMessage.includes('Build') || cleanMessage.includes('Patch'))
-        ) {
-          continue
-        }
-
-        lines.push(`📅 ${date} - ${author}`)
-        lines.push(`   ${cleanMessage}`)
-        lines.push('')
-      }
-    }
-
-    return lines.join('\n')
-  })
-
 // Helper function to prompt user for confirmation
 const promptUser = (message: string): Effect.Effect<boolean, never> =>
   Effect.async<boolean, never>((resume) => {

package/src/cli/commands/setup.ts

@@ -11,6 +11,7 @@ import { AppConfig } from '@/schemas/config'
 import { Schema } from '@effect/schema'
 import { input, password } from '@inquirer/prompts'
 import { spawn } from 'node:child_process'
+import { normalizeGerritHost } from '@/utils/url-parser'
 
 // Check if a command exists on the system
 const checkCommandExists = (command: string): Promise<boolean> =>
@@ -209,7 +210,7 @@ const setupEffect = (configService: ConfigServiceImpl) =>
 
                 // Build flat config
                 const configData = {
-                  host: host.trim().replace(/\/$/, ''), // Remove trailing slash
+                  host: normalizeGerritHost(host),
                   username: username.trim(),
                   password: passwordValue,
                   ...(aiToolCommand && {

package/src/cli/index.ts

@@ -56,7 +56,7 @@ function getVersion(): string {
     const packageJsonPath = join(__dirname, '..', '..', 'package.json')
     const packageJson = JSON.parse(readFileSync(packageJsonPath, 'utf8'))
     return packageJson.version || '0.0.0'
-  } catch (error) {
+  } catch {
     // Fallback version if package.json can't be read
     return '0.0.0'
   }

package/src/services/git-worktree.ts

@@ -84,7 +84,11 @@ export class NotGitRepoError
   readonly name = 'NotGitRepoError'
 }
 
-export type GitError = WorktreeCreationError | PatchsetFetchError | DirtyRepoError | NotGitRepoError
+export type GitWorktreeError =
+  | WorktreeCreationError
+  | PatchsetFetchError
+  | DirtyRepoError
+  | NotGitRepoError
 
 // Worktree info
 export interface WorktreeInfo {
@@ -99,8 +103,8 @@ export interface WorktreeInfo {
 const runGitCommand = (
   args: string[],
   options: { cwd?: string } = {},
-): Effect.Effect<string, GitError, never> =>
-  Effect.async<string, GitError, never>((resume) => {
+): Effect.Effect<string, GitWorktreeError, never> =>
+  Effect.async<string, GitWorktreeError, never>((resume) => {
     const child = spawn('git', args, {
       cwd: options.cwd || process.cwd(),
       stdio: ['ignore', 'pipe', 'pipe'],
@@ -156,23 +160,6 @@ const validateGitRepo = (): Effect.Effect<void, NotGitRepoError, never> =>
     Effect.map(() => undefined),
   )
 
-// Check if working directory is clean
-const validateCleanRepo = (): Effect.Effect<void, DirtyRepoError, never> =>
-  pipe(
-    runGitCommand(['status', '--porcelain']),
-    Effect.mapError(() => new DirtyRepoError({ message: 'Failed to check repository status' })),
-    Effect.flatMap((output) =>
-      output.trim() === ''
-        ? Effect.succeed(undefined)
-        : Effect.fail(
-            new DirtyRepoError({
-              message:
-                'Working directory has uncommitted changes. Please commit or stash changes before review.',
-            }),
-          ),
-    ),
-  )
-
 // Generate unique worktree path
 const generateWorktreePath = (changeId: string): string => {
   const timestamp = Date.now()
@@ -199,7 +186,7 @@ const buildRefspec = (changeNumber: string, patchsetNumber: number = 1): string
 }
 
 // Get the current HEAD commit hash to avoid branch conflicts
-const getCurrentCommit = (): Effect.Effect<string, GitError, never> =>
+const getCurrentCommit = (): Effect.Effect<string, GitWorktreeError, never> =>
   pipe(
     runGitCommand(['rev-parse', 'HEAD']),
     Effect.map((output) => output.trim()),
@@ -243,11 +230,13 @@ const getLatestPatchsetNumber = (
 
 // GitWorktreeService implementation
 export interface GitWorktreeServiceImpl {
-  validatePreconditions: () => Effect.Effect<void, GitError, never>
-  createWorktree: (changeId: string) => Effect.Effect<WorktreeInfo, GitError, never>
-  fetchAndCheckoutPatchset: (worktreeInfo: WorktreeInfo) => Effect.Effect<void, GitError, never>
+  validatePreconditions: () => Effect.Effect<void, GitWorktreeError, never>
+  createWorktree: (changeId: string) => Effect.Effect<WorktreeInfo, GitWorktreeError, never>
+  fetchAndCheckoutPatchset: (
+    worktreeInfo: WorktreeInfo,
+  ) => Effect.Effect<void, GitWorktreeError, never>
   cleanup: (worktreeInfo: WorktreeInfo) => Effect.Effect<void, never, never>
-  getChangedFiles: () => Effect.Effect<string[], GitError, never>
+  getChangedFiles: () => Effect.Effect<string[], GitWorktreeError, never>
 }
 
 const GitWorktreeServiceImplLive: GitWorktreeServiceImpl = {

package/src/utils/index.ts

@@ -0,0 +1,55 @@
+/**
+ * Utility functions for working with Gerrit
+ * @module utils
+ */
+
+// Change ID utilities
+export {
+  normalizeChangeIdentifier,
+  isChangeId,
+  isChangeNumber,
+  isValidChangeIdentifier,
+  getIdentifierType,
+} from './change-id'
+
+// Git commit utilities
+export {
+  extractChangeIdFromCommitMessage,
+  getLastCommitMessage,
+  getChangeIdFromHead,
+  GitError,
+  NoChangeIdError,
+} from './git-commit'
+
+// URL parsing
+export {
+  extractChangeNumber,
+  normalizeGerritHost,
+  isValidChangeId,
+} from './url-parser'
+
+// Message filtering
+export { filterMeaningfulMessages, sortMessagesByDate } from './message-filters'
+
+// Shell safety
+export { sanitizeCDATA } from './shell-safety'
+
+// Formatters
+export {
+  formatDate,
+  getStatusIndicator,
+  colors,
+} from './formatters'
+
+export {
+  formatCommentsPretty,
+  formatCommentsXml,
+  type CommentWithContext,
+} from './comment-formatters'
+
+export {
+  formatDiffPretty,
+  formatDiffSummary,
+  formatFilesList,
+  extractDiffStats,
+} from './diff-formatters'

package/src/utils/review-prompt-builder.ts

@@ -1,6 +1,5 @@
 import { Effect } from 'effect'
 import { type ApiError, GerritApiService } from '@/api/gerrit'
-import type { CommentInfo, MessageInfo } from '@/schemas/gerrit'
 import { flattenComments } from '@/utils/review-formatters'
 
 export const buildEnhancedPrompt = (

package/src/utils/url-parser.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, test } from 'bun:test'
-import { extractChangeNumber, isValidChangeId } from './url-parser'
+import { extractChangeNumber, isValidChangeId, normalizeGerritHost } from './url-parser'
 
 describe('extractChangeNumber', () => {
   test('extracts change number from standard Gerrit URL', () => {
@@ -121,3 +121,151 @@ describe('isValidChangeId', () => {
     expect(isValidChangeId('-abc')).toBe(false)
   })
 })
+
+describe('normalizeGerritHost', () => {
+  describe('adding protocol', () => {
+    test('adds https:// when no protocol is provided', () => {
+      expect(normalizeGerritHost('gerrit.example.com')).toBe('https://gerrit.example.com')
+    })
+
+    test('adds https:// to hostname with port', () => {
+      expect(normalizeGerritHost('gerrit.example.com:8080')).toBe('https://gerrit.example.com:8080')
+    })
+
+    test('adds https:// to localhost', () => {
+      expect(normalizeGerritHost('localhost:8080')).toBe('https://localhost:8080')
+    })
+
+    test('adds https:// to IP address', () => {
+      expect(normalizeGerritHost('192.168.1.100')).toBe('https://192.168.1.100')
+    })
+
+    test('adds https:// to IP address with port', () => {
+      expect(normalizeGerritHost('192.168.1.100:8080')).toBe('https://192.168.1.100:8080')
+    })
+  })
+
+  describe('preserving existing protocol', () => {
+    test('preserves https:// when already present', () => {
+      expect(normalizeGerritHost('https://gerrit.example.com')).toBe('https://gerrit.example.com')
+    })
+
+    test('preserves http:// when explicitly provided', () => {
+      expect(normalizeGerritHost('http://gerrit.example.com')).toBe('http://gerrit.example.com')
+    })
+
+    test('preserves https:// with port', () => {
+      expect(normalizeGerritHost('https://gerrit.example.com:8080')).toBe(
+        'https://gerrit.example.com:8080',
+      )
+    })
+
+    test('preserves http:// with port', () => {
+      expect(normalizeGerritHost('http://gerrit.example.com:8080')).toBe(
+        'http://gerrit.example.com:8080',
+      )
+    })
+  })
+
+  describe('removing trailing slashes', () => {
+    test('removes single trailing slash', () => {
+      expect(normalizeGerritHost('https://gerrit.example.com/')).toBe('https://gerrit.example.com')
+    })
+
+    test('removes trailing slash from URL without protocol', () => {
+      expect(normalizeGerritHost('gerrit.example.com/')).toBe('https://gerrit.example.com')
+    })
+
+    test('removes trailing slash from URL with port', () => {
+      expect(normalizeGerritHost('https://gerrit.example.com:8080/')).toBe(
+        'https://gerrit.example.com:8080',
+      )
+    })
+
+    test('handles URL without trailing slash', () => {
+      expect(normalizeGerritHost('https://gerrit.example.com')).toBe('https://gerrit.example.com')
+    })
+
+    test('does not remove slash from path', () => {
+      expect(normalizeGerritHost('https://gerrit.example.com/gerrit')).toBe(
+        'https://gerrit.example.com/gerrit',
+      )
+    })
+
+    test('removes trailing slash from path', () => {
+      expect(normalizeGerritHost('https://gerrit.example.com/gerrit/')).toBe(
+        'https://gerrit.example.com/gerrit',
+      )
+    })
+  })
+
+  describe('whitespace handling', () => {
+    test('trims leading whitespace', () => {
+      expect(normalizeGerritHost('  gerrit.example.com')).toBe('https://gerrit.example.com')
+    })
+
+    test('trims trailing whitespace', () => {
+      expect(normalizeGerritHost('gerrit.example.com  ')).toBe('https://gerrit.example.com')
+    })
+
+    test('trims whitespace from URL with protocol', () => {
+      expect(normalizeGerritHost('  https://gerrit.example.com  ')).toBe(
+        'https://gerrit.example.com',
+      )
+    })
+
+    test('trims whitespace and removes trailing slash', () => {
+      expect(normalizeGerritHost('  gerrit.example.com/  ')).toBe('https://gerrit.example.com')
+    })
+  })
+
+  describe('combined scenarios', () => {
+    test('adds protocol and removes trailing slash', () => {
+      expect(normalizeGerritHost('gerrit.example.com/')).toBe('https://gerrit.example.com')
+    })
+
+    test('trims, adds protocol, and removes trailing slash', () => {
+      expect(normalizeGerritHost('  gerrit.example.com/  ')).toBe('https://gerrit.example.com')
+    })
+
+    test('handles subdomain with port', () => {
+      expect(normalizeGerritHost('review.git.example.com:8443')).toBe(
+        'https://review.git.example.com:8443',
+      )
+    })
+
+    test('handles complex URL with path', () => {
+      expect(normalizeGerritHost('gerrit.example.com/gerrit')).toBe(
+        'https://gerrit.example.com/gerrit',
+      )
+    })
+
+    test('normalizes complete real-world example', () => {
+      expect(normalizeGerritHost('gerrit-review.example.org')).toBe(
+        'https://gerrit-review.example.org',
+      )
+    })
+  })
+
+  describe('edge cases', () => {
+    test('handles empty string', () => {
+      // Empty string becomes 'https:/' after normalization (protocol added, then trailing slash removed)
+      expect(normalizeGerritHost('')).toBe('https:/')
+    })
+
+    test('handles whitespace-only string', () => {
+      // Whitespace-only string becomes 'https:/' after normalization
+      expect(normalizeGerritHost('   ')).toBe('https:/')
+    })
+
+    test('handles just a slash', () => {
+      // Just a slash becomes 'https://' (protocol added to '/', then trailing slash removed leaving '//')
+      expect(normalizeGerritHost('/')).toBe('https://')
+    })
+
+    test('handles protocol only', () => {
+      // Protocol only becomes 'https:/' (trailing slash removed)
+      expect(normalizeGerritHost('https://')).toBe('https:/')
+    })
+  })
+})

package/src/utils/url-parser.ts

@@ -53,6 +53,33 @@ export const extractChangeNumber = (input: string): string => {
   }
 }
 
+/**
+ * Normalizes a Gerrit host URL by adding https:// if no protocol is provided
+ * and removing trailing slashes
+ *
+ * @param host - The host URL to normalize (e.g., "gerrit.example.com" or "https://gerrit.example.com")
+ * @returns The normalized URL with protocol and without trailing slash
+ *
+ * @example
+ * normalizeGerritHost("gerrit.example.com") // returns "https://gerrit.example.com"
+ * normalizeGerritHost("gerrit.example.com:8080") // returns "https://gerrit.example.com:8080"
+ * normalizeGerritHost("http://gerrit.example.com") // returns "http://gerrit.example.com"
+ * normalizeGerritHost("https://gerrit.example.com/") // returns "https://gerrit.example.com"
+ */
+export const normalizeGerritHost = (host: string): string => {
+  let normalized = host.trim()
+
+  // Add https:// if no protocol provided
+  if (!normalized.startsWith('http://') && !normalized.startsWith('https://')) {
+    normalized = `https://${normalized}`
+  }
+
+  // Remove trailing slash
+  normalized = normalized.replace(/\/$/, '')
+
+  return normalized
+}
+
 /**
  * Validates if a string is a valid Gerrit change identifier
  *

package/tests/change-id-formats.test.ts

@@ -76,7 +76,7 @@ ${JSON.stringify(mockChange)}`)
     return HttpResponse.text('Not Found', { status: 404 })
   }),
 
-  http.post('*/a/changes/:changeId/revisions/current/review', async ({ params, request }) => {
+  http.post('*/a/changes/:changeId/revisions/current/review', async ({ params }) => {
     const { changeId } = params
     if (changeId === CHANGE_NUMBER || changeId === CHANGE_ID) {
       return HttpResponse.text(`)]}'

package/tests/setup.test.ts

@@ -1,17 +1,13 @@
 import { describe, test, expect } from 'bun:test'
+import { normalizeGerritHost } from '@/utils/url-parser'
 
 describe('Setup Command', () => {
-  describe('URL processing', () => {
-    test('should remove trailing slashes', () => {
-      const url = 'https://gerrit.example.com/'
-      const normalized = url.replace(/\/$/, '')
-      expect(normalized).toBe('https://gerrit.example.com')
-    })
-
-    test('should handle URLs without trailing slashes', () => {
-      const url = 'https://gerrit.example.com'
-      const normalized = url.replace(/\/$/, '')
-      expect(normalized).toBe('https://gerrit.example.com')
+  describe('URL normalization integration', () => {
+    test('should normalize host URL using normalizeGerritHost', () => {
+      // Test that the utility function is working as expected
+      expect(normalizeGerritHost('gerrit.example.com')).toBe('https://gerrit.example.com')
+      expect(normalizeGerritHost('https://gerrit.example.com/')).toBe('https://gerrit.example.com')
+      expect(normalizeGerritHost('gerrit.example.com:8080')).toBe('https://gerrit.example.com:8080')
     })
   })
 

package/tests/unit/git-branch-detection.test.ts

@@ -1,6 +1,6 @@
 import { describe, test, expect } from 'bun:test'
 import { Effect, Layer } from 'effect'
-import { GitWorktreeService, GitWorktreeServiceLive } from '@/services/git-worktree'
+import { GitWorktreeService } from '@/services/git-worktree'
 
 describe('Git Worktree Creation', () => {
   test('should handle commit-based worktree creation in service interface', async () => {
@@ -11,7 +11,6 @@ describe('Git Worktree Creation', () => {
       validatePreconditions: () => Effect.succeed(undefined),
       createWorktree: (changeId: string) => {
         // Simulate commit-based worktree creation (detached HEAD)
-        const currentCommit = 'abc123def456' // Mock commit hash
         return Effect.succeed({
           path: `/tmp/test-worktree-${changeId}`,
           changeId,

package/tests/unit/services/review-strategy.test.ts

@@ -100,7 +100,7 @@ describe('Review Strategy', () => {
       }
     })
 
-    mockChildProcess.stderr.on.mockImplementation((event: string, callback: Function) => {
+    mockChildProcess.stderr.on.mockImplementation((_event: string, _callback: Function) => {
       // No stderr for success
     })
 
@@ -112,7 +112,7 @@ describe('Review Strategy', () => {
   }
 
   const setupFailedExecution = (exitCode = 1, stderr = 'Command failed') => {
-    mockChildProcess.stdout.on.mockImplementation((event: string, callback: Function) => {
+    mockChildProcess.stdout.on.mockImplementation((_event: string, _callback: Function) => {
       // No stdout for failure
     })