ebay-mcp 1.7.2 → 1.7.3

package/README.md

@@ -595,8 +595,7 @@ The diagnostic tool checks:
 If you're still experiencing issues:
 
 1. Check existing [GitHub Issues](https://github.com/YosefHayim/ebay-mcp/issues)
-2. Review [GitHub Discussions](https://github.com/YosefHayim/ebay-mcp/discussions)
-3. Create a new issue with:
+2. Create a new issue with:
    - Your diagnostic report (`npm run diagnose:export`)
    - Steps to reproduce the problem
    - Error messages or logs
@@ -628,7 +627,6 @@ Check current eBay API health, incidents, and fixes:
 
 ### Support
 
-- [GitHub Discussions](https://github.com/YosefHayim/ebay-mcp/discussions) - Community Q&A and general discussions
 - [Issue Tracker](https://github.com/YosefHayim/ebay-mcp/issues) - Bug reports and feature requests
 - [Bug Report Template](BUG_REPORT.md) - Detailed bug reporting guide
 

package/build/api/client-trading.js

@@ -0,0 +1,101 @@
+import axios from 'axios';
+import { XMLBuilder, XMLParser } from 'fast-xml-parser';
+import { apiLogger } from '../utils/logger.js';
+const COMPAT_LEVEL = '1451';
+const SITE_ID = '0';
+export class TradingApiClient {
+    restClient;
+    baseUrl;
+    parser;
+    builder;
+    constructor(restClient) {
+        this.restClient = restClient;
+        const env = restClient.getConfig().environment;
+        this.baseUrl =
+            env === 'sandbox'
+                ? 'https://api.sandbox.ebay.com'
+                : 'https://api.ebay.com';
+        this.parser = new XMLParser({
+            ignoreAttributes: false,
+            removeNSPrefix: true,
+            parseTagValue: true,
+            isArray: (_name) => {
+                const arrayTags = [
+                    'Item',
+                    'Errors',
+                    'Error',
+                    'NameValueList',
+                    'Value',
+                    'ShippingServiceOptions',
+                    'InternationalShippingServiceOption',
+                    'PaymentMethods',
+                    'PictureURL',
+                    'CompatibilityList',
+                    'Variation',
+                ];
+                return arrayTags.includes(_name);
+            },
+        });
+        this.builder = new XMLBuilder({
+            ignoreAttributes: false,
+            format: true,
+            suppressEmptyNode: true,
+        });
+    }
+    getBaseUrl() {
+        return this.baseUrl;
+    }
+    async execute(callName, params) {
+        const token = await this.restClient.getOAuthClient().getAccessToken();
+        const requestTag = `${callName}Request`;
+        const responseTag = `${callName}Response`;
+        const xmlObj = {};
+        xmlObj[requestTag] = {
+            '@_xmlns': 'urn:ebay:apis:eBLBaseComponents',
+            ...params,
+        };
+        const xmlBody = `<?xml version="1.0" encoding="utf-8"?>\n${this.builder.build(xmlObj)}`;
+        apiLogger.debug(`Trading API ${callName}`, { xmlBody });
+        let response;
+        try {
+            response = await axios.post(`${this.baseUrl}/ws/api.dll`, xmlBody, {
+                headers: {
+                    'X-EBAY-API-SITEID': SITE_ID,
+                    'X-EBAY-API-COMPATIBILITY-LEVEL': COMPAT_LEVEL,
+                    'X-EBAY-API-CALL-NAME': callName,
+                    'X-EBAY-API-IAF-TOKEN': token,
+                    'Content-Type': 'text/xml',
+                },
+                timeout: 30000,
+            });
+        }
+        catch (error) {
+            const message = error instanceof Error ? error.message : 'Unknown HTTP error';
+            throw new Error(`Trading API ${callName} request failed: ${message}`);
+        }
+        let parsed;
+        try {
+            parsed = this.parser.parse(response.data);
+        }
+        catch (e) {
+            throw new Error(`Failed to parse Trading API ${callName} response: ${e instanceof Error ? e.message : String(e)}`);
+        }
+        const result = (parsed[responseTag] || parsed);
+        // Log warnings without failing
+        if (result.Ack === 'Warning') {
+            apiLogger.warn(`Trading API ${callName} returned warnings`, {
+                errors: result.Errors,
+            });
+        }
+        // Check for eBay errors
+        if (result.Ack === 'Failure' || result.Ack === 'PartialFailure') {
+            const errors = result.Errors;
+            const firstError = Array.isArray(errors) ? errors[0] : errors;
+            const message = firstError?.ShortMessage ||
+                firstError?.LongMessage ||
+                'Unknown Trading API error';
+            throw new Error(message);
+        }
+        return result;
+    }
+}

package/build/api/index.js

@@ -18,6 +18,8 @@ import { EDeliveryApi } from '../api/other/edelivery.js';
 import { IdentityApi } from '../api/other/identity.js';
 import { TranslationApi } from '../api/other/translation.js';
 import { VeroApi } from '../api/other/vero.js';
+import { TradingApiClient } from '../api/client-trading.js';
+import { TradingApi } from '../api/trading/trading.js';
 /**
  * Main API facade providing access to all eBay APIs
  */
@@ -43,6 +45,7 @@ export class EbaySellerApi {
     translation;
     edelivery;
     developer;
+    trading;
     constructor(config) {
         this.client = new EbayApiClient(config);
         // Initialize API category handlers
@@ -65,6 +68,8 @@ export class EbaySellerApi {
         this.translation = new TranslationApi(this.client);
         this.edelivery = new EDeliveryApi(this.client);
         this.developer = new DeveloperApi(this.client);
+        const tradingClient = new TradingApiClient(this.client);
+        this.trading = new TradingApi(tradingClient);
     }
     /**
      * Initialize the API (load tokens from storage)
@@ -123,3 +128,5 @@ export * from '../api/other/identity.js';
 export * from '../api/other/translation.js';
 export * from '../api/other/vero.js';
 export * from '../api/developer/developer.js';
+export * from '../api/trading/trading.js';
+export * from '../api/client-trading.js';

package/build/api/trading/trading.js

@@ -0,0 +1,78 @@
+export class TradingApi {
+    client;
+    constructor(client) {
+        this.client = client;
+    }
+    async getActiveListings(page = 1, entriesPerPage = 50) {
+        const result = await this.client.execute('GetMyeBaySelling', {
+            ActiveList: {
+                Sort: 'TimeLeft',
+                Pagination: {
+                    EntriesPerPage: entriesPerPage,
+                    PageNumber: page,
+                },
+            },
+        });
+        const activeList = result.ActiveList;
+        const itemArray = activeList?.ItemArray;
+        const items = itemArray?.Item || [];
+        const pagination = activeList?.PaginationResult;
+        const listings = items.map((item) => {
+            const sellingStatus = item.SellingStatus;
+            const currentPrice = sellingStatus?.CurrentPrice;
+            const priceValue = typeof currentPrice === 'object' && currentPrice !== null
+                ? Number(currentPrice['#text'] || 0)
+                : Number(currentPrice || 0);
+            return {
+                itemId: String(item.ItemID || ''),
+                title: String(item.Title || ''),
+                sku: String(item.SKU || ''),
+                quantity: Number(item.Quantity || 0),
+                quantityAvailable: Number(item.QuantityAvailable || 0),
+                currentPrice: priceValue,
+                watchCount: Number(item.WatchCount || 0),
+                listingType: String(item.ListingType || ''),
+            };
+        });
+        return {
+            listings,
+            total: Number(pagination?.TotalNumberOfEntries || 0),
+            totalPages: Number(pagination?.TotalNumberOfPages || 0),
+        };
+    }
+    async getListing(itemId) {
+        if (!itemId)
+            throw new Error('itemId is required');
+        const result = await this.client.execute('GetItem', {
+            ItemID: itemId,
+            DetailLevel: 'ReturnAll',
+        });
+        const items = result.Item;
+        return items?.[0] || result;
+    }
+    async createListing(item) {
+        return await this.client.execute('AddFixedPriceItem', { Item: item });
+    }
+    async reviseListing(itemId, fields) {
+        if (!itemId)
+            throw new Error('itemId is required');
+        return await this.client.execute('ReviseFixedPriceItem', {
+            Item: { ...fields, ItemID: itemId },
+        });
+    }
+    async endListing(itemId, reason = 'NotAvailable') {
+        if (!itemId)
+            throw new Error('itemId is required');
+        return await this.client.execute('EndFixedPriceItem', {
+            ItemID: itemId,
+            EndingReason: reason,
+        });
+    }
+    async relistItem(itemId, modifications) {
+        if (!itemId)
+            throw new Error('itemId is required');
+        return await this.client.execute('RelistFixedPriceItem', {
+            Item: { ...modifications, ItemID: itemId },
+        });
+    }
+}

package/build/auth/oauth.js

@@ -1,6 +1,8 @@
 import axios from 'axios';
 import { getBaseUrl, getDefaultScopes } from '../config/environment.js';
 import { LocaleEnum } from '../types/ebay-enums.js';
+import dotenv from 'dotenv';
+import stringify from 'dotenv-stringify';
 import { existsSync, readFileSync, writeFileSync } from 'fs';
 import { join } from 'path';
 import { authLogger } from '../utils/logger.js';
@@ -10,23 +12,13 @@ import { authLogger } from '../utils/logger.js';
 function updateEnvFile(updates) {
     try {
         const envPath = join(process.cwd(), '.env');
-        let envContent = existsSync(envPath) ? readFileSync(envPath, 'utf-8') : '';
-        // Update each key-value pair
-        for (const [key, value] of Object.entries(updates)) {
-            // Match the key with or without value, handling comments
-            const regex = new RegExp(`^(#\\s*)?${key}=.*$`, 'gm');
-            const newLine = `${key}=${value}`;
-            if (regex.test(envContent)) {
-                // Update existing key (uncomment if needed)
-                envContent = envContent.replace(regex, newLine);
-            }
-            else {
-                // Add new key at the end
-                envContent += `\n${newLine}`;
-            }
-        }
-        writeFileSync(envPath, envContent, 'utf-8');
-        // Tokens updated silently - console output interferes with MCP JSON protocol
+        const existingEnv = existsSync(envPath) ? dotenv.parse(readFileSync(envPath, 'utf-8')) : {};
+        // Merge new updates into existing environment object
+        const mergedEnv = { ...existingEnv, ...updates };
+        // Securely stringify the merged environment object
+        const safeEnvContent = stringify(mergedEnv);
+        // Write the updated content back to the .env file
+        writeFileSync(envPath, safeEnvContent, 'utf-8');
     }
     catch (_error) {
         // Silent failure - error logging interferes with MCP JSON protocol

package/build/tools/definitions/index.js

@@ -21,8 +21,9 @@ import { taxonomyTools } from './taxonomy.js';
 import { communicationTools } from './communication.js';
 import { otherApiTools } from './other.js';
 import { developerTools } from './developer.js';
+import { tradingTools } from './trading.js';
 // Export individual categories
-export { tokenManagementTools, accountTools, inventoryTools, fulfillmentTools, marketingTools, analyticsTools, metadataTools, taxonomyTools, communicationTools, otherApiTools, developerTools, };
+export { tokenManagementTools, accountTools, inventoryTools, fulfillmentTools, marketingTools, analyticsTools, metadataTools, taxonomyTools, communicationTools, otherApiTools, developerTools, tradingTools, };
 // Export all tools as a single array
 export const allTools = [
     ...tokenManagementTools,
@@ -36,4 +37,5 @@ export const allTools = [
     ...communicationTools,
     ...otherApiTools,
     ...developerTools,
+    ...tradingTools,
 ];

package/build/tools/definitions/trading.js

@@ -0,0 +1,76 @@
+import { z } from 'zod';
+export const tradingTools = [
+    {
+        name: 'ebay_get_active_listings',
+        description: 'Get all active fixed-price listings with SKU, quantity, price, and watch count.\n\nUses the Trading API (GetMyeBaySelling). Returns listings created via any method (UI, Trading API, or REST API).\n\nRequired: User OAuth token.',
+        inputSchema: {
+            page: z.number().optional().describe('Page number (default 1)'),
+            entriesPerPage: z
+                .number()
+                .optional()
+                .describe('Items per page, max 200 (default 50)'),
+        },
+        annotations: { readOnlyHint: true },
+    },
+    {
+        name: 'ebay_get_listing',
+        description: 'Get full details for a single listing by item ID.\n\nUses the Trading API (GetItem). Returns all listing fields including description, specifics, shipping, and images.\n\nRequired: User OAuth token.',
+        inputSchema: {
+            itemId: z
+                .string()
+                .describe('The eBay item ID (e.g., "167382780779")'),
+        },
+        annotations: { readOnlyHint: true },
+    },
+    {
+        name: 'ebay_create_listing',
+        description: 'Create a new fixed-price listing.\n\nUses the Trading API (AddFixedPriceItem). Requires complete item details.\n\nRequired: User OAuth token.',
+        inputSchema: {
+            item: z
+                .record(z.unknown())
+                .describe('Item details object. Required fields: Title, PrimaryCategory.CategoryID, StartPrice, ConditionID, Country, Currency, DispatchTimeMax, ListingDuration, ListingType ("FixedPriceItem"), Quantity, SKU.'),
+        },
+        annotations: { readOnlyHint: false },
+    },
+    {
+        name: 'ebay_revise_listing',
+        description: 'Revise an existing fixed-price listing. Update quantity, price, title, description, or any other field.\n\nUses the Trading API (ReviseFixedPriceItem). Only send the fields you want to change.\n\nExamples:\n- Update quantity: { "Quantity": 10 }\n- Update price: { "StartPrice": 14.99 }\n- Update title: { "Title": "New Title" }\n- Multiple fields: { "Quantity": 10, "StartPrice": 14.99 }\n\nRequired: User OAuth token.',
+        inputSchema: {
+            itemId: z.string().describe('The eBay item ID to revise'),
+            fields: z
+                .record(z.unknown())
+                .describe('Fields to update (e.g., { "Quantity": 10, "StartPrice": 14.99 })'),
+        },
+        annotations: { readOnlyHint: false },
+    },
+    {
+        name: 'ebay_end_listing',
+        description: 'End/remove an active fixed-price listing.\n\nUses the Trading API (EndFixedPriceItem).\n\nRequired: User OAuth token.',
+        inputSchema: {
+            itemId: z.string().describe('The eBay item ID to end'),
+            reason: z
+                .enum([
+                'NotAvailable',
+                'Incorrect',
+                'LostOrBroken',
+                'OtherListingError',
+                'SellToHighBidder',
+            ])
+                .optional()
+                .describe('Reason for ending (default: NotAvailable)'),
+        },
+        annotations: { readOnlyHint: false, destructiveHint: true },
+    },
+    {
+        name: 'ebay_relist_item',
+        description: 'Relist an ended fixed-price listing, optionally with modifications.\n\nUses the Trading API (RelistFixedPriceItem).\n\nRequired: User OAuth token.',
+        inputSchema: {
+            itemId: z.string().describe('The eBay item ID to relist'),
+            modifications: z
+                .record(z.unknown())
+                .optional()
+                .describe('Optional fields to change when relisting (e.g., { "Quantity": 20 })'),
+        },
+        annotations: { readOnlyHint: false },
+    },
+];

package/build/tools/index.js

@@ -1,5 +1,5 @@
 import { getOAuthAuthorizationUrl, validateScopes } from '../config/environment.js';
-import { accountTools, analyticsTools, communicationTools, developerTools, fulfillmentTools, inventoryTools, marketingTools, metadataTools, otherApiTools, taxonomyTools, tokenManagementTools, } from '../tools/definitions/index.js';
+import { accountTools, analyticsTools, communicationTools, developerTools, fulfillmentTools, inventoryTools, marketingTools, metadataTools, otherApiTools, taxonomyTools, tradingTools, tokenManagementTools, } from '../tools/definitions/index.js';
 import { chatGptTools } from '../tools/tool-definitions.js';
 import { getApiStatusFeed } from '../utils/api-status-feed.js';
 import { convertToTimestamp, validateTokenExpiry } from '../utils/date-converter.js';
@@ -26,6 +26,7 @@ export function getToolDefinitions() {
         ...communicationTools,
         ...otherApiTools,
         ...developerTools,
+        ...tradingTools,
     ];
 }
 /**
@@ -1180,6 +1181,19 @@ export async function executeTool(api, toolName, args) {
             return await api.developer.createSigningKey(args.signingKeyCipher ? { signingKeyCipher: args.signingKeyCipher } : undefined);
         case 'ebay_get_signing_key':
             return await api.developer.getSigningKey(args.signingKeyId);
+        // Trading API - Listing Management
+        case 'ebay_get_active_listings':
+            return await api.trading.getActiveListings(args.page, args.entriesPerPage);
+        case 'ebay_get_listing':
+            return await api.trading.getListing(args.itemId);
+        case 'ebay_create_listing':
+            return await api.trading.createListing(args.item);
+        case 'ebay_revise_listing':
+            return await api.trading.reviseListing(args.itemId, args.fields);
+        case 'ebay_end_listing':
+            return await api.trading.endListing(args.itemId, args.reason);
+        case 'ebay_relist_item':
+            return await api.trading.relistItem(args.itemId, args.modifications);
         default:
             throw new Error(`Unknown tool: ${toolName}`);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ebay-mcp",
-  "version": "1.7.2",
+  "version": "1.7.3",
   "description": "Local MCP server for eBay APIs - provides access to eBay developer functionality through MCP (Model Context Protocol)",
   "type": "module",
   "main": "build/index.js",
@@ -54,11 +54,12 @@
     "url": "https://github.com/YosefHayim/ebay-mcp/issues"
   },
   "dependencies": {
-    "@modelcontextprotocol/sdk": "1.21.1",
+    "@modelcontextprotocol/sdk": "1.26.0",
     "axios": "^1.7.9",
     "chalk": "^5.6.2",
     "cors": "^2.8.5",
     "dotenv": "^17.2.3",
+    "dotenv-stringify": "^3.0.1",
     "express": "^5.1.0",
     "fast-xml-parser": "^5.3.4",
     "helmet": "^8.1.0",