From 577c5714a1a9ba58fd10b343884cdc1d060c6160 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sat, 2 May 2026 09:11:26 +0100 Subject: [PATCH] refactor: hide google meet helper internals --- extensions/google-meet/src/cli.ts | 2 +- extensions/google-meet/src/create.ts | 8 ++------ extensions/google-meet/src/meet.ts | 2 +- extensions/google-meet/src/oauth.ts | 10 +++++----- 4 files changed, 9 insertions(+), 13 deletions(-) diff --git a/extensions/google-meet/src/cli.ts b/extensions/google-meet/src/cli.ts index e2911ec6261..408b7b39c63 100644 --- a/extensions/google-meet/src/cli.ts +++ b/extensions/google-meet/src/cli.ts @@ -1053,7 +1053,7 @@ function renderTranscriptMarkdown(result: GoogleMeetArtifactsResult): string { return `${lines.join("\n")}\n`; } -export function collectGoogleMeetArtifactWarnings( +function collectGoogleMeetArtifactWarnings( result: GoogleMeetArtifactsResult, ): GoogleMeetExportWarning[] { const warnings: GoogleMeetExportWarning[] = []; diff --git a/extensions/google-meet/src/create.ts b/extensions/google-meet/src/create.ts index 613dbbb9f28..04e6d81eb9c 100644 --- a/extensions/google-meet/src/create.ts +++ b/extensions/google-meet/src/create.ts @@ -19,14 +19,14 @@ function normalizeMode(value: unknown): GoogleMeetMode | undefined { return value === "realtime" || value === "transcribe" ? value : undefined; } -export function normalizeGoogleMeetAccessType(value: unknown): GoogleMeetAccessType | undefined { +function normalizeGoogleMeetAccessType(value: unknown): GoogleMeetAccessType | undefined { const normalized = normalizeOptionalString(value)?.toUpperCase().replaceAll("-", "_"); return normalized === "OPEN" || normalized === "TRUSTED" || normalized === "RESTRICTED" ? normalized : undefined; } -export function normalizeGoogleMeetEntryPointAccess( +function normalizeGoogleMeetEntryPointAccess( value: unknown, ): GoogleMeetEntryPointAccess | undefined { const normalized = normalizeOptionalString(value)?.toUpperCase().replaceAll("-", "_"); @@ -84,10 +84,6 @@ function hasGoogleMeetOAuth(config: GoogleMeetConfig, raw: Record): boolean { - return raw.join !== false && raw.join !== "false"; -} - export async function createMeetFromParams(params: { config: GoogleMeetConfig; runtime: OpenClawPluginApi["runtime"]; diff --git a/extensions/google-meet/src/meet.ts b/extensions/google-meet/src/meet.ts index e3c5b9a524b..846979c8356 100644 --- a/extensions/google-meet/src/meet.ts +++ b/extensions/google-meet/src/meet.ts @@ -454,7 +454,7 @@ export async function endGoogleMeetActiveConference(params: { } } -export async function fetchGoogleMeetConferenceRecord(params: { +async function fetchGoogleMeetConferenceRecord(params: { accessToken: string; conferenceRecord: string; }): Promise { diff --git a/extensions/google-meet/src/oauth.ts b/extensions/google-meet/src/oauth.ts index 935756ec6a3..50b324dfb4e 100644 --- a/extensions/google-meet/src/oauth.ts +++ b/extensions/google-meet/src/oauth.ts @@ -6,11 +6,11 @@ import { } from "openclaw/plugin-sdk/provider-auth-runtime"; import { fetchWithSsrFGuard } from "openclaw/plugin-sdk/ssrf-runtime"; -export const GOOGLE_MEET_REDIRECT_URI = "http://localhost:8085/oauth2callback"; -export const GOOGLE_MEET_AUTH_URL = "https://accounts.google.com/o/oauth2/v2/auth"; -export const GOOGLE_MEET_TOKEN_URL = "https://oauth2.googleapis.com/token"; +const GOOGLE_MEET_REDIRECT_URI = "http://localhost:8085/oauth2callback"; +const GOOGLE_MEET_AUTH_URL = "https://accounts.google.com/o/oauth2/v2/auth"; +const GOOGLE_MEET_TOKEN_URL = "https://oauth2.googleapis.com/token"; const GOOGLE_MEET_TOKEN_HOST = "oauth2.googleapis.com"; -export const GOOGLE_MEET_SCOPES = [ +const GOOGLE_MEET_SCOPES = [ "https://www.googleapis.com/auth/meetings.space.created", "https://www.googleapis.com/auth/meetings.space.readonly", "https://www.googleapis.com/auth/meetings.space.settings", @@ -138,7 +138,7 @@ export async function refreshGoogleMeetAccessToken(params: { ); } -export function shouldUseCachedGoogleMeetAccessToken(params: { +function shouldUseCachedGoogleMeetAccessToken(params: { accessToken?: string; expiresAt?: number; now?: number;