fix: ensure the load_data event returns a Promise (#82)

Col0ring · web-flow · commit ca7519f439e0 · 2025-07-02T15:59:50.000+08:00
* fix: ensure the load_data event returns a Proimse

* chore: update deps

* fix: changeset

* fix: ensure the same key and value in the tree data
diff --git a/.changeset/fair-pianos-rule.md b/.changeset/fair-pianos-rule.md
@@ -0,0 +1,7 @@
+---
+'@modelscope-studio/antd': patch
+'@modelscope-studio/frontend': patch
+'modelscope_studio': patch
+---
+
+fix: ensure the same key and value in the tree data
diff --git a/.changeset/hip-apes-bathe.md b/.changeset/hip-apes-bathe.md
@@ -0,0 +1,14 @@
+---
+'@modelscope-studio/legacy-waterfall-gallery': patch
+'@modelscope-studio/legacy-multimodal-input': patch
+'@modelscope-studio/legacy-markdown': patch
+'@modelscope-studio/legacy-compiled': patch
+'@modelscope-studio/legacy-flow': patch
+'@modelscope-studio/lint-config': patch
+'@modelscope-studio/changelog': patch
+'@modelscope-studio/antd': patch
+'@modelscope-studio/frontend': patch
+'modelscope_studio': patch
+---
+
+chore: update deps
diff --git a/.changeset/yellow-tables-dance.md b/.changeset/yellow-tables-dance.md
@@ -0,0 +1,5 @@
+---
+'@modelscope-studio/antd': patch
+---
+
+fix: ensure the load_data event returns a Promise
diff --git a/config/changelog/package.json b/config/changelog/package.json
@@ -25,7 +25,7 @@
   },
   "devDependencies": {
     "@changesets/types": "^6.1.0",
-    "@types/node": "^24.0.0",
+    "@types/node": "^24.0.10",
     "tsup": "^8.5.0"
   }
 }
diff --git a/config/lint-config/package.json b/config/lint-config/package.json
@@ -16,21 +16,21 @@
     }
   },
   "dependencies": {
-    "@eslint/compat": "^1.2.9",
-    "@eslint/js": "^9.28.0",
-    "@typescript-eslint/parser": "^8.34.0",
+    "@eslint/compat": "^1.3.1",
+    "@eslint/js": "^9.30.1",
+    "@typescript-eslint/parser": "^8.35.1",
     "eslint-config-prettier": "^10.1.5",
-    "eslint-import-resolver-typescript": "^4.4.3",
-    "eslint-plugin-import": "^2.31.0",
+    "eslint-import-resolver-typescript": "^4.4.4",
+    "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-jsx-a11y": "^6.10.2",
-    "eslint-plugin-prettier": "5.4.1",
+    "eslint-plugin-prettier": "5.5.1",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^5.2.0",
     "eslint-plugin-react-refresh": "^0.4.20",
     "eslint-plugin-simple-import-sort": "^12.1.0",
-    "eslint-plugin-svelte": "^3.9.2",
-    "globals": "^16.2.0",
-    "postcss": "^8.5.4",
+    "eslint-plugin-svelte": "^3.10.1",
+    "globals": "^16.3.0",
+    "postcss": "^8.5.6",
     "postcss-less": "^6.0.0",
     "stylelint-config-ali": "^2.1.2",
     "stylelint-config-rational-order": "^0.1.2",
@@ -39,7 +39,7 @@
     "stylelint-order": "^7.0.0",
     "stylelint-prettier": "^5.0.3",
     "svelte-eslint-parser": "^1.2.0",
-    "typescript-eslint": "^8.34.0"
+    "typescript-eslint": "^8.35.1"
   },
   "devDependencies": {
     "@types/eslint": "^9.6.1",
diff --git a/frontend/antd/tree-select/tree-select.tsx b/frontend/antd/tree-select/tree-select.tsx
@@ -63,12 +63,28 @@ export const TreeSelect = sveltify<
       const componentProps: TreeSelectProps = useMemo(() => {
         return {
           ...props,
-          loadData: onLoadData,
+          // eslint-disable-next-line require-await
+          loadData: async (...args) => onLoadData?.(...args),
           treeData:
             treeData ||
             renderItems<NonNullable<TreeSelectProps['treeData']>[number]>(
               resolvedSlotItems,
-              { clone: true }
+              {
+                clone: true,
+                itemPropsTransformer: (itemProps) => {
+                  if (
+                    itemProps.value &&
+                    itemProps.key &&
+                    itemProps.value !== itemProps.key
+                  ) {
+                    return {
+                      ...itemProps,
+                      key: undefined,
+                    };
+                  }
+                  return itemProps;
+                },
+              }
             ),
           dropdownRender: slots.dropdownRender
             ? renderParamsSlot({ slots, setSlotParams, key: 'dropdownRender' })
@@ -131,6 +147,7 @@ export const TreeSelect = sveltify<
         treeData,
         treeTitleRenderFunction,
       ]);
+
       return (
         <>
           <div style={{ display: 'none' }}>{children}</div>
diff --git a/frontend/antd/tree/tree.tsx b/frontend/antd/tree/tree.tsx
@@ -65,6 +65,19 @@ export const Tree = sveltify<
               resolvedSlotItems,
               {
                 clone: true,
+                itemPropsTransformer: (itemProps) => {
+                  if (
+                    itemProps.value &&
+                    itemProps.key &&
+                    itemProps.value !== itemProps.key
+                  ) {
+                    return {
+                      ...itemProps,
+                      key: undefined,
+                    };
+                  }
+                  return itemProps;
+                },
               }
             ),
           showLine: slots['showLine.showLeafIcon']
@@ -101,7 +114,8 @@ export const Tree = sveltify<
                   nodeDraggable: draggableNodeDraggableFunction,
                 }
               : draggableFunction || draggable,
-          loadData: onLoadData,
+          // eslint-disable-next-line require-await
+          loadData: async (...args: any[]) => onLoadData?.(...args),
         };
       }, [
         props,
diff --git a/frontend/legacy/Flow/Awaited.svelte b/frontend/legacy/Flow/Awaited.svelte
@@ -62,7 +62,7 @@
       };
     }
   }
-  $: background_props, update_background_props();
+  $: (background_props, update_background_props());
   // process schema
   $: _schema = {
     ...schema,
diff --git a/frontend/legacy/Markdown/Awaited.svelte b/frontend/legacy/Markdown/Awaited.svelte
@@ -43,7 +43,7 @@
       : src.replaceAll('/file=', replaceStr);
   };
   $: _value = redirect_src_url(value);
-  $: label, gradio.dispatch('change');
+  $: (label, gradio.dispatch('change'));
 </script>
 
 <Block
diff --git a/frontend/legacy/Markdown/shared/Markdown.svelte b/frontend/legacy/Markdown/shared/Markdown.svelte
@@ -32,7 +32,7 @@
     dispatch('custom', { tag, tag_index, value: v });
   }
 
-  $: value, dispatch('change');
+  $: (value, dispatch('change'));
 </script>
 
 <div
diff --git a/frontend/legacy/MultimodalInput/shared/Input.svelte b/frontend/legacy/MultimodalInput/shared/Input.svelte
@@ -30,7 +30,7 @@
   let previous_scroll_top = 0;
   let user_has_scrolled_up = false;
 
-  $: value, el && lines !== max_lines && resize({ target: el });
+  $: (value, el && lines !== max_lines && resize({ target: el }));
 
   $: if (value === null) value = '';
 
@@ -68,7 +68,7 @@
     }
     value_is_output = false;
   });
-  $: value, handle_change();
+  $: (value, handle_change());
 
   async function handle_copy(): Promise<void> {
     if ('clipboard' in navigator) {
diff --git a/frontend/legacy/WaterfallGallery/Index.svelte b/frontend/legacy/WaterfallGallery/Index.svelte
@@ -64,7 +64,7 @@
     gradio.dispatch('like', data);
   };
 
-  $: selected_index, dispatch('prop_change', { selected_index });
+  $: (selected_index, dispatch('prop_change', { selected_index }));
 </script>
 
 <Block
diff --git a/frontend/legacy/WaterfallGallery/shared/Gallery.svelte b/frontend/legacy/WaterfallGallery/shared/Gallery.svelte
@@ -256,7 +256,7 @@
     });
   }
   $: if (waterfall_grid_el) {
-    cols, createWaterfall();
+    (cols, createWaterfall());
   }
 
   onDestroy(() => {
diff --git a/frontend/legacy/compiled/package.json b/frontend/legacy/compiled/package.json
@@ -14,7 +14,7 @@
     "@subscribe-kit/core": "^2.1.1",
     "@subscribe-kit/react": "^2.1.1",
     "@ungap/structured-clone": "^1.3.0",
-    "@xyflow/react": "12.6.4",
+    "@xyflow/react": "12.8.1",
     "ajv": "^8.17.1",
     "ajv-i18n": "^4.2.0",
     "deepmerge": "^4.3.1",
diff --git a/frontend/package.json b/frontend/package.json
@@ -9,7 +9,7 @@
     "@ant-design/cssinjs": "^1.23.0",
     "@ant-design/icons": "^6.0.0",
     "@ant-design/x": "^1.4.0",
-    "@babel/standalone": "^7.27.6",
+    "@babel/standalone": "^7.27.7",
     "@gradio/atoms": "0.7.4",
     "@gradio/button": "^0.2.43",
     "@gradio/client": "^1.14.2",
@@ -19,28 +19,28 @@
     "@gradio/upload": "0.11.2",
     "@gradio/utils": "0.9.0",
     "amuchina": "^1.0.12",
-    "antd": "^5.26.0",
+    "antd": "^5.26.3",
     "classnames": "^2.5.1",
     "dayjs": "^1.11.12",
     "dequal": "^2.0.2",
     "github-slugger": "^2.0.0",
     "immer": "^10.1.1",
     "katex": "^0.16.22",
     "lodash-es": "^4.17.21",
-    "marked": "^15.0.12",
-    "marked-gfm-heading-id": "^4.1.1",
-    "marked-highlight": "^2.2.1",
-    "mermaid": "^11.6.0",
+    "marked": "^16.0.0",
+    "marked-gfm-heading-id": "^4.1.2",
+    "marked-highlight": "^2.2.2",
+    "mermaid": "^11.7.0",
     "path-browserify-esm": "^1.0.6",
     "prismjs": "^1.30.0",
     "react": "^18.3.1",
     "react-dom": "^18.3.1",
     "svelte": "^4.2.19",
     "svelte-i18n": "^4.0.1",
-    "wavesurfer.js": "^7.9.5"
+    "wavesurfer.js": "^7.9.9"
   },
   "devDependencies": {
-    "@babel/core": "^7.27.4",
+    "@babel/core": "^7.27.7",
     "@types/babel__core": "^7.20.5",
     "@types/babel__standalone": "^7.1.9",
     "@types/katex": "^0.16.7",
@@ -53,7 +53,7 @@
     "fast-glob": "^3.3.3",
     "less": "^4.3.0",
     "typescript-json-schema": "^0.65.1",
-    "vite": "^6.3.5"
+    "vite": "^7.0.0"
   },
   "main_changeset": true
 }
diff --git a/frontend/utils/renderItems.tsx b/frontend/utils/renderItems.tsx
@@ -12,6 +12,7 @@ export function renderItems<R>(
     fallback?: (item: any) => R;
     clone?: boolean;
     forceClone?: boolean;
+    itemPropsTransformer?: (props: Record<string, any>) => Record<string, any>;
   },
   key?: React.Key
 ): undefined | R[] {
@@ -26,10 +27,15 @@ export function renderItems<R>(
       }
       return item;
     }
-    const result = {
-      ...item.props,
-      key: item.props?.key ?? (key ? `${key}-${i}` : `${i}`),
-    };
+    const result = options?.itemPropsTransformer
+      ? options?.itemPropsTransformer({
+          ...item.props,
+          key: item.props?.key ?? (key ? `${key}-${i}` : `${i}`),
+        })
+      : {
+          ...item.props,
+          key: item.props?.key ?? (key ? `${key}-${i}` : `${i}`),
+        };
 
     let current = result;
     Object.keys(item.slots).forEach((slotKey) => {
diff --git a/package.json b/package.json
@@ -27,26 +27,26 @@
     "@actions/core": "^1.11.1",
     "@actions/exec": "^1.1.1",
     "@actions/github": "^6.0.1",
-    "@changesets/cli": "^2.29.4",
+    "@changesets/cli": "^2.29.5",
     "@commitlint/cli": "^19.8.1",
     "@commitlint/config-conventional": "^19.8.1",
     "@manypkg/get-packages": "^3.0.0",
     "@modelscope-studio/changelog": "workspace:*",
     "@modelscope-studio/lint-config": "workspace:*",
-    "eslint": "^9.28.0",
+    "eslint": "^9.30.1",
     "husky": "^9.1.7",
-    "lint-staged": "^16.1.0",
+    "lint-staged": "^16.1.2",
     "node-fetch": "^3.3.2",
     "npm-run-all": "^4.1.5",
-    "prettier": "^3.5.3",
-    "prettier-plugin-packagejson": "^2.5.15",
+    "prettier": "^3.6.2",
+    "prettier-plugin-packagejson": "^2.5.17",
     "prettier-plugin-svelte": "^3.4.0",
     "rimraf": "^6.0.1",
-    "stylelint": "^16.20.0",
+    "stylelint": "^16.21.0",
     "svelte": "^4.2.19",
-    "svelte-check": "^4.2.1",
+    "svelte-check": "^4.2.2",
     "tslib": "^2.8.1",
-    "tsx": "^4.20.0",
+    "tsx": "^4.20.3",
     "typescript": "^5.8.3"
   },
   "main_changeset": true
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +'@modelscope-studio/antd': patch
 +---
++
 +fix: ensure the load_data event returns a Promise
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@`
`25`	`25`	`},`
`26`	`26`	`"devDependencies": {`
`27`	`27`	`"@changesets/types": "^6.1.0",`
`28`		`- "@types/node": "^24.0.0",`
	`28`	`+ "@types/node": "^24.0.10",`
`29`	`29`	`"tsup": "^8.5.0"`
`30`	`30`	`}`
`31`	`31`	`}`
Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@`
`62`	`62`	`};`
`63`	`63`	`}`
`64`	`64`	`}`
`65`		`- $: background_props, update_background_props();`
	`65`	`+ $: (background_props, update_background_props());`
`66`	`66`	`// process schema`
`67`	`67`	`$: _schema = {`
`68`	`68`	`...schema,`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@`
`32`	`32`	`dispatch('custom', { tag, tag_index, value: v });`
`33`	`33`	`}`
`34`	`34`
`35`		`- $: value, dispatch('change');`
	`35`	`+ $: (value, dispatch('change'));`
`36`	`36`	`</script>`
`37`	`37`
`38`	`38`	`<div`