IEP-1699: Use CDT build environment for idf_tools.py list command

[kolipakakondal]

Description

The AbstractToolsHandler.runCommand() was using System.getenv() which only
includes OS system environment variables, but not the CDT build environment
variables set by the IDE (like IDF_TOOLS_PATH).

Fixes # (IEP-1699)

Type of change

• Bug fix (non-breaking change which fixes an issue)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• Test B

Test Configuration:

• ESP-IDF Version:
• OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

• Component 1
• Component 2

Checklist

• PR Self Reviewed
• Applied Code formatting
• Added Documentation
• Added Unit Test
• Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

• Refactor
  • Improved internal environment setup process to use a unified method for system environment retrieval. No changes to end-user functionality.

[coderabbitai]

📝 Walkthrough

Walkthrough

AbstractToolsHandler.java refactored to obtain system environment variables through a centralized IDFEnvironmentVariables().getSystemEnvMap() method instead of directly instantiating a HashMap and capturing System properties. The HashMap import was removed accordingly.

Changes

Cohort / File(s)	Summary
Environment Variable Centralization bundles/com.espressif.idf.ui/src/com/espressif/idf/ui/update/AbstractToolsHandler.java	Replaced direct HashMap-based system environment capture with centralized IDFEnvironmentVariables().getSystemEnvMap() method call; removed unused HashMap import.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• IEP-1619 DFU flash output truncated in IDE console #1294: Shifts subprocess environment map construction from direct System.getenv() usage to centralized IDFEnvironmentVariables-based environment handling, including PYTHONUNBUFFERED and PATH management.
• IEP-1043: Allow the user to configure IDF_TOOLS_PATH #988: Modifies environment map retrieval in AbstractToolsHandler by replacing direct System.getenv() usage with a centralized environment provider.
• IEP-1395: Removing system env vars from IDE while executing scripts #1110: Changes how the IDE builds and filters the environment for tool/script execution through centralized environment variable management.

Suggested reviewers

• alirana01
• AndriiFilippov

Poem

🐰 A HashMap once sprawled, now unified and clean,
Environment variables centralize—a harmonious scene,
One method to rule them all, with Python unbuffered flow,
Git PATH adjustments still work, as before we go! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title references IEP-1699 and mentions 'idf_tools.py list command', which aligns with the PR's objective of making this command use CDT build environment variables.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch IEP-1699_new1

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}