Skip to content

Commit f026014

Browse files
committed
fix: keep gateway overlay building on main
1 parent da6669e commit f026014

3 files changed

Lines changed: 5 additions & 50 deletions

File tree

src/agents/harness/hook-context.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import type { DiagnosticTraceContext } from "../../infra/diagnostic-trace-context.js";
12
import type {
23
PluginHookAgentContext,
34
PluginHookContextWindowSource,
@@ -6,6 +7,7 @@ import type {
67
export type AgentHarnessHookContext = {
78
runId: string;
89
jobId?: string;
10+
trace?: DiagnosticTraceContext;
911
agentId?: string;
1012
sessionKey?: string;
1113
sessionId?: string;
@@ -24,6 +26,7 @@ export function buildAgentHookContext(params: AgentHarnessHookContext): PluginHo
2426
return {
2527
runId: params.runId,
2628
...(params.jobId ? { jobId: params.jobId } : {}),
29+
...(params.trace ? { trace: params.trace } : {}),
2730
...(params.agentId ? { agentId: params.agentId } : {}),
2831
...(params.sessionKey ? { sessionKey: params.sessionKey } : {}),
2932
...(params.sessionId ? { sessionId: params.sessionId } : {}),

src/auto-reply/reply/agent-runner-memory.ts

Lines changed: 1 addition & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import { runWithModelFallback } from "../../agents/model-fallback.js";
1010
import { listLegacyRuntimeModelProviderAliases } from "../../agents/model-runtime-aliases.js";
1111
import { isCliProvider } from "../../agents/model-selection.js";
1212
import { resolveContextConfigProviderForRuntime } from "../../agents/openai-codex-routing.js";
13-
import { resolveExtraParams } from "../../agents/pi-embedded-runner/extra-params.js";
1413
import { resolveSandboxConfigForAgent, resolveSandboxRuntimeStatus } from "../../agents/sandbox.js";
1514
import {
1615
derivePromptTokens,
@@ -266,38 +265,6 @@ function resolveFollowupContextConfigProvider(params: {
266265
});
267266
}
268267

269-
function parsePositiveInteger(value: unknown): number | undefined {
270-
if (typeof value === "number" && Number.isFinite(value) && value > 0) {
271-
return Math.floor(value);
272-
}
273-
if (typeof value === "string") {
274-
const parsed = Number.parseInt(value, 10);
275-
return Number.isFinite(parsed) && parsed > 0 ? parsed : undefined;
276-
}
277-
return undefined;
278-
}
279-
280-
function resolveServerCompactionThresholdTokens(params: {
281-
cfg: OpenClawConfig;
282-
provider: string;
283-
modelId?: string;
284-
agentId?: string;
285-
}): number | undefined {
286-
if (!params.modelId?.trim()) {
287-
return undefined;
288-
}
289-
const extraParams = resolveExtraParams({
290-
cfg: params.cfg,
291-
provider: params.provider,
292-
modelId: params.modelId,
293-
agentId: params.agentId,
294-
});
295-
if (extraParams?.responsesServerCompaction !== true) {
296-
return undefined;
297-
}
298-
return parsePositiveInteger(extraParams.responsesCompactThreshold);
299-
}
300-
301268
function resolveVisibleMemoryFlushErrorPayloads(payloads?: ReplyPayload[]): ReplyPayload[] {
302269
return (payloads ?? []).filter(
303270
(payload) => payload.isError === true && isRenderablePayload(payload),
@@ -665,12 +632,6 @@ export async function runPreflightCompactionIfNeeded(params: {
665632
params.cfg.agents?.defaults?.compaction?.reserveTokensFloor ??
666633
20_000;
667634
const softThresholdTokens = memoryFlushPlan?.softThresholdTokens ?? 4_000;
668-
const serverCompactionThresholdTokens = resolveServerCompactionThresholdTokens({
669-
cfg: params.cfg,
670-
provider: params.followupRun.run.provider,
671-
modelId: params.followupRun.run.model ?? params.defaultModel,
672-
agentId: params.followupRun.run.agentId,
673-
});
674635
const freshPersistedTokens = resolveFreshSessionTotalTokens(entry);
675636
const persistedTotalTokens = entry.totalTokens;
676637
const hasPersistedTotalTokens =
@@ -732,9 +693,7 @@ export async function runPreflightCompactionIfNeeded(params: {
732693
? projectedTokenCount
733694
: undefined;
734695

735-
const threshold =
736-
serverCompactionThresholdTokens ??
737-
contextWindowTokens - reserveTokensFloor - softThresholdTokens;
696+
const threshold = contextWindowTokens - reserveTokensFloor - softThresholdTokens;
738697
logVerbose(
739698
`preflightCompaction check: sessionKey=${params.sessionKey} ` +
740699
`tokenCount=${tokenCountForCompaction ?? freshPersistedTokens ?? "undefined"} ` +
@@ -743,7 +702,6 @@ export async function runPreflightCompactionIfNeeded(params: {
743702
`persistedFresh=${entry?.totalTokensFresh === true} ` +
744703
`transcriptPromptTokens=${transcriptPromptTokens ?? "undefined"} ` +
745704
`promptTokensEst=${promptTokenEstimate ?? "undefined"} ` +
746-
`serverCompactionThreshold=${serverCompactionThresholdTokens ?? "undefined"} ` +
747705
`activeTranscriptBytes=${activeTranscriptBytes ?? "undefined"} ` +
748706
`maxActiveTranscriptBytes=${maxActiveTranscriptBytes ?? "undefined"} ` +
749707
`sizeTrigger=${shouldCompactByTranscriptBytes}`,
@@ -755,7 +713,6 @@ export async function runPreflightCompactionIfNeeded(params: {
755713
contextWindowTokens,
756714
reserveTokensFloor,
757715
softThresholdTokens,
758-
thresholdTokens: serverCompactionThresholdTokens,
759716
});
760717
const shouldCompact = shouldCompactByTokens || shouldCompactByTranscriptBytes;
761718
if (!shouldCompact) {
@@ -767,7 +724,6 @@ export async function runPreflightCompactionIfNeeded(params: {
767724
`preflightCompaction triggered: sessionKey=${params.sessionKey} ` +
768725
`tokenCount=${tokenCountForCompaction ?? freshPersistedTokens ?? "undefined"} ` +
769726
`threshold=${threshold} trigger=${compactionTrigger} ` +
770-
`serverCompactionThreshold=${serverCompactionThresholdTokens ?? "undefined"} ` +
771727
`activeTranscriptBytes=${activeTranscriptBytes ?? "undefined"} ` +
772728
`maxActiveTranscriptBytes=${maxActiveTranscriptBytes ?? "undefined"}`,
773729
);

src/auto-reply/reply/memory-flush.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ function resolveMemoryFlushGateState<
4444
contextWindowTokens: number;
4545
reserveTokensFloor: number;
4646
softThresholdTokens: number;
47-
thresholdTokens?: number;
4847
}): { entry: TEntry; totalTokens: number; threshold: number } | null {
4948
if (!params.entry) {
5049
return null;
@@ -59,9 +58,7 @@ function resolveMemoryFlushGateState<
5958
const contextWindow = Math.max(1, Math.floor(params.contextWindowTokens));
6059
const reserveTokens = Math.max(0, Math.floor(params.reserveTokensFloor));
6160
const softThreshold = Math.max(0, Math.floor(params.softThresholdTokens));
62-
const configuredThreshold = resolvePositiveTokenCount(params.thresholdTokens);
63-
const threshold =
64-
configuredThreshold ?? Math.max(0, contextWindow - reserveTokens - softThreshold);
61+
const threshold = Math.max(0, contextWindow - reserveTokens - softThreshold);
6562
if (threshold <= 0) {
6663
return null;
6764
}
@@ -107,7 +104,6 @@ export function shouldRunPreflightCompaction(params: {
107104
contextWindowTokens: number;
108105
reserveTokensFloor: number;
109106
softThresholdTokens: number;
110-
thresholdTokens?: number;
111107
}): boolean {
112108
const state = resolveMemoryFlushGateState(params);
113109
return Boolean(state && state.totalTokens >= state.threshold);

0 commit comments

Comments
 (0)