Skip to content

Conversation

@fraxken
Copy link
Member

@fraxken fraxken commented Dec 18, 2025

No description provided.

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 refactors the codebase to leverage Node.js native TypeScript type stripping capabilities, eliminating the need for separate build tools. The changes modernize the development workflow and dependencies while maintaining the same API surface.

Key changes:

  • Migrated from @myunisoft/httpie to native undici for HTTP requests
  • Split monolithic src/index.ts into separate files for better organization (api.ts and github.types.ts)
  • Updated TypeScript configuration to use @openally/config.typescript/esm-ts-next for native type stripping support
  • Simplified build and test scripts to use native Node.js capabilities (node --test)

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tsconfig.json Updated TypeScript configuration to extend ESM-TS-Next config for native type stripping
test/fetch.spec.ts Changed import extension from .js to .ts to support native type stripping
src/index.ts Refactored to re-export from modularized files instead of containing all logic
src/github.types.ts New file containing exported TypeScript interfaces for GitHub API types
src/api.ts New file containing main API logic with migrated HTTP client from httpie to undici
package.json Updated dependencies, removed build tools (tsx, rimraf, glob), simplified scripts, added exact version constraints
.npmrc Added ignore-scripts and save-exact configuration flags
.github/workflows/node.js.yml Updated Node.js version to 24.x and added --ignore-scripts flag to install command
.github/dependabot.yml Added cooldown configuration with 5-day default for both npm and GitHub Actions updates

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@fraxken fraxken merged commit caeb897 into master Dec 18, 2025
8 checks passed
@fraxken fraxken deleted the nodejs-next branch December 18, 2025 18:43
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