Skip to content

fix: shared package for nodenext module resolution#1974

Merged
shikanime merged 1 commit intomainfrom
pr1974
Mar 12, 2026
Merged

fix: shared package for nodenext module resolution#1974
shikanime merged 1 commit intomainfrom
pr1974

Conversation

@shikanime
Copy link
Contributor

@shikanime shikanime commented Mar 5, 2026

Running tests using vitest in the server-nestjs package seems to break module resolution because of nodenext module resolution.

Error: Failed to resolve entry for package "@cpn-console/shared". The package may have incorrect main/module/exports specified in its package.json.

Signed-off-by: William Phetsinorath william.phetsinorath-open@interieur.gouv.fr


Stack created with Sapling. Best reviewed with ReviewStack.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

@shikanime
Copy link
Contributor Author

Est-ce que c'est necessaire, je m'en souviens plus

@shikanime shikanime closed this Mar 9, 2026
auto-merge was automatically disabled March 9, 2026 16:08

Pull request was closed

@shikanime shikanime reopened this Mar 11, 2026
@shikanime shikanime force-pushed the pr1974 branch 2 times, most recently from 1e8a704 to 91d5bd7 Compare March 11, 2026 11:19
@shikanime shikanime enabled auto-merge March 12, 2026 08:13
@shikanime shikanime force-pushed the pr1974 branch 3 times, most recently from 373e435 to c51c821 Compare March 12, 2026 10:49
Running tests using vitest in the server-nestjs package seems to break module resolution because of nodenext module resolution.

```
Error: Failed to resolve entry for package "@cpn-console/shared". The package may have incorrect main/module/exports specified in its package.json.
```

Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
@cloud-pi-native-sonarqube
Copy link

Copy link
Collaborator

@StephaneTrebel StephaneTrebel left a comment

Choose a reason for hiding this comment

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

LGTM, à voir ce que ça donne en aval (server, etc.)

@shikanime shikanime added this pull request to the merge queue Mar 12, 2026
Merged via the queue into main with commit c3c0581 Mar 12, 2026
15 of 16 checks passed
@shikanime shikanime deleted the pr1974 branch March 12, 2026 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

built tech Technical issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants