@optimizely-opal/opal-tool-ocp-sdk 1.0.0-beta.1 → 1.0.0-beta.3

package/dist/validation/ParameterValidator.test.js

@@ -0,0 +1,282 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const ParameterValidator_1 = require("./ParameterValidator");
+const Models_1 = require("../types/Models");
+describe('ParameterValidator', () => {
+    describe('validate', () => {
+        it('should pass validation for valid parameters', () => {
+            const paramDefs = [
+                new Models_1.Parameter('name', Models_1.ParameterType.String, 'User name', true),
+                new Models_1.Parameter('age', Models_1.ParameterType.Integer, 'User age', false),
+                new Models_1.Parameter('score', Models_1.ParameterType.Number, 'User score', false),
+                new Models_1.Parameter('active', Models_1.ParameterType.Boolean, 'Is active', false),
+                new Models_1.Parameter('tags', Models_1.ParameterType.List, 'User tags', false),
+                new Models_1.Parameter('config', Models_1.ParameterType.Dictionary, 'User config', false)
+            ];
+            const validParams = {
+                name: 'John Doe',
+                age: 25,
+                score: 85.5,
+                active: true,
+                tags: ['admin', 'user'],
+                config: { theme: 'dark', language: 'en' }
+            };
+            const result = ParameterValidator_1.ParameterValidator.validate(validParams, paramDefs);
+            expect(result.isValid).toBe(true);
+            expect(result.errors).toHaveLength(0);
+        });
+        it('should fail validation for missing required parameters', () => {
+            const paramDefs = [
+                new Models_1.Parameter('name', Models_1.ParameterType.String, 'User name', true),
+                new Models_1.Parameter('email', Models_1.ParameterType.String, 'User email', true)
+            ];
+            const params = {
+                name: 'John Doe'
+                // email is missing
+            };
+            const result = ParameterValidator_1.ParameterValidator.validate(params, paramDefs);
+            expect(result.isValid).toBe(false);
+            expect(result.errors).toHaveLength(1);
+            expect(result.errors[0]).toEqual({
+                field: 'email',
+                message: "Required parameter 'email' is missing"
+            });
+        });
+        it('should fail validation for wrong parameter types', () => {
+            const paramDefs = [
+                new Models_1.Parameter('name', Models_1.ParameterType.String, 'User name', true),
+                new Models_1.Parameter('age', Models_1.ParameterType.Integer, 'User age', true),
+                new Models_1.Parameter('active', Models_1.ParameterType.Boolean, 'Is active', true),
+                new Models_1.Parameter('tags', Models_1.ParameterType.List, 'User tags', true),
+                new Models_1.Parameter('config', Models_1.ParameterType.Dictionary, 'User config', true)
+            ];
+            const invalidParams = {
+                name: 123, // should be string
+                age: '25', // should be integer
+                active: 'true', // should be boolean
+                tags: 'tag1,tag2', // should be array
+                config: 'invalid' // should be object
+            };
+            const result = ParameterValidator_1.ParameterValidator.validate(invalidParams, paramDefs);
+            expect(result.isValid).toBe(false);
+            expect(result.errors).toHaveLength(5);
+            expect(result.errors[0].field).toBe('name');
+            expect(result.errors[0].message).toContain('must be a string');
+            expect(result.errors[1].field).toBe('age');
+            expect(result.errors[1].message).toContain('must be an integer');
+            expect(result.errors[2].field).toBe('active');
+            expect(result.errors[2].message).toContain('must be a boolean');
+            expect(result.errors[3].field).toBe('tags');
+            expect(result.errors[3].message).toContain('must be an array');
+            expect(result.errors[4].field).toBe('config');
+            expect(result.errors[4].message).toContain('must be an object');
+        });
+        it('should handle null/undefined parameters correctly', () => {
+            const paramDefs = [
+                new Models_1.Parameter('required', Models_1.ParameterType.String, 'Required param', true),
+                new Models_1.Parameter('optional', Models_1.ParameterType.String, 'Optional param', false)
+            ];
+            const result1 = ParameterValidator_1.ParameterValidator.validate(null, paramDefs);
+            expect(result1.isValid).toBe(false);
+            expect(result1.errors).toHaveLength(1);
+            expect(result1.errors[0].field).toBe('required');
+            const result2 = ParameterValidator_1.ParameterValidator.validate(undefined, paramDefs);
+            expect(result2.isValid).toBe(false);
+            expect(result2.errors).toHaveLength(1);
+            expect(result2.errors[0].field).toBe('required');
+        });
+        it('should allow optional parameters to be missing', () => {
+            const paramDefs = [
+                new Models_1.Parameter('name', Models_1.ParameterType.String, 'User name', true),
+                new Models_1.Parameter('age', Models_1.ParameterType.Integer, 'User age', false)
+            ];
+            const params = {
+                name: 'John Doe'
+                // age is optional and missing
+            };
+            const result = ParameterValidator_1.ParameterValidator.validate(params, paramDefs);
+            expect(result.isValid).toBe(true);
+            expect(result.errors).toHaveLength(0);
+        });
+        it('should distinguish between integer and number types', () => {
+            const paramDefs = [
+                new Models_1.Parameter('count', Models_1.ParameterType.Integer, 'Item count', true),
+                new Models_1.Parameter('score', Models_1.ParameterType.Number, 'Score value', true)
+            ];
+            const params1 = {
+                count: 25.5, // should be integer, not float
+                score: 85.5 // number is fine
+            };
+            const result1 = ParameterValidator_1.ParameterValidator.validate(params1, paramDefs);
+            expect(result1.isValid).toBe(false);
+            expect(result1.errors).toHaveLength(1);
+            expect(result1.errors[0].field).toBe('count');
+            expect(result1.errors[0].message).toContain('must be an integer');
+            const params2 = {
+                count: 25, // integer is fine
+                score: 85.5 // number is fine
+            };
+            const result2 = ParameterValidator_1.ParameterValidator.validate(params2, paramDefs);
+            expect(result2.isValid).toBe(true);
+            expect(result2.errors).toHaveLength(0);
+        });
+        it('should handle array vs object distinction', () => {
+            const paramDefs = [
+                new Models_1.Parameter('tags', Models_1.ParameterType.List, 'Tag list', true),
+                new Models_1.Parameter('config', Models_1.ParameterType.Dictionary, 'Config object', true)
+            ];
+            const params = {
+                tags: { 0: 'tag1', 1: 'tag2' }, // object that looks like array
+                config: ['key1', 'key2'] // array instead of object
+            };
+            const result = ParameterValidator_1.ParameterValidator.validate(params, paramDefs);
+            expect(result.isValid).toBe(false);
+            expect(result.errors).toHaveLength(2);
+            expect(result.errors[0].field).toBe('tags');
+            expect(result.errors[0].message).toContain('must be an array');
+            expect(result.errors[1].field).toBe('config');
+            expect(result.errors[1].message).toContain('must be an object');
+        });
+        it('should handle null values for optional parameters', () => {
+            const paramDefs = [
+                new Models_1.Parameter('name', Models_1.ParameterType.String, 'User name', true),
+                new Models_1.Parameter('age', Models_1.ParameterType.Integer, 'User age', false)
+            ];
+            const params = {
+                name: 'John Doe',
+                age: null // null for optional parameter should be allowed
+            };
+            const result = ParameterValidator_1.ParameterValidator.validate(params, paramDefs);
+            expect(result.isValid).toBe(true);
+            expect(result.errors).toHaveLength(0);
+        });
+        it('should handle edge cases for number validation', () => {
+            const paramDefs = [
+                new Models_1.Parameter('score', Models_1.ParameterType.Number, 'Score value', true)
+            ];
+            // Test NaN
+            const params1 = { score: NaN };
+            const result1 = ParameterValidator_1.ParameterValidator.validate(params1, paramDefs);
+            expect(result1.isValid).toBe(false);
+            expect(result1.errors[0].message).toContain('must be a number');
+            // Test Infinity
+            const params2 = { score: Infinity };
+            const result2 = ParameterValidator_1.ParameterValidator.validate(params2, paramDefs);
+            expect(result2.isValid).toBe(true);
+            expect(result2.errors).toHaveLength(0);
+            // Test negative numbers
+            const params3 = { score: -42.5 };
+            const result3 = ParameterValidator_1.ParameterValidator.validate(params3, paramDefs);
+            expect(result3.isValid).toBe(true);
+            expect(result3.errors).toHaveLength(0);
+        });
+        it('should handle edge cases for integer validation', () => {
+            const paramDefs = [
+                new Models_1.Parameter('count', Models_1.ParameterType.Integer, 'Item count', true)
+            ];
+            // Test negative integers
+            const params1 = { count: -5 };
+            const result1 = ParameterValidator_1.ParameterValidator.validate(params1, paramDefs);
+            expect(result1.isValid).toBe(true);
+            expect(result1.errors).toHaveLength(0);
+            // Test zero
+            const params2 = { count: 0 };
+            const result2 = ParameterValidator_1.ParameterValidator.validate(params2, paramDefs);
+            expect(result2.isValid).toBe(true);
+            expect(result2.errors).toHaveLength(0);
+            // Test very large integers
+            const params3 = { count: Number.MAX_SAFE_INTEGER };
+            const result3 = ParameterValidator_1.ParameterValidator.validate(params3, paramDefs);
+            expect(result3.isValid).toBe(true);
+            expect(result3.errors).toHaveLength(0);
+        });
+        it('should handle empty arrays and objects', () => {
+            const paramDefs = [
+                new Models_1.Parameter('tags', Models_1.ParameterType.List, 'Tag list', true),
+                new Models_1.Parameter('config', Models_1.ParameterType.Dictionary, 'Config object', true)
+            ];
+            const params = {
+                tags: [], // empty array should be valid
+                config: {} // empty object should be valid
+            };
+            const result = ParameterValidator_1.ParameterValidator.validate(params, paramDefs);
+            expect(result.isValid).toBe(true);
+            expect(result.errors).toHaveLength(0);
+        });
+        it('should handle special string values', () => {
+            const paramDefs = [
+                new Models_1.Parameter('text', Models_1.ParameterType.String, 'Text value', true)
+            ];
+            // Test empty string
+            const params1 = { text: '' };
+            const result1 = ParameterValidator_1.ParameterValidator.validate(params1, paramDefs);
+            expect(result1.isValid).toBe(true);
+            expect(result1.errors).toHaveLength(0);
+            // Test string with special characters
+            const params2 = { text: 'Hello\nWorld\t!' };
+            const result2 = ParameterValidator_1.ParameterValidator.validate(params2, paramDefs);
+            expect(result2.isValid).toBe(true);
+            expect(result2.errors).toHaveLength(0);
+        });
+        it('should handle multiple validation errors', () => {
+            const paramDefs = [
+                new Models_1.Parameter('name', Models_1.ParameterType.String, 'User name', true),
+                new Models_1.Parameter('age', Models_1.ParameterType.Integer, 'User age', true),
+                new Models_1.Parameter('email', Models_1.ParameterType.String, 'User email', true)
+            ];
+            const params = {
+                name: 123, // wrong type
+                age: '25' // wrong type
+                // email is missing
+            };
+            const result = ParameterValidator_1.ParameterValidator.validate(params, paramDefs);
+            expect(result.isValid).toBe(false);
+            expect(result.errors).toHaveLength(3);
+            expect(result.errors.some((e) => e.field === 'name')).toBe(true);
+            expect(result.errors.some((e) => e.field === 'age')).toBe(true);
+            expect(result.errors.some((e) => e.field === 'email')).toBe(true);
+        });
+        it('should handle tools with no parameter definitions', () => {
+            const result = ParameterValidator_1.ParameterValidator.validate({ someParam: 'value' }, []);
+            expect(result.isValid).toBe(true);
+            expect(result.errors).toHaveLength(0);
+        });
+        it('should handle extra parameters not in definition', () => {
+            const paramDefs = [
+                new Models_1.Parameter('name', Models_1.ParameterType.String, 'User name', true)
+            ];
+            const params = {
+                name: 'John Doe',
+                extraParam: 'should be ignored'
+            };
+            const result = ParameterValidator_1.ParameterValidator.validate(params, paramDefs);
+            expect(result.isValid).toBe(true);
+            expect(result.errors).toHaveLength(0);
+        });
+        it('should handle nested objects and arrays', () => {
+            const paramDefs = [
+                new Models_1.Parameter('config', Models_1.ParameterType.Dictionary, 'Config object', true),
+                new Models_1.Parameter('matrix', Models_1.ParameterType.List, 'Matrix data', true)
+            ];
+            const params = {
+                config: {
+                    nested: {
+                        deep: {
+                            value: 'test'
+                        }
+                    },
+                    array: [1, 2, 3]
+                },
+                matrix: [
+                    [1, 2, 3],
+                    [4, 5, 6],
+                    { nested: 'object in array' }
+                ]
+            };
+            const result = ParameterValidator_1.ParameterValidator.validate(params, paramDefs);
+            expect(result.isValid).toBe(true);
+            expect(result.errors).toHaveLength(0);
+        });
+    });
+});
+//# sourceMappingURL=ParameterValidator.test.js.map

package/dist/validation/ParameterValidator.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ParameterValidator.test.js","sourceRoot":"","sources":["../../src/validation/ParameterValidator.test.ts"],"names":[],"mappings":";;AAAA,6DAA0D;AAC1D,4CAA2D;AAE3D,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IAClC,QAAQ,CAAC,UAAU,EAAE,GAAG,EAAE;QACxB,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;YACrD,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,MAAM,EAAE,WAAW,EAAE,IAAI,CAAC;gBAC9D,IAAI,kBAAS,CAAC,KAAK,EAAE,sBAAa,CAAC,OAAO,EAAE,UAAU,EAAE,KAAK,CAAC;gBAC9D,IAAI,kBAAS,CAAC,OAAO,EAAE,sBAAa,CAAC,MAAM,EAAE,YAAY,EAAE,KAAK,CAAC;gBACjE,IAAI,kBAAS,CAAC,QAAQ,EAAE,sBAAa,CAAC,OAAO,EAAE,WAAW,EAAE,KAAK,CAAC;gBAClE,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,IAAI,EAAE,WAAW,EAAE,KAAK,CAAC;gBAC7D,IAAI,kBAAS,CAAC,QAAQ,EAAE,sBAAa,CAAC,UAAU,EAAE,aAAa,EAAE,KAAK,CAAC;aACxE,CAAC;YAEF,MAAM,WAAW,GAAG;gBAClB,IAAI,EAAE,UAAU;gBAChB,GAAG,EAAE,EAAE;gBACP,KAAK,EAAE,IAAI;gBACX,MAAM,EAAE,IAAI;gBACZ,IAAI,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC;gBACvB,MAAM,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,QAAQ,EAAE,IAAI,EAAE;aAC1C,CAAC;YAEF,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,WAAW,EAAE,SAAS,CAAC,CAAC;YAEnE,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;YAChE,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,MAAM,EAAE,WAAW,EAAE,IAAI,CAAC;gBAC9D,IAAI,kBAAS,CAAC,OAAO,EAAE,sBAAa,CAAC,MAAM,EAAE,YAAY,EAAE,IAAI,CAAC;aACjE,CAAC;YAEF,MAAM,MAAM,GAAG;gBACb,IAAI,EAAE,UAAU;gBAChB,mBAAmB;aACpB,CAAC;YAEF,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;YAE9D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACnC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YACtC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC;gBAC/B,KAAK,EAAE,OAAO;gBACd,OAAO,EAAE,uCAAuC;aACjD,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;YAC1D,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,MAAM,EAAE,WAAW,EAAE,IAAI,CAAC;gBAC9D,IAAI,kBAAS,CAAC,KAAK,EAAE,sBAAa,CAAC,OAAO,EAAE,UAAU,EAAE,IAAI,CAAC;gBAC7D,IAAI,kBAAS,CAAC,QAAQ,EAAE,sBAAa,CAAC,OAAO,EAAE,WAAW,EAAE,IAAI,CAAC;gBACjE,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,IAAI,EAAE,WAAW,EAAE,IAAI,CAAC;gBAC5D,IAAI,kBAAS,CAAC,QAAQ,EAAE,sBAAa,CAAC,UAAU,EAAE,aAAa,EAAE,IAAI,CAAC;aACvE,CAAC;YAEF,MAAM,aAAa,GAAG;gBACpB,IAAI,EAAE,GAAG,EAAE,mBAAmB;gBAC9B,GAAG,EAAE,IAAI,EAAE,oBAAoB;gBAC/B,MAAM,EAAE,MAAM,EAAE,oBAAoB;gBACpC,IAAI,EAAE,WAAW,EAAE,kBAAkB;gBACrC,MAAM,EAAE,SAAS,CAAC,mBAAmB;aACtC,CAAC;YAEF,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,aAAa,EAAE,SAAS,CAAC,CAAC;YAErE,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACnC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAEtC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;YAC5C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;YAE/D,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YAC3C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;YAEjE,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YAC9C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;YAEhE,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;YAC5C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;YAE/D,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YAC9C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;QAClE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;YAC3D,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,UAAU,EAAE,sBAAa,CAAC,MAAM,EAAE,gBAAgB,EAAE,IAAI,CAAC;gBACvE,IAAI,kBAAS,CAAC,UAAU,EAAE,sBAAa,CAAC,MAAM,EAAE,gBAAgB,EAAE,KAAK,CAAC;aACzE,CAAC;YAEF,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;YAC7D,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACpC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YACvC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YAEjD,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC;YAClE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACpC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YACvC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACnD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;YACxD,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,MAAM,EAAE,WAAW,EAAE,IAAI,CAAC;gBAC9D,IAAI,kBAAS,CAAC,KAAK,EAAE,sBAAa,CAAC,OAAO,EAAE,UAAU,EAAE,KAAK,CAAC;aAC/D,CAAC;YAEF,MAAM,MAAM,GAAG;gBACb,IAAI,EAAE,UAAU;gBAChB,8BAA8B;aAC/B,CAAC;YAEF,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;YAE9D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;YAC7D,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,OAAO,EAAE,sBAAa,CAAC,OAAO,EAAE,YAAY,EAAE,IAAI,CAAC;gBACjE,IAAI,kBAAS,CAAC,OAAO,EAAE,sBAAa,CAAC,MAAM,EAAE,aAAa,EAAE,IAAI,CAAC;aAClE,CAAC;YAEF,MAAM,OAAO,GAAG;gBACd,KAAK,EAAE,IAAI,EAAE,+BAA+B;gBAC5C,KAAK,EAAE,IAAI,CAAE,iBAAiB;aAC/B,CAAC;YAEF,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;YAChE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACpC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YACvC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;YAC9C,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;YAElE,MAAM,OAAO,GAAG;gBACd,KAAK,EAAE,EAAE,EAAI,kBAAkB;gBAC/B,KAAK,EAAE,IAAI,CAAE,iBAAiB;aAC/B,CAAC;YAEF,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;YAChE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACzC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;YACnD,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,IAAI,EAAE,UAAU,EAAE,IAAI,CAAC;gBAC3D,IAAI,kBAAS,CAAC,QAAQ,EAAE,sBAAa,CAAC,UAAU,EAAE,eAAe,EAAE,IAAI,CAAC;aACzE,CAAC;YAEF,MAAM,MAAM,GAAG;gBACb,IAAI,EAAE,EAAE,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,MAAM,EAAE,EAAE,+BAA+B;gBAC/D,MAAM,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,0BAA0B;aACpD,CAAC;YAEF,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;YAC9D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACnC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAEtC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;YAC5C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;YAE/D,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YAC9C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;QAClE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;YAC3D,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,MAAM,EAAE,WAAW,EAAE,IAAI,CAAC;gBAC9D,IAAI,kBAAS,CAAC,KAAK,EAAE,sBAAa,CAAC,OAAO,EAAE,UAAU,EAAE,KAAK,CAAC;aAC/D,CAAC;YAEF,MAAM,MAAM,GAAG;gBACb,IAAI,EAAE,UAAU;gBAChB,GAAG,EAAE,IAAI,CAAC,gDAAgD;aAC3D,CAAC;YAEF,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;YAC9D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;YACxD,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,OAAO,EAAE,sBAAa,CAAC,MAAM,EAAE,aAAa,EAAE,IAAI,CAAC;aAClE,CAAC;YAEF,WAAW;YACX,MAAM,OAAO,GAAG,EAAE,KAAK,EAAE,GAAG,EAAE,CAAC;YAC/B,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;YAChE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACpC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;YAEhE,gBAAgB;YAChB,MAAM,OAAO,GAAG,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC;YACpC,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;YAChE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAEvC,wBAAwB;YACxB,MAAM,OAAO,GAAG,EAAE,KAAK,EAAE,CAAC,IAAI,EAAE,CAAC;YACjC,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;YAChE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACzC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;YACzD,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,OAAO,EAAE,sBAAa,CAAC,OAAO,EAAE,YAAY,EAAE,IAAI,CAAC;aAClE,CAAC;YAEF,yBAAyB;YACzB,MAAM,OAAO,GAAG,EAAE,KAAK,EAAE,CAAC,CAAC,EAAE,CAAC;YAC9B,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;YAChE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAEvC,YAAY;YACZ,MAAM,OAAO,GAAG,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;YAC7B,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;YAChE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAEvC,2BAA2B;YAC3B,MAAM,OAAO,GAAG,EAAE,KAAK,EAAE,MAAM,CAAC,gBAAgB,EAAE,CAAC;YACnD,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;YAChE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACzC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,IAAI,EAAE,UAAU,EAAE,IAAI,CAAC;gBAC3D,IAAI,kBAAS,CAAC,QAAQ,EAAE,sBAAa,CAAC,UAAU,EAAE,eAAe,EAAE,IAAI,CAAC;aACzE,CAAC;YAEF,MAAM,MAAM,GAAG;gBACb,IAAI,EAAE,EAAE,EAAE,8BAA8B;gBACxC,MAAM,EAAE,EAAE,CAAC,+BAA+B;aAC3C,CAAC;YAEF,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;YAC9D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;YAC7C,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,MAAM,EAAE,YAAY,EAAE,IAAI,CAAC;aAChE,CAAC;YAEF,oBAAoB;YACpB,MAAM,OAAO,GAAG,EAAE,IAAI,EAAE,EAAE,EAAE,CAAC;YAC7B,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;YAChE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAEvC,sCAAsC;YACtC,MAAM,OAAO,GAAG,EAAE,IAAI,EAAE,iBAAiB,EAAE,CAAC;YAC5C,MAAM,OAAO,GAAG,uCAAkB,CAAC,QAAQ,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;YAChE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACzC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,MAAM,EAAE,WAAW,EAAE,IAAI,CAAC;gBAC9D,IAAI,kBAAS,CAAC,KAAK,EAAE,sBAAa,CAAC,OAAO,EAAE,UAAU,EAAE,IAAI,CAAC;gBAC7D,IAAI,kBAAS,CAAC,OAAO,EAAE,sBAAa,CAAC,MAAM,EAAE,YAAY,EAAE,IAAI,CAAC;aACjE,CAAC;YAEF,MAAM,MAAM,GAAG;gBACb,IAAI,EAAE,GAAG,EAAE,aAAa;gBACxB,GAAG,EAAE,IAAI,CAAC,aAAa;gBACvB,mBAAmB;aACpB,CAAC;YAEF,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;YAC9D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACnC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAEtC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACjE,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAChE,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACpE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;YAC3D,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,EAAE,SAAS,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC,CAAC;YACvE,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;YAC1D,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,MAAM,EAAE,sBAAa,CAAC,MAAM,EAAE,WAAW,EAAE,IAAI,CAAC;aAC/D,CAAC;YAEF,MAAM,MAAM,GAAG;gBACb,IAAI,EAAE,UAAU;gBAChB,UAAU,EAAE,mBAAmB;aAChC,CAAC;YAEF,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;YAC9D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;YACjD,MAAM,SAAS,GAAG;gBAChB,IAAI,kBAAS,CAAC,QAAQ,EAAE,sBAAa,CAAC,UAAU,EAAE,eAAe,EAAE,IAAI,CAAC;gBACxE,IAAI,kBAAS,CAAC,QAAQ,EAAE,sBAAa,CAAC,IAAI,EAAE,aAAa,EAAE,IAAI,CAAC;aACjE,CAAC;YAEF,MAAM,MAAM,GAAG;gBACb,MAAM,EAAE;oBACN,MAAM,EAAE;wBACN,IAAI,EAAE;4BACJ,KAAK,EAAE,MAAM;yBACd;qBACF;oBACD,KAAK,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;iBACjB;gBACD,MAAM,EAAE;oBACN,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;oBACT,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;oBACT,EAAE,MAAM,EAAE,iBAAiB,EAAE;iBAC9B;aACF,CAAC;YAEF,MAAM,MAAM,GAAG,uCAAkB,CAAC,QAAQ,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;YAC9D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@optimizely-opal/opal-tool-ocp-sdk",
-  "version": "1.0.0-beta.1",
+  "version": "1.0.0-beta.3",
   "description": "OCP SDK for Opal tool",
   "scripts": {
     "validate-deps": "node scripts/validate-deps.js",

package/src/function/GlobalToolFunction.test.ts

@@ -36,6 +36,9 @@ jest.mock('@zaiusinc/app-sdk', () => ({
     warn: jest.fn(),
     debug: jest.fn(),
   },
+  LogVisibility: {
+    Zaius: 'zaius'
+  },
 }));
 
 // Create a concrete implementation for testing

package/src/function/GlobalToolFunction.ts

@@ -1,6 +1,7 @@
 import { GlobalFunction, Response, amendLogContext } from '@zaiusinc/app-sdk';
 import { authenticateGlobalRequest, extractAuthData } from '../auth/AuthUtils';
 import { toolsService } from '../service/Service';
+import { ToolLogger } from '../logging/ToolLogger';
 
 /**
  * Abstract base class for global tool-based function execution
@@ -24,6 +25,8 @@ export abstract class GlobalToolFunction extends GlobalFunction {
    * @returns Response as the HTTP response
    */
   public async perform(): Promise<Response> {
+    const startTime = Date.now();
+
     // Extract customer_id from auth data for global context attribution
     const authInfo = extractAuthData(this.request);
     const customerId = authInfo?.authData?.credentials?.customer_id;
@@ -33,6 +36,14 @@ export abstract class GlobalToolFunction extends GlobalFunction {
       customerId: customerId || ''
     });
 
+    ToolLogger.logRequest(this.request);
+
+    const response = await this.handleRequest();
+    ToolLogger.logResponse(this.request, response, Date.now() - startTime);
+    return response;
+  }
+
+  private async handleRequest(): Promise<Response> {
     if (!(await this.authorizeRequest())) {
       return new Response(403, { error: 'Forbidden' });
     }

package/src/function/ToolFunction.test.ts

@@ -36,6 +36,9 @@ jest.mock('@zaiusinc/app-sdk', () => ({
     warn: jest.fn(),
     debug: jest.fn(),
   },
+  LogVisibility: {
+    Zaius: 'zaius'
+  },
 }));
 
 // Create a concrete implementation for testing

package/src/function/ToolFunction.ts

@@ -1,6 +1,7 @@
 import { Function, Response, amendLogContext } from '@zaiusinc/app-sdk';
 import { authenticateRegularRequest } from '../auth/AuthUtils';
 import { toolsService } from '../service/Service';
+import { ToolLogger } from '../logging/ToolLogger';
 
 /**
  * Abstract base class for tool-based function execution
@@ -19,12 +20,28 @@ export abstract class ToolFunction extends Function {
   }
 
   /**
-   * Process the incoming request using the tools service
+   * Process the incoming request with logging
    *
    * @returns Response as the HTTP response
    */
   public async perform(): Promise<Response> {
+    const startTime = Date.now();
     amendLogContext({ opalThreadId: this.request.headers.get('x-opal-thread-id') || '' });
+
+    ToolLogger.logRequest(this.request);
+
+    const response = await this.handleRequest();
+
+    ToolLogger.logResponse(this.request, response, Date.now() - startTime);
+    return response;
+  }
+
+  /**
+   * Handle the core request processing logic
+   *
+   * @returns Response as the HTTP response
+   */
+  private async handleRequest(): Promise<Response> {
     if (!(await this.authorizeRequest())) {
       return new Response(403, { error: 'Forbidden' });
     }
@@ -33,6 +50,7 @@ export abstract class ToolFunction extends Function {
       const isReady = await this.ready();
       return new Response(200, { ready: isReady });
     }
+
     // Pass 'this' as context so decorated methods can use the existing instance
     return toolsService.processRequest(this.request, this);
   }