refactor!: Rename model and providerid to name.

Author: kinyoklion

packages/sdk/server-ai/__tests__/LDAIClientImpl.test.ts

@@ -15,18 +15,18 @@ it('returns config with interpolated messagess', async () => {
   const client = new LDAIClientImpl(mockLdClient);
   const key = 'test-flag';
   const defaultValue: LDAIDefaults = {
-    model: { id: 'test', parameters: { name: 'test-model' } },
+    model: { name: 'test', parameters: { name: 'test-model' } },
     messages: [],
     enabled: true,
   };
 
   const mockVariation = {
     model: {
-      id: 'example-model',
+      name: 'example-model',
       parameters: { name: 'imagination', temperature: 0.7, maxTokens: 4096 },
     },
     provider: {
-      id: 'example-provider',
+      name: 'example-provider',
     },
     messages: [
       { role: 'system', content: 'Hello {{name}}' },
@@ -45,11 +45,11 @@ it('returns config with interpolated messagess', async () => {
 
   expect(result).toEqual({
     model: {
-      id: 'example-model',
+      name: 'example-model',
       parameters: { name: 'imagination', temperature: 0.7, maxTokens: 4096 },
     },
     provider: {
-      id: 'example-provider',
+      name: 'example-provider',
     },
     messages: [
       { role: 'system', content: 'Hello John' },
@@ -64,7 +64,7 @@ it('includes context in variables for messages interpolation', async () => {
   const client = new LDAIClientImpl(mockLdClient);
   const key = 'test-flag';
   const defaultValue: LDAIDefaults = {
-    model: { id: 'test', parameters: { name: 'test-model' } },
+    model: { name: 'test', parameters: { name: 'test-model' } },
     messages: [],
   };
 
@@ -84,12 +84,12 @@ it('handles missing metadata in variation', async () => {
   const client = new LDAIClientImpl(mockLdClient);
   const key = 'test-flag';
   const defaultValue: LDAIDefaults = {
-    model: { id: 'test', parameters: { name: 'test-model' } },
+    model: { name: 'test', parameters: { name: 'test-model' } },
     messages: [],
   };
 
   const mockVariation = {
-    model: { id: 'example-provider', parameters: { name: 'imagination' } },
+    model: { name: 'example-provider', parameters: { name: 'imagination' } },
     messages: [{ role: 'system', content: 'Hello' }],
   };
 
@@ -98,7 +98,7 @@ it('handles missing metadata in variation', async () => {
   const result = await client.config(key, testContext, defaultValue);
 
   expect(result).toEqual({
-    model: { id: 'example-provider', parameters: { name: 'imagination' } },
+    model: { name: 'example-provider', parameters: { name: 'imagination' } },
     messages: [{ role: 'system', content: 'Hello' }],
     tracker: expect.any(Object),
     enabled: false,
@@ -109,8 +109,8 @@ it('passes the default value to the underlying client', async () => {
   const client = new LDAIClientImpl(mockLdClient);
   const key = 'non-existent-flag';
   const defaultValue: LDAIDefaults = {
-    model: { id: 'default-model', parameters: { name: 'default' } },
-    provider: { id: 'default-provider' },
+    model: { name: 'default-model', parameters: { name: 'default' } },
+    provider: { name: 'default-provider' },
     messages: [{ role: 'system', content: 'Default messages' }],
     enabled: true,
   };

packages/sdk/server-ai/src/api/config/LDAIConfig.ts

@@ -7,7 +7,7 @@ export interface LDModelConfig {
   /**
    * The ID of the model.
    */
-  id: string;
+  name: string;
 
   /**
    * Model specific parameters.
@@ -24,7 +24,7 @@ export interface LDProviderConfig {
   /**
    * The ID of the provider.
    */
-  id: string;
+  name: string;
 }
 
 /**