fix: clean up misc_tools_parser.py

[Deez-Automations]

Fixes #814

Few small things I noticed in misc_tools_parser.py:

• NamedTuple was imported but never used anywhere in the file, removed it
• boolean from xmlrpc.client is not the right type for a simple flag param, swapped it for Python's built-in bool
• Typo in the parser name: miscelaneous -> miscellaneous
• logging.error() was being called directly instead of going through the module-level logger — standardized it
• print() inside the loop replaced with logger.info() to match the rest of the codebase
• Added List[defs.Tool] type annotation to parse_tool return type and tool_entries variable, consistent with how other parsers in this directory are typed

No logic changes, just cleanup.

[Copilot]

Pull request overview

Cleanup-focused PR for misc_tools_parser.py (Fixes #814) to improve typing correctness, naming, and logging consistency in the external project parser.

Changes:

• Remove unused imports and replace xmlrpc.client.boolean with built-in bool.
• Fix parser name typo (miscelaneous → miscellaneous).
• Standardize logging by using the module-level logger and replacing print() with logger.info().
• Add List[defs.Tool] return/type annotations for parse_tool and tool_entries.

---

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

[Copilot]

Pull request overview

Cleans up misc_tools_parser.py to improve typing accuracy, fix a parser display-name typo, and make logging consistent with other external project parsers (per #814).

Changes:

• Remove unused imports and replace xmlrpc.client.boolean with built-in bool for dry_run.
• Fix parser name typo (miscelaneous → miscellaneous) and standardize logging (logging.error/print → module logger).
• Add List[defs.Tool] type annotations for parse_tool and tool_entries.

---

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

[Deez-Automations]

Addressed the review comments

• fixed the dry_run early return to avoid UnboundLocalError
• added an empty list check in parse() to prevent IndexError
• improved the error message to include the readme path and repo url

[Deez-Automations]

The e2e workflow failure appears to be a workflow file issue not related to this change, this PR only touches misc_tools_parser.py with no changes to tests or CI config. Happy to investigate further if needed.

[Deez-Automations]

Commits are now signed. The only remaining check is the e2e workflow, the entire workflow file is commented out so it fails immediately with 0s runtime. This is pre-existing and unrelated to this change.