Skip to content

Conversation

@stbestichhh
Copy link
Owner

No description provided.

@stbestichhh stbestichhh self-assigned this Sep 9, 2025
Copilot AI review requested due to automatic review settings September 9, 2025 07:57
@stbestichhh stbestichhh added the enhancement New feature or request label Sep 9, 2025
@stbestichhh stbestichhh linked an issue Sep 9, 2025 that may be closed by this pull request
Copy link
Contributor

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 reduces package size and adds new pagination functionality by replacing the uuid dependency with Node.js built-in crypto module and implementing a findAllPaginated method.

  • Replaced uuid v7 dependency with Node.js built-in randomUUID for ID generation
  • Added findAllPaginated method for paginated queries using Sequelize's findAndCountAll
  • Updated type definitions to support string or number ID generation

Reviewed Changes

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

Show a summary per file
File Description
src/index.ts Updated ID generator to use crypto.randomUUID and added findAllPaginated method implementation
src/abstract.repository.ts Same changes as index.ts - replaced uuid with crypto and added pagination method
src/IRepository.ts Added findAllPaginated method signature to the interface
test/abstract.repository.spec.ts Added test case for the new findAllPaginated functionality
package.json Removed uuid dependency and bumped version to 0.1.10
README.md Updated documentation to reflect UUIDv4 instead of v7
CHANGELOG.md Added changelog entries for version 0.1.10
SECURITY.md Added version 0.1.10 to supported versions table
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@stbestichhh stbestichhh merged commit 5b43774 into master Sep 9, 2025
6 checks passed
@stbestichhh stbestichhh deleted the 22-add-findandcountall branch September 9, 2025 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add findAndCountAll

2 participants