Allow quickPlayMultiplayer and server options to work together for compatibility

Co-authored-by: ci010 <[email protected]>

Author: Copilot

packages/core/README.md

@@ -103,4 +103,15 @@ const proc = launch({
 });
 ```
 
-**Note:** When both `quickPlayMultiplayer` and `server` options are provided, `quickPlayMultiplayer` takes precedence.
+Both `quickPlayMultiplayer` and `server` options can be used together for compatibility:
+
+```ts
+// Both options together for compatibility
+const proc = launch({ 
+  gamePath, 
+  javaPath, 
+  version,
+  quickPlayMultiplayer: 'play.hypixel.net:25565',
+  server: { ip: 'play.hypixel.net', port: 25565 }
+});
+```

packages/core/launch.test.ts

@@ -290,7 +290,7 @@ describe('Launcher', () => {
       expect(args.indexOf('--server')).toBe(-1)
       expect(args.indexOf('--port')).toBe(-1)
     })
-    test('should prefer quickPlayMultiplayer over server option', async ({ mock }) => {
+    test('should allow both quickPlayMultiplayer and server options for compatibility', async ({ mock }) => {
       const server = {
         ip: '192.168.1.1',
         port: 25565,
@@ -303,9 +303,9 @@ describe('Launcher', () => {
         quickPlayMultiplayer: '127.0.0.1:25565',
       })
       expect(args[args.indexOf('--quickPlayMultiplayer') + 1]).toEqual('127.0.0.1:25565')
-      // Should not contain old server arguments when quickPlayMultiplayer is present
-      expect(args.indexOf('--server')).toBe(-1)
-      expect(args.indexOf('--port')).toBe(-1)
+      // Should also contain legacy server arguments for compatibility
+      expect(args[args.indexOf('--server') + 1]).toEqual('192.168.1.1')
+      expect(args[args.indexOf('--port') + 1]).toEqual('25565')
     })
   })
 

packages/core/launch.ts

@@ -812,7 +812,8 @@ export async function generateArguments(options: LaunchOption) {
   }
   if (options.quickPlayMultiplayer) {
     cmd.push('--quickPlayMultiplayer', options.quickPlayMultiplayer)
-  } else if (options.server) {
+  }
+  if (options.server) {
     cmd.push('--server', options.server.ip)
     if (options.server.port) {
       cmd.push('--port', options.server.port.toString())