chore(release): v1.1.33#117
Conversation
Updated `isValidMessage` in `src/index.js` to allow `msg.content` to be an array, enabling full compatibility with OpenAI multimodal API payloads (text + image content objects). Added specific testing to `api.test.js` to guarantee no regressions in standard payload parsing. Co-authored-by: shenald-dev <245350826+shenald-dev@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
@shenald-dev this PR has merge conflicts with To resolve manually: Alternatively, you can try updating your branch with the latest from |
|
Please resolve the merge conflicts with master as indicated. After resolving, ensure CI passes again. Consider adding a test case for multimodal content arrays to validate the new behavior. The changelog entry looks appropriate for this patch release. Reviewed — quality: medium |
Updated `isValidMessage` in `src/index.js` to allow `msg.content` to be an array, enabling full compatibility with OpenAI multimodal API payloads (text + image content objects). Added specific testing to `api.test.js` to guarantee no regressions in standard payload parsing. Co-authored-by: shenald-dev <245350826+shenald-dev@users.noreply.github.com>
|
🔄 Re-evaluating based on recent changes: Please resolve the merge conflicts with master as indicated, then re-run CI. After resolving, consider adding a test case that explicitly verifies Reviewed — quality: medium |
I have successfully resolved the merge conflicts and pushed the required code changes to the current working branch |
|
🔄 Re-evaluating based on recent changes: Please resolve the merge conflicts with master as indicated, then re-run CI. After resolving, consider adding a test case that explicitly verifies Reviewed — quality: medium |
The merge conflicts have already been resolved. Additionally, I added the requested explicit test case |
|
@shenald-dev this PR has merge conflicts with To resolve manually: Alternatively, you can try updating your branch with the latest from |
|
Please resolve the merge conflicts with master as indicated, then re-run CI. After resolving, consider adding a test case that explicitly verifies Reviewed — quality: medium |
Co-authored-by: shenald-dev <245350826+shenald-dev@users.noreply.github.com>
|
🔄 Re-evaluating based on recent changes: Please resolve the merge conflicts with master as indicated, then re-run CI. After resolving, consider adding a test case that explicitly verifies Reviewed — quality: medium |
I have successfully resolved the merge conflicts that were introduced by another commit landing in |
AI-assisted conflict resolution — merged changes from master into chore/release-v1.1.33-2672449651914561363 (3-way merge with ancestor context)
|
@shenald-dev I went through the conflicts and resolved them — merged the changes from ⏳ CI needs to run on the new commits before merging. Will auto-merge on the next cycle if checks pass. 🔧 Used 3-way merge with ancestor context for more accurate resolution. 📦 A backup ref was created: Take a look and let me know if anything looks off 👍 |
AI-assisted conflict resolution — merged changes from master into chore/release-v1.1.33-2672449651914561363 (3-way merge with ancestor context)
|
@shenald-dev I went through the conflicts and resolved them — merged the changes from ⏳ CI needs to run on the new commits before merging. Will auto-merge on the next cycle if checks pass. 🔧 Used 3-way merge with ancestor context for more accurate resolution. 📦 A backup ref was created: Take a look and let me know if anything looks off 👍 |
Updated
isValidMessageAPI validation insrc/index.jsto acceptmsg.contentarrays to correctly support OpenAI multimodal payload schemas. The repository version is bumped to1.1.33. Zero dead code was pruned.PR created automatically by Jules for task 2672449651914561363 started by @shenald-dev