From 10c7f005be20e39e0d66dae392e6f42ec4677c8d Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sat, 8 Aug 2026 02:56:12 -0700 Subject: [PATCH] refactor(github-copilot): narrow runtime facade (#120544) * refactor(github-copilot): narrow runtime facade * style(github-copilot): format runtime exports --- extensions/github-copilot/index.test.ts | 5 ---- extensions/github-copilot/register.runtime.ts | 29 +++---------------- 2 files changed, 4 insertions(+), 30 deletions(-) diff --git a/extensions/github-copilot/index.test.ts b/extensions/github-copilot/index.test.ts index f50c93acdcc6..518720d53d1b 100644 --- a/extensions/github-copilot/index.test.ts +++ b/extensions/github-copilot/index.test.ts @@ -22,7 +22,6 @@ import { afterAll, afterEach, describe, expect, it, vi } from "vitest"; import { runGitHubCopilotDeviceFlow } from "./login.js"; const mocks = vi.hoisted(() => ({ - githubCopilotLoginCommand: vi.fn(), fetchWithSsrFGuard: vi.fn(async (params) => ({ response: await fetch(params.url, params.init), finalUrl: params.url, @@ -50,7 +49,6 @@ vi.mock("./register.runtime.js", () => ({ DEFAULT_COPILOT_API_BASE_URL: "https://api.githubcopilot.test", resolveCopilotRuntimeAuth: mocks.resolveCopilotRuntimeAuth, resolveCopilotStarterModel: mocks.resolveCopilotStarterModel, - githubCopilotLoginCommand: mocks.githubCopilotLoginCommand, fetchCopilotUsage: vi.fn(), })); @@ -663,7 +661,6 @@ describe("github-copilot plugin", () => { message: "GitHub Copilot auth already exists. Re-run login?", initialValue: false, }); - expect(mocks.githubCopilotLoginCommand).not.toHaveBeenCalled(); expect(result).toEqual({ profiles: [ { @@ -1012,7 +1009,6 @@ describe("github-copilot plugin", () => { message: "GitHub Copilot auth already exists. Re-run login?", initialValue: false, }); - expect(mocks.githubCopilotLoginCommand).not.toHaveBeenCalled(); if (!result) { throw new Error("Expected GitHub Copilot auth result"); } @@ -1287,7 +1283,6 @@ describe("github-copilot plugin", () => { message: "GitHub Copilot auth already exists. Re-run login?", initialValue: false, }); - expect(mocks.githubCopilotLoginCommand).not.toHaveBeenCalled(); expect(result.profiles[0]?.credential).toEqual({ type: "token", provider: "github-copilot", diff --git a/extensions/github-copilot/register.runtime.ts b/extensions/github-copilot/register.runtime.ts index 75e4efa6f237..e4f4b2dd82ee 100644 --- a/extensions/github-copilot/register.runtime.ts +++ b/extensions/github-copilot/register.runtime.ts @@ -1,27 +1,6 @@ // Github Copilot plugin module implements register behavior. -import { - coerceSecretRef, - ensureAuthProfileStore, - listProfilesForProvider, -} from "openclaw/plugin-sdk/provider-auth"; -import { githubCopilotLoginCommand } from "./login.js"; -import { PROVIDER_ID, resolveCopilotForwardCompatModel } from "./models.js"; -import { DEFAULT_COPILOT_API_BASE_URL, resolveCopilotRuntimeAuth } from "./runtime-auth.js"; -import { resolveCopilotStarterModel } from "./starter-model.js"; -import { wrapCopilotAnthropicStream, wrapCopilotProviderStream } from "./stream.js"; -import { fetchCopilotUsage } from "./usage.js"; +export { DEFAULT_COPILOT_API_BASE_URL } from "./runtime-auth.js"; +export { resolveCopilotRuntimeAuth } from "./runtime-auth.js"; -export { - coerceSecretRef, - DEFAULT_COPILOT_API_BASE_URL, - ensureAuthProfileStore, - fetchCopilotUsage, - githubCopilotLoginCommand, - listProfilesForProvider, - PROVIDER_ID, - resolveCopilotRuntimeAuth, - resolveCopilotStarterModel, - resolveCopilotForwardCompatModel, - wrapCopilotAnthropicStream, - wrapCopilotProviderStream, -}; +export { resolveCopilotStarterModel } from "./starter-model.js"; +export { fetchCopilotUsage } from "./usage.js";