From d445a4dce49c35242a83a7e17d514045c992231c Mon Sep 17 00:00:00 2001
From: Cheng Liu <chengcyber@users.noreply.github.com>
Date: Thu, 11 Jul 2024 14:19:06 -0700
Subject: [PATCH] feat: setup git configurations when full clone

---
 apps/sparo-lib/src/cli/commands/clone.ts | 59 +++++++++++++-----------
 1 file changed, 31 insertions(+), 28 deletions(-)

diff --git a/apps/sparo-lib/src/cli/commands/clone.ts b/apps/sparo-lib/src/cli/commands/clone.ts
index f81d01d..d63fa3b 100644
--- a/apps/sparo-lib/src/cli/commands/clone.ts
+++ b/apps/sparo-lib/src/cli/commands/clone.ts
@@ -77,16 +77,17 @@ export class CloneCommand implements ICommand<ICloneCommandOptions> {
       directory: directory
     };
 
+    const { full, skipGitConfig } = args;
+
     terminal.writeLine('Initializing working directory...');
     const stopwatch: Stopwatch = Stopwatch.start();
 
-    if (args.full) {
+    if (full) {
       this._gitCloneService.fullClone(cloneOptions);
-      return;
+    } else {
+      this._gitCloneService.bloblessClone(cloneOptions);
     }
 
-    this._gitCloneService.bloblessClone(cloneOptions);
-
     process.chdir(directory);
 
     const { profiles, isNoProfile } = await this._sparoProfileService.preprocessProfileArgs({
@@ -94,37 +95,39 @@ export class CloneCommand implements ICommand<ICloneCommandOptions> {
       addProfilesFromArg: []
     });
 
-    await this._GitSparseCheckoutService.ensureSkeletonExistAndUpdated();
+    if (!full) {
+      this._GitSparseCheckoutService.ensureSkeletonExistAndUpdated();
 
-    // check whether profile exist in local branch
-    if (!isNoProfile) {
-      const targetProfileNames: Set<string> = new Set(profiles);
-      const nonExistProfileNames: string[] = [];
-      for (const targetProfileName of targetProfileNames) {
-        if (!this._sparoProfileService.hasProfileInFS(targetProfileName)) {
-          nonExistProfileNames.push(targetProfileName);
+      // check whether profile exist in local branch
+      if (!isNoProfile) {
+        const targetProfileNames: Set<string> = new Set(profiles);
+        const nonExistProfileNames: string[] = [];
+        for (const targetProfileName of targetProfileNames) {
+          if (!this._sparoProfileService.hasProfileInFS(targetProfileName)) {
+            nonExistProfileNames.push(targetProfileName);
+          }
+        }
+
+        if (nonExistProfileNames.length) {
+          throw new Error(
+            `Clone failed. The following profile(s) are missing in cloned repo: ${Array.from(
+              targetProfileNames
+            ).join(', ')}`
+          );
         }
       }
 
-      if (nonExistProfileNames.length) {
-        throw new Error(
-          `Clone failed. The following profile(s) are missing in cloned repo: ${Array.from(
-            targetProfileNames
-          ).join(', ')}`
-        );
+      // Avoid redundant sync if no profile is given
+      if (!isNoProfile && profiles.size) {
+        // sync local sparse checkout state with given profiles.
+        await this._sparoProfileService.syncProfileState({
+          profiles: isNoProfile ? undefined : profiles
+        });
       }
     }
 
-    // Avoid redundant sync if no profile is given
-    if (!isNoProfile && profiles.size) {
-      // sync local sparse checkout state with given profiles.
-      await this._sparoProfileService.syncProfileState({
-        profiles: isNoProfile ? undefined : profiles
-      });
-    }
-
     // set recommended git config
-    if (!args.skipGitConfig) {
+    if (!skipGitConfig) {
       terminal.writeLine(`Applying recommended configuration...`);
       this._gitService.setRecommendConfig({ overwrite: true });
     }
@@ -139,7 +142,7 @@ export class CloneCommand implements ICommand<ICloneCommandOptions> {
     terminal.writeLine('   ' + Colorize.cyan(`cd ${directory}`));
     terminal.writeLine();
 
-    if (isNoProfile || profiles.size === 0) {
+    if (!full && (isNoProfile || profiles.size === 0)) {
       terminal.writeLine('Your next step is to choose a Sparo profile for checkout.');
       terminal.writeLine('To see available profiles in this repo:');
       terminal.writeLine('   ' + Colorize.cyan('sparo list-profiles'));