chore: Cleanup

[peucastro]

removed dead code/unused dependencies and added formatting check

Summary by CodeRabbit

• Updates

  • Updated browser and mobile home-screen icons for improved branding and display consistency.
  • Streamlined the application by removing unused screens, icons, images, and legacy exchange-related interfaces.
  • Removed outdated staging and data-reset experiences that are no longer available.

• Maintenance

  • Standardized code formatting across the application.
  • Improved automated checks to validate formatting during development workflows.
  • Simplified project configuration and removed unused tooling and dependencies.

[netlify]

✅ Deploy Preview for tts-fe-preview ready!

Name	Link
🔨 Latest commit	dba388c
🔍 Latest deploy log	https://app.netlify.com/projects/tts-fe-preview/deploys/6a9ebbcaa9b96b0008b54faf
😎 Deploy Preview	https://deploy-preview-771--tts-fe-preview.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: Organization UI

Review profile: CHILL

Plan: Team

Run ID: e8a32ed1-e758-4a4a-aa26-d480cbbd8921

📥 Commits

Reviewing files that changed from the base of the PR and between 5a06d11 and dba388c.

⛔ Files ignored due to path filters (13)

• package-lock.json is excluded by !**/package-lock.json
• src/images/homepage1.png is excluded by !**/*.png
• src/images/homepage2.jpg is excluded by !**/*.jpg
• src/images/icon.png is excluded by !**/*.png
• src/images/logo-dark.png is excluded by !**/*.png
• src/images/logo-light.png is excluded by !**/*.png
• src/images/logo-ni2.png is excluded by !**/*.png
• src/images/logo.svg is excluded by !**/*.svg
• src/images/ni-stamp-back.svg is excluded by !**/*.svg
• src/images/ni-stamp.svg is excluded by !**/*.svg
• src/images/old-new-schedule.jpg is excluded by !**/*.jpg
• src/images/select-new-schedule.jpg is excluded by !**/*.jpg
• src/images/wait-new-schedule.jpg is excluded by !**/*.jpg

📒 Files selected for processing (40)

• .github/workflows/codeanalysis.yaml
• 8
• index.html
• package.json
• src/api/backend.ts
• src/api/services/exchangeRequestService.ts
• src/components/config/StagingMode.tsx
• src/components/home/index.ts
• src/components/layout/ComingSoon.tsx
• src/components/planner/ResetDataModal.tsx
• src/components/planner/index.ts
• src/components/planner/schedule/index.ts
• src/components/planner/sidebar/CoursesController/ClassItem.tsx
• src/components/planner/sidebar/CoursesController/ProfessorItem.tsx
• src/components/planner/sidebar/OptionsController.tsx
• src/components/planner/sidebar/SelectedOptionController.tsx
• src/components/planner/sidebar/index.ts
• src/components/planner/sidebar/sessionController/index.ts
• src/components/svgs/Linktree.tsx
• src/components/svgs/NitSigIcon.tsx
• src/components/svgs/fancy/Consult.tsx
• src/components/svgs/fancy/Select.tsx
• src/components/svgs/fancy/Tidy.tsx
• src/components/svgs/index.ts
• src/components/ui/alert-dialog.tsx
• src/components/ui/sidebar.tsx
• src/contexts/PreviewContext.tsx
• src/hooks/use-mobile.tsx
• src/hooks/useDirectExchangeValidation.tsx
• src/hooks/useEligibleExchange.tsx
• src/hooks/useReceivedRequests.tsx
• src/hooks/useSentRequests.tsx
• src/images/index.ts
• src/pages/TimeTableSelector.tsx
• src/pages/index.ts
• src/styles/safelist.txt
• src/test/App.test.tsx
• src/test/reportWebVitals.ts
• src/test/setupTests.ts
• vite.config.ts

💤 Files with no reviewable changes (25)

• src/components/planner/ResetDataModal.tsx
• src/components/svgs/fancy/Select.tsx
• src/test/setupTests.ts
• src/components/planner/sidebar/sessionController/index.ts
• src/components/planner/schedule/index.ts
• src/components/svgs/fancy/Tidy.tsx
• src/hooks/use-mobile.tsx
• src/test/App.test.tsx
• src/components/planner/sidebar/index.ts
• src/hooks/useReceivedRequests.tsx
• src/components/svgs/NitSigIcon.tsx
• src/components/svgs/Linktree.tsx
• src/components/svgs/index.ts
• src/components/ui/sidebar.tsx
• src/hooks/useSentRequests.tsx
• src/styles/safelist.txt
• vite.config.ts
• src/contexts/PreviewContext.tsx
• src/components/svgs/fancy/Consult.tsx
• src/components/config/StagingMode.tsx
• src/components/layout/ComingSoon.tsx
• src/components/home/index.ts
• src/components/ui/alert-dialog.tsx
• src/test/reportWebVitals.ts
• src/hooks/useEligibleExchange.tsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

The PR adds formatting validation, updates dependencies and favicon metadata, removes unused application exports and components, and applies formatting-only changes to API, planner, exchange, and timetable code.

Changes

Application cleanup and tooling

Layer / File(s)	Summary
Tooling and dependency configuration .github/workflows/codeanalysis.yaml, package.json, vite.config.ts	The lint workflow now runs npm run check. Unused dependencies and legacy ESLint configuration are removed. The Vite ESLint plugin is no longer registered.
Unused application surface removal src/components/**, src/hooks/**, src/images/index.ts, src/pages/index.ts	Unused components, hooks, barrel exports, image exports, and the ExchangePage export are removed.
API and component formatting src/api/**, src/components/planner/**, src/hooks/useDirectExchangeValidation.tsx, src/pages/TimeTableSelector.tsx	Quote style, semicolons, indentation, and JSX layout are reformatted without functional changes.
Browser icon metadata index.html	The apple-touch icon path is updated, and 16x16 and 32x32 favicon links are added.

Merge Risk: ⚪ Minimal · up to dba38

This cleanup removes unused code and dependencies, adds formatting validation, and corrects browser icon metadata. No current behavior-affecting risk has been identified.

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (2 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 11 files. (3 skipped: 3…	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	⚠️ Warning	The description states the general purpose of the PR, but it does not include the required issue reference, testing steps, or screenshot section from the repository template.	Add the "Closes #" reference, describe how to test the changes, and include the Screenshots section with screenshots or state that none are required.
Title check	❓ Inconclusive	The title identifies a cleanup change, but "chore: Cleanup" is too generic to describe the removal of dead code, dependency cleanup, favicon fixes, and formatting-check workflow change.	Use a specific title, such as "chore: Remove dead code and add formatting checks".

✅ Passed checks (2 passed)

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

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 11 files. (3 skipped: 3 unsupported.)

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Create stacked PR
• Commit on current branch

---

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

[github-actions]

🌐 Netlify Preview disponível:
https://6a9ebbcaa9b96b0008b54faf--tts-fe-preview.netlify.app