file-entry-cache - changed does not get set on meta.data changes (#906)

jaredwray · Nov 24, 2024 · 7d34f45 · 7d34f45
1 parent e7c87b2
commit 7d34f45
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 6 deletions.
diff --git a/packages/file-entry-cache/package.json b/packages/file-entry-cache/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "file-entry-cache",
-	"version": "10.0.3",
+	"version": "10.0.4",
 	"description": "A lightweight cache for file metadata, ideal for processes that work on a specific set of files and only need to reprocess files that have changed since the last run",
 	"type": "module",
 	"main": "./dist/index.cjs",

diff --git a/packages/file-entry-cache/src/index.ts b/packages/file-entry-cache/src/index.ts
@@ -290,10 +290,6 @@ export class FileEntryCache {
 			return result;
 		}
 
-		if (result.meta.data !== metaCache.data) {
-			result.changed = true;
-		}
-
 		// Set the data from the cache
 		if (result.meta.data === undefined) {
 			result.meta.data = metaCache.data;

diff --git a/packages/file-entry-cache/test/index.test.ts b/packages/file-entry-cache/test/index.test.ts
@@ -188,7 +188,7 @@ describe('getFileDescriptor()', () => {
 		const fileDescriptor4 = fileEntryCache3.getFileDescriptor(testFile1);
 		expect(fileDescriptor4).toBeDefined();
 		expect(fileDescriptor4.meta.data).toEqual(data2);
-		expect(fileDescriptor4.changed).toEqual(true);
+		expect(fileDescriptor4.changed).toEqual(false);
 	});
 
 	test('should return a file descriptor', () => {