Repository navigation
Expose Python environment search paths through public API - #2014
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Stella Huang (StellaHuang95)
added
feature-request
|
🔒 Automated review in progress — Rich Chiodo (@rchiodo) is auto-reviewing this PR. |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
| * `python-envs.workspaceSearchPaths` value for every workspace folder. Relative | ||
| * workspace paths are resolved against the folder that defines them, including in | ||
| * multi-root workspaces. A user/global `workspaceSearchPaths` value and empty | ||
| * entries are ignored, exact duplicate entries are removed, and path separators |
There was a problem hiding this comment.
Warning · Non-blocking recommendation
📍 src/types.ts:1151
Verified isolated production-source probes returned empty and whitespace-only python.venvFolders entries, and whitespace-only python.venvPath, despite this guarantee. Filter blank strings in the canonical resolver before deduplication, preserving nonblank values, and add legacy-settings coverage so native discovery and the public API remain aligned.
[verified]
|
Reviewed commit: Result: 🔴 Verification detailsVerification: Automated verification is best effort. The PR author remains responsible for testing. Summary: [unavailable] Container verification could not start and local execution was not authorized for this PR HEAD: Podman machine 'pyrx-automation' is not running. Test runs: none recorded. |
Rich Chiodo (rchiodo)
left a comment
Rich Chiodo (rchiodo)
left a comment
There was a problem hiding this comment.
Approved via Review Center.
Rich Chiodo (rchiodo)
added
the
review-auto:approved
Bill Schnurr (bschnurr)
left a comment
Bill Schnurr (bschnurr)
left a comment
There was a problem hiding this comment.
Approved via Review Center.
Summary
getAllExtraSearchPaths()on the flat public extension APIValidation
npm run lintnpm run compile-testsnpm run unittest(3,529 passing, 9 pending)npm run compileFixes #960