ExecuteEntityTool: DB metadata based parameter support

[souvikghosh04]

Why make this change?

• Closes Parameter behaviour in MCP tool: ExecuteEntityTool #3588

ExecuteEntityTool validated parameters against config-side Source.Parameters only, rejecting DB-discovered parameters that had no config entry. Defaults were applied in an all-or-nothing fashion (only when the user supplied zero parameters).

What is this change?

• Parameter validation now uses StoredProcedureDefinition.Parameters (the merged DB metadata) as source of truth, accepting any parameter the DB declares.
• Default application iterates ParameterDefinition.HasConfigDefault/ConfigDefaultValue per-missing-parameter, so partial user input gets remaining defaults filled in. This is an intentional behavioral change from the previous all-or-nothing guard.
• Safe-cast dbObject to DatabaseStoredProcedure with graceful error instead of unconditional cast.

How was this tested?

• Unit Tests

Test	Scenario
AcceptsDbDiscoveredParam	Param in DB metadata but not config - accepted
RejectsInvalidParamName (x2)	Param not in DB metadata - error
AcceptsMultipleValidParams	Multiple valid params pass validation
RejectsRequest_WhenAnyParamInvalid	Mix of valid + invalid params - rejected
AppliesConfigDefaults	Missing params filled from HasConfigDefault/ConfigDefaultValue
UserParamsOverride	User-supplied values take precedence over config defaults
DoesNotInjectWithoutDefault	Param without HasConfigDefault is not injected
ZeroParamSP	SP with no parameters succeeds with no input
RejectsNonStoredProcedureEntity	Non-SP entity types return error
ReturnsError_WhenEntityNotFound	Non-existent entity returns EntityNotFound

• Integration Tests

Test	Scenario
Success (x4 DataRows)	GetBook(id), InsertBook(null), InsertBook(override), GetBooks(zero-param)
GetBookById_ReturnsRecord	Validates returned data matches expected
InsertBookWithDefaults_ExecutesSuccessfully	Config defaults produce valid insert
Rejection (x2 DataRows)	Invalid param names rejected with clear error

• Manual Testing

Category	Result
Valid DB-discovered params	Pass
Invalid/unknown/extra params rejected	Pass
Config defaults / partial params / override	Pass
Non-SP entity type gating	Pass
Successful SP execution (zero/single/multi-param)	Pass
Non-existent entity	Pass
SQL injection (parameterized)	Pass
Type mismatch	Pass
Permission denied	Pass
No results for valid param	Pass

Sample Request(s)

NA

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.