Skip to content

Conversation

@chr-hertel
Copy link
Member

@chr-hertel chr-hertel commented Dec 12, 2025

Q A
Bug fix? no
New feature? no
Docs? yes
Issues
License MIT

Follows #1017 - cc @wachterjohannes makes sense?

@chr-hertel chr-hertel added Documentation Improvements or additions to documentation Platform Issues & PRs about the AI Platform component labels Dec 12, 2025
@carsonbot carsonbot changed the title [Platform][Docs] Add docs for message templates [Documentation][Platform][Docs] Add docs for message templates Dec 12, 2025
@chr-hertel chr-hertel requested a review from Copilot December 12, 2025 23:19
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds comprehensive documentation for message templates in the Symfony AI Platform component. The documentation explains how to use dynamic variable substitution in messages for creating reusable message patterns.

Key Changes:

  • Added documentation for string templates with curly brace placeholders
  • Documented expression templates for advanced dynamic evaluation
  • Included setup instructions for manual configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@OskarStark
Copy link
Contributor

I think we should add an example how you can configure templates in ai bundle config

@wachterjohannes
Copy link
Contributor

@OskarStark in the ai-bundle you can just use it. but documenting how that looks like would make sense!

@chr-hertel should we document how the listener has to be configured when no ai-bundle is used? or link to the exanple?

@OskarStark
Copy link
Contributor

Yes let's be explicit and explain it in the docs

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

Labels

Documentation Improvements or additions to documentation Platform Issues & PRs about the AI Platform component Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants