Skip to content

Commit 42d73fd

Browse files
steipetesteipete
committed
refactor: remove dead private helpers
1 parent ffcc0d1 commit 42d73fd

110 files changed

Lines changed: 66 additions & 1408 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎extensions/kilocode/provider-models.ts‎

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import type { KilocodeModelCatalogEntry } from "openclaw/plugin-sdk/provider-model-shared";
21
import type { ModelDefinitionConfig } from "openclaw/plugin-sdk/provider-model-shared";
32
import { createSubsystemLogger } from "openclaw/plugin-sdk/runtime-env";
43
import { normalizeLowercaseStringOrEmpty } from "openclaw/plugin-sdk/text-runtime";
@@ -10,6 +9,15 @@ export const KILOCODE_DEFAULT_MODEL_ID = "kilo/auto";
109
export const KILOCODE_DEFAULT_MODEL_REF = `kilocode/${KILOCODE_DEFAULT_MODEL_ID}`;
1110
export const KILOCODE_DEFAULT_MODEL_NAME = "Kilo Auto";
1211

12+
export type KilocodeModelCatalogEntry = {
13+
id: string;
14+
name: string;
15+
reasoning: boolean;
16+
input: Array<"text" | "image">;
17+
contextWindow?: number;
18+
maxTokens?: number;
19+
};
20+
1321
export const KILOCODE_MODEL_CATALOG: KilocodeModelCatalogEntry[] = [
1422
{
1523
id: KILOCODE_DEFAULT_MODEL_ID,

‎extensions/kilocode/shared.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,4 +9,4 @@ export {
99
KILOCODE_MODEL_CATALOG,
1010
} from "./provider-models.js";
1111

12-
export type { KilocodeModelCatalogEntry } from "openclaw/plugin-sdk/provider-model-shared";
12+
export type { KilocodeModelCatalogEntry } from "./provider-models.js";

‎src/agents/channel-tools.ts‎

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -134,23 +134,6 @@ export function resolveChannelMessageToolHints(params: {
134134
.filter(Boolean);
135135
}
136136

137-
export function resolveChannelMessageToolCapabilities(params: {
138-
cfg?: OpenClawConfig;
139-
channel?: string | null;
140-
accountId?: string | null;
141-
}): string[] {
142-
const channelId = normalizeAnyChannelId(params.channel);
143-
if (!channelId) {
144-
return [];
145-
}
146-
const resolve = getChannelPlugin(channelId)?.agentPrompt?.messageToolCapabilities;
147-
if (!resolve) {
148-
return [];
149-
}
150-
const cfg = params.cfg ?? ({} as OpenClawConfig);
151-
return normalizePromptCapabilities(resolve({ cfg, accountId: params.accountId }));
152-
}
153-
154137
export function resolveChannelPromptCapabilities(params: {
155138
cfg?: OpenClawConfig;
156139
channel?: string | null;

‎src/agents/copilot-dynamic-headers.ts‎

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,7 @@
11
import type { Context } from "@mariozechner/pi-ai";
22
import { buildCopilotIdeHeaders } from "../plugin-sdk/provider-auth.js";
33

4-
export {
5-
buildCopilotIdeHeaders,
6-
COPILOT_EDITOR_PLUGIN_VERSION,
7-
COPILOT_EDITOR_VERSION,
8-
COPILOT_GITHUB_API_VERSION,
9-
COPILOT_USER_AGENT,
10-
} from "../plugin-sdk/provider-auth.js";
4+
export { buildCopilotIdeHeaders } from "../plugin-sdk/provider-auth.js";
115

126
function inferCopilotInitiator(messages: Context["messages"]): "agent" | "user" {
137
const last = messages[messages.length - 1];

‎src/agents/model-auth-label.test.ts‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import { beforeEach, describe, expect, it, vi } from "vitest";
22

33
const mocks = vi.hoisted(() => ({
44
ensureAuthProfileStore: vi.fn(),
5+
externalCliDiscoveryForProviderAuth: vi.fn(() => undefined),
56
loadAuthProfileStoreWithoutExternalProfiles: vi.fn(),
67
resolveAuthProfileOrder: vi.fn(),
78
resolveAuthProfileDisplayLabel: vi.fn(),
@@ -12,6 +13,7 @@ const mocks = vi.hoisted(() => ({
1213

1314
vi.mock("./auth-profiles.js", () => ({
1415
ensureAuthProfileStore: mocks.ensureAuthProfileStore,
16+
externalCliDiscoveryForProviderAuth: mocks.externalCliDiscoveryForProviderAuth,
1517
loadAuthProfileStoreWithoutExternalProfiles: mocks.loadAuthProfileStoreWithoutExternalProfiles,
1618
resolveAuthProfileOrder: mocks.resolveAuthProfileOrder,
1719
resolveAuthProfileDisplayLabel: mocks.resolveAuthProfileDisplayLabel,
@@ -35,6 +37,8 @@ describe("resolveModelAuthLabel", () => {
3537
({ resolveModelAuthLabel } = await import("./model-auth-label.js"));
3638
}
3739
mocks.ensureAuthProfileStore.mockReset();
40+
mocks.externalCliDiscoveryForProviderAuth.mockReset();
41+
mocks.externalCliDiscoveryForProviderAuth.mockReturnValue(undefined);
3842
mocks.loadAuthProfileStoreWithoutExternalProfiles.mockReset();
3943
mocks.resolveAuthProfileOrder.mockReset();
4044
mocks.resolveAuthProfileDisplayLabel.mockReset();

‎src/agents/openai-tool-schema.ts‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
11
import { normalizeToolParameterSchema } from "./pi-tools-parameter-schema.js";
2-
export {
3-
resolveOpenAIStrictToolSetting,
4-
resolvesToNativeOpenAIStrictTools,
5-
} from "./openai-strict-tool-setting.js";
2+
export { resolveOpenAIStrictToolSetting } from "./openai-strict-tool-setting.js";
63

74
type ToolWithParameters = {
85
name?: unknown;

‎src/agents/pi-embedded-runner/cache-ttl.ts‎

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -106,17 +106,3 @@ export function readLastCacheTtlTimestamp(
106106
return null;
107107
}
108108
}
109-
110-
export function appendCacheTtlTimestamp(sessionManager: unknown, data: CacheTtlEntryData): void {
111-
const sm = sessionManager as {
112-
appendCustomEntry?: (customType: string, data: unknown) => void;
113-
};
114-
if (!sm?.appendCustomEntry) {
115-
return;
116-
}
117-
try {
118-
sm.appendCustomEntry(CACHE_TTL_CUSTOM_TYPE, data);
119-
} catch {
120-
// ignore persistence failures
121-
}
122-
}

‎src/agents/pi-embedded-runner/openai-stream-wrappers.ts‎

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -464,15 +464,6 @@ export function createCodexNativeWebSearchWrapper(
464464
});
465465
};
466466
}
467-
export function createCodexDefaultTransportWrapper(baseStreamFn: StreamFn | undefined): StreamFn {
468-
const underlying = baseStreamFn ?? streamSimple;
469-
return (model, context, options) =>
470-
underlying(model, context, {
471-
...options,
472-
transport: options?.transport ?? "auto",
473-
});
474-
}
475-
476467
export function createOpenAIDefaultTransportWrapper(baseStreamFn: StreamFn | undefined): StreamFn {
477468
const underlying = baseStreamFn ?? streamSimple;
478469
return (model, context, options) => {

‎src/agents/pi-embedded-runner/proxy-stream-wrappers.ts‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import type { StreamFn } from "@mariozechner/pi-agent-core";
22
import { streamSimple } from "@mariozechner/pi-ai";
33
import type { ThinkLevel } from "../../auto-reply/thinking.js";
4-
import { isProxyReasoningUnsupportedModelHint } from "../../plugin-sdk/provider-model-shared.js";
54
import { normalizeOptionalLowercaseString, readStringValue } from "../../shared/string-coerce.js";
65
import { resolveProviderRequestPolicy } from "../provider-attribution.js";
76
import { resolveProviderRequestPolicyConfig } from "../provider-request-config.js";
@@ -108,7 +107,9 @@ export function createOpenRouterWrapper(
108107
}
109108

110109
export function isProxyReasoningUnsupported(modelId: string): boolean {
111-
return isProxyReasoningUnsupportedModelHint(modelId);
110+
const trimmed = normalizeOptionalLowercaseString(modelId);
111+
const slashIndex = trimmed?.indexOf("/") ?? -1;
112+
return slashIndex > 0 && trimmed?.slice(0, slashIndex) === "x-ai";
112113
}
113114

114115
export function createKilocodeWrapper(

‎src/agents/pi-embedded-runner/run/attempt.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,6 @@ export {
353353
shouldInjectHeartbeatPrompt,
354354
} from "./attempt.prompt-helpers.js";
355355
export {
356-
buildSessionsYieldContextMessage,
357356
persistSessionsYieldContextMessage,
358357
queueSessionsYieldInterruptMessage,
359358
stripSessionsYieldArtifacts,

0 commit comments

Comments
 (0)