Skip to content

Conversation

@eleanorjboyd
Copy link
Member

No description provided.

Copilot AI and others added 12 commits September 8, 2025 18:31
…icrosoft#783)

When expanding environment manager nodes in the Environment Managers
view, newly created environments (such as those created by `pipenv
install pytest` or `conda create -n myenv`) were not appearing until the
user manually refreshed the view.

This change adds automatic refresh functionality for all environment
managers when their tree node is expanded. The implementation:

- Calls the existing `manager.refresh(undefined)` method before
retrieving environments for any manager type
- Affects all environment managers (Pipenv, Conda, Venv, Poetry, etc.)
providing consistent behavior
- Uses the established refresh mechanism that properly updates
collections and fires change events

**Before:**
1. Run `pipenv install pytest` or create a new environment with any
manager
2. Open Environment Managers view
3. Expand environment manager node
4. New environment is not visible until manual refresh

**After:**
1. Run `pipenv install pytest` or create a new environment with any
manager
2. Open Environment Managers view  
3. Expand environment manager node
4. New environment appears immediately

The fix is minimal and targeted, modifying only 1 line of code in the
`getChildren` method in `EnvManagerView`. Unit tests have been updated
to validate the core logic for any manager type.

Fixes microsoft#782.

<!-- START COPILOT CODING AGENT TIPS -->
---

💡 You can make Copilot smarter by setting up custom instructions,
customizing its development environment and configuring Model Context
Protocol (MCP) servers. Learn more [Copilot coding agent
tips](https://gh.io/copilot-coding-agent-tips) in the docs.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: eleanorjboyd <26030610+eleanorjboyd@users.noreply.github.com>
…xpansion (microsoft#811)

reverting
microsoft@5b29669
as it is not the right way to do it and causes endless refresh looping
…microsoft#764)

Fixes microsoft#631

Co-authored-by: Eleanor Boyd <26030610+eleanorjboyd@users.noreply.github.com>
This was causing the refresh function to return with empty or missing
environments before waiting for all async calls that update the env
collection.

This bug was somehow causing duplicate environments to appear in the
list of environments.

Co-authored-by: Eleanor Boyd <26030610+eleanorjboyd@users.noreply.github.com>
…soft#815)

should help with diagnosing bugs to have the following information in
logging
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants