pierre-review 0.1.21 → 0.1.22

package/dist/db/queries.js

@@ -323,21 +323,40 @@ export async function getTimeline(filters) {
         for (const r of rows)
             reviewStateById.set(r.id, r.state);
     }
-    const timelineEvents = evRows.map((e) => ({
-        id: e.id,
-        repoId: e.repoId,
-        actorId: e.actorId,
-        prId: e.prId,
-        type: e.type,
-        occurredAt: e.occurredAt.toISOString(),
-        threadId: e.type === 'review_comment' && e.refTable === 'review_threads'
-            ? e.refId
-            : null,
-        refId: e.refId,
-        reviewState: e.type === 'review_submitted' && e.refTable === 'reviews' && e.refId != null
-            ? (reviewStateById.get(e.refId) ?? null)
-            : null,
-    }));
+    // Batch-load the derived state of each review_comment event's thread, so the
+    // timeline's "Threads" filter can narrow markers to a specific thread state
+    // (e.g. only resolved) instead of every comment on a PR that has a matching
+    // thread. A single keyed lookup; the thread's id is the event's refId.
+    const threadRefIds = evRows
+        .filter((e) => e.type === 'review_comment' && e.refTable === 'review_threads' && e.refId != null)
+        .map((e) => e.refId);
+    const threadStateById = new Map();
+    if (threadRefIds.length > 0) {
+        const rows = await db
+            .select({ id: reviewThreads.id, state: reviewThreads.derivedState })
+            .from(reviewThreads)
+            .where(inArray(reviewThreads.id, threadRefIds))
+            .execute();
+        for (const r of rows)
+            threadStateById.set(r.id, r.state);
+    }
+    const timelineEvents = evRows.map((e) => {
+        const threadId = e.type === 'review_comment' && e.refTable === 'review_threads' ? e.refId : null;
+        return {
+            id: e.id,
+            repoId: e.repoId,
+            actorId: e.actorId,
+            prId: e.prId,
+            type: e.type,
+            occurredAt: e.occurredAt.toISOString(),
+            threadId,
+            derivedState: threadId != null ? (threadStateById.get(threadId) ?? null) : null,
+            refId: e.refId,
+            reviewState: e.type === 'review_submitted' && e.refTable === 'reviews' && e.refId != null
+                ? (reviewStateById.get(e.refId) ?? null)
+                : null,
+        };
+    });
     return { prs, events: timelineEvents };
 }
 export async function getOpenPrs(filters) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pierre-review",
-  "version": "0.1.21",
+  "version": "0.1.22",
   "description": "Dashboard for tracking your team's GitHub PR activity across repos — local (SQLite + gh) or self-hosted multi-tenant cloud (Postgres + GitHub App).",
   "type": "module",
   "author": "Alex Wakeman",