Remove unused npm watch scripts#41584

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Draft

julien-deramond wants to merge1 commit intomain

base:main

Choose a base branch

frommain-jd-remove-unused-npm-watch-scripts

Draft

Remove unused npm watch scripts#41584

julien-deramond wants to merge1 commit intomainfrommain-jd-remove-unused-npm-watch-scripts

Conversation

Copy link

Member

julien-deramond commentedJul 2, 2025•
edited
Loading

Warning

Heavily draft

This PR follows up on#41574, which introduced automatic Sass recompilation for Astro.

The goal is to identify and removewatch* npm scripts that are no longer necessary.

Currently, these scripts are only triggered through the mainwatch script, which runs when executingnpm run start. There are no known use cases where thewatch-* scripts are run directly.

We'll evaluate them one by one.

🟢`watch-js-docs`

Steps:

Remove thewatch-js-docs npm script.
Runnpm run start.
Editsite/src/assets/stackblitz.js (e.g., change a comment in the HTML).
Confirm that Astro detects the change and recompiles automatically.
Click the “Try it on Stackblitz” button — the updated comment appears as expected.

Conclusion:watch-js-docs is no longer needed and can safely be removed.

TODO