Skip to content

Conversation

n3rdc4ptn
Copy link
Member

… fastify routes

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

andreaskienle
andreaskienle previously approved these changes Jul 9, 2025
Copilot

This comment was marked as outdated.

@andreaskienle andreaskienle requested a review from Copilot July 9, 2025 12:45
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 standardizes code formatting and improves consistency across Fastify routes and plugins.

  • Switched all string delimiters from double to single quotes.
  • Added explicit return statements in route handlers and the onSend hook to ensure payloads are correctly returned.
  • Updated .prettierrc to increase printWidth to 120 for broader line lengths.

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
server/routes/root.js Unified quotes and added return for the root health check
server/routes/feedback.js Unified quotes and removed trailing comma in response object
server/routes/auth-onboarding.js Unified quotes, added return in callbacks, formatted handlers
server/routes/auth-mcp.js Unified quotes, added return in callbacks, formatted handlers
server/plugins/sensible.js Unified quotes
server/plugins/http-proxy.js Unified quotes, formatted preHandler, added return in hooks
server/plugins/auth-utils.js Unified quotes and formatted utility decorators
server/encrypted-session.js Unified quotes, formatted hooks, added return payload in onSend
server/app.js Unified quotes, formatted autoload paths
.prettierrc Increased printWidth from 80 to 120

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@n3rdc4ptn n3rdc4ptn merged commit 833ea4a into main Jul 10, 2025
5 checks passed
@n3rdc4ptn n3rdc4ptn deleted the prettier-and-fastify-fix branch July 10, 2025 08:14
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.

3 participants