Skip to content

Commit 3221889

Browse files
committed
fix(doctor): pass allow-exec into workspace lint drift
1 parent 91c08cf commit 3221889

2 files changed

Lines changed: 15 additions & 1 deletion

File tree

src/flows/doctor-health-contributions.test.ts

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -703,6 +703,7 @@ describe("doctor health contributions", () => {
703703
const ctx = {
704704
cfg: { plugins: { entries: { codex: { enabled: true } } } },
705705
mode: "lint",
706+
allowExecSecretRefs: true,
706707
runtime: { log: vi.fn(), error: vi.fn(), exit: vi.fn() },
707708
} as const;
708709

@@ -722,6 +723,16 @@ describe("doctor health contributions", () => {
722723
expect(mocks.collectWorkspaceStatusHealthFindings).toHaveBeenCalledWith(ctx.cfg, {
723724
pluginVersionDrift,
724725
});
726+
expect(mocks.gatherDaemonStatus).toHaveBeenCalledWith({
727+
rpc: {
728+
timeout: "3000",
729+
json: true,
730+
},
731+
probe: true,
732+
requireRpc: false,
733+
deep: false,
734+
allowExecSecretRefs: true,
735+
});
725736
});
726737

727738
it("passes daemon-context plugin drift into the workspace status note", async () => {

src/flows/doctor-health-contributions.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1698,7 +1698,10 @@ export function resolveDoctorHealthContributions(): DoctorHealthContribution[] {
16981698
await import("../commands/doctor-workspace-status.js");
16991699
const pluginVersionDrift = await collectWorkspaceStatusPluginVersionDrift({
17001700
cfg: ctx.cfg,
1701-
options: { nonInteractive: true },
1701+
options: {
1702+
nonInteractive: true,
1703+
allowExec: ctx.allowExecSecretRefs === true,
1704+
},
17021705
});
17031706
return collectWorkspaceStatusHealthFindings(ctx.cfg, { pluginVersionDrift });
17041707
},

0 commit comments

Comments
 (0)