Skip to content

Conversation

@ZJvandeWeg
Copy link
Member

When building docs, there was a deprecation warning, that was fixed following: https://vite.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated

Related Issue(s)

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

Labels

  • Includes a DB migration? -> add the area:migration label

When building docs, there was a deprecation warning, that was fixed
following: https://vite.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated
@ZJvandeWeg ZJvandeWeg requested a review from joepavitt June 24, 2025 08:05
@joepavitt
Copy link
Contributor

@ZJvandeWeg linting failing as require is no longer available in the NR-side of Dashboard, similarly, Dashboard no longer builds.

Copy link
Contributor

@joepavitt joepavitt left a comment

Choose a reason for hiding this comment

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

Details above

After converting to ES modules, the nodes directory still uses CommonJS
but ESLint was no longer recognizing require, module, __dirname globals.
Added these globals to the nodes ESLint override configuration.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
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