fix: update default model to openai/gpt-5-mini and correct GitHub Mod…

README.md

@@ -39,7 +39,7 @@ jobs:
 | `pr_number` | ✅ | — | Pull request number to review |
 | `api_key` | ✅ | — | API key for the provider. For `github-models`: use `GITHUB_TOKEN`. For `copilot`: use a personal PAT with Copilot subscription |
 | `provider` | | `github-models` | AI provider shorthand (see [Providers](#providers)) |
-| `model` | | `gpt-4o` | Model name |
+| `model` | | `openai/gpt-4o-mini` | Model name |
 | `title` | | `AI Code Review` | Title shown in the PR comment header |
 | `api_url` | | — | Full endpoint URL — overrides provider + api_type defaults |
 | `api_type` | | `chat` | API format: `chat`, `responses`, or `messages` |

action.yml

@@ -26,7 +26,7 @@ inputs:
   provider:
     description: "AI provider shorthand: copilot, github-models, openai, anthropic, openrouter, ollama, xai, zai, google, azure, aws, custom — sets api_url and api_type defaults automatically (azure/aws require api_url). Use github-models to authenticate with GITHUB_TOKEN without needing a personal PAT."
     required: false
-    default: "github-models"
+    default: "github-azure-models"
   api_url:
     description: "Full API endpoint URL (e.g. https://api.anthropic.com/v1/messages). Overrides provider + api_type defaults when set."
     required: false

package.json

@@ -14,5 +14,9 @@
     "@vercel/ncc": "^0.44.0",
     "typescript": "^5.0.0",
     "vitest": "^4.1.9"
+  },
+  "dependencies": {
+    "@azure-rest/ai-inference": "^1.0.0-beta.6",
+    "@azure/core-auth": "^1.10.1"
   }
 }

src/__tests__/config.test.ts

@@ -56,7 +56,7 @@ describe("PROVIDER_BASES", () => {
     );
   });
   it("github-models base URL", () => {
-    expect(PROVIDER_BASES["github-models"]).toBe("https://models.inference.ai.azure.com");
+    expect(PROVIDER_BASES["github-models"]).toBe("https://models.github.ai/inference");
   });
   it("azure/aws/custom have no base URL", () => {
     expect(PROVIDER_BASES.azure).toBeUndefined();
@@ -125,7 +125,7 @@ describe("resolveApiUrl", () => {
   });
   it("constructs GitHub Models chat URL", () => {
     expect(resolveApiUrl("github-models", "chat")).toBe(
-      "https://models.inference.ai.azure.com/chat/completions"
+      "https://models.github.ai/inference/chat/completions"
     );
   });
   it("uses override URL when provided, ignoring provider+type", () => {

src/config.ts

@@ -2,6 +2,7 @@ export type ApiType = "chat" | "responses" | "messages" | "text";
 export type Provider =
   | "copilot"
   | "github-models"
+  | "github-azure-models"
   | "openai"
   | "anthropic"
   | "openrouter"
@@ -21,7 +22,8 @@ export const API_TYPE_PATHS: Record<ApiType, string> = {
 };
 
 export const PROVIDER_BASES: Partial<Record<Provider, string>> = {
-    "github-models": "https://models.inference.ai.azure.com",
+  "github-azure-models": "https://models.inference.ai.azure.com",
+  "github-models": "https://models.github.ai/inference",
   copilot: "https://api.githubcopilot.com",
   openai: "https://api.openai.com/v1",
   anthropic: "https://api.anthropic.com/v1",
@@ -67,6 +69,10 @@ export function isAzureProvider(provider: Provider | undefined, url: string): bo
   return provider === "azure" || url.includes(".openai.azure.com");
 }
 
+export function isGithubModelsProvider(provider: Provider | undefined, url: string): boolean {
+  return provider === "github-models" || url.includes(".inference.ai.azure.com");
+}
+
 export function requiresApiUrl(provider: Provider | undefined): boolean {
   return REQUIRES_API_URL.includes(provider as Provider);
 }

src/index.ts

@@ -7,7 +7,10 @@ import {
   isCopilotUrl,
   isAzureProvider,
   requiresApiUrl,
+  isGithubModelsProvider,
 } from "./config";
+import ModelClient, { isUnexpected } from "@azure-rest/ai-inference";
+import { AzureKeyCredential } from "@azure/core-auth";
 
 // ดึงค่าจาก Environment Variables (รองรับทั้ง GitHub Actions inputs และ env vars ตรง)
 const GH_PAT = process.env.INPUT_GH_PAT ?? process.env.GH_PAT;
@@ -130,6 +133,9 @@ async function askAI(diffText: string): Promise<string> {
   const systemPrompt = `You are an expert Code Reviewer. Review the code and respond in ${LANGUAGE}.`;
   const userPrompt = `Please review the following code diff and suggest improvements. Also point out any security vulnerabilities if present. Respond in ${LANGUAGE}.\n\n${diffText}`;
   console.log(`Using api_type: ${API_TYPE} (${API_URL})`);
+  if (isGithubModelsProvider(PROVIDER, API_URL) && API_TYPE === "chat") {
+    return askGitHubModels(systemPrompt, userPrompt);
+  }
   if (API_TYPE === "chat" || API_TYPE === "text") {
     return askOpenAICompat(systemPrompt, userPrompt);
   }
@@ -142,6 +148,33 @@ async function askAI(diffText: string): Promise<string> {
   return askOpenAICompat(systemPrompt, userPrompt);
 }
 
+// 2b. GitHub Models — ใช้ @azure-rest/ai-inference SDK อย่างเป็นทางการ
+async function askGitHubModels(systemPrompt: string, userPrompt: string): Promise<string> {
+  const endpoint = API_URL.replace(/\/chat\/completions\/?$/, "");
+  const token = GH_PAT ?? API_KEY;
+  const client = ModelClient(endpoint, new AzureKeyCredential(token));
+
+  const response = await client.path("/chat/completions").post({
+    body: {
+      messages: [
+        { role: "system", content: systemPrompt },
+        { role: "user", content: userPrompt },
+      ],
+      model: MODEL,
+      max_tokens: MAX_TOKENS,
+    },
+  });
+
+  if (isUnexpected(response)) {
+    const errText = JSON.stringify(response.body.error ?? response.body);
+    console.error(`GitHub Models API error: ${response.status}`);
+    console.error(`Response: ${errText.slice(0, 500)}`);
+    return `⚠️ GitHub Models API returned status ${response.status}. Response: ${errText.slice(0, 200)}`;
+  }
+
+  return response.body.choices?.[0]?.message?.content ?? "";
+}
+
 async function askOpenAICompat(systemPrompt: string, userPrompt: string): Promise<string> {
   const isChat = API_TYPE !== "text";