Fix Pylint CI score to meet 8.0+ requirement

[Copilot]

Pylint CI was failing with a score of 7.84/10, below the required 8.0 threshold.

Changes

Fixed 380+ Pylint violations across 8 files:

Logging performance (213 fixes)

• Converted f-string interpolation to lazy % formatting in all logging calls

# Before
logger.info(f"Connected to {host.name}")

# After  
logger.info("Connected to %s", host.name)

Code clarity (71 fixes)

• Reordered imports (standard library before third-party)
• Removed unused imports and variables (prefixed with _ where needed)
• Removed unnecessary else blocks after return statements

Exception handling (4 fixes)

• Added proper exception chaining with from e to preserve stack traces

Minor fixes

• Split lines exceeding 100 characters
• Added check=False to subprocess.run() calls
• Used sys.exit() instead of raise SystemExit for consistency

Final score: 8.27/10 | No functionality changes | CodeQL: 0 vulnerabilities

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Fix for Pylint CI</issue_title>
<issue_description>We need an 8.0 or better for the CI job (and, y'know, code readability/maintainability), please fix without changing functionality</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Fix for Pylint CI #36

---

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

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}