## Description The systemMessages field doesn't have a default value in the action configuration when a new OpenAI Vision command query is created. Because systemMessages is not a mandatory field, we can omit the null check. This pull request removes the unnecessary null check that was added. Fixes https://github.com/appsmithorg/appsmith/issues/39395 ## Automation /ok-to-test tags="@tag.All" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!IMPORTANT] > 🟣 🟣 🟣 Your tests are running. > Tests running at: <https://github.com/appsmithorg/appsmith/actions/runs/13559573780> > Commit: 0635d47845106232fd1867a4889975aa2c6802e9 > Workflow: `PR Automation test suite` > Tags: `@tag.All` > Spec: `` > <hr>Thu, 27 Feb 2025 05:28:27 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Bug Fixes** - Improved the stability of message handling by ensuring that system messages are only processed when available, preventing potential errors when data is missing. <!-- end of auto-generated comment: release notes by coderabbit.ai --> |
||
|---|---|---|
| .. | ||
| client | ||
| server | ||
| util | ||