Skip to content

fix(docs): use named minimatch import#2433

Merged
frankensteinke merged 1 commit into
developfrom
fix/docs-minimatch-named-import
Jun 8, 2026
Merged

fix(docs): use named minimatch import#2433
frankensteinke merged 1 commit into
developfrom
fix/docs-minimatch-named-import

Conversation

@frankensteinke

@frankensteinke frankensteinke commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

docs/collections.ts imported minimatch as a default export, but minimatch v10 (which we pinned in #2417) is ESM-only with named exports. This was breaking the docs site.

minimatch v10 is ESM-only and exposes named exports with no default, so
'import minimatch from minimatch' resolves to undefined — webpack warns
('export default not found') and collections.ts would throw
'minimatch is not a function' at runtime. Switch to the named import
'import { minimatch }'. Surfaced once minimatch was declared as a direct
dep at ^10 (#2417); previously an older CJS minimatch with a default
export was hoisted to the top of node_modules and resolved here by
accident.
@frankensteinke
frankensteinke marked this pull request as ready for review June 8, 2026 17:06
@frankensteinke
frankensteinke requested a review from a team as a code owner June 8, 2026 17:06
Copilot AI review requested due to automatic review settings June 8, 2026 17:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 fixes the docs site build by updating the minimatch import in docs/collections.ts to use the named minimatch export, which is required for minimatch@10 (ESM-only).

Changes:

  • Replace the default minimatch import with a named import ({ minimatch }) to match minimatch@10’s export shape.

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

@aws-amplify-us-east-1

Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-2433.d15792l1n26ww3.amplifyapp.com

@frankensteinke
frankensteinke merged commit 8284d94 into develop Jun 8, 2026
12 checks passed
@frankensteinke
frankensteinke deleted the fix/docs-minimatch-named-import branch June 8, 2026 17:18
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