fix(tests): remove dead branch in test_nfkc_normalization_detected

The 'if original != normalized' guard was always True for fullwidth ASCII
(ａｂｃ always maps to abc under NFKC), making the False branch unreachable
and dropping overall branch coverage below the required 100%.

Author: manjunathbhaskar

tests/shared/test_tool_name_validation.py

@@ -297,19 +297,17 @@ def test_decomposed_unicode_detected(self) -> None:
         )
 
     def test_nfkc_normalization_detected(self) -> None:
-        """NFKC normalization changes should be detected."""
-        import unicodedata
-
-        # Fullwidth characters normalize differently in NFKC
-        original = "ａｂｃ"  # Fullwidth ASCII
-        normalized = unicodedata.normalize("NFKC", original)
+        """NFKC normalization changes should be detected.
 
+        Fullwidth ASCII (e.g. ａｂｃ) always normalizes under NFKC, so the
+        validator must flag it unconditionally.
+        """
+        # Fullwidth characters always normalize to ASCII under NFKC
+        original = "ａｂｃ"  # Fullwidth ASCII — NFKC maps these to 'abc'
         result = validate_tool_name(original)
-        # Should detect that normalization would change this
-        if original != normalized:
-            assert not result.is_valid or any("normalize" in w.lower() for w in result.warnings), (
-                "NFKC normalization should be detected"
-            )
+        assert not result.is_valid or any("normalize" in w.lower() for w in result.warnings), (
+            "NFKC normalization should be detected"
+        )
 
 
 class TestUnicodeBoundaryConditions: