fix(ui): type MCP argument resolver results explicitly

This commit is contained in:
Joshua Valluru 2026-09-09 16:57:15 -07:00
parent 70c238dde0
commit 0b21b99ebd
2 changed files with 3 additions and 3 deletions

View file

@ -41,7 +41,7 @@ describe("MCPToolArgumentsForm", () => {
fireEvent.change(screen.getByRole("textbox", { name: "filter.category *" }), {
target: { value: "invoices" },
});
fireEvent.change(screen.getByRole("textbox", { name: "filter", exact: true }), {
fireEvent.change(screen.getByRole("textbox", { name: "filter" }), {
target: { value: '{"category":"receipts","metadata":{"region":"eu"}}' },
});
fireEvent.change(screen.getByRole("spinbutton", { name: "page.limit" }), { target: { value: "7" } });

View file

@ -1,6 +1,6 @@
import React, { forwardRef, useImperativeHandle, useMemo } from "react";
import { CircleHelp } from "lucide-react";
import { useForm, type Resolver } from "react-hook-form";
import { useForm, type Resolver, type ResolverResult } from "react-hook-form";
import { FieldGroup } from "@/components/ui/field";
import { FormField } from "@/components/shared/form/FormField";
import { Input } from "@/components/ui/input";
@ -59,7 +59,7 @@ const collectErrors = (
const buildResolver =
(actualSchema: InputSchema, requiredMessages: Readonly<Record<string, string>> = {}): Resolver<ToolFormValues> =>
(values) => {
(values): ResolverResult<ToolFormValues> => {
const errors = collectErrors(actualSchema, requiredMessages, argumentValues(actualSchema, values));
if (Object.keys(errors).length === 0) return { values, errors: {} };
return {