Skip to content

fix: return null when llm return null#3475

Merged
ulemons merged 4 commits intomainfrom
fix/skip-llm-when-disabled
Oct 7, 2025
Merged

fix: return null when llm return null#3475
ulemons merged 4 commits intomainfrom
fix/skip-llm-when-disabled

Conversation

@ulemons
Copy link
Copy Markdown
Contributor

@ulemons ulemons commented Oct 7, 2025

Changes proposed ✍️

What

add check on all the functions which use the queryLlm function. If the result is null we avoid to parse the json.
So we avoid to throw error when the IS_LLM_ENABLED is false

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.

@ulemons ulemons requested a review from themarolt October 7, 2025 08:28
@ulemons ulemons self-assigned this Oct 7, 2025
@ulemons ulemons added the Bug Created by Linear-GitHub Sync label Oct 7, 2025
@ulemons ulemons merged commit 260b6e5 into main Oct 7, 2025
14 checks passed
@ulemons ulemons deleted the fix/skip-llm-when-disabled branch October 7, 2025 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants