Skip to content

fix: correct install advice, remove false PyPI badge, fix module_path doubling#44

Merged
Coding-Dev-Tools merged 4 commits into
mainfrom
cowork/improve-deploydiff-install-advice
Jul 21, 2026
Merged

fix: correct install advice, remove false PyPI badge, fix module_path doubling#44
Coding-Dev-Tools merged 4 commits into
mainfrom
cowork/improve-deploydiff-install-advice

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

Changes

  1. cli.py + mcp_server.py: Correct click-to-mcp install advice to working git+ URL (click-to-mcp is not on PyPI)
  2. README.md: Remove false PyPI badge, use git+ install instructions, add honesty note
  3. AGENTS.md: Fix broken bare pip install (use self-hosted index/git+)
  4. diff_renderer.py: Remove module_path doubling in rendered address
  5. tests/test_deploydiff.py: Add 3 edge-case tests for renderer fix

All commits reviewed and approved by council_gate_code_edit.

senior-dev-rotation-B and others added 4 commits July 16, 2026 01:06
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).
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@github-actions

Copy link
Copy Markdown

🤖 Automated Code Review

✅ Ruff Lint — No issues

⚠️ Ruff Format — Formatting needed

Would reformat: src/deploydiff/cli.py
Would reformat: src/deploydiff/cloudformation_parser.py
Would reformat: src/deploydiff/cost_estimator.py
Would reformat: src/deploydiff/diff_renderer.py
Would reformat: src/deploydiff/pulumi_parser.py
Would reformat: src/deploydiff/rollback.py
Would reformat: src/deploydiff/terraform_parser.py
Would reformat: tests/test_deploydiff.py
Would reformat: tests/test_edge_cases.py
9 files would be reformatted, 7 files already formatted

✅ Secret Detection — Clean

✅ Large Files — Within limits

📊 Diff Stats — 4 file(s) changed

 AGENTS.md                       |  5 ++-
 README.md                       |  7 +---
 src/deploydiff/diff_renderer.py |  2 -
 tests/test_deploydiff.py        | 85 ++++++++++++++++++++++++++++++++++++++++-
 4 files changed, 90 insertions(+), 9 deletions(-)

Verdict: ⚠️ Warnings — Lint/format issues found. Recommend fixing before merge.

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

@Coding-Dev-Tools
Coding-Dev-Tools merged commit d3b7240 into main Jul 21, 2026
5 of 6 checks passed
@Coding-Dev-Tools
Coding-Dev-Tools deleted the cowork/improve-deploydiff-install-advice branch July 21, 2026 04:17
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