Review of #52:

- The settings import merges Hookshot's keys into the build's own
  longshot_options.ini: every existing line (compile_shaders, vsync, the
  budgets, auto_*, note_* bindings, unknown keys) stays in its place, an
  imported key replaces its value, new keys go at the end.
- A value out of the range of Longshot's menu is skipped and listed with the
  unknown keys instead of refusing the whole import.
- Links on the way to the source, target or backup folder are resolved
  (Saves.Real, which now resolves every part of the path) and the resolved
  folders compared; only the settings files themselves are refused as links.
- An imported game language also becomes the launcher's when one is set, so
  the next update's --language does not undo it; the dialog says so.
- Saves.ImportTarget is always the fixed user the game opens, never another
  profile folder in save/.
- The settings dialog starts with no Longshot version chosen.
- AGENTS.md's pointer to docs/saves.md no longer says "the one bridge".

Co-Authored-By: Claude Opus 5.5 <[email protected]>

@@ -8,3 +8,2 @@
 
-- **Clean break.** The launcher reads nothing of a Hookshot install, its settings, folders, channels, versions or caches,
-  except its saves (`Core/HookshotSaves.cs`). Don't add compatibility with Hookshot or its 2.x builders.
+- **Clean break.** Only Hookshot's saves (`Core/HookshotSaves.cs`) and the compatible game settings explicitly selected for

@@ -0,0 +22,4 @@
+        ImportCommand = new Command(Import, () => CanImport);
+        Targets = main.Service.Registry.Builds.Where(main.Service.IsBuilt).ToList();
+        target = Targets.FirstOrDefault(b => b.Default) ?? (Targets.Count == 1 ? Targets[0] : null);
+        var roots = HookshotSaves.Roots(main.Service.SaveEnv)

@@ -0,0 +46,4 @@
+        var info = new DirectoryInfo(path);
+        if (mustExist && !info.Exists) throw new SaveFailure($"No game folder at {path}.");
+        for (var at = info; at != null; at = at.Parent)
+            if (at.LinkTarget != null || (at.Exists && at.Attributes.HasFlag(FileAttributes.ReparsePoint)))

@@ -0,0 +121,4 @@
+                    valid = values.Length is 1 or 2 && values.All(v => Integer(v.Trim(), pad ? -1 : 0, pad ? 16 : 255));
+                }
+                else { skipped.Add(key); continue; }
+                if (!valid) throw new SaveFailure($"Hookshot setting {key} has an invalid value. Nothing was copied.");

@@ -0,0 +125,4 @@
+                kept[key] = value;
+            }
+            if (kept.Keys.Any(k => k != "options_version"))
+                files.Add("longshot_options.ini", Utf8.GetBytes(string.Join('\n', kept.Select(kv => kv.Key + "=" + kv.Value)) + "\n"));

@@ -0,0 +131,4 @@
+        {
+            var code = Text(language, "language.txt").Split('\n').Select(l => l.Trim()).FirstOrDefault(l => l.Length > 0 && !l.StartsWith('#'));
+            var known = Languages.Normalize(code) ?? throw new SaveFailure("language.txt has no supported game language. Nothing was copied.");
+            files.Add("language.txt", Utf8.GetBytes(known + "\n"));

@@ -282,0 +287,4 @@
+    public static string ImportTarget(SaveEnvironment env)
+    {
+        var profiles = Shared(env).Profiles;
+        return profiles.Count == 0 ? SharedSavePath(env, UserId) : Pick(profiles, "Longshot").Path;

@@ -119,0 +152,4 @@
+  restoration, unsupported values and linked paths.
+
+Settings compatibility was compared on 2026-11-13 against Hookshot
+[`363038d8`](https://github.com/brunobaccari/Hookshot/tree/363038d8903009d8ef06a6f6df87c7c5bde370e1)

@@ -0,0 +135,4 @@
+                    skipped.Add(key);   // (unknown, or out of range: listed and left out, as Longshot would clamp or ignore it)
+                    continue;
+                }
+                skipped.Remove(key);   // (a later valid line of the same key, as the game reads the last one)

@@ -0,0 +155,4 @@
+    /// own settings, such as make_game.py's compile_shaders, vsync, the budgets, auto_*, note_* bindings, and any key it does not
+    /// know), each imported key's value put in place of the old one, and the imported keys the file does not have yet added at
+    /// the end, as make_game.py's write_compile_level and the menu's own save keep the lines they do not write. options_version
+    /// is one of the imported keys: Hookshot's marker, so that Longshot migrates the imported rows; the migrations it runs

@@ -0,0 +159,4 @@
+    /// (gpu/vk/vk_menu.cpp menu_init) are of rows Hookshot has too, none of the Longshot-only lines kept here
+    internal static string Merge(string? existing, IReadOnlyList<(string Key, string Value)> imported)
+    {
+        var values = imported.ToDictionary(kv => kv.Key, kv => kv.Value, StringComparer.Ordinal);

@@ -0,0 +197,4 @@
+        foreach (var line in lines)
+            if (Line(line) is ("options_version", var v)) had = OptionsVersion(v);
+        var now = values.TryGetValue("options_version", out var marker) ? OptionsVersion(marker) : -1;
+        var drop = new HashSet<string>(Migrated.Where(m => !values.ContainsKey(m.Key) && (had < m.Since) != (now < m.Since)).Select(m => m.Key),