resulgit 1.0.5 → 1.0.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "resulgit",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "description": "A powerful CLI tool for version control system operations - clone, commit, push, pull, merge, branch management, and more",
   "main": "resulgit.js",
   "bin": {

package/resulgit.js

@@ -271,7 +271,7 @@ async function cmdClone(opts, cfg) {
   const server = getServer(opts, cfg)
   const token = getToken(opts, cfg)
   const repo = opts.repo
-  const branch = opts.branch
+  const branch = opts.branch || 'main'
   let dest = opts.dest
   if (!repo || !branch) throw new Error('Missing --repo and --branch')
   const spinner = createSpinner('Initializing clone...', opts.json)
@@ -540,10 +540,10 @@ async function cmdDiff(opts) {
     const snap1 = await fetchSnapshotByCommit(server, meta.repoId, commit1, token)
     const snap2 = await fetchSnapshotByCommit(server, meta.repoId, commit2, token)
     const files = filePath ? [filePath] : Object.keys(new Set([...Object.keys(snap1.files), ...Object.keys(snap2.files)]))
-    
+
     let added = 0, deleted = 0, modified = 0
     const stats = []
-    
+
     for (const p of files) {
       const content1 = snap1.files[p] !== undefined ? String(snap1.files[p]) : null
       const content2 = snap2.files[p] !== undefined ? String(snap2.files[p]) : null
@@ -561,7 +561,7 @@ async function cmdDiff(opts) {
           const diff = Math.abs(lines2.length - lines1.length)
           if (showStat) stats.push({ path: p, added: lines2.length > lines1.length ? diff : 0, deleted: lines1.length > lines2.length ? diff : 0 })
         }
-        
+
         if (!showStat) {
           if (opts.json === 'true') {
             print({ path: p, old: content1, new: content2 }, true)
@@ -585,7 +585,7 @@ async function cmdDiff(opts) {
         }
       }
     }
-    
+
     if (showStat) {
       if (opts.json === 'true') {
         print({ added, deleted, modified, files: stats }, true)
@@ -607,7 +607,7 @@ async function cmdDiff(opts) {
     const parentSnap = parentId ? await fetchSnapshotByCommit(server, meta.repoId, parentId, token) : { files: {}, commitId: '' }
 
     const files = filePath ? [filePath] : Object.keys(new Set([...Object.keys(parentSnap.files), ...Object.keys(commitSnap.files)]))
-    
+
     if (showStat) {
       let added = 0, deleted = 0, modified = 0
       const stats = []
@@ -640,7 +640,7 @@ async function cmdDiff(opts) {
       }
       return
     }
-    
+
     for (const p of files) {
       const oldContent = parentSnap.files[p] !== undefined ? String(parentSnap.files[p]) : null
       const newContent = commitSnap.files[p] !== undefined ? String(commitSnap.files[p]) : null
@@ -726,7 +726,7 @@ async function cmdRm(opts) {
   const cfg = loadConfig()
   const server = getServer(opts, cfg) || meta.server
   const token = getToken(opts, cfg) || meta.token
-  
+
   // Handle --cached flag (remove from index but keep file)
   if (opts.cached === 'true') {
     // Mark file for deletion in next commit but don't delete from filesystem
@@ -736,7 +736,7 @@ async function cmdRm(opts) {
     try {
       const s = await fs.promises.readFile(localPath, 'utf8')
       localMeta = JSON.parse(s)
-    } catch {}
+    } catch { }
     if (!localMeta.removedFiles) localMeta.removedFiles = []
     if (!localMeta.removedFiles.includes(pathArg)) {
       localMeta.removedFiles.push(pathArg)
@@ -750,7 +750,7 @@ async function cmdRm(opts) {
     }
     return
   }
-  
+
   const u = new URL(`/api/repositories/${meta.repoId}/files`, server)
   u.searchParams.set('branch', meta.branch)
   u.searchParams.set('path', pathArg)
@@ -776,7 +776,7 @@ async function cmdCommit(opts) {
       spinnerUpdate(spinner, 'Staging all changes...')
       await cmdAdd({ dir, all: 'true', json: opts.json })
     }
-    
+
     // Handle --amend flag
     if (opts.amend === 'true') {
       spinnerUpdate(spinner, 'Amending last commit...')
@@ -823,7 +823,7 @@ async function cmdCommit(opts) {
     const local = await collectLocal(dir)
     const files = {}
     for (const [p, v] of Object.entries(local)) files[p] = v.content
-    
+
     // Execute pre-commit hook
     spinnerUpdate(spinner, 'Running pre-commit hook...')
     try {
@@ -836,7 +836,7 @@ async function cmdCommit(opts) {
       if (err.message === 'pre-commit hook failed') throw err
       // Hook doesn't exist or other error, continue
     }
-    
+
     localMeta.pendingCommit = { message, files, createdAt: Date.now() }
     // Clear conflicts if they were resolved
     if (localMeta.conflicts) {
@@ -844,12 +844,12 @@ async function cmdCommit(opts) {
     }
     await fs.promises.mkdir(metaDir, { recursive: true })
     await fs.promises.writeFile(localPath, JSON.stringify(localMeta, null, 2))
-    
+
     // Execute post-commit hook
     try {
       await hooks.executeHook(dir, 'post-commit', { message, files: Object.keys(files) })
-    } catch {}
-    
+    } catch { }
+
     spinnerSuccess(spinner, `Staged changes for commit: "${message}"`)
     print({ pendingCommit: message }, opts.json === 'true')
   } catch (err) {
@@ -1395,7 +1395,7 @@ async function cmdPush(opts) {
       if (err.message === 'pre-push hook failed') throw err
       // Hook doesn't exist or other error, continue
     }
-    
+
     const body = { message: localMeta.pendingCommit?.message || (opts.message || 'Push'), files: merged, branchName: remoteMeta.branch }
     spinnerUpdate(spinner, `Pushing to '${remoteMeta.branch}'...`)
     const url = new URL(`/api/repositories/${remoteMeta.repoId}/commits`, server).toString()
@@ -1404,12 +1404,12 @@ async function cmdPush(opts) {
     localMeta.baseFiles = merged
     localMeta.pendingCommit = null
     await fs.promises.writeFile(localPath, JSON.stringify(localMeta, null, 2))
-    
+
     // Execute post-push hook
     try {
       await hooks.executeHook(dir, 'post-push', { branch: remoteMeta.branch, commitId: localMeta.baseCommitId })
-    } catch {}
-    
+    } catch { }
+
     spinnerSuccess(spinner, `Pushed to '${remoteMeta.branch}' (commit: ${(data.id || '').slice(0, 7)})`)
     print({ pushed: localMeta.baseCommitId }, opts.json === 'true')
   } catch (err) {
@@ -1758,7 +1758,7 @@ async function cmdBranch(sub, opts) {
       } catch { }
       process.stdout.write(color('Branches:\n', 'bold'))
       let list = (data.branches || []).map(b => ({ name: b.name, commitId: b.commitId || '', date: b.lastCommitDate || b.createdAt || '' }))
-      
+
       // Handle --sort option
       if (opts.sort) {
         const sortBy = opts.sort.replace(/^-/, '') // Remove leading dash
@@ -1770,7 +1770,7 @@ async function cmdBranch(sub, opts) {
           })
         }
       }
-      
+
       for (const b of list) {
         const isCur = b.name === current
         const mark = isCur ? color('*', 'green') : ' '
@@ -1838,7 +1838,7 @@ async function cmdSwitch(opts) {
   const cfg = loadConfig()
   const server = getServer(opts, cfg) || meta.server
   const token = getToken(opts, cfg) || meta.token
-  
+
   // Handle -c flag (create and switch)
   if (create) {
     // Check if branch exists
@@ -1850,7 +1850,7 @@ async function cmdSwitch(opts) {
       await cmdBranch('create', { dir, name: branch, base: meta.branch, repo: meta.repoId, server, token })
     }
   }
-  
+
   await pullToDir(meta.repoId, branch, dir, server, token)
   print({ repoId: meta.repoId, branch, dir }, opts.json === 'true')
 }
@@ -2301,8 +2301,10 @@ async function cmdInit(opts) {
   const cfg = loadConfig()
   const server = getServer(opts, cfg)
   const token = getToken(opts, cfg)
-  const repo = opts.repo || ''
-  const branch = opts.branch || 'main'
+  // Repository name must be a non‑empty string; validate format
+  const repo = opts.repo ? validation.validateRepoName(opts.repo) : ''
+  // Branch name defaults to 'main' if not supplied
+  const branch = opts.branch ? opts.branch : 'main'
   const metaDir = path.join(dir, '.vcs-next')
   const gitDir = path.join(dir, '.git')
   await fs.promises.mkdir(metaDir, { recursive: true })
@@ -2602,38 +2604,38 @@ async function cmdBlame(opts) {
   const dir = path.resolve(opts.dir || '.')
   const filePath = opts.path
   if (!filePath) throw new errors.ValidationError('Missing --path', 'path')
-  
+
   const validPath = validation.validateFilePath(filePath)
   const meta = readRemoteMeta(dir)
   const cfg = loadConfig()
   const server = getServer(opts, cfg) || meta.server
   const token = getToken(opts, cfg) || meta.token
-  
+
   const spinner = createSpinner(`Getting blame for ${validPath}...`, opts.json)
-  
+
   try {
     // Get file content
     const local = await collectLocal(dir)
     if (!local[validPath]) {
       throw new errors.FileSystemError(`File not found: ${validPath}`, validPath, 'read')
     }
-    
+
     // Get commits
     const commitsUrl = new URL(`/api/repositories/${meta.repoId}/commits`, server)
     commitsUrl.searchParams.set('branch', meta.branch)
     const commitsRes = await fetch(commitsUrl.toString(), {
       headers: token ? { Authorization: `Bearer ${token}` } : {}
     })
-    
+
     if (!commitsRes.ok) {
       throw new errors.NetworkError('Failed to fetch commits', commitsRes.status, commitsUrl.toString())
     }
-    
+
     const commits = await commitsRes.json()
     const blameData = parseBlame(local[validPath].content, commits, validPath)
-    
+
     spinnerSuccess(spinner, `Blame for ${validPath}`)
-    
+
     if (opts.json === 'true') {
       print(formatBlameJson(blameData), false)
     } else {
@@ -2651,17 +2653,17 @@ async function cmdLog(opts) {
   const cfg = loadConfig()
   const server = getServer(opts, cfg) || meta.server
   const token = getToken(opts, cfg) || meta.token
-  
+
   const spinner = createSpinner('Fetching commit history...', opts.json)
-  
+
   try {
     const url = new URL(`/api/repositories/${meta.repoId}/commits`, server)
     if (opts.branch) url.searchParams.set('branch', opts.branch)
     else url.searchParams.set('branch', meta.branch)
-    
+
     const data = await request('GET', url.toString(), null, token)
     let commits = Array.isArray(data) ? data : []
-    
+
     // Filter by file path if provided
     const filePath = opts.path
     if (filePath) {
@@ -2674,7 +2676,7 @@ async function cmdLog(opts) {
       }
       commits = filteredCommits
     }
-    
+
     // Filter by content pattern (-G flag)
     const pattern = opts.G || opts.pattern
     if (pattern) {
@@ -2684,7 +2686,7 @@ async function cmdLog(opts) {
         const commitSnap = await fetchSnapshotByCommit(server, meta.repoId, commit.id || commit._id, token)
         const parentId = (Array.isArray(commit.parents) && commit.parents[0]) || ''
         const parentSnap = parentId ? await fetchSnapshotByCommit(server, meta.repoId, parentId, token) : { files: {} }
-        
+
         // Check if pattern matches in any file changed in this commit
         const allPaths = new Set([...Object.keys(parentSnap.files), ...Object.keys(commitSnap.files)])
         let matches = false
@@ -2702,9 +2704,9 @@ async function cmdLog(opts) {
       }
       commits = filteredCommits
     }
-    
+
     spinnerSuccess(spinner, `Found ${commits.length} commits`)
-    
+
     if (opts.json === 'true') {
       print(commits, true)
     } else if (opts.oneline === 'true') {
@@ -2733,7 +2735,7 @@ async function cmdLog(opts) {
 
 async function cmdHook(sub, opts) {
   const dir = path.resolve(opts.dir || '.')
-  
+
   if (sub === 'list') {
     const hooksList = await hooks.listHooks(dir)
     if (opts.json === 'true') {
@@ -2751,18 +2753,18 @@ async function cmdHook(sub, opts) {
     }
     return
   }
-  
+
   if (sub === 'install') {
     const hookName = opts.name
     if (!hookName) throw new Error('Missing --name')
-    
+
     let script = opts.script
     if (!script && opts.sample === 'true') {
       script = hooks.SAMPLE_HOOKS[hookName]
       if (!script) throw new Error(`No sample available for ${hookName}`)
     }
     if (!script) throw new Error('Missing --script or use --sample')
-    
+
     const result = await hooks.installHook(dir, hookName, script)
     print(result, opts.json === 'true')
     if (opts.json !== 'true') {
@@ -2770,20 +2772,20 @@ async function cmdHook(sub, opts) {
     }
     return
   }
-  
+
   if (sub === 'remove') {
     const hookName = opts.name
     if (!hookName) throw new Error('Missing --name')
-    
+
     const result = await hooks.removeHook(dir, hookName)
     print(result, opts.json === 'true')
     return
   }
-  
+
   if (sub === 'show') {
     const hookName = opts.name
     if (!hookName) throw new Error('Missing --name')
-    
+
     const result = await hooks.readHook(dir, hookName)
     if (result.content) {
       process.stdout.write(result.content + '\n')
@@ -2792,7 +2794,7 @@ async function cmdHook(sub, opts) {
     }
     return
   }
-  
+
   throw new Error('Unknown hook subcommand. Use: list, install, remove, show')
 }
 
@@ -2800,13 +2802,13 @@ async function cmdGrep(opts) {
   const dir = path.resolve(opts.dir || '.')
   const pattern = opts.pattern || opts.p || ''
   if (!pattern) throw new Error('Missing --pattern or -p')
-  
+
   const meta = readRemoteMeta(dir)
   const local = await collectLocal(dir)
   const results = []
-  
+
   const regex = new RegExp(pattern, opts.ignoreCase === 'true' ? 'i' : '')
-  
+
   for (const [filePath, fileData] of Object.entries(local)) {
     const lines = fileData.content.split(/\r?\n/)
     for (let i = 0; i < lines.length; i++) {
@@ -2819,7 +2821,7 @@ async function cmdGrep(opts) {
       }
     }
   }
-  
+
   if (opts.json === 'true') {
     print(results, true)
   } else {
@@ -2836,10 +2838,10 @@ async function cmdLsFiles(opts) {
   const cfg = loadConfig()
   const server = getServer(opts, cfg) || meta.server
   const token = getToken(opts, cfg) || meta.token
-  
+
   const remote = await fetchRemoteFilesMap(server, meta.repoId, meta.branch, token)
   const files = Object.keys(remote.map)
-  
+
   if (opts.json === 'true') {
     print(files.map(f => ({ path: f })), true)
   } else {
@@ -2853,13 +2855,13 @@ async function cmdReflog(opts) {
   const dir = path.resolve(opts.dir || '.')
   const metaDir = path.join(dir, '.vcs-next')
   const reflogPath = path.join(metaDir, 'reflog.json')
-  
+
   let reflog = []
   try {
     const content = await fs.promises.readFile(reflogPath, 'utf8')
     reflog = JSON.parse(content)
-  } catch {}
-  
+  } catch { }
+
   if (opts.json === 'true') {
     print(reflog, true)
   } else {
@@ -2878,14 +2880,14 @@ async function cmdCatFile(opts) {
   const dir = path.resolve(opts.dir || '.')
   const type = opts.type || ''
   const object = opts.object || ''
-  
+
   if (!type || !object) throw new Error('Missing --type and --object')
-  
+
   const meta = readRemoteMeta(dir)
   const cfg = loadConfig()
   const server = getServer(opts, cfg) || meta.server
   const token = getToken(opts, cfg) || meta.token
-  
+
   if (type === 'blob') {
     const snap = await fetchSnapshotByCommit(server, meta.repoId, object, token)
     const filePath = opts.path || ''
@@ -2912,12 +2914,12 @@ async function cmdCatFile(opts) {
 async function cmdRevParse(opts) {
   const dir = path.resolve(opts.dir || '.')
   const rev = opts.rev || 'HEAD'
-  
+
   const meta = readRemoteMeta(dir)
   const cfg = loadConfig()
   const server = getServer(opts, cfg) || meta.server
   const token = getToken(opts, cfg) || meta.token
-  
+
   if (rev === 'HEAD') {
     const info = await request('GET', new URL(`/api/repositories/${meta.repoId}/branches`, server).toString(), null, token)
     const found = (info.branches || []).find(b => b.name === meta.branch)
@@ -2938,20 +2940,20 @@ async function cmdRevParse(opts) {
 async function cmdDescribe(opts) {
   const dir = path.resolve(opts.dir || '.')
   const commitId = opts.commit || 'HEAD'
-  
+
   const meta = readRemoteMeta(dir)
   const cfg = loadConfig()
   const server = getServer(opts, cfg) || meta.server
   const token = getToken(opts, cfg) || meta.token
-  
+
   // Get tags
   const tagsUrl = new URL(`/api/repositories/${meta.repoId}/tags`, server)
   const tags = await request('GET', tagsUrl.toString(), null, token)
   const tagsList = Array.isArray(tags) ? tags : []
-  
+
   // Find nearest tag (simplified - just find any tag)
   const nearestTag = tagsList[0]
-  
+
   if (nearestTag) {
     const desc = `${nearestTag.name}-0-g${commitId.slice(0, 7)}`
     print(opts.json === 'true' ? { tag: nearestTag.name, commitId, describe: desc } : desc, opts.json === 'true')
@@ -2966,16 +2968,16 @@ async function cmdShortlog(opts) {
   const cfg = loadConfig()
   const server = getServer(opts, cfg) || meta.server
   const token = getToken(opts, cfg) || meta.token
-  
+
   const url = new URL(`/api/repositories/${meta.repoId}/commits`, server)
   if (opts.branch) url.searchParams.set('branch', opts.branch)
   else url.searchParams.set('branch', meta.branch)
-  
+
   const commits = await request('GET', url.toString(), null, token)
   const commitsList = Array.isArray(commits) ? commits : []
-  
+
   const stats = generateCommitStats(commitsList)
-  
+
   if (opts.json === 'true') {
     print(stats, true)
   } else {
@@ -3044,8 +3046,8 @@ function help() {
     '    >>>>>>> incoming (incoming changes)',
     '  Resolve conflicts manually, then commit and push. Push is blocked until conflicts are resolved.',
     '  pr list|create|merge [--dir <path>] [--title <t>] [--id <id>] [--source <branch>] [--target <branch>]',
-    '  clone <url> [--dest <dir>] | --repo <id> --branch <name> [--dest <dir>] [--server <url>] [--token <token>]',
-    '  init [--dir <path>] [--repo <id>] [--server <url>] [--branch <name>] [--token <tok>]',
+    '  clone <url> [--dest <dir>] | --repo <name> --branch <name> [--dest <dir>] [--server <url>] [--token <token>]',
+    '  init [--dir <path>] [--repo <name>] [--server <url>] [--branch <name>] [--token <tok>]',
     '  remote show|set-url|set-token [--dir <path>] [--server <url>] [--token <tok>]',
     '  config list|get|set [--key <k>] [--value <v>]',
     '  clean [--dir <path>] [--force]',
@@ -3078,13 +3080,20 @@ async function main() {
     return
   }
   if (cmd[0] === 'clone') {
-    if (!opts.url && cmd[1] && !cmd[1].startsWith('--')) opts.url = cmd[1]
+    const arg = cmd[1];
+    if (arg && !arg.startsWith('--')) {
+      if (arg.includes('://')) {
+        opts.url = arg;
+      } else {
+        opts.repo = arg;
+      }
+    }
     if (opts.url) {
-      await cmdCloneFromUrl(opts, cfg)
+      await cmdCloneFromUrl(opts, cfg);
     } else {
-      await cmdClone(opts, cfg)
+      await cmdClone(opts, cfg);
     }
-    return
+    return;
   }
   if (cmd[0] === 'status') {
     await cmdStatus(opts)