sitevision-cli 1.0.0-beta.29 → 1.0.0-beta.30

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 1.0.0-beta.30
+
+- Dev and watch no longer rebuild on file events that change nothing, such as
+  editor metadata, swap files or `.DS_Store`. Ignored events are logged as
+  warnings so a stray trigger can be traced.
+- The log names the event and path of the file that started a rebuild, for
+  example `change src/index.tsx`.
+
 ## 1.0.0-beta.29
 
 - After an update, the shell opens What's new with the changes since the

package/dist/utils/tasks.js

@@ -342,10 +342,42 @@ export function startDev(project, options) {
             building = false;
         }
     };
-    const onChange = (name, file) => {
-        log('fs', `changed ${file ?? name}`);
+    // Size + mtime of every watched file. An event that leaves this unchanged
+    // (editor metadata, xattrs, identical re-save) is noise and must not
+    // rebuild, or a stray event during sign/deploy loops forever.
+    const snapshot = () => {
+        const entries = [];
+        for (const name of WATCH_TARGETS) {
+            const target = path.join(root, name);
+            const stat = fs.statSync(target, { throwIfNoEntry: false });
+            if (!stat)
+                continue;
+            const files = (stat.isDirectory()
+                ? fs
+                    .readdirSync(target, { recursive: true, withFileTypes: true })
+                    .filter(entry => entry.isFile())
+                    .map(entry => path.join(entry.parentPath, entry.name))
+                : [target]).filter(file => !path.basename(file).startsWith('.'));
+            for (const file of files) {
+                const info = fs.statSync(file, { throwIfNoEntry: false });
+                if (info)
+                    entries.push(`${file}:${info.size}:${info.mtimeMs}`);
+            }
+        }
+        return entries.toSorted().join('\n');
+    };
+    let fingerprint = '';
+    const onChange = (name, event, file) => {
+        const where = file ? path.join(name, file) : name;
         clearTimeout(debounce);
         debounce = setTimeout(() => {
+            const next = snapshot();
+            if (next === fingerprint) {
+                log('fs', `${event} ${where} · no file changed, ignored`, 'warn');
+                return;
+            }
+            fingerprint = next;
+            log('fs', `${event} ${where}`);
             void rebuild();
         }, 300);
     };
@@ -355,10 +387,11 @@ export function startDev(project, options) {
             if (!fs.existsSync(target))
                 continue;
             const isDir = fs.statSync(target).isDirectory();
-            watchers.push(fs.watch(target, { recursive: isDir }, (_event, file) => {
-                onChange(name, file);
+            watchers.push(fs.watch(target, { recursive: isDir }, (event, file) => {
+                onChange(name, event, file);
             }));
         }
+        fingerprint = snapshot();
         log('svc', `watching ${WATCH_TARGETS.join(', ')}`);
     };
     void (async () => {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "sitevision-cli",
-	"version": "1.0.0-beta.29",
+	"version": "1.0.0-beta.30",
 	"license": "MIT",
 	"bin": {
 		"svc": "dist/cli.js"