Alphabetically Sort SOCIALS List for Improved User Experience by pisum-sativum · Pull Request #8270 · JhaSourav07/commitpulse

pisum-sativum · 2026-07-22T16:44:03Z

Description

Automatically sorts the SOCIALS list alphabetically by platform name before it is exported. When this list is mapped over to render the UI selection grid, the alphabetical sorting makes it significantly easier for users to quickly scan and find their desired platform (e.g., 'CodePen' or 'Snapchat') without having to read through the entire list manually.

Pillar

🎨 Pillar 1 — New Theme Design
📐 Pillar 2 — Geometric SVG Improvement
🕐 Pillar 3 — Timezone Logic Optimization
🛠️ Other (Bug fix, refactoring, docs)

Visual Preview

(No visual changes to the design itself, but the dropdown order of socials is now alphabetized)

Checklist before requesting a review:

I have read the CONTRIBUTING.md file.
I have tested these changes locally (localhost:3000/api/streak?user=YOUR_USERNAME).
I have run npm run format and npm run lint locally and resolved all errors (CI will fail otherwise).
My commits follow the Conventional Commits format (e.g., feat(themes): ..., fix(calculate): ...).
I have updated README.md if I added a new theme or URL parameter.
I have started the repo.
I have made sure that i have only one commit to merge in this PR.
The SVG output matches the CommitPulse "premium quality" aesthetic standard (no raw elements, smooth animations, correct fonts).
(Recommended) I joined the CommitPulse Discord community for contributor discussions, mentorship, and faster PR support.

…nt timeout on large repos

…cial[]

vercel · 2026-07-22T16:44:08Z

@pisum-sativum is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

retenta-bot · 2026-07-22T16:44:11Z

This PR addresses issue #6501 by ensuring that the SOCIALS list is sorted alphabetically, making it easier for users to find their desired platform. This aligns with previous decisions aimed at improving user experience, such as fixing theme rendering based on system preferences and ensuring social URLs are correctly formatted. Thank you for your contribution, and I look forward to your updates!

Copilot

Pull request overview

This PR aims to improve the generator UI’s socials selection experience by exporting the SOCIALS registry in alphabetical order, making the platform grid easier to scan.

Changes:

Alphabetically sorts the exported SOCIALS array by name.
Adjusts MAX_ACTION_PAGES in CI analytics fetching (note: unrelated to the PR’s stated scope).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
app/generator/data/socials.ts	Sorts the exported socials registry alphabetically to improve UI scanning.
services/github/ci-analytics.ts	Changes pagination limit for GitHub Actions analytics fetching (scope mismatch with PR goal).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

 const GITHUB_REST_URL = 'https://api.github.com';
 const MAX_REPO_PAGES = 2;
-const MAX_ACTION_PAGES = 2;
+const MAX_ACTION_PAGES = 1;
 const MAX_FETCH_TARGETS = 5;


    placeholder: 'e.g. https://huggingface.co/yourname',
  },
-];
+].sort((a, b) => a.name.localeCompare(b.name));


    placeholder: 'e.g. https://huggingface.co/yourname',
  },
-];
+].sort((a, b) => a.name.localeCompare(b.name));


pisum-sativum · 2026-07-22T17:05:02Z

@JhaSourav07 @Aamod007 I have done the necessary changes kindly check and merge with appropriate labels.

github-actions · 2026-07-22T17:15:18Z

📦 Next.js Bundle Size Report (Gzipped Sizes)

✨ No significant bundle size changes detected.

📊 Summary of Totals

Category	PR Size	Base Size	Difference
Total JS	3984.87 KB	3984.83 KB	+0.04 KB (+0.00%)
Total CSS	331.41 KB	331.41 KB	0 B

Aamod007

Thanks for adding alphabetical sorting to the SOCIALS list! This is a great user experience improvement.

However, it looks like this PR includes an unrelated change to \services/github/ci-analytics.ts.

To fix this, please revert the changes to \ci-analytics.ts\ using \git checkout\ or \git restore\ so that this PR is focused entirely on the socials sorting.

Looking forward to getting this merged once the branch is cleaned up!

Label Justification:

\level:intermediate: Assigned based on 2 files changed.
\ ype:feature: Improves the UI by organizing the socials list alphabetically.
\mentor:Aamod007: Assigned as required.

github-actions · 2026-07-23T18:57:17Z

🚨 Hey @pisum-sativum, the CI Pipeline is failing on this PR and it has been marked as status:blocked.

Please fix the issues before this can be reviewed. Here's how:

1. Run checks locally before pushing:

npm run format:check   # Check Prettier formatting
npm run lint           # Run ESLint
npm run typecheck      # TypeScript type check
npm run test           # Run unit tests (Vitest)
npm run build          # Verify production build passes

2. Auto-fix common issues:

npm run format         # Auto-fix formatting with Prettier
npm run lint -- --fix  # Auto-fix lint errors where possible

3. Check the full failure log here:
👉 View CI Run

Once you push a fix and the CI passes, the status:blocked label will be removed automatically. 💪

pisum-sativum · 2026-07-24T07:49:15Z

Thanks for adding alphabetical sorting to the SOCIALS list! This is a great user experience improvement.

However, it looks like this PR includes an unrelated change to \services/github/ci-analytics.ts.

To fix this, please revert the changes to \ci-analytics.ts\ using \git checkout\ or \git restore\ so that this PR is focused entirely on the socials sorting.

Looking forward to getting this merged once the branch is cleaned up!

Label Justification:

\level:intermediate: Assigned based on 2 files changed.

\ ype:feature: Improves the UI by organizing the socials list alphabetically.

\mentor:Aamod007: Assigned as required.

@Aamod007 I have done the necessary changes required but the CI is failing again and again I tried to fix it but I don't think so I can. Maybe after sometime will try ;(

Aamod007

The sorting is done and the SOCIALS list is now alphabetical. The change to wrap the array in parentheses for type safety is correct. However, CI is still failing — please check the logs and resolve.

One thing to note: the PR also carries changes from other PRs (InteractiveViewer.tsx, BurnoutRiskTable.tsx, types/achievements.ts) that seem to be formatting-only or from a shared base. Could you verify your branch is based on the latest main to avoid carrying unrelated changes?

Label verification:

level:intermediate — large file restructuring (600+ line reorder) with type wrapping logic
quality:clean — clean
type:feature — UX improvement via sorted list
mentor:Aamod007 — assigned

Aamod007

Re-reviewing: the only CI failure was Vercel authorization (not code-related). All code checks pass. The Vercel deploy auth is a repo-level config issue. Approving.

pisum-sativum · 2026-07-27T14:42:22Z

@Aamod007 @JhaSourav07 Kindly check and merge if possible with correct labels.

pisum-sativum · 2026-07-28T07:39:45Z

@Aamod007 @JhaSourav07 Kindly merge this pr so that i can take up new issues since the bots is not allowing me to take up any more new issues .

pisum-sativum · 2026-07-31T07:50:24Z

@Aamod007 @JhaSourav07 If no change is required then kindly check and merge.

pisum-sativum · 2026-08-02T07:44:46Z

@Aamod007 @JhaSourav07 Pls check and merge

github-actions · 2026-08-06T03:13:43Z

⚠️ Hey @pisum-sativum, this pull request has been inactive for 3 days. It will be automatically closed in 2 days if no further activity occurs.

If you are still working on this, please push your latest changes or leave a comment to keep it active.

Aamod007

Excellent work addressing the GitHub GraphQL rate limit issues and token exposure vulnerabilities! Fixing the abort signal deduplication logic ensures we don't accidentally exhaust our rate limits under concurrent load, and gracefully sanitizing the fetch errors prevents token leaks. Approved!

pisum-sativum · 2026-08-06T06:52:35Z

@Aamod007 @JhaSourav07 Pls merge this pr asap

pisum-sativum · 2026-08-06T14:42:02Z

@JhaSourav07 Pls merge the prs asap because i have to resolve the merge conflict again and again.

pisum-sativum · 2026-08-06T20:28:45Z

@Aamod007 @JhaSourav07 Pls merge this pr asap

pisum-sativum · 2026-08-07T13:04:41Z

@JhaSourav07 could you please merge all the PRs I've worked on? I've been requesting this for several days now but they are still pending. I would really appreciate it if you could review and merge them as soon as possible. Thank you!

pisum-sativum · 2026-08-08T22:18:08Z

@JhaSourav07 Can u pls merge this pr

pisum-sativum · 2026-08-09T18:34:57Z

@JhaSourav07 @Aamod007 pls merge the prs i have worked on pls

pisum-sativum added 4 commits July 22, 2026 02:13

fix(ci-analytics): parallelize API requests with Promise.all to preve…

f299da2

…nt timeout on large repos

feat: Automatically sort SOCIALS list alphabetically

5a9ecf1

Merge remote-tracking branch 'origin/main' into fix-socials-sort

e6f1ded

fix: resolve TS2322 type error in SOCIALS sort by casting array as So…

6978cd3

…cial[]

Copilot AI review requested due to automatic review settings July 22, 2026 16:44

retenta-bot Bot changed the title ~~Fix socials sort~~ Alphabetically Sort SOCIALS List for Improved User Experience Jul 22, 2026

Copilot started reviewing on behalf of pisum-sativum July 22, 2026 16:44 View session

Copilot AI reviewed Jul 22, 2026

View reviewed changes

github-actions Bot added the status:blocked This PR is blocked due to a failing CI check. label Jul 22, 2026

github-actions Bot removed the status:blocked This PR is blocked due to a failing CI check. label Jul 22, 2026

Aamod007 added level:intermediate Moderate complexity tasks type:feature New features, additions, or enhancements mentor:Aamod007 labels Jul 23, 2026

Aamod007 requested changes Jul 23, 2026

View reviewed changes

fix: revert unrelated changes to ci-analytics.ts

09201bb

github-actions Bot added the status:blocked This PR is blocked due to a failing CI check. label Jul 23, 2026

pisum-sativum added 2 commits July 24, 2026 00:29

Merge remote-tracking branch 'origin/main' into fix-socials-sort

d93f777

style: format files with Prettier

c081574

pisum-sativum force-pushed the fix-socials-sort branch from be3ac67 to c081574 Compare July 23, 2026 19:08

pisum-sativum added 5 commits July 24, 2026 00:45

fix(burnout): add missing Next.js Image import in BurnoutRiskTable

4e06efa

Merge remote-tracking branch 'origin/main' into fix-socials-sort

c3bede7

style: format BurnoutRiskTable with updated Prettier config

2d701a8

fix(burnout): remove duplicate Image import causing build failure

7d7ea21

style: reformat files to match origin/main prettier config

272b0df

Aamod007 requested changes Jul 27, 2026

View reviewed changes

Aamod007 approved these changes Jul 27, 2026

View reviewed changes

Aamod007 approved these changes Aug 6, 2026

View reviewed changes

Conversation

pisum-sativum commented Jul 22, 2026

Description

Pillar

Visual Preview

Checklist before requesting a review:

Uh oh!

vercel Bot commented Jul 22, 2026

Uh oh!

retenta-bot Bot commented Jul 22, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

pisum-sativum commented Jul 22, 2026

Uh oh!

github-actions Bot commented Jul 22, 2026

📦 Next.js Bundle Size Report (Gzipped Sizes)

📊 Summary of Totals

Uh oh!

Aamod007 left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jul 23, 2026

Uh oh!

pisum-sativum commented Jul 24, 2026

Uh oh!

Aamod007 left a comment

Choose a reason for hiding this comment

Uh oh!

Aamod007 left a comment

Choose a reason for hiding this comment

Uh oh!

pisum-sativum commented Jul 27, 2026

Uh oh!

pisum-sativum commented Jul 28, 2026

Uh oh!

pisum-sativum commented Jul 31, 2026

Uh oh!

pisum-sativum commented Aug 2, 2026

Uh oh!

github-actions Bot commented Aug 6, 2026

Uh oh!

Aamod007 left a comment

Choose a reason for hiding this comment

Uh oh!

pisum-sativum commented Aug 6, 2026

Uh oh!

pisum-sativum commented Aug 6, 2026

Uh oh!

pisum-sativum commented Aug 6, 2026

Uh oh!

pisum-sativum commented Aug 7, 2026

Uh oh!

pisum-sativum commented Aug 8, 2026

Uh oh!

pisum-sativum commented Aug 9, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants