smashimo/oh-my-pi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

style: formatted merge resolutions with biome

Browse Source
This commit is contained in:
can1357
2026-07-18 20:14:30 +02:00
parent cbd6164e3a
commit 449e402a16
1 changed files with 26 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/coding-agent/test/selector-settings-side-effects.test.ts
+26 -26
View File
@@ -68,34 +68,34 @@ describe("selector setting side effects", () => {
});
for (const id of ["terminal.showImages", "showImages"]) {
for (const visible of [false, true]) {
it(`updates every image owner and rebuilds the transcript when ${id}=${visible}`, () => {
const setShowImages = vi.fn();
const setImagesVisible = vi.fn();
const clearInlineImages = vi.fn();
const resetDisplay = vi.fn();
const tool = Object.create(ToolExecutionComponent.prototype) as ToolExecutionComponent;
tool.setShowImages = setShowImages;
const assistant = Object.create(AssistantMessageComponent.prototype) as AssistantMessageComponent;
assistant.setImagesVisible = setImagesVisible;
const controller = new SelectorController({
chatContainer: { children: [tool, assistant] },
ui: { clearInlineImages, resetDisplay },
} as unknown as InteractiveModeContext);
for (const visible of [false, true]) {
it(`updates every image owner and rebuilds the transcript when ${id}=${visible}`, () => {
const setShowImages = vi.fn();
const setImagesVisible = vi.fn();
const clearInlineImages = vi.fn();
const resetDisplay = vi.fn();
const tool = Object.create(ToolExecutionComponent.prototype) as ToolExecutionComponent;
tool.setShowImages = setShowImages;
const assistant = Object.create(AssistantMessageComponent.prototype) as AssistantMessageComponent;
assistant.setImagesVisible = setImagesVisible;
const controller = new SelectorController({
chatContainer: { children: [tool, assistant] },
ui: { clearInlineImages, resetDisplay },
} as unknown as InteractiveModeContext);
controller.handleSettingChange(id, visible);
controller.handleSettingChange(id, visible);
expect(setShowImages).toHaveBeenCalledWith(visible);
expect(setImagesVisible).toHaveBeenCalledWith(visible);
expect(clearInlineImages).toHaveBeenCalledTimes(visible ? 0 : 1);
expect(resetDisplay).toHaveBeenCalledTimes(1);
if (!visible) {
expect(clearInlineImages.mock.invocationCallOrder[0]).toBeLessThan(
resetDisplay.mock.invocationCallOrder[0],
);
}
});
}
expect(setShowImages).toHaveBeenCalledWith(visible);
expect(setImagesVisible).toHaveBeenCalledWith(visible);
expect(clearInlineImages).toHaveBeenCalledTimes(visible ? 0 : 1);
expect(resetDisplay).toHaveBeenCalledTimes(1);
if (!visible) {
expect(clearInlineImages.mock.invocationCallOrder[0]).toBeLessThan(
resetDisplay.mock.invocationCallOrder[0],
);
}
});
}
}
it("clears stale default role thinking when auto is selected", async () => {