Skip to content

refactor: simplify MCP integrations and remove unused code - #11

Merged
Tony363 merged 4 commits into
mainfrom
refactor/mcp-simplification
Dec 16, 2025
Merged

refactor: simplify MCP integrations and remove unused code#11
Tony363 merged 4 commits into
mainfrom
refactor/mcp-simplification

style: apply ruff formatting

ae4dd22
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / CodeQL succeeded Dec 16, 2025 in 3s

5 new alerts

New alerts in code changed by this pull request

  • 3 warnings
  • 2 notes

See annotations below for details.

View all branch alerts.

Annotations

Check warning on line 101 in setup/components/mcp.py

See this annotation in the file changed.

Code scanning / CodeQL

Signature mismatch in overriding method Warning

This method requires 1 positional argument, whereas overridden
Component.install
requires 2.
This call
correctly calls the base method, but does not match the signature of the overriding method.

Check warning on line 131 in setup/components/mcp.py

See this annotation in the file changed.

Code scanning / CodeQL

Signature mismatch in overriding method Warning

This method requires 1 positional argument, whereas overridden
Component.update
requires 2.

Check warning on line 108 in setup/components/mcp_docs.py

See this annotation in the file changed.

Code scanning / CodeQL

Signature mismatch in overriding method Warning

This method requires 1 positional argument, whereas overridden
Component.install
requires 2.
This call
correctly calls the base method, but does not match the signature of the overriding method.

Check notice on line 132 in setup/components/mcp_docs.py

See this annotation in the file changed.

Code scanning / CodeQL

Empty except Note

'except' clause does nothing but pass and there is no explanatory comment.

Check notice on line 850 in setup/utils/security.py

See this annotation in the file changed.

Code scanning / CodeQL

Module is imported more than once Note

This import of module logging is redundant, as it was previously imported
on line 30
.