git-ripper 1.4.6 → 1.4.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-ripper",
-  "version": "1.4.6",
+  "version": "1.4.7",
   "description": "CLI tool that lets you download specific folders from GitHub repositories without cloning the entire repo.",
   "main": "src/index.js",
   "type": "module",

package/src/archiver.js

@@ -88,7 +88,7 @@ export const createArchive = (sourceDir, outputPath) => {
         const size = archive.pointer();
         console.log(
           chalk.green(
-            `✓ Archive created: ${outputPath} (${(size / 1024 / 1024).toFixed(
+            `Archive created: ${outputPath} (${(size / 1024 / 1024).toFixed(
               2
             )} MB)`
           )

package/src/downloader.js

@@ -52,7 +52,9 @@ const fetchFolderContents = async (owner, repo, branch, folderPath) => {
   if (!effectiveBranch) {
     // If no branch is specified, fetch the default branch for the repository
     try {
-      const repoInfoUrl = `https://api.github.com/repos/${owner}/${repo}`;
+      const repoInfoUrl = `https://api.github.com/repos/${encodeURIComponent(
+        owner
+      )}/${encodeURIComponent(repo)}`;
       const repoInfoResponse = await axios.get(repoInfoUrl);
       effectiveBranch = repoInfoResponse.data.default_branch;
       if (!effectiveBranch) {
@@ -75,7 +77,11 @@ const fetchFolderContents = async (owner, repo, branch, folderPath) => {
     }
   }
 
-  const apiUrl = `https://api.github.com/repos/${owner}/${repo}/git/trees/${effectiveBranch}?recursive=1`;
+  const apiUrl = `https://api.github.com/repos/${encodeURIComponent(
+    owner
+  )}/${encodeURIComponent(repo)}/git/trees/${encodeURIComponent(
+    effectiveBranch
+  )}?recursive=1`;
 
   try {
     const response = await axios.get(apiUrl);
@@ -164,7 +170,9 @@ const downloadFile = async (owner, repo, branch, filePath, outputPath) => {
     // This check might be redundant if fetchFolderContents already resolved it,
     // but it's a good fallback for direct downloadFile calls if any.
     try {
-      const repoInfoUrl = `https://api.github.com/repos/${owner}/${repo}`;
+      const repoInfoUrl = `https://api.github.com/repos/${encodeURIComponent(
+        owner
+      )}/${encodeURIComponent(repo)}`;
       const repoInfoResponse = await axios.get(repoInfoUrl);
       effectiveBranch = repoInfoResponse.data.default_branch;
       if (!effectiveBranch) {
@@ -191,10 +199,16 @@ const downloadFile = async (owner, repo, branch, filePath, outputPath) => {
     }
   }
 
-  const baseUrl = `https://raw.githubusercontent.com/${owner}/${repo}`;
+  const baseUrl = `https://raw.githubusercontent.com/${encodeURIComponent(
+    owner
+  )}/${encodeURIComponent(repo)}`;
+  const encodedFilePath = filePath
+    .split("/")
+    .map((part) => encodeURIComponent(part))
+    .join("/");
   const fileUrlPath = effectiveBranch
-    ? `/${effectiveBranch}/${filePath}`
-    : `/${filePath}`; // filePath might be at root
+    ? `/${encodeURIComponent(effectiveBranch)}/${encodedFilePath}`
+    : `/${encodedFilePath}`; // filePath might be at root
   const url = `${baseUrl}${fileUrlPath}`;
 
   try {
@@ -483,7 +497,7 @@ const downloadFolder = async (
       // Don't claim success if files failed to download
       if (succeeded === 0) {
         console.log(
-          chalk.red(`❌ Download failed: No files were downloaded successfully`)
+          chalk.red(`Download failed: No files were downloaded successfully`)
         );
         return {
           success: false,
@@ -492,7 +506,7 @@ const downloadFolder = async (
           isEmpty: false,
         };
       } else {
-        console.log(chalk.yellow(`⚠️  Download completed with errors`));
+        console.log(chalk.yellow(`Download completed with errors`));
         return {
           success: false,
           filesDownloaded: succeeded,
@@ -502,7 +516,7 @@ const downloadFolder = async (
       }
     } else {
       console.log(
-        chalk.green(`✅ All ${succeeded} files downloaded successfully!`)
+        chalk.green(`All ${succeeded} files downloaded successfully!`)
       );
       console.log(chalk.green(`Folder cloned successfully!`));
       return {
@@ -514,7 +528,7 @@ const downloadFolder = async (
     }
   } catch (error) {
     // Log the specific error details
-    console.error(chalk.red(`❌ Error downloading folder: ${error.message}`));
+    console.error(chalk.red(`Error downloading folder: ${error.message}`));
 
     // Re-throw the error so the main CLI can exit with proper error code
     throw error;
@@ -539,9 +553,17 @@ const downloadFolderWithResume = async (
   }
 
   const resumeManager = new ResumeManager();
-  const url = `https://github.com/${owner}/${repo}/tree/${branch || "main"}/${
-    folderPath || ""
-  }`;
+  const encodedFolderPath = folderPath
+    ? folderPath
+        .split("/")
+        .map((part) => encodeURIComponent(part))
+        .join("/")
+    : "";
+  const url = `https://github.com/${encodeURIComponent(
+    owner
+  )}/${encodeURIComponent(repo)}/tree/${encodeURIComponent(
+    branch || "main"
+  )}/${encodedFolderPath}`;
 
   // Clear checkpoint if force restart is requested
   if (forceRestart) {
@@ -554,14 +576,14 @@ const downloadFolderWithResume = async (
   if (checkpoint) {
     console.log(
       chalk.blue(
-        `🔄 Found previous download from ${new Date(
+        `Found previous download from ${new Date(
           checkpoint.timestamp
         ).toLocaleString()}`
       )
     );
     console.log(
       chalk.blue(
-        `📊 Progress: ${checkpoint.downloadedFiles.length}/${checkpoint.totalFiles} files completed`
+        `Progress: ${checkpoint.downloadedFiles.length}/${checkpoint.totalFiles} files completed`
       )
     );
 
@@ -587,11 +609,11 @@ const downloadFolderWithResume = async (
     if (corruptedCount > 0) {
       console.log(
         chalk.yellow(
-          `🔧 Detected ${corruptedCount} corrupted files, will re-download`
+          `Detected ${corruptedCount} corrupted files, will re-download`
         )
       );
     }
-    console.log(chalk.green(`✅ Verified ${validFiles.length} existing files`));
+    console.log(chalk.green(`Verified ${validFiles.length} existing files`));
   }
 
   console.log(
@@ -640,7 +662,7 @@ const downloadFolderWithResume = async (
       );
       console.log(
         chalk.cyan(
-          `📥 Starting download of ${totalFiles} files from ${chalk.white(
+          `Starting download of ${totalFiles} files from ${chalk.white(
             owner + "/" + repo
           )}...`
         )
@@ -648,7 +670,7 @@ const downloadFolderWithResume = async (
     } else {
       // Update total files in case repository changed
       checkpoint.totalFiles = totalFiles;
-      console.log(chalk.cyan(`📥 Resuming download...`));
+      console.log(chalk.cyan(`Resuming download...`));
     }
 
     // Get remaining files to download
@@ -663,13 +685,13 @@ const downloadFolderWithResume = async (
     });
 
     if (remainingFiles.length === 0) {
-      console.log(chalk.green(`🎉 All files already downloaded!`));
+      console.log(chalk.green(`All files already downloaded!`));
       resumeManager.cleanupCheckpoint(url, outputDir);
       return;
     }
 
     console.log(
-      chalk.cyan(`📥 Downloading ${remainingFiles.length} remaining files...`)
+      chalk.cyan(`Downloading ${remainingFiles.length} remaining files...`)
     );
 
     // Setup progress bar
@@ -753,10 +775,8 @@ const downloadFolderWithResume = async (
         if (error.name === "SIGINT") {
           resumeManager.saveCheckpoint(checkpoint);
           progressBar.stop();
-          console.log(
-            chalk.blue(`\n⏸️  Download interrupted. Progress saved.`)
-          );
-          console.log(chalk.blue(`💡 Run the same command again to resume.`));
+          console.log(chalk.blue(`\nDownload interrupted. Progress saved.`));
+          console.log(chalk.blue(`Run the same command again to resume.`));
           return;
         }
 
@@ -793,13 +813,13 @@ const downloadFolderWithResume = async (
       }
 
       console.log(
-        chalk.blue(`💡 Run the same command again to retry failed downloads`)
+        chalk.blue(`Run the same command again to retry failed downloads`)
       );
 
       // Don't claim success if files failed to download
       if (succeeded === 0) {
         console.log(
-          chalk.red(`❌ Download failed: No files were downloaded successfully`)
+          chalk.red(`Download failed: No files were downloaded successfully`)
         );
         return {
           success: false,
@@ -808,7 +828,7 @@ const downloadFolderWithResume = async (
           isEmpty: false,
         };
       } else {
-        console.log(chalk.yellow(`⚠️  Download completed with errors`));
+        console.log(chalk.yellow(`Download completed with errors`));
         return {
           success: false,
           filesDownloaded: succeeded,
@@ -818,7 +838,7 @@ const downloadFolderWithResume = async (
       }
     } else {
       console.log(
-        chalk.green(`🎉 All ${succeeded} files downloaded successfully!`)
+        chalk.green(`All ${succeeded} files downloaded successfully!`)
       );
       resumeManager.cleanupCheckpoint(url, outputDir);
       console.log(chalk.green(`Folder cloned successfully!`));
@@ -835,7 +855,7 @@ const downloadFolderWithResume = async (
       resumeManager.saveCheckpoint(checkpoint);
     }
 
-    console.error(chalk.red(`❌ Error downloading folder: ${error.message}`));
+    console.error(chalk.red(`Error downloading folder: ${error.message}`));
     throw error;
   }
 };

package/src/index.js

@@ -76,7 +76,7 @@ const initializeCLI = () => {
             return;
           }
 
-          console.log(chalk.cyan("\n📋 Download Checkpoints:"));
+          console.log(chalk.cyan("\nDownload Checkpoints:"));
           checkpoints.forEach((cp, index) => {
             console.log(chalk.blue(`\n${index + 1}. ID: ${cp.id}`));
             console.log(`   URL: ${cp.url}`);
@@ -151,12 +151,12 @@ const initializeCLI = () => {
         if (error) {
           const failMsg =
             operationType === "archive"
-              ? `❌ Archive creation failed: ${error.message}`
-              : `❌ Download failed: ${error.message}`;
+              ? `Archive creation failed: ${error.message}`
+              : `Download failed: ${error.message}`;
           console.error(chalk.red(failMsg));
           process.exit(1);
         } else if (!createArchive && result && !result.success) {
-          console.error(chalk.red(`❌ Download failed`));
+          console.error(chalk.red(`Download failed`));
           process.exit(1);
         } else if (!createArchive && result && result.isEmpty) {
           console.log("Operation completed - no files to download!");

package/src/parser.js

@@ -16,10 +16,10 @@ export function parseGitHubUrl(url) {
   }
 
   // Extract components from the matched pattern
-  const owner = match[1];
-  const repo = match[2];
-  const branch = match[3]; // Branch might not be in the URL for root downloads
-  const folderPath = match[4] || ""; // Empty string if no folder path
+  const owner = decodeURIComponent(match[1]);
+  const repo = decodeURIComponent(match[2]);
+  const branch = match[3] ? decodeURIComponent(match[3]) : ""; // Branch is an empty string if not present
+  const folderPath = match[4] ? decodeURIComponent(match[4]) : ""; // Empty string if no folder path
 
   // Additional validation
   if (!owner || !repo) {