diff --git a/src/lib/init/ui/logging-ui.ts b/src/lib/init/ui/logging-ui.ts index 75f14e883..a977d6e6f 100644 --- a/src/lib/init/ui/logging-ui.ts +++ b/src/lib/init/ui/logging-ui.ts @@ -83,10 +83,10 @@ export class LoggingUI implements WizardUI { // ── Lifecycle ───────────────────────────────────────────────────── - banner(art: string): void { - // Plain stderr write, no markdown rendering — the banner already - // contains its own ANSI styling and shouldn't be re-processed. - this.stderr.write(`\n${art}\n\n`); + banner(_art: string): void { + // No-op — LoggingUI is used in non-TTY / CI contexts where the + // large ASCII banner wastes vertical space and adds noise for + // programmatic consumers (agents, scripts, piped output). } intro(title: string): void { @@ -157,12 +157,12 @@ export class LoggingUI implements WizardUI { start: (message?: string) => { active = true; if (message) { - this.writeLine(this.stdout, `... ${this.renderInline(message)}`); + this.writeLine(this.stdout, this.renderInline(message)); } }, message: (message?: string) => { if (active && message) { - this.writeLine(this.stdout, `... ${this.renderInline(message)}`); + this.writeLine(this.stdout, this.renderInline(message)); } }, stop: (message?: string, code: SpinnerExitCode = 0) => { diff --git a/test/lib/init/ui/logging-ui.test.ts b/test/lib/init/ui/logging-ui.test.ts index f121dead9..8777763ed 100644 --- a/test/lib/init/ui/logging-ui.test.ts +++ b/test/lib/init/ui/logging-ui.test.ts @@ -135,7 +135,7 @@ describe("LoggingUI spinner", () => { spinner.message("Still working"); spinner.stop("Done", 0); const lines = stdout().split("\n").filter(Boolean); - expect(lines).toEqual(["... Working", "... Still working", "ok: Done"]); + expect(lines).toEqual(["Working", "Still working", "ok: Done"]); }); test("error stop routes to stderr with error prefix", () => { @@ -143,7 +143,7 @@ describe("LoggingUI spinner", () => { const spinner = ui.spinner(); spinner.start("Working"); spinner.stop("Boom", 1); - expect(stdout()).toBe("... Working\n"); + expect(stdout()).toBe("Working\n"); expect(stderr()).toBe("error: Boom\n"); }); @@ -170,7 +170,7 @@ describe("LoggingUI spinner", () => { spinner.stop("Done"); spinner.message("ignored"); const lines = stdout().split("\n").filter(Boolean); - expect(lines).toEqual(["... Working", "ok: Done"]); + expect(lines).toEqual(["Working", "ok: Done"]); }); }); @@ -248,11 +248,11 @@ describe("LoggingUI disposal", () => { }); describe("LoggingUI banner", () => { - test("writes art to stderr wrapped in newlines", () => { + test("is a no-op in non-TTY context", () => { const { ui, stdout, stderr } = createUI(); ui.banner(" ███ SENTRY ███"); expect(stdout()).toBe(""); - expect(stderr()).toBe("\n ███ SENTRY ███\n\n"); + expect(stderr()).toBe(""); }); });