Skip to content

Conversation

@BartoszBlizniak
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings December 23, 2025 10:37
@BartoszBlizniak BartoszBlizniak merged commit 76c8ff5 into master Dec 23, 2025
19 of 20 checks passed
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 bumps the version from 2.0.0 to 2.0.1 and replaces the axios and node-fetch dependencies with native Node.js fetch API to address Node.js 24 deprecation warnings and reduce bundle size.

Key Changes:

  • Removed axios and node-fetch dependencies in favor of native fetch API
  • Implemented fetchWithTimeout function to replace axios HTTP requests
  • Updated error logging to handle native fetch response headers

Reviewed changes

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

Show a summary per file
File Description
package.json Version bumped to 2.0.1, removed axios and node-fetch dependencies
src/oidc-auth.js Removed axios import, added fetchWithTimeout function, updated authentication and validation to use native fetch
src/download-cli.js Replaced node-fetch with native fetch, simplified stream piping using pipeline
eslint.config.js Added clearTimeout and AbortController to ESLint globals
dist/37.index.js Updated bundled distribution file with new module IDs and exports
CHANGELOG.md Added entry for version 2.0.1 and duplicate entry in 2.0.0 section

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

- Updated test workflows to run on Node 24
- Updated documentation to v2
- OIDC Audience defaults to `https://github.com/{org-name}` from `api://AzureADTokenExchange`
- Replaced `axios` with native `fetch` API to fix Node.js 24 `url.parse()` deprecation warning (DEP0169) and reduce bundle size
Copy link

Copilot AI Dec 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This changelog entry is duplicated between version 2.0.0 and version 2.0.1. The change should only appear in version 2.0.1 where it was actually implemented. Remove this line from the 2.0.0 section.

Suggested change
- Replaced `axios` with native `fetch` API to fix Node.js 24 `url.parse()` deprecation warning (DEP0169) and reduce bundle size

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants