Skip to content

[CONTENT] Add RA for code-based agents#728

Draft
seeobjectively wants to merge 13 commits intoSAP:devfrom
seeobjectively:RA-0005-11-code-based-agents
Draft

[CONTENT] Add RA for code-based agents#728
seeobjectively wants to merge 13 commits intoSAP:devfrom
seeobjectively:RA-0005-11-code-based-agents

Conversation

@seeobjectively
Copy link
Copy Markdown
Contributor

@seeobjectively seeobjectively commented Nov 25, 2025

What reference architecture does this PR apply to?

RA0005-5-1

Who should review your contribution? (Use @mention)

@jmsrpp

Checklist before submitting

  • My commits are only for the reference architecture mentioned above.
  • I have followed the folder structure in the main README

@seeobjectively seeobjectively requested a review from a team as a code owner November 25, 2025 19:31
@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented Nov 25, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions
Copy link
Copy Markdown

Preview website is available here.

@seeobjectively seeobjectively changed the title [CONTENT] Add RA for content-based agents [CONTENT] Add RA for code-based agents Nov 25, 2025
@github-actions
Copy link
Copy Markdown

Preview website is available here.

@github-actions
Copy link
Copy Markdown

Preview website is available here.

@cernus76 cernus76 added ready for review This need to be reviewed reference-architecture Content contributions for new or updated reference architectures content labels Nov 27, 2025
@github-actions
Copy link
Copy Markdown

Preview website is available here.

@kay-schmitteckert
Copy link
Copy Markdown
Contributor

@cernus76, @jmsrpp @seeobjectively, this topic is already covered at https://architecture.learning.sap.com/docs/ref-arch/e5eb3b9b1d/5 and updated with my latest PR #733.

@seeobjectively, please merge your content into the existing RA

@seeobjectively
Copy link
Copy Markdown
Contributor Author

seeobjectively commented Dec 2, 2025

@kay-schmitteckert I've switched to the subpage 'deep dive' approach we discussed, I expect the this page will grow as more partnerships/patterns emerge. I've also added a link on the parent page to navigate for more details that looks like this -

For a detailed guide on building code-based agents, including deployment approaches with Kyma Runtime, LLM access patterns using LiteLLM, and framework-specific implementation details, see Building Code-Based Agents.

Let me know what you think.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 2, 2025

Preview website is available here.

1 similar comment
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 2, 2025

Preview website is available here.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 4, 2025

Preview website is available here.

@seeobjectively
Copy link
Copy Markdown
Contributor Author

@jmsrpp, I heard back from @kay-schmitteckert on this and he's okay with us going ahead with this. He's eventually going to refactor all the GenAI/Agents pages. Could you take a look at this change?

@cernus76
Copy link
Copy Markdown
Contributor

cernus76 commented Dec 4, 2025

Side note: I discussed this with Kay, and we might move the content to a new RA focused on agents.

@cernus76 cernus76 added change requested and removed ready for review This need to be reviewed labels Dec 5, 2025
@navyakhurana
Copy link
Copy Markdown
Contributor

Hi @seeobjectively,
Thank you for the changes. Just a request, as per TechEd it was quoted as Pro code agents rather than Code based agents.
Can we make the change to this RA? Have aligned with @kay-schmitteckert on the naming convention.

Let me know what you think.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 5, 2025

Preview website is available here.

@seeobjectively
Copy link
Copy Markdown
Contributor Author

Thanks for letting me know, @navyakhurana! We're looking at making a few more changes/additions to this shortly, so I'll push everything together.

@cernus76
Copy link
Copy Markdown
Contributor

Putting this PR in draft status. Once it's ready for review, please add the "ready for review" label.

@cernus76 cernus76 marked this pull request as draft December 17, 2025 14:20
@sap-email-compliance
Copy link
Copy Markdown

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.

@github-actions
Copy link
Copy Markdown

Preview website is available here.

@cernus76
Copy link
Copy Markdown
Contributor

@seeobjectively @jmsrpp Any updates on that one?

@cernus76
Copy link
Copy Markdown
Contributor

Please let me know once the content review is done. I've marked it as Draft but you can switch back to Ready for review (please add the corresponding label when ready). Thanks!

@github-actions
Copy link
Copy Markdown

Preview website is available here.

@github-actions
Copy link
Copy Markdown

Preview website is available here.

@seeobjectively
Copy link
Copy Markdown
Contributor Author

here

Thanks for checking, @cernus76 - this was on hold because we had higher priority topics come in, we'll resume working on it shortly.

@github-actions
Copy link
Copy Markdown

Preview website is available here.

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

Labels

change requested content do not merge reference-architecture Content contributions for new or updated reference architectures

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants