@mittwald/flow-codemods 1.1.8 → 1.1.9

package/dist/cli/upgrade.d.ts

@@ -13,6 +13,13 @@ export interface UpgradeDeps {
     isDirty: (cwd: string) => boolean;
     readInstalledVersion: (cwd: string, name: string) => string | undefined;
     log: (message: string) => void;
+    /**
+     * Emit ANSI colour in the by-hand list. Off by default so a test sees plain
+     * text.
+     */
+    color?: boolean;
+    /** Terminal width to wrap the by-hand list's prose to. */
+    width?: number;
 }
 export declare const defaultUpgradeDeps: (cwd: string) => UpgradeDeps;
 /**

package/dist/cli/upgrade.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"upgrade.d.ts","sourceRoot":"","sources":["../../src/cli/upgrade.ts"],"names":[],"mappings":"AAGA,OAAO,EAAc,KAAK,YAAY,EAAE,MAAM,uBAAuB,CAAC;AAItE,OAAO,EAIL,KAAK,aAAa,EACnB,MAAM,eAAe,CAAC;AAEvB,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AAEvD,OAAO,EAAE,UAAU,EAAsB,MAAM,uBAAuB,CAAC;AACvE,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,WAAW,CAAC;AAI/C,MAAM,WAAW,WAAW;IAC1B,GAAG,EAAE,MAAM,CAAC;IACZ,aAAa,EAAE,OAAO,aAAa,CAAC;IACpC,OAAO,EAAE,aAAa,CAAC;IACvB,UAAU,EAAE,OAAO,UAAU,CAAC;IAC9B,2EAA2E;IAC3E,MAAM,EAAE,CAAC,OAAO,EAAE,YAAY,EAAE,KAAK,OAAO,CAAC,YAAY,EAAE,CAAC,CAAC;IAC7D,OAAO,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC;IAClC,oBAAoB,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,KAAK,MAAM,GAAG,SAAS,CAAC;IACxE,GAAG,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,CAAC;CAChC;AAED,eAAO,MAAM,kBAAkB,QAAS,MAAM,KAAG,WAS/C,CAAC;AA2BH;;;;;;;;;;;;;;GAcG;AACH,eAAO,MAAM,UAAU,WACb,aAAa,QACf,WAAW,KAChB,OAAO,CAAC,MAAM,CAiNhB,CAAC"}
+{"version":3,"file":"upgrade.d.ts","sourceRoot":"","sources":["../../src/cli/upgrade.ts"],"names":[],"mappings":"AAGA,OAAO,EAAc,KAAK,YAAY,EAAE,MAAM,uBAAuB,CAAC;AAItE,OAAO,EAIL,KAAK,aAAa,EACnB,MAAM,eAAe,CAAC;AAEvB,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AAEvD,OAAO,EAAE,UAAU,EAAsB,MAAM,uBAAuB,CAAC;AACvE,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,WAAW,CAAC;AAI/C,MAAM,WAAW,WAAW;IAC1B,GAAG,EAAE,MAAM,CAAC;IACZ,aAAa,EAAE,OAAO,aAAa,CAAC;IACpC,OAAO,EAAE,aAAa,CAAC;IACvB,UAAU,EAAE,OAAO,UAAU,CAAC;IAC9B,2EAA2E;IAC3E,MAAM,EAAE,CAAC,OAAO,EAAE,YAAY,EAAE,KAAK,OAAO,CAAC,YAAY,EAAE,CAAC,CAAC;IAC7D,OAAO,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC;IAClC,oBAAoB,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,KAAK,MAAM,GAAG,SAAS,CAAC;IACxE,GAAG,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,CAAC;IAC/B;;;OAGG;IACH,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,0DAA0D;IAC1D,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,eAAO,MAAM,kBAAkB,QAAS,MAAM,KAAG,WAS/C,CAAC;AA2BH;;;;;;;;;;;;;;GAcG;AACH,eAAO,MAAM,UAAU,WACb,aAAa,QACf,WAAW,KAChB,OAAO,CAAC,MAAM,CAqNhB,CAAC"}

package/dist/cli/upgrade.js

@@ -187,26 +187,20 @@ export const runUpgrade = async (parsed, deps) => {
             log(`  ${entry.id}: ${result.changed} file(s) changed`);
         }
     }
-    // Selection has no lower bound (see `selectEntries`), so this loop can run
-    // every codemod in the catalogue on a project that never crossed a version
-    // at all — "N run, 0 changed" is the confirmation that there was nothing to
-    // catch up on, not a list of new work.
-    if (ranCount > 0) {
-        log(`\n${ranCount} codemod${ranCount === 1 ? "" : "s"} run, ${changedCount} changed something.`);
-    }
     if (byHand.length > 0) {
         log(`\n${byHand.length} migration(s) in this range have no codemod — apply them by hand:\n`);
-        // `frame: false` — this already printed the heading above and, further
-        // down, its own aggregate ("N codemods run, N changed something");
-        // renderList's own frame (the range/legend on top, the counts at the
-        // bottom) would just repeat both for the same list. `range` is still
-        // passed so each entry gets its catch-up mark — a manual entry that
-        // shipped at or before `current` may already be done, and the mark is
-        // the only thing left that says so now that hiding it is gone.
+        // `frame: false` — this already printed the heading above, and the
+        // closing summaries below cover the same aggregate ("N codemods run, N
+        // changed something") renderList's own frame would otherwise repeat.
+        // `range` is still passed so each entry gets its catch-up mark — a manual
+        // entry that shipped at or before `current` may already be done, and the
+        // mark is the only thing left that says so now that hiding it is gone.
         log(renderList({
             entries: byHand,
             range: { from: current, to: target },
             json: false,
+            color: deps.color,
+            width: deps.width,
             // The same path this run used, so a command copied out of the by-hand
             // block works instead of falling back to a hardcoded `src`.
             path: displaySourcePath(parsed.path, cwd),
@@ -217,6 +211,16 @@ export const runUpgrade = async (parsed, deps) => {
     else {
         log("\nNo migration in this range required a change by hand.");
     }
+    // Both closing summaries trail everything else, not just each other: a
+    // multi-screen by-hand list would otherwise scroll them out of sight before
+    // the reader reaches the end — exactly when they want the counts. Selection
+    // has no lower bound (see `selectEntries`), so this loop can run every
+    // codemod in the catalogue on a project that never crossed a version at
+    // all — "N run, 0 changed" is the confirmation that there was nothing to
+    // catch up on, not a list of new work.
+    if (ranCount > 0) {
+        log(`\n${ranCount} codemod${ranCount === 1 ? "" : "s"} run, ${changedCount} changed something.`);
+    }
     if (incomplete.length > 0) {
         log(`\n${incomplete.length} codemod(s) did not complete: ${incomplete.join(", ")}`);
     }

package/dist/cli.js

@@ -107,6 +107,10 @@ const main = async () => {
             return runUpgrade(parsed, {
                 ...defaultUpgradeDeps(process.cwd()),
                 choose,
+                // Same rule as `list`: colour only for a person at a real terminal,
+                // off under `NO_COLOR` or a pipe so the output stays greppable.
+                color: process.stdout.isTTY === true && process.env.NO_COLOR === undefined,
+                width: Math.min(Math.max(process.stdout.columns ?? 80, 60), 100),
             });
         }
         default:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mittwald/flow-codemods",
-  "version": "1.1.8",
+  "version": "1.1.9",
   "type": "module",
   "description": "Codemods and an upgrade CLI for consumers of Flow, mittwald's design system",
   "homepage": "https://flow.mittwald.de",
@@ -37,5 +37,5 @@
     "vitest": "^4.1.11",
     "yaml": "^2.8.1"
   },
-  "gitHead": "691eebdf4fa32d4b6b91e2b963c03135ba1875c3"
+  "gitHead": "304a819a825704d52f912100ed3042fcd42d7946"
 }