@app-connect/core 1.7.10 → 1.7.11

package/package.json

@@ -1,70 +1,71 @@
-{
-  "name": "@app-connect/core",
-  "version": "1.7.10",
-  "description": "RingCentral App Connect Core",
-  "main": "index.js",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/ringcentral/rc-unified-crm-extension.git"
-  },
-  "keywords": [
-    "RingCentral",
-    "App Connect"
-  ],
-  "author": "RingCentral Labs",
-  "license": "MIT",
-  "peerDependencies": {
-    "axios": "^1.12.2",
-    "express": "^4.22.1",
-    "moment": "^2.29.4",
-    "moment-timezone": "^0.5.39",
-    "pg": "^8.8.0",
-    "sequelize": "^6.29.0"
-  },
-  "dependencies": {
-    "@aws-sdk/client-dynamodb": "^3.751.0",
-    "@aws-sdk/client-s3": "^3.947.0",
-    "@aws-sdk/s3-request-presigner": "^3.947.0",
-    "body-parser": "^1.20.4",
-    "body-parser-xml": "^2.0.5",
-    "client-oauth2": "^4.3.3",
-    "cors": "^2.8.5",
-    "country-state-city": "^3.2.1",
-    "dotenv": "^16.0.3",
-    "dynamoose": "^4.0.3",
-    "jsonwebtoken": "^9.0.0",
-    "mixpanel": "^0.18.0",
-    "shortid": "^2.2.17",
-    "tz-lookup": "^6.1.25",
-    "ua-parser-js": "^1.0.38"
-  },
-  "scripts": {
-    "test": "jest",
-    "test:watch": "jest --watch",
-    "test:coverage": "jest --coverage",
-    "test:ci": "jest --ci --coverage --watchAll=false"
-  },
-  "devDependencies": {
-    "@eslint/js": "^9.22.0",
-    "@octokit/rest": "^19.0.5",
-    "axios": "^1.12.2",
-    "eslint": "^9.22.0",
-    "express": "^4.22.1",
-    "globals": "^16.0.0",
-    "jest": "^29.3.1",
-    "moment": "^2.29.4",
-    "moment-timezone": "^0.5.39",
-    "nock": "^13.2.9",
-    "pg": "^8.8.0",
-    "sequelize": "^6.29.0",
-    "sqlite3": "^5.1.2",
-    "supertest": "^6.3.1"
-  },
-  "overrides": {
-    "js-object-utilities": "2.2.1"
-  },
-  "bugs": {
-    "url": "https://github.com/ringcentral/rc-unified-crm-extension/issues"
-  },
-  "homepage": "https://github.com/ringcentral/rc-unified-crm-extension#readme"
-}
+{
+  "name": "@app-connect/core",
+  "version": "1.7.11",
+  "description": "RingCentral App Connect Core",
+  "main": "index.js",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/ringcentral/rc-unified-crm-extension.git"
+  },
+  "keywords": [
+    "RingCentral",
+    "App Connect"
+  ],
+  "author": "RingCentral Labs",
+  "license": "MIT",
+  "peerDependencies": {
+    "axios": "^1.12.2",
+    "express": "^4.22.1",
+    "moment": "^2.29.4",
+    "moment-timezone": "^0.5.39",
+    "pg": "^8.8.0",
+    "sequelize": "^6.29.0"
+  },
+  "dependencies": {
+    "@aws-sdk/client-dynamodb": "^3.751.0",
+    "@modelcontextprotocol/sdk": "^1.21.1",
+    "@aws-sdk/client-s3": "^3.947.0",
+    "@aws-sdk/s3-request-presigner": "^3.947.0",
+    "body-parser": "^1.20.4",
+    "body-parser-xml": "^2.0.5",
+    "client-oauth2": "^4.3.3",
+    "cors": "^2.8.5",
+    "country-state-city": "^3.2.1",
+    "dotenv": "^16.0.3",
+    "dynamoose": "^4.0.3",
+    "jsonwebtoken": "^9.0.0",
+    "mixpanel": "^0.18.0",
+    "shortid": "^2.2.17",
+    "tz-lookup": "^6.1.25",
+    "ua-parser-js": "^1.0.38"
+  },
+  "scripts": {
+    "test": "jest",
+    "test:watch": "jest --watch",
+    "test:coverage": "jest --coverage",
+    "test:ci": "jest --ci --coverage --watchAll=false"
+  },
+  "devDependencies": {
+    "@eslint/js": "^9.22.0",
+    "@octokit/rest": "^19.0.5",
+    "axios": "^1.12.2",
+    "eslint": "^9.22.0",
+    "express": "^4.22.1",
+    "globals": "^16.0.0",
+    "jest": "^29.3.1",
+    "moment": "^2.29.4",
+    "moment-timezone": "^0.5.39",
+    "nock": "^13.2.9",
+    "pg": "^8.8.0",
+    "sequelize": "^6.29.0",
+    "sqlite3": "^5.1.2",
+    "supertest": "^6.3.1"
+  },
+  "overrides": {
+    "js-object-utilities": "2.2.1"
+  },
+  "bugs": {
+    "url": "https://github.com/ringcentral/rc-unified-crm-extension/issues"
+  },
+  "homepage": "https://github.com/ringcentral/rc-unified-crm-extension#readme"
+}

package/releaseNotes.json

@@ -1,4 +1,16 @@
 {
+    "1.7.11": {
+        "global": [
+            {
+                "type": "New",
+                "description": "Shared-SMS logging"
+            },
+            {
+                "type": "New",
+                "description": "MCP tools"
+            }
+        ]
+    },
     "1.7.10": {
         "global": [
             {

package/test/handlers/log.test.js

@@ -705,6 +705,9 @@ describe('Log Handler', () => {
         createMessageLog: jest.fn().mockResolvedValue({
           logId: 'msg-log-new',
           returnMessage: { message: 'Message logged', messageType: 'success', ttl: 2000 }
+        }),
+        updateMessageLog: jest.fn().mockResolvedValue({
+          returnMessage: { message: 'Message updated', messageType: 'success', ttl: 2000 }
         })
       };
       connectorRegistry.getConnector.mockReturnValue(mockConnector);
@@ -733,8 +736,9 @@ describe('Log Handler', () => {
 
       // Assert
       expect(result.successful).toBe(true);
-      // Only the new message should be logged
-      expect(mockConnector.createMessageLog).toHaveBeenCalledTimes(1);
+      // msg-1 is skipped (already logged), msg-2 uses updateMessageLog because same conversationId exists
+      expect(mockConnector.createMessageLog).toHaveBeenCalledTimes(0);
+      expect(mockConnector.updateMessageLog).toHaveBeenCalledTimes(1);
     });
   });
 

package/test/lib/logger.test.js

@@ -0,0 +1,206 @@
+const { Logger, LOG_LEVELS } = require('../../lib/logger');
+
+describe('Logger', () => {
+  let originalEnv;
+  let consoleSpy;
+  let consoleErrorSpy;
+
+  beforeEach(() => {
+    originalEnv = process.env.NODE_ENV;
+    consoleSpy = jest.spyOn(console, 'log').mockImplementation();
+    consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation();
+  });
+
+  afterEach(() => {
+    process.env.NODE_ENV = originalEnv;
+    consoleSpy.mockRestore();
+    consoleErrorSpy.mockRestore();
+  });
+
+  describe('Log Levels', () => {
+    test('should only log messages at or above the configured level', () => {
+      const logger = new Logger({ level: 'WARN' });
+
+      logger.debug('debug message');
+      logger.info('info message');
+      logger.warn('warn message');
+      logger.error('error message');
+
+      // Only warn and error should be logged
+      expect(consoleSpy).not.toHaveBeenCalled();
+      expect(consoleErrorSpy).toHaveBeenCalledTimes(2);
+    });
+
+    test('should log all messages when level is DEBUG', () => {
+      const logger = new Logger({ level: 'DEBUG' });
+
+      logger.debug('debug message');
+      logger.info('info message');
+      logger.warn('warn message');
+      logger.error('error message');
+
+      expect(consoleSpy).toHaveBeenCalledTimes(2); // debug and info
+      expect(consoleErrorSpy).toHaveBeenCalledTimes(2); // warn and error
+    });
+  });
+
+  describe('Production vs Development', () => {
+    test('should output JSON in production', () => {
+      process.env.NODE_ENV = 'production';
+      const logger = new Logger({ level: 'INFO' });
+
+      logger.info('test message', { userId: '123' });
+
+      const logOutput = consoleSpy.mock.calls[0][0];
+      const parsed = JSON.parse(logOutput);
+
+      expect(parsed).toHaveProperty('timestamp');
+      expect(parsed).toHaveProperty('level', 'INFO');
+      expect(parsed).toHaveProperty('message', 'test message');
+      expect(parsed).toHaveProperty('userId', '123');
+    });
+
+    test('should output human-readable format in development', () => {
+      process.env.NODE_ENV = 'development';
+      const logger = new Logger({ level: 'INFO' });
+
+      logger.info('test message', { userId: '123' });
+
+      const logOutput = consoleSpy.mock.calls[0][0];
+      expect(logOutput).toContain('[INFO]');
+      expect(logOutput).toContain('test message');
+    });
+  });
+
+  describe('Error Handling', () => {
+    test('should extract error information from Error objects', () => {
+      process.env.NODE_ENV = 'production';
+      const logger = new Logger({ level: 'ERROR' });
+      const error = new Error('Test error');
+      error.response = {
+        status: 500,
+        data: { message: 'Server error' },
+      };
+
+      logger.error('Operation failed', { error });
+
+      const logOutput = consoleErrorSpy.mock.calls[0][0];
+      const parsed = JSON.parse(logOutput);
+
+      expect(parsed).toHaveProperty('errorMessage', 'Test error');
+      expect(parsed).toHaveProperty('errorStack');
+      expect(parsed).toHaveProperty('errorStatus', 500);
+      expect(parsed).toHaveProperty('errorResponse');
+      expect(parsed).not.toHaveProperty('error'); // Should be removed
+    });
+  });
+
+  describe('Child Logger', () => {
+    test('should create child logger with default context', () => {
+      process.env.NODE_ENV = 'production';
+      const logger = new Logger({ level: 'INFO' });
+      const childLogger = logger.child({ platform: 'clio', userId: '123' });
+
+      childLogger.info('test message', { operation: 'createLog' });
+
+      const logOutput = consoleSpy.mock.calls[0][0];
+      const parsed = JSON.parse(logOutput);
+
+      expect(parsed).toHaveProperty('platform', 'clio');
+      expect(parsed).toHaveProperty('userId', '123');
+      expect(parsed).toHaveProperty('operation', 'createLog');
+    });
+
+    test('should support nested child loggers', () => {
+      process.env.NODE_ENV = 'production';
+      const logger = new Logger({ level: 'INFO' });
+      const child1 = logger.child({ platform: 'clio' });
+      const child2 = child1.child({ userId: '123' });
+
+      child2.info('test message');
+
+      const logOutput = consoleSpy.mock.calls[0][0];
+      const parsed = JSON.parse(logOutput);
+
+      expect(parsed).toHaveProperty('platform', 'clio');
+      expect(parsed).toHaveProperty('userId', '123');
+    });
+  });
+
+  describe('API Request Logging', () => {
+    test('should log successful API requests at DEBUG level', () => {
+      const logger = new Logger({ level: 'DEBUG' });
+
+      logger.logApiRequest({
+        method: 'GET',
+        url: 'https://api.example.com/users',
+        status: 200,
+        duration: 150,
+        platform: 'clio',
+      });
+
+      expect(consoleSpy).toHaveBeenCalledTimes(1);
+      expect(consoleErrorSpy).not.toHaveBeenCalled();
+    });
+
+    test('should log failed API requests at ERROR level', () => {
+      const logger = new Logger({ level: 'ERROR' });
+      const error = new Error('Request failed');
+
+      logger.logApiRequest({
+        method: 'POST',
+        url: 'https://api.example.com/logs',
+        status: 500,
+        duration: 300,
+        platform: 'clio',
+        error,
+      });
+
+      expect(consoleErrorSpy).toHaveBeenCalledTimes(1);
+      expect(consoleSpy).not.toHaveBeenCalled();
+    });
+
+    test('should log 4xx errors at WARN level', () => {
+      const logger = new Logger({ level: 'WARN' });
+
+      logger.logApiRequest({
+        method: 'GET',
+        url: 'https://api.example.com/contact/999',
+        status: 404,
+        duration: 100,
+        platform: 'clio',
+      });
+
+      expect(consoleErrorSpy).toHaveBeenCalledTimes(1);
+    });
+  });
+
+  describe('Database Query Logging', () => {
+    test('should log successful queries at DEBUG level', () => {
+      const logger = new Logger({ level: 'DEBUG' });
+
+      logger.logDatabaseQuery({
+        operation: 'SELECT',
+        table: 'users',
+        duration: 50,
+      });
+
+      expect(consoleSpy).toHaveBeenCalledTimes(1);
+    });
+
+    test('should log failed queries at ERROR level', () => {
+      const logger = new Logger({ level: 'ERROR' });
+      const error = new Error('Constraint violation');
+
+      logger.logDatabaseQuery({
+        operation: 'INSERT',
+        table: 'call_logs',
+        duration: 20,
+        error,
+      });
+
+      expect(consoleErrorSpy).toHaveBeenCalledTimes(1);
+    });
+  });
+});
+