@pnpm/cli.default-reporter 1100.3.6 → 1100.3.8

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # @pnpm/default-reporter
 
+## 1100.3.8
+
+### Patch Changes
+
+- Keep the interactive `minimumReleaseAge` approval prompt visible during `pnpm install`. The progress reporter now pauses its redraws while a prompt is waiting for input instead of overwriting it, so the install no longer hangs on a question the user cannot see [#13019](https://github.com/pnpm/pnpm/issues/13019).
+
+- Updated dependencies:
+  - @pnpm/config.reader@1101.12.1
+  - @pnpm/core-loggers@1100.2.3
+
+## 1100.3.7
+
+### Patch Changes
+
+- Updated dependencies:
+  - @pnpm/cli.meta@1100.0.9
+  - @pnpm/config.reader@1101.12.0
+  - @pnpm/core-loggers@1100.2.2
+  - @pnpm/deps.inspection.peers-issues-renderer@1100.0.7
+  - @pnpm/types@1101.4.0
+
 ## 1100.3.6
 
 ### Patch Changes

package/lib/index.js

@@ -38,10 +38,26 @@ export function initDefaultReporter(opts) {
     }
     const stream = opts.useStderr ? proc.stderr : proc.stdout;
     const write = stream.write.bind(stream);
-    const diff = createDiffer({
+    const newDiffer = () => createDiffer({
         height: stream.rows,
         width: stream.columns ?? outputMaxWidth,
     });
+    let diff = newDiffer();
+    // Hold redraws while an interactive prompt owns the terminal (see PromptMessage).
+    let promptActive = false;
+    const onLog = (log) => {
+        if (log.name !== 'pnpm:prompt')
+            return;
+        if (log.action === 'start') {
+            promptActive = true;
+        }
+        else {
+            promptActive = false;
+            // Drop the differ's now-stale frame: the terminal below it changed while paused.
+            diff = newDiffer();
+        }
+    };
+    opts.streamParser.on('data', onLog);
     const subscription = output$
         .subscribe({
         complete() { }, // eslint-disable-line:no-empty
@@ -51,6 +67,8 @@ export function initDefaultReporter(opts) {
         next: logUpdate,
     });
     function logUpdate(view) {
+        if (promptActive)
+            return;
         // A new line should always be appended in case a prompt needs to appear.
         // Without a new line the prompt will be joined with the previous output.
         // An example of such prompt may be seen by running: pnpm update --interactive
@@ -68,6 +86,7 @@ export function initDefaultReporter(opts) {
     }
     return () => {
         subscription.unsubscribe();
+        opts.streamParser.removeListener('data', onLog);
     };
 }
 export function toOutput$(opts) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pnpm/cli.default-reporter",
-  "version": "1100.3.6",
+  "version": "1100.3.8",
   "description": "The default reporter of pnpm",
   "keywords": [
     "pnpm",
@@ -32,14 +32,14 @@
     "pnpm-render": "bin/pnpm-render.mjs"
   },
   "dependencies": {
-    "@pnpm/cli.meta": "1100.0.8",
-    "@pnpm/config.reader": "1101.11.2",
-    "@pnpm/core-loggers": "1100.2.1",
-    "@pnpm/deps.inspection.peers-issues-renderer": "1100.0.6",
+    "@pnpm/cli.meta": "1100.0.9",
+    "@pnpm/config.reader": "1101.12.1",
+    "@pnpm/core-loggers": "1100.2.3",
+    "@pnpm/deps.inspection.peers-issues-renderer": "1100.0.7",
     "@pnpm/error": "1100.0.1",
     "@pnpm/installing.dedupe.issues-renderer": "1100.0.1",
     "@pnpm/installing.dedupe.types": "1100.0.1",
-    "@pnpm/types": "1101.3.2",
+    "@pnpm/types": "1101.4.0",
     "@pnpm/util.lex-comparator": "^4.0.1",
     "ansi-diff": "^1.2.0",
     "boxen": "npm:@zkochan/boxen@5.1.2",
@@ -59,7 +59,7 @@
   },
   "devDependencies": {
     "@jest/globals": "30.4.1",
-    "@pnpm/cli.default-reporter": "1100.3.6",
+    "@pnpm/cli.default-reporter": "1100.3.8",
     "@pnpm/logger": "1100.0.0",
     "@types/normalize-path": "^3.0.2",
     "@types/ramda": "0.31.1",

package/lib/cli.d.ts

@@ -1 +0,0 @@
-export declare function runCli(argv: readonly string[]): Promise<void>;

package/lib/cli.js

@@ -1,50 +0,0 @@
-import { EventEmitter } from 'node:events';
-import readline from 'node:readline';
-import { initDefaultReporter } from './index.js';
-export async function runCli(argv) {
-    const cmd = argv[0] ?? 'install';
-    const emitter = new EventEmitter();
-    const streamParser = {
-        on: (event, reporter) => {
-            emitter.on(event, reporter);
-        },
-        removeListener: (event, reporter) => {
-            emitter.removeListener(event, reporter);
-        },
-    };
-    const close = initDefaultReporter({
-        streamParser,
-        context: { argv: [cmd] },
-        reportingOptions: {
-            throttleProgress: 200,
-        },
-    });
-    // initDefaultReporter registers its 'data' listener via setTimeout(0); wait
-    // a tick so events emitted from the readline loop below aren't dropped.
-    await new Promise((resolve) => {
-        setTimeout(resolve, 0);
-    });
-    try {
-        const rl = readline.createInterface({ input: process.stdin, crlfDelay: Infinity });
-        for await (const line of rl) {
-            if (!line)
-                continue;
-            let parsed;
-            try {
-                parsed = JSON.parse(line);
-            }
-            catch {
-                continue;
-            }
-            // Guard against valid JSON that isn't a log object (e.g. `null`,
-            // numbers, strings) — the reporter dispatches on `log.name`.
-            if (parsed == null || typeof parsed !== 'object')
-                continue;
-            emitter.emit('data', parsed);
-        }
-    }
-    finally {
-        close();
-    }
-}
-//# sourceMappingURL=cli.js.map

package/lib/constants.d.ts

@@ -1 +0,0 @@
-export declare const EOL = "\n";

package/lib/constants.js

@@ -1,4 +0,0 @@
-// In terminals, always Unix line endings are used
-// even on Windows
-export const EOL = '\n';
-//# sourceMappingURL=constants.js.map

package/lib/index.d.ts

@@ -1,54 +0,0 @@
-import type { Config, ConfigContext } from '@pnpm/config.reader';
-import type * as logs from '@pnpm/core-loggers';
-import type { LogLevel, StreamParser } from '@pnpm/logger';
-import * as Rx from 'rxjs';
-import type { FilterPkgsDiff } from './reporterForClient/reportSummary.js';
-import { formatWarn } from './reporterForClient/utils/formatWarn.js';
-export { formatWarn };
-export declare function initDefaultReporter(opts: {
-    useStderr?: boolean;
-    streamParser: StreamParser<logs.Log>;
-    reportingOptions?: {
-        appendOnly?: boolean;
-        logLevel?: LogLevel;
-        streamLifecycleOutput?: boolean;
-        aggregateOutput?: boolean;
-        throttleProgress?: number;
-        outputMaxWidth?: number;
-        hideAddedPkgsProgress?: boolean;
-        hideProgressPrefix?: boolean;
-        hideLifecycleOutput?: boolean;
-        hideLifecyclePrefix?: boolean;
-        approveBuildsInstructionText?: string;
-    };
-    context: {
-        argv: string[];
-        config?: Config & ConfigContext;
-        env?: NodeJS.ProcessEnv;
-        process?: NodeJS.Process;
-    };
-    filterPkgsDiff?: FilterPkgsDiff;
-}): () => void;
-export declare function toOutput$(opts: {
-    streamParser: StreamParser<logs.Log>;
-    reportingOptions?: {
-        appendOnly?: boolean;
-        logLevel?: LogLevel;
-        outputMaxWidth?: number;
-        streamLifecycleOutput?: boolean;
-        aggregateOutput?: boolean;
-        throttleProgress?: number;
-        hideAddedPkgsProgress?: boolean;
-        hideProgressPrefix?: boolean;
-        hideLifecycleOutput?: boolean;
-        hideLifecyclePrefix?: boolean;
-        approveBuildsInstructionText?: string;
-    };
-    context: {
-        argv: string[];
-        config?: Config & ConfigContext;
-        env?: NodeJS.ProcessEnv;
-        process?: NodeJS.Process;
-    };
-    filterPkgsDiff?: FilterPkgsDiff;
-}): Rx.Observable<string>;

package/lib/mergeOutputs.d.ts

@@ -1,4 +0,0 @@
-import * as Rx from 'rxjs';
-export declare function mergeOutputs(outputs: Array<Rx.Observable<Rx.Observable<{
-    msg: string;
-}>>>): Rx.Observable<string>;

package/lib/mergeOutputs.js

@@ -1,69 +0,0 @@
-import * as Rx from 'rxjs';
-import { filter, map, mergeAll, scan } from 'rxjs/operators';
-import { EOL } from './constants.js';
-export function mergeOutputs(outputs) {
-    let blockNo = 0;
-    let fixedBlockNo = 0;
-    let started = false;
-    let previousOutput = null;
-    return Rx.merge(...outputs).pipe(map((log) => {
-        let currentBlockNo = -1;
-        let currentFixedBlockNo = -1;
-        return log.pipe(map((msg) => {
-            if (msg.fixed) {
-                if (currentFixedBlockNo === -1) {
-                    currentFixedBlockNo = fixedBlockNo++;
-                }
-                return {
-                    blockNo: currentFixedBlockNo,
-                    fixed: true,
-                    msg: msg.msg,
-                };
-            }
-            if (currentBlockNo === -1) {
-                currentBlockNo = blockNo++;
-            }
-            return {
-                blockNo: currentBlockNo,
-                fixed: false,
-                msg: typeof msg === 'string' ? msg : msg.msg, // eslint-disable-line
-                prevFixedBlockNo: currentFixedBlockNo,
-            };
-        }));
-    }), mergeAll(), scan((acc, log) => {
-        if (log.fixed) {
-            acc.fixedBlocks[log.blockNo] = log.msg;
-        }
-        else {
-            delete acc.fixedBlocks[log.prevFixedBlockNo];
-            acc.blocks[log.blockNo] = log.msg;
-        }
-        return acc;
-    }, { fixedBlocks: [], blocks: [] }), map((sections) => {
-        const fixedBlocks = sections.fixedBlocks.filter(Boolean);
-        const nonFixedPart = sections.blocks.filter(Boolean).join(EOL);
-        if (fixedBlocks.length === 0) {
-            return nonFixedPart;
-        }
-        const fixedPart = fixedBlocks.join(EOL);
-        if (!nonFixedPart) {
-            return fixedPart;
-        }
-        return `${nonFixedPart}${EOL}${fixedPart}`;
-    }), filter((msg) => {
-        if (started) {
-            return true;
-        }
-        if (msg === '')
-            return false;
-        started = true;
-        return true;
-    }), filter((msg) => {
-        if (msg !== previousOutput) {
-            previousOutput = msg;
-            return true;
-        }
-        return false;
-    }));
-}
-//# sourceMappingURL=mergeOutputs.js.map

package/lib/reportError.d.ts

@@ -1,3 +0,0 @@
-import type { Config } from '@pnpm/config.reader';
-import type { Log } from '@pnpm/core-loggers';
-export declare function reportError(logObj: Log, config?: Config): string | null;