log4brains@1.0.0-beta.8 preview and build not working even with the base setup #27

rahul-gottupulla · 2021-03-08T14:06:16Z

Bug Report

Description

The "next" package is being referred internally and it installed the latest automatically. Earlier this week, when I installed log4brains@1.0.0-beta.8 "next 10.0.7" was being used and everything is working as expected, Now it's referring to the "next 10.0.8" and things started to break.

Steps to Reproduce

Installed the latest version of the log4brains.
Ran "log4brains init" to create the .log4brains.yml and docs folder.
"log4brains preview" is failing with the below error.
"log4brains build" is executing till the end but the outputs are not being generated.

Environment

Log4brains version: 1.0.0-beta.8
Node.js version: v15.10.0
OS and its version: Windows 10
Browser information: NA

Possible Solution

Fix the reference to the NEXTJS.

esselius · 2021-03-08T14:44:53Z

Same here:

❯ DEBUG=1 npx log4brains preview                                                                                                                                                                                                                                  
Run `next start`...
[==  ] Log4brains is starting... Error


 FATAL  Cannot read property 'incrementalOptions' of undefined

TypeError: Cannot read property 'incrementalOptions' of undefined
    at previewCommand (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/@log4brains/web/dist/cli/commands/preview.js:27:26)
    at Command.program.command.description.option.option.action (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/@log4brains/web/dist/cli/cli.js:13:16)
    at Command.listener [as _actionHandler] (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/commander/index.js:426:31)
    at Command._parseCommand (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/commander/index.js:1002:14)
    at Command._dispatchSubcommand (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/commander/index.js:953:18)
    at Command._parseCommand (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/commander/index.js:970:12)
    at Command.parse (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/commander/index.js:801:10)
    at Command.parseAsync (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/commander/index.js:828:10)
    at Object.<anonymous> (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/dist/main.js:26:9)
    at Generator.next (<anonymous>)
    at bl (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/esm/esm.js:1:245412)
    at kl (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/esm/esm.js:1:247659)
    at Object.u (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/esm/esm.js:1:287740)
    at Object.o (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/esm/esm.js:1:287137)
    at Object.<anonymous> (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/esm/esm.js:1:284879)
    at Object.apply (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/esm/esm.js:1:199341)

[Next] 
info  - Collecting page data  
[Next]  info  - Generating static pages (5/5)
info  - Finalizing page optimization  
[Next] 
[Next]  Page                                                             Size     First Load JS
┌ ● / (ISR: 1 Seconds)                                           229 B           475 kB
├   /_app                                                        0 B            47.3 kB
├ ○ /404                                                         1.21 kB        48.5 kB
├ ● /adr/[...slug]                                               238 B           475 kB
├   ├ /adr/20210306-use-log4brains-to-manage-the-adrs
├   └ /adr/20210306-use-markdown-architectural-decision-records
├ λ /api/adr                                                     0 B            47.3 kB
├ λ /api/adr/[...slugAndMore]                                    0 B            47.3 kB
└ λ /api/search-index                                            0 B            47.3 kB
+ First Load JS shared by all                                    47.3 kB
  ├ chunks/a3e17150ead73d62fba7a33a50672ae3f72841a5.bea17f.js    20.3 kB
  ├ chunks/commons.4040d7.js                                     17.3 kB
  ├ chunks/main.f303bd.js                                        6.34 kB
  ├ chunks/pages/_app.f3033c.js                                  2.67 kB
  ├ chunks/webpack.50bee0.js                                     751 B
  └ css/d50ac9751859b3eb4bb6.css                                 390 B
[Next] 
[Next]  λ  (Lambda)  server-side renders at runtime (uses getInitialProps or getServerSideProps)
○  (Static)  automatically rendered as static HTML (uses no initial props)
●  (SSG)     automatically generated as static HTML + JSON (uses getStaticProps)
   (ISR)     incremental static regeneration (uses revalidate in getStaticProps)
[Next] 
Run `next export`...
[Next]  
[Next]   FATAL  The "path" argument must be of type string. Received type undefined
[Next]  
[Next]  TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type undefined
    at validateString (internal/validators.js:125:11)
    at join (path.js:1147:7)
    at /Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/next/dist/export/index.js:3:363
    at tracer.withSpan (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/next/dist/build/tracer.js:3:476)
    at NoopTracer.withSpan (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/@opentelemetry/api/build/src/trace/NoopTracer.js:47:16)
    at ProxyTracer.withSpan (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/@opentelemetry/api/build/src/trace/ProxyTracer.js:36:34)
    at traceAsyncFn (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/next/dist/build/tracer.js:3:435)
    at exportApp (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/next/dist/export/index.js:2:475)
    at execNext (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/@log4brains/web/dist/cli/commands/build.js:45:15)
    at execNext (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/@log4brains/web/dist/lib/console.js:20:11)
    at buildCommand (/Users/peteresselius/.npm/prefix/lib/node_modules/log4brains/node_modules/@log4brains/web/dist/cli/commands/build.js:44:11)
[Next]

sahanatroam · 2021-03-16T19:59:43Z

any update on this:
following is the line that's breaking

log4brains/packages/web/src/cli/commands/preview.ts

Line 41 in 50e15a5

    
           app.incrementalCache.incrementalOptions.dev = true; // eslint-disable-line @typescript-eslint/no-unsafe-member-access

thomvaill#27

ThomasCarca · 2021-03-17T16:37:50Z

I'm having the same issue. Tried downgrading log4brains but it did not work unfortunately.

mischak · 2021-03-21T21:21:52Z

Same issue here, neither preview nor build is working, not on Ubuntu and not on Windows.
My env:
Ubuntu 18.04.5 LTS
Node v14.16.0
npm 6.14.11

marc-bouvier · 2021-03-23T22:56:07Z

Reproduced in

WSL2 : Ubuntu 20.04
Node : v14.16.0
Npm : 7.7.0

See: /thomvaill/log4brains#27

See: /thomvaill/log4brains#27 Original-Commit: TeoZosa/cookiecutter-cruft-poetry-tox-pre-commit-ci-cd@5c8b260

See: /thomvaill/log4brains#27

See: /thomvaill/log4brains#27 Original-Commit: TeoZosa/cookiecutter-cruft-poetry-tox-pre-commit-ci-cd@12dab5d

See: /thomvaill/log4brains#27

See: /thomvaill/log4brains#27 Original-Commit: TeoZosa/cookiecutter-cruft-poetry-tox-pre-commit-ci-cd@59c807d

because we rely on many "hacks" of Next.js, we cannot use semver

thomvaill · 2021-04-30T09:13:21Z

Hi all!
I am very sorry for my long absence...
Leaving a bug like this one not resolved for 2 months is just a shame!! 😱
I had a lot of work in March and got the Covid at the beginning of April (hopefully not severe!) but from now on I will be able to work on the project again in my spare time! 🙏

And hopefully, the bug is now fixed! You can test by running:

npm uninstall -g log4brains
npm install -g log4brains

I also tried with npm update -g log4brains but I encountered this error:

/usr/local/lib/node_modules/log4brains/node_modules/next/dist/compiled/compression/index.js:1
ReferenceError: __nccwpck_require__ is not defined
    at __webpack_require__ (/usr/local/lib/node_modules/log4brains/node_modules/next/dist/compiled/compression/index.js:1)
    at /usr/local/lib/node_modules/log4brains/node_modules/next/dist/compiled/compression/index.js:1
    at Object.<anonymous> (/usr/local/lib/node_modules/log4brains/node_modules/next/dist/compiled/compression/index.js:1)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1131:10)

It seems like Next.js does not like to be downgraded!

The fix is explained in this PR: #30
As you can see, it was because I didn't pin correctly the version of Next.js.
It broke very fast because I rely heavily on Next.js internal architecture, but I fear a similar issue can happen again in the future with some other dependencies.
That's why I started this discussion to look for a solution: #33
I would love to hear your opinions on this!

esselius · 2021-05-03T06:04:55Z

@thomvaill Hope you're feeling better! :)

I think everyone's excited to have the best ADR-tool working again! :)

Dutzu · 2022-05-31T11:54:50Z

Hey there, Issue seems to have popped up again in 1.0.0-beta.12. Same line, same error....

mangoldd · 2022-07-12T08:21:15Z

I also got the error (starting) with 1.0.0-beta.12. Updating to 1.0.0-beta.13 did not help.

[==  ] Log4brains is starting... Error

 FATAL  Cannot read property 'incrementalOptions' of undefined

I also tried:

npm uninstall -g log4brains
npm install -g log4brains

...but it did not help. Did anyone already open a new bug or is there a workaround?

thomvaill · 2022-07-14T09:59:05Z

Ah... I also had an issue when upgrading from 1.0.0-beta.12 to 1.0.0-beta.13 but I thought it was because of my local development setup.
Could you try this to see if it fixes the problem?

npm uninstall -g log4brains
rm -rf "$(npm root -g)/log4brains"
npm install -g log4brains

thomvaill · 2022-07-14T10:03:13Z

I close this issue. Let's continue the discussion on #80

rahul-gottupulla added the bug Something isn't working label Mar 8, 2021

sahanatroam added a commit to sahanatroam/log4brains that referenced this issue Mar 16, 2021

chore: make sure incrementalCache exists prior to destrcture

60ba85b

thomvaill#27

sahanatroam mentioned this issue Mar 16, 2021

chore: make sure incrementalCache exists prior to de-strcture #28

Closed

TeoZosa added a commit to TeoZosa/cookiecutter-cruft-poetry-tox-pre-commit-ci-cd that referenced this issue Mar 31, 2021

💚 Temporarily disable log4brains until fixed

5c8b260

See: /thomvaill/log4brains#27

TeoZosa added a commit to TeoZosa/cookiecutter-cruft-poetry-tox-pre-commit-ci-cd that referenced this issue Mar 31, 2021

💚 Temporarily disable log4brains until fixed

12dab5d

See: /thomvaill/log4brains#27

TeoZosa added a commit to TeoZosa/cookiecutter-cruft-poetry-tox-pre-commit-ci-cd that referenced this issue Mar 31, 2021

💚 Temporarily disable log4brains until fixed

59c807d

See: /thomvaill/log4brains#27

fire mentioned this issue Apr 28, 2021

log4brains is not maintained #29

Closed

thomvaill added a commit that referenced this issue Apr 29, 2021

fix: #27 pin nextjs to a specific version

b48d556

because we rely on many "hacks" of Next.js, we cannot use semver

thomvaill mentioned this issue Apr 29, 2021

Fix Next.js dependency #30

Merged

thomvaill closed this as completed in #30 Apr 29, 2021

thomvaill added a commit that referenced this issue Apr 29, 2021

fix: #27 pin nextjs to a specific version

320e1ba

because we rely on many "hacks" of Next.js, we cannot use semver

thomvaill reopened this Apr 29, 2021

thomvaill mentioned this issue Apr 30, 2021

fix(web): caniuse-lite outdated warning #32

Merged

thomvaill closed this as completed Apr 30, 2021

thomvaill mentioned this issue Apr 30, 2021

ESM cache not cleared during an upgrade #20

Open

Dutzu mentioned this issue May 31, 2022

cannot read property 'incrementalOptions' of undefined #78

Closed

thomvaill reopened this Jul 14, 2022

thomvaill closed this as completed Jul 14, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

log4brains@1.0.0-beta.8 preview and build not working even with the base setup #27

log4brains@1.0.0-beta.8 preview and build not working even with the base setup #27

rahul-gottupulla commented Mar 8, 2021

esselius commented Mar 8, 2021

sahanatroam commented Mar 16, 2021

ThomasCarca commented Mar 17, 2021

mischak commented Mar 21, 2021

marc-bouvier commented Mar 23, 2021

thomvaill commented Apr 30, 2021

esselius commented May 3, 2021

Dutzu commented May 31, 2022

mangoldd commented Jul 12, 2022

thomvaill commented Jul 14, 2022

thomvaill commented Jul 14, 2022

log4brains@1.0.0-beta.8 preview and build not working even with the base setup #27

log4brains@1.0.0-beta.8 preview and build not working even with the base setup #27

Comments

rahul-gottupulla commented Mar 8, 2021

Bug Report

Description

Steps to Reproduce

Environment

Possible Solution

esselius commented Mar 8, 2021

sahanatroam commented Mar 16, 2021

ThomasCarca commented Mar 17, 2021

mischak commented Mar 21, 2021

marc-bouvier commented Mar 23, 2021

thomvaill commented Apr 30, 2021

esselius commented May 3, 2021

Dutzu commented May 31, 2022

mangoldd commented Jul 12, 2022

thomvaill commented Jul 14, 2022

thomvaill commented Jul 14, 2022