Skip to content

Fix ignored paths in webpack watch options#7384

Open
Steve-Mcl wants to merge 2 commits into
mainfrom
7383-dx-syntax-error-in-watcher
Open

Fix ignored paths in webpack watch options#7384
Steve-Mcl wants to merge 2 commits into
mainfrom
7383-dx-syntax-error-in-watcher

Conversation

@Steve-Mcl
Copy link
Copy Markdown
Contributor

Description

This pull request makes a minor adjustment to the watchOptions.ignored configuration in the webpack.config.js file to fix the terminal error witnessed on windows dev time CLI (see the related issue OP in #7383 for details)

  • Changed the ignored file pattern from getPath('frontend/dist/**') to a glob string '**/frontend/dist/**' in the watchOptions.ignored array to improve reliability and consistency.

Note

Untested on *nix

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
  • Link to Changelog Entry PR, or note why one is not needed.

Labels

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

@Steve-Mcl Steve-Mcl linked an issue Jun 5, 2026 that may be closed by this pull request
@Steve-Mcl Steve-Mcl requested a review from cstns June 5, 2026 12:36
Copy link
Copy Markdown
Contributor

@hardillb hardillb left a comment

Choose a reason for hiding this comment

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

Assuming you've tested this 😉

@Steve-Mcl
Copy link
Copy Markdown
Contributor Author

Assuming you've tested this 😉

Yup. But only on windows. The terminal error desists and the watcher rebuilds correctly

Just need someone on MacOs to test (but I am certain it will be fine if it is on linux)

@n-lark - perhaps you could check this (assuming you are on MacOS)

@n-lark
Copy link
Copy Markdown
Contributor

n-lark commented Jun 5, 2026

Hey! Confirmed on macOS. Pulled the branch, ran the watcher, edited a frontend file and it rebuilt cleanly with no error and no runaway loop. Works great here in the land of the superior OS 🍏✅

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.

DX: Syntax error in watcher

3 participants