github/thedotmack-claude-mem
1
0
Fork 0
mirror of https://github.com/thedotmack/claude-mem.git synced 2026-07-03 12:32:32 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(banner): play unconditionally; only honor CLAUDE_MEM_NO_BANNER

Browse Source 
The 128-col / TTY / CI / NO_COLOR gates silently swallowed the banner in
narrower terminals, CI logs, and any non-TTY pipe — including Docker runs
where -it should preserve the experience but column width was the wrong
gate. Remove the implicit gates; keep the explicit opt-out only.

If a frame wraps in a narrow terminal, that's better than the banner
not playing at all.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Alex Newman
2026-04-30 22:42:22 -07:00
parent 44ea917473
commit eb6fc15788
1 changed files with 1 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/npx-cli/banner.ts
View File

@@ -101,12 +101,8 @@ function writeTaglineRow(text: string): string {
}
export function isBannerEnabled(): boolean {
if (!process.stdout.isTTY) return false;
if (process.env.CI) return false;
if (process.env.CLAUDE_MEM_NO_BANNER) return false;
if (process.env.NO_COLOR) return false;
const cols = process.stdout.columns ?? 0;
return cols >= BANNER.width;
return true;
}
const sleep = (ms: number) => new Promise<void>((r) => setTimeout(r, ms));