Refactor const to functions (#114)

Author: bluwy

src/core.ts

@@ -21,17 +21,20 @@ function getBaseRefSha(
   return baseRef.target.oid;
 }
 
-const isAlreadyExistingRefError = (error: unknown) =>
-  typeof error === "object" &&
-  error !== null &&
-  "status" in error &&
-  "message" in error &&
-  typeof error.status === "number" &&
-  typeof error.message === "string" &&
-  error.status === 422 &&
-  error.message.includes("Reference already exists");
-
-const createCommit = async ({
+function isAlreadyExistingRefError(error: unknown) {
+  return (
+    typeof error === "object" &&
+    error !== null &&
+    "status" in error &&
+    "message" in error &&
+    typeof error.status === "number" &&
+    typeof error.message === "string" &&
+    error.status === 422 &&
+    error.message.includes("Reference already exists")
+  );
+}
+
+async function createCommit({
   octokit,
   refId,
   baseSha,
@@ -40,7 +43,7 @@ const createCommit = async ({
 }: Pick<CommitFilesFromBase64Args, "octokit" | "message" | "fileChanges"> & {
   refId: string;
   baseSha: string;
-}) => {
+}) {
   // we have to stick to GraphQL here as with REST, each file change would become a separate API call
   return createCommitOnBranchQuery(octokit, {
     input: {
@@ -52,9 +55,9 @@ const createCommit = async ({
       fileChanges,
     },
   });
-};
+}
 
-export const commitFilesFromBase64 = async ({
+export async function commitFilesFromBase64({
   octokit,
   owner,
   repo,
@@ -63,7 +66,7 @@ export const commitFilesFromBase64 = async ({
   force = false,
   message,
   fileChanges,
-}: CommitFilesFromBase64Args): Promise<CommitFilesResult> => {
+}: CommitFilesFromBase64Args): Promise<CommitFilesResult> {
   const baseRef = resolveGitRef(base);
   const targetRef = `refs/heads/${branch}`;
 
@@ -225,4 +228,4 @@ export const commitFilesFromBase64 = async ({
   return {
     refId: newCommit.createCommitOnBranch?.ref?.id ?? null,
   };
-};
+}

src/git.ts

@@ -9,12 +9,12 @@ import type {
 } from "./interface.ts";
 import { resolveGitRef } from "./utils.ts";
 
-export const commitChangesFromRepo = async ({
+export async function commitChangesFromRepo({
   cwd: workingDirectory,
   recursivelyFindRoot = true,
   filterFiles,
   ...otherArgs
-}: CommitChangesFromRepoArgs): Promise<CommitFilesResult> => {
+}: CommitChangesFromRepoArgs): Promise<CommitFilesResult> {
   const ref = resolveGitRef(otherArgs.base ?? { commit: "HEAD" });
   const cwd = path.resolve(workingDirectory);
   const repoRoot = recursivelyFindRoot ? await findGitRoot(cwd) : cwd;
@@ -36,7 +36,7 @@ export const commitChangesFromRepo = async ({
       commit: refSha,
     },
   });
-};
+}
 
 // Exported for testing only
 export async function getFileChanges(