gent-cli 14.0.0 → 15.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gent-cli",
-  "version": "14.0.0",
+  "version": "15.0.0",
   "description": "A modern, Git-like version control CLI with cloud sync, AI-powered superpowers (ask/review/docs/changelog), and zero-friction setup (gent setup/doctor/config).",
   "main": "src/index.js",
   "bin": {

package/src/commands/pet.js

@@ -402,63 +402,83 @@ const up = (n) => `\x1b[${n}A`;
  * @param {boolean} opts.altScreen  use the alternate screen buffer
  * @returns {Promise<void>}
  */
-function play(sceneName, { loop = false, footer = true, goodbye = false, altScreen = false, maxTicks = Infinity } = {}) {
-    return new Promise((resolve) => {
-        const scene = SCENES[sceneName] || SCENES.idle;
-        const cv = new Canvas(CV_W, CV_H);
-        const state = { count: 0, tip: TIPS[Math.floor(Math.random() * TIPS.length)] };
-        const totalTicks = loop ? Infinity : Math.min(scene.cycle + 1, maxTicks); // one clean cycle, capped
-        let t = 0;
-        let printed = false;
-        let done = false;
-
-        const footerLine = () => footer
-            ? chalk.gray('  ') +
-              (loop ? chalk.gray('Ctrl+C to leave') : C.body('Genti')) +
-              chalk.gray('   ·   more scenes: ') + C.say('gent pet push|pull|merge')
-            : '';
-
-        const paint = () => {
-            cv.clear();
-            scene.fn(cv, t, state);
-            if (sceneName === 'idle' && loop && t > 0 && t % scene.cycle === 0) {
-                state.tip = TIPS[Math.floor(Math.random() * TIPS.length)];
-            }
-            const lines = cv.render().split('\n');
-            lines.push(footerLine());
-            const block = lines.map(l => CLEAR_LINE + l).join('\n');
-            if (altScreen) {
-                process.stdout.write(HOME + block);
-            } else {
-                if (printed) process.stdout.write(up(lines.length));
-                process.stdout.write(block + '\n');
-            }
-            printed = true;
-        };
-
-        const finish = () => {
-            if (done) return;
-            done = true;
-            clearInterval(timer);
-            process.removeListener('SIGINT', onSig);
-            process.stdout.write(SHOW + (altScreen ? LEAVE_ALT : ''));
-            if (goodbye) {
-                console.log(C.body('  Genti waves.') + chalk.gray(' Come back with ') + C.say('gent pet') + chalk.gray('.'));
-            }
-            resolve();
-        };
+function runLoop(sceneName, {
+    loop = false, footer = true, goodbye = false, altScreen = false,
+    maxTicks = Infinity, clearOnStop = false, exitOnSig = false,
+} = {}) {
+    const scene = SCENES[sceneName] || SCENES.idle;
+    const cv = new Canvas(CV_W, CV_H);
+    const state = { count: 0, tip: TIPS[Math.floor(Math.random() * TIPS.length)] };
+    const totalTicks = loop ? Infinity : Math.min(scene.cycle + 1, maxTicks);
+    const blockH = CV_H + (footer ? 1 : 0);
+    let t = 0;
+    let printed = false;
+    let done = false;
+    let resolveFn;
+    const promise = new Promise((r) => { resolveFn = r; });
+
+    const footerLine = () => footer
+        ? chalk.gray('  ') +
+          (loop ? chalk.gray('Ctrl+C to cancel') : C.body('Genti')) +
+          chalk.gray('   ·   more scenes: ') + C.say('gent pet push|pull|merge')
+        : '';
+
+    const paint = () => {
+        cv.clear();
+        scene.fn(cv, t, state);
+        if (sceneName === 'idle' && loop && t > 0 && t % scene.cycle === 0) {
+            state.tip = TIPS[Math.floor(Math.random() * TIPS.length)];
+        }
+        const lines = cv.render().split('\n');
+        lines.push(footerLine());
+        const block = lines.map(l => CLEAR_LINE + l).join('\n');
+        if (altScreen) {
+            process.stdout.write(HOME + block);
+        } else {
+            if (printed) process.stdout.write(up(lines.length));
+            process.stdout.write(block + '\n');
+        }
+        printed = true;
+    };
+
+    // Erase the inline block so the caller can print clean output in its place.
+    const eraseBlock = () => {
+        if (!printed) return;
+        process.stdout.write(up(blockH));
+        for (let i = 0; i < blockH; i++) process.stdout.write(CLEAR_LINE + (i < blockH - 1 ? '\n' : ''));
+        process.stdout.write(up(blockH - 1));
+    };
+
+    const stop = () => {
+        if (done) return;
+        done = true;
+        clearInterval(timer);
+        process.removeListener('SIGINT', onSig);
+        if (clearOnStop && !altScreen) eraseBlock();
+        process.stdout.write(SHOW + (altScreen ? LEAVE_ALT : ''));
+        if (goodbye) {
+            console.log(C.body('  Genti waves.') + chalk.gray(' Come back with ') + C.say('gent pet') + chalk.gray('.'));
+        }
+        resolveFn();
+    };
 
-        const onSig = () => finish();
+    const onSig = () => { stop(); if (exitOnSig) process.exit(130); };
 
-        process.stdout.write((altScreen ? ENTER_ALT + HOME : '') + HIDE);
+    process.stdout.write((altScreen ? ENTER_ALT + HOME : '') + HIDE);
+    paint();
+    const timer = setInterval(() => {
+        t++;
+        if (t >= totalTicks) { paint(); return stop(); }
         paint();
-        const timer = setInterval(() => {
-            t++;
-            if (t >= totalTicks) { paint(); return finish(); }
-            paint();
-        }, FRAME_MS);
-        process.on('SIGINT', onSig);
-    });
+    }, FRAME_MS);
+    process.on('SIGINT', onSig);
+
+    return { promise, stop };
+}
+
+// Play a scene to completion (or until Ctrl+C when looping). Resolves when done.
+function play(sceneName, opts = {}) {
+    return runLoop(sceneName, opts).promise;
 }
 
 /**
@@ -528,6 +548,32 @@ async function celebrate(scene) {
     } catch (_) { /* ignore — decoration only */ }
 }
 
+/**
+ * Public: run `task` while Genti animates as the live loader — carrying the
+ * crate to the cloud on push, home on pull, etc. The animation loops until the
+ * task settles, then erases itself so the command can print its own result.
+ *
+ * Safe + transparent: with no TTY / GENT_NO_PET / CI it simply awaits the task
+ * with no animation. Always returns (or throws) exactly what `task` does.
+ *
+ * @param {string} scene           'push' | 'pull' | 'merge' | …
+ * @param {() => Promise<any>} task the real async work (e.g. the network call)
+ * @returns {Promise<any>}
+ */
+async function during(scene, task) {
+    if (!process.stdout.isTTY || process.env.GENT_NO_PET || process.env.CI || !SCENES[scene]) {
+        return task();
+    }
+    const anim = runLoop(scene, { loop: true, footer: true, altScreen: false, clearOnStop: true, exitOnSig: true });
+    try {
+        return await task();
+    } finally {
+        anim.stop();
+        await anim.promise;
+    }
+}
+
 module.exports = petCommand;
 module.exports.celebrate = celebrate;
 module.exports.banner = banner;
+module.exports.during = during;

package/src/commands/pull.js

@@ -73,14 +73,15 @@ async function pull(remoteName, branchName, options) {
 
         // 1. Fetch commits + objects for this branch in a single call. `since`
         //    lets the server send only what we don't have on a fast-forward.
-        spinner.text = `Fetching updates for ${branch}...`;
+        // Genti walks a crate home from the cloud while we fetch.
+        spinner.stop();
         let pullData;
         try {
             const pullUrl = buildRepoUrl(API_ENDPOINTS.REPO_PULL, repoInfo);
             const query = localHead
                 ? `?branch=${encodeURIComponent(branch)}&since=${encodeURIComponent(localHead)}`
                 : `?branch=${encodeURIComponent(branch)}`;
-            pullData = await apiClient.get(pullUrl + query);
+            pullData = await pet.during('pull', () => apiClient.get(pullUrl + query));
         } catch (error) {
             if (error.response?.status === 404) {
                 spinner.succeed(chalk.green('Remote branch not found — nothing to pull'));
@@ -138,7 +139,6 @@ async function pull(remoteName, branchName, options) {
 
             spinner.succeed(chalk.green(`Fast-forward: ${newCount} new commit(s)`));
             console.log(chalk.gray(`  ${remote}/${branch} → ${remoteHead.substring(0, 7)}`));
-            await pet.celebrate('pull');
         } else {
             // Diverged — need 3-way merge
             spinner.text = 'Branches diverged, merging...';
@@ -194,7 +194,6 @@ async function pull(remoteName, branchName, options) {
             } else {
                 spinner.succeed(chalk.green(`Merged ${newCount} remote commit(s)`));
                 console.log(chalk.gray(`  Merge commit: ${mergeCommit.hash.substring(0, 7)}`));
-                await pet.celebrate('pull');
             }
         }
     } catch (error) {

package/src/commands/push.js

@@ -232,20 +232,19 @@ async function push(remoteName, branchName, options) {
             tags: tagsToPush
         };
 
-        // Send to backend
+        // Send to backend — Genti carries the crate to the cloud while we wait.
         const pushUrl = buildRepoUrl(API_ENDPOINTS.REPO_PUSH, repoInfo);
-        const response = await apiClient.post(pushUrl, payload);
+        spinner.stop();
+        const response = await pet.during('push', () => apiClient.post(pushUrl, payload));
 
         // Update remote ref
         config.remoteRefs[`${remote}/${branch}`] = localHead;
         await writeJSON(configPath, config);
 
-        spinner.succeed(chalk.green(`Pushed ${commitsToPush.length} commit(s) to ${remote}/${branch}`));
+        console.log(chalk.green(`✔ Pushed ${commitsToPush.length} commit(s) to ${remote}/${branch}`));
         console.log(chalk.gray(`  ${localHead.substring(0, 7)} → ${remote}/${branch}`));
         console.log(chalk.gray(`  ${packBlobs.length} blob(s), ${packTrees.length} tree(s) transferred`));
 
-        await pet.celebrate('push');
-
     } catch (error) {
         spinner.fail(chalk.red('Push failed'));