Skip to content

fix: add missing dev dependencies for docs development#95

Merged
ShutdownRepo merged 1 commit intoThePorgs:mainfrom
skyf0l:fix/deps
Sep 19, 2025
Merged

fix: add missing dev dependencies for docs development#95
ShutdownRepo merged 1 commit intoThePorgs:mainfrom
skyf0l:fix/deps

Conversation

@skyf0l
Copy link
Contributor

@skyf0l skyf0l commented Sep 16, 2025

This PR adds vitepress and rollup as dev dependencies in package.json. This resolves issues with missing modules when running npm run docs:dev or npm run docs:build and ensures the documentation development server starts correctly. No other changes were made.

Before:

$ npm run docs:dev

> docs:dev
> vitepress dev docs

/home/skyf0l/repos/Exegol-docs/node_modules/rollup/dist/native.js:64
                throw new Error(
                      ^

Error: Cannot find module @rollup/rollup-linux-x64-gnu. npm has a bug related to optional dependencies (https://github.com/npm/cli/issues/4828). Please try `npm i` again after removing both package-lock.json and node_modules directory.
    at requireWithFriendlyError (/home/skyf0l/repos/Exegol-docs/node_modules/rollup/dist/native.js:64:9)
    at Object.<anonymous> (/home/skyf0l/repos/Exegol-docs/node_modules/rollup/dist/native.js:73:76)
    at Module._compile (node:internal/modules/cjs/loader:1554:14)
    at Object..js (node:internal/modules/cjs/loader:1706:10)
    at Module.load (node:internal/modules/cjs/loader:1289:32)
    at Function._load (node:internal/modules/cjs/loader:1108:12)
    at TracingChannel.traceSync (node:diagnostics_channel:322:14)
    at wrapModuleLoad (node:internal/modules/cjs/loader:220:24)
    at cjsLoader (node:internal/modules/esm/translators:262:5)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:196:7) {
  [cause]: Error: Cannot find module '@rollup/rollup-linux-x64-gnu'
  Require stack:
  - /home/skyf0l/repos/Exegol-docs/node_modules/rollup/dist/native.js
      at Function._resolveFilename (node:internal/modules/cjs/loader:1225:15)
      at Function._load (node:internal/modules/cjs/loader:1055:27)
      at TracingChannel.traceSync (node:diagnostics_channel:322:14)
      at wrapModuleLoad (node:internal/modules/cjs/loader:220:24)
      at Module.require (node:internal/modules/cjs/loader:1311:12)
      at require (node:internal/modules/helpers:136:16)
      at requireWithFriendlyError (/home/skyf0l/repos/Exegol-docs/node_modules/rollup/dist/native.js:46:10)
      at Object.<anonymous> (/home/skyf0l/repos/Exegol-docs/node_modules/rollup/dist/native.js:73:76)
      at Module._compile (node:internal/modules/cjs/loader:1554:14)
      at Object..js (node:internal/modules/cjs/loader:1706:10) {
    code: 'MODULE_NOT_FOUND',
    requireStack: [
      '/home/skyf0l/repos/Exegol-docs/node_modules/rollup/dist/native.js'
    ]
  }
}

Node.js v22.14.0

@ShutdownRepo ShutdownRepo merged commit 2ee2c87 into ThePorgs:main Sep 19, 2025
@skyf0l skyf0l deleted the fix/deps branch September 22, 2025 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants