Fix fundamental schema issue with required fields defined incorrectly on JsonSchemaType

[olaservo]

Fixes a critical schema architecture problem identified by @max-stytch where the JsonSchemaType definition violated the JSON Schema specification. JsonSchemaType incorrectly defined required as boolean on individual properties instead of string[] array on parent object schema, leading to unexpected behavior in some tools testing scenarios and several open issues.

Motivation and Context

Related observed issues:

• Issue Empty string parameters are not sent (validation error missing required argument) #406: Empty string parameters incorrectly converted to null/undefined
• Multiple form validation failures across ToolsTab and DynamicJsonForm components
• Broken prop.required logic throughout the codebase

How Has This Been Tested?

Did some basic testing but still working on more methodically testing the different scenarios. Thought it would be worth opening now for visibility's sake.

Breaking Changes

Technically yes due to internal API changes, however these all restore the correct behavior behind the scenes and don't require any changes on the user side:

• generateDefaultValue() function signature changed to include propertyName and parentSchema parameters
• JsonSchemaType.required changed from boolean to string[]
• Components using prop.required logic needed updates

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation update

Checklist

• I have read the MCP Documentation
• My code follows the repository's style guidelines
• New and existing tests pass locally
• I have added appropriate error handling
• I have added or updated documentation as needed

[cliffhall]

We could use the isPropertyRequired helper here and avoid the duplication.

Suggested change

	const isRequired =
	parentSchema?.required?.includes(propertyName || "") ?? false;
	const isRequired = isPropertyRequired(propertyName, parentSchema);

[cliffhall]

Suggested change

	const isPropertyRequired = schema.required?.includes(key) ?? false;
	if (isPropertyRequired) {
	const value = generateDefaultValue(prop, key, schema);
	if (value !== undefined) {
	obj[key] = value;
	}
	}
	if (isPropertyRequired(key, schema)) {
	const value = generateDefaultValue(prop, key, schema);
	if (value !== undefined) {
	obj[key] = value;
	}
	}

[cliffhall]

Found a couple of places where isPropertyRequired helper can be used.