@ryanatkn/gro 0.136.1 → 0.137.0

package/dist/filer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"filer.d.ts","sourceRoot":"../src/lib/","sources":["filer.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAC,WAAW,EAAC,MAAM,yBAAyB,CAAC;AAGzD,OAAO,KAAK,EAAC,OAAO,EAAC,MAAM,WAAW,CAAC;AACvC,OAAO,EACN,SAAS,EAET,KAAK,cAAc,EACnB,KAAK,OAAO,IAAI,iBAAiB,EAEjC,MAAM,gBAAgB,CAAC;AAUxB,MAAM,WAAW,WAAW;IAC3B,EAAE,EAAE,OAAO,CAAC;IACZ;;;OAGG;IACH,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,GAAG,CAAC,OAAO,EAAE,WAAW,CAAC,CAAC;IACtC,YAAY,EAAE,GAAG,CAAC,OAAO,EAAE,WAAW,CAAC,CAAC;CACxC;AAED,MAAM,MAAM,aAAa,GAAG,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;AAEhD,MAAM,MAAM,eAAe,GAAG,CAAC,MAAM,EAAE,cAAc,EAAE,WAAW,EAAE,WAAW,KAAK,IAAI,CAAC;AAEzF,MAAM,WAAW,OAAO;IACvB,SAAS,CAAC,EAAE,OAAO,SAAS,CAAC;IAC7B,iBAAiB,CAAC,EAAE,OAAO,CAAC,WAAW,CAAC,iBAAiB,EAAE,WAAW,CAAC,CAAC,CAAC;CACzE;AAED,qBAAa,KAAK;;IACjB,KAAK,EAAE,GAAG,CAAC,OAAO,EAAE,WAAW,CAAC,CAAa;gBAKjC,OAAO,GAAE,OAAsB;IAU3C,SAAS,OAAQ,OAAO,KAAG,WAAW,GAAG,SAAS,CAEhD;IAEF,aAAa,OAAQ,OAAO,KAAG,WAAW,CAWxC;IA4HI,KAAK,CAAC,QAAQ,EAAE,eAAe,GAAG,OAAO,CAAC,aAAa,CAAC;IAKxD,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;CAQ5B"}
+{"version":3,"file":"filer.d.ts","sourceRoot":"../src/lib/","sources":["filer.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAC,WAAW,EAAC,MAAM,yBAAyB,CAAC;AAGzD,OAAO,KAAK,EAAC,OAAO,EAAC,MAAM,WAAW,CAAC;AACvC,OAAO,EACN,SAAS,EAET,KAAK,cAAc,EACnB,KAAK,OAAO,IAAI,iBAAiB,EAEjC,MAAM,gBAAgB,CAAC;AAUxB,MAAM,WAAW,WAAW;IAC3B,EAAE,EAAE,OAAO,CAAC;IACZ;;;OAGG;IACH,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,GAAG,CAAC,OAAO,EAAE,WAAW,CAAC,CAAC;IACtC,YAAY,EAAE,GAAG,CAAC,OAAO,EAAE,WAAW,CAAC,CAAC;CACxC;AAED,MAAM,MAAM,aAAa,GAAG,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;AAEhD,MAAM,MAAM,eAAe,GAAG,CAAC,MAAM,EAAE,cAAc,EAAE,WAAW,EAAE,WAAW,KAAK,IAAI,CAAC;AAEzF,MAAM,WAAW,OAAO;IACvB,SAAS,CAAC,EAAE,OAAO,SAAS,CAAC;IAC7B,iBAAiB,CAAC,EAAE,OAAO,CAAC,WAAW,CAAC,iBAAiB,EAAE,WAAW,CAAC,CAAC,CAAC;CACzE;AAED,qBAAa,KAAK;;IACjB,KAAK,EAAE,GAAG,CAAC,OAAO,EAAE,WAAW,CAAC,CAAa;gBAKjC,OAAO,GAAE,OAAsB;IAU3C,SAAS,OAAQ,OAAO,KAAG,WAAW,GAAG,SAAS,CAEhD;IAEF,aAAa,OAAQ,OAAO,KAAG,WAAW,CAWxC;IA6HI,KAAK,CAAC,QAAQ,EAAE,eAAe,GAAG,OAAO,CAAC,aAAa,CAAC;IAKxD,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;CAQ5B"}

package/dist/filer.js

@@ -40,44 +40,44 @@ export class Filer {
     #update(id) {
         const file = this.get_or_create(id);
         const new_contents = existsSync(id) ? readFileSync(id, 'utf8') : null;
-        const contents_changed = file.contents !== new_contents;
+        if (file.contents === new_contents) {
+            return null;
+        }
         file.contents = new_contents;
-        if (contents_changed) {
-            const dir = dirname(file.id);
-            const dependencies_before = new Set(file.dependencies.keys());
-            const dependencies_removed = new Set(dependencies_before);
-            const imported = file.contents ? parse_imports(file.id, file.contents) : [];
-            for (const specifier of imported) {
-                // TODO logic is duplicated from loader
-                const path = map_sveltekit_aliases(specifier, aliases);
-                // The specifier `path` has now been mapped to its final form, so we can inspect it.
-                if (path[0] === '.' || path[0] === '/') {
-                    const { path_id } = resolve_specifier(path, dir);
-                    dependencies_removed.delete(path_id);
-                    if (!dependencies_before.has(path_id)) {
-                        const d = this.get_or_create(path_id);
-                        file.dependencies.set(d.id, d);
-                        d.dependents.set(file.id, file);
-                    }
+        const dir = dirname(file.id);
+        const dependencies_before = new Set(file.dependencies.keys());
+        const dependencies_removed = new Set(dependencies_before);
+        const imported = file.contents ? parse_imports(file.id, file.contents) : [];
+        for (const specifier of imported) {
+            // TODO logic is duplicated from loader
+            const path = map_sveltekit_aliases(specifier, aliases);
+            // The specifier `path` has now been mapped to its final form, so we can inspect it.
+            if (path[0] === '.' || path[0] === '/') {
+                const { path_id } = resolve_specifier(path, dir);
+                dependencies_removed.delete(path_id);
+                if (!dependencies_before.has(path_id)) {
+                    const d = this.get_or_create(path_id);
+                    file.dependencies.set(d.id, d);
+                    d.dependents.set(file.id, file);
                 }
             }
-            // update any removed dependencies
-            for (const dependency_removed of dependencies_removed) {
-                const deleted1 = file.dependencies.delete(dependency_removed);
-                if (!deleted1)
-                    throw Error('expected to delete1 ' + file.id); // TODO @many delete if correct
-                const dependency_removed_file = this.get_or_create(dependency_removed);
-                const deleted2 = dependency_removed_file.dependents.delete(file.id);
-                if (!deleted2)
-                    throw Error('expected to delete2 ' + file.id); // TODO @many delete if correct
-            }
+        }
+        // update any removed dependencies
+        for (const dependency_removed of dependencies_removed) {
+            const deleted1 = file.dependencies.delete(dependency_removed);
+            if (!deleted1)
+                throw Error('expected to delete1 ' + file.id); // TODO @many delete if correct
+            const dependency_removed_file = this.get_or_create(dependency_removed);
+            const deleted2 = dependency_removed_file.dependents.delete(file.id);
+            if (!deleted2)
+                throw Error('expected to delete2 ' + file.id); // TODO @many delete if correct
         }
         return file;
     }
     #remove(id) {
         const file = this.get_by_id(id);
         if (!file)
-            return; // this is safe because the object would exist if any other file referenced it as a dependency or dependent
+            return null; // this is safe because the object would exist if any other file referenced it as a dependency or dependent
         file.contents = null; // clear contents in case it gets re-added later, we want the change to be detected
         let found = false;
         for (const d of this.files.values()) {

package/dist/package.js

@@ -1,7 +1,7 @@
 // generated by src/lib/package.gen.ts
 export const package_json = {
     name: '@ryanatkn/gro',
-    version: '0.136.1',
+    version: '0.137.0',
     description: 'task runner and toolkit extending SvelteKit',
     motto: 'generate, run, optimize',
     glyph: '🌰',
@@ -264,7 +264,7 @@ export const package_json = {
 };
 export const src_json = {
     name: '@ryanatkn/gro',
-    version: '0.136.1',
+    version: '0.137.0',
     modules: {
         '.': {
             path: 'index.ts',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ryanatkn/gro",
-  "version": "0.136.1",
+  "version": "0.137.0",
   "description": "task runner and toolkit extending SvelteKit",
   "motto": "generate, run, optimize",
   "glyph": "🌰",

package/src/lib/filer.ts

@@ -74,51 +74,52 @@ export class Filer {
 		return file;
 	};
 
-	#update(id: Path_Id): Source_File {
+	#update(id: Path_Id): Source_File | null {
 		const file = this.get_or_create(id);
 		const new_contents = existsSync(id) ? readFileSync(id, 'utf8') : null;
-		const contents_changed = file.contents !== new_contents;
+		if (file.contents === new_contents) {
+			return null;
+		}
+
 		file.contents = new_contents;
 
-		if (contents_changed) {
-			const dir = dirname(file.id);
-
-			const dependencies_before = new Set(file.dependencies.keys());
-			const dependencies_removed = new Set(dependencies_before);
-
-			const imported = file.contents ? parse_imports(file.id, file.contents) : [];
-			for (const specifier of imported) {
-				// TODO logic is duplicated from loader
-				const path = map_sveltekit_aliases(specifier, aliases);
-
-				// The specifier `path` has now been mapped to its final form, so we can inspect it.
-				if (path[0] === '.' || path[0] === '/') {
-					const {path_id} = resolve_specifier(path, dir);
-					dependencies_removed.delete(path_id);
-					if (!dependencies_before.has(path_id)) {
-						const d = this.get_or_create(path_id);
-						file.dependencies.set(d.id, d);
-						d.dependents.set(file.id, file);
-					}
+		const dir = dirname(file.id);
+
+		const dependencies_before = new Set(file.dependencies.keys());
+		const dependencies_removed = new Set(dependencies_before);
+
+		const imported = file.contents ? parse_imports(file.id, file.contents) : [];
+		for (const specifier of imported) {
+			// TODO logic is duplicated from loader
+			const path = map_sveltekit_aliases(specifier, aliases);
+
+			// The specifier `path` has now been mapped to its final form, so we can inspect it.
+			if (path[0] === '.' || path[0] === '/') {
+				const {path_id} = resolve_specifier(path, dir);
+				dependencies_removed.delete(path_id);
+				if (!dependencies_before.has(path_id)) {
+					const d = this.get_or_create(path_id);
+					file.dependencies.set(d.id, d);
+					d.dependents.set(file.id, file);
 				}
 			}
+		}
 
-			// update any removed dependencies
-			for (const dependency_removed of dependencies_removed) {
-				const deleted1 = file.dependencies.delete(dependency_removed);
-				if (!deleted1) throw Error('expected to delete1 ' + file.id); // TODO @many delete if correct
-				const dependency_removed_file = this.get_or_create(dependency_removed);
-				const deleted2 = dependency_removed_file.dependents.delete(file.id);
-				if (!deleted2) throw Error('expected to delete2 ' + file.id); // TODO @many delete if correct
-			}
+		// update any removed dependencies
+		for (const dependency_removed of dependencies_removed) {
+			const deleted1 = file.dependencies.delete(dependency_removed);
+			if (!deleted1) throw Error('expected to delete1 ' + file.id); // TODO @many delete if correct
+			const dependency_removed_file = this.get_or_create(dependency_removed);
+			const deleted2 = dependency_removed_file.dependents.delete(file.id);
+			if (!deleted2) throw Error('expected to delete2 ' + file.id); // TODO @many delete if correct
 		}
 
 		return file;
 	}
 
-	#remove(id: Path_Id): Source_File | undefined {
+	#remove(id: Path_Id): Source_File | null {
 		const file = this.get_by_id(id);
-		if (!file) return; // this is safe because the object would exist if any other file referenced it as a dependency or dependent
+		if (!file) return null; // this is safe because the object would exist if any other file referenced it as a dependency or dependent
 
 		file.contents = null; // clear contents in case it gets re-added later, we want the change to be detected
 
@@ -177,7 +178,7 @@ export class Filer {
 
 	#on_change: Watcher_Change_Callback = (change) => {
 		if (change.is_directory) return;
-		let source_file: Source_File | undefined;
+		let source_file: Source_File | null;
 		switch (change.type) {
 			case 'add':
 			case 'update': {

package/src/lib/package.ts

@@ -5,7 +5,7 @@ import type {Src_Json} from './src_json.js';
 
 export const package_json = {
 	name: '@ryanatkn/gro',
-	version: '0.136.1',
+	version: '0.137.0',
 	description: 'task runner and toolkit extending SvelteKit',
 	motto: 'generate, run, optimize',
 	glyph: '🌰',
@@ -270,7 +270,7 @@ export const package_json = {
 
 export const src_json = {
 	name: '@ryanatkn/gro',
-	version: '0.136.1',
+	version: '0.137.0',
 	modules: {
 		'.': {
 			path: 'index.ts',