Skip to content

remove psutil as a required dep (except in a windows)#5498

Merged
adhami3310 merged 3 commits into
mainfrom
remove-psutil-as-a-required-dep-(except-in-windows)
Jun 25, 2025
Merged

remove psutil as a required dep (except in a windows)#5498
adhami3310 merged 3 commits into
mainfrom
remove-psutil-as-a-required-dep-(except-in-windows)

Conversation

@adhami3310
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Significant refactoring to make psutil an optional dependency except for Windows systems, reducing installation requirements and improving cross-platform compatibility.

  • Moved psutil from required to optional dependency in pyproject.toml, now only required for Windows systems
  • Implemented lazy loading of psutil in reflex/testing.py, reflex/utils/exec.py, and scripts/wait_for_listening_port.py
  • Replaced psutil port checking with standard socket library in reflex/utils/processes.py
  • Removed memory reporting functionality from telemetry system in reflex/utils/telemetry.py
  • Added 2-second timeout to PyPI version check in reflex/utils/prerequisites.py to prevent hanging

9 files reviewed, 1 comment
Edit PR Review Bot Settings | Greptile

Comment thread reflex/utils/processes.py
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jun 25, 2025

CodSpeed Performance Report

Merging #5498 will not alter performance

Comparing remove-psutil-as-a-required-dep-(except-in-windows) (3de5353) with main (38bcc06)

Summary

✅ 8 untouched benchmarks

Copy link
Copy Markdown
Collaborator

@masenf masenf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's cool

@adhami3310 adhami3310 merged commit e99dbe2 into main Jun 25, 2025
40 of 41 checks passed
@adhami3310 adhami3310 deleted the remove-psutil-as-a-required-dep-(except-in-windows) branch June 25, 2025 22:45
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.

2 participants