storymaps 1.2.1 → 1.2.2

package/lib/commands/push.js

@@ -7,7 +7,8 @@ import { resolve } from 'node:path';
 import chalk from 'chalk';
 import { password } from '@inquirer/prompts';
 import jsyaml from 'js-yaml';
-import { importFromYaml } from '../core/yaml.js';
+import { exportToYaml, importFromYaml, yamlObjToJson } from '../core/yaml.js';
+import { printChangeSummary } from '../util/summary.js';
 import { resolveTarget } from '../util/identity.js';
 import { getMap, putMap, postMap, ApiError } from '../util/api.js';
 import { detectFormat, readCache, writeCache } from '../util/file.js';
@@ -70,6 +71,14 @@ export async function pushCommand(target, opts) {
 
     const contentType = format === 'json' ? 'application/json' : 'text/yaml';
 
+    // Show a change summary before pushing
+    try {
+        const { data: remoteData } = await getMap(baseUrl, id, 'yaml');
+        const remoteNorm = exportToYaml(yamlObjToJson(jsyaml.load(remoteData)));
+        const localNorm = exportToYaml(yamlObjToJson(jsyaml.load(content)));
+        if (remoteNorm !== localNorm) printChangeSummary(remoteNorm, localNorm);
+    } catch { /* non-fatal */ }
+
     const cache = await readCache(id);
     let etag = opts.force ? undefined : cache.etag;
     let lockPasswordHash = cache.hash;

package/lib/util/summary.js

@@ -42,26 +42,43 @@ export function changeSummary(oldYaml, newYaml) {
         lines.push(chalk.red(`  - slice: ${s}`));
     }
 
-    // Count story-level changes per slice
-    const countStories = (map) => {
-        const counts = {};
+    // Detect card-level changes per slice (name-aware, handles renames)
+    const collectCardNames = (map) => {
+        const result = {};
         for (const slice of (map.slices || [])) {
-            const stories = slice.stories || {};
-            counts[slice.name] = Object.values(stories)
-                .reduce((n, cards) => n + (Array.isArray(cards) ? cards.length : 0), 0);
+            const names = [];
+            for (const cards of Object.values(slice.stories || {})) {
+                if (Array.isArray(cards)) names.push(...cards.map(c => c.name).filter(Boolean));
+            }
+            result[slice.name] = names;
         }
-        return counts;
+        return result;
     };
-    const oldCounts = countStories(old);
-    const newCounts = countStories(cur);
-    for (const name of new Set([...Object.keys(oldCounts), ...Object.keys(newCounts)])) {
-        const oc = oldCounts[name] || 0;
-        const nc = newCounts[name] || 0;
-        if (oc !== nc) {
-            const delta = nc - oc;
-            const word = Math.abs(delta) === 1 ? 'story' : 'stories';
-            if (delta > 0) lines.push(chalk.green(`  + ${name}: ${delta} ${word}`));
-            else lines.push(chalk.red(`  - ${name}: ${Math.abs(delta)} ${word}`));
+    const oldCards = collectCardNames(old);
+    const newCards = collectCardNames(cur);
+    for (const sliceName of new Set([...Object.keys(oldCards), ...Object.keys(newCards)])) {
+        const oldNames = oldCards[sliceName] || [];
+        const newNames = newCards[sliceName] || [];
+        const oldCounts = new Map();
+        const newCounts = new Map();
+        for (const n of oldNames) oldCounts.set(n, (oldCounts.get(n) || 0) + 1);
+        for (const n of newNames) newCounts.set(n, (newCounts.get(n) || 0) + 1);
+        const allNames = new Set([...oldCounts.keys(), ...newCounts.keys()]);
+        let added = 0, removed = 0;
+        for (const n of allNames) {
+            const o = oldCounts.get(n) || 0;
+            const c = newCounts.get(n) || 0;
+            if (c > o) added += c - o;
+            if (o > c) removed += o - c;
+        }
+        if (added === 0 && removed === 0) continue;
+        const net = newNames.length - oldNames.length;
+        if (net === 0) {
+            const word = added === 1 ? 'card' : 'cards';
+            lines.push(chalk.yellow(`  ~ ${sliceName}: ${added} ${word} renamed`));
+        } else {
+            if (added) lines.push(chalk.green(`  + ${sliceName}: ${added} ${added === 1 ? 'card' : 'cards'}`));
+            if (removed) lines.push(chalk.red(`  - ${sliceName}: ${removed} ${removed === 1 ? 'card' : 'cards'}`));
         }
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "storymaps",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "description": "CLI tool for storymaps.io - import, export, validate, and manage user story maps from the terminal",
   "type": "module",
   "bin": {