refactor: trim internal helper exports

This commit is contained in:
Peter Steinberger
2026-05-02 07:18:57 +01:00
parent 5e35112d21
commit 5acfc89175
7 changed files with 13 additions and 46 deletions

View File

@@ -20,7 +20,7 @@ export function readFlagValue(args, name) {
return undefined;
}
export function consumeStringFlag(argv, index, flag, currentValue) {
function consumeStringFlag(argv, index, flag, currentValue) {
if (argv[index] !== flag) {
return null;
}
@@ -30,18 +30,7 @@ export function consumeStringFlag(argv, index, flag, currentValue) {
};
}
export function consumeStringListFlag(argv, index, flag) {
if (argv[index] !== flag) {
return null;
}
const value = argv[index + 1];
return {
nextIndex: index + 1,
value: typeof value === "string" && value.length > 0 ? value : null,
};
}
export function consumeIntFlag(argv, index, flag, currentValue, options = {}) {
function consumeIntFlag(argv, index, flag, currentValue, options = {}) {
if (argv[index] !== flag) {
return null;
}
@@ -53,7 +42,7 @@ export function consumeIntFlag(argv, index, flag, currentValue, options = {}) {
};
}
export function consumeFloatFlag(argv, index, flag, currentValue, options = {}) {
function consumeFloatFlag(argv, index, flag, currentValue, options = {}) {
if (argv[index] !== flag) {
return null;
}
@@ -84,25 +73,6 @@ export function stringFlag(flag, key) {
};
}
export function stringListFlag(flag, key) {
return {
consume(argv, index) {
const option = consumeStringListFlag(argv, index, flag);
if (!option) {
return null;
}
return {
nextIndex: option.nextIndex,
apply(target) {
if (option.value) {
target[key].push(option.value);
}
},
};
},
};
}
function createAssignedValueFlag(consumeOption) {
return {
consume(argv, index, args) {

View File

@@ -70,9 +70,7 @@ export function loadPersistedAuthProfileState(agentDir?: string): AuthProfileSta
return coerceAuthProfileState(loadJsonFile(resolveAuthStatePath(agentDir)));
}
export function buildPersistedAuthProfileState(
store: AuthProfileState,
): AuthProfileStateStore | null {
function buildPersistedAuthProfileState(store: AuthProfileState): AuthProfileStateStore | null {
const state = coerceAuthProfileState(store);
if (!state.order && !state.lastGood && !state.usageStats) {
return null;

View File

@@ -150,7 +150,7 @@ function applyWhamCooldownResult(params: {
};
}
export async function probeWhamForCooldown(
async function probeWhamForCooldown(
store: AuthProfileStore,
profileId: string,
): Promise<WhamCooldownProbeResult | null> {

View File

@@ -4,7 +4,7 @@ export function isRecord(value: unknown): value is Record<string, unknown> {
return typeof value === "object" && value !== null && !Array.isArray(value);
}
export function normalizeStringArray(value: unknown): string[] | undefined {
function normalizeStringArray(value: unknown): string[] | undefined {
return Array.isArray(value) && value.every((entry) => typeof entry === "string")
? [...value]
: undefined;

View File

@@ -227,7 +227,7 @@ function resolveCliImageRoot(params: { backend: CliBackendConfig; workspaceDir:
return path.join(resolvePreferredOpenClawTmpDir(), "openclaw-cli-images");
}
export function appendImagePathsToPrompt(prompt: string, paths: string[], prefix = ""): string {
function appendImagePathsToPrompt(prompt: string, paths: string[], prefix = ""): string {
if (!paths.length) {
return prompt;
}

View File

@@ -18,8 +18,7 @@ import {
import { getBlockedNetworkModeReason } from "./network-mode.js";
// Targeted denylist: host paths that should never be exposed inside sandbox containers.
// Exported for reuse in security audit collectors.
export const BLOCKED_HOST_PATHS = [
const BLOCKED_HOST_PATHS = [
"/etc",
"/private/etc",
"/proc",
@@ -92,18 +91,18 @@ function parseBindSpec(bind: string): ParsedBindSpec {
* Parse the host/source path from a Docker bind mount string.
* Format: `source:target[:mode]`
*/
export function parseBindSourcePath(bind: string): string {
function parseBindSourcePath(bind: string): string {
return parseBindSpec(bind).source.trim();
}
export function parseBindTargetPath(bind: string): string {
function parseBindTargetPath(bind: string): string {
return parseBindSpec(bind).target.trim();
}
/**
* Normalize a POSIX path: resolve `.`, `..`, collapse `//`, strip trailing `/`.
*/
export function normalizeHostPath(raw: string): string {
function normalizeHostPath(raw: string): string {
return normalizeSandboxHostPath(raw);
}
@@ -135,7 +134,7 @@ export function getBlockedBindReason(bind: string): BlockedBindReason | null {
return null;
}
export function getBlockedReasonForSourcePath(
function getBlockedReasonForSourcePath(
sourceNormalized: string,
blockedHostPaths: string[],
): BlockedBindReason | null {

View File

@@ -133,5 +133,5 @@ export function isInstalledPluginEnabled(
rootConfig: config,
enabledByDefault: record.enabledByDefault,
});
return state.enabled && (record.enabled || state.explicitlyEnabled === true);
return state.enabled && (record.enabled || state.explicitlyEnabled);
}