vrr/openclaw
2
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-07-10 00:11:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 22

Revert "fix(ci): annotate unsafe boundary casts"

Browse source 
This reverts commit 8a40f90f62.
This commit is contained in:
Peter Steinberger 2026-05-31 04:03:52 -04:00
parent 4471335d26
commit 0ac725278d
8 changed files with 9 additions and 33 deletions
Download patch file Download diff file Expand all files Collapse all files

1
extensions/anthropic-vertex/region.ts
View file

@ -120,7 +120,6 @@ function resolveAnthropicVertexProjectIdFromAdc(
return undefined;
}
try {
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- ADC credentials are external JSON; only optional project ids are consumed.
const parsed = JSON.parse(readFileSync(credentialsPath, "utf8")) as AdcProjectFile;
return (
normalizeOptionalSecretInput(parsed.project_id) ||

5
extensions/anthropic-vertex/stream-runtime.ts
View file

@ -115,7 +115,6 @@ function createAnthropicVertexOnPayload(params: {
function applyPolicy(payload: unknown): unknown {
if (payload && typeof payload === "object" && !Array.isArray(payload)) {
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Anthropic payload policy mutates open provider JSON params.
applyAnthropicPayloadPolicyToParams(payload as Record<string, unknown>, policy);
}
return payload;
@ -150,7 +149,6 @@ export function createAnthropicVertexStreamFn(
});
return (model, context, options) => {
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- This stream adapter is registered only for Anthropic Messages models.
const transportModel = model as Model<"anthropic-messages"> & {
api: string;
baseUrl?: string;
@ -190,8 +188,7 @@ export function createAnthropicVertexStreamFn(
const budgets = options.thinkingBudgets;
opts.thinkingBudgetTokens =
(budgets && options.reasoning in budgets
? // oxlint-disable-next-line typescript/no-unsafe-type-assertion -- The key presence check above narrows runtime indexing for configured budgets.
budgets[options.reasoning as keyof typeof budgets]
? budgets[options.reasoning as keyof typeof budgets]
: undefined) ?? 10000;
}
} else {

2
extensions/mattermost/src/mattermost/accounts.ts
View file

@ -67,9 +67,7 @@ function mergeMattermostAccountConfig(
accountId: string,
): MattermostAccountConfig {
return resolveMergedAccountConfig<MattermostAccountConfig>({
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Mattermost channel config is plugin-owned open config merged by the account resolver.
channelConfig: cfg.channels?.mattermost as MattermostAccountConfig | undefined,
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Mattermost account entries are plugin-owned open config records.
accounts: cfg.channels?.mattermost?.accounts as
| Record<string, Partial<MattermostAccountConfig>>
| undefined,

5
extensions/mattermost/src/mattermost/client.ts
View file

@ -82,7 +82,6 @@ function buildMattermostApiUrl(baseUrl: string, path: string): string {
export async function readMattermostError(res: Response): Promise<string> {
const contentType = res.headers.get("content-type") ?? "";
if (contentType.includes("application/json")) {
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Mattermost error payloads are external JSON; only optional message is consumed.
const data = (await res.json()) as { message?: string } | undefined;
if (data?.message) {
return data.message;
@ -152,17 +151,14 @@ export function createMattermostClient(params: {
}
if (res.status === 204) {
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Generic request callers type the expected endpoint body; 204 has no body.
return undefined as T;
}
const contentType = res.headers.get("content-type") ?? "";
if (contentType.includes("application/json")) {
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Generic request callers type the expected Mattermost endpoint body.
return (await res.json()) as T;
}
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Generic request callers type the expected Mattermost endpoint body.
return (await res.text()) as T;
};
@ -598,7 +594,6 @@ export async function uploadMattermostFile(
throw new Error(`Mattermost API ${res.status} ${res.statusText}: ${detail || "unknown error"}`);
}
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Mattermost upload response is external JSON; required file id is checked below.
const data = (await res.json()) as { file_infos?: MattermostFileInfo[] };
const info = data.file_infos?.[0];
if (!info?.id) {

1
extensions/microsoft/speech-provider.ts
View file

@ -165,7 +165,6 @@ export async function listMicrosoftVoices(): Promise<SpeechVoiceOption[]> {
});
}
await assertOkOrThrowProviderError(response, "Microsoft voices API error");
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Microsoft voice-list payload is external JSON filtered before returning options.
const voices = (await response.json()) as MicrosoftVoiceListEntry[];
return Array.isArray(voices)
? voices

3
extensions/vydra/shared.ts
View file

@ -42,8 +42,7 @@ type VydraJobPayload = {
function asObject(value: unknown): Record<string, unknown> | undefined {
return typeof value === "object" && value !== null && !Array.isArray(value)
? // oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Vydra webhook payloads are external JSON objects walked defensively by key.
(value as Record<string, unknown>)
? (value as Record<string, unknown>)
: undefined;
}

21
packages/speech-core/src/tts.ts
View file

@ -351,15 +351,13 @@ function resolveTtsRuntimeConfig(cfg: OpenClawConfig): OpenClawConfig {
function asProviderConfig(value: unknown): SpeechProviderConfig {
return typeof value === "object" && value !== null && !Array.isArray(value)
? // oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Speech provider configs are open JSON records normalized at the plugin boundary.
(value as SpeechProviderConfig)
? (value as SpeechProviderConfig)
: {};
}
function asProviderConfigMap(value: unknown): Record<string, unknown> {
return typeof value === "object" && value !== null && !Array.isArray(value)
? // oxlint-disable-next-line typescript/no-unsafe-type-assertion -- TTS provider config maps are open JSON records keyed by provider id.
(value as Record<string, unknown>)
? (value as Record<string, unknown>)
: {};
}
@ -730,7 +728,6 @@ function readPrefs(prefsPath: string): TtsUserPrefs {
if (!existsSync(prefsPath)) {
return {};
}
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- User prefs are trusted local JSON and unknown keys are ignored by readers.
return JSON.parse(readFileSync(prefsPath, "utf8")) as TtsUserPrefs;
} catch {
return {};
@ -903,14 +900,12 @@ export function resolveExplicitTtsOverrides(params: {
`TTS provider "${selectedProvider}" ignored the requested model or voice overrides.`,
);
}
if (!providerOverrides) {
throw new Error(`TTS provider "${selectedProvider}" did not return talk overrides.`);
}
const overridesRecord = providerOverrides as SpeechProviderOverrides;
return {
provider: selectedProvider,
providerOverrides: {
[provider.id]: providerOverrides,
[provider.id]: overridesRecord,
},
};
}
@ -1820,10 +1815,9 @@ export async function textToSpeechTelephony(params: {
config,
provider: resolvedProvider.provider,
});
const synthesizeTelephony = resolvedProvider.provider.synthesizeTelephony;
if (!synthesizeTelephony) {
throw new Error(`TTS provider "${resolvedProvider.provider.id}" lost telephony support`);
}
const synthesizeTelephony = resolvedProvider.provider.synthesizeTelephony as NonNullable<
typeof resolvedProvider.provider.synthesizeTelephony
>;
const prepared = await prepareSpeechSynthesis({
provider: resolvedProvider.provider,
text: params.text,
@ -2050,7 +2044,6 @@ export async function maybeApplyTtsToPayload(params: {
textForAudio = `${textForAudio.slice(0, config.maxTextLength - 3)}...`;
}
} catch (err) {
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Only the message is used for verbose diagnostics after a caught summarizer failure.
const error = err as Error;
logVerbose(`TTS: summarization failed, truncating instead: ${error.message}`);
textForAudio = `${textForAudio.slice(0, maxLength - 3)}...`;

4
packages/speech-core/voice-models.ts
View file

@ -101,7 +101,6 @@ export function voiceProviderSupportsModel(
}
export function resolveVoiceModelRefs(config: unknown): VoiceModelRef[] {
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Voice model config is an open config object; fields are normalized individually below.
const voiceModel = config as VoiceModelConfig | undefined;
if (typeof voiceModel === "string") {
const parsed = parseVoiceModelRef(voiceModel);
@ -213,7 +212,6 @@ export function getVoiceProviderConfig<TConfig extends Record<string, unknown>>(
const configuredKeys = Object.keys(params.providerConfigs);
for (const candidate of candidates) {
if (Object.hasOwn(params.providerConfigs, candidate)) {
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Empty provider config preserves the caller-specific config record type.
return params.providerConfigs[candidate] ?? ({} as TConfig);
}
const normalizedCandidate = normalizeLowercaseString(candidate);
@ -221,11 +219,9 @@ export function getVoiceProviderConfig<TConfig extends Record<string, unknown>>(
(key) => normalizeLowercaseString(key) === normalizedCandidate,
);
if (matchingKey) {
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Empty provider config preserves the caller-specific config record type.
return params.providerConfigs[matchingKey] ?? ({} as TConfig);
}
}
// oxlint-disable-next-line typescript/no-unsafe-type-assertion -- Empty provider config preserves the caller-specific config record type.
return {} as TConfig;
}