Fix linting issues: remove unused imports and variables

Author: claude

cli/xml_lib/assertions.py

@@ -4,7 +4,7 @@
 import json
 from datetime import datetime
 from pathlib import Path
-from typing import List, Optional, TYPE_CHECKING
+from typing import List, TYPE_CHECKING
 from lxml import etree
 from cryptography.hazmat.primitives import hashes, serialization
 from cryptography.hazmat.primitives.asymmetric import padding, rsa

cli/xml_lib/guardrails.py

@@ -2,7 +2,6 @@
 
 import re
 from dataclasses import dataclass, field
-from datetime import datetime
 from pathlib import Path
 from typing import Any, Dict, List, Optional
 from lxml import etree
@@ -127,7 +126,7 @@ def validate(self, project_path: Path) -> GuardrailResult:
                     violations = self._check_rule(doc, xml_file, rule)
                     result.errors.extend(violations)
 
-                except Exception as e:
+                except Exception:
                     # Skip files that can't be parsed
                     continue
 
@@ -155,7 +154,7 @@ def _check_rule(
                 # Cross-file checks require multiple documents - skip for now
                 pass
 
-        except Exception as e:
+        except Exception:
             # Rule execution failed - log but don't fail validation
             pass
 
@@ -195,7 +194,7 @@ def _check_xpath(
                     )
                 )
 
-        except Exception as e:
+        except Exception:
             # XPath evaluation failed
             pass
 

cli/xml_lib/pptx_composer.py

@@ -1,13 +1,11 @@
 """OOXML composer that maps XML guidance to PowerPoint presentations."""
 
 from dataclasses import dataclass
-from datetime import datetime
 from pathlib import Path
-from typing import Dict, List, Optional
+from typing import List, Optional
 from lxml import etree
 from pptx import Presentation
-from pptx.util import Inches, Pt
-from pptx.enum.text import PP_ALIGN
+from pptx.util import Pt
 
 from xml_lib.telemetry import TelemetrySink
 
@@ -193,7 +191,7 @@ def _render_summary(self, summary: etree._Element, prs: Presentation) -> None:
         next_action = summary.find("next-action")
         if next_action is not None and next_action.text:
             content_lines.append("")
-            content_lines.append(f"Next Action:")
+            content_lines.append("Next Action:")
             content_lines.append(f"  {next_action.text}")
 
         # Set body text

cli/xml_lib/telemetry.py

@@ -3,10 +3,10 @@
 import json
 import sqlite3
 from abc import ABC, abstractmethod
-from dataclasses import asdict, dataclass
+from dataclasses import dataclass
 from datetime import datetime
 from pathlib import Path
-from typing import Any, Dict, Optional
+from typing import Any, Dict
 
 try:
     import psycopg2

cli/xml_lib/validator.py

@@ -172,7 +172,7 @@ def _validate_relaxng(
 
         try:
             self.relaxng_lifecycle.assertValid(doc)
-        except etree.DocumentInvalid as e:
+        except etree.DocumentInvalid:
             # Try guardrails schema if lifecycle fails
             if self.relaxng_guardrails:
                 try: