Skip to content

Conversation

@danwkennedy
Copy link
Contributor

Description

We're upgrading all the packages in this repo to ESM modules. @actions/glob can go next since all its dependencies are ESM modules.

@danwkennedy danwkennedy requested a review from a team as a code owner January 29, 2026 02:12
Copilot AI review requested due to automatic review settings January 29, 2026 02:12
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 converts the @actions/glob package from CommonJS to an ESM module, as part of a broader effort to upgrade all packages in the toolkit repository. The conversion includes updating TypeScript configuration, adding .js extensions to all relative imports, updating package metadata, and bumping the major version to 0.6.0 to reflect the breaking change.

Changes:

  • Updated package configuration to ESM format with "type": "module" and proper exports field
  • Modified all relative imports across source and test files to include .js extensions (required for ESM)
  • Upgraded dependency @actions/core from ^2.0.3 to ^3.0.0 (ESM version)
  • Added TypeScript ESM module configuration with "module": "node16" and "moduleResolution": "node16"

Reviewed changes

Copilot reviewed 16 out of 18 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/glob/package.json Added ESM package configuration ("type": "module", exports field), bumped version to 0.6.0, upgraded @actions/core dependency
packages/glob/package-lock.json Updated lockfile to reflect version changes and dependency upgrades
packages/glob/tsconfig.json Added ESM module configuration for TypeScript compilation
packages/glob/src/*.ts Added .js extensions to all relative imports in source files
packages/glob/tests/*.test.ts Added .js extensions to all relative imports in test files
packages/glob/RELEASES.md Added release notes for 0.6.0 documenting the breaking ESM change and reformatted previous entries
jest.config.js Added @actions/glob to module name mapper and transform ignore patterns for ESM support
Files not reviewed (1)
  • packages/glob/package-lock.json: Language not supported

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@danwkennedy danwkennedy merged commit 7a0147b into main Jan 29, 2026
16 of 17 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.

3 participants