fix: code quality and safety improvements

[saurabhhhcodes]

Summary by CodeRabbit

• Bug Fixes
  • Improved handling of failed activity feed requests, providing more reliable error detection.
  • Maintained existing cross-origin access behavior while improving origin validation.

[netlify]

✅ Deploy Preview for github-spy ready!

Name	Link
🔨 Latest commit	d60b49e
🔍 Latest deploy log	https://app.netlify.com/projects/github-spy/deploys/6a6f54dd6d8d0800081b7941
😎 Deploy Preview	https://deploy-preview-756--github-spy.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 0c96c04b-dfdf-4cc6-8d88-5cdf6eb5a1c4

📥 Commits

Reviewing files that changed from the base of the PR and between 53f820b and d60b49e.

📒 Files selected for processing (2)

• backend/server.js
• src/components/ActivityFeed.tsx

---

📝 Walkthrough

Walkthrough

The PR updates the CORS allowlist membership check and adds HTTP response validation before ActivityFeed parses GitHub API data.

Changes

CORS allowlist

Layer / File(s)	Summary
CORS origin membership check backend/server.js	The allowlist check uses includes instead of an indexOf comparison.

Activity feed response handling

Layer / File(s)	Summary
GitHub response validation src/components/ActivityFeed.tsx	The component throws for unsuccessful responses before parsing the response body. Existing error handling processes the failure.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• GitMetricsLab/github_tracker#749: Implements the same changes in backend/server.js and src/components/ActivityFeed.tsx.
• GitMetricsLab/github_tracker#753: Implements the same changes in backend/server.js and src/components/ActivityFeed.tsx.

Suggested reviewers: mehul-m-prajapati

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The pull request has no description and does not provide the required issue, summary, testing, or change-type details.	Add the required sections and document the related issue, changes, testing, screenshots if applicable, and type of change.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the code quality and safety changes in the pull request.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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