@foresthubai/workflow-builder 0.4.3 → 0.4.5

package/src/inputs/ParameterEditor.tsx

@@ -1,515 +1,519 @@
-import { Input } from "../components/ui/input";
-import { Textarea } from "../components/ui/textarea";
-import { Label } from "../components/ui/label";
-import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue } from "../components/ui/select";
-import { Switch } from "../components/ui/switch";
-import { Alert, AlertDescription } from "../components/ui/alert";
-import { Button } from "../components/ui/button";
-import { AddButton } from "../components/ui/add-button";
-import { ToggleGroup, ToggleGroupItem } from "../components/ui/toggle-group";
-import { AlertTriangle, Trash2 } from "lucide-react";
-import type { DataType, Expression, Reference } from "@foresthubai/workflow-core";
-import type {
-  ExpressionParam,
-  ChannelSelectParam,
-  MemorySelectParam,
-  ModelSelectParam,
-  Parameter,
-  StringParam,
-} from "@foresthubai/workflow-core/parameter";
-import {
-  resolveCapabilities,
-  resolveExpressionType,
-  resolveChannelTypes,
-  resolveMemoryTypes,
-  resolveModelTypes,
-} from "@foresthubai/workflow-core/parameter";
-import { useTranslation } from "react-i18next";
-import { useAvailableVariables } from "../hooks/useAvailableVariables";
-import { useEditorStore } from "../stores/editorStore";
-import { varKey, refToLookupKey } from "@foresthubai/workflow-core/variable";
-import type { Channel } from "@foresthubai/workflow-core/channel";
-import type { Memory, MemoryRef } from "@foresthubai/workflow-core/memory";
-import type { ModelCapability } from "@foresthubai/workflow-core/model";
-import ExpressionInput from "./ExpressionInput";
-import { getParamDescription } from "../utils/translation";
-
-/** Shared Select component for all reference-select parameter types */
-function ReferenceSelect({
-  value,
-  options,
-  isStale,
-  loading,
-  placeholder,
-  onChange,
-}: {
-  value: string | undefined;
-  options: { value: string; label: string }[];
-  isStale: boolean;
-  loading?: boolean;
-  placeholder: string;
-  onChange: (value: string | undefined) => void;
-}) {
-  const NONE = "__none__";
-  const selectValue = isStale ? undefined : (value ?? NONE);
-
-  return (
-    <Select value={selectValue} onValueChange={(v) => onChange(v === NONE ? undefined : v)} disabled={loading}>
-      <SelectTrigger>
-        <SelectValue
-          placeholder={
-            isStale ? <span className="text-destructive">Deleted reference</span> : loading ? "Loading..." : placeholder
-          }
-        />
-      </SelectTrigger>
-      <SelectContent>
-        <SelectItem value={NONE}>
-          <span className="text-muted-foreground italic pr-0.5">None</span>
-        </SelectItem>
-        {options.map((opt) => (
-          <SelectItem key={opt.value} value={opt.value}>
-            {opt.label}
-          </SelectItem>
-        ))}
-      </SelectContent>
-    </Select>
-  );
-}
-
-function getExpressionPlaceholder(dataType: DataType): string {
-  switch (dataType) {
-    case "int":
-    case "float":
-      return "${var1} + ${var2}";
-    case "bool":
-      return "${var1} > ${var2}";
-    case "string":
-      return "hello ${var1}";
-    default:
-      return "${var1}";
-  }
-}
-
-interface ParameterEditorProps {
-  canvasId: string;
-  parameter: Parameter;
-  value: unknown;
-  allArguments: Record<string, unknown>;
-  onChange: (value: unknown) => void;
-  errors?: string[];
-  translationPrefix?: string;
-}
-
-const ParameterEditor = ({
-  canvasId,
-  parameter,
-  value,
-  allArguments,
-  onChange,
-  errors,
-  translationPrefix,
-}: ParameterEditorProps) => {
-  const { t } = useTranslation();
-  // Do NOT fall back to parameter.default here — each input type handles undefined individually.
-  // This preserves the distinction between "user cleared field" and "default value".
-  const currentValue = value;
-  const { list: variableList, lookup: variables } = useAvailableVariables(canvasId);
-  const channels = useEditorStore((s) => s.channels);
-  const memory = useEditorStore((s) => s.memory);
-  const models = useEditorStore((s) => s.models);
-  const availableModels = useEditorStore((s) => s.availableModels);
-
-  const renderInput = () => {
-    switch (parameter.type) {
-      case "variableSelect": {
-        const ref = currentValue as Reference | undefined;
-        const selectedKey = ref?.varId ? refToLookupKey(ref) : undefined;
-        const isStale = !!(selectedKey && !variables[selectedKey]);
-        const options = variableList.map((v) => ({ value: varKey(v), label: `${v.name} (${v.dataType})` }));
-
-        return (
-          <ReferenceSelect
-            value={selectedKey}
-            options={options}
-            isStale={isStale}
-            placeholder="Select variable..."
-            onChange={(key) => {
-              if (!key) {
-                onChange(undefined);
-                return;
-              }
-              const variable = variables[key];
-              if (!variable) return;
-              const newRef: Reference =
-                variable.kind === "node"
-                  ? { srcId: variable.nodeId, varId: variable.outputId }
-                  : variable.kind === "declared"
-                    ? { srcId: "declared", varId: variable.uid }
-                    : { srcId: "fnarg", varId: variable.uid };
-              onChange(newRef);
-            }}
-          />
-        );
-      }
-
-      case "expression": {
-        const exprParam = parameter as ExpressionParam;
-        // Resolve expressionType — static, args-only lambda, or derived from a referenced variable
-        const resolvedType: DataType = resolveExpressionType(exprParam, allArguments, variables);
-        // Ensure we have a valid expression object (create empty one if undefined for safety)
-        const exprValue: Expression =
-          currentValue && typeof currentValue === "object" && "expression" in currentValue
-            ? (currentValue as Expression)
-            : { expression: "", references: [], dataType: resolvedType };
-        return (
-          <ExpressionInput
-            value={exprValue}
-            onChange={onChange}
-            expressionType={resolvedType}
-            availableVariables={variables}
-            placeholder={getExpressionPlaceholder(resolvedType)}
-          />
-        );
-      }
-
-      case "string":
-        if ((parameter as StringParam).multiline) {
-          return (
-            <Textarea
-              value={(currentValue as string) || ""}
-              onChange={(e) => onChange(e.target.value)}
-              placeholder={String(parameter.default ?? "")}
-              rows={4}
-            />
-          );
-        }
-        return (
-          <Input
-            value={(currentValue as string) || ""}
-            onChange={(e) => onChange(e.target.value)}
-            placeholder={String(parameter.default ?? "")}
-          />
-        );
-
-      case "int":
-        return (
-          <Input
-            type="number"
-            step={1}
-            value={currentValue != null ? (currentValue as number) : ""}
-            onChange={(e) => {
-              const numValue = parseInt(e.target.value, 10);
-              onChange(isNaN(numValue) ? undefined : numValue);
-            }}
-            placeholder={parameter.default?.toString() || "0"}
-          />
-        );
-
-      case "float":
-        return (
-          <Input
-            type="number"
-            step="any"
-            className="[&::-webkit-inner-spin-button]:appearance-none [&::-webkit-outer-spin-button]:appearance-none [-moz-appearance:textfield]"
-            value={currentValue != null ? (currentValue as number) : ""}
-            onChange={(e) => {
-              const numValue = parseFloat(e.target.value);
-              onChange(isNaN(numValue) ? undefined : numValue);
-            }}
-            placeholder={parameter.default?.toString() || "0"}
-          />
-        );
-
-      case "bool": {
-        const boolValue = currentValue as boolean;
-        return (
-          <div className="flex items-center space-x-2">
-            <Switch checked={boolValue} onCheckedChange={onChange} />
-            <span className="text-sm text-muted-foreground">{boolValue ? "Enabled" : "Disabled"}</span>
-          </div>
-        );
-      }
-
-      case "selection": {
-        const NONE = "__none__";
-        const selectValue = (currentValue as string) ?? (parameter.optional ? NONE : "");
-        return (
-          <Select value={selectValue} onValueChange={(v) => onChange(v === NONE ? undefined : v)}>
-            <SelectTrigger>
-              <SelectValue placeholder="Select..." />
-            </SelectTrigger>
-            <SelectContent>
-              {parameter.optional && (
-                <SelectItem value={NONE}>
-                  <span className="text-muted-foreground italic pr-0.5">None</span>
-                </SelectItem>
-              )}
-              {parameter.options.map((option) => (
-                <SelectItem key={option.value} value={option.value}>
-                  {option.label}
-                </SelectItem>
-              ))}
-            </SelectContent>
-          </Select>
-        );
-      }
-
-      case "memorySelect": {
-        const memoryParam = parameter as ParameterEditorProps["parameter"] & MemorySelectParam;
-        const allowedTypes = resolveMemoryTypes(memoryParam, allArguments);
-        const matching = Object.values(memory).filter((m: Memory) => allowedTypes.includes(m.type));
-
-        const selectedId = currentValue as string | undefined;
-        const isStale = !!(selectedId && !matching.some((m) => m.id === selectedId));
-        const options = matching.map((m) => ({ value: m.id, label: m.label }));
-
-        return (
-          <ReferenceSelect
-            value={selectedId}
-            options={options}
-            isStale={isStale}
-            placeholder={t("selectMemory", "Select memory...")}
-            onChange={(v) => onChange(v)}
-          />
-        );
-      }
-
-      case "modelSelect": {
-        const modelParam = parameter as ParameterEditorProps["parameter"] & ModelSelectParam;
-        const allowedTypes = resolveModelTypes(modelParam, allArguments);
-        const requiredCaps = resolveCapabilities(modelParam, allArguments);
-        const hasAllCaps = (caps: ModelCapability[]) =>
-          !requiredCaps?.length || requiredCaps.every((c) => caps.includes(c));
-
-        // Static catalog (props): the always-available models the llmproxy supports.
-        const catalogOptions = availableModels
-          .filter((m) => hasAllCaps(m.capabilities))
-          .map((m) => ({ value: m.id, label: m.label }));
-
-        // Declared custom models of a compatible type (capabilities default to chat).
-        const customOptions = Object.values(models)
-          .filter(
-            (m) =>
-              allowedTypes.includes(m.type) && hasAllCaps((m.arguments.capabilities as ModelCapability[]) ?? ["chat"]),
-          )
-          .map((m) => ({ value: m.id, label: m.label }));
-
-        // Catalog first; drop customs shadowed by a catalog id.
-        const catalogIds = new Set(catalogOptions.map((o) => o.value));
-        const options = [...catalogOptions, ...customOptions.filter((o) => !catalogIds.has(o.value))];
-
-        const selectedId = currentValue as string | undefined;
-        const isStale = !!(selectedId && !options.some((o) => o.value === selectedId));
-
-        if (options.length === 0) {
-          return (
-            <Alert variant="destructive">
-              <AlertTriangle className="h-4 w-4" />
-              <AlertDescription>{t("noModelsAvailable")}</AlertDescription>
-            </Alert>
-          );
-        }
-
-        return (
-          <ReferenceSelect
-            value={selectedId}
-            options={options}
-            isStale={isStale}
-            placeholder={t("selectModel", "Select model...")}
-            onChange={(v) => onChange(v)}
-          />
-        );
-      }
-
-      case "time":
-        return <Input type="time" value={(currentValue as string) ?? ""} onChange={(e) => onChange(e.target.value)} />;
-
-      case "weekdays": {
-        const DAYS = [
-          { code: "mon", label: "Mon" },
-          { code: "tue", label: "Tue" },
-          { code: "wed", label: "Wed" },
-          { code: "thu", label: "Thu" },
-          { code: "fri", label: "Fri" },
-          { code: "sat", label: "Sat" },
-          { code: "sun", label: "Sun" },
-        ];
-        const selected = (currentValue as string[]) || [];
-        const toggle = (code: string) => {
-          const next = selected.includes(code) ? selected.filter((d) => d !== code) : [...selected, code];
-          onChange(next);
-        };
-        return (
-          <div className="space-y-1.5">
-            <div className="flex gap-1">
-              {DAYS.map((day) => (
-                <button
-                  key={day.code}
-                  type="button"
-                  onClick={() => toggle(day.code)}
-                  className={`px-2 py-1 text-xs rounded-md border transition-colors ${
-                    selected.includes(day.code)
-                      ? "bg-primary text-primary-foreground border-primary"
-                      : "bg-field text-muted-foreground border-input hover:bg-muted/50"
-                  }`}
-                >
-                  {day.label}
-                </button>
-              ))}
-            </div>
-          </div>
-        );
-      }
-
-      case "memory-refs": {
-        const refs = (currentValue as MemoryRef[] | undefined) ?? [];
-        // memory-refs bind to MemoryFile-type memories only.
-        const allFiles = Object.values(memory).filter((m) => m.type === "MemoryFile");
-        const allFilesById = new Map(allFiles.map((m) => [m.id, m]));
-
-        const replace = (index: number, next: MemoryRef) => onChange(refs.map((r, i) => (i === index ? next : r)));
-        const remove = (index: number) => onChange(refs.filter((_, i) => i !== index));
-        const add = () => {
-          // Pre-select the first unused memory file (if any) so adding a row
-          // immediately gives the user a valid binding to start tweaking.
-          const usedIds = new Set(refs.map((r) => r.id));
-          const firstUnused = allFiles.find((m) => !usedIds.has(m.id));
-          onChange([...refs, { id: firstUnused?.id ?? "", mode: "r" as const }]);
-        };
-
-        // Always render existing refs (even when there are 0 memory files left)
-        // so the user can delete dangling references after the underlying memory
-        // file is removed. Mirror of the output-binding pattern in NodeConfigPanel.
-        const canAdd = allFiles.length > refs.length;
-
-        return (
-          <div className="space-y-2">
-            {refs.length === 0 && allFiles.length === 0 && (
-              <p className="text-xs text-muted-foreground italic px-2 py-1">
-                {t(
-                  "noMemoryFilesForAgent",
-                  "No memory files declared yet. Add one from the Memory tab in the sidebar.",
-                )}
-              </p>
-            )}
-            {refs.map((ref, index) => {
-              const file = ref.id ? allFilesById.get(ref.id) : undefined;
-              const isStale = !!(ref.id && !file);
-              const usedByOthers = new Set(refs.filter((_, i) => i !== index).map((r) => r.id));
-              const selectableFiles = allFiles.filter((m) => m.id === ref.id || !usedByOthers.has(m.id));
-
-              return (
-                <div key={index} className="rounded-lg bg-card shadow-sm border border-border p-2 space-y-2 transition-all hover:shadow-md">
-                  <div className="flex items-center gap-2">
-                    <Select value={ref.id || undefined} onValueChange={(id) => replace(index, { ...ref, id })}>
-                      <SelectTrigger className="h-7 text-xs flex-1">
-                        <SelectValue
-                          placeholder={
-                            isStale ? (
-                              <span className="text-destructive">{t("memoryRefStale", "Deleted memory file")}</span>
-                            ) : (
-                              t("selectMemoryFile", "Select memory file...")
-                            )
-                          }
-                        />
-                      </SelectTrigger>
-                      <SelectContent>
-                        {selectableFiles.map((m) => (
-                          <SelectItem key={m.id} value={m.id}>
-                            {m.label}
-                          </SelectItem>
-                        ))}
-                      </SelectContent>
-                    </Select>
-                    <Button
-                      variant="ghost"
-                      size="icon"
-                      className="h-7 w-7 shrink-0"
-                      onClick={() => remove(index)}
-                      aria-label={t("removeMemoryRef", "Remove memory ref")}
-                    >
-                      <Trash2 className="w-3.5 h-3.5" />
-                    </Button>
-                  </div>
-                  <ToggleGroup
-                    type="single"
-                    value={ref.mode}
-                    onValueChange={(v) => v && replace(index, { ...ref, mode: v as "r" | "rw" })}
-                    className="gap-0 justify-start"
-                  >
-                    <ToggleGroupItem value="r" size="sm" variant="outline" className="h-7 px-3 text-xs rounded-r-none">
-                      {t("memoryModeRead", "Read")}
-                    </ToggleGroupItem>
-                    <ToggleGroupItem
-                      value="rw"
-                      size="sm"
-                      variant="outline"
-                      className="h-7 px-3 text-xs rounded-l-none -ml-px"
-                    >
-                      {t("memoryModeReadWrite", "Read + Write")}
-                    </ToggleGroupItem>
-                  </ToggleGroup>
-                </div>
-              );
-            })}
-            {canAdd && <AddButton onClick={add}>{t("addMemoryRef", "Add memory ref")}</AddButton>}
-          </div>
-        );
-      }
-
-      case "channelSelect": {
-        const channelParam = parameter as ParameterEditorProps["parameter"] & ChannelSelectParam;
-        const allowedTypes = resolveChannelTypes(channelParam, allArguments);
-        const matching = Object.values(channels).filter((v: Channel) => allowedTypes.includes(v.type));
-
-        const selectedId = currentValue as string | undefined;
-        const isStale = !!(selectedId && !matching.some((v) => v.id === selectedId));
-        const options = matching.map((v) => ({ value: v.id, label: v.label }));
-
-        return (
-          <ReferenceSelect
-            value={selectedId}
-            options={options}
-            isStale={isStale}
-            placeholder={t("selectChannel")}
-            onChange={(v) => onChange(v)}
-          />
-        );
-      }
-
-      // Exhaustiveness check - fails if a parameter type is not handled
-      default: {
-        const _exhaustive: never = parameter;
-        return _exhaustive;
-      }
-    }
-  };
-
-  const hasError = errors && errors.length > 0;
-
-  return (
-    <div className={`space-y-2 ${hasError ? "ring-1 ring-destructive rounded-md p-2" : ""}`}>
-      <Label className="text-sm font-medium">
-        {parameter.label}
-        {parameter.optional !== true && <span className="text-destructive ml-1">*</span>}
-      </Label>
-      {parameter.description && (
-        <p className="text-xs text-muted-foreground">
-          {translationPrefix ? getParamDescription(t, translationPrefix, parameter) : parameter.description}
-        </p>
-      )}
-      {renderInput()}
-      {hasError && (
-        <div className="space-y-0.5">
-          {errors.map((msg, i) => (
-            <p key={i} className="text-xs text-destructive">
-              {msg}
-            </p>
-          ))}
-        </div>
-      )}
-    </div>
-  );
-};
-
-export default ParameterEditor;
+// SPDX-License-Identifier: AGPL-3.0-only
+// Copyright (c) 2026 ForestHub. All rights reserved.
+// For commercial licensing, contact root@foresthub.ai
+
+import { Input } from "../components/ui/input";
+import { Textarea } from "../components/ui/textarea";
+import { Label } from "../components/ui/label";
+import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue } from "../components/ui/select";
+import { Switch } from "../components/ui/switch";
+import { Alert, AlertDescription } from "../components/ui/alert";
+import { Button } from "../components/ui/button";
+import { AddButton } from "../components/ui/add-button";
+import { ToggleGroup, ToggleGroupItem } from "../components/ui/toggle-group";
+import { AlertTriangle, Trash2 } from "lucide-react";
+import type { DataType, Expression, Reference } from "@foresthubai/workflow-core";
+import type {
+  ExpressionParam,
+  ChannelSelectParam,
+  MemorySelectParam,
+  ModelSelectParam,
+  Parameter,
+  StringParam,
+} from "@foresthubai/workflow-core/parameter";
+import {
+  resolveCapabilities,
+  resolveExpressionType,
+  resolveChannelTypes,
+  resolveMemoryTypes,
+  resolveModelTypes,
+} from "@foresthubai/workflow-core/parameter";
+import { useTranslation } from "react-i18next";
+import { useAvailableVariables } from "../hooks/useAvailableVariables";
+import { useEditorStore } from "../stores/editorStore";
+import { varKey, refToLookupKey } from "@foresthubai/workflow-core/variable";
+import type { Channel } from "@foresthubai/workflow-core/channel";
+import type { Memory, MemoryRef } from "@foresthubai/workflow-core/memory";
+import type { ModelCapability } from "@foresthubai/workflow-core/model";
+import ExpressionInput from "./ExpressionInput";
+import { getParamDescription } from "../utils/translation";
+
+/** Shared Select component for all reference-select parameter types */
+function ReferenceSelect({
+  value,
+  options,
+  isStale,
+  loading,
+  placeholder,
+  onChange,
+}: {
+  value: string | undefined;
+  options: { value: string; label: string }[];
+  isStale: boolean;
+  loading?: boolean;
+  placeholder: string;
+  onChange: (value: string | undefined) => void;
+}) {
+  const NONE = "__none__";
+  const selectValue = isStale ? undefined : (value ?? NONE);
+
+  return (
+    <Select value={selectValue} onValueChange={(v) => onChange(v === NONE ? undefined : v)} disabled={loading}>
+      <SelectTrigger>
+        <SelectValue
+          placeholder={
+            isStale ? <span className="text-destructive">Deleted reference</span> : loading ? "Loading..." : placeholder
+          }
+        />
+      </SelectTrigger>
+      <SelectContent>
+        <SelectItem value={NONE}>
+          <span className="text-muted-foreground italic pr-0.5">None</span>
+        </SelectItem>
+        {options.map((opt) => (
+          <SelectItem key={opt.value} value={opt.value}>
+            {opt.label}
+          </SelectItem>
+        ))}
+      </SelectContent>
+    </Select>
+  );
+}
+
+function getExpressionPlaceholder(dataType: DataType): string {
+  switch (dataType) {
+    case "int":
+    case "float":
+      return "${var1} + ${var2}";
+    case "bool":
+      return "${var1} > ${var2}";
+    case "string":
+      return "hello ${var1}";
+    default:
+      return "${var1}";
+  }
+}
+
+interface ParameterEditorProps {
+  canvasId: string;
+  parameter: Parameter;
+  value: unknown;
+  allArguments: Record<string, unknown>;
+  onChange: (value: unknown) => void;
+  errors?: string[];
+  translationPrefix?: string;
+}
+
+const ParameterEditor = ({
+  canvasId,
+  parameter,
+  value,
+  allArguments,
+  onChange,
+  errors,
+  translationPrefix,
+}: ParameterEditorProps) => {
+  const { t } = useTranslation();
+  // Do NOT fall back to parameter.default here — each input type handles undefined individually.
+  // This preserves the distinction between "user cleared field" and "default value".
+  const currentValue = value;
+  const { list: variableList, lookup: variables } = useAvailableVariables(canvasId);
+  const channels = useEditorStore((s) => s.channels);
+  const memory = useEditorStore((s) => s.memory);
+  const models = useEditorStore((s) => s.models);
+  const availableModels = useEditorStore((s) => s.availableModels);
+
+  const renderInput = () => {
+    switch (parameter.type) {
+      case "variableSelect": {
+        const ref = currentValue as Reference | undefined;
+        const selectedKey = ref?.varId ? refToLookupKey(ref) : undefined;
+        const isStale = !!(selectedKey && !variables[selectedKey]);
+        const options = variableList.map((v) => ({ value: varKey(v), label: `${v.name} (${v.dataType})` }));
+
+        return (
+          <ReferenceSelect
+            value={selectedKey}
+            options={options}
+            isStale={isStale}
+            placeholder="Select variable..."
+            onChange={(key) => {
+              if (!key) {
+                onChange(undefined);
+                return;
+              }
+              const variable = variables[key];
+              if (!variable) return;
+              const newRef: Reference =
+                variable.kind === "node"
+                  ? { srcId: variable.nodeId, varId: variable.outputId }
+                  : variable.kind === "declared"
+                    ? { srcId: "declared", varId: variable.uid }
+                    : { srcId: "fnarg", varId: variable.uid };
+              onChange(newRef);
+            }}
+          />
+        );
+      }
+
+      case "expression": {
+        const exprParam = parameter as ExpressionParam;
+        // Resolve expressionType — static, args-only lambda, or derived from a referenced variable
+        const resolvedType: DataType = resolveExpressionType(exprParam, allArguments, variables);
+        // Ensure we have a valid expression object (create empty one if undefined for safety)
+        const exprValue: Expression =
+          currentValue && typeof currentValue === "object" && "expression" in currentValue
+            ? (currentValue as Expression)
+            : { expression: "", references: [], dataType: resolvedType };
+        return (
+          <ExpressionInput
+            value={exprValue}
+            onChange={onChange}
+            expressionType={resolvedType}
+            availableVariables={variables}
+            placeholder={getExpressionPlaceholder(resolvedType)}
+          />
+        );
+      }
+
+      case "string":
+        if ((parameter as StringParam).multiline) {
+          return (
+            <Textarea
+              value={(currentValue as string) || ""}
+              onChange={(e) => onChange(e.target.value)}
+              placeholder={String(parameter.default ?? "")}
+              rows={4}
+            />
+          );
+        }
+        return (
+          <Input
+            value={(currentValue as string) || ""}
+            onChange={(e) => onChange(e.target.value)}
+            placeholder={String(parameter.default ?? "")}
+          />
+        );
+
+      case "int":
+        return (
+          <Input
+            type="number"
+            step={1}
+            value={currentValue != null ? (currentValue as number) : ""}
+            onChange={(e) => {
+              const numValue = parseInt(e.target.value, 10);
+              onChange(isNaN(numValue) ? undefined : numValue);
+            }}
+            placeholder={parameter.default?.toString() || "0"}
+          />
+        );
+
+      case "float":
+        return (
+          <Input
+            type="number"
+            step="any"
+            className="[&::-webkit-inner-spin-button]:appearance-none [&::-webkit-outer-spin-button]:appearance-none [-moz-appearance:textfield]"
+            value={currentValue != null ? (currentValue as number) : ""}
+            onChange={(e) => {
+              const numValue = parseFloat(e.target.value);
+              onChange(isNaN(numValue) ? undefined : numValue);
+            }}
+            placeholder={parameter.default?.toString() || "0"}
+          />
+        );
+
+      case "bool": {
+        const boolValue = currentValue as boolean;
+        return (
+          <div className="flex items-center space-x-2">
+            <Switch checked={boolValue} onCheckedChange={onChange} />
+            <span className="text-sm text-muted-foreground">{boolValue ? "Enabled" : "Disabled"}</span>
+          </div>
+        );
+      }
+
+      case "selection": {
+        const NONE = "__none__";
+        const selectValue = (currentValue as string) ?? (parameter.optional ? NONE : "");
+        return (
+          <Select value={selectValue} onValueChange={(v) => onChange(v === NONE ? undefined : v)}>
+            <SelectTrigger>
+              <SelectValue placeholder="Select..." />
+            </SelectTrigger>
+            <SelectContent>
+              {parameter.optional && (
+                <SelectItem value={NONE}>
+                  <span className="text-muted-foreground italic pr-0.5">None</span>
+                </SelectItem>
+              )}
+              {parameter.options.map((option) => (
+                <SelectItem key={option.value} value={option.value}>
+                  {option.label}
+                </SelectItem>
+              ))}
+            </SelectContent>
+          </Select>
+        );
+      }
+
+      case "memorySelect": {
+        const memoryParam = parameter as ParameterEditorProps["parameter"] & MemorySelectParam;
+        const allowedTypes = resolveMemoryTypes(memoryParam, allArguments);
+        const matching = Object.values(memory).filter((m: Memory) => allowedTypes.includes(m.type));
+
+        const selectedId = currentValue as string | undefined;
+        const isStale = !!(selectedId && !matching.some((m) => m.id === selectedId));
+        const options = matching.map((m) => ({ value: m.id, label: m.label }));
+
+        return (
+          <ReferenceSelect
+            value={selectedId}
+            options={options}
+            isStale={isStale}
+            placeholder={t("selectMemory", "Select memory...")}
+            onChange={(v) => onChange(v)}
+          />
+        );
+      }
+
+      case "modelSelect": {
+        const modelParam = parameter as ParameterEditorProps["parameter"] & ModelSelectParam;
+        const allowedTypes = resolveModelTypes(modelParam, allArguments);
+        const requiredCaps = resolveCapabilities(modelParam, allArguments);
+        const hasAllCaps = (caps: ModelCapability[]) =>
+          !requiredCaps?.length || requiredCaps.every((c) => caps.includes(c));
+
+        // Static catalog (props): the always-available models the llmproxy supports.
+        const catalogOptions = availableModels
+          .filter((m) => hasAllCaps(m.capabilities))
+          .map((m) => ({ value: m.id, label: m.label }));
+
+        // Declared custom models of a compatible type (capabilities default to chat).
+        const customOptions = Object.values(models)
+          .filter(
+            (m) =>
+              allowedTypes.includes(m.type) && hasAllCaps((m.arguments.capabilities as ModelCapability[]) ?? ["chat"]),
+          )
+          .map((m) => ({ value: m.id, label: m.label }));
+
+        // Catalog first; drop customs shadowed by a catalog id.
+        const catalogIds = new Set(catalogOptions.map((o) => o.value));
+        const options = [...catalogOptions, ...customOptions.filter((o) => !catalogIds.has(o.value))];
+
+        const selectedId = currentValue as string | undefined;
+        const isStale = !!(selectedId && !options.some((o) => o.value === selectedId));
+
+        if (options.length === 0) {
+          return (
+            <Alert variant="destructive">
+              <AlertTriangle className="h-4 w-4" />
+              <AlertDescription>{t("noModelsAvailable")}</AlertDescription>
+            </Alert>
+          );
+        }
+
+        return (
+          <ReferenceSelect
+            value={selectedId}
+            options={options}
+            isStale={isStale}
+            placeholder={t("selectModel", "Select model...")}
+            onChange={(v) => onChange(v)}
+          />
+        );
+      }
+
+      case "time":
+        return <Input type="time" value={(currentValue as string) ?? ""} onChange={(e) => onChange(e.target.value)} />;
+
+      case "weekdays": {
+        const DAYS = [
+          { code: "mon", label: "Mon" },
+          { code: "tue", label: "Tue" },
+          { code: "wed", label: "Wed" },
+          { code: "thu", label: "Thu" },
+          { code: "fri", label: "Fri" },
+          { code: "sat", label: "Sat" },
+          { code: "sun", label: "Sun" },
+        ];
+        const selected = (currentValue as string[]) || [];
+        const toggle = (code: string) => {
+          const next = selected.includes(code) ? selected.filter((d) => d !== code) : [...selected, code];
+          onChange(next);
+        };
+        return (
+          <div className="space-y-1.5">
+            <div className="flex gap-1">
+              {DAYS.map((day) => (
+                <button
+                  key={day.code}
+                  type="button"
+                  onClick={() => toggle(day.code)}
+                  className={`px-2 py-1 text-xs rounded-md border transition-colors ${
+                    selected.includes(day.code)
+                      ? "bg-primary text-primary-foreground border-primary"
+                      : "bg-field text-muted-foreground border-input hover:bg-muted/50"
+                  }`}
+                >
+                  {day.label}
+                </button>
+              ))}
+            </div>
+          </div>
+        );
+      }
+
+      case "memory-refs": {
+        const refs = (currentValue as MemoryRef[] | undefined) ?? [];
+        // memory-refs bind to MemoryFile-type memories only.
+        const allFiles = Object.values(memory).filter((m) => m.type === "MemoryFile");
+        const allFilesById = new Map(allFiles.map((m) => [m.id, m]));
+
+        const replace = (index: number, next: MemoryRef) => onChange(refs.map((r, i) => (i === index ? next : r)));
+        const remove = (index: number) => onChange(refs.filter((_, i) => i !== index));
+        const add = () => {
+          // Pre-select the first unused memory file (if any) so adding a row
+          // immediately gives the user a valid binding to start tweaking.
+          const usedIds = new Set(refs.map((r) => r.id));
+          const firstUnused = allFiles.find((m) => !usedIds.has(m.id));
+          onChange([...refs, { id: firstUnused?.id ?? "", mode: "r" as const }]);
+        };
+
+        // Always render existing refs (even when there are 0 memory files left)
+        // so the user can delete dangling references after the underlying memory
+        // file is removed. Mirror of the output-binding pattern in NodeConfigPanel.
+        const canAdd = allFiles.length > refs.length;
+
+        return (
+          <div className="space-y-2">
+            {refs.length === 0 && allFiles.length === 0 && (
+              <p className="text-xs text-muted-foreground italic px-2 py-1">
+                {t(
+                  "noMemoryFilesForAgent",
+                  "No memory files declared yet. Add one from the Memory tab in the sidebar.",
+                )}
+              </p>
+            )}
+            {refs.map((ref, index) => {
+              const file = ref.id ? allFilesById.get(ref.id) : undefined;
+              const isStale = !!(ref.id && !file);
+              const usedByOthers = new Set(refs.filter((_, i) => i !== index).map((r) => r.id));
+              const selectableFiles = allFiles.filter((m) => m.id === ref.id || !usedByOthers.has(m.id));
+
+              return (
+                <div key={index} className="rounded-lg bg-card shadow-sm border border-border p-2 space-y-2 transition-all hover:shadow-md">
+                  <div className="flex items-center gap-2">
+                    <Select value={ref.id || undefined} onValueChange={(id) => replace(index, { ...ref, id })}>
+                      <SelectTrigger className="h-7 text-xs flex-1">
+                        <SelectValue
+                          placeholder={
+                            isStale ? (
+                              <span className="text-destructive">{t("memoryRefStale", "Deleted memory file")}</span>
+                            ) : (
+                              t("selectMemoryFile", "Select memory file...")
+                            )
+                          }
+                        />
+                      </SelectTrigger>
+                      <SelectContent>
+                        {selectableFiles.map((m) => (
+                          <SelectItem key={m.id} value={m.id}>
+                            {m.label}
+                          </SelectItem>
+                        ))}
+                      </SelectContent>
+                    </Select>
+                    <Button
+                      variant="ghost"
+                      size="icon"
+                      className="h-7 w-7 shrink-0"
+                      onClick={() => remove(index)}
+                      aria-label={t("removeMemoryRef", "Remove memory ref")}
+                    >
+                      <Trash2 className="w-3.5 h-3.5" />
+                    </Button>
+                  </div>
+                  <ToggleGroup
+                    type="single"
+                    value={ref.mode}
+                    onValueChange={(v) => v && replace(index, { ...ref, mode: v as "r" | "rw" })}
+                    className="gap-0 justify-start"
+                  >
+                    <ToggleGroupItem value="r" size="sm" variant="outline" className="h-7 px-3 text-xs rounded-r-none">
+                      {t("memoryModeRead", "Read")}
+                    </ToggleGroupItem>
+                    <ToggleGroupItem
+                      value="rw"
+                      size="sm"
+                      variant="outline"
+                      className="h-7 px-3 text-xs rounded-l-none -ml-px"
+                    >
+                      {t("memoryModeReadWrite", "Read + Write")}
+                    </ToggleGroupItem>
+                  </ToggleGroup>
+                </div>
+              );
+            })}
+            {canAdd && <AddButton onClick={add}>{t("addMemoryRef", "Add memory ref")}</AddButton>}
+          </div>
+        );
+      }
+
+      case "channelSelect": {
+        const channelParam = parameter as ParameterEditorProps["parameter"] & ChannelSelectParam;
+        const allowedTypes = resolveChannelTypes(channelParam, allArguments);
+        const matching = Object.values(channels).filter((v: Channel) => allowedTypes.includes(v.type));
+
+        const selectedId = currentValue as string | undefined;
+        const isStale = !!(selectedId && !matching.some((v) => v.id === selectedId));
+        const options = matching.map((v) => ({ value: v.id, label: v.label }));
+
+        return (
+          <ReferenceSelect
+            value={selectedId}
+            options={options}
+            isStale={isStale}
+            placeholder={t("selectChannel")}
+            onChange={(v) => onChange(v)}
+          />
+        );
+      }
+
+      // Exhaustiveness check - fails if a parameter type is not handled
+      default: {
+        const _exhaustive: never = parameter;
+        return _exhaustive;
+      }
+    }
+  };
+
+  const hasError = errors && errors.length > 0;
+
+  return (
+    <div className={`space-y-2 ${hasError ? "ring-1 ring-destructive rounded-md p-2" : ""}`}>
+      <Label className="text-sm font-medium">
+        {parameter.label}
+        {parameter.optional !== true && <span className="text-destructive ml-1">*</span>}
+      </Label>
+      {parameter.description && (
+        <p className="text-xs text-muted-foreground">
+          {translationPrefix ? getParamDescription(t, translationPrefix, parameter) : parameter.description}
+        </p>
+      )}
+      {renderInput()}
+      {hasError && (
+        <div className="space-y-0.5">
+          {errors.map((msg, i) => (
+            <p key={i} className="text-xs text-destructive">
+              {msg}
+            </p>
+          ))}
+        </div>
+      )}
+    </div>
+  );
+};
+
+export default ParameterEditor;