gitshift 2.1.0 → 2.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitshift",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "GitHub Account Switcher CLI",
   "main": "server.js",
   "bin": {

package/src/commands/add.js

@@ -1,23 +1,8 @@
-import {
-    confirm,
-    input,
-} from "@inquirer/prompts";
-
+import { confirm, input } from "@inquirer/prompts";
 import ora from "ora";
-
-import {
-    getProfile,
-    saveProfile,
-} from "../services/profile.js";
-
-import {
-    generateSSHKey,
-} from "../services/ssh.js";
-
-import {
-    error,
-    success,
-} from "../utils/logger.js";
+import { getProfile, saveProfile } from "../services/profile.js";
+import { generateSSHKey } from "../services/ssh.js";
+import { error, success } from "../utils/logger.js";
 
 export async function addCommand() {
     try {
@@ -25,6 +10,12 @@ export async function addCommand() {
             message: "Profile Name",
         });
 
+        if (!name.trim()) {
+            throw new Error(
+                "Profile name is required"
+            );
+        }
+
         if (getProfile(name)) {
             error(`Profile "${name}" already exists`);
 
@@ -36,10 +27,22 @@ export async function addCommand() {
             message: "GitHub Username",
         });
 
+        if (!username.trim()) {
+            throw new Error(
+                "GitHub username is required"
+            );
+        }
+
         const email = await input({
             message: "Email",
         });
 
+        if (!email.trim()) {
+            throw new Error(
+                "Email is required"
+            );
+        }
+
         const createSSH = await confirm({
             message: "Generate SSH key automatically?",
             default: true,
@@ -80,9 +83,7 @@ export async function addCommand() {
             sshKey,
         });
 
-        success(
-            `Profile "${name}" saved locally`
-        );
+        success(`Profile "${name}" saved locally`);
     } catch (err) {
         if (err && err.name === "ExitPromptError") {
             // User canceled the prompt (Ctrl+C / SIGINT). Exit gracefully.

package/src/commands/auto.js

@@ -1,43 +1,31 @@
 
-import {
-    getFolderMappings,
-    getProfile,
-    setCurrentProfile,
-} from "../services/profile.js";
-
-import {
-    setGitUser,
-} from "../services/git.js";
-
-import {
-    success,
-} from "../utils/logger.js";
+import { setGitUser } from "../services/git.js";
+import { getFolderMappings, getProfile, setCurrentProfile } from "../services/profile.js";
+import { success } from "../utils/logger.js";
 
 export async function autoCommand() {
     const cwd = process.cwd();
 
     const mappings = getFolderMappings();
 
-    const matched =
-        mappings
-            .sort(
-                (a, b) =>
-                    b.path.length -
-                    a.path.length
+    const matched = mappings
+        .sort(
+            (a, b) =>
+                b.path.length -
+                a.path.length
+        )
+        .find((mapping) =>
+            cwd.startsWith(
+                mapping.path
             )
-            .find((mapping) =>
-                cwd.startsWith(
-                    mapping.path
-                )
-            );
+        );
 
     if (!matched) {
         console.log("\nNo matching profile\n");
         return;
     }
 
-    const profile =
-        getProfile(matched.profile);
+    const profile = getProfile(matched.profile);
 
     if (!profile) {
         return;

package/src/commands/current.js

@@ -1,21 +1,15 @@
-import {
-    getCurrentProfile,
-    getProfile,
-} from "../services/profile.js";
-
+import { getCurrentProfile, getProfile, } from "../services/profile.js";
 import { info } from "../utils/logger.js";
 
 export async function currentCommand() {
-    const current =
-        getCurrentProfile();
+    const current = getCurrentProfile();
 
     if (!current) {
         info("No active profile");
         return;
     }
 
-    const profile =
-        getProfile(current);
+    const profile = getProfile(current);
 
     if (!profile) {
         info("Profile not found");
@@ -23,23 +17,9 @@ export async function currentCommand() {
     }
 
     console.log();
-
-    console.log(
-        `Profile : ${profile.name}`
-    );
-
-    console.log(
-        `Username: ${profile.username}`
-    );
-
-    console.log(
-        `Email   : ${profile.email}`
-    );
-
-    console.log(
-        `SSH Key : ${profile.sshKey || "Not configured"
-        }`
-    );
-
+    console.log(`Profile : ${profile.name}`);
+    console.log(`Username: ${profile.username}`);
+    console.log(`Email   : ${profile.email}`);
+    console.log(`SSH Key : ${profile.sshKey || "Not configured"}`);
     console.log();
 }

package/src/commands/doctor.js

@@ -1,24 +1,13 @@
 import { execa } from "execa";
+import { error, success } from "../utils/logger.js";
 
-import {
-    error,
-    success,
-} from "../utils/logger.js";
-
-async function check(
-    name,
-    command,
-    args = []
-) {
+async function check(name, command, args = []) {
     try {
         await execa(command, args);
-
         success(`${name} installed`);
-
         return true;
     } catch {
         error(`${name} missing`);
-
         return false;
     }
 }
@@ -26,25 +15,14 @@ async function check(
 export async function doctorCommand() {
     console.log();
 
-    await check("Git", "git", [
-        "--version",
-    ]);
-
-    await check("SSH", "ssh", [
-        "-V",
-    ]);
+    await check("Git", "git", ["--version",]);
+    await check("SSH", "ssh", ["-V",]);
 
-    const ghInstalled = await check(
-        "GitHub CLI",
-        "gh",
-        ["--version"]
-    );
+    const ghInstalled = await check("GitHub CLI", "gh", ["--version"]);
 
     if (!ghInstalled) {
         console.log();
-        console.log(
-            "Hint: Install GitHub CLI (macOS): `brew install gh`"
-        );
+        console.log("Hint: Install GitHub CLI (macOS): `brew install gh`");
         console.log("Then authenticate with: `gh auth login`");
     }
 

package/src/commands/link.js

@@ -1,33 +1,14 @@
+import { confirm, input, select } from "@inquirer/prompts";
 import fs from "fs-extra";
+import ora from "ora";
 import path from "path";
+import { addFolderMapping, getProfile, getProfiles, saveProfile } from "../services/profile.js";
+import { generateSSHKey } from "../services/ssh.js";
+import { error, success } from "../utils/logger.js";
 
-import {
-    confirm,
-    input,
-    select,
-} from "@inquirer/prompts";
-
-import {
-    addFolderMapping,
-    getProfiles,
-    saveProfile,
-} from "../services/profile.js";
-
-import {
-    generateSSHKey,
-} from "../services/ssh.js";
+export async function linkCommand(folder) {
 
-import ora from "ora";
-import {
-    error,
-    success,
-} from "../utils/logger.js";
-
-export async function linkCommand(
-    folder
-) {
     const fullPath = path.resolve(folder);
-
     const exists = await fs.pathExists(fullPath);
 
     if (!exists) {
@@ -36,33 +17,28 @@ export async function linkCommand(
     }
 
     let profiles = getProfiles();
-
     let selectedProfile;
 
     if (profiles.length === 0) {
         console.log("\nNo profiles found.\n");
-
         selectedProfile = await createProfile();
     } else {
-        const choice =
-            await select({
-                message: "Select Profile",
-                choices: [
-                    ...profiles.map(
-                        (
-                            profile
-                        ) => ({
-                            name: `${profile.name} (${profile.username})`,
-                            value:
-                                profile.name,
-                        })
-                    ),
-                    {
-                        name: "+ Create New Profile",
-                        value: "__create__",
-                    },
-                ],
-            });
+        const choice = await select({
+            message: "Select Profile",
+            choices: [
+                ...profiles.map(
+                    (profile) => ({
+                        name: `${profile.name} (${profile.username})`,
+                        value:
+                            profile.name,
+                    })
+                ),
+                {
+                    name: "+ Create New Profile",
+                    value: "__create__",
+                },
+            ],
+        });
 
         if (choice === "__create__") {
             selectedProfile = await createProfile();
@@ -73,9 +49,7 @@ export async function linkCommand(
 
     try {
         addFolderMapping(selectedProfile, fullPath);
-
         success(`Linked ${fullPath}`);
-
         success(`Profile: ${selectedProfile}`);
     } catch (err) {
         error(err.message);
@@ -83,21 +57,38 @@ export async function linkCommand(
 }
 
 async function createProfile() {
-    const name =
-        await input({
-            message: "Profile Name",
-        });
+    const name = await input({
+        message: "Profile Name",
+    });
+
+    if (!name.trim()) {
+        throw new Error("Profile name is required");
+    }
+
+    if (getProfile(name)) {
+        error(`Profile "${name}" already exists`);
+        process.exitCode = 1;
+        return;
+    }
 
     const username =
         await input({
             message: "GitHub Username",
         });
 
+    if (!username.trim()) {
+        throw new Error("GitHub username is required");
+    }
+
     const email =
         await input({
             message: "Email",
         });
 
+    if (!email.trim()) {
+        throw new Error("Email is required");
+    }
+
     const shouldCreateSSH =
         await confirm({
             message: "Generate SSH Key?",
@@ -111,7 +102,6 @@ async function createProfile() {
 
         try {
             sshKey = await generateSSHKey(name, email);
-
             spinner.succeed("SSH key generated");
         } catch (err) {
             spinner.fail("Unable to generate SSH key");

package/src/commands/links.js

@@ -1,6 +1,4 @@
-import {
-    getFolderMappings,
-} from "../services/profile.js";
+import { getFolderMappings } from "../services/profile.js";
 
 export async function linksCommand() {
     const mappings = getFolderMappings();

package/src/commands/remove.js

@@ -1,37 +1,19 @@
-import {
-    getCurrentProfile,
-    getProfile,
-    removeProfile,
-    setCurrentProfile,
-} from "../services/profile.js";
+import { getCurrentProfile, getProfile, removeProfile, setCurrentProfile } from "../services/profile.js";
+import { error, success } from "../utils/logger.js";
 
-import {
-    error,
-    success,
-} from "../utils/logger.js";
-
-export async function removeCommand(
-    profileName
-) {
-    const profile =
-        getProfile(profileName);
+export async function removeCommand(profileName) {
+    const profile = getProfile(profileName);
 
     if (!profile) {
         error("Profile not found");
-
         return;
     }
 
     removeProfile(profileName);
 
-    if (
-        getCurrentProfile() ===
-        profileName
-    ) {
+    if (getCurrentProfile() === profileName) {
         setCurrentProfile(null);
     }
 
-    success(
-        `Profile "${profileName}" removed`
-    );
+    success(`Profile "${profileName}" removed`);
 }

package/src/commands/scan.js

@@ -1,24 +1,8 @@
+import { input, select } from "@inquirer/prompts";
 import path from "path";
-
-import {
-    input,
-    select,
-} from "@inquirer/prompts";
-
-import {
-    findSSHKeys,
-} from "../services/scan.js";
-
-import {
-    getProfiles,
-    saveProfile,
-} from "../services/profile.js";
-
-import {
-    error,
-    info,
-    success,
-} from "../utils/logger.js";
+import { getProfiles, saveProfile } from "../services/profile.js";
+import { findSSHKeys } from "../services/scan.js";
+import { error, info, success } from "../utils/logger.js";
 
 export async function scanCommand() {
     try {
@@ -30,9 +14,7 @@ export async function scanCommand() {
         }
 
         const existing = getProfiles();
-
         const importedPaths = existing.map((profile) => profile.sshKey);
-
         const available = keys.filter((key) => !importedPaths.includes(key));
 
         if (!available.length) {
@@ -57,18 +39,25 @@ export async function scanCommand() {
             message: "GitHub Username",
         });
 
+        if (!username.trim()) {
+            throw new Error("GitHub username is required");
+        }
+
         const email = await input({
             message: "Email",
         });
 
+        if (!email.trim()) {
+            throw new Error("Email is required");
+        }
+
         try {
             saveProfile({
                 name: profileName,
                 username,
                 email,
                 sshKey: selected,
-                source:
-                    "imported",
+                source: "imported",
             });
 
             success(`Existing SSH Key Imported "${profileName}"`);

package/src/commands/unlink.js

@@ -1,17 +1,9 @@
 import path from "path";
-
-import {
-    removeFolderMapping,
-} from "../services/profile.js";
-
+import { removeFolderMapping } from "../services/profile.js";
 import { success } from "../utils/logger.js";
 
-export async function unlinkCommand(
-    folder
-) {
+export async function unlinkCommand(folder) {
     const fullPath = path.resolve(folder);
-
     removeFolderMapping(fullPath);
-
     success(`Removed ${fullPath}`);
 }

package/src/commands/use.js

@@ -1,57 +1,27 @@
 import ora from "ora";
+import { setGitUser } from "../services/git.js";
+import { getProfile, setCurrentProfile } from "../services/profile.js";
+import { error, success } from "../utils/logger.js";
 
-import {
-    getProfile,
-    setCurrentProfile,
-} from "../services/profile.js";
-
-import {
-    setGitUser,
-} from "../services/git.js";
-
-import {
-    error,
-    success,
-} from "../utils/logger.js";
-
-export async function useCommand(
-    profileName
-) {
-    const profile =
-        getProfile(profileName);
+export async function useCommand(profileName) {
+    const profile = getProfile(profileName);
 
     if (!profile) {
-        error(
-            `Profile "${profileName}" not found`
-        );
-
+        error(`Profile "${profileName}" not found`);
         return;
     }
 
-    const spinner = ora(
-        "Switching profile..."
-    ).start();
+    const spinner = ora("Switching profile...").start();
 
     try {
-        await setGitUser(
-            profile.username,
-            profile.email
-        );
-
+        await setGitUser(profile.username, profile.email);
         setCurrentProfile(profile.name);
 
-        spinner.succeed(
-            "Profile switched"
-        );
+        spinner.succeed("Profile switched");
+        success(`Current profile: ${profile.name}`);
 
-        success(
-            `Current profile: ${profile.name}`
-        );
     } catch (err) {
-        spinner.fail(
-            "Unable to switch profile"
-        );
-
+        spinner.fail("Unable to switch profile");
         error(err.message);
     }
 }

package/src/server.js

@@ -62,9 +62,7 @@ async function checkForUpdates() {
         const latestVersion = response.data.version;
 
         if (latestVersion && compareVersions(version, latestVersion) < 0) {
-            console.log(
-                `A new version of ${name} is available: ${version} -> ${latestVersion}`,
-            );
+            console.log(`A new version of ${name} is available: ${version} -> ${latestVersion}`);
             console.log(`Run: npm install -g ${name}`);
         }
     } catch (error) {

package/src/services/git.js

@@ -1,9 +1,6 @@
 import { execa } from "execa";
 
-export async function setGitUser(
-    name,
-    email
-) {
+export async function setGitUser(name, email) {
     await execa("git", [
         "config",
         "--global",

package/src/services/profile.js

@@ -9,20 +9,27 @@ export function getProfiles() {
 }
 
 export function saveProfile(profile) {
-    const profiles = getProfiles();
 
+    if (
+        !profile.name?.trim() ||
+        !profile.username?.trim() ||
+        !profile.email?.trim()
+    ) {
+        throw new Error(
+            "Invalid profile data"
+        );
+    }
+
+    const profiles = getProfiles();
     const exists = profiles.find(
         (item) => item.name === profile.name
     );
 
     if (exists) {
-        throw new Error(
-            `Profile "${profile.name}" already exists`
-        );
+        throw new Error(`Profile "${profile.name}" already exists`);
     }
 
     profiles.push(profile);
-
     config.set("profiles", profiles);
 }
 
@@ -44,9 +51,7 @@ export function removeProfile(name) {
     config.set("profiles", filtered);
 }
 
-export function setCurrentProfile(
-    name
-) {
+export function setCurrentProfile(name) {
     if (!name) {
         config.delete("current");
         return;
@@ -63,12 +68,8 @@ export function getFolderMappings() {
     return config.get("folderMappings", []);
 }
 
-export function addFolderMapping(
-    profile,
-    folderPath
-) {
+export function addFolderMapping(profile, folderPath) {
     const mappings = getFolderMappings();
-
     const exists = mappings.find((item) => item.path === folderPath);
 
     if (exists) {
@@ -76,20 +77,10 @@ export function addFolderMapping(
     }
 
     mappings.push({ profile, path: folderPath, });
-
     config.set("folderMappings", mappings);
 }
 
-export function removeFolderMapping(
-    folderPath
-) {
+export function removeFolderMapping(folderPath) {
     const mappings = getFolderMappings();
-
-    config.set(
-        "folderMappings",
-        mappings.filter(
-            (item) =>
-                item.path !== folderPath
-        )
-    );
+    config.set("folderMappings", mappings.filter((item) => item.path !== folderPath));
 }

package/src/services/scan.js

@@ -5,9 +5,7 @@ import path from "path";
 
 export async function findSSHKeys() {
     const sshDir = path.join(os.homedir(), ".ssh");
-
-    const exists =
-        await fs.pathExists(sshDir);
+    const exists = await fs.pathExists(sshDir);
 
     if (!exists) {
         return [];

package/src/services/ssh.js

@@ -13,32 +13,21 @@ function toSafeKeyName(profileName) {
     return normalized || "profile";
 }
 
-export async function generateSSHKey(
-    profileName,
-    email
-) {
-    const safeProfileName = toSafeKeyName(
-        profileName
-    );
-
+export async function generateSSHKey(profileName, email) {
+    const safeProfileName = toSafeKeyName(profileName);
     const keyPath = path.join(
         os.homedir(),
         ".ssh",
         `gitshift-${safeProfileName}`
     );
 
-    const exists = await fs.pathExists(
-        keyPath
-    );
+    const exists = await fs.pathExists(keyPath);
 
     if (exists) {
         return keyPath;
     }
 
-    await fs.ensureDir(
-        path.dirname(keyPath)
-    );
-
+    await fs.ensureDir(path.dirname(keyPath));
     await execa("ssh-keygen", [
         "-t",
         "ed25519",
@@ -53,14 +42,9 @@ export async function generateSSHKey(
     return keyPath;
 }
 
-export async function getPublicKey(
-    privateKeyPath
-) {
-    const publicKey =
-        `${privateKeyPath}.pub`;
-
-    const exists =
-        await fs.pathExists(publicKey);
+export async function getPublicKey(privateKeyPath) {
+    const publicKey = `${privateKeyPath}.pub`;
+    const exists = await fs.pathExists(publicKey);
 
     if (!exists) {
         return null;

package/src/utils/logger.js

@@ -1,10 +1,7 @@
 import chalk from "chalk";
 
-export const success = (msg) =>
-    console.log(chalk.green(`✓ ${msg}`));
+export const success = (msg) => console.log(chalk.green(`✓ ${msg}`));
 
-export const error = (msg) =>
-    console.log(chalk.red(`✗ ${msg}`));
+export const error = (msg) => console.log(chalk.red(`✗ ${msg}`));
 
-export const info = (msg) =>
-    console.log(chalk.cyan(msg));
+export const info = (msg) => console.log(chalk.cyan(msg));