jvcs 1.6.9 → 1.7.2

package/controllers/add.js

@@ -1,4 +1,4 @@
-const { checkGlobalConfig, getGlobalConfig, checkforjvcs } = require("./utility")
+const { getGlobalConfig, checkforjvcs } = require("./utility")
 const path = require("path")
 const fssync = require("fs")
 const chalk = require("chalk")
@@ -15,7 +15,7 @@ async function getFileHash(filepath) {
 // load .jvcsignore file
 async function loadIgnorePatterns() {
     const ignorePath = path.join(process.cwd(), ".jvcsignore")
-    if(!fssync.existsSync(ignorePath)) return []
+    if (!fssync.existsSync(ignorePath)) return []
 
     const content = await fs.readFile(ignorePath, "utf-8")
     return content
@@ -61,26 +61,20 @@ async function addCmd(paths) {
 
     if (!paths || paths.length === 0) {
         console.log(chalk.yellow("Please specify files or folders to add."));
-        return
+        process.exit(1);
     }
 
-    if (!checkGlobalConfig()) {
-        console.log(chalk.red("No existing session found. Please login or signup."))
-        console.log(chalk.green("jvcs --help for help"))
-        return
-    }
-
-    let configData = getGlobalConfig()
+    const configData = getGlobalConfig()
 
     if (!configData) {
         console.log(chalk.red("No existing session found. Please login or signup."))
         console.log(chalk.green("jvcs --help for help"))
-        return
+        process.exit(1);
     }
 
     if (!checkforjvcs()) {
         console.log(chalk.red("Repository is not initialized or is deleted. Please create it."))
-        return
+        process.exit(1);
     }
 
     const repoPath = path.join(process.cwd(), ".jvcs")
@@ -99,6 +93,43 @@ async function addCmd(paths) {
     // load ignore patterns
     const ignorePatterns = await loadIgnorePatterns()
 
+    // -------------------------------------------------------------
+    // NEW: Handle Deletions sync before adding new files
+    // -------------------------------------------------------------
+    let deletionHandledFor = new Set();
+
+    for (const key of Object.keys(hashData)) {
+        const cwdPath = path.join(process.cwd(), key);
+
+        // Check if the staged file falls under any of the target paths
+        let isUnderTarget = false;
+        if (paths.length === 1 && paths[0] === ".") {
+            isUnderTarget = true;
+        } else {
+            for (const p of paths) {
+                const targetPath = path.resolve(process.cwd(), p);
+                const relToTarget = path.relative(targetPath, cwdPath);
+                // If it's the exact file or inside the target directory
+                if (relToTarget === "" || (!relToTarget.startsWith("..") && !path.isAbsolute(relToTarget))) {
+                    isUnderTarget = true;
+                    break;
+                }
+            }
+        }
+
+        if (isUnderTarget && !fssync.existsSync(cwdPath)) {
+            // File was deleted from working directory, remove it from staging
+            delete hashData[key];
+            const stagedTarget = path.join(staging, key);
+            if (fssync.existsSync(stagedTarget)) {
+                await fs.rm(stagedTarget);
+            }
+            console.log(chalk.red(`Removed deleted file: ${key}`));
+            deletionHandledFor.add(cwdPath);
+        }
+    }
+
+
     let targets = []
     if (paths.length === 1 && paths[0] === ".") {
         let rootEntries = await fs.readdir(process.cwd(), { withFileTypes: true })
@@ -114,7 +145,10 @@ async function addCmd(paths) {
         try {
 
             if (!fssync.existsSync(target)) {
-                console.log(chalk.red(`Path not found: ${target}`))
+                // Only warn if we didn't just handle its deletion
+                if (!deletionHandledFor.has(target)) {
+                    console.log(chalk.yellow(`Path not found: ${path.relative(process.cwd(), target)}`));
+                }
                 continue
             }
 
@@ -155,7 +189,7 @@ async function addCmd(paths) {
             }
         }
         catch (error) {
-            console.log(chalk.red(`Unexpected error: ${error.message}`));
+            console.log(chalk.red(`Unexpected error processing ${target}: ${error.message}`));
         }
     }
 

package/controllers/begin.js

@@ -10,14 +10,24 @@ const fs = require("fs")
 
 async function beginCmd() {
 
-    const config = path.join(require("os").homedir(),".jvcs","config.json")
+    const config = path.join(require("os").homedir(), ".jvcs", "config.json")
     let isInitialized = false
     let configData = null
-    if(fs.existsSync(config)) {
-        isInitialized = true
-        configData = JSON.parse(fs.readFileSync(config,"utf-8"))
+    if (fs.existsSync(config)) {
+        try {
+            configData = JSON.parse(fs.readFileSync(config, "utf-8"))
+            isInitialized = true
+        }
+        catch (error) {
+            // corrupted file, delete it and force re-login
+            try {
+                fs.unlinkSync(config)
+            }
+            catch (e) { }
+            isInitialized = false
+        }
     }
-    
+
     if (isInitialized) {
         console.log(chalk.green(`You are already logged in as ${configData.username} (${configData.email})`))
         console.log(chalk.yellow(`[1] Continue \n[2] Logout \nChoose an option (1/2) : `))
@@ -38,14 +48,20 @@ async function beginCmd() {
             }
         ])
 
-        console.log(`Your choice was ${answer.choice}`)
         if (answer.choice === "1") {
             console.log(chalk.green("Continuing as current user..."))
             console.log(chalk.green("jvcs --help for help"))
         }
         else {
             console.log(chalk.green("logging out..."))
-            fs.unlinkSync(config)
+            try {
+                if (fs.existsSync(config)) {
+                    fs.unlinkSync(config)
+                }
+            }
+            catch (error) {
+                console.log(chalk.yellow("Warning: Could not remove config file properly."));
+            }
             console.log(chalk.green("Logged out Successfully"))
             console.log(chalk.green("Please login or signup again (jvcs begin) to use version control system"))
             console.log(chalk.green("jvcs --help for help"))
@@ -80,6 +96,14 @@ async function beginCmd() {
             {
                 type: 'input',
                 name: 'username',
+                validate: function (input) {
+                    // Ensure no spaces or weird special characters in username
+                    if (!/^[a-zA-Z0-9_]+$/.test(input)) {
+                        return "Username can only contain alphanumeric characters and underscores";
+                    }
+                    if (input.length < 3) return "Username must be at least 3 characters long";
+                    return true;
+                },
                 filter: function (input) {
                     return input.trim()
                 }
@@ -139,6 +163,7 @@ async function beginCmd() {
         }
         catch (error) {
             console.log(chalk.red(error))
+            process.exit(1);
         }
     }
     else {
@@ -194,6 +219,7 @@ async function beginCmd() {
         }
         catch (error) {
             console.log(chalk.red(error))
+            process.exit(1);
         }
     }
 }

package/controllers/init.js

@@ -6,16 +6,16 @@ const { getGlobalConfig, checkGlobalConfig } = require("./utility")
 
 async function initCmd() {
 
-    if(!checkGlobalConfig()) {
+    if (!checkGlobalConfig()) {
         console.log(chalk.red("No existing session found. Please login or signup."))
         console.log(chalk.green("jvcs --help for help"))
         return
     }
-    
+
     let configData = null
     configData = getGlobalConfig()
-    
-    if(!configData) {
+
+    if (!configData) {
         console.log(chalk.red("No existing session found. Please login or signup."))
         console.log(chalk.green("jvcs --help for help"))
         return
@@ -25,36 +25,45 @@ async function initCmd() {
     const cwd = process.cwd()
     const home = require("os").homedir()
 
-    if(cwd ===  home) {
+    if (cwd === home) {
         console.log(chalk.red("Cannot initialize a repository in your home directory."));
         console.log(chalk.yellow("Hint: navigate to a project folder and run 'jvcs init' there."));
         process.exit(1);
     }
 
-    const repoPath = path.join(cwd,".jvcs")
-    if(fssync.existsSync(repoPath)) {
+    const repoPath = path.join(cwd, ".jvcs")
+    if (fssync.existsSync(repoPath)) {
         console.log(chalk.yellow("Repository already is initialized."));
-        process.exit(1);    
+        process.exit(0);
     }
 
-    fssync.mkdirSync(repoPath)
-    fssync.mkdirSync(path.join(repoPath,"commits"))
-    fssync.mkdirSync(path.join(repoPath,"staging"))
-    
-    const config = {
-        repoName: path.basename(cwd),
-        createdAt: new Date().toISOString(),
-        remote: null,
-        owner : {
-            username: configData.username,
-            email: configData.email
+    try {
+        fssync.mkdirSync(repoPath)
+        fssync.mkdirSync(path.join(repoPath, "commits"))
+        fssync.mkdirSync(path.join(repoPath, "staging"))
+
+        const config = {
+            repoName: path.basename(cwd),
+            createdAt: new Date().toISOString(),
+            remote: null,
+            owner: {
+                username: configData.username,
+                email: configData.email
+            }
         }
-    }
 
-    fssync.writeFileSync(path.join(repoPath, "config.json"), JSON.stringify(config, null, 2))
-    fssync.writeFileSync(path.join(repoPath, "HEAD"), "");
+        fssync.writeFileSync(path.join(repoPath, "config.json"), JSON.stringify(config, null, 2))
+        fssync.writeFileSync(path.join(repoPath, "HEAD"), "");
 
-    console.log(chalk.green(`Initialized empty JVCS repository`));
+        console.log(chalk.green(`Initialized empty JVCS repository in ${repoPath}`));
+    }
+    catch (error) {
+        if (fssync.existsSync(repoPath)) {
+            fssync.rmSync(repoPath, { recursive: true, force: true });
+        }
+        console.log(chalk.red(`Failed to initialize repository: ${error.message}`));
+        process.exit(1);
+    }
 }
 
 module.exports = initCmd

package/index.js

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 
 const yargs = require("yargs");
 const { hideBin } = require("yargs/helpers");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jvcs",
-  "version": "1.6.9",
+  "version": "1.7.2",
   "bin": {
     "jvcs": "index.js"
   },