IEP-1768 java.lang.Exception: Resource ... does not exist

[sigmaaa]

Description

Added a check for project existence

Fixes # (IEP-1768)

Type of change

Please delete options that are not relevant.

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

How has this been tested?

Create multiple projects and delete them. Track logs for the resource not exists exception

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

• Bug Fixes
  • Enhanced validation logic to prevent errors when encountering invalid or missing projects, improving overall stability when working with project configurations.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d75acaf1-552c-43b9-a115-f60007d7b994

📥 Commits

Reviewing files that changed from the base of the PR and between 559f443 and 07ee6b6.

📒 Files selected for processing (1)

• bundles/com.espressif.idf.core/src/com/espressif/idf/core/util/IDFUtil.java

---

📝 Walkthrough

Walkthrough

The updateProjectBuildFolder method in IDFUtil now performs a more robust project validation check. The guard clause now returns early if the resolved project is either null or does not exist, replacing the previous null-only check.

Changes

Project validation enhancement

Layer / File(s)	Summary
Project null and existence validation bundles/com.espressif.idf.core/src/com/espressif/idf/core/util/IDFUtil.java	The early-return check in updateProjectBuildFolder now guards against both null projects and non-existent projects by combining the null check with !project.exists().

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

• kolipakakondal
• AndriiFilippov

Poem

🐰 A guard grows stronger, standing tall,
When null and missing both heed the call,
One line added, defense set right,
Projects validated day and night! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title references issue IEP-1768 and describes the exception being fixed, directly corresponding to the main change (adding a null/existence check for projects).
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch IEP-1768

---

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.}