abapgit-agent 1.11.0 → 1.11.2

package/abap/guidelines/workflow-detailed.md

@@ -0,0 +1,255 @@
+---
+layout: default
+title: Development Workflow (Detailed)
+nav_order: 15
+parent: ABAP Coding Guidelines
+grand_parent: ABAP Development
+---
+
+# Development Workflow (Detailed)
+
+```
+1. Read .abapGitAgent → get folder value AND workflow.mode
+       │
+       ▼
+2. Research → use ref command for unfamiliar topics
+       │
+       ▼
+3. Write code → place in correct folder (e.g., src/zcl_*.clas.abap)
+       │
+       ▼
+4. Syntax check (for CLAS, INTF, PROG, DDLS only)
+       │
+       ├─► CLAS/INTF/PROG/DDLS → abapgit-agent syntax --files <file>
+       │       │
+       │       ├─► Errors? → Fix locally (no commit needed), re-run syntax
+       │       │
+       │       └─► Clean ✅ → Proceed to commit
+       │
+       └─► Other types (FUGR, TABL, etc.) → Skip syntax, go to commit
+               │
+               ▼
+5. Commit and push → git add . && git commit && git push
+       │
+       ▼
+6. Activate → abapgit-agent pull --files src/file.clas.abap
+       │         (behaviour depends on .abapgit-agent.json — see AI Tool Guidelines)
+       │
+       ▼
+7. Verify → Check pull output
+   - **"Error updating where-used list"** → SYNTAX ERROR (use inspect for details)
+   - Objects in "Failed Objects Log" → Syntax error (use inspect)
+   - Objects NOT appearing at all → XML metadata issue (check abapgit.md)
+       │
+       ▼
+8. (Optional) Run unit tests → abapgit-agent unit --files <testclass> (AFTER successful pull)
+```
+
+**Syntax Command - Supported Object Types:**
+
+| Object Type | Syntax Command | What to Do |
+|-------------|----------------|------------|
+| CLAS (classes) | ✅ Supported | Run `syntax` before commit |
+| CLAS (test classes: .testclasses.abap) | ✅ Supported | Run `syntax` before commit |
+| INTF (interfaces) | ✅ Supported | Run `syntax` before commit |
+| PROG (programs) | ✅ Supported | Run `syntax` before commit |
+| DDLS (CDS views) | ✅ Supported | Run `syntax` before commit (requires annotations) |
+| FUGR (function groups) | ❌ Not supported | Skip syntax, use `pull` then `inspect` |
+| TABL/DTEL/DOMA/MSAG/SHLP | ❌ Not supported | Skip syntax, just `pull` |
+| All other types | ❌ Not supported | Skip syntax, just `pull` |
+
+**IMPORTANT**:
+- **Use `syntax` BEFORE commit** for CLAS/INTF/PROG/DDLS - catches errors early, no git pollution
+- **Syntax checks files INDEPENDENTLY** - syntax checker doesn't have access to uncommitted files
+- **For dependent files** (interface + class): Create/activate underlying object FIRST, then dependent object (see workflow below)
+- **DDLS requires proper annotations** - CDS views need `@AbapCatalog.sqlViewName`, view entities don't
+- **ALWAYS push to git BEFORE running pull** - abapGit reads from git
+- **Use `inspect` AFTER pull** for unsupported types or if pull fails
+
+**Working with dependent objects (RECOMMENDED APPROACH):**
+
+When creating objects with dependencies (e.g., interface → class), create and activate the underlying object FIRST:
+
+```bash
+# Step 1: Create interface, syntax check, commit, activate
+vim src/zif_my_interface.intf.abap
+abapgit-agent syntax --files src/zif_my_interface.intf.abap  # ✅ Works (no dependencies)
+git add src/zif_my_interface.intf.abap src/zif_my_interface.intf.xml
+git commit -m "feat: add interface"
+git push
+abapgit-agent pull --files src/zif_my_interface.intf.abap   # Interface now activated
+
+# Step 2: Create class, syntax check, commit, activate
+vim src/zcl_my_class.clas.abap
+abapgit-agent syntax --files src/zcl_my_class.clas.abap     # ✅ Works (interface already activated)
+git add src/zcl_my_class.clas.abap src/zcl_my_class.clas.xml
+git commit -m "feat: add class implementing interface"
+git push
+abapgit-agent pull --files src/zcl_my_class.clas.abap
+```
+
+**Benefits:**
+- ✅ Syntax checking works for both objects
+- ✅ Each step is validated independently
+- ✅ Easier to debug if something fails
+- ✅ Cleaner workflow
+
+**Alternative approach (when interface design is uncertain):**
+
+If the interface might need changes while implementing the class, commit both together:
+
+```bash
+# Create both files
+vim src/zif_my_interface.intf.abap
+vim src/zcl_my_class.clas.abap
+
+# Skip syntax (files depend on each other), commit together
+git add src/zif_my_interface.intf.abap src/zif_my_interface.intf.xml
+git add src/zcl_my_class.clas.abap src/zcl_my_class.clas.xml
+git commit -m "feat: add interface and implementing class"
+git push
+
+# Pull both together
+abapgit-agent pull --files src/zif_my_interface.intf.abap,src/zcl_my_class.clas.abap
+
+# Use inspect if errors occur
+abapgit-agent inspect --files src/zcl_my_class.clas.abap
+```
+
+**Use this approach when:**
+- ❌ Interface design is still evolving
+- ❌ Multiple iterations expected
+
+**Working with mixed file types:**
+When modifying multiple files of different types (e.g., 1 class + 1 CDS view):
+1. Run `syntax` on independent supported files (CLAS, INTF, PROG, DDLS)
+2. Commit ALL files together (including unsupported types)
+3. Push and pull ALL files together
+
+Example:
+```bash
+# Check syntax on independent files only
+abapgit-agent syntax --files src/zcl_my_class.clas.abap,src/zc_my_view.ddls.asddls
+
+# Commit and push all files
+git add src/zcl_my_class.clas.abap src/zc_my_view.ddls.asddls
+git commit -m "feat: add class and CDS view"
+git push
+
+# Pull all files together
+abapgit-agent pull --files src/zcl_my_class.clas.abap,src/zc_my_view.ddls.asddls
+```
+
+**When to use syntax vs inspect vs view**:
+- **syntax**: Check LOCAL code BEFORE commit (CLAS, INTF, PROG, DDLS)
+- **inspect**: Check ACTIVATED code AFTER pull (all types, runs Code Inspector)
+- **view**: Understand object STRUCTURE (not for debugging errors)
+
+### Quick Decision Tree for AI
+
+**When user asks to modify/create ABAP code:**
+
+```
+1. Identify file extension(s) AND dependencies
+   ├─ .clas.abap or .clas.testclasses.abap → CLAS ✅ syntax supported
+   ├─ .intf.abap → INTF ✅ syntax supported
+   ├─ .prog.abap → PROG ✅ syntax supported
+   ├─ .ddls.asddls → DDLS ✅ syntax supported (requires proper annotations)
+   └─ All other extensions → ❌ syntax not supported
+
+2. Check for dependencies:
+   ├─ Interface + implementing class? → DEPENDENT (interface is underlying)
+   ├─ Class A uses class B? → DEPENDENT (class B is underlying)
+   ├─ CDS view uses table? → INDEPENDENT (table already exists)
+   └─ Unrelated bug fixes across files? → INDEPENDENT
+
+3. For SUPPORTED types (CLAS/INTF/PROG/DDLS):
+   ├─ INDEPENDENT files → Run syntax → Fix errors → Commit → Push → Pull
+   │
+   └─ DEPENDENT files (NEW objects):
+       ├─ RECOMMENDED: Create underlying object first (interface, base class, etc.)
+       │   1. Create underlying object → Syntax → Commit → Push → Pull
+       │   2. Create dependent object → Syntax (works!) → Commit → Push → Pull
+       │   ✅ Benefits: Both syntax checks work, cleaner workflow
+       │
+       └─ ALTERNATIVE: If interface design uncertain, commit both together
+           → Skip syntax → Commit both → Push → Pull → (if errors: inspect)
+
+4. For UNSUPPORTED types (FUGR, TABL, etc.):
+   Write code → Skip syntax → Commit → Push → Pull → (if errors: inspect)
+
+5. For MIXED types (some supported + some unsupported):
+   Write all code → Run syntax on independent supported files ONLY → Commit ALL → Push → Pull ALL
+```
+
+**Example workflows:**
+
+**Scenario 1: Interface + Class (RECOMMENDED)**
+```bash
+# Step 1: Interface first
+vim src/zif_calculator.intf.abap
+abapgit-agent syntax --files src/zif_calculator.intf.abap  # ✅ Works
+git commit -am "feat: add calculator interface" && git push
+abapgit-agent pull --files src/zif_calculator.intf.abap    # Interface activated
+
+# Step 2: Class next
+vim src/zcl_calculator.clas.abap
+abapgit-agent syntax --files src/zcl_calculator.clas.abap  # ✅ Works (interface exists!)
+git commit -am "feat: implement calculator" && git push
+abapgit-agent pull --files src/zcl_calculator.clas.abap
+```
+
+**Scenario 2: Multiple independent classes**
+```bash
+# All syntax checks work (no dependencies)
+vim src/zcl_class1.clas.abap src/zcl_class2.clas.abap
+abapgit-agent syntax --files src/zcl_class1.clas.abap,src/zcl_class2.clas.abap
+git commit -am "feat: add utility classes" && git push
+abapgit-agent pull --files src/zcl_class1.clas.abap,src/zcl_class2.clas.abap
+```
+
+**Error indicators after pull:**
+- ❌ **"Error updating where-used list"** → SYNTAX ERROR - run `inspect` for details
+- ❌ **Objects in "Failed Objects Log"** → SYNTAX ERROR - run `inspect`
+- ❌ **Objects NOT appearing at all** → XML metadata issue (check `ref --topic abapgit`)
+- ⚠️ **"Activated with warnings"** → Code Inspector warnings - run `inspect` to see details
+
+### Commands
+
+```bash
+# 1. Syntax check LOCAL code BEFORE commit (CLAS, INTF, PROG, DDLS)
+abapgit-agent syntax --files src/zcl_my_class.clas.abap
+abapgit-agent syntax --files src/zc_my_view.ddls.asddls
+abapgit-agent syntax --files src/zcl_class1.clas.abap,src/zif_intf1.intf.abap,src/zc_view.ddls.asddls
+
+# 2. Pull/activate AFTER pushing to git
+abapgit-agent pull --files src/zcl_class1.clas.abap,src/zcl_class2.clas.abap
+
+# Override conflict detection for a single pull (e.g. deliberate branch switch)
+abapgit-agent pull --files src/zcl_class1.clas.abap --conflict-mode ignore
+
+# 3. Inspect AFTER pull (for errors or unsupported types)
+abapgit-agent inspect --files src/zcl_class1.clas.abap
+
+# Run unit tests (after successful pull)
+abapgit-agent unit --files src/zcl_test1.clas.testclasses.abap,src/zcl_test2.clas.testclasses.abap
+
+# View object definitions (multiple objects)
+abapgit-agent view --objects ZCL_CLASS1,ZCL_CLASS2,ZIF_INTERFACE
+
+# Preview table data (multiple tables/views)
+abapgit-agent preview --objects ZTABLE1,ZTABLE2
+
+# Explore table structures
+abapgit-agent view --objects ZTABLE --type TABL
+
+# Display package tree
+abapgit-agent tree --package \$MY_PACKAGE
+
+# Investigate runtime errors (ST22 short dumps)
+abapgit-agent dump                                          # Last 7 days
+abapgit-agent dump --user DEVELOPER --date TODAY           # Today's dumps for a user
+abapgit-agent dump --program ZMY_PROGRAM                   # Dumps from a specific program
+abapgit-agent dump --error TIME_OUT                        # Dumps by error type
+abapgit-agent dump --user DEVELOPER --detail 1             # Full detail of first result
+```

package/package.json

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

package/src/commands/debug.js

@@ -34,6 +34,7 @@ const { spawn } = require('child_process');
 const { AdtHttp }      = require('../utils/adt-http');
 const { DebugSession } = require('../utils/debug-session');
 const debugStateModule = require('../utils/debug-state');
+const { printHttpError } = require('../utils/format-error');
 const {
   saveActiveSession,
   loadActiveSession,
@@ -60,11 +61,16 @@ function hasFlag(args, flag) {
 }
 
 /**
- * Determine ADT object URI from object name (class/interface vs program).
+ * Determine ADT object URI from object name (class/interface vs program vs include).
  * Must use /source/main suffix for classes — verified by live testing: ADT
  * rejects breakpoints set on the class root URI but accepts /source/main.
  *   classes  → /sap/bc/adt/oo/classes/<name_lowercase>/source/main
+ *   includes → /sap/bc/adt/programs/includes/<name_lowercase>
  *   programs → /sap/bc/adt/programs/programs/<name_lowercase>
+ *
+ * Class method includes are named <ClassName padded to 30 chars with '='>CM<suffix>
+ * e.g. ZCL_ABGAGT_AGENT=============CM00D
+ * These must be routed to the programs/includes ADT endpoint.
  */
 function objectUri(name) {
   const upper = (name || '').toUpperCase();
@@ -325,17 +331,16 @@ async function cmdSet(args, config, adt) {
     return;
   }
 
-  await adt.fetchCsrfToken();
-  const body = buildBreakpointsXml(config.user, updated);
-
   let resp;
   try {
+    await adt.fetchCsrfToken();
+    const body = buildBreakpointsXml(config.user, updated);
     resp = await adt.post('/sap/bc/adt/debugger/breakpoints', body, {
       contentType: 'application/xml',
       headers: { Accept: 'application/xml' }
     });
   } catch (err) {
-    console.error(`\n  Error: ${err.message || JSON.stringify(err)}\n`);
+    printHttpError(err, { prefix: '  Error' });
     process.exit(1);
   }
 
@@ -357,6 +362,35 @@ async function cmdSet(args, config, adt) {
   });
   if (_saveBpState) _saveBpState(config, updatedWithServerIds);
 
+  // Immediately re-validate the newly added breakpoints: ADT accepts the POST
+  // with HTTP 200 even for invalid positions (e.g. comment lines), but then
+  // silently drops them.  Re-validating here gives immediate feedback instead
+  // of only discovering the failure on the next "debug list".
+  const addedBps = updatedWithServerIds.filter(bp =>
+    added.some(a => a.uri === bp.uri && a.line === bp.line)
+  );
+  const { stale: newlyStale } = await refreshBreakpoints(config, adt, addedBps);
+  if (newlyStale.length > 0) {
+    // Remove stale from state
+    const stillValid = updatedWithServerIds.filter(bp =>
+      !newlyStale.some(s => s.uri === bp.uri && s.line === bp.line)
+    );
+    if (_saveBpState) _saveBpState(config, stillValid);
+
+    if (jsonOutput) {
+      console.log(JSON.stringify({ error: 'Breakpoint not accepted by server', stale: newlyStale.map(b => ({ object: b.object, line: b.line, error: b.error })) }));
+    } else {
+      newlyStale.forEach(({ object, line, error }) => {
+        console.error(`\n  ❌ Breakpoint not accepted: ${object} line ${line}`);
+        console.error(`     Reason: ${error || 'Not registered on server'}`);
+        console.error(`     Tip: line must be an executable statement — not a comment, blank line,`);
+        console.error(`          DATA declaration, or continuation line of a multi-line call.\n`);
+      });
+      process.exit(1);
+    }
+    return;
+  }
+
   if (jsonOutput) {
     const out = added.map(a => {
       const sr = serverResults.find(r => r.uri === a.uri && r.line === a.line);
@@ -461,7 +495,7 @@ async function cmdDelete(args, config, adt) {
       try {
         await adt.delete(`/sap/bc/adt/debugger/breakpoints/${encodeURIComponent(bpId)}`);
       } catch (err) {
-        console.error(`\n  Error: ${err.message || JSON.stringify(err)}\n`);
+        printHttpError(err, { prefix: '  Error' });
         process.exit(1);
       }
       if (jsonOutput) {
@@ -488,11 +522,11 @@ async function cmdDelete(args, config, adt) {
       try {
         await adt.delete(`/sap/bc/adt/debugger/breakpoints?clientId=${encodeURIComponent(ADT_CLIENT_ID)}`);
       } catch (err2) {
-        console.error(`\n  Error: ${err2.message || JSON.stringify(err2)}\n`);
+        printHttpError(err2, { prefix: '  Error' });
         process.exit(1);
       }
     } else {
-      console.error(`\n  Error: ${err.message || JSON.stringify(err)}\n`);
+      printHttpError(err, { prefix: '  Error' });
       process.exit(1);
     }
   }
@@ -622,7 +656,7 @@ async function cmdAttach(args, config, adt) {
           );
           process.exit(1);
         }
-        console.error(`\n  Error from ADT listener: ${err.message || JSON.stringify(err)}\n`);
+        printHttpError(err, { prefix: '  Error from ADT listener' });
         process.exit(1);
       }
 
@@ -708,7 +742,7 @@ async function cmdAttach(args, config, adt) {
     try {
       await session.attach(sessionId, (config.user || '').toUpperCase());
     } catch (e) {
-      console.error(`\n  Error during attach: ${e.message || JSON.stringify(e)}\n`);
+      printHttpError(e, { prefix: '  Error during attach' });
       if (e.body) console.error('  Response body:', e.body.substring(0, 400));
       process.exit(1);
     }
@@ -802,7 +836,7 @@ async function cmdStep(args, config, adt) {
     try {
       resp = await sendDaemonCommand(socketPath, { cmd: 'step', type }, 60000);
     } catch (err) {
-      console.error(`\n  Error: ${err.message}\n`);
+      printHttpError(err, { prefix: '  Error' });
       process.exit(1);
     }
     if (!resp.ok) {
@@ -846,7 +880,7 @@ async function cmdStep(args, config, adt) {
       );
       process.exit(1);
     }
-    console.error(`\n  Error: ${err.message || JSON.stringify(err)}${err.body ? '\n  Body: ' + err.body.substring(0, 600) : ''}\n`);
+    printHttpError(err, { prefix: '  Error' });
     process.exit(1);
   }
 
@@ -892,7 +926,7 @@ async function cmdVars(args, config, adt) {
     try {
       resp = await sendDaemonCommand(socketPath, { cmd: 'vars', name: nameFilter }, 60000);
     } catch (err) {
-      console.error(`\n  Error: ${err.message}\n`);
+      printHttpError(err, { prefix: '  Error' });
       process.exit(1);
     }
     if (!resp.ok) {
@@ -923,7 +957,7 @@ async function cmdVars(args, config, adt) {
       );
       process.exit(1);
     }
-    console.error(`\n  Error: ${err.message || JSON.stringify(err)}\n`);
+    printHttpError(err, { prefix: '  Error' });
     process.exit(1);
   }
 
@@ -962,7 +996,7 @@ async function cmdExpand(expandName, sessionId, socketPath, config, adt, jsonOut
     try {
       result = await session.expandPath(pathParts);
     } catch (err) {
-      console.error(`\n  Error: ${err.message}\n`);
+      printHttpError(err, { prefix: '  Error' });
       process.exit(1);
     }
     const { variable: target, children } = result;
@@ -977,7 +1011,7 @@ async function cmdExpand(expandName, sessionId, socketPath, config, adt, jsonOut
     try {
       resp = await sendDaemonCommand(socketPath, { cmd: 'vars', name: null }, 60000);
     } catch (err) {
-      console.error(`\n  Error: ${err.message}\n`);
+      printHttpError(err, { prefix: '  Error' });
       process.exit(1);
     }
     if (!resp.ok) {
@@ -1009,7 +1043,7 @@ async function cmdExpand(expandName, sessionId, socketPath, config, adt, jsonOut
     try {
       resp = await sendDaemonCommand(socketPath, { cmd: 'expand', id: target.id, meta }, 60000);
     } catch (err) {
-      console.error(`\n  Error: ${err.message}\n`);
+      printHttpError(err, { prefix: '  Error' });
       process.exit(1);
     }
     if (!resp.ok) {
@@ -1070,7 +1104,7 @@ async function cmdStack(args, config, adt) {
     try {
       resp = await sendDaemonCommand(socketPath, { cmd: 'stack' }, 60000);
     } catch (err) {
-      console.error(`\n  Error: ${err.message}\n`);
+      printHttpError(err, { prefix: '  Error' });
       process.exit(1);
     }
     if (!resp.ok) {
@@ -1101,7 +1135,7 @@ async function cmdStack(args, config, adt) {
       );
       process.exit(1);
     }
-    console.error(`\n  Error: ${err.message || JSON.stringify(err)}\n`);
+    printHttpError(err, { prefix: '  Error' });
     process.exit(1);
   }
 
@@ -1164,7 +1198,7 @@ async function cmdTerminate(args, config, adt) {
       );
       process.exit(1);
     }
-    console.error(`\n  Error: ${err.message || JSON.stringify(err)}\n`);
+    printHttpError(err, { prefix: '  Error' });
     process.exit(1);
   }
 

package/src/commands/inspect.js

@@ -3,11 +3,12 @@
  */
 
 const pathModule = require('path');
+const { printHttpError } = require('../utils/format-error');
 
 /**
  * Inspect all files in one request
  */
-async function inspectAllFiles(files, csrfToken, config, variant, http) {
+async function inspectAllFiles(files, csrfToken, config, variant, http, verbose = false) {
   // Convert files to uppercase names
   const fileNames = files.map(f => {
     const baseName = pathModule.basename(f).toUpperCase();
@@ -45,7 +46,7 @@ async function inspectAllFiles(files, csrfToken, config, variant, http) {
 
     return results;
   } catch (error) {
-    console.error(`\n  Error: ${error.message}`);
+    printHttpError(error, { verbose });
     process.exit(1);
   }
 }
@@ -149,6 +150,7 @@ module.exports = {
     const { loadConfig, AbapHttp } = context;
 
     const jsonOutput = args.includes('--json');
+    const verbose = args.includes('--verbose');
     const filesArgIndex = args.indexOf('--files');
     if (filesArgIndex === -1 || filesArgIndex + 1 >= args.length) {
       console.error('Error: --files parameter required');
@@ -178,7 +180,7 @@ module.exports = {
     const csrfToken = await http.fetchCsrfToken();
 
     // Send all files in one request
-    const results = await inspectAllFiles(filesSyntaxCheck, csrfToken, config, variant, http);
+    const results = await inspectAllFiles(filesSyntaxCheck, csrfToken, config, variant, http, verbose);
 
     // JSON output mode
     if (jsonOutput) {

package/src/commands/pull.js

@@ -2,6 +2,8 @@
  * Pull command - Pull and activate ABAP objects from git repository
  */
 
+const { printHttpError } = require('../utils/format-error');
+
 module.exports = {
   name: 'pull',
   description: 'Pull and activate ABAP objects from git repository',
@@ -10,6 +12,7 @@ module.exports = {
 
   async execute(args, context) {
     const { loadConfig, AbapHttp, gitUtils, getTransport, getSafeguards, getConflictSettings, getTransportSettings } = context;
+    const verbose = args.includes('--verbose');
 
     // Check project-level safeguards
     const safeguards = getSafeguards();
@@ -404,7 +407,7 @@ module.exports = {
       if (error._isPullError) {
         throw error;
       }
-      console.error(`\n❌ Error: ${error.message}`);
+      printHttpError(error, { verbose });
       process.exit(1);
     }
   }

package/src/commands/transport.js

@@ -5,6 +5,7 @@
 const VALID_SCOPES = ['mine', 'task', 'all'];
 const VALID_SUBCOMMANDS = ['list', 'create', 'check', 'release'];
 const VALID_TYPES = ['workbench', 'customizing'];
+const { printHttpError } = require('../utils/format-error');
 
 module.exports = {
   name: 'transport',
@@ -16,6 +17,7 @@ module.exports = {
     const { loadConfig, AbapHttp, getTransportSettings } = context;
 
     const jsonOutput = args.includes('--json');
+    const verbose = args.includes('--verbose');
 
     // Determine subcommand (first positional arg, default to 'list')
     const subcommand = args[0] && !args[0].startsWith('-') ? args[0] : 'list';
@@ -65,7 +67,7 @@ module.exports = {
           break;
       }
     } catch (error) {
-      console.error(`❌ Error: ${error.message}`);
+      printHttpError(error, { verbose });
       process.exit(1);
     }
   },

package/src/commands/unit.js

@@ -4,11 +4,12 @@
 
 const pathModule = require('path');
 const fs = require('fs');
+const { formatHttpError } = require('../utils/format-error');
 
 /**
  * Run unit test for a single file
  */
-async function runUnitTestForFile(sourceFile, csrfToken, config, coverage, http, jsonOutput = false) {
+async function runUnitTestForFile(sourceFile, csrfToken, config, coverage, http, jsonOutput = false, verbose = false) {
   if (!jsonOutput) {
     console.log(`  Running unit test for: ${sourceFile}`);
   }
@@ -123,14 +124,12 @@ async function runUnitTestForFile(sourceFile, csrfToken, config, coverage, http,
     }
 
     if (!jsonOutput) {
-      console.error(`\n  ❌ Error: ${error.message}`);
-      if (error.statusCode) {
-        console.error(`     Status Code: ${error.statusCode}`);
-      }
-      if (error.body && typeof error.body === 'string') {
-        // Show first 500 chars of error body
-        const preview = error.body.substring(0, 500);
-        console.error(`     Response: ${preview}${error.body.length > 500 ? '...' : ''}`);
+      console.error(`\n  ❌ Error: ${formatHttpError(error)}`);
+      if (verbose && error.body) {
+        console.error('\n--- Raw response body ---');
+        const raw = typeof error.body === 'object' ? JSON.stringify(error.body, null, 2) : String(error.body);
+        console.error(raw);
+        console.error('--- End of response body ---');
       }
     }
 
@@ -148,6 +147,7 @@ module.exports = {
     const { loadConfig, AbapHttp } = context;
 
     const jsonOutput = args.includes('--json');
+    const verbose = args.includes('--verbose');
     const filesArgIndex = args.indexOf('--files');
     if (filesArgIndex === -1 || filesArgIndex + 1 >= args.length) {
       console.error('Error: --files parameter required');
@@ -177,7 +177,7 @@ module.exports = {
     let hasErrors = false;
 
     for (const sourceFile of files) {
-      const result = await runUnitTestForFile(sourceFile, csrfToken, config, coverage, http, jsonOutput);
+      const result = await runUnitTestForFile(sourceFile, csrfToken, config, coverage, http, jsonOutput, verbose);
       if (result) {
         results.push(result);