Merge pull request #1299 from corbitsdev/cl-9726-bump-corbitsxai-provider-to-012-and-drop-duplicated-0293-client

fix(xai): use provider client header constants

Author: TheGreatAxios

bun.lock

@@ -89,7 +89,7 @@
     "@corbits/codex-provider": "0.1.1",
     "@corbits/oauth-core": "0.2.0",
     "@corbits/openai-responses": "0.2.1",
-    "@corbits/xai-provider": "0.1.1",
+    "@corbits/xai-provider": "0.1.2",
     "@intx/agent": "workspace:*",
     "@intx/authz": "workspace:*",
     "@intx/crypto": "0.3.0",

package.json

@@ -1,4 +1,9 @@
 import { describe, expect, test } from "bun:test";
+import {
+  XAI_CLIENT_IDENTIFIER,
+  XAI_CLIENT_VERSION,
+  XAI_USER_AGENT,
+} from "@corbits/xai-provider";
 
 import { xaiAuthHeadersForToken } from "./auth-headers.js";
 
@@ -17,6 +22,9 @@ describe("xaiAuthHeadersForToken", () => {
     expect(headers.authorization).toBe(
       `Bearer ${accessTokenWithSub("user-123")}`,
     );
+    expect(headers["user-agent"]).toBe(XAI_USER_AGENT);
+    expect(headers["x-grok-client-identifier"]).toBe(XAI_CLIENT_IDENTIFIER);
+    expect(headers["x-grok-client-version"]).toBe(XAI_CLIENT_VERSION);
   });
 
   test("omits x-grok-user-id when the JWT sub carries CR, LF, or NUL", () => {

src/auth/xai/auth-headers.test.ts

@@ -1,10 +1,9 @@
-import { xaiUserIdFromAccessToken } from "@corbits/xai-provider";
-
 import {
   XAI_CLIENT_IDENTIFIER,
   XAI_CLIENT_VERSION,
   XAI_USER_AGENT,
-} from "./constants.js";
+  xaiUserIdFromAccessToken,
+} from "@corbits/xai-provider";
 
 export function xaiAuthHeadersForToken(token: {
   readonly access: string;

src/auth/xai/auth-headers.ts

@@ -39,14 +39,6 @@ export const XAI_CALLBACK_HOST = callbackTarget.host;
 export const XAI_CALLBACK_PORT = callbackTarget.port;
 export const XAI_CALLBACK_PATH = callbackTarget.path;
 
-// The CLI chat proxy speaks the OpenAI Responses API and authenticates the
-// caller by client headers in addition to the bearer token. Values mirror the
-// grok CLI's own request (captured live). Do not replace these with a product
-// user-agent — the proxy and billing surfaces expect the grok-shell identity.
-export const XAI_CLIENT_IDENTIFIER = "grok-shell";
-export const XAI_CLIENT_VERSION = "0.2.93";
-export const XAI_USER_AGENT = "grok-shell/0.2.93 (macos; aarch64)";
-
 // Cap every token request to the xAI proxy. The refresh runs on the
 // send path before the inference fetch arms its inactivity/total timers, so a
 // stalled token endpoint would otherwise freeze the agent at turn 0.

src/auth/xai/constants.ts

@@ -18,7 +18,7 @@ import {
 describe("xAI OAuth provider projection", () => {
   test("default models lead with the CLI coding model", () => {
     expect(XAI_DEFAULT_MODELS.length).toBeGreaterThan(1);
-    expect(XAI_DEFAULT_MODELS[0]).toBe("grok-4.5");
+    expect(XAI_DEFAULT_MODELS[0]).toBe("grok-4.7");
   });
 
   test("skips grok-4.7 insert when the vendor list already includes it", () => {

src/config/xai-providers.test.ts

@@ -1,4 +1,8 @@
 import { describe, expect, test } from "bun:test";
+import {
+  XAI_CLIENT_IDENTIFIER,
+  XAI_CLIENT_VERSION,
+} from "@corbits/xai-provider";
 import { BEARER_CREDENTIAL_SENTINEL } from "@intx/inference";
 import type {
   ConversationTurn,
@@ -254,8 +258,8 @@ describe("grok-responses buildRequest", () => {
     );
     expect(req.url).toBe("/responses");
     expect(req.headers["authorization"]).toBe(BEARER_CREDENTIAL_SENTINEL);
-    expect(req.headers["x-grok-client-identifier"]).toBe("grok-shell");
-    expect(req.headers["x-grok-client-version"]).toBe("0.2.93");
+    expect(req.headers["x-grok-client-identifier"]).toBe(XAI_CLIENT_IDENTIFIER);
+    expect(req.headers["x-grok-client-version"]).toBe(XAI_CLIENT_VERSION);
     expect(req.headers["x-grok-model-override"]).toBe("grok-4.5");
     expect(req.headers["x-grok-user-id"]).toBe("user-123");
     expect(req.headers["accept"]).toBe("text/event-stream");