Sweep: Fix logical error in book by category endpoint

[sweep-ai[bot]]

PR Feedback (click)

• [ ] 👍 Sweep Did Well
• [ ] 👎 Sweep Needs Improvement

  Description

  This pull request addresses a logical error in the book by category endpoint within our book management system. Previously, the validation for category names directly used a regex within the route handler, which not only cluttered the code but also made it less maintainable. This update refactors the validation logic by moving it into a separate utility function, thereby improving code readability and maintainability.

Summary

• Moved category name validation logic from the routes/book.routes.js file to a new utility function in utils/validateInput.js.
• Introduced a new case category in the validateInput function to check for alphabetic characters in the category name, ensuring that the validation logic is reusable and easily maintainable.
• Modified the category endpoint in routes/book.routes.js to use the new validateInput utility function for category name validation, simplifying the endpoint logic and improving code readability.
• This refactor ensures that the endpoint now correctly returns a 404 status code for invalid category names before attempting to query the database, which optimizes performance and error handling.

Files modified:

• routes/book.routes.js: Refactored to use the new validation logic.
• utils/validateInput.js: Enhanced to include category name validation.

Fixes #142.

---

🎉 Latest improvements to Sweep:

• New dashboard launched for real-time tracking of Sweep issues, covering all stages from search to coding.
• Integration of OpenAI's latest Assistant API for more efficient and reliable code planning and editing, improving speed by 3x.
• Use the GitHub issues extension for creating Sweep issues directly from your editor.

---

💡 To get Sweep to edit this pull request, you can:

• Comment below, and Sweep can edit the entire PR
• Comment on a file, Sweep will only modify the commented file
• Edit the original issue to get Sweep to recreate the PR from scratch

This is an automated message generated by Sweep AI.

[sweep-ai[bot]]

Rollback Files For Sweep

• [ ] Rollback changes to utils/validateInput.js
• [ ] Rollback changes to routes/book.routes.js

This is an automated message generated by Sweep AI.

[sweep-ai[bot]]

Apply Sweep Rules to your PR?

• [ ] Apply: All docstrings and comments should be up to date.
• [ ] Apply: All new business logic should have corresponding unit tests.
• [ ] Apply: Refactor large functions to be more modular.
• [ ] Apply: Add docstrings to all functions and file headers.

This is an automated message generated by Sweep AI.