fix: move pragma no cover to defensive else branch

ROOT CAUSE:
strict-no-cover was failing because pragma was on the if statement
which IS covered. The uncovered branch is the else case.

CHANGES:
- Moved pragma from if statement to else branch (continue statement)
- Added comment explaining why this is defensive code

IMPACT:
- strict-no-cover will now pass
- Tests remain fully functional
- Defensive code properly marked

Refs: #2086

Author: peter-luminova

src/mcp/server/mcpserver/tools/base.py

@@ -125,8 +125,11 @@ async def run(
 
                 deps = find_dependency_parameters(self.fn)
                 for dep_name in self.dependency_kwarg_names:
-                    if dep_name in deps:  # pragma: no cover (defensive, should always be in deps)
+                    if dep_name in deps:
                         direct_args[dep_name] = await dependency_resolver.resolve(dep_name, deps[dep_name])
+                    else:
+                        # Defensive: should never happen since dependency_kwarg_names is built from deps
+                        continue  # pragma: no cover
 
             result = await self.fn_metadata.call_fn_with_arg_validation(
                 self.fn,