@otto-code/protocol 0.7.5 → 0.7.6

package/dist/git-remote.d.ts

@@ -1,6 +1,13 @@
 export interface GitRemoteLocation {
     transport: "scp" | "ssh" | "http" | "https";
     host: string;
+    /**
+     * Explicit non-default port from the remote (e.g. a self-hosted forge on
+     * `:60443`), or undefined for a default-port or scp-form remote. Kept separate
+     * from `host` so host-identity matching (forge detection, cloud-host checks)
+     * stays port-agnostic; only consumers that reconstruct a URL (web links) use it.
+     */
+    port?: string;
     path: string;
 }
 export interface GitHubRemoteIdentity {
@@ -15,4 +22,14 @@ export declare function parseGitRemoteLocation(remoteUrl: string): GitRemoteLoca
 export declare function parseGitHubRemoteIdentity(path: string): GitHubRemoteIdentity | null;
 export declare function isGitHubHost(host: string): boolean;
 export declare function normalizeHost(host: string): string;
+/**
+ * Whether `repo` is already a complete git remote (a URL or scp-like address)
+ * rather than `owner/repo` shorthand that still needs a clone protocol picked.
+ *
+ * Clients (app + CLI) and the daemon must agree on this classification: the
+ * daemon treats `parseGitRemoteLocation(repo) !== null` as "complete remote"
+ * and everything else as shorthand, so reuse the same parser here instead of a
+ * separate regex that would drift (e.g. accepting `git://` the parser rejects).
+ */
+export declare function isCompleteGitRemote(repo: string): boolean;
 //# sourceMappingURL=git-remote.d.ts.map

package/dist/git-remote.js

@@ -1,5 +1,10 @@
 const GITHUB_HOSTS = new Set(["github.com", "ssh.github.com"]);
 const BITBUCKET_CLOUD_HOSTS = new Set(["bitbucket.org", "altssh.bitbucket.org"]);
+const DEFAULT_PORT_BY_PROTOCOL = {
+    "https:": "443",
+    "http:": "80",
+    "ssh:": "22",
+};
 const TRANSPORT_BY_PROTOCOL = {
     "https:": "https",
     "http:": "http",
@@ -26,7 +31,9 @@ export function parseGitRemoteLocation(remoteUrl) {
     const trimmed = remoteUrl.trim();
     if (!trimmed)
         return null;
-    const scpLike = trimmed.match(/^[^@]+@([^:]+):(.+)$/u);
+    // scp form has no scheme. Testing it first would match `ssh://git@host:22/x`
+    // — `[^@]+` happily eats `ssh://git` — and swallow the port into the path.
+    const scpLike = trimmed.includes("://") ? null : trimmed.match(/^[^@]+@([^:]+):(.+)$/u);
     if (scpLike) {
         const host = normalizeHost(scpLike[1] ?? "");
         const path = normalizeRemotePath(scpLike[2] ?? "");
@@ -55,7 +62,9 @@ export function parseGitRemoteLocation(remoteUrl) {
     const normalizedPath = normalizeRemotePath(path);
     if (!isValidRemoteHost(host) || !normalizedPath)
         return null;
-    return { transport, host, path: normalizedPath };
+    const protocol = parsed.protocol.toLowerCase();
+    const port = parsed.port && parsed.port !== DEFAULT_PORT_BY_PROTOCOL[protocol] ? parsed.port : undefined;
+    return { transport, host, port, path: normalizedPath };
 }
 export function parseGitHubRemoteIdentity(path) {
     const segments = path.split("/").filter(Boolean);
@@ -81,4 +90,16 @@ function normalizeRemotePath(path) {
 function isValidRemoteHost(host) {
     return /^[a-z0-9](?:[a-z0-9._-]*[a-z0-9])?$/u.test(host);
 }
+/**
+ * Whether `repo` is already a complete git remote (a URL or scp-like address)
+ * rather than `owner/repo` shorthand that still needs a clone protocol picked.
+ *
+ * Clients (app + CLI) and the daemon must agree on this classification: the
+ * daemon treats `parseGitRemoteLocation(repo) !== null` as "complete remote"
+ * and everything else as shorthand, so reuse the same parser here instead of a
+ * separate regex that would drift (e.g. accepting `git://` the parser rejects).
+ */
+export function isCompleteGitRemote(repo) {
+    return parseGitRemoteLocation(repo) !== null;
+}
 //# sourceMappingURL=git-remote.js.map