@bernierllc/ai-provider-openai 1.0.0 → 1.0.2

package/CHANGELOG.md

@@ -0,0 +1,17 @@
+# @bernierllc/ai-provider-openai
+
+## 1.0.2
+
+### Patch Changes
+
+- Updated dependencies [[`73c1d1e`](https://github.com/bernierllc/tools/commit/73c1d1e46114cf2ccfd5b74b83743361247f4502)]:
+  - @bernierllc/logger@1.1.0
+  - @bernierllc/ai-provider-core@1.0.3
+
+## 1.0.1
+
+### Patch Changes
+
+- Updated dependencies [[`24899fa`](https://github.com/bernierllc/tools/commit/24899fa4eaf80ae329f7bd74483cf97a132f5d4c)]:
+  - @bernierllc/logger@1.0.4
+  - @bernierllc/ai-provider-core@1.0.2

package/__tests__/OpenAIProvider.test.ts

@@ -27,24 +27,30 @@ describe('OpenAIProvider', () => {
   beforeEach(() => {
     jest.clearAllMocks();
 
-    // Setup mock OpenAI client
+    // Setup mock OpenAI client with properly typed Jest mocks
+    const mockChatCompletionsCreate = jest.fn() as jest.Mock;
+    const mockEmbeddingsCreate = jest.fn() as jest.Mock;
+    const mockModerationsCreate = jest.fn() as jest.Mock;
+    const mockModelsList = jest.fn() as jest.Mock;
+    const mockModelsRetrieve = jest.fn() as jest.Mock;
+
     mockClient = {
       chat: {
         completions: {
-          create: jest.fn() as any
+          create: mockChatCompletionsCreate as any
         }
       },
       embeddings: {
-        create: jest.fn() as any
+        create: mockEmbeddingsCreate as any
       },
       moderations: {
-        create: jest.fn() as any
+        create: mockModerationsCreate as any
       },
       models: {
-        list: jest.fn() as any,
-        retrieve: jest.fn() as any
+        list: mockModelsList as any,
+        retrieve: mockModelsRetrieve as any
       }
-    } as any;
+    } as unknown as jest.Mocked<OpenAI>;
 
     (OpenAI as jest.MockedClass<typeof OpenAI>).mockImplementation(() => mockClient);
 
@@ -110,7 +116,7 @@ describe('OpenAIProvider', () => {
         }
       };
 
-      mockClient.chat.completions.create.mockResolvedValue(mockResponse as any);
+      (mockClient.chat.completions.create as jest.Mock).mockResolvedValue(mockResponse as any);
 
       const result = await provider.complete({
         messages: [
@@ -145,7 +151,7 @@ describe('OpenAIProvider', () => {
         ]
       };
 
-      mockClient.chat.completions.create.mockResolvedValue(mockResponse as any);
+      (mockClient.chat.completions.create as jest.Mock).mockResolvedValue(mockResponse as any);
 
       const result = await provider.complete({
         messages: [{ role: 'user', content: 'Hello' }]
@@ -156,7 +162,7 @@ describe('OpenAIProvider', () => {
     });
 
     it('should pass all request parameters to OpenAI', async () => {
-      mockClient.chat.completions.create.mockResolvedValue({
+      (mockClient.chat.completions.create as jest.Mock).mockResolvedValue({
         choices: [{ message: { content: 'test' }, finish_reason: 'stop' }]
       } as any);
 
@@ -186,7 +192,7 @@ describe('OpenAIProvider', () => {
     });
 
     it('should use default model if not specified', async () => {
-      mockClient.chat.completions.create.mockResolvedValue({
+      (mockClient.chat.completions.create as jest.Mock).mockResolvedValue({
         choices: [{ message: { content: 'test' }, finish_reason: 'stop' }]
       } as any);
 
@@ -225,7 +231,7 @@ describe('OpenAIProvider', () => {
         {}
       );
 
-      mockClient.chat.completions.create.mockRejectedValue(apiError);
+      (mockClient.chat.completions.create as jest.Mock).mockRejectedValue(apiError);
 
       const result = await provider.complete({
         messages: [{ role: 'user', content: 'Test' }]
@@ -264,7 +270,7 @@ describe('OpenAIProvider', () => {
         }
       ];
 
-      mockClient.chat.completions.create.mockResolvedValue({
+      (mockClient.chat.completions.create as jest.Mock).mockResolvedValue({
         async *[Symbol.asyncIterator]() {
           for (const chunk of mockChunks) {
             yield chunk;
@@ -297,7 +303,7 @@ describe('OpenAIProvider', () => {
         { choices: [{ delta: { content: 'test' }, finish_reason: 'stop' }] }
       ];
 
-      mockClient.chat.completions.create.mockResolvedValue({
+      (mockClient.chat.completions.create as jest.Mock).mockResolvedValue({
         async *[Symbol.asyncIterator]() {
           for (const chunk of mockChunks) {
             yield chunk;
@@ -339,7 +345,7 @@ describe('OpenAIProvider', () => {
         }
       };
 
-      mockClient.embeddings.create.mockResolvedValue(mockResponse as any);
+      (mockClient.embeddings.create as jest.Mock).mockResolvedValue(mockResponse as any);
 
       const result = await provider.generateEmbeddings({
         input: ['text 1', 'text 2']
@@ -358,7 +364,7 @@ describe('OpenAIProvider', () => {
     });
 
     it('should use default embedding model if not specified', async () => {
-      mockClient.embeddings.create.mockResolvedValue({
+      (mockClient.embeddings.create as jest.Mock).mockResolvedValue({
         data: [{ embedding: [1, 2, 3] }],
         usage: { prompt_tokens: 5, total_tokens: 5 }
       } as any);
@@ -375,7 +381,7 @@ describe('OpenAIProvider', () => {
     });
 
     it('should handle API errors', async () => {
-      mockClient.embeddings.create.mockRejectedValue(new Error('API Error'));
+      (mockClient.embeddings.create as jest.Mock).mockRejectedValue(new Error('API Error'));
 
       const result = await provider.generateEmbeddings({
         input: 'test'
@@ -422,7 +428,7 @@ describe('OpenAIProvider', () => {
         ]
       };
 
-      mockClient.moderations.create.mockResolvedValue(mockResponse as any);
+      (mockClient.moderations.create as jest.Mock).mockResolvedValue(mockResponse as any);
 
       const result = await provider.moderate('test content');
 
@@ -443,7 +449,7 @@ describe('OpenAIProvider', () => {
         ]
       };
 
-      mockClient.moderations.create.mockResolvedValue(mockResponse as any);
+      (mockClient.moderations.create as jest.Mock).mockResolvedValue(mockResponse as any);
 
       const result = await provider.moderate('clean content');
 
@@ -452,7 +458,7 @@ describe('OpenAIProvider', () => {
     });
 
     it('should handle API errors', async () => {
-      mockClient.moderations.create.mockRejectedValue(new Error('API Error'));
+      (mockClient.moderations.create as jest.Mock).mockRejectedValue(new Error('API Error'));
 
       const result = await provider.moderate('test');
 
@@ -474,7 +480,7 @@ describe('OpenAIProvider', () => {
         ]
       };
 
-      mockClient.models.list.mockResolvedValue(mockResponse as any);
+      (mockClient.models.list as jest.Mock).mockResolvedValue(mockResponse as any);
 
       const models = await provider.getAvailableModels();
 
@@ -486,7 +492,7 @@ describe('OpenAIProvider', () => {
     });
 
     it('should return cached models if API fails', async () => {
-      mockClient.models.list.mockRejectedValue(new Error('API Error'));
+      (mockClient.models.list as jest.Mock).mockRejectedValue(new Error('API Error'));
 
       const models = await provider.getAvailableModels();
 
@@ -497,7 +503,7 @@ describe('OpenAIProvider', () => {
 
   describe('checkHealth()', () => {
     it('should return healthy status when API is accessible', async () => {
-      mockClient.models.retrieve.mockResolvedValue({ id: 'gpt-3.5-turbo' } as any);
+      (mockClient.models.retrieve as jest.Mock).mockResolvedValue({ id: 'gpt-3.5-turbo' } as any);
 
       const health = await provider.checkHealth();
 
@@ -507,7 +513,7 @@ describe('OpenAIProvider', () => {
     });
 
     it('should return unavailable status when API fails', async () => {
-      mockClient.models.retrieve.mockRejectedValue(new Error('Connection failed'));
+      (mockClient.models.retrieve as jest.Mock).mockRejectedValue(new Error('Connection failed'));
 
       const health = await provider.checkHealth();
 
@@ -519,7 +525,7 @@ describe('OpenAIProvider', () => {
 
   describe('isAvailable()', () => {
     it('should return true when provider is healthy', async () => {
-      mockClient.models.retrieve.mockResolvedValue({ id: 'gpt-3.5-turbo' } as any);
+      (mockClient.models.retrieve as jest.Mock).mockResolvedValue({ id: 'gpt-3.5-turbo' } as any);
 
       const available = await provider.isAvailable();
 
@@ -527,7 +533,7 @@ describe('OpenAIProvider', () => {
     });
 
     it('should return false when provider is unavailable', async () => {
-      mockClient.models.retrieve.mockRejectedValue(new Error('API Error'));
+      (mockClient.models.retrieve as jest.Mock).mockRejectedValue(new Error('API Error'));
 
       const available = await provider.isAvailable();
 

package/__tests__/openai-specific.test.ts

@@ -25,13 +25,16 @@ describe('OpenAI-Specific Features', () => {
   beforeEach(() => {
     jest.clearAllMocks();
 
+    // Setup mock OpenAI client with properly typed Jest mocks
+    const mockChatCompletionsCreate = jest.fn() as jest.Mock;
+
     mockClient = {
       chat: {
         completions: {
-          create: jest.fn() as any
+          create: mockChatCompletionsCreate as any
         }
       }
-    } as any;
+    } as unknown as jest.Mocked<OpenAI>;
 
     (OpenAI as jest.MockedClass<typeof OpenAI>).mockImplementation(() => mockClient);
 
@@ -79,7 +82,7 @@ describe('OpenAI-Specific Features', () => {
         }
       };
 
-      mockClient.chat.completions.create.mockResolvedValue(mockResponse as any);
+      (mockClient.chat.completions.create as jest.Mock).mockResolvedValue(mockResponse as any);
 
       const result = await provider.completionWithFunctions({
         messages: [{ role: 'user', content: 'What is the weather in San Francisco?' }],
@@ -132,7 +135,7 @@ describe('OpenAI-Specific Features', () => {
         }
       };
 
-      mockClient.chat.completions.create.mockResolvedValue(mockResponse as any);
+      (mockClient.chat.completions.create as jest.Mock).mockResolvedValue(mockResponse as any);
 
       const result = await provider.completionWithFunctions({
         messages: [{ role: 'user', content: 'Hello' }],
@@ -185,7 +188,8 @@ describe('OpenAI-Specific Features', () => {
       });
 
       expect(result.success).toBe(true);
-      expect(result.metadata?.functionCall?.name).toBe('get_time');
+      expect(result.metadata?.functionCall).toBeDefined();
+      expect((result.metadata?.functionCall as any)?.name).toBe('get_time');
     });
 
     it('should handle API errors', async () => {
@@ -200,7 +204,7 @@ describe('OpenAI-Specific Features', () => {
         }
       ];
 
-      mockClient.chat.completions.create.mockRejectedValue(new Error('API Error'));
+      (mockClient.chat.completions.create as jest.Mock).mockRejectedValue(new Error('API Error'));
 
       const result = await provider.completionWithFunctions({
         messages: [{ role: 'user', content: 'Test' }],
@@ -232,7 +236,7 @@ describe('OpenAI-Specific Features', () => {
         }
       };
 
-      mockClient.chat.completions.create.mockResolvedValue(mockResponse as any);
+      (mockClient.chat.completions.create as jest.Mock).mockResolvedValue(mockResponse as any);
 
       const result = await provider.analyzeImage(
         'https://example.com/sunset.jpg',
@@ -300,7 +304,7 @@ describe('OpenAI-Specific Features', () => {
         {}
       );
 
-      mockClient.chat.completions.create.mockRejectedValue(apiError);
+      (mockClient.chat.completions.create as jest.Mock).mockRejectedValue(apiError);
 
       const result = await provider.analyzeImage(
         'invalid-url',
@@ -319,7 +323,7 @@ describe('OpenAI-Specific Features', () => {
         {}
       );
 
-      mockClient.chat.completions.create.mockRejectedValue(rateLimitError);
+      (mockClient.chat.completions.create as jest.Mock).mockRejectedValue(rateLimitError);
 
       const result = await provider.analyzeImage(
         'https://example.com/image.jpg',

package/jest.config.cjs

@@ -8,7 +8,8 @@ Redistribution or use in other products or commercial offerings is not permitted
 
 module.exports = {
   preset: 'ts-jest',
-  testEnvironment: 'node',
+  // Use fixed environment to handle Node.js v25+ localStorage issue
+  testEnvironment: '<rootDir>/../../../jest-environment-node-fixed.cjs',
   roots: ['<rootDir>/__tests__'],
   testMatch: ['**/__tests__/**/*.test.ts'],
   collectCoverageFrom: [

package/package.json

@@ -1,17 +1,9 @@
 {
   "name": "@bernierllc/ai-provider-openai",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "OpenAI API adapter implementing the unified AI provider interface",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
-  "scripts": {
-    "build": "tsc",
-    "test": "jest --watch",
-    "test:run": "jest",
-    "test:coverage": "jest --coverage",
-    "lint": "eslint src --ext .ts",
-    "clean": "rm -rf dist"
-  },
   "keywords": [
     "ai",
     "openai",
@@ -27,10 +19,10 @@
   "author": "Bernier LLC",
   "license": "SEE LICENSE IN LICENSE",
   "dependencies": {
-    "@bernierllc/ai-provider-core": "^1.0.1",
-    "@bernierllc/retry-policy": "^0.1.3",
-    "@bernierllc/logger": "^1.0.1",
-    "openai": "^4.20.0"
+    "openai": "^4.20.0",
+    "@bernierllc/ai-provider-core": "1.0.3",
+    "@bernierllc/retry-policy": "0.1.6",
+    "@bernierllc/logger": "1.1.0"
   },
   "devDependencies": {
     "@types/jest": "^29.5.0",
@@ -59,5 +51,17 @@
       "logger": "required",
       "docs-suite": "ready"
     }
+  },
+  "publishConfig": {
+    "access": "public",
+    "registry": "https://registry.npmjs.org/"
+  },
+  "scripts": {
+    "build": "tsc",
+    "test": "jest --watch",
+    "test:run": "jest",
+    "test:coverage": "jest --coverage",
+    "lint": "eslint src --ext .ts",
+    "clean": "rm -rf dist"
   }
-}
+}