chore: Remove unused imports/variables and improve exception handling clarity

[Copilot]

Addresses code review feedback from PR #7 by removing unused code and improving exception handling documentation.

Changes

Unused imports removed:

• Test files: pytest, math, os from test modules that don't use them
• Type hints: Dict, Any, field from modules that don't reference these types
• Stdlib: dataclass, math, os where not needed

Unused variables removed:

• scatter return values in api/report.py where plot objects weren't used

Exception handling improvements:

• Added explanatory comments to bare pass statements in except blocks
• Fixed variable shadowing where nested exception handlers reused e:

  except ImportError as e:
      # ... outer handling ...
      try:
          # ...
      except ImportError as e2:  # Changed from 'e'
          # ... inner handling ...

Test fixes:

• Defined missing MAX_RESULTS = 5 constant in tests/test_design_builder.py
• Removed unused imports from tests/test_mas_dataclasses.py (6 unused dataclass imports)

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.