From 9848587052203485c24f4eaed1927053c40feab9 Mon Sep 17 00:00:00 2001
From: Shoubhit Dash <shoubhit2005@gmail.com>
Date: Wed, 24 Jun 2026 23:29:06 +0530
Subject: [PATCH] refactor(llm): name provider model identity

---
 packages/llm/src/provider-package.ts            |  2 +-
 packages/llm/src/providers/amazon-bedrock.ts    |  4 ++--
 packages/llm/src/providers/anthropic.ts         |  4 ++--
 packages/llm/src/providers/openai-compatible.ts |  4 ++--
 packages/llm/src/providers/openai.ts            | 10 +++++-----
 5 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/packages/llm/src/provider-package.ts b/packages/llm/src/provider-package.ts
index 977a211864..fd878a9d06 100644
--- a/packages/llm/src/provider-package.ts
+++ b/packages/llm/src/provider-package.ts
@@ -10,7 +10,7 @@ export interface Settings extends Readonly<Record<string, unknown>> {
 }
 
 export interface Definition<ProviderSettings extends Settings = Settings> {
-  readonly model: (id: string, settings: ProviderSettings) => Model
+  readonly model: (modelID: string, settings: ProviderSettings) => Model
 }
 
 export * as ProviderPackage from "./provider-package"
diff --git a/packages/llm/src/providers/amazon-bedrock.ts b/packages/llm/src/providers/amazon-bedrock.ts
index e578d614c6..4e5040c9ec 100644
--- a/packages/llm/src/providers/amazon-bedrock.ts
+++ b/packages/llm/src/providers/amazon-bedrock.ts
@@ -50,7 +50,7 @@ export const configure = (input: Config = {}) => {
 }
 
 export const provider = configure()
-export const model: ProviderPackage.Definition<Settings>["model"] = (id, settings) => {
+export const model: ProviderPackage.Definition<Settings>["model"] = (modelID, settings) => {
   if (settings.auth === "bearer" && settings.apiKey === undefined)
     throw new Error("Amazon Bedrock bearer auth requires apiKey")
   if (settings.auth === "sigv4" && settings.apiKey !== undefined)
@@ -64,5 +64,5 @@ export const model: ProviderPackage.Definition<Settings>["model"] = (id, setting
     http: settings.body === undefined ? undefined : { body: { ...settings.body } },
     limits: settings.limits,
     region: settings.region,
-  }).model(id)
+  }).model(modelID)
 }
diff --git a/packages/llm/src/providers/anthropic.ts b/packages/llm/src/providers/anthropic.ts
index 8ef0293eeb..1bf8bf569c 100644
--- a/packages/llm/src/providers/anthropic.ts
+++ b/packages/llm/src/providers/anthropic.ts
@@ -38,11 +38,11 @@ export const configure = (input: Config = {}) => {
 }
 
 export const provider = configure()
-export const model: ProviderPackage.Definition<Settings>["model"] = (id, settings) =>
+export const model: ProviderPackage.Definition<Settings>["model"] = (modelID, settings) =>
   configure({
     apiKey: settings.apiKey,
     baseURL: settings.baseURL,
     headers: settings.headers === undefined ? undefined : { ...settings.headers },
     http: settings.body === undefined ? undefined : { body: { ...settings.body } },
     limits: settings.limits,
-  }).model(id)
+  }).model(modelID)
diff --git a/packages/llm/src/providers/openai-compatible.ts b/packages/llm/src/providers/openai-compatible.ts
index 36d9b3fd5a..8b5d23eed7 100644
--- a/packages/llm/src/providers/openai-compatible.ts
+++ b/packages/llm/src/providers/openai-compatible.ts
@@ -63,7 +63,7 @@ export const provider = {
   configure,
 }
 
-export const model: ProviderPackage.Definition<Settings>["model"] = (id, settings) =>
+export const model: ProviderPackage.Definition<Settings>["model"] = (modelID, settings) =>
   configure({
     apiKey: settings.apiKey,
     baseURL: settings.baseURL,
@@ -71,7 +71,7 @@ export const model: ProviderPackage.Definition<Settings>["model"] = (id, setting
     http: settings.body === undefined ? undefined : { body: { ...settings.body } },
     limits: settings.limits,
     provider: settings.provider,
-  }).model(id)
+  }).model(modelID)
 
 export const baseten = define(profiles.baseten)
 export const cerebras = define(profiles.cerebras)
diff --git a/packages/llm/src/providers/openai.ts b/packages/llm/src/providers/openai.ts
index b40a94514b..6b49b9def2 100644
--- a/packages/llm/src/providers/openai.ts
+++ b/packages/llm/src/providers/openai.ts
@@ -76,15 +76,15 @@ const config = (settings: Settings): Config => ({
   queryParams: settings.queryParams === undefined ? undefined : { ...settings.queryParams },
 })
 
-export const model: ProviderPackage.Definition<Settings>["model"] = (id, settings) => {
+export const model: ProviderPackage.Definition<Settings>["model"] = (modelID, settings) => {
   const configured = configure(config(settings))
-  if (settings.transport === undefined || settings.transport === "http") return configured.responses(id)
-  if (settings.transport === "websocket") return configured.responsesWebSocket(id)
+  if (settings.transport === undefined || settings.transport === "http") return configured.responses(modelID)
+  if (settings.transport === "websocket") return configured.responsesWebSocket(modelID)
   throw new Error(`Unsupported OpenAI Responses transport: ${String(settings.transport)}`)
 }
 
-export const chatModel: ProviderPackage.Definition<Settings>["model"] = (id, settings) =>
-  configure(config(settings)).chat(id)
+export const chatModel: ProviderPackage.Definition<Settings>["model"] = (modelID, settings) =>
+  configure(config(settings)).chat(modelID)
 export const responses = provider.responses
 export const responsesWebSocket = provider.responsesWebSocket
 export const chat = provider.chat