fix(status): keep plain json off security audit path

This commit is contained in:
Vincent Koc
2026-04-06 13:41:13 +01:00
parent a2b065b090
commit 35af6cc49c
2 changed files with 14 additions and 3 deletions

View File

@@ -105,8 +105,9 @@ export async function statusCommand(
await runStatusJsonCommand({
opts,
runtime,
includeSecurityAudit: true,
includeSecurityAudit: opts.all === true,
includePluginCompatibility: true,
suppressHealthErrors: true,
scanStatusJsonFast: async (scanOpts, runtimeForScan) =>
await loadStatusScanFastJsonModule().then(({ scanStatusJsonFast }) =>
scanStatusJsonFast(scanOpts, runtimeForScan),

View File

@@ -721,8 +721,7 @@ describe("statusCommand", () => {
expect(payload.sessions.recent[0].totalTokensFresh).toBe(true);
expect(payload.sessions.recent[0].remainingTokens).toBe(5000);
expect(payload.sessions.recent[0].flags).toContain("verbose:on");
expect(payload.securityAudit.summary.critical).toBe(1);
expect(payload.securityAudit.summary.warn).toBe(1);
expect(payload.securityAudit).toBeUndefined();
expect(payload.gatewayService.label).toBe("LaunchAgent");
expect(payload.nodeService.label).toBe("LaunchAgent");
expect(payload.pluginCompatibility).toEqual({
@@ -736,6 +735,17 @@ describe("statusCommand", () => {
byStatus: expect.objectContaining({ queued: 0, running: 0 }),
}),
);
expect(mocks.runSecurityAudit).not.toHaveBeenCalled();
});
it("includes security audit in JSON when all is requested", async () => {
mocks.hasPotentialConfiguredChannels.mockReturnValue(false);
await statusCommand({ json: true, all: true }, runtime as never);
const payload = JSON.parse(String(runtimeLogMock.mock.calls[0]?.[0]));
expect(payload.securityAudit.summary.critical).toBe(1);
expect(payload.securityAudit.summary.warn).toBe(1);
expect(mocks.runSecurityAudit).toHaveBeenCalledWith(
expect.objectContaining({
includeFilesystem: true,