Skip to content

Refactor duplicated display helpers#1050

Draft
ryokun6 wants to merge 3 commits intomainfrom
cursor/cleanup-duplicated-helpers-3c62
Draft

Refactor duplicated display helpers#1050
ryokun6 wants to merge 3 commits intomainfrom
cursor/cleanup-duplicated-helpers-3c62

Conversation

@ryokun6
Copy link
Copy Markdown
Owner

@ryokun6 ryokun6 commented Apr 25, 2026

Summary

  • Extract shared username initials formatting into the contacts utility.
  • Extract shared dashboard city helpers, including popular city list, label formatting, and Nominatim search mapping.
  • Extract shared Kugou image URL formatting and Redis JSON parsing helpers.
  • Remove the unused getTheme export.
  • Add focused Bun tests for extracted helper behavior.

Testing

  • export PATH="/root/.bun/bin:$PATH" && bun test tests/test-contacts-vcard.test.ts tests/test-dashboard-city-search.test.ts tests/test-kugou-image-url.test.ts tests/test-parse-json-utils.test.ts
  • export PATH="/root/.bun/bin:$PATH" && bun run build
Open in Web Open in Cursor 

Co-authored-by: Ryo Lu <me@ryo.lu>
@ryos-deploy
Copy link
Copy Markdown

ryos-deploy Bot commented Apr 25, 2026

The preview deployment for ryos-dev is ready. 🟢

Open Preview | Open Build Logs | Open Application Logs

Last updated at: 2026-04-25 10:56:10 CET

cursoragent and others added 2 commits April 25, 2026 10:32
Co-authored-by: Ryo Lu <me@ryo.lu>
Co-authored-by: Ryo Lu <me@ryo.lu>
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