abapgit-agent 1.5.0 → 1.6.1

package/bin/abgagt

@@ -0,0 +1,24 @@
+#!/usr/bin/env node
+// abgagt - Short alias for abapgit-agent
+// Usage: abgagt <command> [options]
+//
+// This is a wrapper script that forwards all arguments to abapgit-agent
+//
+// Examples:
+//   abgagt pull
+//   abgagt ref "CORRESPONDING"
+//   abgagt inspect --files zcl_my_class.clas.abap
+
+const path = require('path');
+const { spawn } = require('child_process');
+
+const agentPath = path.join(__dirname, 'abapgit-agent');
+const args = process.argv.slice(2);
+
+const child = spawn('node', [agentPath, ...args], {
+  stdio: 'inherit'
+});
+
+child.on('exit', (code) => {
+  process.exit(code);
+});

package/package.json

@@ -1,10 +1,16 @@
 {
   "name": "abapgit-agent",
-  "version": "1.5.0",
+  "version": "1.6.1",
   "description": "ABAP Git Agent - Pull and activate ABAP code via abapGit from any git repository",
   "main": "src/index.js",
+  "files": [
+    "bin/",
+    "src/",
+    "abap/guidelines/"
+  ],
   "bin": {
-    "abapgit-agent": "bin/abapgit-agent"
+    "abapgit-agent": "bin/abapgit-agent",
+    "abgagt": "bin/abgagt"
   },
   "scripts": {
     "start": "node src/server.js",

package/src/abap-client.js

@@ -379,7 +379,7 @@ class ABAPClient {
     return await this.request('POST', '/tree', data, { csrfToken: this.csrfToken });
   }
 
-  async preview(objects, type = null, limit = 10) {
+  async preview(objects, type = null, limit = 10, where = null, columns = null) {
     // Fetch CSRF token first
     await this.fetchCsrfToken();
 
@@ -392,10 +392,73 @@ class ABAPClient {
       data.type = type;
     }
 
-    logger.info('Previewing data', { objects, type, limit: data.limit, service: 'abapgit-agent' });
+    if (where) {
+      // Convert ISO date format (YYYY-MM-DD) to ABAP DATS format (YYYYMMDD)
+      // This handles date literals in WHERE clauses like "FLDATE = '2024-10-24'"
+      data.where = this.convertDatesInWhereClause(where);
+    }
+
+    if (columns) {
+      data.columns = columns;
+    }
+
+    logger.info('Previewing data', { objects, type, limit: data.limit, where: data.where, service: 'abapgit-agent' });
 
     return await this.request('POST', '/preview', data, { csrfToken: this.csrfToken });
   }
+
+  /**
+   * Convert ISO date formats to ABAP DATS format in WHERE clause
+   * @param {string} whereClause - SQL WHERE clause
+   * @returns {string} - WHERE clause with dates converted to YYYYMMDD format
+   */
+  convertDatesInWhereClause(whereClause) {
+    if (!whereClause) return whereClause;
+
+    // Pattern to match ISO date format: 'YYYY-MM-DD'
+    // Uses negative lookbehind and lookahead to ensure we're matching complete dates
+    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}'`;
+    });
+  }
+
+  /**
+   * List objects in an ABAP package
+   * @param {string} packageName - ABAP package name
+   * @param {string} type - Comma-separated object types to filter (optional, e.g., 'CLAS,INTF')
+   * @param {string} name - Name pattern to filter (optional)
+   * @param {number} limit - Maximum number of objects to return (default: 100, max: 1000)
+   * @param {number} offset - Offset for pagination (default: 0)
+   * @returns {object} List result with objects, by_type, and total
+   */
+  async list(packageName, type = null, name = null, limit = 100, offset = 0) {
+    // Fetch CSRF token first
+    await this.fetchCsrfToken();
+
+    const data = {
+      package: packageName,
+      limit: Math.min(Math.max(1, limit), 1000),
+      offset: Math.max(0, offset)
+    };
+
+    if (type) {
+      data.type = type;
+    }
+
+    if (name) {
+      data.name = name;
+    }
+
+    logger.info('Listing objects', { package: packageName, type, name, limit: data.limit, offset: data.offset, service: 'abapgit-agent' });
+
+    return await this.request('POST', '/list', data, { csrfToken: this.csrfToken });
+  }
 }
 
 // Singleton instance

package/src/agent.js

@@ -192,11 +192,20 @@ class ABAPGitAgent {
     }
   }
 
-  async preview(objects, type = null, limit = 10) {
-    logger.info('Previewing data', { objects, type, limit });
+  /**
+   * Preview data from ABAP tables or CDS views
+   * @param {Array} objects - Array of table/view names
+   * @param {string} type - Object type (TABL, DDLS, etc.)
+   * @param {number} limit - Maximum rows to return
+   * @param {string} where - WHERE clause filter
+   * @param {Array} columns - Array of column names to display
+   * @returns {object} Preview result with rows, fields, and metadata
+   */
+  async preview(objects, type = null, limit = 10, where = null, columns = null) {
+    logger.info('Previewing data', { objects, type, limit, where, columns });
 
     try {
-      const result = await this.abap.preview(objects, type, limit);
+      const result = await this.abap.preview(objects, type, limit, where, columns);
       return {
         success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
         command: result.COMMAND || result.command || 'PREVIEW',
@@ -210,6 +219,51 @@ class ABAPGitAgent {
       throw new Error(`Preview command failed: ${error.message}`);
     }
   }
+
+  /**
+   * List objects in an ABAP package
+   * @param {string} packageName - ABAP package name
+   * @param {string} type - Comma-separated object types to filter (optional)
+   * @param {string} name - Name pattern to filter (optional)
+   * @param {number} limit - Maximum number of objects to return (default: 100)
+   * @param {number} offset - Offset for pagination (default: 0)
+   * @returns {object} List result with objects, by_type, total, and error
+   */
+  async list(packageName, type = null, name = null, limit = 100, offset = 0) {
+    logger.info('Listing objects', { package: packageName, type, name, limit, offset });
+
+    try {
+      const result = await this.abap.list(packageName, type, name, limit, offset);
+
+      // Normalize objects array to use lowercase keys
+      const rawObjects = result.OBJECTS || result.objects || [];
+      const normalizedObjects = rawObjects.map(obj => ({
+        type: obj.TYPE || obj.type,
+        name: obj.NAME || obj.name
+      }));
+
+      // Normalize by_type array to use lowercase keys
+      const rawByType = result.BY_TYPE || result.by_type || [];
+      const normalizedByType = rawByType.map(item => ({
+        type: item.TYPE || item.type,
+        count: item.COUNT || item.count || 0
+      }));
+
+      return {
+        success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
+        command: result.COMMAND || result.command || 'LIST',
+        package: result.PACKAGE || result.package || packageName,
+        message: result.MESSAGE || result.message || '',
+        objects: normalizedObjects,
+        by_type: normalizedByType,
+        total: result.TOTAL || result.total || 0,
+        error: result.ERROR || result.error || null
+      };
+    } catch (error) {
+      logger.error('List command failed', { error: error.message });
+      throw new Error(`List command failed: ${error.message}`);
+    }
+  }
 }
 
 module.exports = {

package/src/config.js

@@ -24,7 +24,7 @@ function loadConfig() {
   if (fs.existsSync(configPath)) {
     config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
   } else {
-    // Load from environment variables
+    // Load from environment variables only when no config file exists
     config = {
       host: process.env.ABAP_HOST,
       sapport: parseInt(process.env.ABAP_PORT, 10) || 443,