Skip to content

update backend dependencies #148

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 5, 2025
Merged

update backend dependencies #148

merged 2 commits into from
May 5, 2025

Conversation

adamrefaey
Copy link
Collaborator

  • chore: update dependencies and refactor perplexity module
  • Updated AWS SDK and NestJS dependencies to their latest versions.
  • Removed PerplexityController and related DTOs as they are no longer needed.
  • Adjusted AppModule to remove references to the deleted PerplexityController.
  • Updated route handling in AppModule for better clarity.

- Updated AWS SDK and NestJS dependencies to their latest versions.
- Removed PerplexityController and related DTOs as they are no longer needed.
- Adjusted AppModule to remove references to the deleted PerplexityController.
- Updated route handling in AppModule for better clarity.
@adamrefaey adamrefaey requested review from GuidoBR and Copilot May 4, 2025 09:26
@adamrefaey adamrefaey self-assigned this May 4, 2025
Copilot

This comment was marked as outdated.

@adamrefaey adamrefaey requested a review from Copilot May 4, 2025 09:56
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates backend dependencies, refactors the perplexity module, and adjusts route handling in the AppModule. Key changes include:

  • Updating the AWS SDK and NestJS dependencies and refactoring vitest.config.ts to use asynchronous configuration.
  • Removing the PerplexityController and related DTOs, while leaving the PerplexityService intact.
  • Changing the route wildcard in AppModule from '*' to '*path' for improved clarity.

Reviewed Changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
backend/vitest.config.ts Refactored configuration to support async import of tsconfigPaths.
backend/src/controllers/perplexity/*.ts Removed unused controller and DTO files.
backend/src/app.module.ts Removed PerplexityController and updated the wildcard route pattern.
backend/eslint.config.js Introduced a new ESLint configuration.
backend/.eslintrc.js Removed in favor of the new ESLint configuration.
Files not reviewed (2)
  • backend/.eslintignore: Language not supported
  • backend/package.json: Language not supported
Comments suppressed due to low confidence (2)

backend/.eslintrc.js:1

  • Confirm that the removal of .eslintrc.js in favor of eslint.config.js fully captures all necessary ESLint rules and overrides to maintain consistent linting.
module.exports = { ... }

backend/src/app.module.ts:35

  • Ensure that the new wildcard pattern 'path' matches all intended routes, as it may behave differently compared to the previous '' pattern.
.forRoutes('*path');

@GuidoBR GuidoBR merged commit 50a4143 into main May 5, 2025
1 check passed
@adamrefaey adamrefaey deleted the upgrade-backend-packages branch May 5, 2025 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants