abapgit-agent 1.13.1 → 1.13.3

package/README.md

@@ -115,7 +115,6 @@ npm run pull -- --url <git-url> --branch main
 | Installation & Setup | [INSTALL.md](INSTALL.md) |
 | All Commands Overview | [docs/commands.md](docs/commands.md) |
 | REST API Reference | [API.md](API.md) |
-| Error Handling | [ERROR_HANDLING.md](ERROR_HANDLING.md) |
 
 ## Dependent Projects
 

package/abap/CLAUDE.md

@@ -359,7 +359,7 @@ edit src/zcl_auth_handler.clas.abap
 abapgit-agent syntax --files src/zcl_auth_handler.clas.abap
 git add . && git commit -m "feat: add authentication handler"
 git push origin main
-abapgit-agent pull --files src/zcl_auth_handler.clas.abap
+abapgit-agent pull --files src/zcl_auth_handler.clas.abap --sync-xml
 ```
 
 ### AI Tool Guidelines
@@ -425,6 +425,11 @@ abapgit-agent pull --files src/zcl_auth_handler.clas.abap
 1. ✗ Do not run `abapgit-agent transport release`
 2. ✓ Inform the user that transport release is disabled for this project
 
+**After every pull that creates or modifies ABAP objects:**
+1. ✓ Always pass `--sync-xml` — rewrites any XML metadata files that differ from the ABAP serializer output, amends the commit, and re-pulls so git and the ABAP system stay in sync
+2. ✓ If pull output shows `⚠️  X XML file(s) differ from serializer output`, re-run immediately with `--sync-xml`
+3. ✗ Never leave a pull without `--sync-xml` when you authored the objects — abapGit will show **M (modified)** permanently otherwise
+
 ---
 
 ### Quick Decision Tree for AI
@@ -435,11 +440,11 @@ abapgit-agent pull --files src/zcl_auth_handler.clas.abap
 Modified ABAP files?
 ├─ CLAS/INTF/PROG/DDLS files?
 │  ├─ Independent files (no cross-dependencies)?
-│  │  └─ ✅ Use: syntax → commit → push → pull
+│  │  └─ ✅ Use: syntax → commit → push → pull --sync-xml
 │  └─ Dependent files (interface + class, class uses class)?
-│     └─ ✅ Use: skip syntax → commit → push → pull
+│     └─ ✅ Use: skip syntax → commit → push → pull --sync-xml
 └─ Other types (DDLS, FUGR, TABL, etc.)?
-   └─ ✅ Use: skip syntax → commit → push → pull → (if errors: inspect)
+   └─ ✅ Use: skip syntax → commit → push → pull --sync-xml → (if errors: inspect)
 ```
 
 → See `guidelines/workflow-detailed.md` — run: `abapgit-agent ref --topic workflow-detailed`

package/abap/CLAUDE.slim.md

@@ -10,6 +10,8 @@ Read the full ABAP development guide by running:
 abapgit-agent guide
 ```
 
+**Never pipe `abapgit-agent guide` or `abapgit-agent ref` through `head`, `tail`, or any other truncation command. Always read the full output.**
+
 This guide covers: development workflow, ABAP syntax guidelines, object naming, unit testing, and debugging.
 
 > **Humans:** Full guide online at https://sylvoscai.github.io/abapgit-agent/pages/abap-coding-guidelines.html

package/abap/guidelines/abapgit.md

@@ -26,16 +26,12 @@ Structure          | z*.stru.abap                | z*.stru.xml
 Table Type         | z*.ttyp.abap                | z*.ttyp.xml
 ```
 
-```
-Key XML Settings:
-  Class EXPOSURE:   2=Public, 3=Protected, 4=Private
-  Class STATE:      1=Active
-  Table TABCLASS:   TRANSP, POOL, CLUSTER
-  Table DELIVERY:   A=Application, C=Customizing
-  CDS SOURCE_TYPE:  W=View Entity (modern), V=View (legacy)
-  Test Class XML:   <WITH_UNIT_TESTS>X</WITH_UNIT_TESTS>
-  Local Classes:    <CLSCCINCL>X</CLSCCINCL>
-```
+> **CRITICAL: Always write XML files with a UTF-8 BOM (`\ufeff`) as the very first character**, before `<?xml ...`.
+> Without the BOM, abapGit shows the object as **"M" (modified)** after every pull because the
+> serializer always produces XML with BOM — and every byte matters.
+
+> **CRITICAL: Only include fields that abapGit's serializer actually writes. Never add fields with
+> default values.** Extra fields cause a permanent "M" (modified) diff. Follow the exact templates below.
 
 **Searchable keywords**: class xml, interface xml, table xml, cds xml, test class, exposure, serializer, abapgit
 
@@ -46,14 +42,50 @@ abapGit needs XML files to:
 - Store object attributes (description, exposure, state, etc.)
 - Handle object-specific configurations
 
+## Field Presence Rules (CRITICAL)
+
+abapGit's serializer **omits fields that have their default value**. Writing extra fields causes permanent
+"M" (modified) status in abapGit UI. Follow these rules strictly:
+
+### CLAS — Field Presence Rules
+
+| Field | Include when | Omit when |
+|---|---|---|
+| `CLSNAME` | Always | — |
+| `LANGU` | Always | — |
+| `DESCRIPT` | Always | — |
+| `CATEGORY` | Non-standard class (`40`=exception, `05`=test double) | Standard class (default `00`) — **omit** |
+| `EXPOSURE` | **Never for CLAS** | **Always omit** — public (`2`) is the default and is never written |
+| `STATE` | Always (`1`) | — |
+| `CLSCCINCL` | `.clas.locals_def.abap` file exists | No local class files — **omit** |
+| `FIXPT` | Always (`X`) | — |
+| `UNICODE` | Always (`X`) | — |
+| `WITH_UNIT_TESTS` | `.clas.testclasses.abap` file exists | No test class file — **omit** |
+| `MSG_ID` | Class has a message class | No message class — **omit** |
+
+**Field order**: `CLSNAME → LANGU → DESCRIPT → [CATEGORY] → STATE → [CLSCCINCL] → FIXPT → UNICODE → [WITH_UNIT_TESTS] → [MSG_ID]`
+
+### INTF — Field Presence Rules
+
+| Field | Include when | Omit when |
+|---|---|---|
+| `CLSNAME` | Always | — |
+| `LANGU` | Always | — |
+| `DESCRIPT` | Always | — |
+| `EXPOSURE` | Always (`2`) | — (interfaces always have EXPOSURE, unlike classes) |
+| `STATE` | Always (`1`) | — |
+| `UNICODE` | Always (`X`) | — |
+
 ## Object Types and XML Templates
 
 ### Class (CLAS)
 
 **Filename**: `src/zcl_my_class.clas.xml`
 
+**Standard public class** (no local includes, no test class):
+
 ```xml
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8"?>
 <abapGit version="v1.0.0" serializer="LCL_OBJECT_CLAS" serializer_version="v1.0.0">
  <asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
   <asx:values>
@@ -61,29 +93,85 @@ abapGit needs XML files to:
     <CLSNAME>ZCL_MY_CLASS</CLSNAME>
     <LANGU>E</LANGU>
     <DESCRIPT>Description of the class</DESCRIPT>
-    <EXPOSURE>2</EXPOSURE>
     <STATE>1</STATE>
+    <FIXPT>X</FIXPT>
     <UNICODE>X</UNICODE>
+   </VSEOCLASS>
+  </asx:values>
+ </asx:abap>
+</abapGit>
+```
+
+**Class with test class** (`.clas.testclasses.abap` exists — add `WITH_UNIT_TESTS`):
+
+```xml
+<?xml version="1.0" encoding="utf-8"?>
+<abapGit version="v1.0.0" serializer="LCL_OBJECT_CLAS" serializer_version="v1.0.0">
+ <asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
+  <asx:values>
+   <VSEOCLASS>
+    <CLSNAME>ZCL_MY_CLASS</CLSNAME>
+    <LANGU>E</LANGU>
+    <DESCRIPT>Description of the class</DESCRIPT>
+    <STATE>1</STATE>
     <FIXPT>X</FIXPT>
+    <UNICODE>X</UNICODE>
+    <WITH_UNIT_TESTS>X</WITH_UNIT_TESTS>
    </VSEOCLASS>
   </asx:values>
  </asx:abap>
 </abapGit>
 ```
 
-**Key Fields**:
-- `CLSNAME`: Class name (must match filename)
-- `DESCRIPT`: Class description
-- `EXPOSURE`: Exposure (2 = Public, 3 = Protected, 4 = Private)
-- `STATE`: State (1 = Active)
-- `UNICODE`: Unicode encoding (X = Yes)
-- `FIXPT`: Fixed-point arithmetic (X = Yes) - **Always include for correct decimal arithmetic**
+**Class with local includes** (`.clas.locals_def.abap` exists — add `CLSCCINCL` before `FIXPT`):
 
-**Note**: `<FIXPT>X</FIXPT>` is default for modern ABAP. Without it, decimals treated as integers.
+```xml
+<?xml version="1.0" encoding="utf-8"?>
+<abapGit version="v1.0.0" serializer="LCL_OBJECT_CLAS" serializer_version="v1.0.0">
+ <asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
+  <asx:values>
+   <VSEOCLASS>
+    <CLSNAME>ZCL_MY_CLASS</CLSNAME>
+    <LANGU>E</LANGU>
+    <DESCRIPT>Description of the class</DESCRIPT>
+    <STATE>1</STATE>
+    <CLSCCINCL>X</CLSCCINCL>
+    <FIXPT>X</FIXPT>
+    <UNICODE>X</UNICODE>
+    <WITH_UNIT_TESTS>X</WITH_UNIT_TESTS>
+   </VSEOCLASS>
+  </asx:values>
+ </asx:abap>
+</abapGit>
+```
+
+**Exception class** (`CATEGORY>40` — add `CATEGORY` after `DESCRIPT`):
+
+```xml
+<?xml version="1.0" encoding="utf-8"?>
+<abapGit version="v1.0.0" serializer="LCL_OBJECT_CLAS" serializer_version="v1.0.0">
+ <asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
+  <asx:values>
+   <VSEOCLASS>
+    <CLSNAME>ZCX_MY_EXCEPTION</CLSNAME>
+    <LANGU>E</LANGU>
+    <DESCRIPT>My exception</DESCRIPT>
+    <CATEGORY>40</CATEGORY>
+    <STATE>1</STATE>
+    <CLSCCINCL>X</CLSCCINCL>
+    <FIXPT>X</FIXPT>
+    <UNICODE>X</UNICODE>
+   </VSEOCLASS>
+  </asx:values>
+ </asx:abap>
+</abapGit>
+```
 
-**Local Classes**: If the class has local classes (e.g., test doubles), add:
-- `<WITH_UNIT_TESTS>X</WITH_UNIT_TESTS>` - for test classes
-- `<CLSCCINCL>X</CLSCCINCL>` - for local class definitions
+**Key rules**:
+- ❌ **Never include `<EXPOSURE>`** in a CLAS XML — public (2) is the default and abapGit omits it
+- ✅ Always include `<FIXPT>X</FIXPT>` and `<UNICODE>X</UNICODE>`
+- ✅ Add `<WITH_UNIT_TESTS>X</WITH_UNIT_TESTS>` only when `.clas.testclasses.abap` exists
+- ✅ Add `<CLSCCINCL>X</CLSCCINCL>` only when `.clas.locals_def.abap` exists
 
 **Local Class Files**:
 | File | Purpose |
@@ -98,7 +186,7 @@ abapGit needs XML files to:
 **Filename**: `src/zif_my_interface.intf.xml`
 
 ```xml
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8"?>
 <abapGit version="v1.0.0" serializer="LCL_OBJECT_INTF" serializer_version="v1.0.0">
  <asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
   <asx:values>
@@ -115,6 +203,9 @@ abapGit needs XML files to:
 </abapGit>
 ```
 
+**Key rules**:
+- ✅ `<EXPOSURE>2</EXPOSURE>` is **always present** for interfaces (unlike classes where it is omitted)
+
 ---
 
 ### Program (PROG)
@@ -122,13 +213,13 @@ abapGit needs XML files to:
 **Filename**: `src/zmy_program.prog.xml`
 
 ```xml
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8"?>
 <abapGit version="v1.0.0" serializer="LCL_OBJECT_PROG" serializer_version="v1.0.0">
  <asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
   <asx:values>
    <PROGDIR>
     <NAME>ZMY_PROGRAM</NAME>
-    <SUBC>I</SUBC>
+    <SUBC>1</SUBC>
     <RLOAD>E</RLOAD>
     <FIXPT>X</FIXPT>
     <UCCHECK>X</UCCHECK>
@@ -140,8 +231,12 @@ abapGit needs XML files to:
 
 **Key Fields**:
 - `NAME`: Program name
-- `SUBC`: Subc (I = Include, 1 = Executable, F = Function Group, M = Module Pool, S = Subroutine Pool)
-- `RLOAD`: Rload (E = External, I = Internal)
+- `SUBC`: Program type (`1`=Executable, `I`=Include, `F`=Function Group, `M`=Module Pool, `S`=Subroutine Pool)
+- `RLOAD`: `E`=External
+- `FIXPT`: Fixed-point arithmetic (`X`=Yes) — include for executables
+- `UCCHECK`: Unicode checks active (`X`=Yes)
+
+**Note**: The serializer may also write a `<TPOOL>` section after `<PROGDIR>` if the program has a title text. You do not need to write this when creating new programs — abapGit will add it on the next pull if needed.
 
 ---
 
@@ -150,7 +245,7 @@ abapGit needs XML files to:
 **Filename**: `src/zmy_table.tabl.xml`
 
 ```xml
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8"?>
 <abapGit version="v1.0.0" serializer="LCL_OBJECT_TABL" serializer_version="v1.0.0">
  <asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
   <asx:values>
@@ -168,9 +263,20 @@ abapGit needs XML files to:
 
 **Key Fields**:
 - `TABNAME`: Table name
-- `DDTEXT`: Description (NOT DESCRIPT)
-- `TABCLASS`: Table class (TRANSP = Transparent)
-- `CONTFLAG`: Delivery class (A = Application table)
+- `DDTEXT`: Description (**not** `DESCRIPT`)
+- `TABCLASS`: `TRANSP`=Transparent (most common), `POOL`, `CLUSTER`
+- `CONTFLAG`: Delivery class — `A`=Application, `C`=Customizing, `S`=System, `G`=Customizing protected
+
+**Note**: When abapGit serializes an existing table it also writes `<DD09L>` (technical settings) and `<DD03P_TABLE>` (field definitions). These sections are generated automatically from the ABAP Dictionary on pull — you only need the `<DD02V>` header when creating a new table. After the first pull the XML will be expanded with those sections.
+
+**`DD03P` field-level rules** (apply when editing an existing table XML that includes `<DD03P_TABLE>`):
+
+| Rule | Detail |
+|---|---|
+| `SHLPORIGIN` | Include `<SHLPORIGIN>D</SHLPORIGIN>` on fields where the Dictionary provides a value help (e.g. fields with a domain that has fixed values or a search help). Omit on fields with no value help. |
+| Field order for raw-type fields | For fields with no `ROLLNAME` (raw type, e.g. `CHAR`, `NUMC`), the serializer writes `<MASK>` **before** `<DDTEXT>`. For fields with a `ROLLNAME`, only `ROLLNAME` appears (no `MASK` or `DDTEXT`). |
+
+**Why this matters**: Missing `SHLPORIGIN` or wrong `MASK`/`DDTEXT` order causes a permanent diff between git and the system-serialized XML.
 
 ---
 
@@ -181,7 +287,7 @@ abapGit needs XML files to:
 The XML format is identical for both types — only `SOURCE_TYPE` differs:
 
 ```xml
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8"?>
 <abapGit version="v1.0.0" serializer="LCL_OBJECT_DDLS" serializer_version="v1.0.0">
  <asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
   <asx:values>
@@ -208,7 +314,7 @@ The XML format is identical for both types — only `SOURCE_TYPE` differs:
 **Filename**: `src/zmy_dtel.dtel.xml`
 
 ```xml
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8"?>
 <abapGit version="v1.0.0" serializer="LCL_OBJECT_DTEL" serializer_version="v1.0.0">
  <asx:abap xmlns:asx="http://www.sap.com/abapxml" version="1.0">
   <asx:values>
@@ -231,9 +337,9 @@ The XML format is identical for both types — only `SOURCE_TYPE` differs:
 
 **Key Fields**:
 - `ROLLNAME`: Data element name
-- `DDTEXT`: Description (NOT DESCRIPT)
-- `DATATYPE`: Data type (CHAR, NUMC, etc.)
-- `LENG`: Length (e.g., 000010 for 10 characters)
+- `DDTEXT`: Description (**not** `DESCRIPT`)
+- `DATATYPE`: Data type (`CHAR`, `NUMC`, `DATS`, `TIMS`, `INT4`, etc.)
+- `LENG`: Length padded to 6 digits (e.g. `000010` for 10 characters)
 
 ---
 

package/bin/abapgit-agent

@@ -34,32 +34,7 @@ async function main() {
   const args = process.argv.slice(2);
   const command = args[0];
 
-  // 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'),
-    dump: require('../src/commands/dump'),
-    debug: require('../src/commands/debug'),
-    run: require('../src/commands/run'),
-    ref: require('../src/commands/ref'),
-    guide: require('../src/commands/guide'),
-    init: require('../src/commands/init'),
-    pull: require('../src/commands/pull'),
-    upgrade: require('../src/commands/upgrade'),
-    transport: require('../src/commands/transport')
-  };
+  const commandModules = require('../src/commands/index');
 
   // Check if this is a modular command
   if (commandModules[command] || command === '--help' || command === '-h') {

package/package.json

@@ -1,8 +1,7 @@
 {
   "name": "abapgit-agent",
-  "version": "1.13.1",
+  "version": "1.13.3",
   "description": "ABAP Git Agent - Pull and activate ABAP code via abapGit from any git repository",
-  "main": "src/index.js",
   "files": [
     "bin/",
     "src/",
@@ -18,8 +17,6 @@
     "abgagt": "bin/abgagt"
   },
   "scripts": {
-    "start": "node src/server.js",
-    "dev": "nodemon src/server.js",
     "test": "jest",
     "test:all": "node tests/run-all.js",
     "test:unit": "jest",
@@ -30,6 +27,7 @@
     "test:cmd:demo": "node tests/run-all.js --cmd --demo",
     "test:cmd:syntax": "node tests/run-all.js --cmd --command=syntax",
     "test:cmd:pull": "node tests/run-all.js --cmd --command=pull",
+    "test:sync-xml": "node tests/run-all.js --sync-xml",
     "test:cmd:inspect": "node tests/run-all.js --cmd --command=inspect",
     "test:cmd:unit": "node tests/run-all.js --cmd --command=unit",
     "test:cmd:view": "node tests/run-all.js --cmd --command=view",
@@ -51,16 +49,13 @@
     "unrelease": "node scripts/unrelease.js"
   },
   "dependencies": {
-    "cors": "^2.8.5",
     "dotenv": "^16.3.1",
-    "express": "^4.18.2",
     "node-fetch": "^2.7.0",
     "uuid": "^9.0.0",
     "winston": "^3.11.0"
   },
   "devDependencies": {
     "jest": "^29.7.0",
-    "nodemon": "^3.0.1",
     "supertest": "^7.2.2"
   },
   "engines": {

package/src/commands/index.js

@@ -0,0 +1,33 @@
+'use strict';
+
+/**
+ * abapgit-agent command registry
+ *
+ * Each entry maps a CLI command name to its command module.
+ * Command modules must export: { name, description, requiresAbapConfig, execute }
+ */
+module.exports = {
+  help:      require('./help'),
+  health:    require('./health'),
+  status:    require('./status'),
+  create:    require('./create'),
+  delete:    require('./delete'),
+  import:    require('./import'),
+  inspect:   require('./inspect'),
+  unit:      require('./unit'),
+  syntax:    require('./syntax'),
+  tree:      require('./tree'),
+  list:      require('./list'),
+  view:      require('./view'),
+  preview:   require('./preview'),
+  where:     require('./where'),
+  dump:      require('./dump'),
+  debug:     require('./debug'),
+  run:       require('./run'),
+  ref:       require('./ref'),
+  guide:     require('./guide'),
+  init:      require('./init'),
+  pull:      require('./pull'),
+  upgrade:   require('./upgrade'),
+  transport: require('./transport')
+};

package/src/commands/pull.js

@@ -5,16 +5,45 @@
 const { printHttpError } = require('../utils/format-error');
 const fs = require('fs');
 const pathModule = require('path');
-
+const { execSync } = require('child_process');
+
+// Calculate display width accounting for emoji (2 cells) vs ASCII (1 cell)
+function 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) { i += 1; continue; }  // Zero width joiner
+    // 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
+function padToWidth(str, width) {
+  const s = str || '';
+  const currentWidth = calcWidth(s);
+  return s + ' '.repeat(Math.max(0, width - currentWidth));
+}
 module.exports = {
   name: 'pull',
   description: 'Pull and activate ABAP objects from git repository',
   requiresAbapConfig: true,
   requiresVersionCheck: true,
+  _calcWidth: calcWidth,
+  _padToWidth: padToWidth,
 
   async execute(args, context) {
     const { loadConfig, AbapHttp, gitUtils, getTransport, getSafeguards, getConflictSettings, getTransportSettings } = context;
     const verbose = args.includes('--verbose');
+    const syncXml = args.includes('--sync-xml');
 
     // Check project-level safeguards
     const safeguards = getSafeguards();
@@ -171,13 +200,13 @@ module.exports = {
       }
     }
 
-    await this.pull(gitUrl, branch, files, transportRequest, loadConfig, AbapHttp, jsonOutput, undefined, conflictMode, verbose);
+    await this.pull(gitUrl, branch, files, transportRequest, loadConfig, AbapHttp, jsonOutput, undefined, conflictMode, verbose, syncXml);
   },
 
-  async pull(gitUrl, branch = 'main', files = null, transportRequest = null, loadConfig, AbapHttp, jsonOutput = false, gitCredentials = undefined, conflictMode = 'abort', verbose = false) {
+  async pull(gitUrl, branch = 'main', files = null, transportRequest = null, loadConfig, AbapHttp, jsonOutput = false, gitCredentials = undefined, conflictMode = 'abort', verbose = false, syncXml = false, isRepull = false) {
     const TERM_WIDTH = process.stdout.columns || 80;
 
-    if (!jsonOutput) {
+    if (!jsonOutput && !isRepull) {
       console.log(`\n🚀 Starting pull for: ${gitUrl}`);
       console.log(`   Branch: ${branch}`);
       if (files && files.length > 0) {
@@ -281,44 +310,6 @@ module.exports = {
         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!`);
@@ -449,6 +440,71 @@ module.exports = {
         throw err;
       }
 
+      // --- Post-pull XML sync ---
+      // abapGit's status calculation already identified which XML files differ
+      // (match=false) — only those are returned in local_xml_files.
+      const localXmlFiles = result.local_xml_files || result.LOCAL_XML_FILES || [];
+
+      if (localXmlFiles.length > 0) {
+        const diffFiles = [];
+        for (const f of localXmlFiles) {
+          const relPath = ((f.path || f.PATH || '') + (f.filename || f.FILENAME || '')).replace(/^\//, '');
+          const absPath = pathModule.join(process.cwd(), relPath);
+          if (!fs.existsSync(absPath)) continue;
+          const incoming = Buffer.from(f.data || f.DATA, 'base64');
+          // Double-check: only write if bytes actually differ (guard against encoding quirks)
+          const current = fs.readFileSync(absPath);
+          if (!current.equals(incoming)) {
+            diffFiles.push({ relPath, absPath, incoming });
+          }
+        }
+
+        if (diffFiles.length > 0 && !syncXml) {
+          console.log(`\n⚠️  ${diffFiles.length} XML file(s) differ from serializer output:`);
+          for (const f of diffFiles) console.log(`   ${f.relPath}`);
+          console.log(`   Run with --sync-xml to accept serializer output and amend the last commit`);
+        } else if (diffFiles.length > 0 && syncXml) {
+          console.log(`\n🔄 Syncing ${diffFiles.length} XML file(s) to match serializer output:`);
+          for (const f of diffFiles) console.log(`   ${f.relPath}`);
+
+          // 1. Write serializer XML to disk
+          for (const f of diffFiles) fs.writeFileSync(f.absPath, f.incoming);
+
+          // 2. Stage changed XML files
+          const quotedPaths = diffFiles.map(f => `"${f.relPath}"`).join(' ');
+          execSync(`git add ${quotedPaths}`, { cwd: process.cwd() });
+
+          // 3. Amend last commit
+          execSync('git commit --amend --no-edit', { cwd: process.cwd() });
+
+          // 4. Push with force-with-lease; if no upstream, set it automatically
+          let pushed = false;
+          try {
+            execSync('git push --force-with-lease', { cwd: process.cwd(), stdio: 'pipe' });
+            pushed = true;
+          } catch (pushErr) {
+            const msg = (pushErr.stderr || pushErr.stdout || pushErr.message || '').toString();
+            if (msg.includes('no upstream branch') || msg.includes('has no upstream')) {
+              // Branch not yet pushed — set upstream and force push (amend requires force)
+              execSync(`git push --force-with-lease --set-upstream origin ${branch}`, { cwd: process.cwd(), stdio: 'pipe' });
+              pushed = true;
+            }
+            // Any other push error (no remote at all, auth failure, etc.) → skip silently
+          }
+
+          if (pushed) {
+            console.log(`   Re-pulling so ABAP system matches the amended commit...`);
+            // 5. Re-pull so ABAP system matches the amended commit (no sync loop)
+            await this.pull(gitUrl, branch, files, transportRequest, loadConfig, AbapHttp, jsonOutput, gitCredentials, conflictMode, verbose, false, true);
+            console.log(`\n✅ Synced ${diffFiles.length} XML file(s), amended commit, re-pulled`);
+          } else {
+            // No remote at all — files are written and committed locally
+            console.log(`\n✅ Synced ${diffFiles.length} XML file(s), amended commit`);
+            console.log(`   Push skipped (no remote). Push manually to sync with remote.`);
+          }
+        }
+      }
+
       return result;
     } catch (error) {
       if (error._isPullError) {

package/src/commands/view.js

@@ -167,6 +167,8 @@ module.exports = {
   description: 'View ABAP object definitions from ABAP system',
   requiresAbapConfig: true,
   requiresVersionCheck: true,
+  _buildMethodLineMap: buildMethodLineMap,
+  _findFirstExecutableLine: findFirstExecutableLine,
 
   async execute(args, context) {
     const { loadConfig, AbapHttp } = context;

package/src/agent.js

@@ -1,512 +0,0 @@
-/**
- * ABAP Git Agent - Main agent class
- * Uses AbapHttp for all ABAP communication
- */
-
-const { AbapHttp } = require('./utils/abap-http');
-const { getAbapConfig } = require('./config');
-const logger = require('./logger');
-
-class ABAPGitAgent {
-  constructor() {
-    this.config = getAbapConfig();
-    this.http = new AbapHttp(this.config);
-  }
-
-  /**
-   * Pull repository and activate objects
-   * @param {string} repoUrl - Git repository URL
-   * @param {string} branch - Branch name (default: main)
-   * @param {string} username - Git username (optional)
-   * @param {string} password - Git password/token (optional)
-   * @param {Array} files - Specific files to pull (optional)
-   * @param {string} transportRequest - Transport request number (optional)
-   * @returns {object} Pull result with success, job_id, message, error_detail
-   */
-  async pull(repoUrl, branch = 'main', username = null, password = null, files = null, transportRequest = null) {
-    logger.info('Starting pull operation', { repoUrl, branch, username: !!username, files, transportRequest });
-
-    try {
-      const csrfToken = await this.http.fetchCsrfToken();
-
-      const data = {
-        url: repoUrl,
-        branch: branch,
-        username: username || this.config.gitUsername,
-        password: password || this.config.gitPassword
-      };
-
-      if (files && files.length > 0) {
-        data.files = files;
-      }
-
-      if (transportRequest) {
-        data.transport_request = transportRequest;
-      }
-
-      const result = await this.http.post('/sap/bc/z_abapgit_agent/pull', data, { csrfToken });
-
-      // Return the result directly from ABAP (handle uppercase keys from /UI2/CL_JSON)
-      return {
-        success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
-        job_id: result.JOB_ID || result.job_id,
-        message: result.MESSAGE || result.message,
-        error_detail: result.ERROR_DETAIL || result.error_detail || null,
-        activated_count: result.ACTIVATED_COUNT || result.activated_count || 0,
-        failed_count: result.FAILED_COUNT || result.failed_count || 0,
-        activated_objects: result.ACTIVATED_OBJECTS || result.activated_objects || [],
-        failed_objects: result.FAILED_OBJECTS || result.failed_objects || []
-      };
-
-    } catch (error) {
-      logger.error('Pull failed', { error: error.message });
-      throw new Error(`Pull failed: ${error.message}`);
-    }
-  }
-
-  /**
-   * Health check
-   * @returns {object} Health status
-   */
-  async healthCheck() {
-    try {
-      const result = await this.http.get('/sap/bc/z_abapgit_agent/health');
-      return {
-        status: 'healthy',
-        abap: 'connected',
-        version: result.version || '1.4.0'
-      };
-    } catch (error) {
-      return {
-        status: 'unhealthy',
-        abap: 'disconnected',
-        error: error.message
-      };
-    }
-  }
-
-  /**
-   * Check syntax of an ABAP object (legacy - not used by CLI)
-   * @param {string} objectType - ABAP object type (e.g., 'CLAS', 'PROG', 'INTF')
-   * @param {string} objectName - ABAP object name
-   * @returns {object} Syntax check result with errors (if any)
-   */
-  async syntaxCheck(objectType, objectName) {
-    logger.info('Starting syntax check', { objectType, objectName });
-
-    try {
-      const csrfToken = await this.http.fetchCsrfToken();
-      const data = {
-        object_type: objectType,
-        object_name: objectName
-      };
-
-      const result = await this.http.post('/sap/bc/z_abapgit_agent/syntax-check', data, { csrfToken });
-      return {
-        success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
-        object_type: result.OBJECT_TYPE || result.object_type,
-        object_name: result.OBJECT_NAME || result.object_name,
-        error_count: result.ERROR_COUNT || result.error_count || 0,
-        errors: result.ERRORS || result.errors || []
-      };
-    } catch (error) {
-      logger.error('Syntax check failed', { error: error.message });
-      throw new Error(`Syntax check failed: ${error.message}`);
-    }
-  }
-
-  /**
-   * Check syntax of ABAP source code directly (without pull/activation)
-   * Supported types: CLAS, INTF, PROG
-   * @param {Array} objects - Array of {type, name, source, locals_def?, locals_imp?}
-   * @param {string} uccheck - Unicode check mode ('X' for Standard, '5' for Cloud)
-   * @returns {object} Syntax check results with success, results array
-   */
-  async syntaxCheckSource(objects, uccheck = 'X') {
-    logger.info('Starting source syntax check', { objectCount: objects.length, uccheck });
-
-    try {
-      const csrfToken = await this.http.fetchCsrfToken();
-      const data = {
-        objects: objects,
-        uccheck: uccheck
-      };
-
-      const result = await this.http.post('/sap/bc/z_abapgit_agent/syntax', data, { csrfToken });
-      const success = result.SUCCESS === 'X' || result.SUCCESS === true ||
-                      result.success === 'X' || result.success === true;
-      const results = result.RESULTS || result.results || [];
-      const message = result.MESSAGE || result.message || '';
-
-      // Normalize results
-      const normalizedResults = results.map(r => ({
-        object_type: r.OBJECT_TYPE || r.object_type,
-        object_name: r.OBJECT_NAME || r.object_name,
-        success: r.SUCCESS === 'X' || r.SUCCESS === true || r.success === 'X' || r.success === true,
-        error_count: r.ERROR_COUNT || r.error_count || 0,
-        errors: r.ERRORS || r.errors || [],
-        warnings: r.WARNINGS || r.warnings || [],
-        message: r.MESSAGE || r.message || ''
-      }));
-
-      return {
-        success,
-        message,
-        results: normalizedResults
-      };
-    } catch (error) {
-      logger.error('Source syntax check failed', { error: error.message });
-      throw new Error(`Source syntax check failed: ${error.message}`);
-    }
-  }
-
-  /**
-   * Run unit tests for package or objects
-   * @param {string} packageName - Package name to run tests for (optional)
-   * @param {Array} objects - Array of {object_type, object_name} objects (optional)
-   * @returns {object} Unit test results with test_count, passed_count, failed_count, results
-   */
-  async unitCheck(packageName = null, objects = []) {
-    logger.info('Starting unit tests', { package: packageName, objects });
-
-    try {
-      const csrfToken = await this.http.fetchCsrfToken();
-      const data = {};
-
-      if (packageName) {
-        data.package = packageName;
-      }
-
-      if (objects && objects.length > 0) {
-        data.objects = objects;
-      }
-
-      const result = await this.http.post('/sap/bc/z_abapgit_agent/unit', data, { csrfToken });
-      return {
-        success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
-        test_count: result.TEST_COUNT || result.test_count || 0,
-        passed_count: result.PASSED_COUNT || result.passed_count || 0,
-        failed_count: result.FAILED_COUNT || result.failed_count || 0,
-        message: result.MESSAGE || result.message || '',
-        errors: result.ERRORS || result.errors || []
-      };
-    } catch (error) {
-      logger.error('Unit tests failed', { error: error.message });
-      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 csrfToken = await this.http.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;
-      }
-
-      const result = await this.http.post('/sap/bc/z_abapgit_agent/create', data, { csrfToken });
-      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 csrfToken = await this.http.fetchCsrfToken();
-      const data = {
-        url: repoUrl
-      };
-
-      if (message) {
-        data.message = message;
-      }
-
-      const result = await this.http.post('/sap/bc/z_abapgit_agent/import', data, { csrfToken });
-      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}`);
-    }
-  }
-
-  /**
-   * Get package hierarchy tree
-   * @param {string} packageName - ABAP package name
-   * @param {number} depth - Maximum depth to traverse (default: 3)
-   * @param {boolean} includeObjects - Include object counts breakdown
-   * @returns {object} Tree result with hierarchy, summary, and metadata
-   */
-  async tree(packageName, depth = 3, includeObjects = false) {
-    logger.info('Getting package tree', { package: packageName, depth, includeObjects });
-
-    try {
-      const csrfToken = await this.http.fetchCsrfToken();
-      const data = {
-        package: packageName,
-        depth: Math.min(Math.max(1, depth), 10),
-        include_objects: includeObjects
-      };
-
-      const result = await this.http.post('/sap/bc/z_abapgit_agent/tree', data, { csrfToken });
-      return {
-        success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
-        command: result.COMMAND || result.command || 'TREE',
-        package: result.PACKAGE || result.package,
-        message: result.MESSAGE || result.message || '',
-        hierarchy: result.HIERARCHY || result.hierarchy || null,
-        summary: result.SUMMARY || result.summary || null,
-        error: result.ERROR || result.error || null
-      };
-    } catch (error) {
-      logger.error('Tree command failed', { error: error.message });
-      throw new Error(`Tree command failed: ${error.message}`);
-    }
-  }
-
-  /**
-   * 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 {number} offset - Number of rows to skip
-   * @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 = 100, offset = 0, where = null, columns = null) {
-    logger.info('Previewing data', { objects, type, limit, offset, where, columns });
-
-    try {
-      const csrfToken = await this.http.fetchCsrfToken();
-      const data = {
-        objects: objects,
-        limit: Math.min(Math.max(1, limit), 500),
-        offset: Math.max(0, offset)
-      };
-
-      if (type) {
-        data.type = type;
-      }
-
-      if (where) {
-        // Convert ISO date format (YYYY-MM-DD) to ABAP DATS format (YYYYMMDD)
-        data.where = this.convertDatesInWhereClause(where);
-      }
-
-      if (columns) {
-        data.columns = columns;
-      }
-
-      const result = await this.http.post('/sap/bc/z_abapgit_agent/preview', data, { csrfToken });
-      return {
-        success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
-        command: result.COMMAND || result.command || 'PREVIEW',
-        message: result.MESSAGE || result.message || '',
-        objects: result.OBJECTS || result.objects || [],
-        summary: result.SUMMARY || result.summary || null,
-        pagination: result.PAGINATION || result.pagination || null,
-        error: result.ERROR || result.error || null
-      };
-    } catch (error) {
-      logger.error('Preview command failed', { error: error.message });
-      throw new Error(`Preview command failed: ${error.message}`);
-    }
-  }
-
-  /**
-   * 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;
-
-    const isoDatePattern = /'\\d{4}-\\d{2}-\\d{2}'/g;
-    return whereClause.replace(isoDatePattern, (match) => {
-      const dateContent = match.slice(1, -1);
-      const [year, month, day] = dateContent.split('-');
-      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)
-   * @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 csrfToken = await this.http.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;
-      }
-
-      const result = await this.http.post('/sap/bc/z_abapgit_agent/list', data, { csrfToken });
-
-      // 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}`);
-    }
-  }
-
-  /**
-   * View ABAP object definitions
-   * @param {Array} objects - Array of object names to view
-   * @param {string} type - Object type (optional, e.g., 'CLAS', 'TABL')
-   * @returns {object} View result with object definitions
-   */
-  async view(objects, type = null) {
-    logger.info('Viewing objects', { objects, type });
-
-    try {
-      const csrfToken = await this.http.fetchCsrfToken();
-      const data = {
-        objects: objects
-      };
-
-      if (type) {
-        data.type = type;
-      }
-
-      const result = await this.http.post('/sap/bc/z_abapgit_agent/view', data, { csrfToken });
-      return {
-        success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
-        command: result.COMMAND || result.command || 'VIEW',
-        objects: result.OBJECTS || result.objects || [],
-        error: result.ERROR || result.error || null
-      };
-    } catch (error) {
-      logger.error('View command failed', { error: error.message });
-      throw new Error(`View command failed: ${error.message}`);
-    }
-  }
-
-  /**
-   * Find where-used list for ABAP objects
-   * @param {Array} objects - Array of object names to search
-   * @param {string} type - Object type (optional)
-   * @param {number} limit - Maximum results (default: 100, max: 500)
-   * @param {number} offset - Number of results to skip (default: 0)
-   * @returns {object} Where-used result with found objects
-   */
-  async where(objects, type = null, limit = 100, offset = 0) {
-    logger.info('Finding where-used', { objects, type, limit, offset });
-
-    try {
-      const csrfToken = await this.http.fetchCsrfToken();
-      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 this.http.post('/sap/bc/z_abapgit_agent/where-used', data, { csrfToken });
-      return {
-        success: result.SUCCESS === 'X' || result.success === 'X' || result.success === true,
-        command: result.COMMAND || result.command || 'WHERE',
-        objects: result.OBJECTS || result.objects || [],
-        message: result.MESSAGE || result.message || '',
-        pagination: result.PAGINATION || result.pagination || null,
-        error: result.ERROR || result.error || null
-      };
-    } catch (error) {
-      logger.error('Where command failed', { error: error.message });
-      throw new Error(`Where command failed: ${error.message}`);
-    }
-  }
-}
-
-module.exports = {
-  ABAPGitAgent
-};

package/src/index.js

@@ -1,48 +0,0 @@
-/**
- * ABAP Git Agent - Package Entry Point
- *
- * Exports functions for programmatic use:
- *   const { pull, healthCheck } = require('abapgit-agent');
- */
-
-const { ABAPGitAgent } = require('./agent');
-
-/**
- * Pull repository and activate objects
- * @param {string} repoUrl - Git repository URL
- * @param {string} branch - Branch name (default: main)
- * @param {string} username - Git username (optional)
- * @param {string} password - Git password/token (optional)
- * @returns {object} Pull result with success, job_id, message, error_detail
- */
-async function pull(repoUrl, branch = 'main', username = null, password = null) {
-  const agent = new ABAPGitAgent();
-  return await agent.pull(repoUrl, branch, username, password);
-}
-
-/**
- * Check agent health
- * @returns {object} Health status
- */
-async function healthCheck() {
-  const agent = new ABAPGitAgent();
-  return await agent.healthCheck();
-}
-
-/**
- * Check if integration is configured for current directory
- * @returns {boolean} True if .abapGitAgent exists
- */
-function isConfigured() {
-  const fs = require('fs');
-  const path = require('path');
-  const repoConfigPath = path.join(process.cwd(), '.abapGitAgent');
-  return fs.existsSync(repoConfigPath);
-}
-
-module.exports = {
-  pull,
-  healthCheck,
-  isConfigured,
-  ABAPGitAgent
-};

package/src/server.js

@@ -1,116 +0,0 @@
-/**
- * HTTP Server for Claude Integration
- */
-
-const express = require('express');
-const cors = require('cors');
-const { ABAPGitAgent } = require('./agent');
-const { getAgentConfig } = require('./config');
-const logger = require('./logger');
-
-class Server {
-  constructor() {
-    this.app = express();
-    this.agent = new ABAPGitAgent();
-    this.agentConfig = getAgentConfig();
-
-    this.setupMiddleware();
-    this.setupRoutes();
-  }
-
-  setupMiddleware() {
-    this.app.use(cors());
-    this.app.use(express.json());
-
-    // Request logging
-    this.app.use((req, res, next) => {
-      logger.debug(`${req.method} ${req.path}`, { body: req.body });
-      next();
-    });
-
-    // Error handling
-    this.app.use((err, req, res, next) => {
-      logger.error('Request error', { error: err.message, stack: err.stack });
-      res.status(500).json({
-        success: false,
-        error: err.message
-      });
-    });
-  }
-
-  setupRoutes() {
-    // Health check
-    this.app.get('/api/health', async (req, res) => {
-      try {
-        const health = await this.agent.healthCheck();
-        res.json(health);
-      } catch (error) {
-        res.status(503).json({
-          status: 'unhealthy',
-          error: error.message
-        });
-      }
-    });
-
-    // Pull repository (synchronous - returns immediately with result)
-    this.app.post('/api/pull', async (req, res) => {
-      try {
-        const { url, branch, username, password } = req.body;
-
-        if (!url) {
-          return res.status(400).json({
-            success: false,
-            error: 'Missing required parameter: url'
-          });
-        }
-
-        const result = await this.agent.pull(url, branch, username, password);
-        res.json(result);
-
-      } catch (error) {
-        logger.error('Pull failed', { error: error.message });
-        res.status(500).json({
-          success: false,
-          error: error.message
-        });
-      }
-    });
-  }
-
-  start() {
-    const port = this.agentConfig.port || 3000;
-
-    this.server = this.app.listen(port, () => {
-      logger.info(`ABAP AI Bridge server started on port ${port}`);
-      console.log(`\n🚀 ABAP AI Bridge is running!`);
-      console.log(`   Health:   http://localhost:${port}/api/health`);
-      console.log(`   Pull:     POST http://localhost:${port}/api/pull`);
-      console.log(`\n📚 API Documentation:`);
-      console.log(`   POST /api/pull { "url": "git-url", "branch": "main" }`);
-    });
-
-    // Graceful shutdown
-    process.on('SIGTERM', () => this.shutdown());
-    process.on('SIGINT', () => this.shutdown());
-  }
-
-  shutdown() {
-    logger.info('Shutting down server...');
-    if (this.server) {
-      this.server.close(() => {
-        logger.info('Server closed');
-        process.exit(0);
-      });
-    }
-  }
-}
-
-// Start server if run directly
-if (require.main === module) {
-  const server = new Server();
-  server.start();
-}
-
-module.exports = {
-  Server
-};