refactor: mark BoardCandidatesLayout props as read-only

[ThetaLogN]

Resolves #3405

Following the SonarCloud recommendation, this PR refactors the BoardCandidatesLayout component by marking its props as read-only.

Checklist

• Required: I followed the contributing workflow
• Required: I verified that my code works as intended and resolves the issue as described
• Required: I ran make check-test locally: all warnings addressed, tests passed
• I used AI for code, documentation, tests, or communication related to this PR

[coderabbitai]

Summary by CodeRabbit

• Refactor
  • Updated component prop types to be read-only, tightening static type definitions for improved code quality and consistency across the UI layer.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

The BoardCandidatesLayout component's props parameter type was changed to use Readonly<{ children: React.ReactNode }> instead of { children: React.ReactNode }, altering only the TypeScript type-level immutability without changing runtime behavior.

Changes

Cohort / File(s)	Summary
Type annotation update for component props frontend/src/app/board/[year]/candidates/layout.tsx	Updated component parameter type from { children: React.ReactNode } to Readonly<{ children: React.ReactNode }> to mark props as read-only.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
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.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: marking BoardCandidatesLayout props as read-only, which matches the primary objective of the changeset.
Description check	✅ Passed	The description is directly related to the changeset, explaining the refactoring follows SonarCloud recommendations and links to issue #3405 which documents the requirement.
Linked Issues check	✅ Passed	The PR successfully implements the requirement from issue #3405 by updating the BoardCandidatesLayout component props to be marked as read-only using the Readonly type.
Out of Scope Changes check	✅ Passed	The changes are focused solely on marking the BoardCandidatesLayout props as read-only, with no unrelated modifications detected in the changeset.

_{✏️ 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

---

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.}

[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]

LGTM