fix(quality): resolve SonarQube violations

- S1488: return stream expression directly in validateCollectionExists
  instead of assigning to local variable
- S7467: replace unused caught exceptions with _ (Java 25 unnamed
  variables) in CollectionService and SolrConfigUrlNormalizationTest

Signed-off-by: Aditya Parikh <aditya.m.parikh@gmail.com>

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: adityamparikh <aditya.m.parikh@gmail.com>

Author: adityamparikh

src/main/java/org/apache/solr/mcp/server/metadata/CollectionService.java

@@ -361,7 +361,7 @@ public List<String> listCollections() {
 
 				return new ArrayList<>(coreResponse.getCoreStatus().keySet());
 			}
-		} catch (SolrServerException | IOException e) {
+		} catch (SolrServerException | IOException _) {
 			return new ArrayList<>();
 		}
 	}
@@ -604,7 +604,7 @@ public CacheStats getCacheMetrics(String collection) {
 			}
 
 			return stats;
-		} catch (SolrServerException | IOException | RuntimeException e) {
+		} catch (SolrServerException | IOException | RuntimeException _) {
 			// RuntimeException covers SolrException subclasses (e.g. RemoteSolrException)
 			// thrown when the /admin/mbeans endpoint is unavailable (removed in Solr 10).
 			return null; // Return null instead of empty object
@@ -778,7 +778,7 @@ public HandlerStats getHandlerMetrics(String collection) {
 			}
 
 			return stats;
-		} catch (SolrServerException | IOException | RuntimeException e) {
+		} catch (SolrServerException | IOException | RuntimeException _) {
 			// RuntimeException covers SolrException subclasses (e.g. RemoteSolrException)
 			// thrown when the /admin/mbeans endpoint is unavailable (removed in Solr 10).
 			return null; // Return null instead of empty object
@@ -964,10 +964,8 @@ private boolean validateCollectionExists(String collection) {
 			// Check if any of the returned collections start with the collection name (for
 			// shard
 			// names)
-			boolean shardMatch = collections.stream().anyMatch(c -> c.startsWith(collection + SHARD_SUFFIX));
-
-			return shardMatch;
-		} catch (Exception e) {
+			return collections.stream().anyMatch(c -> c.startsWith(collection + SHARD_SUFFIX));
+		} catch (Exception _) {
 			return false;
 		}
 	}

src/test/java/org/apache/solr/mcp/server/config/SolrConfigUrlNormalizationTest.java

@@ -51,7 +51,7 @@ void testUrlNormalization(String inputUrl, String expectedUrl) {
 
 		try {
 			client.close();
-		} catch (Exception e) {
+		} catch (Exception _) {
 			// Ignore close errors in test
 		}
 	}
@@ -68,7 +68,7 @@ void testUrlWithoutTrailingSlash() {
 
 		try {
 			client.close();
-		} catch (Exception e) {
+		} catch (Exception _) {
 			// Ignore close errors in test
 		}
 	}
@@ -85,7 +85,7 @@ void testUrlWithTrailingSlashButNoSolrPath() {
 
 		try {
 			client.close();
-		} catch (Exception e) {
+		} catch (Exception _) {
 			// Ignore close errors in test
 		}
 	}
@@ -102,7 +102,7 @@ void testUrlWithSolrPathButNoTrailingSlash() {
 
 		try {
 			client.close();
-		} catch (Exception e) {
+		} catch (Exception _) {
 			// Ignore close errors in test
 		}
 	}
@@ -119,7 +119,7 @@ void testUrlAlreadyProperlyFormatted() {
 
 		try {
 			client.close();
-		} catch (Exception e) {
+		} catch (Exception _) {
 			// Ignore close errors in test
 		}
 	}