fix: layer editor UX and review follow-ups · Comfy-Org/ComfyUI_frontend@eb4b556

Verify lint and format: src/composables/useCopySystemInfo.test.ts#L69

src/composables/useCopySystemInfo.test.ts:69:5: vi.resetAllMocks() is redundant in a Vitest hook because Vitest performs this cleanup automatically.

Verify lint and format: src/platform/workflow/templates/utils/templateRanking.test.ts#L24

src/platform/workflow/templates/utils/templateRanking.test.ts:24:7: Expect takes at most 1 argument

Verify lint and format: src/base/common/downloadUtil.test.ts#L32

src/base/common/downloadUtil.test.ts:32:26: Install vi.spyOn() in beforeEach or a test because automatic Vitest cleanup removes earlier mock installations before assertions run.

Verify lint and format: src/base/common/downloadUtil.test.ts#L29

src/base/common/downloadUtil.test.ts:29:26: Install vi.spyOn() in beforeEach or a test because automatic Vitest cleanup removes earlier mock installations before assertions run.

Verify lint and format: src/renderer/extensions/compositor/components/WidgetCompositor.test.ts#L76

src/renderer/extensions/compositor/components/WidgetCompositor.test.ts:76:5: vi.clearAllMocks() is redundant in a Vitest hook because Vitest performs this cleanup automatically.

Verify lint and format: src/utils/videoMetadataUtil.test.ts#L107

src/utils/videoMetadataUtil.test.ts:107:5: vi.unstubAllGlobals() is redundant in a Vitest hook because Vitest performs this cleanup automatically.

Verify lint and format: src/renderer/extensions/layerEditor/engine/editor/selectionMath.test.ts#L223

src/renderer/extensions/layerEditor/engine/editor/selectionMath.test.ts:223:19: Unexpected erasing operation. This expression will always evaluate to zero.

Verify lint and format: src/renderer/extensions/layerEditor/engine/editor/selectionMath.test.ts#L222

src/renderer/extensions/layerEditor/engine/editor/selectionMath.test.ts:222:19: Unexpected erasing operation. This expression will always evaluate to zero.

Verify lint and format: src/renderer/extensions/layerEditor/composables/useLayerEditorSession.test.ts#L153

src/renderer/extensions/layerEditor/composables/useLayerEditorSession.test.ts:153:3: vi.unstubAllGlobals() is redundant in a Vitest hook because Vitest performs this cleanup automatically.

Verify lint and format: src/vitestSetup.test.ts#L13

src/vitestSetup.test.ts:13:5: vi.unstubAllGlobals() is redundant in a Vitest hook because Vitest performs this cleanup automatically.

Run ESLint with auto-fix: src/composables/useCopySystemInfo.test.ts#L69

src/composables/useCopySystemInfo.test.ts:69:5: vi.resetAllMocks() is redundant in a Vitest hook because Vitest performs this cleanup automatically.

Run ESLint with auto-fix: src/platform/workflow/templates/utils/templateRanking.test.ts#L24

src/platform/workflow/templates/utils/templateRanking.test.ts:24:7: Expect takes at most 1 argument

Run ESLint with auto-fix: src/utils/videoMetadataUtil.test.ts#L107

src/utils/videoMetadataUtil.test.ts:107:5: vi.unstubAllGlobals() is redundant in a Vitest hook because Vitest performs this cleanup automatically.

Run ESLint with auto-fix: src/base/common/downloadUtil.test.ts#L32

src/base/common/downloadUtil.test.ts:32:26: Install vi.spyOn() in beforeEach or a test because automatic Vitest cleanup removes earlier mock installations before assertions run.

Run ESLint with auto-fix: src/base/common/downloadUtil.test.ts#L29

src/base/common/downloadUtil.test.ts:29:26: Install vi.spyOn() in beforeEach or a test because automatic Vitest cleanup removes earlier mock installations before assertions run.

Run ESLint with auto-fix: src/renderer/extensions/compositor/components/WidgetCompositor.test.ts#L76

src/renderer/extensions/compositor/components/WidgetCompositor.test.ts:76:5: vi.clearAllMocks() is redundant in a Vitest hook because Vitest performs this cleanup automatically.

Run ESLint with auto-fix: src/renderer/extensions/layerEditor/engine/editor/selectionMath.test.ts#L223

src/renderer/extensions/layerEditor/engine/editor/selectionMath.test.ts:223:19: Unexpected erasing operation. This expression will always evaluate to zero.

Run ESLint with auto-fix: src/renderer/extensions/layerEditor/engine/editor/selectionMath.test.ts#L222

src/renderer/extensions/layerEditor/engine/editor/selectionMath.test.ts:222:19: Unexpected erasing operation. This expression will always evaluate to zero.

Run ESLint with auto-fix: src/vitestSetup.test.ts#L13

src/vitestSetup.test.ts:13:5: vi.unstubAllGlobals() is redundant in a Vitest hook because Vitest performs this cleanup automatically.

Run ESLint with auto-fix: src/renderer/extensions/layerEditor/composables/useLayerEditorSession.test.ts#L153

src/renderer/extensions/layerEditor/composables/useLayerEditorSession.test.ts:153:3: vi.unstubAllGlobals() is redundant in a Vitest hook because Vitest performs this cleanup automatically.

Navigation Menu

Uh oh!

fix: layer editor UX and review follow-ups #28247

lint-and-format

fix: layer editor UX and review follow-ups #28247

Uh oh!

Annotations

lint-and-format

Re-running jobs...