Stop logging ACP non-JSON stdout previews
This commit is contained in:
@@ -148,9 +148,6 @@ export function migrateProviderSettings(
|
|||||||
const nextEntry: ProviderOverride = {};
|
const nextEntry: ProviderOverride = {};
|
||||||
const command = parsedOld.data.command;
|
const command = parsedOld.data.command;
|
||||||
if (command?.mode === "append") {
|
if (command?.mode === "append") {
|
||||||
console.warn(
|
|
||||||
`[Config] Skipping legacy agents.providers.${providerId}.command append mode during provider override migration because it cannot be auto-migrated.`,
|
|
||||||
);
|
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
if (command?.mode === "replace") {
|
if (command?.mode === "replace") {
|
||||||
|
|||||||
@@ -72,10 +72,10 @@ describe("createLoggedNdJsonStream", () => {
|
|||||||
expect(logger.warn).toHaveBeenCalledWith(
|
expect(logger.warn).toHaveBeenCalledWith(
|
||||||
expect.objectContaining({
|
expect.objectContaining({
|
||||||
provider: "gemini",
|
provider: "gemini",
|
||||||
linePreview: "Please visit the following URL to authorize the application:",
|
|
||||||
}),
|
}),
|
||||||
"ACP agent emitted non-JSON stdout; ignoring line",
|
"ACP agent emitted non-JSON stdout; ignoring line",
|
||||||
);
|
);
|
||||||
|
expect(logger.warn.mock.calls[0]?.[0]).not.toHaveProperty("linePreview");
|
||||||
expect(consoleError).not.toHaveBeenCalled();
|
expect(consoleError).not.toHaveBeenCalled();
|
||||||
|
|
||||||
await writer.close();
|
await writer.close();
|
||||||
|
|||||||
@@ -153,8 +153,6 @@ export function createLoggedNdJsonStream(
|
|||||||
{
|
{
|
||||||
err: error,
|
err: error,
|
||||||
provider: options.provider,
|
provider: options.provider,
|
||||||
linePreview:
|
|
||||||
trimmedLine.length > 500 ? `${trimmedLine.slice(0, 500)}...` : trimmedLine,
|
|
||||||
},
|
},
|
||||||
"ACP agent emitted non-JSON stdout; ignoring line",
|
"ACP agent emitted non-JSON stdout; ignoring line",
|
||||||
);
|
);
|
||||||
|
|||||||
Reference in New Issue
Block a user