Skip to content

Commit 7c5a7b3

Browse files
Revert object mapper
1 parent a9989e4 commit 7c5a7b3

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

docs/release-notes/release_notes.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212

1313
### ✨ New Functionality
1414

15-
-
15+
- [Add Orchestration `LlamaGuardFilter`](../guides/ORCHESTRATION_CHAT_COMPLETION.md#chat-completion-filter).
1616

1717
### 📈 Improvements
1818

orchestration/src/main/java/com/sap/ai/sdk/orchestration/OrchestrationJacksonConfiguration.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@
22

33
import static com.sap.ai.sdk.core.JacksonConfiguration.getDefaultObjectMapper;
44

5-
import com.fasterxml.jackson.annotation.JsonAutoDetect.Visibility;
6-
import com.fasterxml.jackson.annotation.PropertyAccessor;
75
import com.fasterxml.jackson.databind.ObjectMapper;
86
import com.fasterxml.jackson.databind.module.SimpleModule;
97
import com.google.common.annotations.Beta;
@@ -51,7 +49,6 @@ public static ObjectMapper getOrchestrationObjectMapper() {
5149
PolymorphicFallbackDeserializer.fromJsonSubTypes(ChatMessage.class))
5250
.setMixInAnnotation(ChatMessage.class, JacksonMixins.NoneTypeInfoMixin.class);
5351
jackson.registerModule(module);
54-
jackson.setVisibility(PropertyAccessor.FIELD, Visibility.ANY);
5552
return jackson;
5653
}
5754
}

sample-code/spring-app/src/main/java/com/sap/ai/sdk/app/Application.java

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,10 @@
11
package com.sap.ai.sdk.app;
22

3+
import static com.sap.ai.sdk.core.JacksonConfiguration.getDefaultObjectMapper;
4+
5+
import com.fasterxml.jackson.annotation.JsonAutoDetect.Visibility;
6+
import com.fasterxml.jackson.annotation.PropertyAccessor;
37
import com.fasterxml.jackson.databind.ObjectMapper;
4-
import com.sap.ai.sdk.orchestration.OrchestrationJacksonConfiguration;
58
import javax.annotation.Nonnull;
69
import org.springframework.boot.SpringApplication;
710
import org.springframework.boot.autoconfigure.SpringBootApplication;
@@ -17,16 +20,16 @@
1720
public class Application {
1821

1922
/**
20-
* Temporary workaround to fix the issue with the Orchestration spec.
23+
* Changes Spring Boot's default object mapper to fix serialization issues.
2124
*
22-
* @return a modified object mapper that works for Orchestration.
25+
* @return a modified object mapper
2326
*/
2427
@Bean
2528
@Primary
2629
@SuppressWarnings("unused")
2730
@Nonnull
2831
public ObjectMapper objectMapper() {
29-
return OrchestrationJacksonConfiguration.getOrchestrationObjectMapper();
32+
return getDefaultObjectMapper().setVisibility(PropertyAccessor.FIELD, Visibility.ANY);
3033
}
3134

3235
/**

0 commit comments

Comments
 (0)