Deduplicate transient cleanup logic and add edge-case tests

cleanupFileInformationCache() now delegates to invalidateForFile()
instead of duplicating the same transient-check logic. Add tests for
invalidateForFile() with no-target entries and missing keys.

Author: cderv

src/project/project-shared.ts

@@ -707,16 +707,9 @@ export class FileInformationCacheMap extends Map<string, FileInformation>
 }
 
 export function cleanupFileInformationCache(project: ProjectContext) {
-  project.fileInformationCache.forEach((entry) => {
-    if (entry?.target?.data) {
-      const data = entry.target.data as {
-        transient?: boolean;
-      };
-      if (data.transient && entry.target?.input) {
-        safeRemoveSync(entry.target?.input);
-      }
-    }
-  });
+  for (const key of [...project.fileInformationCache.keys()]) {
+    project.fileInformationCache.invalidateForFile(key);
+  }
 }
 
 export async function withProjectCleanup<T>(

tests/unit/project/file-information-cache.test.ts

@@ -207,3 +207,42 @@ unitTest(
     );
   },
 );
+
+// deno-lint-ignore require-await
+unitTest(
+  "fileInformationCache - invalidateForFile handles entry with no target",
+  async () => {
+    const project = createMockProjectContext();
+    const sourcePath = join(project.dir, "doc.qmd");
+
+    // Populate cache entry with metadata only (no target)
+    const entry = ensureFileInformationCache(project, sourcePath);
+    entry.metadata = { title: "Test" };
+
+    // Should not throw
+    project.fileInformationCache.invalidateForFile(sourcePath);
+
+    assert(
+      !project.fileInformationCache.has(sourcePath),
+      "Cache entry should be removed even without a target",
+    );
+  },
+);
+
+// deno-lint-ignore require-await
+unitTest(
+  "fileInformationCache - invalidateForFile is a no-op for missing keys",
+  async () => {
+    const project = createMockProjectContext();
+
+    // Should not throw on a key that doesn't exist
+    project.fileInformationCache.invalidateForFile(
+      join(project.dir, "nonexistent.qmd"),
+    );
+
+    assert(
+      project.fileInformationCache.size === 0,
+      "Cache should remain empty",
+    );
+  },
+);