abapgit-agent 1.17.2 → 1.17.4

package/.abapGitAgent.example

@@ -8,5 +8,12 @@
   "protocol": "https",
   "gitUsername": "github-username",
   "gitPassword": "github-token",
-  "referenceFolder": "~/abap-reference"
+  "referenceFolder": "~/abap-reference",
+
+  "testRepos": {
+    "pull":      "https://github.com/your-org/abgagt-pull-test.git",
+    "drop":      "https://github.com/your-org/abgagt-drop-test.git",
+    "customize": "https://github.com/your-org/abgagt-customize-test.git",
+    "lifecycle": "https://github.com/your-org/abgagt-lifecycle-test.git"
+  }
 }

package/.abapgit-agent.example.json

@@ -0,0 +1,19 @@
+{
+  "project": {
+    "name": "MY_PACKAGE",
+    "description": ""
+  },
+
+  "safeguards": {
+    "requireFilesForPull": false,
+    "disablePull": false,
+    "disableRun": false,
+    "disableImport": false,
+    "requireImportMessage": false,
+    "disableProbeClasses": false
+  },
+
+  "conflictDetection": {
+    "mode": "abort"
+  }
+}

package/README.md

@@ -55,6 +55,7 @@ See [Creating New ABAP Projects](docs/install.md#creating-new-abap-projects) to
 abapgit-agent init --package ZMY_PACKAGE   # Initialize local config
 abapgit-agent create                        # Create online repo in ABAP
 abapgit-agent import                        # Import objects from ABAP to git
+abapgit-agent import --branch main          # Import to specific branch
 abapgit-agent delete                        # Delete repo from ABAP
 ```
 
@@ -99,17 +100,30 @@ abapgit-agent health                  # Verify ABAP connection
 # Install dependencies
 npm install
 
-# Run unit tests (no ABAP system needed)
-npm test
-
 # Test a command manually
 node bin/abapgit-agent --help
 node bin/abapgit-agent syntax --files src/zcl_my_class.clas.abap
+```
 
-# Run integration tests against a real ABAP system (requires .abapGitAgent)
-npm run test:integration
+## Running Tests
+
+```bash
+# Unit tests — no ABAP system needed (fast)
+npm test
+
+# Integration tests — requires a configured .abapGitAgent
+npm run test:setup   # one-time setup: clone test repos + activate ABAP objects
+npm run test:all     # full suite (setup runs automatically on first run)
+
+# Run a single suite
+npm run test:cmd         # CLI command tests
+npm run test:drop        # drop command tests
+npm run test:customize   # customize command tests
+npm run test:aunit       # ABAP unit tests
 ```
 
+> **Full integration test guide:** [docs/integration-tests.md](docs/integration-tests.md)
+
 ## Documentation
 
 | Topic | File |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "abapgit-agent",
-  "version": "1.17.2",
+  "version": "1.17.4",
   "description": "ABAP Git Agent - Pull and activate ABAP code via abapGit from any git repository",
   "files": [
     "bin/",
@@ -10,7 +10,8 @@
     "abap/CLAUDE.slim.md",
     "abap/.github/copilot-instructions.md",
     "abap/.github/copilot-instructions.slim.md",
-    ".abapGitAgent.example"
+    ".abapGitAgent.example",
+    ".abapgit-agent.example.json"
   ],
   "bin": {
     "abapgit-agent": "bin/abapgit-agent",
@@ -19,6 +20,7 @@
   "scripts": {
     "test": "jest",
     "test:all": "node tests/run-all.js",
+    "test:setup": "node tests/run-all.js --setup",
     "test:unit": "jest",
     "test:jest": "jest",
     "test:integration": "node tests/run-all.js --cmd",

package/src/commands/customize.js

@@ -17,6 +17,31 @@ module.exports = {
   async execute(args, context) {
     const { loadConfig, AbapHttp, getTransport, getTransportSettings } = context;
 
+    if (args.includes('--help') || args.includes('-h')) {
+      console.log(`
+Usage:
+  abapgit-agent customize <table> --set <field=value> [<field=value>...]
+                          [--transport <TRKORR>] [--no-transport] [--json]
+
+Description:
+  Write a single row to a SAP customizing table (delivery class C or E).
+  The row is identified by the key fields you provide via --set.
+
+Parameters:
+  <table>               Name of the customizing table (e.g. ZTABLE_CONFIG).
+  --set <field=value>   One or more field=value pairs (space-separated after --set).
+  --transport <TRKORR>  Record the change in this transport request.
+  --no-transport        Write without a transport request (local change).
+  --json                Output result as JSON.
+
+Examples:
+  abapgit-agent customize ZTABLE_CONFIG --set KEY=APP VALUE=active
+  abapgit-agent customize ZTABLE_CONFIG --set KEY=APP VALUE=active --transport DEVK900001
+  abapgit-agent customize ZTABLE_CONFIG --set KEY=APP VALUE=active --no-transport
+`);
+      return;
+    }
+
     const jsonOutput = args.includes('--json');
     const verbose    = args.includes('--verbose');
     const noTransport = args.includes('--no-transport');

package/src/commands/import.js

@@ -39,11 +39,14 @@ Prerequisites:
   - Package must have objects to import
 
 Options:
+  --branch     Branch to push to. Auto-detected from current git branch if omitted.
   --message    Commit message (default: "feat: initial import from ABAP package <package>")
 
 Examples:
   abapgit-agent import
   abapgit-agent import --message "Initial import from SAP"
+  abapgit-agent import --branch main
+  abapgit-agent import --branch feature/my-branch --message "Import objects"
 `);
       return;
     }
@@ -87,6 +90,9 @@ Examples:
       commitMessage = args[messageArgIndex + 1];
     }
 
+    const branchArgIndex = args.indexOf('--branch');
+    const branch = branchArgIndex !== -1 ? args[branchArgIndex + 1] : gitUtils.getBranch();
+
     if (safeguards.requireImportMessage && !commitMessage) {
       console.error('❌ Error: import requires a commit message for this project\n');
       console.error('Please provide one with:');
@@ -99,6 +105,7 @@ Examples:
 
     console.log(`\n📦 Starting import job`);
     console.log(`   URL: ${repoUrl}`);
+    console.log(`   Branch: ${branch}`);
     if (commitMessage) {
       console.log(`   Message: ${commitMessage}`);
     }
@@ -107,7 +114,8 @@ Examples:
     const csrfToken = await http.fetchCsrfToken();
 
     const data = {
-      url: repoUrl
+      url: repoUrl,
+      branch
     };
 
     if (commitMessage) {

package/src/commands/init.js

@@ -565,24 +565,14 @@ Uncomment and edit the rows that differ from the defaults in \`guidelines/object
     // Create .abapgit-agent.json with default values (team config, checked into git)
     const projectConfigPath = pathModule.join(process.cwd(), '.abapgit-agent.json');
     if (!fs.existsSync(projectConfigPath)) {
-      const projectConfig = {
-        project: {
-          name: packageName,
-          description: ''
-        },
-        safeguards: {
-          requireFilesForPull: false,
-          disablePull: false,
-          disableRun: false,
-          disableImport: false,
-          requireImportMessage: false,
-          disableProbeClasses: false
-        },
-        conflictDetection: {
-          mode: 'abort'
-        }
-      };
+      const projectConfigSamplePath = pathModule.join(__dirname, '..', '..', '.abapgit-agent.example.json');
+      if (!fs.existsSync(projectConfigSamplePath)) {
+        console.error('Error: .abapgit-agent.example.json not found.');
+        process.exit(1);
+      }
       try {
+        const projectConfig = JSON.parse(fs.readFileSync(projectConfigSamplePath, 'utf8'));
+        projectConfig.project.name = packageName;
         fs.writeFileSync(projectConfigPath, JSON.stringify(projectConfig, null, 2) + '\n');
         console.log(`✅ Created .abapgit-agent.json (team config — commit this to git)`);
       } catch (error) {

package/src/commands/lint.js

@@ -59,13 +59,13 @@ Examples:
     // ── Resolve changed files ─────────────────────────────────────────────────
     let abapFiles;
     if (filesArg) {
-      abapFiles = filesArg.split(',').map(f => f.trim()).filter(f => f.endsWith('.abap'));
+      abapFiles = filesArg.split(',').map(f => f.trim()).filter(f => isLintable(f) && fs.existsSync(f));
     } else {
       abapFiles = detectChangedAbapFiles(baseBranch);
     }
 
     if (abapFiles.length === 0) {
-      console.log('No changed .abap files found — nothing to lint.');
+      console.log('No changed .abap/.asddls files found — nothing to lint.');
       return;
     }
 
@@ -102,34 +102,29 @@ Examples:
     // keep <file> blocks for the originally changed files — suppressing any
     // pre-existing issues in dependency files that were not part of this change.
     try {
-      if (outformat === 'checkstyle') {
-        // Run to a temp file, filter, then write to the final destination.
-        const tempOut = '.abaplint-raw.xml';
-        const abapFilesSet = new Set(abapFiles.map(f => path.resolve(f)));
-        try {
-          const result = spawnSync(
-            `npx @abaplint/cli@latest ${scopedConfig} --outformat checkstyle --outfile ${tempOut}`,
-            { stdio: 'pipe', shell: true }
-          );
-          const raw = fs.existsSync(tempOut) ? fs.readFileSync(tempOut, 'utf8') : '<checkstyle version="8.0"/>';
-          const filtered = filterCheckstyleToFiles(raw, abapFilesSet);
+      const tempOut = '.abaplint-raw.xml';
+      const abapFilesSet = new Set(abapFiles.map(f => path.resolve(f)));
+      try {
+        spawnSync(
+          `npx @abaplint/cli@latest ${scopedConfig} --outformat checkstyle --outfile ${tempOut}`,
+          { stdio: 'pipe', shell: true }
+        );
+        const raw = fs.existsSync(tempOut) ? fs.readFileSync(tempOut, 'utf8') : '<checkstyle version="8.0"/>';
+        const filtered = filterCheckstyleToFiles(raw, abapFilesSet);
+        if (outformat === 'checkstyle') {
           if (outfile) {
             fs.writeFileSync(outfile, filtered);
           } else {
             process.stdout.write(filtered);
           }
-          const issueCount = (filtered.match(/<error /g) || []).length;
-          if (issueCount > 0) process.exitCode = 1;
-        } finally {
-          if (fs.existsSync(tempOut)) fs.unlinkSync(tempOut);
+        } else {
+          // Interactive: print issues as human-readable text, scoped to changed files only.
+          printCheckstyleAsText(filtered);
         }
-      } else {
-        // Interactive: inherit stdio so abaplint's human-readable output flows through.
-        const result = spawnSync(
-          `npx @abaplint/cli@latest ${scopedConfig}`,
-          { stdio: 'inherit', shell: true }
-        );
-        if (result.status !== 0) process.exitCode = result.status;
+        const issueCount = (filtered.match(/<error /g) || []).length;
+        if (issueCount > 0) process.exitCode = 1;
+      } finally {
+        if (fs.existsSync(tempOut)) fs.unlinkSync(tempOut);
       }
     } finally {
       fs.unlinkSync(scopedConfig);
@@ -156,12 +151,12 @@ function detectChangedAbapFiles(baseBranch) {
 
   let diffCmd;
   if (base) {
-    diffCmd = `git diff --name-only ${base}...HEAD -- '*.abap'`;
+    diffCmd = `git diff --name-only ${base}...HEAD -- '*.abap' '*.asddls'`;
   } else {
     // Fall back to uncommitted changes first, then last commit
-    const uncommitted = runGit('git diff --name-only HEAD -- *.abap').filter(Boolean);
+    const uncommitted = runGit(`git diff --name-only HEAD -- '*.abap' '*.asddls'`).filter(Boolean);
     if (uncommitted.length > 0) return filterAbapFiles(uncommitted);
-    diffCmd = `git diff --name-only HEAD~1 HEAD -- '*.abap'`;
+    diffCmd = `git diff --name-only HEAD~1 HEAD -- '*.abap' '*.asddls'`;
   }
 
   return filterAbapFiles(runGit(diffCmd));
@@ -190,7 +185,7 @@ function buildFileIndex(abapDir) {
       const full = path.join(dir, entry.name);
       if (entry.isDirectory()) {
         walk(full);
-      } else if (entry.name.endsWith('.abap') || entry.name.endsWith('.xml')) {
+      } else if (entry.name.endsWith('.abap') || entry.name.endsWith('.xml') || entry.name.endsWith('.asddls')) {
         index.set(entry.name.toLowerCase(), full);
       }
     }
@@ -304,13 +299,47 @@ function filterCheckstyleToFiles(xml, abapFilesSet) {
 }
 
 /**
- * Keep only files that look like ABAP source files
- * (name.type.abap or name.type.subtype.abap).
+ * Print checkstyle XML as human-readable text, mirroring abaplint's default output format:
+ *   path/to/file.clas.abap:line - severity - message (rule)
+ */
+function printCheckstyleAsText(xml) {
+  const fileRe  = /<file\s+name="([^"]*)"([\s\S]*?)<\/file>/g;
+  const errorRe = /<error\s+[^>]*line="(\d+)"[^>]*severity="([^"]*)"[^>]*message="([^"]*)"[^>]*source="([^"]*)"/g;
+  let fileMatch;
+  let issueCount = 0;
+  while ((fileMatch = fileRe.exec(xml)) !== null) {
+    const filePath = fileMatch[1];
+    const block    = fileMatch[2];
+    errorRe.lastIndex = 0;
+    let errMatch;
+    while ((errMatch = errorRe.exec(block)) !== null) {
+      const [, line, severity, message, source] = errMatch;
+      const rule = source.split('.').pop();
+      const text = message.replace(/&amp;/g, '&').replace(/&quot;/g, '"').replace(/&lt;/g, '<').replace(/&gt;/g, '>').replace(/&#39;/g, "'");
+      console.log(`${filePath}:${line} - ${severity} - ${text} (${rule})`);
+      issueCount++;
+    }
+  }
+  if (issueCount === 0) {
+    console.log('No issues found.');
+  }
+}
+
+/**
+ * Returns true if the file is a type abaplint can analyse.
+ * - .abap — all ABAP source files (CLAS, INTF, PROG, FUGR, ENHO, etc.)
+ * - .asddls — CDS view / view entity sources (DDLS)
+ */
+function isLintable(f) {
+  const lower = f.toLowerCase();
+  return lower.endsWith('.abap') || lower.endsWith('.asddls');
+}
+
+/**
+ * Keep only lintable files (name.type.abap / name.type.subtype.abap / name.ddls.asddls)
+ * that still exist on disk. Deleted files appear in git diff output but must not
+ * be passed to abaplint.
  */
 function filterAbapFiles(files) {
-  return files.filter(f => {
-    const parts = path.basename(f).split('.');
-    return (parts.length === 3 || parts.length === 4) &&
-           parts[parts.length - 1].toLowerCase() === 'abap';
-  });
+  return files.filter(f => isLintable(f) && fs.existsSync(f));
 }