Skip to content

Commit feae23d

Browse files
hugocasaclaude
andcommitted
style(cli): amber (not red) for the migration nudge
It's a "you could consolidate" hint, not an error, so color it amber to match the dashboard banner. ANSI only when stderr is a TTY, so piped or redirected output stays plain. Stays on stderr (standard for diagnostics, keeps stdout parseable). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 76f204d commit feae23d

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

bin/src/migrate.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,8 @@ export function warnIfOtherInstances(port: number, listLive: () => InstanceEntry
145145
}
146146
if (others.length === 0) return;
147147
const ports = others.map((entry) => entry.port).join(", ");
148-
console.error(
149-
`Warning: ${others.length} other webmux server(s) detected on port(s) ${ports}. Run \`webmux project migrate\` to consolidate them into this dashboard.`,
150-
);
148+
const message = `Warning: ${others.length} other webmux server(s) detected on port(s) ${ports}. Run \`webmux project migrate\` to consolidate them into this dashboard.`;
149+
// Amber, not red — this is a nudge, not an error. Colorize only on a TTY so
150+
// piped/redirected stderr stays plain (no escape codes in logs).
151+
console.error(process.stderr.isTTY ? `\x1b[38;5;214m${message}\x1b[0m` : message);
151152
}

0 commit comments

Comments
 (0)