Skip and log singleton corrupted project records

Author: alex-rawlings-yyc

src/__tests__/services/projectStorage.test.ts

@@ -491,12 +491,18 @@ describe('projectStorage', () => {
       await expect(listProjects(token)).rejects.toThrow(SyntaxError);
     });
 
-    it('propagates a JSON parse error from getProject as a corrupt-record signal', async () => {
+    it('skips a project whose storage value is corrupt JSON and logs the error', async () => {
       __mockReadUserData
         .mockResolvedValueOnce(JSON.stringify(['abc']))
         .mockResolvedValueOnce('not valid json');
 
-      await expect(listProjects(token)).rejects.toThrow(SyntaxError);
+      const result = await listProjects(token);
+
+      expect(result).toEqual([]);
+      expect(__mockLogger.error).toHaveBeenCalledWith(
+        expect.stringContaining('abc'),
+        expect.any(SyntaxError),
+      );
     });
   });
 });

src/services/projectStorage.ts

@@ -175,16 +175,26 @@ export async function getProject(
 
 /**
  * Returns all stored projects in creation order. Projects whose storage keys are missing (e.g.
- * after a failed delete) are silently omitted.
+ * after a failed delete) are silently omitted. Projects that fail to read or parse are logged and
+ * skipped so a single corrupted record does not prevent access to the rest.
  *
  * @param token - The execution token for storage access.
  * @returns All stored projects, ordered by creation time.
- * @throws {SyntaxError} If `projectIds` or any project's storage value contains invalid JSON.
- * @throws If `papi.storage.readUserData` rejects for any non-ENOENT reason.
+ * @throws {SyntaxError} If `projectIds` contains invalid JSON.
+ * @throws If `papi.storage.readUserData` rejects for any non-ENOENT reason when reading the index.
  */
 export async function listProjects(token: ExecutionToken): Promise<InterlinearProject[]> {
   const ids = await readIds(token);
-  const projects = await Promise.all(ids.map((id) => getProject(token, id)));
+  const projects = await Promise.all(
+    ids.map(async (id) => {
+      try {
+        return await getProject(token, id);
+      } catch (e) {
+        logger.error(`Interlinearizer: failed to read project ${id}:`, e);
+        return undefined;
+      }
+    }),
+  );
   return projects.filter((p): p is InterlinearProject => p !== undefined);
 }