From c75adb8154e78ee52a24ebc7b3845d623cab30ae Mon Sep 17 00:00:00 2001
From: yyh <yuanyouhuilyz@gmail.com>
Date: Tue, 24 Mar 2026 20:06:56 +0800
Subject: [PATCH] fix: type error

---
 .../score-slider/base-slider/index.tsx           | 11 ++++++++---
 .../nodes/list-operator/use-config.helpers.ts    | 14 ++++++++------
 .../use-single-run-form-params.helpers.spec.ts   |  2 +-
 .../__tests__/use-single-run-form-params.spec.ts |  2 +-
 .../workflow/nodes/loop/use-config.helpers.ts    | 16 +++++++++-------
 .../panel/env-panel/__tests__/index.spec.tsx     | 10 ++++++++--
 6 files changed, 35 insertions(+), 20 deletions(-)

diff --git a/web/app/components/base/features/new-feature-panel/annotation-reply/score-slider/base-slider/index.tsx b/web/app/components/base/features/new-feature-panel/annotation-reply/score-slider/base-slider/index.tsx
index 509426c08e..78e414457c 100644
--- a/web/app/components/base/features/new-feature-panel/annotation-reply/score-slider/base-slider/index.tsx
+++ b/web/app/components/base/features/new-feature-panel/annotation-reply/score-slider/base-slider/index.tsx
@@ -16,7 +16,7 @@ const Slider: React.FC<ISliderProps> = ({ className, max, min, step, value, disa
   return (
     <ReactSlider
       disabled={disabled}
-      value={isNaN(value) ? 0 : value}
+      value={Number.isNaN(value) ? 0 : value}
       min={min || 0}
       max={max || 100}
       step={step || 1}
@@ -24,8 +24,13 @@ const Slider: React.FC<ISliderProps> = ({ className, max, min, step, value, disa
       thumbClassName={cn(s['slider-thumb'], 'top-[-7px] h-[18px] w-2 cursor-pointer rounded-[36px] border !border-black/8 bg-white shadow-md')}
       trackClassName={s['slider-track']}
       onChange={onChange}
-      renderThumb={(props, state) => (
-        <div {...props}>
+      renderThumb={({ ref, ...props }, state) => (
+        <div
+          {...props}
+          ref={(node) => {
+            ref(node)
+          }}
+        >
           <div className="relative h-full w-full">
             <div className="absolute left-[50%] top-[-16px] translate-x-[-50%] text-text-primary system-sm-semibold">
               {(state.valueNow / 100).toFixed(2)}
diff --git a/web/app/components/workflow/nodes/list-operator/use-config.helpers.ts b/web/app/components/workflow/nodes/list-operator/use-config.helpers.ts
index 114a1f0ac5..7d213033f7 100644
--- a/web/app/components/workflow/nodes/list-operator/use-config.helpers.ts
+++ b/web/app/components/workflow/nodes/list-operator/use-config.helpers.ts
@@ -5,6 +5,13 @@ import { VarType as WorkflowVarType } from '../../types'
 import { getOperators } from '../if-else/utils'
 import { OrderBy } from './types'
 
+const filterableVarTypes: readonly VarType[] = [
+  WorkflowVarType.arrayNumber,
+  WorkflowVarType.arrayString,
+  WorkflowVarType.arrayBoolean,
+  WorkflowVarType.arrayFile,
+]
+
 export const getItemVarType = (varType?: VarType) => {
   switch (varType) {
     case WorkflowVarType.arrayNumber:
@@ -33,12 +40,7 @@ export const getItemVarTypeShowName = (itemVarType?: VarType, hasVariable?: bool
 export const supportsSubVariable = (varType?: VarType) => varType === WorkflowVarType.arrayFile
 
 export const canFilterVariable = (varPayload: Var) => {
-  return [
-    WorkflowVarType.arrayNumber,
-    WorkflowVarType.arrayString,
-    WorkflowVarType.arrayBoolean,
-    WorkflowVarType.arrayFile,
-  ].includes(varPayload.type)
+  return filterableVarTypes.includes(varPayload.type)
 }
 
 export const buildFilterCondition = ({
diff --git a/web/app/components/workflow/nodes/loop/__tests__/use-single-run-form-params.helpers.spec.ts b/web/app/components/workflow/nodes/loop/__tests__/use-single-run-form-params.helpers.spec.ts
index 7d8438b07c..21b5b1f5b2 100644
--- a/web/app/components/workflow/nodes/loop/__tests__/use-single-run-form-params.helpers.spec.ts
+++ b/web/app/components/workflow/nodes/loop/__tests__/use-single-run-form-params.helpers.spec.ts
@@ -24,7 +24,7 @@ vi.mock('../../_base/components/variable/utils', () => ({
   isSystemVar: (...args: unknown[]) => mockIsSystemVar(...args),
 }))
 
-const createNode = (id: string, title: string, type = BlockEnum.Tool): Node => ({
+const createNode = (id: string, title: string, type: BlockEnum = BlockEnum.Tool): Node => ({
   id,
   position: { x: 0, y: 0 },
   data: {
diff --git a/web/app/components/workflow/nodes/loop/__tests__/use-single-run-form-params.spec.ts b/web/app/components/workflow/nodes/loop/__tests__/use-single-run-form-params.spec.ts
index 7ff89492cd..3a952c4843 100644
--- a/web/app/components/workflow/nodes/loop/__tests__/use-single-run-form-params.spec.ts
+++ b/web/app/components/workflow/nodes/loop/__tests__/use-single-run-form-params.spec.ts
@@ -43,7 +43,7 @@ const createLoopNode = (overrides: Partial<LoopNodeType> = {}): LoopNodeType =>
   ...overrides,
 })
 
-const createVariableNode = (id: string, title: string, type = BlockEnum.Tool): Node => ({
+const createVariableNode = (id: string, title: string, type: BlockEnum = BlockEnum.Tool): Node => ({
   id,
   position: { x: 0, y: 0 },
   data: {
diff --git a/web/app/components/workflow/nodes/loop/use-config.helpers.ts b/web/app/components/workflow/nodes/loop/use-config.helpers.ts
index edeca0b766..677244e299 100644
--- a/web/app/components/workflow/nodes/loop/use-config.helpers.ts
+++ b/web/app/components/workflow/nodes/loop/use-config.helpers.ts
@@ -6,14 +6,16 @@ import { ValueType, VarType } from '../../types'
 import { LogicalOperator } from './types'
 import { getOperators } from './utils'
 
+const loopInputVarTypes: readonly VarType[] = [
+  VarType.array,
+  VarType.arrayString,
+  VarType.arrayNumber,
+  VarType.arrayObject,
+  VarType.arrayFile,
+]
+
 export const canUseAsLoopInput = (variable: Var) => {
-  return [
-    VarType.array,
-    VarType.arrayString,
-    VarType.arrayNumber,
-    VarType.arrayObject,
-    VarType.arrayFile,
-  ].includes(variable.type)
+  return loopInputVarTypes.includes(variable.type)
 }
 
 export const updateErrorHandleMode = (
diff --git a/web/app/components/workflow/panel/env-panel/__tests__/index.spec.tsx b/web/app/components/workflow/panel/env-panel/__tests__/index.spec.tsx
index cc1499daaf..30ab7d8176 100644
--- a/web/app/components/workflow/panel/env-panel/__tests__/index.spec.tsx
+++ b/web/app/components/workflow/panel/env-panel/__tests__/index.spec.tsx
@@ -34,12 +34,18 @@ const {
   mockVariableTriggerState: {
     savePayload: undefined as EnvironmentVariable | undefined,
   },
-  mockUpdateEnvironmentVariables: vi.fn(() => Promise.resolve()),
+  mockUpdateEnvironmentVariables: vi.fn<({
+    appId,
+    environmentVariables,
+  }: {
+    appId: string
+    environmentVariables: EnvironmentVariable[]
+  }) => Promise<void>>(() => Promise.resolve()),
   mockSocketEmit: vi.fn(),
 }))
 
 vi.mock('@/service/workflow', () => ({
-  updateEnvironmentVariables: (...args: unknown[]) => mockUpdateEnvironmentVariables(...args),
+  updateEnvironmentVariables: (payload: { appId: string, environmentVariables: EnvironmentVariable[] }) => mockUpdateEnvironmentVariables(payload),
 }))
 
 vi.mock('@/app/components/workflow/collaboration/core/websocket-manager', () => ({