Skip to content

Conversation

OskarStark
Copy link
Contributor

@OskarStark OskarStark commented Sep 13, 2025

Q A
Bug fix? no
New feature? yes
Docs? yes
Issues Follows #576
License MIT
  • Renamed system_prompt configuration key to prompt for clearer naming
  • Renamed nested 'prompt' key to 'text' in array configuration to avoid confusion
  • Updated all configuration examples in demo
  • Updated documentation with new configuration structure
  • Updated all tests to use new configuration structure
  • Maintained backward compatibility through configuration normalization

I prefer text, as this way, we can introduce file and validate that only one can be set. I am planning to support twig and json prompts in the future.

@OskarStark OskarStark self-assigned this Sep 13, 2025
@OskarStark OskarStark added the BC Break Breaking the Backwards Compatibility Promise label Sep 13, 2025
@carsonbot carsonbot added Status: Needs Review AI Bundle Issues & PRs about the AI integration bundle Feature New feature labels Sep 13, 2025
@OskarStark OskarStark force-pushed the rename-system-prompt-to-prompt branch from 432375b to d74409e Compare September 13, 2025 09:00
- Renamed system_prompt configuration key to prompt for clearer naming
- Renamed nested 'prompt' key to 'text' in array configuration to avoid confusion
- Updated all configuration examples in demo
- Updated documentation with new configuration structure
- Updated all tests to use new configuration structure
- Maintained backward compatibility through configuration normalization
@OskarStark OskarStark force-pushed the rename-system-prompt-to-prompt branch from d74409e to 2220387 Compare September 13, 2025 09:03
@OskarStark OskarStark merged commit c8e0630 into symfony:main Sep 13, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI Bundle Issues & PRs about the AI integration bundle BC Break Breaking the Backwards Compatibility Promise Feature New feature Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants