fix: remove unused className prop from AnchorTitle component

[lavanyayadawad30-lab]

Proposed change

Resolves #3805

This PR fixes a SonarCloud code smell in frontend/src/components/AnchorTitle.tsx where the className prop was defined in AnchorTitleProps but never used in the component.

The unused prop has been removed to improve code clarity and maintainability and to resolve Sonar rule typescript:S6767.
No functional behavior is changed.

Checklist:-
✅I followed the contributing workflow
✅ I verified that my code works as intended and resolves the issue as described
✅I ran checks locally (or verified no additional tests were required for this change)
❌ I used AI for code, documentation, tests, or communication related to this PR

[coderabbitai]

Summary by CodeRabbit

Release Notes

• Refactor

  • Renamed ContributorsList component prop from label to title throughout the application for improved semantic naming and API consistency
  • Simplified AnchorTitle component interface by removing optional className prop support, reducing surface area while preserving visual behavior

• Tests

  • Updated component test mocks and test cases to align with prop naming updates and component interface changes

Walkthrough

Removed the unused className prop from AnchorTitle's public props and updated all call sites and tests to stop passing className; also renamed ContributorsList prop label → title across code and tests.

Changes

Cohort / File(s)	Summary
AnchorTitle component frontend/src/components/AnchorTitle.tsx	Removed className?: string from AnchorTitleProps and stopped accepting/propagating className in the component.
ContributorsList & usages frontend/src/components/ContributorsList.tsx, frontend/src/components/CardDetailsPage.tsx, frontend/src/app/page.tsx, frontend/src/app/about/page.tsx	Renamed public prop label?: string → title?: string on ContributorsList; internal mapping preserves default label text. Call sites updated to use title.
AnchorTitle call-site updates frontend/src/components/Milestones.tsx, frontend/src/components/RecentIssues.tsx, frontend/src/components/RecentPullRequests.tsx, frontend/src/components/RecentReleases.tsx, frontend/src/components/CardDetailsPage.tsx	Removed className="..." arguments when instantiating <AnchorTitle ... />, leaving only title prop.
Tests / Mocks frontend/__tests__/unit/components/Milestones.test.tsx, frontend/__tests__/unit/components/ContributorsList.test.tsx, frontend/__tests__/unit/components/CardDetailsPage.test.tsx	Updated mocks and expectations to match AnchorTitle prop removal and ContributorsList prop rename (label → title), and adjusted rendered text assertions accordingly.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• adding feature of anchor links #960: Modifies AnchorTitle prop usage; directly related to adding/removing className.
• Improved AnchorTitle component used slug to get href from title #1434: Alters AnchorTitle's public interface (prop changes and slug behavior), related to the component API changes here.
• Feature/added milestone component tests #2026: Touches Milestones tests and AnchorTitle mock usage, overlapping with updated test mocks in this PR.

Suggested reviewers

• kasya

🚥 Pre-merge checks | ✅ 1 | ❌ 4

❌ Failed checks (2 warnings, 2 inconclusive)

Check name	Status	Explanation	Resolution
Out of Scope Changes check	⚠️ Warning	Multiple out-of-scope changes present: ContributorsList prop renamed from label to title across 6 files, plus className removals from multiple AnchorTitle usages beyond the component definition.	Limit changes to the AnchorTitle component fix only. Handle the label→title rename and other AnchorTitle className removals in separate PRs aligned with their respective issues.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	The description relates to the changeset and references issue #3805, but contradicts the actual implementation showing className reintroduction.	Clarify whether the PR removes or reintroduces the className prop to resolve confusion between description and implementation.
Linked Issues check	❓ Inconclusive	The PR partially addresses issue #3805: className was removed from AnchorTitleProps but was then reintroduced in prop signatures, creating inconsistency with the original issue's expected resolution.	Verify if className should be removed entirely per issue #3805 or reintroduced with proper usage to fully resolve SonarCloud rule typescript:S6767.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: removing the unused className prop from AnchorTitle, though implementation differs from PR description.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

frontend/src/components/ContributorsList.tsx (1)

13-13: Public API rename label → title is scope creep beyond the PR objective.

The PR objective (issue #3805) is to address the unused className prop on AnchorTitle. Renaming ContributorsListProps.label to title is an unrelated breaking API change that increases the surface area and risk of this PR. Consider splitting this rename into a separate PR.

Also, the destructuring alias title: label = 'Contributors' (accepting title externally but using label internally) is a bit confusing. If you keep the rename, consider using title consistently throughout the component body as well.

Also applies to: 19-21

frontend/__tests__/unit/components/ContributorsList.test.tsx (1)

303-308: Test description says "label" but the prop is now title.

The test description 'uses custom label when provided' still references "label" after the prop was renamed to title. Consider updating the test description for consistency.

Suggested description updates

-    it('uses custom label when provided', () => {
+    it('uses custom title when provided', () => {

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cubic-dev-ai]

No issues found across 1 file

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

[arkid15r]

I don't think this is enough.

This is there for a reason.

[lavanyayadawad30-lab]

Hi @arkid15r , Thank you for the feedback!

I've now updated the fix to properly use the className prop instead of removing it:

✅ Added className?: string back
✅ Accept className parameter in the component function
✅ Apply className to the root div element using template literals

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[arkid15r]

@lavanyayadawad30-lab thanks for your contribution!

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.38%. Comparing base (31047c7) to head (ae91917).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3822   +/-   ##
=======================================
  Coverage   95.38%   95.38%           
=======================================
  Files         463      463           
  Lines       14543    14543           
  Branches     2061     2061           
=======================================
  Hits        13872    13872           
  Misses        328      328           
  Partials      343      343           

Flag	Coverage Δ
backend	95.67% <ø> (ø)
frontend	94.61% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
frontend/src/app/about/page.tsx	96.70% <ø> (ø)
frontend/src/app/page.tsx	91.35% <ø> (ø)
frontend/src/components/AnchorTitle.tsx	100.00% <ø> (ø)
frontend/src/components/CardDetailsPage.tsx	94.92% <ø> (ø)
frontend/src/components/ContributorsList.tsx	100.00% <100.00%> (ø)
frontend/src/components/Milestones.tsx	93.75% <ø> (ø)
frontend/src/components/RecentIssues.tsx	100.00% <ø> (ø)
frontend/src/components/RecentPullRequests.tsx	100.00% <ø> (ø)
frontend/src/components/RecentReleases.tsx	100.00% <ø> (ø)

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 31047c7...ae91917. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.