Address code review feedback: fix import pollution and add missing UIClassifier enum

[Copilot]

Progress Plan

• Add missing UIClassifier enum to exports
• Remove redundant enum conversion in create_server_config (rely on ServerConfig.init)
• Verify all linting and type checking passes
• Test functionality works correctly

Changes

This PR addresses code review feedback on the enum refactoring by:

1. Replacing wildcard imports with explicit enum re-exports in pyoverkiz/enums/__init__.py

   • Exports all 23 enum classes via explicit imports (including previously missing UIClassifier)
   • Eliminates namespace pollution (StrEnum, Enum, unique, etc. no longer leak into package)
   • Maintains backward compatibility for all import styles

2. Simplifying enum conversion in pyoverkiz/utils.py

   • create_server_config() passes values directly to ServerConfig
   • Relies on ServerConfig.__init__() to handle string → enum conversion internally
   • Uses # type: ignore[arg-type] comments for mypy compatibility

Testing

• ✅ All ruff linting checks pass
• ✅ All mypy type checks pass (27 source files)
• ✅ Functional tests confirm all 23 enums import and work correctly
• ✅ create_server_config() properly converts strings to enums via ServerConfig.__init__()
• ✅ No namespace pollution

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• astral.sh
  • Triggering command: /usr/bin/curl curl -LsSf REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[Copilot]

Pull request overview

Refactors enum exports and server config creation to address review feedback by reducing namespace pollution from wildcard imports and removing duplicated enum coercion logic.

Changes:

• Replaced pyoverkiz/enums/__init__.py wildcard imports with explicit enum re-exports via a fixed __all__ list.
• Simplified create_server_config() in pyoverkiz/utils.py by removing local string→enum conversion and adding type: ignore markers.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
pyoverkiz/utils.py	Removes local enum coercion when constructing ServerConfig.
pyoverkiz/enums/init.py	Switches to explicit re-exports to prevent leaking helper/imported names into pyoverkiz.enums.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.