feenkcom
The Iceberg git CLI implementation uses the wrong kind of IceRemote objects
The Iceberg git CLI implementation uses the wrong kind of IceRemote objects and it should probably use its own variant.
In particular, IceGitCliRepository>>#remotes returns (sub) instances of IceGitRemote but there are methods on that class (IceGitRemote>>#remoteBranchNamed:ifPresent:ifAbsent: and IceGitRemote>>#doFetchWithRefSpec:) that assume the Libgit2 FFI implementation.
The problem is that we can't simply subclass and override those methods since there is a hierarchy under IceGitRemote. It looks as of the subclasses mostly deal with parsing different URLs into parts, but do not implement much real behaviour.
Another problem is that IceGitRemote is referred to as a factory in many places (the key subclass creation/dispatch method being IceGitRemote class>>#url:).
