Ensure install progress msg stays in one line (#12165)

microsoft · Nov 24, 2022 · 9b75af3 · 9b75af3
1 parent cda5378
commit 9b75af3
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/kernels/installer/moduleInstaller.node.ts b/src/kernels/installer/moduleInstaller.node.ts
@@ -113,13 +113,16 @@ export abstract class ModuleInstaller implements IModuleInstaller {
                 });
             }
             let lastStdErr: string | undefined;
+            const ticker = ['', '.', '..', '...'];
+            let counter = 0;
             if (observable) {
                 observable.out.subscribe({
                     next: (output) => {
+                        const suffix = ticker[counter % 4];
+                        const trimmedOutput = output.out.trim();
+                        counter += 1;
                         const message =
-                            output.out.length > 100
-                                ? `${output.out.substring(0, 50)}...${output.out.substring(output.out.length - 50)}`
-                                : output.out;
+                            trimmedOutput.length > 30 ? `${trimmedOutput.substring(0, 30)}${suffix}` : trimmedOutput;
                         progress.report({ message });
                         traceInfo(output.out);
                         if (output.source === 'stderr') {