@mcp-abap-adt/core 2.5.2 → 2.6.1

package/dist/handlers/table/handleCreateTable.js

@@ -1,118 +0,0 @@
-"use strict";
-/**
- * CreateTable Handler - ABAP Table Creation via ADT API
- *
- * Uses TableBuilder from @mcp-abap-adt/adt-clients for all operations.
- * Session and lock management handled internally by builder.
- *
- * Workflow: validate -> create -> lock -> update -> check -> unlock -> (activate)
- */
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.TOOL_DEFINITION = void 0;
-exports.handleCreateTable = handleCreateTable;
-const utils_1 = require("../../lib/utils");
-const utils_2 = require("../../lib/utils");
-const transportValidation_js_1 = require("../../utils/transportValidation.js");
-const adt_clients_1 = require("@mcp-abap-adt/adt-clients");
-exports.TOOL_DEFINITION = {
-    name: "CreateTable",
-    description: "Create a new ABAP table via the ADT API using provided DDL. Mirrors Eclipse ADT behaviour with status/check runs, lock handling, activation and verification.",
-    inputSchema: {
-        type: "object",
-        properties: {
-            table_name: {
-                type: "string",
-                description: "Table name (e.g., ZZ_TEST_TABLE_001). Must follow SAP naming conventions."
-            },
-            ddl_code: {
-                type: "string",
-                description: "Complete DDL code for table creation. Example: '@EndUserText.label : \\'My Table\\' @AbapCatalog.tableCategory : #TRANSPARENT define table ztst_table { key client : abap.clnt not null; key id : abap.char(10); name : abap.char(255); }'"
-            },
-            package_name: {
-                type: "string",
-                description: "Package name (e.g., ZOK_LOCAL, $TMP for local objects)"
-            },
-            transport_request: {
-                type: "string",
-                description: "Transport request number (e.g., E19K905635). Required for transportable packages."
-            }
-        },
-        required: ["table_name", "ddl_code", "package_name"]
-    }
-};
-/**
- * Main handler for CreateTable MCP tool
- *
- * Uses TableBuilder from @mcp-abap-adt/adt-clients for all operations
- * Session and lock management handled internally by builder
- */
-async function handleCreateTable(args) {
-    try {
-        const createTableArgs = args;
-        // Validate required parameters
-        if (!createTableArgs?.table_name) {
-            throw new utils_1.McpError(utils_1.ErrorCode.InvalidParams, 'Table name is required');
-        }
-        if (!createTableArgs?.ddl_code) {
-            throw new utils_1.McpError(utils_1.ErrorCode.InvalidParams, 'DDL code is required');
-        }
-        if (!createTableArgs?.package_name) {
-            throw new utils_1.McpError(utils_1.ErrorCode.InvalidParams, 'Package name is required');
-        }
-        // Validate transport_request: required for non-$TMP packages
-        (0, transportValidation_js_1.validateTransportRequest)(createTableArgs.package_name, createTableArgs.transport_request);
-        const connection = (0, utils_2.getManagedConnection)();
-        const tableName = createTableArgs.table_name.toUpperCase();
-        utils_2.logger.info(`Starting table creation: ${tableName}`);
-        try {
-            // Create client
-            const client = new adt_clients_1.CrudClient(connection);
-            const shouldActivate = true; // Default to true for tables
-            // Validate
-            await client.validateTable(tableName);
-            // Create
-            await client.createTable(tableName, createTableArgs.package_name, createTableArgs.transport_request);
-            // Lock
-            await client.lockTable(tableName);
-            const lockHandle = client.getLockHandle();
-            // Update with DDL code
-            await client.updateTable(tableName, createTableArgs.ddl_code, lockHandle);
-            // Check
-            await client.checkTable(tableName);
-            // Unlock
-            await client.unlockTable(tableName, lockHandle);
-            // Activate
-            if (shouldActivate) {
-                await client.activateTable(tableName);
-            }
-            utils_2.logger.info(`✅ CreateTable completed successfully: ${tableName}`);
-            return (0, utils_2.return_response)({
-                data: JSON.stringify({
-                    success: true,
-                    table_name: tableName,
-                    package_name: createTableArgs.package_name,
-                    transport_request: createTableArgs.transport_request || 'local',
-                    activated: shouldActivate,
-                    message: `Table ${tableName} created successfully${shouldActivate ? ' and activated' : ''}`
-                })
-            });
-        }
-        catch (error) {
-            utils_2.logger.error(`Error creating table ${tableName}:`, error);
-            // Check if table already exists
-            if (error.message?.includes('already exists') || error.response?.status === 409) {
-                throw new utils_1.McpError(utils_1.ErrorCode.InvalidParams, `Table ${tableName} already exists. Please delete it first or use a different name.`);
-            }
-            const errorMessage = error.response?.data
-                ? (typeof error.response.data === 'string' ? error.response.data : JSON.stringify(error.response.data))
-                : error.message || String(error);
-            throw new utils_1.McpError(utils_1.ErrorCode.InternalError, `Failed to create table ${tableName}: ${errorMessage}`);
-        }
-    }
-    catch (error) {
-        if (error instanceof utils_1.McpError) {
-            throw error;
-        }
-        return (0, utils_2.return_error)(error);
-    }
-}

package/dist/handlers/table/handleGetTable.d.ts

@@ -1,16 +0,0 @@
-import * as z from 'zod';
-export declare const TOOL_DEFINITION: {
-    readonly name: "GetTable";
-    readonly description: "Retrieve ABAP table structure.";
-    readonly inputSchema: {
-        readonly table_name: z.ZodString;
-    };
-};
-export declare function handleGetTable(args: any): Promise<{
-    isError: boolean;
-    content: {
-        type: string;
-        text: any;
-    }[];
-}>;
-//# sourceMappingURL=handleGetTable.d.ts.map

package/dist/handlers/table/handleGetTable.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"handleGetTable.d.ts","sourceRoot":"","sources":["../../../src/handlers/table/handleGetTable.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,CAAC,MAAM,KAAK,CAAC;AAEzB,eAAO,MAAM,eAAe;;;;;;CAMlB,CAAC;AAuCX,wBAAsB,cAAc,CAAC,IAAI,EAAE,GAAG;;;;;;GAsD7C"}

package/dist/handlers/table/handleGetTable.js

@@ -1,140 +0,0 @@
-"use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.TOOL_DEFINITION = void 0;
-exports.handleGetTable = handleGetTable;
-const utils_1 = require("../../lib/utils");
-const adt_clients_1 = require("@mcp-abap-adt/adt-clients");
-const fast_xml_parser_1 = require("fast-xml-parser");
-const writeResultToFile_1 = require("../../lib/writeResultToFile");
-const z = __importStar(require("zod"));
-exports.TOOL_DEFINITION = {
-    name: "GetTable",
-    description: "Retrieve ABAP table structure.",
-    inputSchema: {
-        table_name: z.string().describe("Name of the ABAP table")
-    }
-};
-function parseTableXml(xml) {
-    const parser = new fast_xml_parser_1.XMLParser({
-        ignoreAttributes: false,
-        attributeNamePrefix: '',
-        parseAttributeValue: true,
-        trimValues: true
-    });
-    const result = parser.parse(xml);
-    // DDIC Table (TABL/DT)
-    if (result['ddic:table']) {
-        const t = result['ddic:table'];
-        const fields = Array.isArray(t['ddic:fields']?.['ddic:field'])
-            ? t['ddic:fields']['ddic:field']
-            : t['ddic:fields']?.['ddic:field']
-                ? [t['ddic:fields']['ddic:field']]
-                : [];
-        return {
-            name: t['adtcore:name'],
-            objectType: 'table',
-            description: t['adtcore:description'],
-            package: t['adtcore:packageRef']?.['adtcore:name'] || null,
-            fields: fields.map(f => ({
-                name: f['ddic:name'],
-                dataType: f['ddic:dataType'],
-                length: parseInt(f['ddic:length'], 10),
-                decimals: parseInt(f['ddic:decimals'] || '0', 10),
-                key: f['ddic:keyFlag'] === 'true',
-                description: f['ddic:description']
-            }))
-        };
-    }
-    // fallback: return raw
-    return { raw: result };
-}
-async function handleGetTable(args) {
-    try {
-        if (!args?.table_name) {
-            throw new utils_1.McpError(utils_1.ErrorCode.InvalidParams, 'Table name is required');
-        }
-        const connection = (0, utils_1.getManagedConnection)();
-        const client = new adt_clients_1.CrudClient(connection);
-        await client.readTable(args.table_name);
-        const response = client.getReadResult();
-        if (!response) {
-            throw new utils_1.McpError(utils_1.ErrorCode.InternalError, 'Failed to read table');
-        }
-        // Parse XML responses; fall back to returning the payload as-is
-        if (typeof response.data === 'string' && response.data.trim().startsWith('<?xml')) {
-            const resultObj = parseTableXml(response.data);
-            const result = {
-                isError: false,
-                content: [
-                    {
-                        type: "text",
-                        text: JSON.stringify(resultObj, null, 2)
-                    }
-                ]
-            };
-            if (args.filePath) {
-                (0, writeResultToFile_1.writeResultToFile)(JSON.stringify(result, null, 2), args.filePath);
-            }
-            return result;
-        }
-        else {
-            const plainResult = {
-                isError: false,
-                content: [
-                    {
-                        type: "text",
-                        text: response.data
-                    }
-                ]
-            };
-            if (args.filePath) {
-                (0, writeResultToFile_1.writeResultToFile)(response.data, args.filePath);
-            }
-            return plainResult;
-        }
-    }
-    catch (error) {
-        return {
-            isError: true,
-            content: [
-                {
-                    type: "text",
-                    text: error instanceof Error ? error.message : String(error)
-                }
-            ]
-        };
-    }
-}

package/dist/handlers/table/handleGetTableContents.d.ts

@@ -1,56 +0,0 @@
-/**
- * SAP ADT Data Preview API Implementation - COMPLETED
- *
- * TASK: Fix the SAP ADT data preview API implementation for getting table contents.
- * The task involved correcting the POST request format to match Eclipse ADT behavior,
- * which sends SQL SELECT statements with explicit field lists instead of "SELECT *" queries.
- *
- * COMPLETED FIXES:
- * ✅ Fixed makeAdtRequest function to use POST instead of GET for table contents
- * ✅ Added Content-Type "text/plain; charset=utf-8" for SQL queries
- * ✅ Improved cleanup() function in utils.ts to properly clear axios interceptors
- * ✅ Updated Jest configuration with forceExit and increased timeouts
- * ✅ Added comprehensive logging to handleGetTableContents for debugging
- * ✅ Fixed CDS field parsing to correctly extract field names from "define table" syntax
- * ✅ Generate proper SQL SELECT statements with explicit field lists for any table
- * ✅ Tested with T000, T100S, and TOBJ tables - all working successfully
- * ✅ Added test to verify generated SQL matches expected format
- *
- * IMPLEMENTATION DETAILS:
- * - CDS field parsing using line-by-line regex matching for field definitions
- * - SQL generation with table-prefixed field names (e.g., "T000~MANDT")
- * - Fallback mechanisms for different table structure formats
- * - Proper error handling and logging throughout the process
- * - All 19 tests passing including new SQL format validation test
- *
- * The implementation now correctly sends SQL SELECT statements like:
- * "SELECT T000~MANDT, T000~MTEXT, T000~ORT01, T000~MWAER, T000~ADRNR, ... FROM T000"
- * instead of "SELECT * FROM T000" which SAP ADT doesn't support.
- */
-export declare const TOOL_DEFINITION: {
-    readonly name: "GetTableContents";
-    readonly description: "Retrieve contents of an ABAP table.";
-    readonly inputSchema: {
-        readonly type: "object";
-        readonly properties: {
-            readonly table_name: {
-                readonly type: "string";
-                readonly description: "Name of the ABAP table";
-            };
-            readonly max_rows: {
-                readonly type: "number";
-                readonly description: "Maximum number of rows to retrieve";
-                readonly default: 100;
-            };
-        };
-        readonly required: readonly ["table_name"];
-    };
-};
-export declare function handleGetTableContents(args: any): Promise<{
-    isError: boolean;
-    content: {
-        type: string;
-        text: string;
-    }[];
-}>;
-//# sourceMappingURL=handleGetTableContents.d.ts.map

package/dist/handlers/table/handleGetTableContents.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"handleGetTableContents.d.ts","sourceRoot":"","sources":["../../../src/handlers/table/handleGetTableContents.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4BG;AAQH,eAAO,MAAM,eAAe;;;;;;;;;;;;;;;;;;CAoBlB,CAAC;AAoKX,wBAAsB,sBAAsB,CAAC,IAAI,EAAE,GAAG;;;;;;GAoNrD"}