fix(models): Prevent trailing newlines in generated SQL queries

[pankaj-bind]

The should query model test case in tests/models/modelsRestApiClient.test.ts was failing due to a strict equality check on the generated SQL query. The actual query contained a trailing newline character that was not present in the expected output.

Root Cause:
The queryModel, batchQueryModel, and finetuneModel methods in src/models/modelsRestApiClient.ts constructed SQL queries by joining an array of clauses with \n. If an optional clause (e.g., the USING or LIMIT clause) was empty, the .join('\n') method would still append a newline separator for that empty item, resulting in an incorrect trailing newline at the end of the final query string.

To fix this, a .filter(Boolean) has been added to the array of clauses in all three affected methods (queryModel, batchQueryModel, and finetuneModel). This removes any empty or falsy strings from the array before they are joined, ensuring that no unnecessary newlines are appended.

npm test -- tests/models/modelsRestApiClient.test.ts

This change aligns these methods with the pattern already used in trainModel and retrainModel in the same file, ensuring consistent and correct SQL query generation across the module.

[entelligence-ai-pr-reviews]

Unable to Process PR Review

The author of this PR does not exist on Entelligence Dashboard. Please add the user to Entelligence AI here to enable reviews for this user.

[github-actions]

Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

---

I have read the CLA Document and I hereby sign the CLA

---

_{You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.}