chore: make Resource.uri and Resource.name required

Author: LucaButBoring

mcp/src/main/java/io/modelcontextprotocol/spec/McpSchema.java

@@ -550,6 +550,14 @@ public Builder annotations(Annotations annotations) {
 			}
 
 			public Resource build() {
+				if (uri == null) {
+					throw new IllegalStateException("uri cannot be null");
+				}
+
+				if (name == null) {
+					throw new IllegalStateException("name cannot be null");
+				}
+
 				return new Resource(uri, name, description, mimeType, size, annotations);
 			}
 		}

mcp/src/test/java/io/modelcontextprotocol/spec/McpSchemaTests.java

@@ -318,6 +318,36 @@ void testResourceBuilder() throws Exception {
 							{"uri":"resource://test","name":"Test Resource","description":"A test resource","mimeType":"text/plain","size":256,"annotations":{"audience":["user","assistant"],"priority":0.8}}"""));
 	}
 
+	@Test
+	void testResourceBuilderUriRequired() {
+		McpSchema.Annotations annotations = new McpSchema.Annotations(
+				Arrays.asList(McpSchema.Role.USER, McpSchema.Role.ASSISTANT), 0.8);
+
+		McpSchema.Resource.Builder resourceBuilder = McpSchema.Resource.builder()
+			.name("Test Resource")
+			.description("A test resource")
+			.mimeType("text/plain")
+			.size(256L)
+			.annotations(annotations);
+
+		assertThatThrownBy(resourceBuilder::build).isInstanceOf(IllegalStateException.class);
+	}
+
+	@Test
+	void testResourceBuilderNameRequired() {
+		McpSchema.Annotations annotations = new McpSchema.Annotations(
+				Arrays.asList(McpSchema.Role.USER, McpSchema.Role.ASSISTANT), 0.8);
+
+		McpSchema.Resource.Builder resourceBuilder = McpSchema.Resource.builder()
+			.uri("resource://test")
+			.description("A test resource")
+			.mimeType("text/plain")
+			.size(256L)
+			.annotations(annotations);
+
+		assertThatThrownBy(resourceBuilder::build).isInstanceOf(IllegalStateException.class);
+	}
+
 	@Test
 	void testResourceTemplate() throws Exception {
 		McpSchema.Annotations annotations = new McpSchema.Annotations(Arrays.asList(McpSchema.Role.USER), 0.5);