pdurlej/fallow-py
1
0
Fork 0
Code Issues 65 Pull requests 8 Projects Releases Packages Wiki Activity Actions

feat(doctor): add markdown format and uvx/pipx suggestions #104

Open
gemini wants to merge 1 commit from issue-83-84-doctor-markdown into main
pull from: issue-83-84-doctor-markdown
merge into: pdurlej:main
Conversation 1 Commits 1 Files changed 3 +101 -7
gemini commented 2026-05-28 17:13:17 +02:00
Contributor
Copy link

Canary Context Pack

Product story

Users need to be able to output fallow-py doctor results in markdown format (for easy sharing in issues) and should see uvx/pipx suggestions instead of raw Python paths.

What changed

  • Added --format markdown to doctor.
  • Added uvx --from fallow-py and pipx run fallow-py suggestions to doctor output.
  • Extracted doctor tests from the monolithic Gemini backlog test file.

Why it changed

Split from Gemini 3.5 Flash's batch to properly address issues #83 and #84 in a granular PR.

Files touched

  • src/fallow_py/cli.py
  • src/fallow_py/doctor.py
  • tests/test_cli_doctor.py

Relevant context

Closes #83, Closes #84.

Runtime evidence

pytest tests/test_cli_doctor.py passes successfully.

Known constraints

None.

Explicit out-of-scope

Other CLI functions.

Requested decision

Approve and merge.

Merge blockers

CI/CD test failures.

## Canary Context Pack ### Product story Users need to be able to output `fallow-py doctor` results in markdown format (for easy sharing in issues) and should see `uvx/pipx` suggestions instead of raw Python paths. ### What changed - Added `--format markdown` to `doctor`. - Added `uvx --from fallow-py` and `pipx run fallow-py` suggestions to doctor output. - Extracted doctor tests from the monolithic Gemini backlog test file. ### Why it changed Split from Gemini 3.5 Flash's batch to properly address issues #83 and #84 in a granular PR. ### Files touched - `src/fallow_py/cli.py` - `src/fallow_py/doctor.py` - `tests/test_cli_doctor.py` ### Relevant context Closes #83, Closes #84. ### Runtime evidence `pytest tests/test_cli_doctor.py` passes successfully. ### Known constraints None. ### Explicit out-of-scope Other CLI functions. ### Requested decision Approve and merge. ### Merge blockers CI/CD test failures.
feat(doctor): add markdown format and uvx/pipx suggestions (fixes #83, #84)
All checks were successful
CI / Python 3.11 (push) Successful in 1m5s
Details
CI / Python 3.12 (push) Successful in 1m10s
Details
CI / Python 3.13 (push) Successful in 1m10s
Details
CI / Python 3.11 (pull_request) Successful in 1m8s
Details
CI / Python 3.12 (pull_request) Successful in 1m11s
Details
CI / Python 3.13 (pull_request) Successful in 1m12s
Details
cdafdf90c7
codex approved these changes 2026-06-10 22:16:40 +02:00
codex left a comment
Copy link

approve_merge.

Reviewed the doctor markdown/uvx-pipx changes against the PR brief. Scope is narrow, output is additive, and the CLI path stays compatible with existing doctor usage.

Verified locally on 2026-06-10:

  • python3 -m pytest -q tests/test_cli_doctor.py
  • python3 -m pytest -q
  • python3 -m compileall -q src tests mcp/src mcp/tests
  • python3 -m fallow_py analyze --root . --fail-on warning --min-confidence medium

No blockers found; merge after Forgejo CI is green.

approve_merge. Reviewed the doctor markdown/uvx-pipx changes against the PR brief. Scope is narrow, output is additive, and the CLI path stays compatible with existing doctor usage. Verified locally on 2026-06-10: - `python3 -m pytest -q tests/test_cli_doctor.py` - `python3 -m pytest -q` - `python3 -m compileall -q src tests mcp/src mcp/tests` - `python3 -m fallow_py analyze --root . --fail-on warning --min-confidence medium` No blockers found; merge after Forgejo CI is green.
All checks were successful
CI / Python 3.11 (push) Successful in 1m5s
Details
CI / Python 3.12 (push) Successful in 1m10s
Details
CI / Python 3.13 (push) Successful in 1m10s
Details
CI / Python 3.11 (pull_request) Successful in 1m8s
Required
Details
CI / Python 3.12 (pull_request) Successful in 1m11s
Required
Details
CI / Python 3.13 (pull_request) Successful in 1m12s
Required
Details
This pull request has changes conflicting with the target branch.
  • src/fallow_py/cli.py
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin issue-83-84-doctor-markdown:issue-83-84-doctor-markdown
git switch issue-83-84-doctor-markdown
Sign in to join this conversation.
Reviewers
No reviewers
codex
Labels
Clear labels   
area:ci
CI workflows, packaging, release

  
area:docs
README, philosophy, dogfood, rules ref

  
area:engineering
Code, AST, graph, MCP runtime

  
area:framework-fp
False-positive heuristics for frameworks

  
area:test-coverage
Dedicated tests for critical modules

  
dogfood:fn
Dogfood evidence: false negative (missed)

  
dogfood:fp
Dogfood evidence: false positive

  
dogfood:friction
Dogfood: UX friction

  
dogfood:tp
Dogfood evidence: true positive (real catch)

  
phase:b
Phase B - engineering hardening

  
phase:c
Phase C - Show HN polish

  
severity:critical
Silent bugs, data loss, security

  
severity:high
Real defects, FP confirmed, crashes

  
severity:low
Hygiene, perf micro-opts

  
severity:medium
Hardening, drift detection, edge cases

  
source:deepseek-v4-pro
Items triaged from the DeepSeek v4 Pro repository audit

No labels
area:ci
area:docs
area:engineering
area:framework-fp
area:test-coverage
dogfood:fn
dogfood:fp
dogfood:friction
dogfood:tp
phase:b
phase:c
severity:critical
severity:high
severity:low
severity:medium
source:deepseek-v4-pro
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
pdurlej/fallow-py!104
No description provided.