fix(llm_client): ensure description is handled correctly in result

sumansaurabh · sumansaurabh · commit 17ffb10a5018 · 2025-03-07T05:46:22.000+05:30
diff --git a/penify_hook/llm_client.py b/penify_hook/llm_client.py
@@ -140,7 +140,7 @@ def generate_commit_summary(self, diff: str, message: str, generate_description:
                         "description": description
                     }
             
-            if not generate_description:
+            if not generate_description and 'description' in result:
                 # If description is missing and user requested it, add a placeholder
                 del result['description']
             return result
@@ -150,5 +150,4 @@ def generate_commit_summary(self, diff: str, message: str, generate_description:
             print(f"Error generating commit summary with LLM: {e}")
             return {
                 "title": "Update code",
-                "description": f"Changes were made to the repository.\n\nUser message: {message}"
             }

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,7 @@ def generate_commit_summary(self, diff: str, message: str, generate_description:`
`140`	`140`	`"description": description`
`141`	`141`	`}`
`142`	`142`
`143`		`- if not generate_description:`
	`143`	`+ if not generate_description and 'description' in result:`
`144`	`144`	`# If description is missing and user requested it, add a placeholder`
`145`	`145`	`del result['description']`
`146`	`146`	`return result`
`@@ -150,5 +150,4 @@ def generate_commit_summary(self, diff: str, message: str, generate_description:`
`150`	`150`	`print(f"Error generating commit summary with LLM: {e}")`
`151`	`151`	`return {`
`152`	`152`	`"title": "Update code",`
`153`		`- "description": f"Changes were made to the repository.\n\nUser message: {message}"`
`154`	`153`	`}`