fix: version check error if not connection

[vdboots]

User description

fixes issues where application will crash if there is no connection available.

---

PR Type

Bug fix

---

Description

• Fixed crash when no internet connection during version check.

• Added specific handling for HttpRequestException in VersionCheckService.

---

Changes walkthrough 📝

	Relevant files
Error handling	VersionCheckService.cs Added error handling for network issues in version check  src/Aspirate.Services/Implementations/VersionCheckService.cs Added a catch block for HttpRequestException. Logged a user-friendly message for network errors. Prevented application crash when unable to reach api.nuget.org. +5/-0

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[qodo-code-review]

Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Error Handling The new catch block for HttpRequestException swallows the exception details. Consider logging the exception details for debugging purposes. catch (HttpRequestException) { logger.MarkupLine($"[red]Network error: Unable to reach api.nuget.org. Please check your internet connection.[/]"); return; }

[qodo-code-review]

Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Improve network error handling specificity Add specific HTTP status code handling in the catch block to provide more precise error messages for different network scenarios (e.g., 404, 503, timeout). src/Aspirate.Services/Implementations/VersionCheckService.cs [150-154] -catch (HttpRequestException) +catch (HttpRequestException ex) { - logger.MarkupLine($"[red]Network error: Unable to reach api.nuget.org. Please check your internet connection.[/]"); + var message = ex.StatusCode switch + { + System.Net.HttpStatusCode.NotFound => "NuGet API endpoint not found", + System.Net.HttpStatusCode.ServiceUnavailable => "NuGet API is temporarily unavailable", + _ => "Unable to reach api.nuget.org. Please check your internet connection" + }; + logger.MarkupLine($"[red]Network error: {message}[/]"); return; } Apply this suggestion Suggestion importance[1-10]: 7 __ Why: The suggestion enhances error handling by providing more specific error messages based on HTTP status codes, improving the user experience by giving clearer feedback about the nature of network failures.	Medium

[prom3theu5]

Good catch