Skip to content

Conversation

buenoC6
Copy link

@buenoC6 buenoC6 commented Sep 25, 2025

Description

The "./src/" entry in the exports field was invalid since folder exports must point to a directory (ending with "/"), not to file patterns. This entry is also redundant with "./src/*" and caused errors when parsing package.json in strict environments (e.g. Node 16+).

Added a patch changeset.

The "./src/" entry in the exports field was invalid since folder exports must point to a directory (ending with "/"), not to file patterns. 
This entry is also redundant with "./src/*" and caused errors when parsing package.json in strict environments (e.g. Node 16+).


Signed-off-by: Alexian Moins <[email protected]>
Copy link

changeset-bot bot commented Sep 25, 2025

🦋 Changeset detected

Latest commit: 2afa63a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@forgerock/javascript-sdk Patch
@forgerock/ping-protect Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ryanbas21
Copy link
Contributor

ryanbas21 commented Sep 29, 2025

Description

The "./src/" entry in the exports field was invalid since folder exports must point to a directory (ending with "/"), not to file patterns. This entry is also redundant with "./src/*" and caused errors when parsing package.json in strict environments (e.g. Node 16+).

Added a patch changeset.

Hey, thanks for this, and good catch. sorry i missed this for so long. I'll talk to product about a patch release for this in the future, i can't promise when appreciate the PR.

I may need to opent his myself since we don't use pull_request_target on ci for security and we need a key to run the commands in ci

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