Skip to content

fix: make cat face symmetrical - #120

Open
missayi wants to merge 1 commit into
ramarivera:mainfrom
missayi:fix/cat-face-symmetry
Open

fix: make cat face symmetrical#120
missayi wants to merge 1 commit into
ramarivera:mainfrom
missayi:fix/cat-face-symmetry

Conversation

@missayi

@missayi missayi commented May 24, 2026

Copy link
Copy Markdown

The cat art had uneven spacing around the eyes (( {E} {E})) making the face look off-centre. Updated all three animation frames to ( {E} {E} ) across server/art.ts, README.md, and CONTRIBUTING.md. Also fixed alignment of other species in readme grid (no changes to buddy art for these needed, though).


One thing I noticed while working on this: the local dev install (claude-buddy/ subdirectory created by bun run install-buddy) isn't in .gitignore. It's untracked so it won't cause problems, but it might be worth adding to avoid confusion for future contributors. Happy to add that in a follow-up if useful!

@missayi
missayi force-pushed the fix/cat-face-symmetry branch from 25a8361 to f4aed4a Compare May 24, 2026 07:07
@missayi

missayi commented May 24, 2026

Copy link
Copy Markdown
Author

I originally made it narrower but realised this is to small for many of the different eye styles and updated to a little wider instead so it vis still 12 char across total but now has a space between left eye and side of face, too. Looks like:
Screenshot 2026-05-24 at 5 24 09 pm

@ramarivera

Copy link
Copy Markdown
Owner

Hey @missayi — thanks for this fix! The only thing blocking it from getting CI + review is the DCO check: the commit(s) are missing a Signed-off-by line.

Quick fix from your branch:

git commit --amend -s --no-edit   # single commit
# or, for multiple commits:
git rebase --signoff origin/main
git push --force-with-lease

Once DCO goes green we can run CI and get this reviewed — it's a small, wanted change.

🤖 Created with the help of AI (Claude Fable 5).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants