fix(provider): support port ranges for tuic and hysteria2

- Add extractFirstPort helper to parse port ranges
- Enable port fallback to first port from port range if not specified
- Extract common TUIC fields to reduce code duplication
- Add validation to ensure port is available for both protocols
- Make TUIC version optional with default value of 5
- Update tests to reflect new behavior

Fixes port configuration when nodes only provide port ranges.

Author: geekdada

src/provider/ClashProvider.ts

@@ -533,51 +533,44 @@ export const parseClashConfig = (
         case 'tuic': {
           let input: TuicNodeConfigInput
 
-          if (item.version >= 5) {
+          const port = item.port ?? extractFirstPort(item.ports)
+
+          if (!port) {
+            throw new SurgioError('Tuic 节点配置校验失败，未指定端口或端口范围')
+          }
+
+          const tuicCommonFields = {
+            type: NodeTypeEnum.Tuic as const,
+            nodeName: item.name,
+            hostname: item.server,
+            port,
+            ...('skip-cert-verify' in item
+              ? { skipCertVerify: item['skip-cert-verify'] === true }
+              : null),
+            tls13: tls13 ?? false,
+            ...('sni' in item ? { sni: item.sni } : null),
+            ...('alpn' in item ? { alpn: item.alpn } : null),
+            ...('ports' in item
+              ? {
+                  portHopping: item.ports,
+                }
+              : null),
+            ...('hop-interval' in item
+              ? { portHoppingInterval: item['hop-interval'] }
+              : null),
+          }
+
+          if (item.uuid) {
             input = {
-              type: NodeTypeEnum.Tuic,
-              version: item.version,
-              nodeName: item.name,
-              hostname: item.server,
-              port: item.port,
+              ...tuicCommonFields,
               password: item.password,
               uuid: item.uuid,
-              ...('skip-cert-verify' in item
-                ? { skipCertVerify: item['skip-cert-verify'] === true }
-                : null),
-              tls13: tls13 ?? false,
-              ...('sni' in item ? { sni: item.sni } : null),
-              ...('alpn' in item ? { alpn: item.alpn } : null),
-              ...('ports' in item
-                ? {
-                    portHopping: item.ports,
-                  }
-                : null),
-              ...('hop-interval' in item
-                ? { portHoppingInterval: item['hop-interval'] }
-                : null),
+              version: item.version ?? 5,
             }
           } else {
             input = {
-              type: NodeTypeEnum.Tuic,
-              nodeName: item.name,
-              hostname: item.server,
-              port: item.port,
+              ...tuicCommonFields,
               token: item.token,
-              ...('skip-cert-verify' in item
-                ? { skipCertVerify: item['skip-cert-verify'] === true }
-                : null),
-              tls13: tls13 ?? false,
-              ...('sni' in item ? { sni: item.sni } : null),
-              ...('alpn' in item ? { alpn: item.alpn } : null),
-              ...('ports' in item
-                ? {
-                    portHopping: item.ports,
-                  }
-                : null),
-              ...('hop-interval' in item
-                ? { portHoppingInterval: item['hop-interval'] }
-                : null),
             }
           }
 
@@ -601,11 +594,19 @@ export const parseClashConfig = (
             )
           }
 
+          const port = item.port ?? extractFirstPort(item.ports)
+
+          if (!port) {
+            throw new SurgioError(
+              'Hysteria2 节点配置校验失败，未指定端口或端口范围',
+            )
+          }
+
           const input: Hysteria2NodeConfigInput = {
             type: NodeTypeEnum.Hysteria2,
             nodeName: item.name,
             hostname: item.server,
-            port: item.port,
+            port,
             password: item.auth || item.password,
             ...(item.down
               ? { downloadBandwidth: parseBitrate(item.down) }
@@ -729,6 +730,10 @@ function resolveUdpRelay(val?: boolean, defaultVal = false): boolean {
   return defaultVal
 }
 
+function extractFirstPort(ports: string): number {
+  return Number(ports.split(/[;,-]/)[0])
+}
+
 function resolveVmessHttpHeaders(
   headers: Record<string, string[]>,
 ): Record<string, string> {

src/provider/__tests__/ClashProvider.test.ts

@@ -1070,7 +1070,6 @@ test('parseClashConfig tuic configurations', (t) => {
         name: 'tuic-v5',
         server: 'example.com',
         port: 443,
-        version: 5,
         uuid: 'uuid',
         password: 'password',
         'skip-cert-verify': true,
@@ -1095,12 +1094,12 @@ test('parseClashConfig tuic configurations', (t) => {
     [
       {
         type: NodeTypeEnum.Tuic,
-        version: 5,
         nodeName: 'tuic-v5',
         hostname: 'example.com',
         port: 443,
         password: 'password',
         uuid: 'uuid',
+        version: 5,
         skipCertVerify: true,
         tls13: false,
         sni: 'sni.example.com',

src/utils/__tests__/surge.test.ts

@@ -542,10 +542,10 @@ test('getSurgeNodes - Tuic', (t) => {
         port: 443,
         password: 'password',
         uuid: 'uuid',
+        version: 5,
         alpn: ['h3'],
         sni: 'sni.example.com',
         skipCertVerify: true,
-        version: 5,
         ecn: true,
       },
     ]),