Fix/update deepseek not support response format (#45)

* fix: fix deepseek llm not support response_format type

* delete comment

* Update llm_request_handler.py with Lint and bug fixes

The condition if self.response_schema and not any(...) has three outcomes folded into two branches:
response_schema exists AND model is not deepseek --> json_schema (correct)
response_schema exists AND model is deepseek --> falls to else, gets json_object (correct)
response_schema is None (any model) --> falls to else, gets json_object (BUG -- previously no response_format was set at all)
The fix: the else with json_object should only apply when we have a schema but the provider doesn't support json_schema. When there's no schema, we should skip response_format entirely (original behavior).

* Update llm_request_handler.py with lint issues

---------

Co-authored-by: pengzy2 <pengzy2@knownsec.com>
Co-authored-by: Vineeth Sai Narajala <vnarajal@cisco.com>

Author: 3 people

skill_scanner/core/analyzers/llm_request_handler.py

@@ -211,14 +211,19 @@ async def _make_litellm_request(self, messages: list[dict[str, str]], context: s
                 # Format: response_format={ "type": "json_schema", "json_schema": { "name": "...", "schema": {...}, "strict": true } }
                 # Works for: OpenAI, Anthropic Claude, Gemini (via LiteLLM), Bedrock, Vertex AI, Groq, Ollama, Databricks
                 if self.response_schema:
-                    request_params["response_format"] = {
-                        "type": "json_schema",
-                        "json_schema": {
-                            "name": "security_analysis_response",
-                            "schema": self.response_schema,
-                            "strict": True,  # Enforce strict schema compliance - prevents extra fields
-                        },
-                    }
+                    model_lower = self.provider_config.model.lower()
+                    unsupported_json_schema_providers = ["deepseek"]
+                    if any(p in model_lower for p in unsupported_json_schema_providers):
+                        request_params["response_format"] = {"type": "json_object"}
+                    else:
+                        request_params["response_format"] = {
+                            "type": "json_schema",
+                            "json_schema": {
+                                "name": "security_analysis_response",
+                                "schema": self.response_schema,
+                                "strict": True,
+                            },
+                        }
 
                 response = await acompletion(**request_params)
                 content: str = response.choices[0].message.content or ""