Skip to content

feat: support using RG scoped prompt templates #722

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
n-o-u-r-h-a-n wants to merge 30 commits into
base: main
Choose a base branch
from
Open

feat: support using RG scoped prompt templates #722

n-o-u-r-h-a-n wants to merge 30 commits into from
+455 −31

Conversation

@n-o-u-r-h-a-n
Copy link
Contributor

@n-o-u-r-h-a-n n-o-u-r-h-a-n commented Jan 21, 2026 •
edited
Loading

Context

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

RG scoped prompt templates are now supported in Orchestration.

Feature scope:

  • Updated TemplateConfig with reference and byId methods for RG enum + changed names of old methods to explicitly be used for the tenant enum only.
  • Updated Orchestration Service class with methods for RG scope.
  • Testing
    • Added Tests in OrchestrationTest and OrchestrationUnitTest classes.
    • Updated testTemplateReferenceConstruction() in OrchestrationConvenienceUnitTest.

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

@sap-email-compliance
Copy link

sap-email-compliance bot commented Jan 21, 2026

SAP employees are expected to use their SAP-email address for commits related to their work. Our compliance check has detected usage of an email other than a SAP one by a SAP employee. Please update your pull request accordingly.

If you think this is wrong or need any assistance, please contact ospo@sap.com.

@n-o-u-r-h-a-n
Copy link
Contributor Author

n-o-u-r-h-a-n commented Jan 21, 2026

Documentation and Release notes are still not yet updated.

CharlesDuboisSAP
CharlesDuboisSAP reviewed Jan 22, 2026
View reviewed changes
Copy link
Contributor

@CharlesDuboisSAP CharlesDuboisSAP left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rpanackal rpanackal rpanackal left review comments

@CharlesDuboisSAP CharlesDuboisSAP Awaiting requested review from CharlesDuboisSAP

@newtork newtork Awaiting requested review from newtork

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@n-o-u-r-h-a-n @newtork @rpanackal @CharlesDuboisSAP @bot-sdk-js