allow product selection when specifying VR mode

fixes #123

CHANGELOG.md

@@ -8,6 +8,11 @@
 
   This error happens intermittently with Steam licenses.
 
+### Breaking changes
+- `/vr` no longer implies `/autorun`. This allows for users to select a game version while also specifying VR mode.
+
+  If you relied on this behavior, you'll need to append `/autorun` to your launch options going forward.
+
 ## [0.9.0] - 2023-09-12
 
 ### New Features

src/MinEdLauncher/Settings.fs

@@ -143,7 +143,7 @@ let parseArgs defaults (findCbLaunchDir: Platform -> Result<string,string>) (arg
             | "-epicapp", Some id             -> { s with Platform = epicArg (AppId id) }
             | "/oculus", Some nonce           -> { s with Platform = Oculus nonce; ForceLocal = true }
             | "/restart", PosDouble delay     -> { s with Restart = delay * 1000. |> int64 |> Some }
-            | "/vr", _                        -> { s with DisplayMode = Vr; AutoRun = true }
+            | "/vr", _                        -> { s with DisplayMode = Vr }
             | "/novr", _                      -> { s with DisplayMode = Pancake }
             | "/autorun", _                   -> { s with AutoRun = true }
             | "/autoquit", _                  -> { s with AutoQuit = true }

tests/Settings.fs

@@ -73,7 +73,7 @@ let tests =
         test "Matches /vr" {
             let settings = parse [| "/vr" |]
             Expect.equal settings.DisplayMode Vr ""
-            Expect.equal settings.AutoRun true "VR mode should autorun the game"
+            Expect.equal settings.AutoRun false "VR mode should only autorun the game if /autorun is specified"
         }
         test "Matches /novr" {
             let settings = parse [| "/novr" |]