refactor(github-copilot): localize internal helpers (#101393)

vincentkoc · web-flow · commit b66220948882 · 2026-07-06T22:58:46.000-07:00
diff --git a/extensions/github-copilot/connection-bound-ids.ts b/extensions/github-copilot/connection-bound-ids.ts
@@ -70,7 +70,7 @@ export function rewriteCopilotConnectionBoundResponseIds(input: unknown): boolea
   return sanitizeCopilotReplayResponseIds(input);
 }
 
-export function sanitizeCopilotReplayResponsePayloadIds(payload: unknown): boolean {
+function sanitizeCopilotReplayResponsePayloadIds(payload: unknown): boolean {
   if (!payload || typeof payload !== "object") {
     return false;
   }
diff --git a/extensions/github-copilot/login.ts b/extensions/github-copilot/login.ts
@@ -260,12 +260,12 @@ function normalizeGitHubDeviceUserCode(raw: string): string {
   return userCode;
 }
 
-export type GitHubCopilotDeviceFlowResult =
+type GitHubCopilotDeviceFlowResult =
   | { status: "authorized"; accessToken: string }
   | { status: "access_denied" }
   | { status: "expired" };
 
-export type GitHubCopilotDeviceFlowIO = {
+type GitHubCopilotDeviceFlowIO = {
   showCode(args: { verificationUrl: string; userCode: string; expiresInMs: number }): Promise<void>;
   openUrl?: (url: string) => Promise<void>;
 };
diff --git a/extensions/github-copilot/models.ts b/extensions/github-copilot/models.ts
@@ -245,7 +245,7 @@ function asCopilotApiModelEntry(value: unknown): CopilotApiModelEntry {
   return value as CopilotApiModelEntry;
 }
 
-export type FetchCopilotModelCatalogParams = {
+type FetchCopilotModelCatalogParams = {
   /** Short-lived Copilot API token (from `resolveCopilotApiToken`). */
   copilotApiToken: string;
   /** Resolved baseUrl from the same token-exchange response. */
diff --git a/extensions/github-copilot/stream.ts b/extensions/github-copilot/stream.ts
@@ -34,7 +34,7 @@ function inferCopilotInitiator(messages: Context["messages"]): "agent" | "user"
   return last.role === "user" ? "user" : "agent";
 }
 
-export function hasCopilotVisionInput(messages: Context["messages"]): boolean {
+function hasCopilotVisionInput(messages: Context["messages"]): boolean {
   return messages.some((message) => {
     if (message.role === "user" && Array.isArray(message.content)) {
       return message.content.some((item) => containsCopilotContentType(item, "image"));
diff --git a/extensions/github-copilot/token.ts b/extensions/github-copilot/token.ts
@@ -3,5 +3,4 @@ export {
   DEFAULT_COPILOT_API_BASE_URL,
   deriveCopilotApiBaseUrlFromToken,
   resolveCopilotApiToken,
-  type CachedCopilotToken,
 } from "openclaw/plugin-sdk/provider-auth";

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ export function rewriteCopilotConnectionBoundResponseIds(input: unknown): boolea`
`70`	`70`	`return sanitizeCopilotReplayResponseIds(input);`
`71`	`71`	`}`
`72`	`72`
`73`		`-export function sanitizeCopilotReplayResponsePayloadIds(payload: unknown): boolean {`
	`73`	`+function sanitizeCopilotReplayResponsePayloadIds(payload: unknown): boolean {`
`74`	`74`	`if (!payload \|\| typeof payload !== "object") {`
`75`	`75`	`return false;`
`76`	`76`	`}`
Original file line number	Diff line number	Diff line change
`@@ -245,7 +245,7 @@ function asCopilotApiModelEntry(value: unknown): CopilotApiModelEntry {`
`245`	`245`	`return value as CopilotApiModelEntry;`
`246`	`246`	`}`
`247`	`247`
`248`		`-export type FetchCopilotModelCatalogParams = {`
	`248`	`+type FetchCopilotModelCatalogParams = {`
`249`	`249`	/** Short-lived Copilot API token (from `resolveCopilotApiToken`). */
`250`	`250`	`copilotApiToken: string;`
`251`	`251`	`/** Resolved baseUrl from the same token-exchange response. */`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ function inferCopilotInitiator(messages: Context["messages"]): "agent" \| "user"`
`34`	`34`	`return last.role === "user" ? "user" : "agent";`
`35`	`35`	`}`
`36`	`36`
`37`		`-export function hasCopilotVisionInput(messages: Context["messages"]): boolean {`
	`37`	`+function hasCopilotVisionInput(messages: Context["messages"]): boolean {`
`38`	`38`	`return messages.some((message) => {`
`39`	`39`	`if (message.role === "user" && Array.isArray(message.content)) {`
`40`	`40`	`return message.content.some((item) => containsCopilotContentType(item, "image"));`