abap-local-client 1.0.0 → 1.1.1

package/bin/config.cjs

@@ -1,8 +1,3 @@
-#!/usr/bin/env node
-// abap-local-config entry point (CommonJS wrapper for ESM package)
-(async () => {
-    await import('../config-cli.mjs');
-})().catch(err => {
-    console.error('Failed to start abap-local-config:', err.message);
-    process.exit(1);
-});
+#!/usr/bin/env node
+// abap-local-config entry point (CJS wrapper → dynamic import ESM module)
+(async()=>{await import('../config-cli.mjs')})().catch(e=>{console.error(e.message);process.exit(1)});

package/bin/sso.cjs

@@ -0,0 +1,3 @@
+#!/usr/bin/env node
+// abap-local-client entry point (CJS wrapper → dynamic import ESM module)
+(async()=>{await import('../sso-sap-client.mjs')})().catch(e=>{console.error(e.message);process.exit(1)});

package/cli-config.cjs

@@ -0,0 +1,2 @@
+// abap-local-config — CommonJS wrapper for ESM entry point
+import('../config-cli.mjs');

package/cli.cjs

@@ -0,0 +1,3 @@
+// abap-local-client — CommonJS wrapper for ESM entry point
+// This wrapper is what npm's "bin" generates: node ./cli.js
+import('../sso-sap-client.mjs');

package/package.json

@@ -1,15 +1,13 @@
 {
   "name": "abap-local-client",
-  "version": "1.0.0",
+  "version": "1.1.1",
   "description": "SAP Local Client — WebSocket bridge with Basic, SPNEGO, X.509, SAML & SNC authentication",
   "main": "sso-sap-client.mjs",
   "type": "module",
   "scripts": {
     "start": "node sso-sap-client.mjs",
-    "dev": "node sso-sap-client.mjs",
     "sso": "node sso-sap-client.mjs",
-    "rfc": "node saprouter-client.mjs",
-    "kill": "node kill-client.js",
+    "config": "node config-cli.mjs",
     "test": "node test-integration.mjs",
     "prepublishOnly": "node test-integration.mjs"
   },

package/xml-utils.js

@@ -0,0 +1,60 @@
+import { XMLParser } from 'fast-xml-parser';
+
+// Create XML parser instance
+const xmlParser = new XMLParser({
+  ignoreAttributes: false,
+  attributeNamePrefix: '@_'
+});
+
+/**
+ * Parse SAP XML response and check for errors
+ * Returns: { hasErrors: boolean, errorMessages: string[], parsedData: object }
+ */
+export function parseSAPResponse(xmlString) {
+  if (typeof xmlString !== 'string' || !xmlString.includes('<?xml')) {
+    return { hasErrors: false, errorMessages: [], parsedData: null };
+  }
+
+  try {
+    const parsed = xmlParser.parse(xmlString);
+    
+    // Check for syntax check errors (chkrun:checkMessage with type="E")
+    if (parsed['chkrun:checkRunReports']) {
+      const report = parsed['chkrun:checkRunReports']['chkrun:checkReport'];
+      const messageList = report?.['chkrun:checkMessageList'];
+      
+      if (messageList && messageList['chkrun:checkMessage']) {
+        const messages = Array.isArray(messageList['chkrun:checkMessage']) 
+          ? messageList['chkrun:checkMessage']
+          : [messageList['chkrun:checkMessage']];
+
+        const errorMessages = [];
+        for (const msg of messages) {
+          if (msg['@_chkrun:type'] === 'E') {
+            let line = null;
+            if (msg['@_chkrun:uri'] && msg['@_chkrun:uri'].includes('#start=')) {
+              const match = msg['@_chkrun:uri'].match(/#start=(\d+),(\d+)/);
+              if (match) {
+                line = parseInt(match[1]);
+              }
+            }
+            const text = msg['@_chkrun:shortText'] || 'Unknown error';
+            const lineInfo = line ? ` (line ${line})` : '';
+            errorMessages.push(`${text}${lineInfo}`);
+          }
+        }
+
+        if (errorMessages.length > 0) {
+          return { hasErrors: true, errorMessages, parsedData: parsed };
+        }
+      }
+    }
+
+    // Add other error patterns here if needed (e.g., activation errors, etc.)
+    
+    return { hasErrors: false, errorMessages: [], parsedData: parsed };
+  } catch (error) {
+    console.error('   ⚠️  XML parsing failed:', error.message);
+    return { hasErrors: false, errorMessages: [], parsedData: null };
+  }
+}