cleanup: Removed redundant SOURCE_METADATA assignments in TextReader and unwrapped some unnecessary conditional checks. (#3627)

Fixes #3627

Signed-off-by: Sun Yuhan <[email protected]>
Co-authored-by: Sun Yuhan <[email protected]>
(cherry picked from commit c1ee22d)

Author: sunyuhan1998

spring-ai-commons/src/main/java/org/springframework/ai/reader/TextReader.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2023-2024 the original author or authors.
+ * Copyright 2023-2025 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -90,7 +90,6 @@ public List<Document> get() {
 
 			// Inject source information as a metadata.
 			this.customMetadata.put(CHARSET_METADATA, this.charset.name());
-			this.customMetadata.put(SOURCE_METADATA, this.resource.getFilename());
 			this.customMetadata.put(SOURCE_METADATA, getResourceIdentifier(this.resource));
 
 			return List.of(new Document(document, this.customMetadata));
@@ -111,9 +110,7 @@ protected String getResourceIdentifier(Resource resource) {
 		// Try to get the URI
 		try {
 			URI uri = resource.getURI();
-			if (uri != null) {
-				return uri.toString();
-			}
+			return uri.toString();
 		}
 		catch (IOException ignored) {
 			// If getURI() throws an exception, we'll try the next method
@@ -122,9 +119,7 @@ protected String getResourceIdentifier(Resource resource) {
 		// Try to get the URL
 		try {
 			URL url = resource.getURL();
-			if (url != null) {
-				return url.toString();
-			}
+			return url.toString();
 		}
 		catch (IOException ignored) {
 			// If getURL() throws an exception, we'll fall back to getDescription()