Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Types not correctly imported #1

Open
sifferhans opened this issue Nov 7, 2023 · 6 comments · Fixed by unjs/nitro#1908 or unjs/unimport#303
Open

Types not correctly imported #1

sifferhans opened this issue Nov 7, 2023 · 6 comments · Fixed by unjs/nitro#1908 or unjs/unimport#303

Comments

@sifferhans
Copy link
Contributor

sifferhans commented Nov 7, 2023 •
edited

It seems like the types generated in .nuxt/types/nitro-imports.d.ts imports from the wrong files.
The utilities (setUserSession, getUserSession etc.) are being imported from the .mjs files as seen in the screenshot below.

CleanShot 2023-11-07 at 13 52 19

This causes everything to be typed as any, as in this githubEventHandler example:

CleanShot 2023-11-07 at 13 55 07

I guess this might have something to do with the way the utils are imported here?

CleanShot 2023-11-07 at 13 56 22
@danielroe
Copy link
Collaborator

This seems like an issue in unimport rather than this module.

@danielroe
Copy link
Collaborator

added a workaround but we still need to address in unimport... (cc: @antfu)

@Gerbuuun
Copy link
Contributor

Gerbuuun commented Nov 9, 2023

What is the workaround? The server utilities have correct typings but the useUserSession composable does not.

@danielroe danielroe reopened this Nov 9, 2023
@migueloliveiradev
Copy link

Same problem

antfu added a commit to unjs/nitro that referenced this issue Nov 13, 2023
@antfu
fix: scan dirs for unimport on initialization
3a63c6c 
Atinux/nuxt-auth-utils#1
@antfu antfu mentioned this issue Nov 13, 2023
Merged
8 tasks
@kingyue737

This comment was marked as off-topic.

Sign in to view
@productdevbook
Copy link

@antfu unjs/nitro#1908 dont fixed this issua :/ i will test nitro 2.8.1

@productdevbook productdevbook mentioned this issue Dec 5, 2023
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
6 participants
@Gerbuuun @sifferhans @danielroe @productdevbook @kingyue737 @migueloliveiradev