Skip to content

Commit beebe69

Browse files
author
John Doe
committed
refactor: fix utils logic
1 parent 3b3d25d commit beebe69

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

packages/utils/src/lib/execute-process.ts

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import {
88
import type { Readable, Writable } from 'node:stream';
99
import { formatCommandLog } from './command.js';
1010
import { isVerbose } from './env.js';
11-
import { logger } from './logger.js';
11+
import { type Logger, logger } from './logger.js';
1212
import { calcDuration } from './reports/utils.js';
1313

1414
/**
@@ -106,6 +106,7 @@ export type ProcessConfig = Omit<
106106
args?: string[];
107107
observer?: ProcessObserver;
108108
ignoreExitCode?: boolean;
109+
verbose?: boolean;
109110
};
110111

111112
/**
@@ -157,7 +158,7 @@ export type ProcessObserver = {
157158
*/
158159
export function executeProcess(
159160
cfg: ProcessConfig,
160-
logger: { log: (str: string) => void } = ui().logger,
161+
loggerInstance: Logger = logger,
161162
): Promise<ProcessResult> {
162163
const {
163164
command,
@@ -170,7 +171,7 @@ export function executeProcess(
170171
const { onStdout, onStderr, onError, onComplete } = observer ?? {};
171172

172173
if (isVerbose() || verbose === true) {
173-
logger.log(
174+
loggerInstance.info(
174175
formatCommandLog({
175176
command,
176177
args,
@@ -179,7 +180,7 @@ export function executeProcess(
179180
);
180181
}
181182
const bin = [command, ...(args ?? [])].join(' ');
182-
return logger.command(
183+
return loggerInstance.command(
183184
bin,
184185
() =>
185186
new Promise((resolve, reject) => {
@@ -219,12 +220,12 @@ export function executeProcess(
219220
spawnedProcess.on('close', (code, signal) => {
220221
const result: ProcessResult = { bin, code, signal, stdout, stderr };
221222
if (code === 0 || ignoreExitCode) {
222-
logger.debug(output);
223+
loggerInstance.debug(output);
223224
onComplete?.();
224225
resolve(result);
225226
} else {
226227
// ensure stdout and stderr are logged to help debug failure
227-
logger.debug(output, { force: true });
228+
loggerInstance.debug(output, { force: true });
228229
const error = new ProcessError(result);
229230
onError?.(error);
230231
reject(error);

0 commit comments

Comments
 (0)