abapgit-agent 1.16.2 → 1.16.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "abapgit-agent",
-  "version": "1.16.2",
+  "version": "1.16.3",
   "description": "ABAP Git Agent - Pull and activate ABAP code via abapGit from any git repository",
   "files": [
     "bin/",

package/src/commands/import.js

@@ -50,14 +50,26 @@ Examples:
 
     // Get parameters from config
     const safeguards = getSafeguards();
+
+    // disableImport check — bypass if current user is in importAllowedUsers
     if (safeguards.disableImport) {
-      console.error('❌ Error: import command is disabled for this project\n');
-      if (safeguards.reason) {
-        console.error(`Reason: ${safeguards.reason}\n`);
+      const config0 = loadConfig();
+      const currentUser = (config0.user || '').toUpperCase();
+      const allowedUsers = safeguards.importAllowedUsers || [];
+      const isAllowed = allowedUsers.length > 0 && allowedUsers.includes(currentUser);
+
+      if (!isAllowed) {
+        console.error('❌ Error: import command is disabled for this project\n');
+        if (safeguards.reason) {
+          console.error(`Reason: ${safeguards.reason}\n`);
+        }
+        if (allowedUsers.length > 0) {
+          console.error(`Allowed users: ${allowedUsers.join(', ')}`);
+        }
+        console.error('The import command has been disabled in .abapgit-agent.json');
+        console.error('Please contact the project maintainer to enable it.');
+        process.exit(1);
       }
-      console.error('The import command has been disabled in .abapgit-agent.json');
-      console.error('Please contact the project maintainer to enable it.');
-      process.exit(1);
     }
 
     // Get parameters from config
@@ -75,6 +87,16 @@ Examples:
       commitMessage = args[messageArgIndex + 1];
     }
 
+    if (safeguards.requireImportMessage && !commitMessage) {
+      console.error('❌ Error: import requires a commit message for this project\n');
+      console.error('Please provide one with:');
+      console.error('  abapgit-agent import --message "Your descriptive message"\n');
+      if (safeguards.reason) {
+        console.error(`Reason: ${safeguards.reason}\n`);
+      }
+      process.exit(1);
+    }
+
     console.log(`\n📦 Starting import job`);
     console.log(`   URL: ${repoUrl}`);
     if (commitMessage) {

package/src/config.js

@@ -119,11 +119,21 @@ function getSafeguards() {
   const projectConfig = loadProjectConfig();
 
   if (projectConfig?.safeguards) {
+    // importAllowedUsers: accept string (single) or array; normalize to uppercase array
+    const rawAllowed = projectConfig.safeguards.importAllowedUsers;
+    let importAllowedUsers = null;
+    if (rawAllowed) {
+      const arr = Array.isArray(rawAllowed) ? rawAllowed : [rawAllowed];
+      importAllowedUsers = arr.map(u => String(u).toUpperCase());
+    }
+
     return {
       requireFilesForPull: projectConfig.safeguards.requireFilesForPull === true,
       disablePull: projectConfig.safeguards.disablePull === true,
       disableRun: projectConfig.safeguards.disableRun === true,
       disableImport: projectConfig.safeguards.disableImport === true,
+      requireImportMessage: projectConfig.safeguards.requireImportMessage === true,
+      importAllowedUsers,
       disableProbeClasses: projectConfig.safeguards.disableProbeClasses === true,
       reason: projectConfig.safeguards.reason || null
     };
@@ -135,6 +145,8 @@ function getSafeguards() {
     disablePull: false,
     disableRun: false,
     disableImport: false,
+    requireImportMessage: false,
+    importAllowedUsers: null,
     disableProbeClasses: false,
     reason: null
   };