fix(git): rewrite logical ip upstream aliases

bot_bottle/git_gate.py

@@ -166,6 +166,17 @@ def git_gate_render_gitconfig(
     for entry in entries:
         out.append(f'[url "git://{gate_host}/{entry.Name}.git"]\n')
         out.append(f"\tinsteadOf = {entry.Upstream}\n")
+        if entry.RemoteKey and entry.RemoteKey != entry.UpstreamHost:
+            port = (
+                f":{entry.UpstreamPort}"
+                if entry.UpstreamPort and entry.UpstreamPort != "22"
+                else ""
+            )
+            alias = (
+                f"ssh://{entry.UpstreamUser}@{entry.RemoteKey}{port}/"
+                f"{entry.UpstreamPath}"
+            )
+            out.append(f"\tinsteadOf = {alias}\n")
     return "".join(out)
 
 
@@ -404,4 +415,3 @@ class GitGate(ABC):
             access_hook_script=access_hook,
             upstreams=upstreams,
         )
-

bot_bottle/manifest.py

@@ -85,6 +85,7 @@ class GitEntry:
     IdentityFile: str
     KnownHostKey: str = ""
     ExtraHosts: Mapping[str, str] = field(default_factory=_empty_str_dict)
+    RemoteKey: str = ""
     UpstreamUser: str = ""
     UpstreamHost: str = ""
     UpstreamPort: str = ""
@@ -158,6 +159,7 @@ class GitEntry:
             IdentityFile=ident,
             KnownHostKey=khk,
             ExtraHosts=extra_hosts,
+            RemoteKey=host_key or host,
             UpstreamUser=user,
             UpstreamHost=host,
             UpstreamPort=port,

tests/unit/test_manifest_git.py

@@ -217,6 +217,7 @@ class TestGitEntryCrossValidation(unittest.TestCase):
             "agents": {"demo": {"skills": [], "prompt": "", "bottle": "dev"}},
         })
         e = m.bottles["dev"].git[0]
+        self.assertEqual("gitea.dideric.is", e.RemoteKey)
         self.assertEqual("100.78.141.42", e.UpstreamHost)
         self.assertEqual("30009", e.UpstreamPort)
 

tests/unit/test_provision_git.py

@@ -10,6 +10,7 @@ from bot_bottle.git_gate import (
     GIT_GATE_HOSTNAME,
     git_gate_render_gitconfig,
 )
+from bot_bottle.manifest import Manifest
 from tests.fixtures import fixture_minimal, fixture_with_git
 
 
@@ -59,6 +60,29 @@ class TestGitGateGitconfigRender(unittest.TestCase):
             '[url "git://192.168.20.2:9418/bot-bottle.git"]', out,
         )
 
+    def test_ip_upstream_also_rewrites_logical_remote_key(self):
+        m = Manifest.from_json_obj({
+            "bottles": {"dev": {"git": {"remotes": {
+                "gitea.dideric.is": {
+                    "Name": "bot-bottle",
+                    "Upstream": "ssh://git@100.78.141.42:30009/didericis/bot-bottle.git",
+                    "IdentityFile": "/dev/null",
+                },
+            }}}},
+            "agents": {"demo": {"skills": [], "prompt": "", "bottle": "dev"}},
+        })
+        out = git_gate_render_gitconfig(m.bottles["dev"].git, GIT_GATE_HOSTNAME)
+        self.assertIn(
+            "\tinsteadOf = "
+            "ssh://git@100.78.141.42:30009/didericis/bot-bottle.git",
+            out,
+        )
+        self.assertIn(
+            "\tinsteadOf = "
+            "ssh://git@gitea.dideric.is:30009/didericis/bot-bottle.git",
+            out,
+        )
+
 
 if __name__ == "__main__":
     unittest.main()