mirror of
https://github.com/openclaw/openclaw.git
synced 2026-04-12 09:41:11 +00:00
agents: address runtime truthfulness review fixes
This commit is contained in:
@@ -5,11 +5,7 @@ import {
|
||||
buildTextObservationFields,
|
||||
sanitizeForConsole,
|
||||
} from "./pi-embedded-error-observation.js";
|
||||
import {
|
||||
classifyFailoverReason,
|
||||
classifyProviderRuntimeFailureKind,
|
||||
formatAssistantErrorText,
|
||||
} from "./pi-embedded-helpers.js";
|
||||
import { classifyFailoverReason, formatAssistantErrorText } from "./pi-embedded-helpers.js";
|
||||
import {
|
||||
consumePendingToolMediaReply,
|
||||
hasAssistantVisibleReply,
|
||||
@@ -55,10 +51,6 @@ export function handleAgentEnd(ctx: EmbeddedPiSubscribeContext): void | Promise<
|
||||
const failoverReason = classifyFailoverReason(rawError ?? "", {
|
||||
provider: lastAssistant.provider,
|
||||
});
|
||||
const providerRuntimeFailureKind = classifyProviderRuntimeFailureKind({
|
||||
message: rawError ?? "",
|
||||
provider: lastAssistant.provider,
|
||||
});
|
||||
const errorText = (friendlyError || lastAssistant.errorMessage || "LLM request failed.").trim();
|
||||
const observedError = buildApiErrorObservationFields(rawError, {
|
||||
provider: lastAssistant.provider,
|
||||
@@ -83,7 +75,6 @@ export function handleAgentEnd(ctx: EmbeddedPiSubscribeContext): void | Promise<
|
||||
model: lastAssistant.model,
|
||||
provider: lastAssistant.provider,
|
||||
...observedError,
|
||||
providerRuntimeFailureKind,
|
||||
consoleMessage: `embedded run agent end: runId=${safeRunId} isError=true model=${safeModel} provider=${safeProvider} error=${safeErrorText}${rawErrorConsoleSuffix}`,
|
||||
});
|
||||
} else {
|
||||
|
||||
@@ -111,6 +111,14 @@ export async function resolveCommandsSystemPromptBundle(
|
||||
defaultModel: defaultModelLabel,
|
||||
},
|
||||
});
|
||||
const fullAccessState = resolveEmbeddedFullAccessState({
|
||||
sandboxEnabled: true,
|
||||
execElevated: {
|
||||
enabled: params.elevated.enabled,
|
||||
allowed: params.elevated.allowed,
|
||||
defaultLevel: (params.resolvedElevatedLevel ?? "off") as "on" | "off" | "ask" | "full",
|
||||
},
|
||||
});
|
||||
const sandboxInfo = sandboxRuntime.sandboxed
|
||||
? {
|
||||
enabled: true,
|
||||
@@ -119,18 +127,10 @@ export async function resolveCommandsSystemPromptBundle(
|
||||
elevated: {
|
||||
allowed: params.elevated.allowed,
|
||||
defaultLevel: (params.resolvedElevatedLevel ?? "off") as "on" | "off" | "ask" | "full",
|
||||
fullAccessAvailable: resolveEmbeddedFullAccessState({
|
||||
sandboxEnabled: true,
|
||||
execElevated: {
|
||||
enabled: params.elevated.enabled,
|
||||
allowed: params.elevated.allowed,
|
||||
defaultLevel: (params.resolvedElevatedLevel ?? "off") as
|
||||
| "on"
|
||||
| "off"
|
||||
| "ask"
|
||||
| "full",
|
||||
},
|
||||
}).available,
|
||||
fullAccessAvailable: fullAccessState.available,
|
||||
...(fullAccessState.blockedReason
|
||||
? { fullAccessBlockedReason: fullAccessState.blockedReason }
|
||||
: {}),
|
||||
},
|
||||
}
|
||||
: { enabled: false };
|
||||
|
||||
@@ -228,7 +228,7 @@ export async function runPreparedReply(
|
||||
const fullAccessState = resolveEmbeddedFullAccessState({
|
||||
sandboxEnabled: resolveSandboxRuntimeStatus({
|
||||
cfg,
|
||||
sessionKey: ctx.SessionKey,
|
||||
sessionKey,
|
||||
}).sandboxed,
|
||||
execElevated: {
|
||||
enabled: elevatedEnabled,
|
||||
|
||||
Reference in New Issue
Block a user