Please address the comments from this code review:
## Overall Comments
- Consider using a common base Agent type (e.g. from 'http' or 'https') as the return signature instead of the specific HttpsProxyAgent or SocksProxyAgent to simplify downstream usage.
- The protocol parsing branch only handles 'socks'; you may want to support variants like 'socks5'/'socks4' or normalize protocol strings to cover more proxy types.
- Rename the 'splited' variable to something like 'parts' for clarity and correct grammar.

## Individual Comments

### Comment 1
<location> `src/utils/makeProxyAgent.ts:12` </location>
<code_context>
 };

-export function makeProxyAgent(proxy: Proxy | string) {
+function selectProxyAgent(protocol: string, uri: string):  HttpsProxyAgent<string> | SocksProxyAgent {
+  switch (protocol) {
+    case 'http':
+      return new HttpsProxyAgent(uri);
+    case 'socks':
+      return new SocksProxyAgent(uri);
+    default:
+      throw new Error(`Unsupported proxy protocol: ${protocol}`);
+    }
</code_context>

<issue_to_address>
The protocol check in selectProxyAgent is too strict and may not handle variants like 'https' or 'socks5'.

The switch should account for protocol variants like 'https' and 'socks5' by normalizing the input and mapping common variants to the correct agent.
</issue_to_address>

<suggested_fix>
<<<<<<< SEARCH
function selectProxyAgent(protocol: string, uri: string):  HttpsProxyAgent<string> | SocksProxyAgent {
  switch (protocol) {
    case 'http':
      return new HttpsProxyAgent(uri);
    case 'socks':
      return new SocksProxyAgent(uri);
    default:
      throw new Error(`Unsupported proxy protocol: ${protocol}`);
    }
}
=======
function selectProxyAgent(protocol: string, uri: string):  HttpsProxyAgent<string> | SocksProxyAgent {
  // Normalize protocol and map variants
  const normalized = protocol.trim().toLowerCase();
  let mapped: string;
  switch (normalized) {
    case 'http':
    case 'https':
      mapped = 'http';
      break;
    case 'socks':
    case 'socks5':
    case 'socks4':
      mapped = 'socks';
      break;
    default:
      throw new Error(`Unsupported proxy protocol: ${protocol}`);
  }

  switch (mapped) {
    case 'http':
      return new HttpsProxyAgent(uri);
    case 'socks':
      return new SocksProxyAgent(uri);
    default:
      throw new Error(`Unsupported proxy protocol: ${protocol}`);
  }
}
>>>>>>> REPLACE

</suggested_fix>

### Comment 2
<location> `src/utils/makeProxyAgent.ts:25` </location>
<code_context>
+export function makeProxyAgent(proxy: Proxy | string): HttpsProxyAgent<string> | SocksProxyAgent {
   if (typeof proxy === 'string') {
-    return new HttpsProxyAgent(proxy);
+    const splited = proxy.split('://');
+    if (splited.length !== 2) {
+      throw new Error('Invalid proxy string format. Expected format: "protocol://host:port"');
+    }
</code_context>

<issue_to_address>
Splitting on '://' may not be robust for all valid proxy URIs.

Using 'split' may not handle all valid proxy URIs, especially those with embedded credentials or multiple '://'. Using the URL constructor is recommended for more reliable parsing.
</issue_to_address>