💪 Refacotr codes to find fetchURL

Author: lambdalisue

commit_url.ts

@@ -11,13 +11,12 @@ export async function getCommitURL(
   commitish: string,
   options: Options = {},
 ): Promise<URL> {
-  if (!options.remote) {
-    const remote = await getRemoteContains(commitish, options);
-    return getCommitURL(commitish, { ...options, remote: remote ?? "origin" });
-  }
-  const fetchURL = await getRemoteFetchURL(options.remote, options);
+  const remote = options.remote ??
+    await getRemoteContains(commitish, options) ??
+    "origin";
+  const fetchURL = await getRemoteFetchURL(remote, options);
   if (!fetchURL) {
-    throw new Error(`Remote '${options.remote}' has no fetch URL`);
+    throw new Error(`No remote '${remote}' found`);
   }
   const hostingService = await getHostingService(fetchURL, options);
   return hostingService.getCommitURL(fetchURL, commitish);

home_url.ts

@@ -10,13 +10,12 @@ export type Options = ExecuteOptions & {
 export async function getHomeURL(
   options: Options = {},
 ): Promise<URL> {
-  if (!options.remote) {
-    const remote = await getRemoteContains("HEAD", options);
-    return getHomeURL({ ...options, remote: remote ?? "origin" });
-  }
-  const fetchURL = await getRemoteFetchURL(options.remote, options);
+  const remote = options.remote ??
+    await getRemoteContains("HEAD", options) ??
+    "origin";
+  const fetchURL = await getRemoteFetchURL(remote, options);
   if (!fetchURL) {
-    throw new Error(`Remote '${options.remote}' has no fetch URL`);
+    throw new Error(`No remote '${remote}' found or failed to get fetch URL.`);
   }
   const hostingService = await getHostingService(fetchURL, options);
   return hostingService.getHomeURL(fetchURL);

object_url.ts

@@ -13,16 +13,12 @@ export async function getObjectURL(
   path: string,
   options: Options = {},
 ): Promise<URL> {
-  if (!options.remote) {
-    const remote = await getRemoteContains(commitish, options);
-    return getObjectURL(commitish, path, {
-      ...options,
-      remote: remote ?? "origin",
-    });
-  }
-  const fetchURL = await getRemoteFetchURL(options.remote, options);
+  const remote = options.remote ??
+    await getRemoteContains(commitish, options) ??
+    "origin";
+  const fetchURL = await getRemoteFetchURL(remote, options);
   if (!fetchURL) {
-    throw new Error(`Remote '${options.remote}' has no fetch URL`);
+    throw new Error(`No remote '${remote}' found`);
   }
   const hostingService = await getHostingService(fetchURL, options);
   const [normPath, range] = parsePath(path);

pull_request_url.ts

@@ -16,16 +16,12 @@ export async function getPullRequestURL(
   commitish: string,
   options: Options = {},
 ): Promise<URL> {
-  if (!options.remote) {
-    const remote = await getRemoteContains(commitish, options);
-    return getPullRequestURL(commitish, {
-      ...options,
-      remote: remote ?? "origin",
-    });
-  }
-  const fetchURL = await getRemoteFetchURL(options.remote, options);
+  const remote = options.remote ??
+    await getRemoteContains(commitish, options) ??
+    "origin";
+  const fetchURL = await getRemoteFetchURL(remote, options);
   if (!fetchURL) {
-    throw new Error(`Remote '${options.remote}' has no fetch URL`);
+    throw new Error(`No remote '${remote}' found`);
   }
   const hostingService = await getHostingService(fetchURL, options);
   if (!hostingService.getPullRequestURL) {
@@ -35,7 +31,7 @@ export async function getPullRequestURL(
   }
   const pr = await getPullRequestContains(
     commitish,
-    options.remote,
+    remote,
     options,
   );
   if (!pr) {