Skip to content

style: apply ruff formatting across codebase#36

Merged
Coding-Dev-Tools merged 10 commits into
masterfrom
improve/click-to-mcp-20260630
Jul 3, 2026
Merged

style: apply ruff formatting across codebase#36
Coding-Dev-Tools merged 10 commits into
masterfrom
improve/click-to-mcp-20260630

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

Summary

Apply ruff code formatting across the entire codebase. All changes are formatting only — no logic changes.

Changes

  • Reformat multiline function signatures to single-line where they fit
  • Normalize single quotes to double quotes consistently
  • Add blank lines after docstrings (PEP 257)
  • Restructure __all__ list to one item per line
  • Reformat dict literals and click.option decorators

Testing

CI will run test/coverage/security jobs on this branch.

…ify port help text

Reviewer-B identified two issues missed by Reviewer-A:
1. Removed spurious # pragma: allowlist secret annotations from
   non-secret client name strings (claude-desktop, cursor, vscode,
   windsurf, cline) — these are not secrets, and adding annotations
   to non-secrets reduces secret-scanning effectiveness.
2. Clarified port help text to note streamable-http default is 8001.
Reformat multiline function signatures, normalize quotes, add
blank lines after docstrings, and restructure __all__ list.
All changes are formatting only — no logic changes.
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

🤖 Automated Code Review

✅ Ruff Lint — No issues

✅ Ruff Format — Clean

✅ Secret Detection — Clean

✅ Large Files — Within limits

📊 Diff Stats — 24 file(s) changed

 click_to_mcp/__init__.py              | 17 +++---
 click_to_mcp/__main__.py              |  1 +
 click_to_mcp/_version.py              |  1 +
 click_to_mcp/adapter.py               | 10 ++--
 click_to_mcp/cli.py                   | 50 ++++++++++++------
 click_to_mcp/discover.py              | 26 +++++----
 click_to_mcp/http_server.py           | 16 +++---
 click_to_mcp/server.py                | 29 ++++++----
 click_to_mcp/streamable_http.py       | 58 +++++++-------------
 examples/api_contract_guardian_mcp.py | 12 ++++-
 examples/quick_start.py               |  5 +-
 tests/test_cli_config.py              | 63 +++++++++++++++-------
 tests/test_cli_error_paths.py         |  1 +
 tests/test_click_to_mcp.py            | 86 ++++++++++++++++++------------
 tests/test_config.py                  | 15 ++----
 tests/test_entry_points.py            |  5 +-
 tests/test_http_transport.py          | 23 ++++----
 tests/test_init_functions.py          | 16 ++++--
 tests/test_lint_regression.py         |  9 ++--
 tests/test_list_tools.py              | 12 +++--
 tests/test_main.py                    |  3 +-
 tests/test_mcp_edge_cases.py          | 67 +++++++++++++++---------
 tests/test_server_stdio.py            | 99 +++++++++++++++++++++--------------
 tests/test_streamable_http.py         |  4 +-
 24 files changed, 376 insertions(+), 252 deletions(-)

Verdict: ✅ Pass — No issues found.

Automated by Coding-Dev-Tools/.github reusable workflow.

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner Author

[Blocked — conflict after #35] This ruff format PR now conflicts with master in click_to_mcp/server.py after #35 merged. Cleanest fix: rebase onto master and re-run ruff format . rather than hand-merging, so none of #35's changes are dropped. Not auto-resolving.

@Coding-Dev-Tools Coding-Dev-Tools merged commit b856efe into master Jul 3, 2026
7 checks passed
@Coding-Dev-Tools Coding-Dev-Tools deleted the improve/click-to-mcp-20260630 branch July 3, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant