fix(cli): keep build/lint green
This commit is contained in:
@@ -278,8 +278,7 @@ function parseCliJsonl(
|
|||||||
}
|
}
|
||||||
const item = isRecord(parsed.item) ? parsed.item : null;
|
const item = isRecord(parsed.item) ? parsed.item : null;
|
||||||
if (item && typeof item.text === "string") {
|
if (item && typeof item.text === "string") {
|
||||||
const type =
|
const type = typeof item.type === "string" ? item.type.toLowerCase() : "";
|
||||||
typeof item.type === "string" ? item.type.toLowerCase() : "";
|
|
||||||
if (!type || type.includes("message")) {
|
if (!type || type.includes("message")) {
|
||||||
texts.push(item.text);
|
texts.push(item.text);
|
||||||
}
|
}
|
||||||
@@ -633,8 +632,9 @@ export async function runCliAgent(params: {
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
const outputMode =
|
const outputMode = useResume
|
||||||
useResume ? backend.resumeOutput ?? backend.output : backend.output;
|
? (backend.resumeOutput ?? backend.output)
|
||||||
|
: backend.output;
|
||||||
|
|
||||||
if (outputMode === "text") {
|
if (outputMode === "text") {
|
||||||
return { text: stdout, sessionId: undefined };
|
return { text: stdout, sessionId: undefined };
|
||||||
@@ -656,7 +656,7 @@ export async function runCliAgent(params: {
|
|||||||
meta: {
|
meta: {
|
||||||
durationMs: Date.now() - started,
|
durationMs: Date.now() - started,
|
||||||
agentMeta: {
|
agentMeta: {
|
||||||
sessionId: output.sessionId ?? sessionIdSent,
|
sessionId: output.sessionId ?? sessionIdSent ?? params.sessionId,
|
||||||
provider: params.provider,
|
provider: params.provider,
|
||||||
model: modelId,
|
model: modelId,
|
||||||
usage: output.usage,
|
usage: output.usage,
|
||||||
|
|||||||
@@ -378,17 +378,13 @@ describeLive("gateway live (cli backend)", () => {
|
|||||||
{ expectFinal: true },
|
{ expectFinal: true },
|
||||||
);
|
);
|
||||||
if (resumePayload?.status !== "ok") {
|
if (resumePayload?.status !== "ok") {
|
||||||
throw new Error(
|
throw new Error(`resume status=${String(resumePayload?.status)}`);
|
||||||
`resume status=${String(resumePayload?.status)}`,
|
|
||||||
);
|
|
||||||
}
|
}
|
||||||
const resumeText = extractPayloadText(resumePayload?.result);
|
const resumeText = extractPayloadText(resumePayload?.result);
|
||||||
if (providerId === "codex-cli") {
|
if (providerId === "codex-cli") {
|
||||||
expect(resumeText).toContain(`CLI-RESUME-${resumeNonce}`);
|
expect(resumeText).toContain(`CLI-RESUME-${resumeNonce}`);
|
||||||
} else {
|
} else {
|
||||||
expect(resumeText).toContain(
|
expect(resumeText).toContain(`CLI backend RESUME OK ${resumeNonce}.`);
|
||||||
`CLI backend RESUME OK ${resumeNonce}.`,
|
|
||||||
);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user