abapgit-agent 1.7.2 → 1.8.1

package/bin/abapgit-agent

@@ -19,1478 +19,48 @@
 const pathModule = require('path');
 const fs = require('fs');
 
-// Get terminal width for responsive table
-const getTermWidth = () => process.stdout.columns || 80;
-const TERM_WIDTH = getTermWidth();
-
-const COOKIE_FILE = '.abapgit_agent_cookies.txt';
-
-/**
- * Convert ISO date formats (YYYY-MM-DD) to ABAP DATS format (YYYYMMDD) in WHERE clause
- * This allows users to use familiar ISO date formats while ensuring compatibility with ABAP SQL
- * @param {string} whereClause - SQL WHERE clause
- * @returns {string} - WHERE clause with dates converted to YYYYMMDD format
- */
-function convertDatesInWhereClause(whereClause) {
-  if (!whereClause) return whereClause;
-
-  // Pattern to match ISO date format: 'YYYY-MM-DD'
-  const isoDatePattern = /'\d{4}-\d{2}-\d{2}'/g;
-
-  return whereClause.replace(isoDatePattern, (match) => {
-    // Extract YYYY, MM, DD from 'YYYY-MM-DD'
-    const dateContent = match.slice(1, -1); // Remove quotes: YYYY-MM-DD
-    const [year, month, day] = dateContent.split('-');
-    // Return in ABAP format: 'YYYYMMDD'
-    return `'${year}${month}${day}'`;
-  });
-}
-
-/**
- * Get CLI version from package.json
- */
-function getCliVersion() {
-  const packageJsonPath = pathModule.join(__dirname, '..', 'package.json');
-  if (fs.existsSync(packageJsonPath)) {
-    const pkg = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-    return pkg.version || '1.0.0';
-  }
-  return '1.0.0';
-}
-
-/**
- * Check version compatibility between CLI and ABAP API
- */
-async function checkVersionCompatibility() {
-  const cliVersion = getCliVersion();
-
-  try {
-    const config = loadConfig();
-    const https = require('https');
-    const url = new URL(`/sap/bc/z_abapgit_agent/health`, `https://${config.host}:${config.sapport}`);
-
-    return new Promise((resolve) => {
-      const options = {
-        hostname: url.hostname,
-        port: url.port,
-        path: url.pathname,
-        method: 'GET',
-        headers: {
-          'Authorization': `Basic ${Buffer.from(`${config.user}:${config.password}`).toString('base64')}`,
-          'sap-client': config.client,
-          'sap-language': config.language || 'EN',
-          'Content-Type': 'application/json'
-        },
-        agent: new https.Agent({ rejectUnauthorized: false })
-      };
-
-      const req = https.request(options, (res) => {
-        let body = '';
-        res.on('data', chunk => body += chunk);
-        res.on('end', () => {
-          try {
-            const result = JSON.parse(body);
-            const apiVersion = result.version || '1.0.0';
-
-            if (cliVersion !== apiVersion) {
-              console.log(`\n⚠️  Version mismatch: CLI ${cliVersion}, ABAP API ${apiVersion}`);
-              console.log('   Some commands may not work correctly.');
-              console.log('   Update ABAP code: abapgit-agent pull\n');
-            }
-            resolve({ cliVersion, apiVersion, compatible: cliVersion === apiVersion });
-          } catch (e) {
-            resolve({ cliVersion, apiVersion: null, compatible: false, error: e.message });
-          }
-        });
-      });
-
-      req.on('error', (e) => {
-        resolve({ cliVersion, apiVersion: null, compatible: false, error: e.message });
-      });
-      req.end();
-    });
-  } catch (error) {
-    return { cliVersion, apiVersion: null, compatible: false, error: error.message };
-  }
-}
-
-/**
- * Load configuration from .abapGitAgent in current working directory
- */
-function loadConfig() {
-  const repoConfigPath = pathModule.join(process.cwd(), '.abapGitAgent');
-
-  if (fs.existsSync(repoConfigPath)) {
-    return JSON.parse(fs.readFileSync(repoConfigPath, 'utf8'));
-  }
-
-  // Fallback to environment variables
-  return {
-    host: process.env.ABAP_HOST,
-    sapport: parseInt(process.env.ABAP_PORT, 10) || 443,
-    client: process.env.ABAP_CLIENT || '100',
-    user: process.env.ABAP_USER,
-    password: process.env.ABAP_PASSWORD,
-    language: process.env.ABAP_LANGUAGE || 'EN',
-    gitUsername: process.env.GIT_USERNAME,
-    gitPassword: process.env.GIT_PASSWORD,
-    transport: process.env.ABAP_TRANSPORT
-  };
-}
-
-/**
- * Get transport request from config or environment
- */
-function getTransport() {
-  const config = loadConfig();
-  return config.transport;
-}
-
-/**
- * Check if ABAP integration is configured for this repo
- */
-function isAbapIntegrationEnabled() {
-  const repoConfigPath = pathModule.join(process.cwd(), '.abapGitAgent');
-  return fs.existsSync(repoConfigPath);
-}
-
-/**
- * Get git remote URL from .git/config
- */
-function getGitRemoteUrl() {
-  const gitConfigPath = pathModule.join(process.cwd(), '.git', 'config');
-
-  if (!fs.existsSync(gitConfigPath)) {
-    return null;
-  }
-
-  const content = fs.readFileSync(gitConfigPath, 'utf8');
-  const remoteMatch = content.match(/\[remote "origin"\]/);
-
-  if (!remoteMatch) {
-    return null;
-  }
-
-  const urlMatch = content.match(/	url = (.+)/);
-  return urlMatch ? urlMatch[1].trim() : null;
-}
-
-/**
- * Get current git branch
- */
-function getGitBranch() {
-  const headPath = pathModule.join(process.cwd(), '.git', 'HEAD');
-
-  if (!fs.existsSync(headPath)) {
-    return 'main';
-  }
-
-  const content = fs.readFileSync(headPath, 'utf8').trim();
-  const match = content.match(/ref: refs\/heads\/(.+)/);
-  return match ? match[1] : 'main';
-}
-
-/**
- * Read cookies from file
- * Supports both Netscape format and simple cookie string format
- */
-function readNetscapeCookies() {
-  const cookiePath = pathModule.join(process.cwd(), COOKIE_FILE);
-  if (!fs.existsSync(cookiePath)) return '';
-
-  const content = fs.readFileSync(cookiePath, 'utf8').trim();
-  if (!content) return '';
-
-  // Check if it's Netscape format (has tabs)
-  if (content.includes('\t')) {
-    const lines = content.split('\n');
-    const cookies = [];
-
-    for (const line of lines) {
-      const trimmed = line.trim();
-      if (!trimmed || (trimmed.startsWith('#') && !trimmed.startsWith('#HttpOnly'))) continue;
-
-      const parts = trimmed.split('\t');
-      if (parts.length >= 7) {
-        cookies.push(`${parts[5]}=${parts[6]}`);
-      }
-    }
-
-    return cookies.join('; ');
-  }
-
-  // Simple format - just return as-is
-  return content;
-}
-
-/**
- * Save cookies to Netscape format cookie file
- */
-function saveCookies(cookies, host) {
-  const cookiePath = pathModule.join(process.cwd(), COOKIE_FILE);
-
-  // Parse cookies and convert to Netscape format
-  const cookieList = cookies.split(';').map(c => c.trim()).filter(Boolean);
-  const lines = ['# Netscape HTTP Cookie File', '# https://curl.se/docs/http-cookies.html', '# This file was generated by libcurl! Edit at your own risk.'];
-
-  for (const cookie of cookieList) {
-    const [name, ...valueParts] = cookie.split('=');
-    const value = valueParts.join('=');
-
-    // HttpOnly cookies get #HttpOnly_ prefix
-    const domain = name.includes('SAP') || name.includes('MYSAP') || name.includes('SAPLOGON')
-      ? `#HttpOnly_${host}`
-      : host;
-
-    // Format: domain, flag, path, secure, expiration, name, value
-    lines.push(`${domain}\tFALSE\t/\tFALSE\t0\t${name.trim()}\t${value}`);
-  }
-
-  fs.writeFileSync(cookiePath, lines.join('\n'));
-}
-
-/**
- * Fetch CSRF token using GET /health with X-CSRF-Token: fetch
- */
-async function fetchCsrfToken(config) {
-  const https = require('https');
-  const url = new URL(`/sap/bc/z_abapgit_agent/health`, `https://${config.host}:${config.sapport}`);
-
-  return new Promise((resolve, reject) => {
-    // Clear stale cookies before fetching new token
-    const cookiePath = pathModule.join(process.cwd(), COOKIE_FILE);
-    if (fs.existsSync(cookiePath)) {
-      fs.unlinkSync(cookiePath);
-    }
-
-    const cookieHeader = readNetscapeCookies();
-
-    const options = {
-      hostname: url.hostname,
-      port: url.port,
-      path: url.pathname,
-      method: 'GET',
-      headers: {
-        'Authorization': `Basic ${Buffer.from(`${config.user}:${config.password}`).toString('base64')}`,
-        'sap-client': config.client,
-        'sap-language': config.language || 'EN',
-        'X-CSRF-Token': 'fetch',
-        'Content-Type': 'application/json',
-        ...(cookieHeader && { 'Cookie': cookieHeader })
-      },
-      agent: new https.Agent({ rejectUnauthorized: false })
-    };
-
-    const req = https.request(options, (res) => {
-      const csrfToken = res.headers['x-csrf-token'];
-
-      // Save new cookies from response
-      const setCookie = res.headers['set-cookie'];
-      if (setCookie) {
-        const cookies = Array.isArray(setCookie)
-          ? setCookie.map(c => c.split(';')[0]).join('; ')
-          : setCookie.split(';')[0];
-        saveCookies(cookies, config.host);
-      }
-
-      let body = '';
-      res.on('data', chunk => body += chunk);
-      res.on('end', () => {
-        resolve(csrfToken);
-      });
-    });
-
-    req.on('error', reject);
-    req.end();
-  });
-}
-
-/**
- * Make HTTP request to ABAP REST endpoint
- */
-function request(method, urlPath, data = null, options = {}) {
-  return new Promise((resolve, reject) => {
-    const https = require('https');
-    const http = require('http');
-    const config = loadConfig();
-    const url = new URL(urlPath, `https://${config.host}:${config.sapport}`);
-
-    const headers = {
-      'Content-Type': 'application/json',
-      'sap-client': config.client,
-      'sap-language': config.language || 'EN',
-      ...options.headers
-    };
-
-    // Add authorization
-    headers['Authorization'] = `Basic ${Buffer.from(`${config.user}:${config.password}`).toString('base64')}`;
-
-    // Add CSRF token for POST
-    if (method === 'POST' && options.csrfToken) {
-      headers['X-CSRF-Token'] = options.csrfToken;
-    }
-
-    // Add cookies if available
-    const cookieHeader = readNetscapeCookies();
-    if (cookieHeader) {
-      headers['Cookie'] = cookieHeader;
-    }
-
-    const reqOptions = {
-      hostname: url.hostname,
-      port: url.port,
-      path: url.pathname,
-      method,
-      headers,
-      agent: new https.Agent({ rejectUnauthorized: false })
-    };
-
-    const req = (url.protocol === 'https:' ? https : http).request(reqOptions, (res) => {
-      let body = '';
-      res.on('data', chunk => body += chunk);
-      res.on('end', () => {
-        try {
-          // Handle unescaped newlines from ABAP - replace actual newlines with \n
-          const cleanedBody = body.replace(/\n/g, '\\n');
-          resolve(JSON.parse(cleanedBody));
-        } catch (e) {
-          // Fallback: try to extract JSON from response
-          const jsonMatch = body.match(/\{[\s\S]*\}/);
-          if (jsonMatch) {
-            try {
-              resolve(JSON.parse(jsonMatch[0].replace(/\n/g, '\\n')));
-            } catch (e2) {
-              resolve({ raw: body, error: e2.message });
-            }
-          } else {
-            resolve({ raw: body, error: e.message });
-          }
-        }
-      });
-    });
-
-    req.on('error', reject);
-
-    if (data) {
-      req.write(JSON.stringify(data));
-    }
-    req.end();
-  });
-}
-
-/**
- * Inspect ABAP source file
- * Reads file content and sends to /inspect
- */
-async function syntaxCheckSource(sourceFile, csrfToken, config) {
-  console.log(`  Syntax check for file: ${sourceFile}`);
-
-  try {
-    // Read file content
-    const absolutePath = pathModule.isAbsolute(sourceFile)
-      ? sourceFile
-      : pathModule.join(process.cwd(), sourceFile);
-
-    if (!fs.existsSync(absolutePath)) {
-      console.error(`  ❌ File not found: ${absolutePath}`);
-      return;
-    }
-
-    const fileContent = fs.readFileSync(absolutePath, 'utf8');
-
-    // Extract source name from file path (basename with extension)
-    // e.g., "zcl_my_class.clas.abap" -> "ZCL_MY_CLASS.CLASS.ABAP"
-    const sourceName = pathModule.basename(sourceFile).toUpperCase();
-
-    // Send files array to syntax-check endpoint
-    const data = {
-      files: [sourceName]
-    };
-
-    const result = await request('POST', '/sap/bc/z_abapgit_agent/inspect', data, { csrfToken: csrfToken });
-
-    // Handle new table-based result format (array) or old single result
-    let results = [];
-    if (Array.isArray(result)) {
-      results = result;
-    } else {
-      // Convert old single result to array format for compatibility
-      results = [{
-        OBJECT_TYPE: 'UNKNOWN',
-        OBJECT_NAME: pathModule.basename(sourceFile),
-        SUCCESS: result.SUCCESS !== undefined ? result.SUCCESS === 'X' || result.SUCCESS === true : result.success,
-        ERROR_COUNT: result.ERROR_COUNT || result.error_count || 0,
-        ERRORS: result.ERRORS || result.errors || [],
-        WARNINGS: result.warnings || []
-      }];
-    }
-
-    // Process each result
-    let hasErrors = false;
-    for (const res of results) {
-      // Handle both uppercase and lowercase keys
-      const success = res.SUCCESS !== undefined ? res.SUCCESS : res.success;
-      const objectType = res.OBJECT_TYPE !== undefined ? res.OBJECT_TYPE : res.object_type;
-      const objectName = res.OBJECT_NAME !== undefined ? res.OBJECT_NAME : res.object_name;
-      const errorCount = res.ERROR_COUNT !== undefined ? res.ERROR_COUNT : (res.error_count || 0);
-      const errors = res.ERRORS !== undefined ? res.ERRORS : (res.errors || []);
-      const warnings = res.WARNINGS !== undefined ? res.WARNINGS : (res.warnings || []);
-
-      console.log('\n');
-
-      if (errorCount > 0 || warnings.length > 0) {
-        if (errorCount > 0) {
-          console.log(`❌ ${objectType} ${objectName} - Syntax check failed (${errorCount} error(s)):`);
-        } else {
-          console.log(`⚠️  ${objectType} ${objectName} - Syntax check passed with warnings (${warnings.length}):`);
-        }
-        console.log('\nErrors:');
-        console.log('─'.repeat(60));
-
-        for (const err of errors) {
-          const line = err.LINE || err.line || '?';
-          const column = err.COLUMN || err.column || '?';
-          const text = err.TEXT || err.text || 'Unknown error';
-
-          console.log(`  Line ${line}, Column ${column}:`);
-          console.log(`    ${text}`);
-          console.log('');
-        }
-
-        // Show warnings if any
-        if (warnings.length > 0) {
-          console.log('Warnings:');
-          console.log('─'.repeat(60));
-          for (const warn of warnings) {
-            const line = warn.LINE || warn.line || '?';
-            const text = warn.MESSAGE || warn.message || 'Unknown warning';
-            console.log(`  Line ${line}: ${text}`);
-          }
-        }
-        hasErrors = true;
-      } else if (success === true || success === 'X') {
-        console.log(`✅ ${objectType} ${objectName} - Syntax check passed`);
-      } else {
-        console.log(`⚠️  ${objectType} ${objectName} - Syntax check returned unexpected status`);
-      }
-    }
-
-    return result;
-  } catch (error) {
-    console.error(`\n  Error: ${error.message}`);
-    process.exit(1);
-  }
-}
-
-/**
- * Inspect all files in one request
- */
-async function inspectAllFiles(files, csrfToken, config, variant = null) {
-  // Convert files to uppercase names (same as syntaxCheckSource does)
-  const fileNames = files.map(f => {
-    const baseName = pathModule.basename(f).toUpperCase();
-    return baseName;
-  });
-
-  try {
-    // Send all files in one request
-    const data = {
-      files: fileNames
-    };
-
-    // Add variant if specified
-    if (variant) {
-      data.variant = variant;
-    }
-
-    const result = await request('POST', '/sap/bc/z_abapgit_agent/inspect', data, { csrfToken: csrfToken });
-
-    // Handle both table result and old single result
-    let results = [];
-    if (Array.isArray(result)) {
-      results = result;
-    } else {
-      // Convert single result to array format
-      results = [{
-        OBJECT_TYPE: 'UNKNOWN',
-        OBJECT_NAME: files.join(', '),
-        SUCCESS: result.SUCCESS !== undefined ? result.SUCCESS === 'X' || result.SUCCESS === true : result.success,
-        ERROR_COUNT: result.ERROR_COUNT || result.error_count || 0,
-        ERRORS: result.ERRORS || result.errors || [],
-        WARNINGS: result.warnings || []
-      }];
-    }
-
-    return results;
-  } catch (error) {
-    console.error(`\n  Error: ${error.message}`);
-    process.exit(1);
-  }
-}
-
-/**
- * Process a single inspect result
- */
-async function processInspectResult(res) {
-  // Handle both uppercase and lowercase keys
-  const success = res.SUCCESS !== undefined ? res.SUCCESS : res.success;
-  const objectType = res.OBJECT_TYPE !== undefined ? res.OBJECT_TYPE : res.object_type;
-  const objectName = res.OBJECT_NAME !== undefined ? res.OBJECT_NAME : res.object_name;
-  const errorCount = res.ERROR_COUNT !== undefined ? res.ERROR_COUNT : (res.error_count || 0);
-  const errors = res.ERRORS !== undefined ? res.ERRORS : (res.errors || []);
-  const warnings = res.WARNINGS !== undefined ? res.WARNINGS : (res.warnings || []);
-  const infos = res.INFOS !== undefined ? res.INFOS : (res.infos || []);
-
-  if (errorCount > 0 || warnings.length > 0 || infos.length > 0) {
-    if (errorCount > 0) {
-      console.log(`❌ ${objectType} ${objectName} - Syntax check failed (${errorCount} error(s)):`);
-    } else {
-      const total = warnings.length + infos.length;
-      console.log(`⚠️  ${objectType} ${objectName} - Syntax check passed with warnings (${total}):`);
-    }
-    console.log('\nErrors:');
-    console.log('─'.repeat(60));
-
-    for (const err of errors) {
-      const line = err.LINE || err.line || '?';
-      const column = err.COLUMN || err.column || '?';
-      const text = err.TEXT || err.text || 'Unknown error';
-      const methodName = err.METHOD_NAME || err.method_name;
-      const sobjname = err.SOBJNAME || err.sobjname;
-
-      if (methodName) {
-        console.log(`  Method: ${methodName}`);
-      }
-      console.log(`  Line ${line}, Column ${column}:`);
-      if (sobjname && sobjname.includes('====')) {
-        console.log(`    Include: ${sobjname}`);
-      }
-      console.log(`    ${text}`);
-      console.log('');
-    }
-
-    // Show warnings if any
-    if (warnings.length > 0) {
-      console.log('Warnings:');
-      console.log('─'.repeat(60));
-      for (const warn of warnings) {
-        const line = warn.LINE || warn.line || '?';
-        const text = warn.MESSAGE || warn.message || 'Unknown warning';
-        const methodName = warn.METHOD_NAME || warn.method_name;
-        const sobjname = warn.SOBJNAME || warn.sobjname;
-
-        if (methodName) {
-          console.log(`  Method: ${methodName}`);
-        }
-        console.log(`  Line ${line}:`);
-        if (sobjname && sobjname.includes('====')) {
-          console.log(`    Include: ${sobjname}`);
-        }
-        console.log(`    ${text}`);
-      }
-    }
-
-    // Show infos if any
-    if (infos.length > 0) {
-      console.log('Info:');
-      console.log('─'.repeat(60));
-      for (const info of infos) {
-        const line = info.LINE || info.line || '?';
-        const text = info.MESSAGE || info.message || 'Unknown info';
-        const methodName = info.METHOD_NAME || info.method_name;
-        const sobjname = info.SOBJNAME || info.sobjname;
-
-        if (methodName) {
-          console.log(`  Method: ${methodName}`);
-        }
-        console.log(`  Line ${line}:`);
-        if (sobjname && sobjname.includes('====')) {
-          console.log(`    Include: ${sobjname}`);
-        }
-        console.log(`    ${text}`);
-      }
-    }
-  } else if (success === true || success === 'X') {
-    console.log(`✅ ${objectType} ${objectName} - Syntax check passed`);
-  } else {
-    console.log(`⚠️  ${objectType} ${objectName} - Syntax check returned unexpected status`);
-  }
-}
-
-/**
- * Run unit tests for package or objects
- */
-async function runUnitTests(options) {
-  console.log('\n  Running unit tests');
-
-  try {
-    const config = loadConfig();
-
-    // Fetch CSRF token first
-    const csrfToken = await fetchCsrfToken(config);
-
-    const data = {};
-
-    if (options.package) {
-      data.package = options.package;
-    }
-
-    if (options.objects && options.objects.length > 0) {
-      data.objects = options.objects;
-    }
-
-    const result = await request('POST', '/sap/bc/z_abapgit_agent/unit', data, { csrfToken });
-
-    // Display raw result for debugging
-    if (process.env.DEBUG) {
-      console.log('Raw result:', JSON.stringify(result, null, 2));
-    }
-
-    // Handle uppercase keys from ABAP
-    const success = result.SUCCESS || result.success;
-    const testCount = result.TEST_COUNT || result.test_count || 0;
-    const passedCount = result.PASSED_COUNT || result.passed_count || 0;
-    const failedCount = result.FAILED_COUNT || result.failed_count || 0;
-    const message = result.MESSAGE || result.message || '';
-    const errors = result.ERRORS || result.errors || [];
-
-    console.log('\n');
-
-    if (success === 'X' || success === true) {
-      console.log(`✅ All tests passed!`);
-    } else {
-      console.log(`❌ Some tests failed`);
-    }
-
-    console.log(`   ${message}`);
-    console.log(`   Tests: ${testCount} | Passed: ${passedCount} | Failed: ${failedCount}`);
-
-    if (failedCount > 0 && errors.length > 0) {
-      console.log('\nFailed Tests:');
-      console.log('─'.repeat(60));
-
-      for (const err of errors) {
-        const className = err.CLASS_NAME || err.class_name || '?';
-        const methodName = err.METHOD_NAME || err.method_name || '?';
-        const errorKind = err.ERROR_KIND || err.error_kind || '';
-        const errorText = err.ERROR_TEXT || err.error_text || 'Unknown error';
-
-        console.log(`   ✗ ${className}=>${methodName}`);
-        if (errorKind) {
-          console.log(`     Kind: ${errorKind}`);
-        }
-        console.log(`     Error: ${errorText}`);
-        console.log('');
-      }
-    }
-
-    return result;
-  } catch (error) {
-    console.error(`\n  Error: ${error.message}`);
-    process.exit(1);
-  }
-}
-
-/**
- * Run unit test for a single file
- */
-async function runUnitTestForFile(sourceFile, csrfToken, config, coverage = false) {
-  console.log(`  Running unit test for: ${sourceFile}`);
-
-  try {
-    // Read file content
-    const absolutePath = pathModule.isAbsolute(sourceFile)
-      ? sourceFile
-      : pathModule.join(process.cwd(), sourceFile);
-
-    if (!fs.existsSync(absolutePath)) {
-      console.error(`  ❌ File not found: ${absolutePath}`);
-      return;
-    }
-
-    // Extract object type and name from file path
-    // e.g., "zcl_my_test.clas.abap" -> CLAS, ZCL_MY_TEST
-    const fileName = pathModule.basename(sourceFile).toUpperCase();
-    const parts = fileName.split('.');
-    if (parts.length < 3) {
-      console.error(`  ❌ Invalid file format: ${sourceFile}`);
-      return;
-    }
-
-    // obj_name is first part (may contain path), obj_type is second part
-    const objType = parts[1] === 'CLASS' ? 'CLAS' : parts[1];
-    let objName = parts[0];
-
-    // Handle subdirectory paths
-    const lastSlash = objName.lastIndexOf('/');
-    if (lastSlash >= 0) {
-      objName = objName.substring(lastSlash + 1);
-    }
-
-    // Send files array to unit endpoint (ABAP expects string_table of file names)
-    const data = {
-      files: [sourceFile],
-      coverage: coverage
-    };
-
-    const result = await request('POST', '/sap/bc/z_abapgit_agent/unit', data, { csrfToken });
-
-    // Handle uppercase keys from ABAP
-    const success = result.SUCCESS || result.success;
-    const testCount = result.TEST_COUNT || result.test_count || 0;
-    const passedCount = result.PASSED_COUNT || result.passed_count || 0;
-    const failedCount = result.FAILED_COUNT || result.failed_count || 0;
-    const message = result.MESSAGE || result.message || '';
-    const errors = result.ERRORS || result.errors || [];
-
-    // Handle coverage data
-    const coverageStats = result.COVERAGE_STATS || result.coverage_stats;
-
-    if (testCount === 0) {
-      console.log(`  ➖ ${objName} - No unit tests`);
-    } else if (success === 'X' || success === true) {
-      console.log(`  ✅ ${objName} - All tests passed`);
-    } else {
-      console.log(`  ❌ ${objName} - Tests failed`);
-    }
-
-    console.log(`     Tests: ${testCount} | Passed: ${passedCount} | Failed: ${failedCount}`);
-
-    // Display coverage if available
-    if (coverage && coverageStats) {
-      const totalLines = coverageStats.TOTAL_LINES || coverageStats.total_lines || 0;
-      const coveredLines = coverageStats.COVERED_LINES || coverageStats.covered_lines || 0;
-      const coverageRate = coverageStats.COVERAGE_RATE || coverageStats.coverage_rate || 0;
-
-      console.log(`     📊 Coverage: ${coverageRate}%`);
-      console.log(`        Total Lines: ${totalLines}`);
-      console.log(`        Covered Lines: ${coveredLines}`);
-    }
-
-    if (failedCount > 0 && errors.length > 0) {
-      for (const err of errors) {
-        const className = err.CLASS_NAME || err.class_name || '?';
-        const methodName = err.METHOD_NAME || err.method_name || '?';
-        const errorText = err.ERROR_TEXT || err.error_text || 'Unknown error';
-        console.log(`     ✗ ${className}=>${methodName}: ${errorText}`);
-      }
-    }
-
-    return result;
-  } catch (error) {
-    console.error(`\n  Error: ${error.message}`);
-  }
-}
-
-/**
- * Run tree command and return raw result
- */
-async function runTreeCommand(packageName, depth, includeTypes, csrfToken, config) {
-  const data = {
-    package: packageName,
-    depth: depth,
-    include_objects: includeTypes
-  };
-
-  return await request('POST', '/sap/bc/z_abapgit_agent/tree', data, { csrfToken });
-}
-
-/**
- * Display tree output in human-readable format
- */
-async function displayTreeOutput(packageName, depth, includeTypes) {
-  const config = loadConfig();
-  const csrfToken = await fetchCsrfToken(config);
-
-  console.log(`\n  Getting package tree for: ${packageName}`);
-
-  const result = await runTreeCommand(packageName, depth, includeTypes, csrfToken, config);
-
-  // Handle uppercase keys from ABAP
-  const success = result.SUCCESS || result.success;
-  const error = result.ERROR || result.error;
-
-  if (!success || error) {
-    console.error(`\n  ❌ Error: ${error || 'Failed to get tree'}`);
-    return;
-  }
-
-  // Parse hierarchy structure (ABAP returns flat nodes with parent refs)
-  const nodes = result.NODES || result.nodes || [];
-  const rootPackage = result.PACKAGE || result.package || packageName;
-  const parentPackage = result.PARENT_PACKAGE || result.parent_package;
-  const totalPackages = result.TOTAL_PACKAGES || result.total_packages || 0;
-  const totalObjects = result.TOTAL_OBJECTS || result.total_objects || 0;
-  const objectTypes = result.OBJECTS || result.objects || [];
-
-  console.log(`\n  Package Tree: ${rootPackage}`);
-
-  // Display parent info if available
-  if (parentPackage && parentPackage !== rootPackage) {
-    console.log(`  ⬆️  Parent: ${parentPackage}`);
-  }
-
-  console.log('');
-
-  // Build and display tree from flat nodes list
-  const lines = buildTreeLinesFromNodes(nodes, '', true);
-  for (const line of lines) {
-    console.log(`  ${line}`);
-  }
-
-  console.log('');
-  console.log('  Summary');
-  console.log(`  PACKAGES: ${totalPackages}`);
-  console.log(`  OBJECTS: ${totalObjects}`);
-
-  // Display object types if available
-  if (includeTypes && objectTypes.length > 0) {
-    const typeStr = objectTypes.map(t => `${t.OBJECT || t.object}=${t.COUNT || t.count}`).join(' ');
-    console.log(`  TYPES: ${typeStr}`);
-  }
-}
-
-/**
- * Build tree display lines from flat nodes list
- */
-function buildTreeLinesFromNodes(nodes, prefix, isLast) {
-  const lines = [];
-
-  if (nodes.length === 0) {
-    return lines;
-  }
-
-  // First node is the root
-  const root = nodes[0];
-  const icon = '📦';
-  lines.push(`${prefix}${isLast ? '└─ ' : '├─ '} ${icon} ${root.PACKAGE || root.package}`);
-
-  // Get children (nodes with depth > 0, grouped by depth)
-  const children = nodes.filter(n => (n.DEPTH || n.depth) > 0);
-
-  // Group children by depth
-  const byDepth = {};
-  children.forEach(n => {
-    const d = n.DEPTH || n.depth;
-    if (!byDepth[d]) byDepth[d] = [];
-    byDepth[d].push(n);
-  });
-
-  // Process depth 1 children
-  const depth1 = byDepth[1] || [];
-  const newPrefix = prefix + (isLast ? '   ' : '│  ');
-
-  for (let i = 0; i < depth1.length; i++) {
-    const child = depth1[i];
-    const isSubLast = i === depth1.length - 1;
-    const childLines = buildChildLines(child, newPrefix, isSubLast, byDepth);
-    lines.push(...childLines);
-  }
-
-  return lines;
-}
-
-function buildChildLines(node, prefix, isLast, byDepth) {
-  const lines = [];
-  const icon = '📦';
-  lines.push(`${prefix}${isLast ? '└─ ' : '├─ '} ${icon} ${node.PACKAGE || node.package}`);
-
-  const newPrefix = prefix + (isLast ? '   ' : '│  ');
-  const nodeDepth = node.DEPTH || node.depth;
-  const children = byDepth[nodeDepth + 1] || [];
-
-  // Find children of this node
-  const myChildren = children.filter(n => (n.PARENT || n.parent) === (node.PACKAGE || node.package));
-
-  for (let i = 0; i < myChildren.length; i++) {
-    const child = myChildren[i];
-    const isSubLast = i === myChildren.length - 1;
-    const childLines = buildChildLines(child, newPrefix, isSubLast, byDepth);
-    lines.push(...childLines);
-  }
-
-  return lines;
-}
-
-/**
- * Pull and activate repository
- */
-async function pull(gitUrl, branch = 'main', files = null, transportRequest = null) {
-  console.log(`\n🚀 Starting pull for: ${gitUrl}`);
-  console.log(`   Branch: ${branch}`);
-  if (files && files.length > 0) {
-    console.log(`   Files: ${files.join(', ')}`);
-  }
-  if (transportRequest) {
-    console.log(`   Transport Request: ${transportRequest}`);
-  }
-
-  try {
-    const config = loadConfig();
-
-    // Fetch CSRF token first
-    const csrfToken = await fetchCsrfToken(config);
-
-    // Prepare request data with git credentials
-    const data = {
-      url: gitUrl,
-      branch: branch,
-      username: config.gitUsername,
-      password: config.gitPassword
-    };
-
-    // Add files array if specified
-    if (files && files.length > 0) {
-      data.files = files;
-    }
-
-    // Add transport request if specified
-    if (transportRequest) {
-      data.transport_request = transportRequest;
-    }
-
-    const result = await request('POST', '/sap/bc/z_abapgit_agent/pull', data, { csrfToken });
-
-    console.log('\n');
-
-    // Display raw result for debugging
-    if (process.env.DEBUG) {
-      console.log('Raw result:', JSON.stringify(result, null, 2));
-    }
-
-    // Handle uppercase keys from ABAP
-    const success = result.SUCCESS || result.success;
-    const jobId = result.JOB_ID || result.job_id;
-    const message = result.MESSAGE || result.message;
-    const errorDetail = result.ERROR_DETAIL || result.error_detail;
-    const transportRequestUsed = result.TRANSPORT_REQUEST || result.transport_request;
-    const activatedCount = result.ACTIVATED_COUNT || result.activated_count || 0;
-    const failedCount = result.FAILED_COUNT || result.failed_count || 0;
-    const logMessages = result.LOG_MESSAGES || result.log_messages || [];
-    const activatedObjects = result.ACTIVATED_OBJECTS || result.activated_objects || [];
-    const failedObjects = result.FAILED_OBJECTS || result.failed_objects || [];
-
-    // Icon mapping for message types
-    const getIcon = (type) => {
-      const icons = {
-        'S': '✅',  // Success
-        'E': '❌',  // Error
-        'W': '⚠️',  // Warning
-        'A': '🛑'   // Abort
-      };
-      return icons[type] || '';
-    };
-
-    // Calculate display width accounting for emoji (2 cells) vs ASCII (1 cell)
-    const calcWidth = (str) => {
-      if (!str) return 0;
-      let width = 0;
-      let i = 0;
-      while (i < str.length) {
-        const code = str.codePointAt(i);
-        if (!code) break;
-        // Variation selectors (FE00-FE0F) and ZWJ (200D) take 0 width
-        if (code >= 0xFE00 && code <= 0xFE0F) {
-          i += 1;
-          continue;
-        }
-        if (code === 0x200D) {  // Zero width joiner
-          i += 1;
-          continue;
-        }
-        // Emoji and wide characters take 2 cells
-        if (code > 0xFFFF) {
-          width += 2;
-          i += 2;  // Skip surrogate pair
-        } else if (code > 127) {
-          width += 2;
-          i += 1;
-        } else {
-          width += 1;
-          i += 1;
-        }
-      }
-      return width;
-    };
-
-    // Pad string to display width
-    const padToWidth = (str, width) => {
-      const s = str || '';
-      const currentWidth = calcWidth(s);
-      return s + ' '.repeat(Math.max(0, width - currentWidth));
-    };
-
-    if (success === 'X' || success === true) {
-      console.log(`✅ Pull completed successfully!`);
-      console.log(`   Job ID: ${jobId || 'N/A'}`);
-      console.log(`   Message: ${message || 'N/A'}`);
-    } else {
-      console.log(`❌ Pull completed with errors!`);
-      console.log(`   Job ID: ${jobId || 'N/A'}`);
-      console.log(`   Message: ${message || 'N/A'}`);
-    }
-
-    // Display error detail if available
-    if (errorDetail && errorDetail.trim()) {
-      console.log(`\n📋 Error Details:`);
-      console.log('─'.repeat(TERM_WIDTH));
-      // Handle escaped newlines from ABAP JSON
-      const formattedDetail = errorDetail.replace(/\\n/g, '\n');
-      console.log(formattedDetail);
-    }
-
-    // Display all messages as table (from log_messages)
-    if (logMessages && logMessages.length > 0) {
-      console.log(`\n📋 Pull Log (${logMessages.length} messages):`);
-
-      // Calculate column widths based on terminal width
-      const tableWidth = Math.min(TERM_WIDTH, 120);
-      const iconCol = 4;  // Fixed width for icon column
-      const objCol = 28;
-      const msgCol = tableWidth - iconCol - objCol - 6; // Account for vertical lines (3 chars)
-
-      // Pad string to display width using calcWidth for emoji support
-      const padToWidth = (str, width) => {
-        const s = str || '';
-        const currentWidth = calcWidth(s);
-        return s + ' '.repeat(Math.max(0, width - currentWidth));
-      };
-
-      const headerLine = '─'.repeat(iconCol) + '┼' + '─'.repeat(objCol) + '┼' + '─'.repeat(msgCol);
-      const headerText = padToWidth('Icon', iconCol) + '│' + padToWidth('Object', objCol) + '│' + 'Message'.substring(0, msgCol);
-      const borderLine = '─'.repeat(tableWidth);
-
-      console.log(borderLine);
-      console.log(headerText);
-      console.log(headerLine);
-
-      for (const msg of logMessages) {
-        const icon = getIcon(msg.TYPE);
-        const objType = msg.OBJ_TYPE || '';
-        const objName = msg.OBJ_NAME || '';
-        const obj = objType && objName ? `${objType} ${objName}` : '';
-        const text = msg.TEXT || '';
-
-        // Truncate long text
-        const displayText = text.length > msgCol ? text.substring(0, msgCol - 3) + '...' : text;
-
-        console.log(padToWidth(icon, iconCol) + '│' + padToWidth(obj.substring(0, objCol), objCol) + '│' + displayText);
-      }
-    }
-
-    // Extract objects with errors from log messages (type 'E' = Error)
-    const failedObjectsFromLog = [];
-    const objectsWithErrors = new Set();
-
-    for (const msg of logMessages) {
-      if (msg.TYPE === 'E' && msg.OBJ_TYPE && msg.OBJ_NAME) {
-        const objKey = `${msg.OBJ_TYPE} ${msg.OBJ_NAME}`;
-        objectsWithErrors.add(objKey);
-        failedObjectsFromLog.push({
-          OBJ_TYPE: msg.OBJ_TYPE,
-          OBJ_NAME: msg.OBJ_NAME,
-          TEXT: msg.TEXT || 'Activation failed',
-          EXCEPTION: msg.EXCEPTION || ''
-        });
-      }
-    }
-
-    // Filter activated objects - only include objects without errors
-    const filteredActivatedObjects = activatedObjects.filter(obj => {
-      const objKey = obj.OBJ_TYPE && obj.OBJ_NAME ? `${obj.OBJ_TYPE} ${obj.OBJ_NAME}` : '';
-      return objKey && !objectsWithErrors.has(objKey);
-    });
-
-    // Display unique activated objects (excluding objects with errors)
-    if (filteredActivatedObjects && filteredActivatedObjects.length > 0) {
-      console.log(`\n📦 Activated Objects (${filteredActivatedObjects.length}):`);
-      console.log('─'.repeat(TERM_WIDTH));
-      for (const obj of filteredActivatedObjects) {
-        console.log(`✅ ${obj.OBJ_TYPE} ${obj.OBJ_NAME}`);
-      }
-    }
-
-    // Display failed objects log (all error messages, duplicates allowed)
-    if (failedObjectsFromLog.length > 0) {
-      console.log(`\n❌ Failed Objects Log (${failedObjectsFromLog.length} entries):`);
-      console.log('─'.repeat(TERM_WIDTH));
-      for (const obj of failedObjectsFromLog) {
-        const objKey = obj.OBJ_TYPE && obj.OBJ_NAME ? `${obj.OBJ_TYPE} ${obj.OBJ_NAME}` : '';
-        let text = obj.TEXT || 'Activation failed';
-        // Include exception text if available
-        if (obj.EXCEPTION && obj.EXCEPTION.trim()) {
-          text = `${text}\nException: ${obj.EXCEPTION}`;
-        }
-        console.log(`❌ ${objKey}: ${text}`);
-      }
-    } else if (failedObjects && failedObjects.length > 0) {
-      // Fallback to API failed_objects if no errors in log
-      console.log(`\n❌ Failed Objects Log (${failedObjects.length}):`);
-      console.log('─'.repeat(TERM_WIDTH));
-      for (const obj of failedObjects) {
-        const objKey = obj.OBJ_TYPE && obj.OBJ_NAME ? `${obj.OBJ_TYPE} ${obj.OBJ_NAME}` : '';
-        let text = obj.TEXT || 'Activation failed';
-        // Include exception text if available
-        if (obj.EXCEPTION && obj.EXCEPTION.trim()) {
-          text = `${text}\nException: ${obj.EXCEPTION}`;
-        }
-        console.log(`❌ ${objKey}: ${text}`);
-      }
-    } else if (failedCount > 0) {
-      console.log(`\n❌ Failed Objects Log (${failedCount})`);
-    }
-
-    return result;
-  } catch (error) {
-    console.error(`\n❌ Error: ${error.message}`);
-    process.exit(1);
-  }
-}
-
-/**
- * Copy a file if source exists (helper for init --update)
- * @param {string} srcPath - Source file path
- * @param {string} destPath - Destination file path
- * @param {string} label - Label for console output
- * @param {boolean} createParentDir - Whether to create parent directory
- * @returns {Promise<void>}
- */
-async function copyFileIfExists(srcPath, destPath, label, createParentDir = false) {
-  try {
-    if (fs.existsSync(srcPath)) {
-      if (createParentDir) {
-        const parentDir = pathModule.dirname(destPath);
-        if (!fs.existsSync(parentDir)) {
-          fs.mkdirSync(parentDir, { recursive: true });
-        }
-      }
-      fs.copyFileSync(srcPath, destPath);
-      console.log(`✅ Updated ${label}`);
-    } else {
-      console.log(`⚠️  ${label} not found in abapgit-agent`);
-    }
-  } catch (error) {
-    console.error(`Error copying ${label}: ${error.message}`);
-  }
-}
-
-/**
- * Copy guidelines folder (helper for init --update)
- * @param {string} srcPath - Source folder path
- * @param {string} destPath - Destination folder path
- * @param {boolean} overwrite - Whether to overwrite existing files
- * @returns {Promise<void>}
- */
-async function copyGuidelinesFolder(srcPath, destPath, overwrite = false) {
-  try {
-    if (fs.existsSync(srcPath)) {
-      // Create destination directory if needed
-      if (!fs.existsSync(destPath)) {
-        fs.mkdirSync(destPath, { recursive: true });
-      }
-
-      const files = fs.readdirSync(srcPath);
-      let copied = 0;
-
-      for (const file of files) {
-        if (file.endsWith('.md')) {
-          const srcFile = pathModule.join(srcPath, file);
-          const destFile = pathModule.join(destPath, file);
-
-          // Skip if file exists and not overwriting
-          if (fs.existsSync(destFile) && !overwrite) {
-            continue;
-          }
-
-          fs.copyFileSync(srcFile, destFile);
-          copied++;
-        }
-      }
-
-      if (copied > 0) {
-        console.log(`✅ Updated guidelines/ (${copied} files)`);
-      } else {
-        console.log(`⚠️  No guideline files found`);
-      }
-    } else {
-      console.log(`⚠️  guidelines folder not found in abapgit-agent`);
-    }
-  } catch (error) {
-    console.error(`Error copying guidelines: ${error.message}`);
-  }
-}
-
-/**
- * Run init command - Initialize local configuration
- */
-async function runInit(args) {
-  const folderArgIndex = args.indexOf('--folder');
-  const packageArgIndex = args.indexOf('--package');
-  const updateMode = args.includes('--update');
-
-  // Get parameters
-  let folder = folderArgIndex !== -1 && folderArgIndex + 1 < args.length
-    ? args[folderArgIndex + 1]
-    : '/src/';
-
-  // Normalize folder path: ensure it starts with / and ends with /
-  folder = folder.trim();
-  if (!folder.startsWith('/')) {
-    folder = '/' + folder;
-  }
-  if (!folder.endsWith('/')) {
-    folder = folder + '/';
-  }
-
-  const packageName = packageArgIndex !== -1 && packageArgIndex + 1 < args.length
-    ? args[packageArgIndex + 1]
-    : null;
-
-  // Validate package is required for non-update mode
-  if (!updateMode && !packageName) {
-    console.error('Error: --package is required');
-    console.error('Usage: abapgit-agent init --folder /src --package ZMY_PACKAGE');
-    console.error('       abapgit-agent init --update    # Update files to latest version');
-    process.exit(1);
-  }
-
-  // Check if current folder is git repo root
-  const gitDir = pathModule.join(process.cwd(), '.git');
-  if (!fs.existsSync(gitDir)) {
-    console.error('Error: Current folder is not a git repository.');
-    console.error('Run this command from the root folder of your git repository.');
-    process.exit(1);
-  }
-
-  // In update mode, just copy the files and return
-  if (updateMode) {
-    console.log(`\n🔄 Updating abapGit Agent files`);
-    console.log('');
-
-    // Copy CLAUDE.md
-    await copyFileIfExists(
-      pathModule.join(__dirname, '..', 'abap', 'CLAUDE.md'),
-      pathModule.join(process.cwd(), 'CLAUDE.md'),
-      'CLAUDE.md'
-    );
-
-    // Copy copilot-instructions.md
-    await copyFileIfExists(
-      pathModule.join(__dirname, '..', 'abap', '.github', 'copilot-instructions.md'),
-      pathModule.join(process.cwd(), '.github', 'copilot-instructions.md'),
-      '.github/copilot-instructions.md',
-      true  // create parent dir
-    );
-
-    // Copy guidelines folder to project root
-    await copyGuidelinesFolder(
-      pathModule.join(__dirname, '..', 'abap', 'guidelines'),
-      pathModule.join(process.cwd(), 'guidelines'),
-      true  // overwrite
-    );
-
-    console.log(`
-📋 Update complete!
-   Run 'abapgit-agent ref --list-topics' to see available topics.
-`);
-    return;
-  }
-
-  // Validate package is required
-  if (!packageName) {
-    console.error('Error: --package is required');
-    console.error('Usage: abapgit-agent init --folder /src --package ZMY_PACKAGE');
-    console.error('       abapgit-agent init --update    # Update files to latest version');
-    process.exit(1);
-  }
-
-  console.log(`\n🚀 Initializing abapGit Agent for local repository`);
-  console.log(`   Folder: ${folder}`);
-  console.log(`   Package: ${packageName}`);
-  console.log('');
-
-  // Detect git remote URL
-  const gitUrl = getGitRemoteUrl();
-  if (!gitUrl) {
-    console.error('Error: No git remote configured.');
-    console.error('Configure a remote with: git remote add origin <url>');
-    process.exit(1);
-  }
-  console.log(`📌 Git remote: ${gitUrl}`);
-
-  // Check if .abapGitAgent already exists
-  const configPath = pathModule.join(process.cwd(), '.abapGitAgent');
-  if (fs.existsSync(configPath)) {
-    console.error('Error: .abapGitAgent already exists.');
-    console.error('To reinitialize, delete the existing file first.');
-    process.exit(1);
-  }
-
-  // Copy .abapGitAgent.example to .abapGitAgent
-  const samplePath = pathModule.join(__dirname, '..', '.abapGitAgent.example');
-  if (!fs.existsSync(samplePath)) {
-    console.error('Error: .abapGitAgent.example not found.');
-    process.exit(1);
-  }
-
-  try {
-    // Read sample and update with package/folder
-    const sampleContent = fs.readFileSync(samplePath, 'utf8');
-    const config = JSON.parse(sampleContent);
-    config.package = packageName;
-    config.folder = folder;
-
-    // Write updated config
-    fs.writeFileSync(configPath, JSON.stringify(config, null, 2) + '\n');
-    console.log(`✅ Created .abapGitAgent`);
-  } catch (error) {
-    console.error(`Error creating .abapGitAgent: ${error.message}`);
-    process.exit(1);
-  }
-
-  // Update .gitignore
-  const gitignorePath = pathModule.join(process.cwd(), '.gitignore');
-  const ignoreEntries = ['.abapGitAgent', '.abapgit_agent_cookies.txt'];
-  let existingIgnore = '';
-
-  if (fs.existsSync(gitignorePath)) {
-    existingIgnore = fs.readFileSync(gitignorePath, 'utf8');
-  }
+// Import utilities
+const gitUtils = require('../src/utils/git-utils');
+const versionCheck = require('../src/utils/version-check');
+const validators = require('../src/utils/validators');
+const { AbapHttp } = require('../src/utils/abap-http');
+const { loadConfig, getTransport, isAbapIntegrationEnabled } = require('../src/config');
 
-  let updated = false;
-  let newIgnoreContent = existingIgnore;
-
-  for (const entry of ignoreEntries) {
-    if (!newIgnoreContent.includes(entry)) {
-      if (newIgnoreContent && !newIgnoreContent.endsWith('\n')) {
-        newIgnoreContent += '\n';
-      }
-      newIgnoreContent += entry + '\n';
-      updated = true;
-    }
-  }
-
-  if (updated) {
-    fs.writeFileSync(gitignorePath, newIgnoreContent);
-    console.log(`✅ Updated .gitignore`);
-  } else {
-    fs.writeFileSync(gitignorePath, newIgnoreContent);
-    console.log(`✅ .gitignore already up to date`);
-  }
-
-  // Copy CLAUDE.md
-  const claudeMdPath = pathModule.join(__dirname, '..', 'abap', 'CLAUDE.md');
-  const localClaudeMdPath = pathModule.join(process.cwd(), 'CLAUDE.md');
-  try {
-    if (fs.existsSync(claudeMdPath)) {
-      fs.copyFileSync(claudeMdPath, localClaudeMdPath);
-      console.log(`✅ Created CLAUDE.md`);
-    } else {
-      console.log(`⚠️  CLAUDE.md not found in abap/ directory`);
-    }
-  } catch (error) {
-    console.error(`Error copying CLAUDE.md: ${error.message}`);
-  }
-
-  // Copy copilot-instructions.md for GitHub Copilot
-  const copilotMdPath = pathModule.join(__dirname, '..', 'abap', '.github', 'copilot-instructions.md');
-  const githubDir = pathModule.join(process.cwd(), '.github');
-  const localCopilotMdPath = pathModule.join(githubDir, 'copilot-instructions.md');
-  try {
-    if (fs.existsSync(copilotMdPath)) {
-      // Ensure .github directory exists
-      if (!fs.existsSync(githubDir)) {
-        fs.mkdirSync(githubDir, { recursive: true });
-      }
-      fs.copyFileSync(copilotMdPath, localCopilotMdPath);
-      console.log(`✅ Created .github/copilot-instructions.md`);
-    } else {
-      console.log(`⚠️  copilot-instructions.md not found in abap/ directory`);
-    }
-  } catch (error) {
-    console.error(`Error copying copilot-instructions.md: ${error.message}`);
-  }
-
-  // Copy guidelines folder to project root
-  const guidelinesSrcPath = pathModule.join(__dirname, '..', 'abap', 'guidelines');
-  const guidelinesDestPath = pathModule.join(process.cwd(), 'guidelines');
-  try {
-    if (fs.existsSync(guidelinesSrcPath)) {
-      if (!fs.existsSync(guidelinesDestPath)) {
-        // Create guidelines directory
-        fs.mkdirSync(guidelinesDestPath, { recursive: true });
-        // Copy all files from guidelines folder
-        const files = fs.readdirSync(guidelinesSrcPath);
-        for (const file of files) {
-          if (file.endsWith('.md')) {
-            fs.copyFileSync(
-              pathModule.join(guidelinesSrcPath, file),
-              pathModule.join(guidelinesDestPath, file)
-            );
-          }
-        }
-        console.log(`✅ Created guidelines/ (${files.filter(f => f.endsWith('.md')).length} files)`);
-      } else {
-        console.log(`⚠️  guidelines/ already exists, skipped`);
-      }
-    } else {
-      console.log(`⚠️  guidelines folder not found in abap/ directory`);
-    }
-  } catch (error) {
-    console.error(`Error copying guidelines: ${error.message}`);
-  }
-
-  // Create folder
-  const folderPath = pathModule.join(process.cwd(), folder);
-  try {
-    if (!fs.existsSync(folderPath)) {
-      fs.mkdirSync(folderPath, { recursive: true });
-      console.log(`✅ Created folder: ${folder}`);
-
-      // Create .gitkeep
-      const gitkeepPath = pathModule.join(folderPath, '.gitkeep');
-      fs.writeFileSync(gitkeepPath, '');
-    } else {
-      console.log(`✅ Folder already exists: ${folder}`);
-    }
-  } catch (error) {
-    console.error(`Error creating folder: ${error.message}`);
-  }
-
-  console.log(`
-📋 Next steps:
-   1. Edit .abapGitAgent with your ABAP credentials (host, user, password)
-   2. Run 'abapgit-agent create --import' to create online repository
-   3. Run 'abapgit-agent pull' to activate objects
-`);
-}
-
-/**
- * Check agent health
- */
-async function healthCheck() {
-  try {
-    const result = await request('GET', '/sap/bc/z_abapgit_agent/health');
-    console.log(JSON.stringify(result, null, 2));
-    return result;
-  } catch (error) {
-    console.error(`Health check failed: ${error.message}`);
-    process.exit(1);
-  }
-}
+// Get terminal width for responsive table
+const getTermWidth = () => process.stdout.columns || 80;
+const TERM_WIDTH = getTermWidth();
 
-/**
- * Main CLI
- */
 async function main() {
   const args = process.argv.slice(2);
   const command = args[0];
 
-  // Check if ABAP integration is enabled for this repo
-  // (skip for init, help, ref commands - these don't need ABAP connection)
-  const noAbapRequired = ['init', 'help', '--help', '-h', 'ref'];
-  if (command && !noAbapRequired.includes(command)) {
-    if (!isAbapIntegrationEnabled()) {
+  // Load command modules (Phase 2 refactoring)
+  const commandModules = {
+    help: require('../src/commands/help'),
+    health: require('../src/commands/health'),
+    status: require('../src/commands/status'),
+    create: require('../src/commands/create'),
+    delete: require('../src/commands/delete'),
+    import: require('../src/commands/import'),
+    inspect: require('../src/commands/inspect'),
+    unit: require('../src/commands/unit'),
+    syntax: require('../src/commands/syntax'),
+    tree: require('../src/commands/tree'),
+    list: require('../src/commands/list'),
+    view: require('../src/commands/view'),
+    preview: require('../src/commands/preview'),
+    where: require('../src/commands/where'),
+    ref: require('../src/commands/ref'),
+    init: require('../src/commands/init'),
+    pull: require('../src/commands/pull')
+  };
+
+  // Check if this is a modular command
+  if (commandModules[command] || command === '--help' || command === '-h') {
+    const cmd = command === '--help' || command === '-h' ? commandModules.help : commandModules[command];
+
+    // Check if command requires ABAP configuration
+    if (cmd.requiresAbapConfig && !isAbapIntegrationEnabled()) {
       console.log(`
 ⚠️  ABAP Git Agent not configured for this repository.
 
@@ -1510,1397 +80,36 @@ To enable integration:
 `);
       process.exit(1);
     }
-  }
-
-  // Version compatibility check for commands that interact with ABAP
-  const abapCommands = ['create', 'delete', 'import', 'pull', 'inspect', 'unit', 'tree', 'view', 'preview', 'list'];
-  if (command && abapCommands.includes(command)) {
-    await checkVersionCompatibility();
-  }
-
-  try {
-    switch (command) {
-      case 'init':
-        await runInit(args);
-        return;  // Don't check ABAP integration for init
-
-      case 'create': {
-        const helpIndex = args.findIndex(a => a === '--help' || a === '-h');
-
-        // Show help if requested
-        if (helpIndex !== -1) {
-          console.log(`
-Usage:
-  abapgit-agent create
-
-Description:
-  Create abapGit online repository in ABAP system.
-  Auto-detects URL from git remote and package from .abapGitAgent.
-
-Prerequisites:
-  - Run "abapgit-agent init" first
-  - Edit .abapGitAgent with credentials (host, user, password)
-
-Examples:
-  abapgit-agent create                    # Create repo in ABAP
-`);
-          return;
-        }
-
-        // Get parameters from config
-        const config = loadConfig();
-        const repoUrl = getGitRemoteUrl();
-
-        if (!repoUrl) {
-          console.error('Error: No git remote configured. Please configure a remote origin.');
-          process.exit(1);
-        }
-
-        if (!config.package) {
-          console.error('Error: Package not configured. Run "abapgit-agent init" first or set package in .abapGitAgent.');
-          process.exit(1);
-        }
-
-        const branch = getGitBranch();
-
-        // Extract repo name from git URL
-        const repoName = repoUrl.split('/').pop().replace('.git', '');
-
-        console.log(`\n🚀 Creating online repository`);
-        console.log(`   URL: ${repoUrl}`);
-        console.log(`   Package: ${config.package}`);
-        console.log(`   Folder: ${config.folder || '/src/'}`);
-        console.log(`   Name: ${repoName}`);
-        console.log(`   Branch: ${branch}`);
-
-        const csrfToken = await fetchCsrfToken(config);
-
-        const data = {
-          url: repoUrl,
-          package: config.package,
-          name: repoName,
-          branch: branch,
-          folder: config.folder || '/src/'
-        };
-
-        if (config.gitUsername) {
-          data.username = config.gitUsername;
-        }
-
-        if (config.gitPassword) {
-          data.password = config.gitPassword;
-        }
-
-        const result = await request('POST', '/sap/bc/z_abapgit_agent/create', data, { csrfToken });
-
-        console.log('\n');
-
-        // Handle uppercase keys from ABAP
-        const success = result.SUCCESS || result.success;
-        const repoKey = result.REPO_KEY || result.repo_key;
-        const createdRepoName = result.REPO_NAME || result.repo_name;
-        const message = result.MESSAGE || result.message;
-        const error = result.ERROR || result.error;
-
-        if (success === 'X' || success === true) {
-          console.log(`✅ Repository created successfully!`);
-          console.log(`   URL: ${repoUrl}`);
-          console.log(`   Package: ${config.package}`);
-          console.log(`   Name: ${createdRepoName || repoName}`);
-        } else {
-          console.log(`❌ Failed to create repository`);
-          console.log(`   Error: ${error || message || 'Unknown error'}`);
-          process.exit(1);
-        }
-        break;
-      }
-
-      case 'delete': {
-        const helpIndex = args.findIndex(a => a === '--help' || a === '-h');
-
-        // Show help if requested
-        if (helpIndex !== -1) {
-          console.log(`
-Usage:
-  abapgit-agent delete
-
-Description:
-  Delete abapGit online repository from ABAP system.
-  Auto-detects URL from git remote of current directory.
-
-Prerequisites:
-  - Run "abapgit-agent create" first
-
-Examples:
-  abapgit-agent delete   # Delete repo for current git remote
-`);
-          return;
-        }
-
-        // Get URL from current git remote
-        const config = loadConfig();
-        const repoUrl = getGitRemoteUrl();
-
-        if (!repoUrl) {
-          console.error('Error: No git remote configured. Please configure a remote origin.');
-          process.exit(1);
-        }
-
-        console.log(`\n🗑️  Deleting online repository`);
-        console.log(`   URL: ${repoUrl}`);
-
-        const csrfToken = await fetchCsrfToken(config);
-
-        const data = {
-          url: repoUrl
-        };
-
-        const result = await request('POST', '/sap/bc/z_abapgit_agent/delete', data, { csrfToken });
-
-        console.log('\n');
-
-        // Handle uppercase keys from ABAP
-        const success = result.SUCCESS || result.success;
-        const repoKey = result.REPO_KEY || result.repo_key;
-        const message = result.MESSAGE || result.message;
-        const error = result.ERROR || result.error;
-
-        if (success === 'X' || success === true) {
-          console.log(`✅ Repository deleted successfully!`);
-          console.log(`   Key: ${repoKey}`);
-        } else {
-          console.log(`❌ Failed to delete repository`);
-          console.log(`   Error: ${error || message || 'Unknown error'}`);
-          process.exit(1);
-        }
-        break;
-      }
-
-      case 'import': {
-        const helpIndex = args.findIndex(a => a === '--help' || a === '-h');
-
-        // Show help if requested
-        if (helpIndex !== -1) {
-          console.log(`
-Usage:
-  abapgit-agent import [--message <message>]
-
-Description:
-  Import existing objects from package to git repository.
-  Uses the git remote URL to find the abapGit online repository.
-
-Prerequisites:
-  - Run "abapgit-agent create" first or create repository in abapGit UI
-  - Package must have objects to import
-
-Options:
-  --message    Commit message (default: "feat: initial import from ABAP package <package>")
-
-Examples:
-  abapgit-agent import
-  abapgit-agent import --message "Initial import from SAP"
-`);
-          return;
-        }
-
-        // Get parameters from config
-        const config = loadConfig();
-        const repoUrl = getGitRemoteUrl();
-
-        if (!repoUrl) {
-          console.error('Error: No git remote configured. Please configure a remote origin.');
-          process.exit(1);
-        }
-
-        const messageArgIndex = args.indexOf('--message');
-        let commitMessage = null;
-        if (messageArgIndex !== -1 && messageArgIndex + 1 < args.length) {
-          commitMessage = args[messageArgIndex + 1];
-        }
-
-        console.log(`\n📦 Importing objects to git repository`);
-        console.log(`   URL: ${repoUrl}`);
-        if (commitMessage) {
-          console.log(`   Message: ${commitMessage}`);
-        }
-
-        const csrfToken = await fetchCsrfToken(config);
-
-        const data = {
-          url: repoUrl
-        };
-
-        if (commitMessage) {
-          data.message = commitMessage;
-        }
-
-        if (config.gitUsername) {
-          data.username = config.gitUsername;
-        }
-
-        if (config.gitPassword) {
-          data.password = config.gitPassword;
-        }
-
-        const result = await request('POST', '/sap/bc/z_abapgit_agent/import', data, { csrfToken });
-
-        console.log('\n');
-
-        // Handle uppercase keys from ABAP
-        const success = result.SUCCESS || result.success;
-        const filesStaged = result.FILES_STAGED || result.files_staged;
-        const abapCommitMessage = result.COMMIT_MESSAGE || result.commit_message;
-        const error = result.ERROR || result.error;
-
-        if (success === 'X' || success === true) {
-          console.log(`✅ Objects imported successfully!`);
-          console.log(`   Files staged: ${filesStaged}`);
-          console.log(`   Commit: ${commitMessage || abapCommitMessage}`);
-        } else {
-          console.log(`❌ Import failed`);
-          console.log(`   Error: ${error || resultMessage || 'Unknown error'}`);
-          process.exit(1);
-        }
-        break;
-      }
-
-      case 'pull':
-        const urlArgIndex = args.indexOf('--url');
-        const branchArgIndex = args.indexOf('--branch');
-        const filesArgIndex = args.indexOf('--files');
-        const transportArgIndex = args.indexOf('--transport');
-
-        // Auto-detect git remote URL if not provided
-        let gitUrl = urlArgIndex !== -1 ? args[urlArgIndex + 1] : null;
-        let branch = branchArgIndex !== -1 ? args[branchArgIndex + 1] : getGitBranch();
-        let files = null;
-
-        // Transport: CLI arg takes priority, then config/environment, then null
-        let transportRequest = null;
-        if (transportArgIndex !== -1 && transportArgIndex + 1 < args.length) {
-          // Explicit --transport argument
-          transportRequest = args[transportArgIndex + 1];
-        } else {
-          // Fall back to config or environment variable
-          transportRequest = getTransport();
-        }
-
-        if (filesArgIndex !== -1 && filesArgIndex + 1 < args.length) {
-          files = args[filesArgIndex + 1].split(',').map(f => f.trim());
-        }
-
-        if (!gitUrl) {
-          gitUrl = getGitRemoteUrl();
-          if (!gitUrl) {
-            console.error('Error: Not in a git repository or no remote configured.');
-            console.error('Either run from a git repo, or specify --url <git-url>');
-            process.exit(1);
-          }
-          console.log(`📌 Auto-detected git remote: ${gitUrl}`);
-        }
-
-        await pull(gitUrl, branch, files, transportRequest);
-        break;
-
-      case 'health':
-        await healthCheck();
-        break;
-
-      case 'status':
-        if (isAbapIntegrationEnabled()) {
-          console.log('✅ ABAP Git Agent is ENABLED');
-          console.log('   Config location:', pathModule.join(process.cwd(), '.abapGitAgent'));
-
-          // Check if repo exists in ABAP
-          const config = loadConfig();
-          const repoUrl = getGitRemoteUrl();
-
-          if (repoUrl) {
-            try {
-              const csrfToken = await fetchCsrfToken(config);
-              const result = await request('POST', '/sap/bc/z_abapgit_agent/status', { url: repoUrl }, { csrfToken });
-
-              const status = result.status || result.STATUS || result.SUCCESS;
-              if (status === 'Found' || status === 'X' || status === true) {
-                console.log('   Repository: ✅ Created');
-                console.log(`      Package: ${result.PACKAGE || result.package}`);
-                console.log(`      URL: ${repoUrl}`);
-                console.log(`      Key: ${result.REPO_KEY || result.repo_key}`);
-              } else {
-                console.log('   Repository: ❌ Not created in ABAP system');
-                console.log(`      URL: ${repoUrl}`);
-                console.log('   Run "abapgit-agent create" to create repository');
-              }
-            } catch (err) {
-              console.log('   Repository: ⚠️ Unable to check status');
-              console.log('   Error:', err.message);
-            }
-          } else {
-            console.log('   No git remote configured');
-          }
-        } else {
-          console.log('❌ ABAP Git Agent is NOT configured');
-        }
-        break;
-
-      case 'inspect': {
-        const filesArgIndex = args.indexOf('--files');
-        if (filesArgIndex === -1 || filesArgIndex + 1 >= args.length) {
-          console.error('Error: --files parameter required');
-          console.error('Usage: abapgit-agent inspect --files <file1>,<file2>,... [--variant <check-variant>]');
-          console.error('Example: abapgit-agent inspect --files zcl_my_class.clas.abap');
-          console.error('Example: abapgit-agent inspect --files zcl_my_class.clas.abap --variant ALL_CHECKS');
-          process.exit(1);
-        }
-
-        const filesSyntaxCheck = args[filesArgIndex + 1].split(',').map(f => f.trim());
-
-        // Parse optional --variant parameter
-        const variantArgIndex = args.indexOf('--variant');
-        const variant = variantArgIndex !== -1 ? args[variantArgIndex + 1] : null;
-
-        console.log(`\n  Inspect for ${filesSyntaxCheck.length} file(s)`);
-        if (variant) {
-          console.log(`  Using variant: ${variant}`);
-        }
-        console.log('');
-
-        const config = loadConfig();
-        const csrfToken = await fetchCsrfToken(config);
-
-        // Send all files in one request
-        const results = await inspectAllFiles(filesSyntaxCheck, csrfToken, config, variant);
-
-        // Process results
-        for (const result of results) {
-          await processInspectResult(result);
-        }
-        break;
-      }
-
-      case 'unit': {
-        const filesArgIndex = args.indexOf('--files');
-        if (filesArgIndex === -1 || filesArgIndex + 1 >= args.length) {
-          console.error('Error: --files parameter required');
-          console.error('Usage: abapgit-agent unit --files <file1>,<file2>,... [--coverage]');
-          console.error('Example: abapgit-agent unit --files zcl_my_test.clas.abap');
-          console.error('Example: abapgit-agent unit --files zcl_my_test.clas.abap --coverage');
-          process.exit(1);
-        }
-
-        const files = args[filesArgIndex + 1].split(',').map(f => f.trim());
-
-        // Check for coverage option
-        const coverage = args.includes('--coverage');
-
-        console.log(`\n  Running unit tests for ${files.length} file(s)${coverage ? ' (with coverage)' : ''}`);
-        console.log('');
-
-        const config = loadConfig();
-        const csrfToken = await fetchCsrfToken(config);
-
-        for (const sourceFile of files) {
-          await runUnitTestForFile(sourceFile, csrfToken, config, coverage);
-        }
-        break;
-      }
-
-      case 'tree': {
-        const packageArgIndex = args.indexOf('--package');
-        if (packageArgIndex === -1) {
-          console.error('Error: --package parameter required');
-          console.error('Usage: abapgit-agent tree --package <package> [--depth <n>] [--include-types] [--json]');
-          console.error('Example: abapgit-agent tree --package ZMY_PACKAGE');
-          process.exit(1);
-        }
-
-        // Check if package value is missing (happens when shell variable expands to empty)
-        if (packageArgIndex + 1 >= args.length) {
-          console.error('Error: --package parameter value is missing');
-          console.error('');
-          console.error('Tip: If you are using a shell variable, make sure to quote it:');
-          console.error('  abapgit-agent tree --package "$ZMY_PACKAGE"');
-          console.error('  or escape the $ character:');
-          console.error('  abapgit-agent tree --package \\$ZMY_PACKAGE');
-          console.error('');
-          console.error('Usage: abapgit-agent tree --package <package> [--depth <n>] [--include-types] [--json]');
-          console.error('Example: abapgit-agent tree --package ZMY_PACKAGE');
-          process.exit(1);
-        }
-
-        const packageName = args[packageArgIndex + 1];
-
-        // Check for empty/whitespace-only package name
-        if (!packageName || packageName.trim() === '') {
-          console.error('Error: --package parameter cannot be empty');
-          console.error('Usage: abapgit-agent tree --package <package> [--depth <n>] [--include-types] [--json]');
-          console.error('Example: abapgit-agent tree --package ZMY_PACKAGE');
-          process.exit(1);
-        }
-
-        // Optional depth parameter
-        const depthArgIndex = args.indexOf('--depth');
-        let depth = 3;
-        if (depthArgIndex !== -1 && depthArgIndex + 1 < args.length) {
-          depth = parseInt(args[depthArgIndex + 1], 10);
-          if (isNaN(depth) || depth < 1) {
-            console.error('Error: --depth must be a positive number');
-            process.exit(1);
-          }
-        }
-
-        // Optional include-types parameter (--include-objects is deprecated alias)
-        const includeTypes = args.includes('--include-types') || args.includes('--include-objects');
-
-        // Optional json parameter
-        const jsonOutput = args.includes('--json');
-
-        if (jsonOutput) {
-          const config = loadConfig();
-          const csrfToken = await fetchCsrfToken(config);
-          const result = await runTreeCommand(packageName, depth, includeTypes, csrfToken, config);
-          console.log(JSON.stringify(result, null, 2));
-        } else {
-          await displayTreeOutput(packageName, depth, includeTypes);
-        }
-        break;
-      }
-
-      case 'list': {
-        const packageArgIndex = args.indexOf('--package');
-        if (packageArgIndex === -1) {
-          console.error('Error: --package parameter required');
-          console.error('Usage: abapgit-agent list --package <package> [--type <types>] [--name <pattern>] [--limit <n>] [--offset <n>] [--json]');
-          console.error('Example: abapgit-agent list --package $ZMY_PACKAGE');
-          console.error('Example: abapgit-agent list --package $ZMY_PACKAGE --type CLAS,INTF');
-          process.exit(1);
-        }
-
-        // Check if package value is missing
-        if (packageArgIndex + 1 >= args.length) {
-          console.error('Error: --package parameter value is missing');
-          process.exit(1);
-        }
-
-        const packageName = args[packageArgIndex + 1];
 
-        // Validate package name
-        if (!packageName || packageName.trim() === '') {
-          console.error('Error: --package parameter cannot be empty');
-          process.exit(1);
-        }
+    // Load config if needed
+    const config = cmd.requiresAbapConfig ? loadConfig() : null;
 
-        // Optional type parameter
-        const typeArgIndex = args.indexOf('--type');
-        const type = typeArgIndex !== -1 && typeArgIndex + 1 < args.length ? args[typeArgIndex + 1] : null;
-
-        // Optional name pattern
-        const nameArgIndex = args.indexOf('--name');
-        const name = nameArgIndex !== -1 && nameArgIndex + 1 < args.length ? args[nameArgIndex + 1] : null;
-
-        // Optional limit
-        const limitArgIndex = args.indexOf('--limit');
-        let limit = 100;
-        if (limitArgIndex !== -1 && limitArgIndex + 1 < args.length) {
-          limit = parseInt(args[limitArgIndex + 1], 10);
-          if (isNaN(limit) || limit < 1) {
-            console.error('Error: --limit must be a positive number');
-            process.exit(1);
-          }
-          if (limit > 1000) {
-            console.error('Error: --limit value too high (max: 1000)');
-            process.exit(1);
-          }
-        }
-
-        // Optional offset
-        const offsetArgIndex = args.indexOf('--offset');
-        let offset = 0;
-        if (offsetArgIndex !== -1 && offsetArgIndex + 1 < args.length) {
-          offset = parseInt(args[offsetArgIndex + 1], 10);
-          if (isNaN(offset) || offset < 0) {
-            console.error('Error: --offset must be a non-negative number');
-            process.exit(1);
-          }
-        }
-
-        // Optional json parameter
-        const jsonOutput = args.includes('--json');
-
-        const config = loadConfig();
-        const csrfToken = await fetchCsrfToken(config);
-
-        const data = {
-          package: packageName,
-          limit: limit,
-          offset: offset
-        };
-
-        if (type) {
-          data.type = type;
-        }
-
-        if (name) {
-          data.name = name;
-        }
-
-        const result = await request('POST', '/sap/bc/z_abapgit_agent/list', data, { csrfToken });
-
-        // Handle uppercase keys from ABAP
-        const success = result.SUCCESS || result.success;
-        const error = result.ERROR || result.error;
-        const objects = result.OBJECTS || result.objects || [];
-        const byType = result.BY_TYPE || result.by_type || [];
-        const total = result.TOTAL || result.total || 0;
-
-        if (!success || error) {
-          console.error(`\n  Error: ${error || 'Failed to list objects'}`);
-          process.exit(1);
-        }
-
-        if (jsonOutput) {
-          console.log(JSON.stringify(result, null, 2));
-        } else {
-          // Display human-readable output
-          let title = `Objects in ${packageName}`;
-          if (type) {
-            title += ` (${type} only`;
-            if (total !== objects.length) {
-              title += `, Total: ${total}`;
-            }
-            title += ')';
-          } else if (total !== objects.length) {
-            title += ` (Total: ${total})`;
-          }
-          console.log(`\n${title}\n`);
-
-          // Group objects by type
-          const objectsByType = {};
-          for (const obj of objects) {
-            const objType = (obj.TYPE || obj.type || '???').toUpperCase();
-            if (!objectsByType[objType]) {
-              objectsByType[objType] = [];
-            }
-            objectsByType[objType].push(obj.NAME || obj.name);
-          }
-
-          // Display grouped objects
-          for (const objType of Object.keys(objectsByType).sort()) {
-            const objNames = objectsByType[objType];
-            console.log(`  ${objType} (${objNames.length})`);
-            for (const objName of objNames) {
-              console.log(`    ${objName}`);
-            }
-            console.log('');
-          }
-        }
-        break;
-      }
-
-      case 'view': {
-        const objectsArgIndex = args.indexOf('--objects');
-        if (objectsArgIndex === -1 || objectsArgIndex + 1 >= args.length) {
-          console.error('Error: --objects parameter required');
-          console.error('Usage: abapgit-agent view --objects <obj1>,<obj2>,... [--type <type>] [--json]');
-          console.error('Example: abapgit-agent view --objects ZCL_MY_CLASS');
-          console.error('Example: abapgit-agent view --objects ZCL_CLASS1,ZCL_CLASS2 --type CLAS');
-          process.exit(1);
-        }
-
-        const objects = args[objectsArgIndex + 1].split(',').map(o => o.trim().toUpperCase());
-        const typeArg = args.indexOf('--type');
-        const type = typeArg !== -1 ? args[typeArg + 1].toUpperCase() : null;
-        const jsonOutput = args.includes('--json');
-
-        console.log(`\n  Viewing ${objects.length} object(s)`);
-
-        const config = loadConfig();
-        const csrfToken = await fetchCsrfToken(config);
-
-        const data = {
-          objects: objects
-        };
-
-        if (type) {
-          data.type = type;
-        }
-
-        const result = await request('POST', '/sap/bc/z_abapgit_agent/view', data, { csrfToken });
-
-        // Handle uppercase keys from ABAP
-        const success = result.SUCCESS || result.success;
-        const viewObjects = result.OBJECTS || result.objects || [];
-        const message = result.MESSAGE || result.message || '';
-        const error = result.ERROR || result.error;
-
-        if (!success || error) {
-          console.error(`\n  Error: ${error || 'Failed to view objects'}`);
-          break;
-        }
-
-        if (jsonOutput) {
-          console.log(JSON.stringify(result, null, 2));
-        } else {
-          console.log(`\n  ${message}`);
-          console.log('');
-
-          for (let i = 0; i < viewObjects.length; i++) {
-            const obj = viewObjects[i];
-            const objName = obj.NAME || obj.name || `Object ${i + 1}`;
-            const objType = obj.TYPE || obj.type || '';
-            const objTypeText = obj.TYPE_TEXT || obj.type_text || '';
-            const description = obj.DESCRIPTION || obj.description || '';
-            const methods = obj.METHODS || obj.methods || [];
-            const components = obj.COMPONENTS || obj.components || [];
-            const notFound = obj.NOT_FOUND || obj.not_found || false;
-
-            // Check if object was not found
-            if (notFound) {
-              console.log(`  ❌ ${objName} (${objTypeText})`);
-              console.log(`     Object not found: ${objName}`);
-              continue;
-            }
-
-            console.log(`  📖 ${objName} (${objTypeText})`);
-            if (description) {
-              console.log(`     ${description}`);
-            }
-
-            // Display source code for classes, interfaces, CDS views, and programs/source includes
-            const source = obj.SOURCE || obj.source || '';
-            if (source && (objType === 'INTF' || objType === 'Interface' || objType === 'CLAS' || objType === 'Class' || objType === 'DDLS' || objType === 'CDS View' || objType === 'PROG' || objType === 'Program')) {
-              console.log('');
-              // Replace escaped newlines with actual newlines and display
-              const displaySource = source.replace(/\\n/g, '\n');
-              const lines = displaySource.split('\n');
-              for (const line of lines) {
-                console.log(`  ${line}`);
-              }
-            }
-
-            if (methods.length > 0) {
-              console.log(`     Methods: ${methods.length}`);
-              for (const method of methods.slice(0, 5)) {
-                const name = method.NAME || method.name || '';
-                const visibility = method.VISIBILITY || method.visibility || '';
-                console.log(`       - ${visibility} ${name}`);
-              }
-              if (methods.length > 5) {
-                console.log(`       ... and ${methods.length - 5} more`);
-              }
-            }
-
-            if (components.length > 0) {
-              // Check if this is a data element (DTEL) - show domain info in property format
-              if (objType === 'DTEL' || objType === 'Data Element') {
-                const propWidth = 18;
-                const valueWidth = 40;
-
-                // Build separator with corners
-                const sep = '┌' + '─'.repeat(propWidth + 2) + '┬' + '─'.repeat(valueWidth + 2) + '┐';
-                const mid = '├' + '─'.repeat(propWidth + 2) + '┼' + '─'.repeat(valueWidth + 2) + '┤';
-                const end = '└' + '─'.repeat(propWidth + 2) + '┴' + '─'.repeat(valueWidth + 2) + '┘';
-
-                // Helper to build row
-                const buildPropRow = (property, value) => {
-                  return '│ ' + String(property || '').padEnd(propWidth) + ' │ ' +
-                         String(value || '').substring(0, valueWidth).padEnd(valueWidth) + ' │';
-                };
-
-                console.log(`  DATA ELEMENT ${objName}:`);
-                console.log(sep);
-                console.log(buildPropRow('Property', 'Value'));
-                console.log(mid);
-
-                // Collect properties from top-level fields and components
-                const domain = obj.DOMAIN || obj.domain || '';
-                const domainType = obj.DOMAIN_TYPE || obj.domain_type || '';
-                const domainLength = obj.DOMAIN_LENGTH || obj.domain_length || 0;
-                const domainDecimals = obj.DOMAIN_DECIMALS || obj.domain_decimals || 0;
-                const description = obj.DESCRIPTION || obj.description || '';
-
-                if (domainType) {
-                  console.log(buildPropRow('Data Type', domainType));
-                }
-                if (domainLength) {
-                  console.log(buildPropRow('Length', String(domainLength)));
-                }
-                if (domainDecimals) {
-                  console.log(buildPropRow('Decimals', String(domainDecimals)));
-                }
-                if (description) {
-                  console.log(buildPropRow('Description', description));
-                }
-                if (domain) {
-                  console.log(buildPropRow('Domain', domain));
-                }
-
-                console.log(end);
-              } else if (objType === 'TTYP' || objType === 'Table Type') {
-                // Show TTYP details as simple text lines
-                console.log('');
-                for (const comp of components) {
-                  const desc = comp.DESCRIPTION || comp.description || '';
-                  if (desc) {
-                    console.log(`   ${desc}`);
-                  }
-                }
-              } else {
-                // Build table display for TABL/STRU with Data Element and Description
-                const colWidths = {
-                  field: 16,      // Max field name length
-                  key: 3,
-                  type: 8,
-                  length: 8,
-                  dataelement: 30, // Max data element name length
-                  description: 60, // Max field description length
-                };
-
-                // Helper to truncate with ellipsis if needed
-                const truncate = (str, maxLen) => {
-                  const s = String(str || '');
-                  if (s.length <= maxLen) return s;
-                  return s.substring(0, maxLen - 1) + '…';
-                };
-
-                // Helper to build row
-                const buildRow = (field, key, type, length, dataelement, description) => {
-                  return '  | ' + truncate(field, colWidths.field).padEnd(colWidths.field) + ' | ' + String(key || '').padEnd(colWidths.key) + ' | ' + truncate(type, colWidths.type).padEnd(colWidths.type) + ' | ' + String(length || '').padStart(colWidths.length) + ' | ' + truncate(dataelement, colWidths.dataelement).padEnd(colWidths.dataelement) + ' | ' + truncate(description, colWidths.description).padEnd(colWidths.description) + ' |';
-                };
-
-                // Build separator line (matches row structure with | at ends and + between columns)
-                const sep = '  |' + '-'.repeat(colWidths.field + 2) + '+' +
-                            '-'.repeat(colWidths.key + 2) + '+' +
-                            '-'.repeat(colWidths.type + 2) + '+' +
-                            '-'.repeat(colWidths.length + 2) + '+' +
-                            '-'.repeat(colWidths.dataelement + 2) + '+' +
-                            '-'.repeat(colWidths.description + 2) + '|';
-
-                // Header
-                console.log(`  TABLE ${objName}:`);
-                console.log(sep);
-                console.log(buildRow('Field', 'Key', 'Type', 'Length', 'Data Elem', 'Description'));
-                console.log(sep);
-
-                // Rows
-                for (const comp of components) {
-                  const key = comp.KEY || comp.key || false ? 'X' : '';
-                  const dataelement = comp.DATAELEMENT || comp.dataelement || '';
-                  const description = comp.DESCRIPTION || comp.description || '';
-                  console.log(buildRow(comp.FIELD || comp.field, key, comp.TYPE || comp.type, comp.LENGTH || comp.length, dataelement, description));
-                }
-
-                console.log(sep);
-              }
-            }
-
-            console.log('');
-          }
-
-        }
-        break;
-      }
-
-      case 'preview': {
-        const objectsArgIndex = args.indexOf('--objects');
-        if (objectsArgIndex === -1 || objectsArgIndex + 1 >= args.length) {
-          console.error('Error: --objects parameter required');
-          console.error('Usage: abapgit-agent preview --objects <table1>,<view1>,... [--type <type>] [--limit <n>] [--offset <n>] [--where <condition>] [--columns <cols>] [--vertical] [--compact] [--json]');
-          console.error('Example: abapgit-agent preview --objects SFLIGHT');
-          console.error('Example: abapgit-agent preview --objects ZC_MY_CDS_VIEW --type DDLS');
-          console.error('Example: abapgit-agent preview --objects SFLIGHT --where "CARRID = \'AA\'"');
-          console.error('Example: abapgit-agent preview --objects SFLIGHT --offset 10 --limit 20');
-          process.exit(1);
-        }
-
-        const objects = args[objectsArgIndex + 1].split(',').map(o => o.trim().toUpperCase());
-        const typeArg = args.indexOf('--type');
-        const type = typeArg !== -1 ? args[typeArg + 1].toUpperCase() : null;
-        const limitArg = args.indexOf('--limit');
-        const limitRaw = limitArg !== -1 ? args[limitArg + 1] : null;
-        const limitParsed = parseInt(limitRaw, 10);
-        const limit = limitRaw && !limitRaw.startsWith('--') && !isNaN(limitParsed) ? Math.max(1, limitParsed) : 100;
-        const offsetArg = args.indexOf('--offset');
-        const offsetRaw = offsetArg !== -1 ? args[offsetArg + 1] : null;
-        const offsetParsed = parseInt(offsetRaw, 10);
-        const offset = offsetRaw && !offsetRaw.startsWith('--') && !isNaN(offsetParsed) ? Math.max(0, offsetParsed) : 0;
-        const whereArg = args.indexOf('--where');
-        const where = whereArg !== -1 ? args[whereArg + 1] : null;
-        const columnsArg = args.indexOf('--columns');
-        const columns = columnsArg !== -1 ? args[columnsArg + 1].split(',').map(c => c.trim().toUpperCase()) : null;
-        const verticalOutput = args.includes('--vertical');
-        const compactOutput = args.includes('--compact');
-        const jsonOutput = args.includes('--json');
-
-        console.log(`\n  Previewing ${objects.length} object(s)`);
-
-        const config = loadConfig();
-        const csrfToken = await fetchCsrfToken(config);
-
-        const data = {
-          objects: objects,
-          limit: Math.min(Math.max(1, limit), 500),
-          offset: Math.max(0, offset)
-        };
-
-        if (type) {
-          data.type = type;
-        }
-
-        if (where) {
-          data.where = convertDatesInWhereClause(where);
-        }
-
-        if (columns) {
-          data.columns = columns;
-        }
-
-        const result = await request('POST', '/sap/bc/z_abapgit_agent/preview', data, { csrfToken });
-
-        // Handle uppercase keys from ABAP
-        const success = result.SUCCESS || result.success;
-        const previewObjects = result.OBJECTS || result.objects || [];
-        const message = result.MESSAGE || result.message || '';
-        const error = result.ERROR || result.error;
-
-        if (!success || error) {
-          console.error(`\n  Error: ${error || 'Failed to preview objects'}`);
-          break;
-        }
-
-        const pagination = result.PAGINATION || result.pagination || null;
-
-        if (jsonOutput) {
-          console.log(JSON.stringify(result, null, 2));
-        } else {
-          // Build pagination message
-          let paginationMsg = '';
-          const paginationTotal = pagination ? (pagination.TOTAL || pagination.total || 0) : 0;
-          const paginationHasMore = pagination ? (pagination.HAS_MORE || pagination.has_more || false) : false;
-          if (pagination && paginationTotal > 0) {
-            // Handle case where offset exceeds total
-            if (offset >= paginationTotal) {
-              paginationMsg = ` (Offset ${offset} exceeds total ${paginationTotal})`;
-            } else {
-              const start = offset + 1;
-              const end = Math.min(offset + limit, paginationTotal);
-              paginationMsg = ` (Showing ${start}-${end} of ${paginationTotal})`;
-              if (paginationHasMore) {
-                paginationMsg += ` — Use --offset ${offset + limit} to see more`;
-              }
-            }
-          }
-
-          console.log(`\n  ${message}${paginationMsg}`);
-          console.log('');
-
-          // Track if columns were explicitly specified
-          const columnsExplicit = columns !== null;
-
-          for (let i = 0; i < previewObjects.length; i++) {
-            const obj = previewObjects[i];
-            const objName = obj.NAME || obj.name || `Object ${i + 1}`;
-            const objType = obj.TYPE || obj.type || '';
-            const objTypeText = obj.TYPE_TEXT || obj.type_text || '';
-            // Parse rows - could be a JSON string or array
-            let rows = obj.ROWS || obj.rows || [];
-            if (typeof rows === 'string') {
-              try {
-                rows = JSON.parse(rows);
-              } catch (e) {
-                rows = [];
-              }
-            }
-            const fields = obj.FIELDS || obj.fields || [];
-            const rowCount = obj.ROW_COUNT || obj.row_count || 0;
-            const totalRows = obj.TOTAL_ROWS || obj.total_rows || 0;
-            const notFound = obj.NOT_FOUND || obj.not_found || false;
-            const accessDenied = obj.ACCESS_DENIED || obj.access_denied || false;
-
-            // Check if object was not found
-            if (notFound) {
-              console.log(`  ❌ ${objName} (${objTypeText})`);
-              console.log(`     Object not found: ${objName}`);
-              continue;
-            }
-
-            // Check if access denied
-            if (accessDenied) {
-              console.log(`  ❌ ${objName} (${objTypeText})`);
-              console.log(`     Access denied to: ${objName}`);
-              continue;
-            }
-
-            console.log(`  📊 Preview: ${objName} (${objTypeText})`);
-
-            // Check for errors first
-            const objError = obj.ERROR || obj.error;
-            if (objError) {
-              console.log(`     ❌ Error: ${objError}`);
-              continue;
-            }
-
-            if (rows.length === 0) {
-              console.log('     No data found');
-              continue;
-            }
-
-            // Get all unique field names from all rows
-            const allFields = new Set();
-            rows.forEach(row => {
-              Object.keys(row).forEach(key => allFields.add(key));
-            });
-            const allFieldNames = Array.from(allFields);
-
-            // Display as table - use fields metadata only if --columns was explicitly specified
-            let fieldNames;
-            let columnsAutoSelected = false;
-            if (columnsExplicit && fields && fields.length > 0) {
-              // Use fields from metadata (filtered by explicit --columns)
-              fieldNames = fields.map(f => f.FIELD || f.field);
-            } else {
-              // Use all fields - let terminal handle wrapping if needed
-              // Terminal width detection is unreliable without a proper TTY
-              fieldNames = allFieldNames;
-            }
-
-            // Calculate column widths - use reasonable defaults
-            const colWidths = {};
-            const maxColWidth = compactOutput ? 10 : 20;
-            fieldNames.forEach(field => {
-              let maxWidth = field.length;
-              rows.forEach(row => {
-                const value = String(row[field] || '');
-                maxWidth = Math.max(maxWidth, value.length);
-              });
-              // Cap at maxColWidth (truncates both headers and data in compact mode)
-              colWidths[field] = Math.min(maxWidth, maxColWidth);
-            });
-
-            // Render output - either vertical or table
-            if (verticalOutput) {
-              // Vertical format: each field on its own line
-              rows.forEach((row, rowIndex) => {
-                if (rows.length > 1) {
-                  console.log(`\n  Row ${rowIndex + 1}:`);
-                  console.log('  ' + '─'.repeat(30));
-                }
-                fieldNames.forEach(field => {
-                  const value = String(row[field] || '');
-                  console.log(`  ${field}: ${value}`);
-                });
-              });
-              console.log('');
-              continue;
-            }
-
-            // Build table header
-            let headerLine = '  ┌';
-            let separatorLine = '  ├';
-            fieldNames.forEach(field => {
-              const width = colWidths[field];
-              headerLine += '─'.repeat(width + 2) + '┬';
-              separatorLine += '─'.repeat(width + 2) + '┼';
-            });
-            headerLine = headerLine.slice(0, -1) + '┐';
-            separatorLine = separatorLine.slice(0, -1) + '┤';
-
-            // Build header row
-            let headerRow = '  │';
-            fieldNames.forEach(field => {
-              const width = colWidths[field];
-              let displayField = String(field);
-              if (displayField.length > width) {
-                displayField = displayField.slice(0, width - 3) + '...';
-              }
-              headerRow += ' ' + displayField.padEnd(width) + ' │';
-            });
-
-            console.log(headerLine);
-            console.log(headerRow);
-            console.log(separatorLine);
-
-            // Build data rows
-            rows.forEach(row => {
-              let dataRow = '  │';
-              fieldNames.forEach(field => {
-                const width = colWidths[field];
-                const value = String(row[field] || '');
-                const displayValue = value.length > width ? value.slice(0, width - 3) + '...' : value;
-                dataRow += ' ' + displayValue.padEnd(width) + ' │';
-              });
-              console.log(dataRow);
-            });
-
-            // Build bottom border
-            let bottomLine = '  └';
-            fieldNames.forEach(field => {
-              const width = colWidths[field];
-              bottomLine += '─'.repeat(width + 2) + '┴';
-            });
-            bottomLine = bottomLine.slice(0, -1) + '┘';
-            console.log(bottomLine);
-
-            // Show row count
-            if (totalRows > rowCount) {
-              console.log(`\n  Showing ${rowCount} of ${totalRows} rows`);
-            } else {
-              console.log(`\n  ${rowCount} row(s)`);
-            }
-
-            // Note about hidden columns only when --columns was explicitly specified
-            const columnsDisplayed = fieldNames.length;
-            let columnsHidden = [];
-
-            if (columnsExplicit) {
-              columnsHidden = obj.COLUMNS_HIDDEN || obj.columns_hidden || [];
-              if (columnsHidden.length > 0) {
-                console.log(`\n  ⚠️  ${columnsHidden.length} more columns hidden (${columnsHidden.join(', ')})`);
-                console.log('     Use --json for full data');
-              }
-            }
-
-            console.log('');
-          }
-        }
-        break;
-      }
-
-      case 'where': {
-        const objectsArgIndex = args.indexOf('--objects');
-        if (objectsArgIndex === -1 || objectsArgIndex + 1 >= args.length) {
-          console.error('Error: --objects parameter required');
-          console.error('Usage: abapgit-agent where --objects <obj1>,<obj2>,... [--type <type>] [--limit <n>] [--offset <n>] [--json]');
-          console.error('Example: abapgit-agent where --objects ZCL_MY_CLASS');
-          console.error('Example: abapgit-agent where --objects ZIF_MY_INTERFACE');
-          console.error('Example: abapgit-agent where --objects CL_SUT_AUNIT_RUNNER --limit 20');
-          console.error('Example: abapgit-agent where --objects CL_SUT_AUNIT_RUNNER --offset 100');
-          process.exit(1);
-        }
-
-        const objects = args[objectsArgIndex + 1].split(',').map(o => o.trim().toUpperCase());
-        const typeArg = args.indexOf('--type');
-        const type = typeArg !== -1 ? args[typeArg + 1].toUpperCase() : null;
-        const limitArg = args.indexOf('--limit');
-        const limitRaw = limitArg !== -1 ? args[limitArg + 1] : null;
-        const limitParsed = parseInt(limitRaw, 10);
-        const limit = limitRaw && !limitRaw.startsWith('--') && !isNaN(limitParsed) ? Math.max(1, limitParsed) : 100;
-        const offsetArg = args.indexOf('--offset');
-        const offsetRaw = offsetArg !== -1 ? args[offsetArg + 1] : null;
-        const offsetParsed = parseInt(offsetRaw, 10);
-        const offset = offsetRaw && !offsetRaw.startsWith('--') && !isNaN(offsetParsed) ? Math.max(0, offsetParsed) : 0;
-        const jsonOutput = args.includes('--json');
-
-        console.log(`\n  Where-used list for ${objects.length} object(s)`);
-
-        const config = loadConfig();
-        const csrfToken = await fetchCsrfToken(config);
-
-        const data = {
-          objects: objects,
-          limit: Math.min(Math.max(1, limit), 500),
-          offset: Math.max(0, offset)
-        };
-
-        if (type) {
-          data.type = type;
-        }
-
-        const result = await request('POST', '/sap/bc/z_abapgit_agent/where', data, { csrfToken });
-
-        // Handle uppercase keys from ABAP
-        const success = result.SUCCESS || result.success;
-        const whereObjects = result.OBJECTS || result.objects || [];
-        const message = result.MESSAGE || result.message || '';
-        const error = result.ERROR || result.error;
-        const pagination = result.PAGINATION || result.pagination || null;
-
-        if (!success || error) {
-          console.error(`\n  Error: ${error || 'Failed to get where-used list'}`);
-          break;
-        }
-
-        if (jsonOutput) {
-          console.log(JSON.stringify(result, null, 2));
-        } else {
-          // Build pagination message
-          let paginationMsg = '';
-          const paginationTotal = pagination.TOTAL || pagination.total || 0;
-          const paginationHasMore = pagination.HAS_MORE || pagination.has_more || false;
-          if (pagination && paginationTotal > 0) {
-            // Handle case where offset exceeds total
-            if (offset >= paginationTotal) {
-              paginationMsg = ` (Offset ${offset} exceeds total ${paginationTotal})`;
-            } else {
-              const start = offset + 1;
-              const end = Math.min(offset + limit, paginationTotal);
-              paginationMsg = ` (Showing ${start}-${end} of ${paginationTotal})`;
-              if (paginationHasMore) {
-                paginationMsg += ` — Use --offset ${offset + limit} to see more`;
-              }
-            }
-          }
-
-          console.log(`\n  ${message}${paginationMsg}`);
-          console.log('');
-
-          for (let i = 0; i < whereObjects.length; i++) {
-            const obj = whereObjects[i];
-            const objName = obj.NAME || obj.name || `Object ${i + 1}`;
-            const objType = obj.TYPE || obj.type || '';
-            const error = obj.ERROR || obj.error || '';
-            const references = obj.REFERENCES || obj.references || [];
-            const count = obj.COUNT || obj.count || 0;
-
-            // Handle object not found error
-            if (error) {
-              console.log(`  ❌ ${objName} (${objType})`);
-              console.log(`     ${error}`);
-              console.log('');
-              continue;
-            }
-
-            if (count === 0) {
-              console.log(`  ❌ ${objName} (${objType})`);
-              console.log(`     No references found`);
-              console.log('');
-              continue;
-            }
-
-            console.log(`  🔍 ${objName} (${objType})`);
-            console.log(`     Found ${count} reference(s):`);
-            console.log('');
-
-            // Display references - one line format: include → method (type) or include (type)
-            for (let j = 0; j < references.length; j++) {
-              const ref = references[j];
-              const includeName = ref.INCLUDE_NAME || ref.include_name || '';
-              const includeType = ref.INCLUDE_TYPE || ref.include_type || '';
-              const methodName = ref.METHOD_NAME || ref.method_name || '';
-
-              let line;
-              if (methodName) {
-                line = `  ${j + 1}. ${includeName} → ${methodName} (${includeType})`;
-              } else {
-                line = `  ${j + 1}. ${includeName} (${includeType})`;
-              }
-              console.log(line);
-            }
-            console.log('');
-          }
-        }
-        break;
-      }
-
-      case 'ref': {
-        const refSearch = require('../src/ref-search');
-        const topicIndex = args.indexOf('--topic');
-        const cloneIndex = args.indexOf('--clone');
-        const nameIndex = args.indexOf('--name');
-        const listTopics = args.includes('--list-topics') || args.includes('-l');
-        const listRepos = args.includes('--list-repos') || args.includes('-r');
-        const jsonOutput = args.includes('--json');
-
-        // Handle --clone option
-        if (cloneIndex !== -1 && cloneIndex + 1 < args.length) {
-          const repoUrl = args[cloneIndex + 1];
-          const name = nameIndex !== -1 && nameIndex + 1 < args.length ? args[nameIndex + 1] : null;
-          const result = refSearch.cloneRepository(repoUrl, name);
-          if (jsonOutput) {
-            console.log(JSON.stringify(result, null, 2));
-          } else {
-            refSearch.displayCloneResult(result);
-          }
-          break;
-        }
-
-        if (listRepos) {
-          const result = await refSearch.listRepositories();
-          if (jsonOutput) {
-            console.log(JSON.stringify(result, null, 2));
-          } else {
-            refSearch.displayRepositories(result);
-          }
-          break;
-        }
-
-        if (listTopics) {
-          const result = await refSearch.listTopics();
-          if (jsonOutput) {
-            console.log(JSON.stringify(result, null, 2));
-          } else {
-            refSearch.displayTopics(result);
-          }
-          break;
-        }
-
-        if (topicIndex !== -1 && topicIndex + 1 < args.length) {
-          const topic = args[topicIndex + 1];
-          const result = await refSearch.getTopic(topic);
-          if (jsonOutput) {
-            console.log(JSON.stringify(result, null, 2));
-          } else {
-            refSearch.displayTopic(result);
-          }
-          break;
-        }
-
-        // Pattern search (default)
-        const patternIndex = args.findIndex((arg, idx) => idx > 0 && !arg.startsWith('--'));
-        if (patternIndex === -1) {
-          console.error('Error: No pattern specified');
-          console.error('');
-          console.error('Usage:');
-          console.error('  abapgit-agent ref <pattern>           Search for pattern');
-          console.error('  abapgit-agent ref --topic <name>      View specific topic');
-          console.error('  abapgit-agent ref --list-topics       List available topics');
-          console.error('  abapgit-agent ref --list-repos       List reference repositories');
-          console.error('  abapgit-agent ref --clone <repo>     Clone a repository');
-          console.error('');
-          console.error('Examples:');
-          console.error('  abapgit-agent ref "CORRESPONDING"');
-          console.error('  abapgit-agent ref --topic exceptions');
-          console.error('  abapgit-agent ref --list-topics');
-          console.error('  abapgit-agent ref --list-repos');
-          console.error('  abapgit-agent ref --clone SAP-samples/abap-cheat-sheets');
-          console.error('  abapgit-agent ref --clone https://github.com/abapGit/abapGit.git');
-          process.exit(1);
-        }
-
-        const pattern = args[patternIndex];
-        const result = await refSearch.searchPattern(pattern);
-
-        if (jsonOutput) {
-          console.log(JSON.stringify(result, null, 2));
-        } else {
-          refSearch.displaySearchResults(result);
-        }
-        break;
-      }
-
-      case 'help':
-      case '--help':
-      case '-h':
-        console.log(`
-ABAP Git Agent
-
-Usage:
-  abapgit-agent <command> [options]
-
-Commands:
-  init --folder <folder> --package <package>
-    Initialize local configuration for an existing git repository.
-  init --update
-    Update existing files (CLAUDE.md, copilot-instructions.md, guidelines) to latest version.
-
-  create
-    Create abapGit online repository in ABAP system.
-    Auto-detects URL from git remote and package from .abapGitAgent.
-
-  import [--message <message>]
-    Import existing objects from package to git repository.
-    Uses the git remote URL to find the abapGit online repository.
-
-  pull [--url <git-url>] [--branch <branch>] [--files <file1,file2,...>] [--transport <request>]
-    Pull and activate repository in ABAP system.
-    Auto-detects git remote and branch from current directory.
-    Use --files to pull only specific files.
-    Use --transport to specify a transport request.
-
-  inspect --files <file1>,<file2>,...
-    Inspect ABAP source file(s) for issues. Currently runs syntax check.
-
-  unit --files <file1>,<file2>,...
-    Run AUnit tests for ABAP test class files (.testclasses.abap)
-
-  tree --package <package> [--depth <n>] [--include-types] [--json]
-    Display package hierarchy tree from ABAP system
-
-  list --package <package> [--type <types>] [--name <pattern>] [--limit <n>] [--offset <n>] [--json]
-    List ABAP objects in a package with filtering and pagination
-
-  view --objects <obj1>,<obj2>,... [--type <type>] [--json]
-    View ABAP object definitions from the ABAP system
-
-  where --objects <obj1>,<obj2>,... [--type <type>] [--limit <n>] [--json]
-    Find where-used list for ABAP objects (classes, interfaces, programs)
-
-  ref <pattern> [--json]
-    Search ABAP reference repositories for patterns. Requires referenceFolder in .abapGitAgent.
-
-  ref --topic <topic> [--json]
-    View specific topic from cheat sheets (exceptions, sql, unit-tests, etc.)
-
-  ref --list-topics
-    List available topics for reference search
-
-  ref --list-repos
-    List all reference repositories in the reference folder
-
-  ref --clone <repo> [--name <folder>]
-    Clone a GitHub repository to the reference folder
-    - Use full URL: https://github.com/user/repo.git
-    - Or short name: user/repo or user/repo (assumes github.com)
-
-  health
-    Check if ABAP REST API is healthy
-
-  status
-    Check if ABAP integration is configured for this repo
+    // Version check if needed
+    if (cmd.requiresVersionCheck && config) {
+      await versionCheck.checkCompatibility(config);
+    }
 
-Examples:
-  abapgit-agent init --folder /src --package ZMY_PACKAGE      # Initialize
-  abapgit-agent init --update                                # Update files to latest
-  abapgit-agent create                                       # Create repo
-  abapgit-agent delete                                       # Delete repo
-  abapgit-agent import                                       # Import objects to git
-  abapgit-agent import --message "Initial import"            # Import with message
-  abapgit-agent pull                                         # Auto-detect from git
-  abapgit-agent pull --branch develop                        # Specific branch
-  abapgit-agent pull --files zcl_my_class.clas.abap          # Specific file
-  abapgit-agent pull --transport DEVK900001                  # With transport
-  abapgit-agent inspect --files zcl_my_class.clas.abap       # Syntax check
-  abapgit-agent unit --files zcl_my_test.clas.testclasses.abap  # Run tests
-  abapgit-agent tree --package \$ZMY_PACKAGE                      # Show package tree
-  abapgit-agent tree --package \$ZMY_PACKAGE --depth 2             # Shallow tree
-  abapgit-agent tree --package \$ZMY_PACKAGE --include-types      # With type counts
-  abapgit-agent tree --package \$ZMY_PACKAGE --json                # JSON output
-  abapgit-agent list --package $ZMY_PACKAGE                      # List all objects
-  abapgit-agent list --package $ZMY_PACKAGE --type CLAS,INTF      # Filter by type
-  abapgit-agent list --package $ZMY_PACKAGE --name ZCL_*         # Filter by name
-  abapgit-agent list --package $ZMY_PACKAGE --limit 50            # Limit results
-  abapgit-agent list --package $ZMY_PACKAGE --json               # JSON output
-  abapgit-agent view --objects ZCL_MY_CLASS                       # View class definition
-  abapgit-agent view --objects ZIF_MY_INTERFACE --type INTF       # View interface
-  abapgit-agent view --objects ZMY_TABLE --type TABL               # View table structure
-  abapgit-agent view --objects ZCL_CLASS1,ZCL_CLASS2 --json        # Multiple objects
-  abapgit-agent where --objects ZCL_SUT_AUNIT_RUNNER               # Find where class is used
-  abapgit-agent where --objects ZIF_MY_INTERFACE                   # Find interface implementations
-  abapgit-agent where --objects CL_SUT_AUNIT_RUNNER --limit 20    # Limit results
-  abapgit-agent where --objects ZIF_MY_INTERFACE --json            # JSON output
-  abapgit-agent ref "CORRESPONDING"                         # Search all reference repos
-  abapgit-agent ref "CX_SY_"                                 # Search exceptions
-  abapgit-agent ref --topic exceptions                       # View exception topic
-  abapgit-agent ref --topic sql                              # View SQL topic
-  abapgit-agent ref --topic unit-tests                       # View unit test topic
-  abapgit-agent ref --list-topics                            # List all topics
-  abapgit-agent ref --list-repos                             # List reference repositories
-  abapgit-agent ref --clone SAP-samples/abap-cheat-sheets    # Clone repo to reference folder
-  abapgit-agent ref --clone https://github.com/user/repo.git # Clone from URL
-  abapgit-agent health
-  abapgit-agent status
-`);
-        break;
+    // Build context for command
+    const context = {
+      config,
+      gitUtils,
+      validators,
+      versionCheck,
+      isAbapIntegrationEnabled,
+      loadConfig,
+      AbapHttp,
+      getTransport
+    };
 
-      default:
-        console.error(`Unknown command: ${command}`);
-        console.error('Use: abapgit-agent help');
-        process.exit(1);
-    }
-  } catch (error) {
-    console.error(`Error: ${error.message}`);
-    process.exit(1);
+    // Execute command
+    await cmd.execute(args.slice(1), context);
+    return;
   }
+
+  // Unknown command - show error
+  console.error(`Unknown command: ${command}`);
+  console.error('Run "abapgit-agent help" for usage information');
+  process.exit(1);
 }
 
 main();