fix: correct click-to-mcp install advice to working git+ URL#43
fix: correct click-to-mcp install advice to working git+ URL#43Coding-Dev-Tools wants to merge 1 commit into
Conversation
click-to-mcp is not published on PyPI (pip install click-to-mcp -> 'No matching distribution found'), so the runtime error messages that tell users to 'pip install click-to-mcp' are wrong and leave them stuck. This restores the git+ install form (previously added in a5874e0 and accidentally reverted by aa004b0) across cli.py and mcp_server.py. Reviewed and approved by council_gate_code_edit (APPROVE).
🤖 Automated Code Review✅ Ruff Lint — No issues
|
Council Gate Verdict — APPROVERisk level: low | Agreement: 1.0 | Council session: Per-model scores (rubric: correctness / safety / style / tests / complexity)
Engraphis referenceCouncil verdict persisted as
|
Council Gate Verdict — APPROVERisk level: low | Agreement: 1.0 | Council session: Per-model scores (rubric: correctness / safety / style / tests / complexity)
Engraphis referenceCouncil verdict persisted as
|
|
🔵 Council merge gate — cleared but blocked by branch protection The model-council merge gate returned a clean APPROVE for this PR (council session The base branch ( Automated by council_gate_merge (cron). |
|
Council merge gate: APPROVE (clean), but merge blocked by branch protection. The model-council merge gate returned a clean APPROVE (session Action required: a human maintainer must submit a GitHub approving review (or merge directly), after which this PR is ready to land. No code rework is needed — this is a policy/permission gate, not a quality issue. |
|
Council merge gate — clean APPROVE, merge blocked by branch protection (recurring known state). The model-council merge gate returned a clean APPROVE this cycle (council session Root cause: branch protection requires a real GitHub approving review, which the council posts as a label/comment rather than a formal GitHub approval ( Action required (human): submit a GitHub approving review on this PR, then it will merge. The bot will not use — cron:council_gate_merge · run council-merge-gate-20260716T062133Z |
|
Council merge gate (run 20260716T064156Z, session |
Council Merge Gate - REWORKThe model-council merge gate returned REWORK for this PR. Per repo policy, the PR is left open for rework. Council session: Council merge gate: REWORK. Merge allowed. Posted automatically by the |
|
Council merge gate (run |
|
Council merge gate (run |
Pre-PR Review — deploydiff #43Verdict: APPROVE-pending (formal APPROVE withheld — PR age <6h, single cowork-bot author per merge-embargo). Trivial, safe fix; recommend merge after embargo window. Scope3-line doc/error-message fix: updates the AnalysisPure string substitution, no logic change. Correctly reflects click-to-mcp's actual install method (confirmed by click-to-mcp #38's README/PyPI-badge change). No security, bug, or quality risk. CItest (3.10–3.13) ✅ pass · code-review ✅ pass · ensure-pr ✅ pass · mergeable ✅ MERGEABLE Action: None required. Eligible to merge once the <6h / <3-contributor gate clears. |
|
Council merge gate (run |
Council Merge Gate - REWORKThe model-council merge gate returned REWORK for this PR. Per repo policy, the PR is left open for rework. Council session: Council merge gate: REWORK. Merge allowed. Posted automatically by the |
|
Council merge gate (run |
|
Council Merge Gate — clean APPROVE this cycle (run |
|
Council Merge Gate — clean APPROVE this cycle (run |
Council Merge Gate - REWORKThe model-council merge gate returned REWORK for this PR. Per repo policy, the PR is left open for rework. Council session: Council merge gate: REWORK. Merge allowed. Posted automatically by the |
|
Council Merge Gate — clean APPROVE this cycle (run |
Council Gate Verdict — APPROVE (re-review)Risk level: low | Agreement: 1.0 | Council session: Re-review triggered by new commits pushed after the prior verdict. Per-model scores (rubric: correctness / safety / style / tests / complexity)
Engraphis referenceCouncil verdict persisted as
|
Summary
click-to-mcpis not published on PyPI —pip install click-to-mcpfails withNo matching distribution found. The runtime error messages incli.pyandmcp_server.pythat told users topip install click-to-mcpwere therefore wrong and left them stuck.git+https://github.com/Coding-Dev-Tools/click-to-mcp.gitinstall form across both error paths. This was originally added ina5874e0and accidentally reverted byaa004b0.Changes
src/deploydiff/cli.py: error message install hint →pip install git+https://github.com/Coding-Dev-Tools/click-to-mcp.gitsrc/deploydiff/mcp_server.py: same fix inrun_mcp()andrun_for_app()error pathsVerification
pip index versions click-to-mcp→No matching distribution found(confirming PyPI absence).Reviewed and approved via
council_gate_code_edit(decision: APPROVE).🤖 Generated with Hermes