abapgit-agent 1.0.0 → 1.1.0

package/docs/pull-command.md

@@ -0,0 +1,188 @@
+# pull Command Requirements
+
+## Overview
+
+Pull and activate ABAP objects from git repository.
+
+## Command
+
+```bash
+# Auto-detect git remote and branch from current directory
+abapgit-agent pull
+
+# Pull specific files only (fast)
+abapgit-agent pull --files zcl_my_class.clas.abap,zcl_other.clas.abap
+
+# Pull from specific branch
+abapgit-agent pull --branch develop
+
+# Pull from specific URL
+abapgit-agent pull --url https://github.com/org/repo.git
+
+# With transport request
+abapgit-agent pull --transport DEVK900001
+
+# Combined options
+abapgit-agent pull --branch develop --files src/zcl_my_class.clas.abap --transport DEVK900001
+```
+
+## Prerequisite
+
+- Current folder is git repo root (unless `--url` specified)
+- Git remote is configured (unless `--url` specified)
+- `.abapGitAgent` exists with valid credentials
+
+## Parameters
+
+| Parameter | Required | Description |
+|-----------|----------|-------------|
+| `--url` | No | Git repository URL (auto-detected if not specified) |
+| `--branch` | No | Branch name (default: current branch) |
+| `--files` | No | Comma-separated list of files to pull |
+| `--transport` | No | Transport request for activation |
+
+---
+
+## Tasks
+
+### 1. Detect Git Remote URL
+
+```bash
+git remote get-url origin
+```
+
+### 2. Detect Current Branch
+
+```bash
+git branch --show-current
+```
+
+Or from `.git/HEAD`
+
+### 3. Load Configuration
+
+Read `.abapGitAgent` for credentials
+
+### 4. Fetch CSRF Token
+
+```bash
+GET /health (with X-CSRF-Token: fetch)
+```
+
+### 5. Make Pull Request
+
+**Endpoint:** `POST /pull`
+
+**Request Body:**
+```json
+{
+  "url": "https://github.com/org/repo.git",
+  "branch": "main",
+  "username": "git-user",
+  "password": "git-token",
+  "files": ["zcl_my_class.clas.abap"],
+  "transport_request": "DEVK900001"
+}
+```
+
+### 6. Display Results
+
+---
+
+## Output
+
+### Success
+
+```
+✅ Pull completed successfully!
+   Job ID: CAIS20260208115649
+   Message: Pull completed successfully
+
+📋 Pull Log (N messages):
+────────────────────────────────────────────────────────────────────────────────
+Icon │Object                      │Message
+─────┼────────────────────────────┼─────────────────────────────────────────
+✅   │CLAS ZCL_MY_CLASS          │Object ZCL_MY_CLASS imported
+...
+
+📦 Activated Objects (N):
+────────────────────────────────────────────────────────────────────────────────
+✅ CLAS ZCL_MY_CLASS
+...
+```
+
+### With Errors
+
+```
+❌ Pull completed with errors!
+   Job ID: CAIS20260208115649
+   Message: Pull completed with errors
+
+📋 Error Details:
+────────────────────────────────────────────────────────────────────────────────
+CLAS ZCL_MY_CLASS: Error updating where-used list
+
+📋 Pull Log (N messages):
+...
+
+❌ Failed Objects Log (M entries):
+────────────────────────────────────────────────────────────────────────────────
+❌ CLAS ZCL_MY_CLASS: Error message text
+Exception: Exception details
+```
+
+---
+
+## File Format
+
+Files are parsed to extract `(obj_type, obj_name)`:
+
+| File Pattern | Object Type | Object Name |
+|--------------|-------------|-------------|
+| `zcl_my_class.clas.abap` | CLAS | ZCL_MY_CLASS |
+| `zif_my_intf.intf.abap` | INTF | ZIF_MY_INTF |
+| `src/zcl_my_class.clas.abap` | CLAS | ZCL_MY_CLASS |
+
+---
+
+## Response Structure
+
+```json
+{
+  "success": "X",
+  "job_id": "CAIS20260208115649",
+  "message": "Pull completed successfully",
+  "activated_count": 15,
+  "failed_count": 1,
+  "activated_objects": [
+    { "obj_type": "CLAS", "obj_name": "ZCL_MY_CLASS" }
+  ],
+  "failed_objects": [
+    { "obj_type": "CLAS", "obj_name": "ZCL_OTHER", "text": "Error message" }
+  ],
+  "log_messages": [...]
+}
+```
+
+---
+
+## Key Behaviors
+
+1. **Activated Objects** - Only includes objects that completed successfully (no errors in log)
+2. **Failed Objects Log** - Shows all error messages (duplicates allowed for multiple errors per object)
+3. **Error Details** - When errors occur, displays error detail section at the top
+
+---
+
+## Example
+
+```bash
+# Full pull
+abapgit-agent pull
+
+# Fast pull - specific files
+abapgit-agent pull --files abap/zcl_my_class.clas.abap
+
+# With transport
+abapgit-agent pull --files abap/zcl_my_class.clas.abap --transport DEVK900001
+```

package/docs/status-command.md

@@ -0,0 +1,68 @@
+# status Command Requirements
+
+## Overview
+
+Check if ABAP integration is configured for the current repository.
+
+## Command
+
+```bash
+abapgit-agent status
+```
+
+## Tasks
+
+### 1. Check Configuration File
+
+Check if `.abapGitAgent` exists in current directory:
+
+```bash
+test -f .abapGitAgent
+```
+
+### 2. Display Status
+
+| Status | Output |
+|--------|--------|
+| Configured | `✅ ABAP Git Agent is ENABLED` with config location |
+| Not configured | `❌ ABAP Git Agent is NOT configured` |
+
+---
+
+## Output Examples
+
+### Configured
+
+```
+✅ ABAP Git Agent is ENABLED
+   Config location: /path/to/repo/.abapGitAgent
+```
+
+### Not Configured
+
+```
+❌ ABAP Git Agent is NOT configured
+```
+
+---
+
+## Used By
+
+- `create` command - validates configuration exists before creating repo
+
+## Related Files
+
+| File | Description |
+|------|-------------|
+| `.abapGitAgent` | Configuration file with ABAP connection details |
+| `.abapGitAgent.sample` | Template file |
+
+---
+
+## Example
+
+```bash
+$ abapgit-agent status
+✅ ABAP Git Agent is ENABLED
+   Config location: /Users/user/project/.abapGitAgent
+```

package/docs/unit-command.md

@@ -0,0 +1,167 @@
+# unit Command Requirements
+
+## Overview
+
+Run AUnit tests for ABAP test classes and display detailed results.
+
+## Command
+
+```bash
+# Single test class file
+abapgit-agent unit --files zcl_my_test.clas.testclasses.abap
+
+# Multiple test class files
+abapgit-agent unit --files zcl_test1.clas.testclasses.abap,zcl_test2.clas.testclasses.abap
+
+# With path
+abapgit-agent unit --files abap/zcl_my_test.clas.testclasses.abap
+```
+
+## Prerequisite
+
+- `.abapGitAgent` exists with valid credentials
+- Files must be test class files (`.testclasses.abap`)
+
+## Parameters
+
+| Parameter | Required | Description |
+|-----------|----------|-------------|
+| `--files` | Yes | Comma-separated list of test class files |
+
+---
+
+## Tasks
+
+### 1. Validate Parameters
+
+- `--files` must be specified
+- Files must be `.testclasses.abap` format
+
+### 2. Load Configuration
+
+Read `.abapGitAgent` for credentials
+
+### 3. Fetch CSRF Token
+
+```bash
+GET /health (with X-CSRF-Token: fetch)
+```
+
+### 4. Make Unit Request
+
+**Endpoint:** `POST /unit`
+
+**Request Body:**
+```json
+{
+  "files": ["ZCL_MY_TEST.CLASS.TESTCLASSES.ABAP"]
+}
+```
+
+### 5. Display Results
+
+---
+
+## Output
+
+### All Tests Passed
+
+```
+  Running unit tests for 2 file(s)
+
+  ✅ ZCL_MY_TEST - All tests passed
+     Tests: 10 | Passed: 10 | Failed: 0
+
+  ✅ ZCL_OTHER_TEST - All tests passed
+     Tests: 5 | Passed: 5 | Failed: 0
+```
+
+### With Failures
+
+```
+  Running unit tests for 1 file(s)
+
+  ❌ ZCL_MY_TEST - Tests failed
+     Tests: 10 | Passed: 8 | Failed: 2
+     ✗ ZCL_MY_TEST=>TEST_METHOD_1: Error description
+     ✗ ZCL_MY_TEST=>TEST_METHOD_2: Another error
+
+Failed Tests:
+────────────────────────────────────────────────────────────────────────────────
+   ✗ ZCL_MY_TEST=>TEST_METHOD_1
+     Error: Expected X but got Y
+```
+
+---
+
+## Response Structure
+
+```json
+{
+  "success": "X",
+  "message": "2 of 10 tests failed",
+  "test_count": 10,
+  "passed_count": 8,
+  "failed_count": 2,
+  "errors": [
+    {
+      "class_name": "ZCL_MY_TEST",
+      "method_name": "TEST_METHOD_1",
+      "error_kind": "ERROR",
+      "error_text": "Expected X but got Y"
+    }
+  ]
+}
+```
+
+---
+
+## Error Handling
+
+| Error | Message |
+|-------|---------|
+| File not found | `File not found: <path>` |
+| Invalid format | `Invalid file format: <file>` |
+| No --files specified | `Error: --files parameter required` |
+| No tests found | `➖ <class> - No unit tests` |
+
+---
+
+## File Format
+
+Test class files must end with `.testclasses.abap`:
+
+| File | Test Class |
+|------|------------|
+| `zcl_my_test.clas.testclasses.abap` | ZCL_MY_TEST |
+| `src/tests/zcl_my_test.clas.testclasses.abap` | ZCL_MY_TEST |
+
+---
+
+## Error Details
+
+When a test fails, output includes:
+- **Class**: The test class name
+- **Method**: Failed test method name (with `=>` notation)
+- **Error Kind**: Type of error (ERROR, FAILURE, etc.)
+- **Error Text**: Detailed error message from AUnit
+
+---
+
+## Example
+
+```bash
+# Run tests
+abapgit-agent unit --files zcl_my_test.clas.testclasses.abap
+
+# Multiple
+abapgit-agent unit --files abap/zcl_test1.clas.testclasses.abap,abap/zcl_test2.clas.testclasses.abap
+```
+
+## Implementation
+
+Uses `CL_SUT_AUNIT_RUNNER` to execute tests:
+- `S_CREATE` - Create test runner
+- `run()` - Execute tests
+- `str_results` - Get test statistics
+- `tab_objects` - Get detailed results

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "abapgit-agent",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "ABAP Git Agent - Pull and activate ABAP code via abapGit from any git repository",
   "main": "src/index.js",
   "bin": {

package/src/abap-client.js

@@ -296,6 +296,66 @@ class ABAPClient {
 
     return await this.request('POST', '/unit', data, { csrfToken: this.csrfToken });
   }
+
+  /**
+   * Create a new online repository
+   * @param {string} repoUrl - Git repository URL
+   * @param {string} packageName - ABAP package name
+   * @param {string} branch - Branch name (default: 'main')
+   * @param {string} displayName - Display name for the repository (optional)
+   * @param {string} name - Repository name (optional)
+   * @param {string} folderLogic - Folder logic: 'PREFIX' or 'FULL' (default: 'PREFIX')
+   * @returns {object} Create result
+   */
+  async create(repoUrl, packageName, branch = 'main', displayName = null, name = null, folderLogic = 'PREFIX') {
+    // Fetch CSRF token first
+    await this.fetchCsrfToken();
+
+    const data = {
+      url: repoUrl,
+      package: packageName,
+      branch: branch
+    };
+
+    if (displayName) {
+      data.display_name = displayName;
+    }
+
+    if (name) {
+      data.name = name;
+    }
+
+    if (folderLogic) {
+      data.folder_logic = folderLogic;
+    }
+
+    logger.info('Creating repository', { repoUrl, packageName, branch, service: 'abapgit-agent' });
+
+    return await this.request('POST', '/create', data, { csrfToken: this.csrfToken });
+  }
+
+  /**
+   * Import existing objects from package to git repository
+   * @param {string} repoUrl - Git repository URL
+   * @param {string} message - Commit message (optional)
+   * @returns {object} Import result with success, files_staged, commit_sha
+   */
+  async import(repoUrl, message = null) {
+    // Fetch CSRF token first
+    await this.fetchCsrfToken();
+
+    const data = {
+      url: repoUrl
+    };
+
+    if (message) {
+      data.message = message;
+    }
+
+    logger.info('Starting import operation', { repoUrl, message, service: 'abapgit-agent' });
+
+    return await this.request('POST', '/import', data, { csrfToken: this.csrfToken });
+  }
 }
 
 // Singleton instance

package/src/agent.js

@@ -112,6 +112,58 @@ class ABAPGitAgent {
       throw new Error(`Unit tests failed: ${error.message}`);
     }
   }
+
+  /**
+   * Create a new online repository
+   * @param {string} repoUrl - Git repository URL
+   * @param {string} packageName - ABAP package name
+   * @param {string} branch - Branch name (default: 'main')
+   * @param {string} displayName - Display name for the repository (optional)
+   * @param {string} name - Repository name (optional)
+   * @param {string} folderLogic - Folder logic: 'PREFIX' or 'FULL' (default: 'PREFIX')
+   * @returns {object} Create result with success, repo_key, repo_name, message
+   */
+  async create(repoUrl, packageName, branch = 'main', displayName = null, name = null, folderLogic = 'PREFIX') {
+    logger.info('Creating repository', { repoUrl, packageName, branch });
+
+    try {
+      const result = await this.abap.create(repoUrl, packageName, branch, displayName, name, folderLogic);
+      return {
+        success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
+        repo_key: result.REPO_KEY || result.repo_key,
+        repo_name: result.REPO_NAME || result.repo_name,
+        message: result.MESSAGE || result.message || '',
+        error: result.ERROR || result.error || null
+      };
+    } catch (error) {
+      logger.error('Create failed', { error: error.message });
+      throw new Error(`Create failed: ${error.message}`);
+    }
+  }
+
+  /**
+   * Import existing objects from package to git repository
+   * @param {string} repoUrl - Git repository URL
+   * @param {string} message - Commit message (optional)
+   * @returns {object} Import result with success, files_staged, commit_sha
+   */
+  async import(repoUrl, message = null) {
+    logger.info('Starting import operation', { repoUrl, message });
+
+    try {
+      const result = await this.abap.import(repoUrl, message);
+      return {
+        success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
+        files_staged: result.FILES_STAGED || result.files_staged || 0,
+        commit_sha: result.COMMIT_SHA || result.commit_sha || '',
+        message: result.MESSAGE || result.message || '',
+        error: result.ERROR || result.error || null
+      };
+    } catch (error) {
+      logger.error('Import failed', { error: error.message });
+      throw new Error(`Import failed: ${error.message}`);
+    }
+  }
 }
 
 module.exports = {