Feat: If a query variable in a data manipulation operator is deleted, a warning message should be displayed to the user. #10427

web/src/components/jsonjoy-builder/components/schema-editor/json-schema-visualizer.tsx

@@ -48,7 +48,7 @@ const JsonSchemaVisualizer: FC<JsonSchemaVisualizerProps> = ({
 
     try {
       const parsedJson = JSON.parse(value);
-      if (onChange) {
+      if (onChange && typeof parsedJson !== 'number') {
         onChange(parsedJson);
       }
     } catch (_error) {

web/src/locales/en.ts

@@ -1646,6 +1646,7 @@ The variable aggregation node (originally the variable assignment node) is a cru
       beginInputTip:
         'By defining input parameters, this content can be accessed by other components in subsequent processes.',
       query: 'Query variables',
+      queryRequired: 'Query is required',
       queryTip: 'Select the variable you want to use',
       agent: 'Agent',
       addAgent: 'Add Agent',

web/src/locales/zh.ts

@@ -1549,6 +1549,7 @@ General：实体和关系提取提示来自 GitHub - microsoft/graphrag：基于
       task: '任务',
       beginInputTip: '通过定义输入参数，此内容可以被后续流程中的其他组件访问。',
       query: '查询变量',
+      queryRequired: '查询变量是必填项',
       queryTip: '选择您想要使用的变量',
       agent: '智能体',
       addAgent: '添加智能体',

web/src/pages/agent/canvas/node/data-operations-node.tsx

@@ -11,11 +11,12 @@ export function DataOperationsNode({
 }: NodeProps<BaseNode<DataOperationsFormSchemaType>>) {
   const { data } = props;
   const { t } = useTranslation();
+  const operations = data.form?.operations;
 
   return (
     <RagNode {...props}>
       <LabelCard>
-        {t(`flow.operationsOptions.${camelCase(data.form?.operations)}`)}
+        {operations && t(`flow.operationsOptions.${camelCase(operations)}`)}
       </LabelCard>
     </RagNode>
   );

web/src/pages/agent/form/components/dynamic-fom-header.tsx

@@ -7,17 +7,24 @@ export type FormListHeaderProps = {
   label: ReactNode;
   tooltip?: string;
   onClick?: () => void;
+  disabled?: boolean;
 };
 
 export function DynamicFormHeader({
   label,
   tooltip,
   onClick,
+  disabled = false,
 }: FormListHeaderProps) {
   return (
     <div className="flex items-center justify-between">
       <FormLabel tooltip={tooltip}>{label}</FormLabel>
-      <Button variant={'ghost'} type="button" onClick={onClick}>
+      <Button
+        variant={'ghost'}
+        type="button"
+        onClick={onClick}
+        disabled={disabled}
+      >
         <Plus />
       </Button>
     </div>

web/src/pages/agent/form/components/query-variable-list.tsx

@@ -2,6 +2,10 @@ import { Button } from '@/components/ui/button';
 import { X } from 'lucide-react';
 import { useFieldArray, useFormContext } from 'react-hook-form';
 import { JsonSchemaDataType } from '../../constant';
+import {
+  flatOptions,
+  useFilterQueryVariableOptionsByTypes,
+} from '../../hooks/use-get-begin-query';
 import { DynamicFormHeader, FormListHeaderProps } from './dynamic-fom-header';
 import { QueryVariable } from './query-variable';
 
@@ -16,6 +20,10 @@ export function QueryVariableList({
   const form = useFormContext();
   const name = 'query';
 
+  let options = useFilterQueryVariableOptionsByTypes(types);
+
+  const secondOptions = flatOptions(options);
+
   const { fields, remove, append } = useFieldArray({
     name: name,
     control: form.control,
@@ -26,14 +34,15 @@ export function QueryVariableList({
       <DynamicFormHeader
         label={label}
         tooltip={tooltip}
-        onClick={() => append({ input: '' })}
+        onClick={() => append({ input: secondOptions.at(0)?.value })}
+        disabled={!secondOptions.length}
       ></DynamicFormHeader>
       <div className="space-y-5">
         {fields.map((field, index) => {
           const nameField = `${name}.${index}.input`;
 
           return (
-            <div key={field.id} className="flex items-center gap-2">
+            <div key={field.id} className="flex gap-2">
               <QueryVariable
                 name={nameField}
                 hideLabel

web/src/pages/agent/form/data-operations-form/index.tsx

@@ -4,6 +4,7 @@ import { Form } from '@/components/ui/form';
 import { Separator } from '@/components/ui/separator';
 import { buildOptions } from '@/utils/form';
 import { zodResolver } from '@hookform/resolvers/zod';
+import { t } from 'i18next';
 import { memo } from 'react';
 import { useForm, useWatch } from 'react-hook-form';
 import { useTranslation } from 'react-i18next';
@@ -25,7 +26,11 @@ import { SelectKeys } from './select-keys';
 import { Updates } from './updates';
 
 export const RetrievalPartialSchema = {
-  query: z.array(z.object({ input: z.string().optional() })),
+  query: z.array(
+    z.object({
+      input: z.string().min(1, { message: t('flow.queryRequired') }),
+    }),
+  ),
   operations: z.string(),
   select_keys: z.array(z.object({ name: z.string().optional() })).optional(),
   remove_keys: z.array(z.object({ name: z.string().optional() })).optional(),

web/src/pages/agent/hooks/use-get-begin-query.tsx

@@ -317,14 +317,18 @@ export const useGetComponentLabelByValue = (nodeId: string) => {
   return getLabel;
 };
 
+export function flatOptions(options: DefaultOptionType[]) {
+  return options.reduce<DefaultOptionType[]>((pre, cur) => {
+    return [...pre, ...cur.options];
+  }, []);
+}
+
 export function useFlattenQueryVariableOptions(nodeId?: string) {
   const { getNode } = useGraphStore((state) => state);
   const nextOptions = useBuildQueryVariableOptions(getNode(nodeId));
 
   const flattenOptions = useMemo(() => {
-    return nextOptions.reduce<DefaultOptionType[]>((pre, cur) => {
+    return flatOptions(nextOptions);
-      return [...pre, ...cur.options];
-    }, []);
   }, [nextOptions]);
 
   return flattenOptions;