Skip to content

Conversation

rpanackal
Copy link
Member

@rpanackal rpanackal commented Jul 24, 2025

Context

AI/ai-sdk-java-backlog#301.

Orchestration has in plan a partial translation feature. Here I was integrating the changes that will come along.

Note: Until the feature is implemented as prescribed in spec here and available in canary, DO NOT MERGE.
Orchestration Spec PR: https://github.tools.sap/AI/llm-orchestration/pull/1637

Feature scope:

  • Breaking: OrchestrationModuleConfig now has SAPDocumentTranslationInput and SAPDocumentTranslationOutput for configuring input and output translation, replacing SAPDocumentTranslation.
    • Affecting withers withInputTranslationConfig and withOutputTranslationConfig.
  • Translation E2E and unit test update for full config.

Definition of Done

  • Functionality scope stated & covered
  • Tests cover the scope above
  • Error handling created / updated & covered by the tests above
  • Aligned changes with the JavaScript SDK
  • Documentation updated
  • Release notes updated

bot-sdk-js and others added 27 commits May 26, 2025 11:10
…onse-type

# Conflicts:
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/DpiMasking.java
…ffix and Embedding property renaming (#469)

* Introduce filtering schema changes and update generated class names

- synchronous suffix removed
- `createConfig` removed from `ContentFilter`
- release notes updated for filtering changes

* Introduce filtering schema changes and update generated class names

- synchronous suffix removed
- `createConfig` removed from `ContentFilter`
- release notes updated for filtering changes

* Release notes and jacoco coverage work around

* Lower min required jacoco coverage complexity and branch rating.

- Release note paraphrasing

* Update e2e for input filters

---------

Co-authored-by: Roshin Rajan Panackal <[email protected]>
- `ConfigToRequestTransformer` (not fixed)
…onse-type

# Conflicts:
#	docs/release_notes.md
#	orchestration/pom.xml
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/JacksonMixins.java
#	sample-code/spring-app/src/test/java/com/sap/ai/sdk/app/controllers/OrchestrationTest.java
…onse-type' into spec-update/orchestration/fix/validate-v2

# Conflicts:
#	sample-code/spring-app/src/test/java/com/sap/ai/sdk/app/controllers/OrchestrationTest.java
…ate/orchestration/fix/validate-v2

# Conflicts:
#	docs/release_notes.md
#	orchestration/pom.xml
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/ContentFilter.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/OrchestrationChatCompletionDelta.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/OrchestrationChatResponse.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/OrchestrationClient.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/OrchestrationHttpExecutor.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/OrchestrationJacksonConfiguration.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/model/AzureContentSafetyInput.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/model/CompletionPostResponseStreaming.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/model/ErrorResponseStreaming.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/model/LLMModuleResult.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/spring/OrchestrationSpringChatDelta.java
#	orchestration/src/main/java/com/sap/ai/sdk/orchestration/spring/OrchestrationSpringChatResponse.java
#	orchestration/src/main/resources/spec/orchestration.yaml
#	orchestration/src/test/java/com/sap/ai/sdk/orchestration/OrchestrationUnitTest.java
#	sample-code/spring-app/src/test/java/com/sap/ai/sdk/app/controllers/OrchestrationTest.java
Base automatically changed from spec-update/orchestration/fix/validate-v2 to main August 7, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants