Shell-startup ASCII art off by default; gallery pruned to four - #93
Draft
yulonglin wants to merge 3 commits into
Draft
Shell-startup ASCII art off by default; gallery pruned to four#93yulonglin wants to merge 3 commits into
yulonglin wants to merge 3 commits into
Conversation
config/start.txt was tracked and printed on every interactive zsh and bash start. It is now untracked and per machine: nothing prints unless `deploy.sh --ascii=NAME` writes it, `--ascii=none` removes it, and a plain deploy leaves the current choice alone. The gallery drops the twelve 120–450-column pieces that no terminal fits and keeps cat, dog, paw and stars (all under 100 columns). Claude-Session: https://claude.ai/code/session_01PCVS6wJHQndykfPN2ESvh7
Restores the exact start.txt from that period (balloon_girl2 minus two blank padding lines top and bottom), 41 lines by 100 display columns. The earlier width figures counted UTF-8 bytes, not columns; the piece fits an ordinary terminal. Claude-Session: https://claude.ai/code/session_01PCVS6wJHQndykfPN2ESvh7
…, stars, waves3 Restores the original balloon_girl, cat2, corgi, paw, paw3 and waves3; drops paw2 and the balloon_girl2 variant. Nine pieces, all 80 columns or narrower. stars.txt keeps the former default (stars2) content. Claude-Session: https://claude.ai/code/session_01PCVS6wJHQndykfPN2ESvh7
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
config/start.txtis no longer tracked or printed by default. It is gitignored and per machine.deploy.sh --ascii=NAMEwrites it fromconfig/ascii_arts/(a.txtsuffix is tolerated);--ascii=noneremoves it; a plain deploy leaves the current choice alone. An unknown name warns and lists what exists.config/zshrc.shnow guards on the file existing, so a shell with no art no longer runscaton a missing path. The bashrc block already guarded.Why
Startup art on every shell was noise. Keeping the flag with a curated gallery preserves the option without the default.
Verified
bash -n/zsh -non the edited scripts.--asciiblock exercised against a scratchDOT_DIR: empty leaves absent,catwrites cat, empty again keeps cat,dog.txtwrites dog,boguswarns and lists the gallery,noneremoves.tests/test_profile_defaults.sh: 18 passed, 0 failed.Effect on merge
Merging deletes the tracked
config/start.txtin the main checkout, so the art stops on the next new shell with no deploy needed. To bring one back on a machine:./deploy.sh --ascii=balloon_girl.https://claude.ai/code/session_01PCVS6wJHQndykfPN2ESvh7