feat: return repo fullname in fork script (#29)

This PR is a follow-up to a missing step in the `fork` script initial
implementation at #3; the fork script failed to return a `repo` which
can be used to in the next step of computation. This was because of a
weird assumption I had during the initial implementation. 😆See my
assumption below...

> I assume that the call to the `"POST /repos/{owner}/{repo}/forks"`
endpoint only assures of initiating a fork process without assuring us
of a response at all. Meaning we might not exactly get a `response.data`
following the call

...but that wasn't true, I found out that a `response.data` actually
comes, but it might just take some time and only in cases where the repo
being forked is huge.... and at the moment forking the project repo
happens in less than 5secs.

### Changes Made

- Returned `fork` repo - this is a repo fullname value returned from the
`isRepositoryForked` helper function; I hereby return it as main
returned value from the `forkRepository` function execution in the
condition where the repo is already forked on a executing user's account
- Returned `response.data.full_name` - this is a newly created fork repo
fullname; Its a value from the response to the `"POST
/repos/{owner}/{repo}/forks"` endpoint call; I hereby return it as main
retuned value from the `forkRepository` function execution in cases
where there was no fork already already found on the executing user's
account
- Cherry picked some changes from #25 to use on here
  - f12f25f
  - 436ceea

📖

Author: babblebey

.env.example

@@ -8,4 +8,4 @@ GITHUB_APP_PRIVATE_KEY="-----BEGIN PRIVATE KEY-----\nMII...und==\n-----END PRIVA
 CRYPTO_SECRET_KEY="secret"
 
 PROJECT_REPO="babblebey/jargons.dev"
-PROJECT_REPO_BRANCH_REF="heads/main"
+PROJECT_REPO_BRANCH_REF="refs/heads/main"

src/lib/branch.js

@@ -39,11 +39,14 @@ export async function createBranch(userOctokit, repoDetails, newBranchName) {
  */
 export async function getBranch(userOctokit, repo, ref) {
   const { repoOwner, repoName } = getRepoParts(repo);
+  const formattedRef = ref.split("/")[0] === "refs" 
+    ? ref.split("/").slice(1).join("/")
+    : ref;
 
   const response = await userOctokit.request("GET /repos/{owner}/{repo}/git/ref/{ref}", {
     owner: repoOwner,
     repo: repoName,
-    ref: ref,
+    ref: formattedRef,
   });
 
   return response.data;

src/lib/fork.js

@@ -15,7 +15,7 @@ export async function forkRepository(userOctokit, repoDetails) {
 
     const fork = await isRepositoryForked(userOctokit, repoFullname, user.login ); 
 
-    if (!!fork) {
+    if (fork) {
       console.log("Repo is already forked!");
 
       const { isUpdated, updateSHA } = await isRepositoryForkUpdated(userOctokit, repoDetails, fork)
@@ -29,19 +29,15 @@ export async function forkRepository(userOctokit, repoDetails) {
         });
       }
 
-      return;
+      return fork;
     }
 
     const response = await userOctokit.request("POST /repos/{owner}/{repo}/forks", {
       owner: repoOwner,
       repo: repoName,
     });
 
-    if (response.status === 202) {
-      console.log("Forking process initiated successfully!");
-    } else {
-      console.log("Error occurred while forking repository.");
-    }
+    return response.data.full_name;
   } catch (error) {
     console.log("Error occurred while forking repository:", error);
   }