sfcc-dev-mcp 1.0.16 → 1.0.20

package/tests/log-validation.test.ts

@@ -1,149 +0,0 @@
-import { validateLogLevel, formatLogMessage } from '../src/core/handlers/validation-helpers.js';
-import { LogLevelValues } from '../src/utils/log-tool-constants.js';
-
-describe('validateLogLevel', () => {
-  it('should accept valid log levels', () => {
-    const validLevels = ['error', 'warn', 'info', 'debug', 'all'];
-
-    validLevels.forEach(level => {
-      expect(() => validateLogLevel(level, 'test_tool')).not.toThrow();
-    });
-  });
-
-  it('should reject invalid log levels', () => {
-    const invalidLevels = ['trace', 'fatal', 'verbose', 'invalid', ''];
-
-    invalidLevels.forEach(level => {
-      expect(() => validateLogLevel(level, 'test_tool'))
-        .toThrow(/Invalid log level/);
-    });
-  });
-
-  it('should include tool name in error message', () => {
-    expect(() => validateLogLevel('invalid', 'test_tool'))
-      .toThrow(/Invalid log level.*test_tool/);
-  });
-
-  it('should handle null and undefined gracefully', () => {
-    expect(() => validateLogLevel(null as any, 'test_tool')).toThrow();
-    expect(() => validateLogLevel(undefined as any, 'test_tool')).toThrow();
-  });
-
-  it('should be case sensitive', () => {
-    expect(() => validateLogLevel('ERROR', 'test_tool')).toThrow();
-    expect(() => validateLogLevel('ALL', 'test_tool')).toThrow();
-    expect(() => validateLogLevel('Debug', 'test_tool')).toThrow();
-  });
-});
-
-describe('LogLevelValues', () => {
-  it('should contain exactly the expected levels', () => {
-    expect(Object.values(LogLevelValues)).toEqual(['error', 'warn', 'info', 'debug', 'all']);
-  });
-});
-
-describe('formatLogMessage', () => {
-  it('should format message with operation only', () => {
-    const result = formatLogMessage('Testing operation', {});
-    expect(result).toBe('Testing operation');
-  });
-
-  it('should format message with job name', () => {
-    const result = formatLogMessage('Testing operation', { jobName: 'TestJob' });
-    expect(result).toBe('Testing operation jobName=TestJob');
-  });
-
-  it('should format message with level', () => {
-    const result = formatLogMessage('Testing operation', { level: 'error' });
-    expect(result).toBe('Testing operation level=error');
-  });
-
-  it('should format message with limit', () => {
-    const result = formatLogMessage('Testing operation', { limit: 50 });
-    expect(result).toBe('Testing operation limit=50');
-  });
-
-  it('should format message with limit of 0', () => {
-    const result = formatLogMessage('Testing operation', { limit: 0 });
-    expect(result).toBe('Testing operation limit=0');
-  });
-
-  it('should format message with pattern', () => {
-    const result = formatLogMessage('Testing operation', { pattern: 'error-pattern' });
-    expect(result).toBe('Testing operation pattern="error-pattern"');
-  });
-
-  it('should format message with pattern containing spaces', () => {
-    const result = formatLogMessage('Testing operation', { pattern: 'error with spaces' });
-    expect(result).toBe('Testing operation pattern="error with spaces"');
-  });
-
-  it('should format message with all parameters', () => {
-    const result = formatLogMessage('Complex operation', {
-      jobName: 'ComplexJob',
-      level: 'debug',
-      limit: 25,
-      pattern: 'search-term',
-    });
-    expect(result).toBe('Complex operation jobName=ComplexJob level=debug limit=25 pattern="search-term"');
-  });
-
-  it('should handle empty strings in parameters', () => {
-    const result = formatLogMessage('Testing operation', {
-      jobName: '',
-      level: '',
-      pattern: '',
-    });
-    expect(result).toBe('Testing operation');
-  });
-
-  it('should handle undefined limit correctly', () => {
-    const result = formatLogMessage('Testing operation', { jobName: 'TestJob', limit: undefined });
-    expect(result).toBe('Testing operation jobName=TestJob');
-  });
-
-  it('should handle special characters in job name', () => {
-    const result = formatLogMessage('Testing operation', { jobName: 'Job-With_Special.Characters' });
-    expect(result).toBe('Testing operation jobName=Job-With_Special.Characters');
-  });
-
-  it('should handle special characters in pattern', () => {
-    const result = formatLogMessage('Testing operation', { pattern: 'pattern-with_special.chars' });
-    expect(result).toBe('Testing operation pattern="pattern-with_special.chars"');
-  });
-
-  it('should preserve order of parameters', () => {
-    const result = formatLogMessage('Operation', {
-      pattern: 'last',
-      jobName: 'first',
-      level: 'second',
-      limit: 123,
-    });
-    // Should maintain the order: jobName, level, limit, pattern
-    expect(result).toBe('Operation jobName=first level=second limit=123 pattern="last"');
-  });
-
-  it('should handle empty operation string', () => {
-    const result = formatLogMessage('', { jobName: 'TestJob' });
-    expect(result).toBe(' jobName=TestJob');
-  });
-
-  it('should format message with filename', () => {
-    const result = formatLogMessage('Reading file', { filename: 'test.log' });
-    expect(result).toBe('Reading file filename=test.log');
-  });
-
-  it('should format message with date', () => {
-    const result = formatLogMessage('Fetching logs', { date: '20240101' });
-    expect(result).toBe('Fetching logs date=20240101');
-  });
-
-  it('should format message with maxBytes and tailOnly', () => {
-    const result = formatLogMessage('Reading file', {
-      filename: 'test.log',
-      maxBytes: 1024,
-      tailOnly: true,
-    });
-    expect(result).toBe('Reading file filename=test.log maxBytes=1024 tailOnly=true');
-  });
-});

package/tests/logger.test.ts

@@ -1,303 +0,0 @@
-import { Logger } from '../src/utils/logger';
-import { existsSync, readFileSync, rmSync } from 'fs';
-import { join } from 'path';
-import { tmpdir } from 'os';
-
-describe('Logger', () => {
-  let logger: Logger;
-  let testLogDir: string;
-
-  beforeEach(() => {
-    // Create a unique test log directory for each test
-    testLogDir = join(tmpdir(), `sfcc-mcp-logs-test-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`);
-
-    // Clean up if directory somehow exists
-    if (existsSync(testLogDir)) {
-      rmSync(testLogDir, { recursive: true, force: true });
-    }
-  });
-
-  afterEach(() => {
-    // Clean up test log directory
-    if (existsSync(testLogDir)) {
-      rmSync(testLogDir, { recursive: true, force: true });
-    }
-  });
-
-  describe('constructor', () => {
-    it('should create logger and create log directory', () => {
-      logger = new Logger('TEST', true, false, testLogDir);
-
-      expect(existsSync(testLogDir)).toBe(true);
-      expect(logger.getLogDirectory()).toBe(testLogDir);
-    });
-
-    it('should create logger with custom context and write logs correctly', () => {
-      logger = new Logger('CUSTOM-CONTEXT', true, false, testLogDir);
-
-      logger.log('test message');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-info.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toMatch(/^\[\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z\] \[CUSTOM-CONTEXT\] test message\n$/);
-    });
-
-    it('should create logger with timestamp disabled', () => {
-      logger = new Logger('TEST', false, false, testLogDir);
-
-      logger.log('test message');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-info.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toBe('[TEST] test message\n');
-    });
-
-    it('should create logger with debug enabled', () => {
-      logger = new Logger('TEST', true, true, testLogDir);
-
-      logger.debug('debug message');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-debug.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toMatch(
-        /^\[\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z\] \[TEST\] \[DEBUG\] debug message\n$/,
-      );
-    });
-  });
-
-  describe('logging methods', () => {
-    beforeEach(() => {
-      logger = new Logger('TEST', true, false, testLogDir);
-    });
-
-    it('should write info messages to info log file', () => {
-      logger.info('info message');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-info.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toMatch(/^\[\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z\] \[TEST\] info message\n$/);
-    });
-
-    it('should write log messages to info log file', () => {
-      logger.log('log message');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-info.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toMatch(/^\[\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z\] \[TEST\] log message\n$/);
-    });
-
-    it('should write warning messages to warn log file', () => {
-      logger.warn('warning message');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-warn.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toMatch(/^\[\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z\] \[TEST\] warning message\n$/);
-    });
-
-    it('should write error messages to error log file', () => {
-      logger.error('error message');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-error.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toMatch(/^\[\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z\] \[TEST\] error message\n$/);
-    });
-
-    it('should write debug messages to debug log file when debug is enabled', () => {
-      logger = new Logger('TEST', true, true, testLogDir);
-      logger.debug('debug message');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-debug.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toMatch(
-        /^\[\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z\] \[TEST\] \[DEBUG\] debug message\n$/,
-      );
-    });
-
-    it('should not write debug messages when debug is disabled', () => {
-      logger = new Logger('TEST', true, false, testLogDir);
-      logger.debug('debug message');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-debug.log');
-      expect(existsSync(logFile)).toBe(false);
-    });
-
-    it('should handle additional arguments', () => {
-      const testObject = { key: 'value', number: 42 };
-      logger.info('message with args', 'string arg', testObject);
-
-      const logFile = join(testLogDir, 'sfcc-mcp-info.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toContain('message with args');
-      expect(logContent).toContain('string arg');
-      expect(logContent).toContain('"key": "value"');
-      expect(logContent).toContain('"number": 42');
-    });
-  });
-
-  describe('debug logging methods', () => {
-    beforeEach(() => {
-      logger = new Logger('TEST', true, true, testLogDir);
-    });
-
-    it('should log method entry', () => {
-      logger.methodEntry('testMethod', { param1: 'value1' });
-
-      const logFile = join(testLogDir, 'sfcc-mcp-debug.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toContain('[DEBUG] Entering method: testMethod with params:');
-      expect(logContent).toContain('"param1":"value1"');
-    });
-
-    it('should log method entry without params', () => {
-      logger.methodEntry('testMethod');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-debug.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toContain('[DEBUG] Entering method: testMethod');
-      expect(logContent).not.toContain('with params:');
-    });
-
-    it('should log method exit', () => {
-      logger.methodExit('testMethod', { result: 'success' });
-
-      const logFile = join(testLogDir, 'sfcc-mcp-debug.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toContain('[DEBUG] Exiting method: testMethod with result:');
-      expect(logContent).toContain('"result":"success"');
-    });
-
-    it('should log method exit without result', () => {
-      logger.methodExit('testMethod');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-debug.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toContain('[DEBUG] Exiting method: testMethod');
-      expect(logContent).not.toContain('with result:');
-    });
-
-    it('should log timing information', () => {
-      const startTime = Date.now() - 100; // Simulate 100ms operation
-      logger.timing('testOperation', startTime);
-
-      const logFile = join(testLogDir, 'sfcc-mcp-debug.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toMatch(/\[DEBUG\] Performance: testOperation took \d+ms/);
-    });
-  });
-
-  describe('utility methods', () => {
-    beforeEach(() => {
-      logger = new Logger('TEST', true, false, testLogDir);
-    });
-
-    it('should create child logger with combined context', () => {
-      const childLogger = logger.createChildLogger('CHILD');
-      childLogger.info('child message');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-info.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toContain('[TEST:CHILD] child message');
-    });
-
-    it('should enable debug logging dynamically', () => {
-      logger.setDebugEnabled(true);
-      logger.debug('now debug is enabled');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-debug.log');
-      expect(existsSync(logFile)).toBe(true);
-
-      const logContent = readFileSync(logFile, 'utf8');
-      expect(logContent).toContain('[DEBUG] now debug is enabled');
-    });
-
-    it('should disable debug logging dynamically', () => {
-      logger = new Logger('TEST', true, true, testLogDir);
-      logger.setDebugEnabled(false);
-      logger.debug('this should not appear');
-
-      const logFile = join(testLogDir, 'sfcc-mcp-debug.log');
-      expect(existsSync(logFile)).toBe(false);
-    });
-
-    it('should return log directory path', () => {
-      const logDirectory = logger.getLogDirectory();
-      expect(logDirectory).toBe(testLogDir);
-      expect(existsSync(logDirectory)).toBe(true);
-    });
-  });
-
-  describe('error handling', () => {
-    it('should handle file write errors gracefully', () => {
-      // Mock appendFileSync to throw an error
-      // eslint-disable-next-line @typescript-eslint/no-require-imports
-      const fsMock = jest.spyOn(require('fs'), 'appendFileSync');
-      fsMock.mockImplementation(() => {
-        throw new Error('File write error');
-      });
-
-      // Mock stderr.write to capture fallback behavior
-      const stderrSpy = jest.spyOn(process.stderr, 'write').mockImplementation(() => true);
-
-      logger = new Logger('TEST', true, false, testLogDir);
-      logger.error('test error message');
-
-      // Should have attempted to write to stderr as fallback
-      expect(stderrSpy).toHaveBeenCalledWith('[LOGGER ERROR] Could not write to log file: Error: File write error\n');
-      expect(stderrSpy).toHaveBeenCalledWith(expect.stringContaining('test error message'));
-
-      stderrSpy.mockRestore();
-      fsMock.mockRestore();
-    });
-
-    it('should not fallback to stderr for non-error log levels', () => {
-      // Mock appendFileSync to throw an error
-      // eslint-disable-next-line @typescript-eslint/no-require-imports
-      const fsMock = jest.spyOn(require('fs'), 'appendFileSync');
-      fsMock.mockImplementation(() => {
-        throw new Error('File write error');
-      });
-
-      // Mock stderr.write to capture fallback behavior
-      const stderrSpy = jest.spyOn(process.stderr, 'write').mockImplementation(() => true);
-
-      logger = new Logger('TEST', true, false, testLogDir);
-      logger.info('test info message');
-
-      // Should not have written to stderr for non-error levels
-      expect(stderrSpy).not.toHaveBeenCalled();
-
-      stderrSpy.mockRestore();
-      fsMock.mockRestore();
-    });
-  });
-});

package/tests/mcp/AGENTS.md

@@ -1,162 +0,0 @@
-# MCP Aegis - AI Agent Guide
-
-**Target Audience**: AI coding assistants, automated testing agents, and AI-powered development tools
-
-## Overview
-
-# MCP Aegis - AI Agent Guide
-
-## 🎯 About This Project
-
-This project uses **MCP Aegis** for testing Model Context Protocol (MCP) servers.
-
-**MCP Aegis** is a comprehensive Node.js testing library for Model Context Protocol (MCP) servers. It provides both **declarative YAML-based testing** and **programmatic JavaScript/TypeScript testing** with 35+ verified pattern matching capabilities including exact numeric equality, floating-point tolerance, decimal precision validation, modular arithmetic, and comprehensive date/timestamp validation.
-
-## 📁 Dedicated Agent Guides
-
-This guide has been restructured into focused, dedicated guides for each testing approach:
-
-### **[YAML Testing Guide](./yaml/AGENTS.md)**
-- **Focus**: Declarative, human-readable test files
-- **Best For**: Protocol compliance, basic tool testing, maintainable test suites
-- **Audience**: Agents generating `.test.mcp.yml` files
-- **Key Features**: 35+ pattern matching types including exact numeric equality, floating-point tolerance, decimal precision validation, modular arithmetic, and date/timestamp validation, CLI debugging, no programming required
-
-### **[Programmatic Testing Guide](./node/AGENTS.md)**  
-- **Focus**: JavaScript/TypeScript API integration
-- **Best For**: Complex validation, existing test suites, multi-step workflows
-- **Audience**: Agents generating `.programmatic.test.js` files
-- **Key Features**: Jest/Mocha integration, dynamic validation, performance testing
-
-## Quick Decision Matrix
-
-| Use Case | YAML Testing | Programmatic Testing |
-|----------|--------------|---------------------|
-| **Protocol compliance** | ✅ Excellent | ✅ Excellent |
-| **Simple tool testing** | ✅ Perfect | ⚠️ Overkill |
-| **Pattern matching** | ✅ 29+ built-in types | ⚠️ Manual assertions |
-| **Complex validation** | ⚠️ Limited logic | ✅ Full JavaScript |
-| **Multi-step workflows** | ❌ Not supported | ✅ Perfect |
-| **Existing test suites** | ❌ Separate runner | ✅ Jest/Mocha/Node.js |
-| **CI/CD integration** | ✅ JSON output | ✅ Standard test runners |
-| **Maintenance** | ✅ Declarative | ⚠️ Code maintenance |
-
-## 📚 Additional Resources
-
-### Documentation
-- **[Complete Documentation](https://aegis.rhino-inquisitor.com/)** - Full guide and reference
-- **[Installation](https://aegis.rhino-inquisitor.com/installation.html)** | **[Quick Start](https://aegis.rhino-inquisitor.com/quick-start.html)**
-- **[Pattern Matching](https://aegis.rhino-inquisitor.com/pattern-matching.html)** | **[Examples](https://aegis.rhino-inquisitor.com/examples.html)**
-
-### MCP Architecture for AI Agents
-```
-AI Agent → MCP Client → MCP Server → Tools/Services
-    ↓
-MCP Aegis → Test Validation → Quality Assurance
-```
-
-**Common Tool Categories**: Data retrieval, content generation, external services, analysis tools, component libraries, knowledge bases
-
-## Universal Setup
-
-Both testing approaches require initial configuration:
-
-### 1. Installation
-```bash
-npm install -g mcp-aegis
-# OR 
-npm install --save-dev mcp-aegis
-# OR use init to auto-install
-npx mcp-aegis init
-```
-
-### 2. Configuration File (`aegis.config.json`)
-```json
-{
-  "name": "My MCP Server",
-  "command": "node",
-  "args": ["./server.js"],
-  "startupTimeout": 5000,
-  "env": {
-    "NODE_ENV": "test"
-  }
-}
-```
-
-### 3. Test Execution
-```bash
-# YAML Testing
-aegis "tests/**/*.test.mcp.yml" --config "aegis.config.json"
-
-# Programmatic Testing  
-node --test "tests/**/*.programmatic.test.js"
-
-# Interactive debugging (NEW): Test individual tools without files
-aegis query --config "aegis.config.json"                        # List tools
-aegis query tool_name '{"param": "value"}' --config "config.json"   # Test tool
-```
-
-## When to Use Each Approach
-
-### Choose YAML Testing When:
-- Testing basic MCP protocol compliance
-- Validating simple tool executions
-- Creating maintainable test suites for non-developers
-- Using built-in pattern matching is sufficient
-- CI/CD needs declarative test definitions
-
-### Choose Programmatic Testing When:
-- Integration with existing JavaScript/TypeScript test suites
-- Complex validation logic required
-- Multi-step agent workflows need testing
-- Dynamic test case generation needed
-- Performance testing and monitoring required
-
-### Use Query Command For:
-- **Rapid development**: Test tools immediately during development
-- **Server validation**: Verify server behavior before writing comprehensive tests
-- **Tool discovery**: Explore available tools and understand their signatures
-- **Debugging**: Inspect exact responses and error conditions
-- **Prototyping**: Quick validation during MCP server development
-
-## Integration Examples
-
-### GitHub Copilot Integration
-Add to `.github/copilot-instructions.md`:
-```markdown
-## MCP Testing Standards
-1. Use MCP Aegis for all MCP server testing
-2. Create both YAML (protocol) and programmatic (complex) tests
-3. Reference dedicated guides:
-   - YAML: ./AGENTS/yaml/AGENTS.md
-   - Code: ./AGENTS/node/AGENTS.md
-4. Follow 35+ verified pattern matching types including numeric comparisons
-5. Ensure JSON-RPC 2.0 compliance
-```
-
-### NPM Scripts Integration
-```json
-{
-  "scripts": {
-    "test:mcp:yaml": "aegis 'tests/**/*.test.mcp.yml' --config './aegis.config.json'",
-    "test:mcp:code": "node --test 'tests/**/*.programmatic.test.js'",
-    "test:mcp:all": "npm run test:mcp:yaml && npm run test:mcp:code",
-    "test:mcp:ci": "npm run test:mcp:yaml -- --json && npm run test:mcp:code",
-    "debug:mcp:tools": "aegis query --config './aegis.config.json'",
-    "debug:mcp:tool": "aegis query"
-  }
-}
-```
-
-**Usage examples**:
-```bash
-npm run test:mcp:all                                    # Run all tests
-npm run debug:mcp:tools                                 # List available tools  
-npm run debug:mcp:tool read_file '{"path": "test.txt"}' # Test specific tool
-```
-
----
-
-**For detailed implementation guidance, see the dedicated guides:**
-- **[YAML Testing Guide](./yaml/AGENTS.md)** - Pattern matching, CLI options, real examples
-- **[Programmatic Testing Guide](./node/AGENTS.md)** - API reference, frameworks, advanced scenarios