Make country optional for ISP and DC proxy types

[sjmiller609]

No description provided.

[mesa-dot-dev]

Mesa Description

This change makes the country parameter optional when creating ISP and DC proxies. If a country is not provided, the system will now automatically select one. This simplifies the proxy creation process for users who don't have a specific country preference.

^{Description generated by Mesa. Update settings}

[mesa-dot-dev]

Performed full review of c3ec0bb...0fe46ad

Analysis

1. The implementation lacks sufficient validation and error handling for the new optional Country parameter, which could lead to unexpected behavior when the parameter is omitted.

2. While backward compatibility is maintained from a syntactic perspective, there's no clear documentation of the semantic implications of omitting the Country parameter - what defaults are used and how this affects routing behavior.

3. The SDK version bump from v0.15.0 to v0.17.0 may introduce additional dependencies or changes that aren't directly related to this feature, potentially increasing upgrade risk.

4. Although test coverage includes both old and new behavior scenarios, it's unclear if edge cases (such as invalid country codes or runtime failures) are adequately tested in the optional parameter context.

Tip

Help

Configure your agents

Mesa Docs

Slash Commands:

• /review - Request a full code review
• /review latest - Review only changes since the last review
• /describe - Generate PR description. This will update the PR body or issue comment depending on your configuration
• /help - Get help with Mesa commands and configuration options

^{4 files reviewed | 0 comments | Edit Agent Settings}

[masnwilliams]

low risk lgtm