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.

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

next-ui recipe stuck "generating file diff" #4189

Open
JackDevAU opened this issue Aug 2, 2023 · 5 comments · May be fixed by #4287
Open

next-ui recipe stuck "generating file diff" #4189

JackDevAU opened this issue Aug 2, 2023 · 5 comments · May be fixed by #4287
Assignees
Labels
kind/bug Something isn't working status/assigned

Comments

@JackDevAU
Copy link

What is the problem?

I've just created a new project, without auth. I wanted to try and use next-ui using the blitz cli but when I use the blitz install next-ui command it fails to "generate file diffs"
When updating "the LabeledTextField wit Next UI's Input Component"

Paste all your error logs here:

  ERROR  The "path" argument must be of type string or an instance of Buffer or URL. Received undefined

 node:fs:592:10

 - Object.openSync (node:fs:592:10)
 - Object.readFileSync (node:fs:468:35)
 -  (recipe-install/node_modules/blitz/dist/installer.cjs:478:44)
 -     at Generator.next (<anonymous>)
 - fulfilled (recipe-install/node_modules/blitz/dist/installer.cjs:439:24)
 - processTicksAndRejections (node:internal/process/task_queues:95:5)

2023-08-02 14:39:49.850 ERROR   []
 TypeError  The "path" argument must be of type string or an instance of Buffer or URL. Received undefined
error stack:
  • fs  Object.openSync
        fs:592
  • fs  Object.readFileSync
        fs:468
  • \recipe-install\node_modules\blitz\dist\installer.cjs
        \recipe-install\node_modules\blitz\dist\installer.cjs:478
  •

  • \recipe-install\node_modules\blitz\dist\installer.cjs       fulfilled
        \recipe-install\node_modules\blitz\dist\installer.cjs:439
  • task_queues processTicksAndRejections
        internal\process\task_queues:95

Paste all relevant code snippets here:

N/A

What are detailed steps to reproduce this?

  1. Using the cli blitz new testing-blitz-next-ui
    1. √ Pick a new project's language » TypeScript
    2. √ Pick your new app template » Minimal — no DB, no auth
    3. √ Install dependencies? » npm
  2. cd into folder
  3. install next-ui blitz install next-ui
  4. See error when "Updating LabeledTextField"

Run blitz -v and paste the output here:

Blitz version: 2.0.0-beta.31 (global)
Blitz version: 2.0.0-beta.31 (local)
Windows 11 | win32-x64 | Node: v18.14.1


 Package manager: npm

  System:
    OS: Windows 10 10.0.22621
    CPU: (16) x64 11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz
    Memory: 6.13 GB / 31.74 GB
  Binaries:
    Node: 18.14.1 - ~\scoop\apps\nvm\current\nodejs\nodejs\node.EXE
    Yarn: 1.22.19 - ~\scoop\apps\yarn\current\bin\yarn.CMD
    npm: 9.3.1 - ~\scoop\apps\nvm\current\nodejs\nodejs\npm.CMD
  npmPackages:
    @blitzjs/auth: 2.0.0-beta.31 => 2.0.0-beta.31
    @blitzjs/next: 2.0.0-beta.31 => 2.0.0-beta.31
    @blitzjs/rpc: 2.0.0-beta.31 => 2.0.0-beta.31
    @prisma/client: Not Found
    blitz: 2.0.0-beta.31 => 2.0.0-beta.31
    next: 13.4.5 => 13.4.5
    prisma: Not Found
    react: 18.2.0 => 18.2.0
    react-dom: 18.2.0 => 18.2.0
    typescript: ^4.8.4 => 4.9.5

Please include below any other applicable logs and screenshots that show your problem:

image
Figure: Stuck generating file diff

@JackDevAU JackDevAU added kind/bug Something isn't working status/triage labels Aug 2, 2023
@bdlowery
Copy link

bdlowery commented Aug 7, 2023

I am experience the same problem, but with the tailwind recipe - #4192

@siddhsuresh siddhsuresh added the status/ready-to-work-on This issue is up for grabs label Aug 14, 2023
@openspaceranger
Copy link

openspaceranger commented Jan 12, 2024

I experienced the same problem with both tailwind and material ui. Is it possible to force it now to diff aside from killing the process?

This is for material-ui

  ERROR  The "path" argument must be of type string or an instance of Buffer or URL. Received undefined

 node:fs:579:10


 - Object.openSync (node:fs:579:10)
 - Object.readFileSync (node:fs:457:35)
 -  (recipe-install/node_modules/blitz/dist/installer.cjs:478:44)
 -     at Generator.next (<anonymous>)
 - fulfilled (recipe-install/node_modules/blitz/dist/installer.cjs:439:24)
 - processTicksAndRejections (node:internal/process/task_queues:95:5)

2024-01-11 22:11:43.166 ERROR   []
 TypeError  ERR_INVALID_ARG_TYPE, The "path" argument must be of type string or an instance of Buffer or URL. Received undefined
error stack:
  • fs  Object.openSync
        fs:579
  • fs  Object.readFileSync
        fs:457
  • \recipe-install\node_modules\blitz\dist\installer.cjs
        \recipe-install\node_modules\blitz\dist\installer.cjs:478
  •

  • \recipe-install\node_modules\blitz\dist\installer.cjs       fulfilled
        \recipe-install\node_modules\blitz\dist\installer.cjs:439
  • task_queues processTicksAndRejections
        internal\process\task_queues:95

Run blitz -v and paste the output here:

 Package manager: npm

  System:
    OS: Windows 10 10.0.19044
    CPU: (8) x64 Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz
    Memory: 14.40 GB / 31.78 GB
  Binaries:
    Node: 20.10.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 4.0.2 - C:\Program Files\nodejs\yarn.CMD
    npm: 10.2.3 - C:\Program Files\nodejs\npm.CMD
  npmPackages:
    @blitzjs/auth: 2.0.2 => 2.0.2
    @blitzjs/next: 2.0.2 => 2.0.2
    @blitzjs/rpc: 2.0.2 => 2.0.2
    @prisma/client: 5.4.2 => 5.4.2
    blitz: 2.0.2 => 2.0.2
    next: 13.5.4 => 13.5.4
    prisma: 5.4.2 => 5.4.2
    react: 18.2.0 => 18.2.0
    react-dom: 18.2.0 => 18.2.0
    typescript: 4.8.4 => 4.8.4

@Doc0x1
Copy link
Contributor

Doc0x1 commented Jan 21, 2024

I'm going to work on getting this issue fixed. Pretty sure I know what the cause is.

@siddhsuresh
Copy link
Member

that's great @Doc0x1! I will assign this issue to you. Feel free to tag me and we can get the fix merged.

@blitzjs-bot blitzjs-bot added status/assigned and removed status/ready-to-work-on This issue is up for grabs labels Jan 21, 2024
@Doc0x1
Copy link
Contributor

Doc0x1 commented Jan 21, 2024

that's great @Doc0x1! I will assign this issue to you. Feel free to tag me and we can get the fix merged.

Okay cool, I'm actually done writing up a discussion discussing the problem with the code at length, since it's a multi-layered problem and I thought it'd be worth trying to get some context/ideas/suggestions from others who might have some good strategies or ideas to lend me. It's here if you wanna check it out: #4286

@Doc0x1 Doc0x1 linked a pull request Jan 22, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working status/assigned
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants