fix: remove commits-only guard in GoalTracker auto-sync (closes #2410)

[Rudrasamadhiya]

Summary

The GoalTracker component's auto-sync logic silently skipped all goal types except unit === "commits". This meant streak, PR, review, and other goal types never triggered background sync, leaving them perpetually stale.

This fix removes the guard clause if (g.unit !== "commits") return false; and replaces it with a comment clarifying that auto-sync applies to all goal types.

Closes #2410

Type of Change

• Bug fix

[github-actions]

GSSoC Label Checklist 🏷️

@Umbrella-io — please apply the appropriate labels before merging:

Difficulty (pick one):

• level:beginner — 20 pts
• level:intermediate — 35 pts
• level:advanced — 55 pts
• level:critical — 80 pts

Quality (optional):

• quality:clean — ×1.2 multiplier
• quality:exceptional — ×1.5 multiplier

Validation (required to score):

• gssoc:approved — counts for points
• gssoc:invalid / gssoc:spam / gssoc:ai-slop — does not score

Type labels (type:*) are auto-detected from files and title. Review and adjust if needed.
Points formula: (difficulty × quality_multiplier) + type_bonus

[github-actions]

🎉 Merged! Thanks for contributing to DevTrack.

If the project has been useful to you, a ⭐ star on the repo is the easiest way to support it — it helps DevTrack get discovered by more developers.

Keep an eye on open issues for your next contribution!