bug #582 [AI Bundle] Correct exception message to mention both text and include_tools (OskarStark)

OskarStark · OskarStark · commit 5308d4d4f9be · 2025-09-13T23:07:02.000+02:00
This PR was merged into the main branch. Discussion ---------- [AI Bundle] Correct exception message to mention both `text` and `include_tools` | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | Docs? | no | Issues | -- | License | MIT Commits ------- 2471398 fix [AI Bundle] Correct exception message to mention both 'text' and 'include_tools' options
diff --git a/src/ai-bundle/config/options.php b/src/ai-bundle/config/options.php
@@ -142,7 +142,7 @@
                                 ->ifArray()
                                 ->then(function (array $v) {
                                     if (!isset($v['text']) && !isset($v['include_tools'])) {
-                                        throw new \InvalidArgumentException('Either "text" must be configured for prompt.');
+                                        throw new \InvalidArgumentException('Either "text" or "include_tools" must be configured for prompt.');
                                     }
 
                                     return $v;

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@`
`142`	`142`	`->ifArray()`
`143`	`143`	`->then(function (array $v) {`
`144`	`144`	`if (!isset($v['text']) && !isset($v['include_tools'])) {`
`145`		`- throw new \InvalidArgumentException('Either "text" must be configured for prompt.');`
	`145`	`+ throw new \InvalidArgumentException('Either "text" or "include_tools" must be configured for prompt.');`
`146`	`146`	`}`
`147`	`147`
`148`	`148`	`return $v;`