Skip to content

Conversation

@fgiova
Copy link
Owner

@fgiova fgiova commented Sep 4, 2025

No description provided.

Copilot AI review requested due to automatic review settings September 4, 2025 08:36
Copy link

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 the project's dependencies and development tooling, migrating from ESLint to Biome for code formatting and linting, while modernizing build configuration and TypeScript types.

  • Updates major dependencies including undici (v5→v7) and @fgiova/aws-signature (v2→v3)
  • Replaces ESLint with Biome for consistent code formatting and linting
  • Modernizes build system with dual ESM/CommonJS support and improved package.json exports

Reviewed Changes

Copilot reviewed 10 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
test/index.ts Reformatted with Biome, updated imports to use node: protocol and type-only imports
test/error.ts Reformatted with Biome and updated imports to use node: protocol
src/schemas.ts Removed extra blank line for code formatting consistency
src/index.ts Major refactoring with improved TypeScript types, Biome formatting, and better type safety
package.json Updated build system, dependencies, and added dual module support
biome.json Added Biome configuration file
README.md Updated documentation and badge references
.github/workflows/node.js.yml Updated Node.js versions and switched coverage reporting to qlty
.eslintrc.js Removed ESLint configuration in favor of Biome
.eslintignore Removed ESLint ignore file
Comments suppressed due to low confidence (1)

src/index.ts:1

  • The splitArray function inside the test has a hardcoded batch size of 10 that doesn't match the updated function signature. Consider adding the maxItems parameter back or documenting why it was removed, as the function still uses Math.floor(index / 10) internally.
import { randomUUID } from "node:crypto";

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

@fgiova fgiova merged commit e513549 into main Sep 4, 2025
7 checks passed
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