@in-the-loop-labs/pair-review 2.6.3 → 2.7.0

package/src/setup/pr-setup.js

@@ -18,6 +18,7 @@ const { normalizeRepository } = require('../utils/paths');
 const { findMainGitRoot } = require('../local-review');
 const { getConfigDir, getMonorepoPath, resolveMonorepoOptions, DEFAULT_CHECKOUT_TIMEOUT_MS } = require('../config');
 const logger = require('../utils/logger');
+const { fireReviewStartedHook } = require('../hooks/payloads');
 const simpleGit = require('simple-git');
 const fs = require('fs').promises;
 const path = require('path');
@@ -70,13 +71,15 @@ async function storePRData(db, prInfo, prData, diff, changedFiles, worktreePath,
       SELECT id FROM pr_metadata WHERE pr_number = ? AND repository = ? COLLATE NOCASE
     `, [prInfo.number, repository]);
 
+    const now = new Date().toISOString();
+
     if (existingPR) {
       // Update existing PR metadata (preserves ID)
       await run(db, `
         UPDATE pr_metadata
         SET title = ?, description = ?, author = ?,
             base_branch = ?, head_branch = ?, pr_data = ?,
-            updated_at = CURRENT_TIMESTAMP
+            updated_at = CURRENT_TIMESTAMP, last_accessed_at = ?
         WHERE id = ?
       `, [
         prData.title,
@@ -85,6 +88,7 @@ async function storePRData(db, prInfo, prData, diff, changedFiles, worktreePath,
         prData.base_branch,
         prData.head_branch,
         JSON.stringify(extendedPRData),
+        now,
         existingPR.id
       ]);
       logger.info(`Updated existing PR metadata (ID: ${existingPR.id})`);
@@ -92,8 +96,8 @@ async function storePRData(db, prInfo, prData, diff, changedFiles, worktreePath,
       // Insert new PR metadata
       const result = await run(db, `
         INSERT INTO pr_metadata
-        (pr_number, repository, title, description, author, base_branch, head_branch, pr_data)
-        VALUES (?, ?, ?, ?, ?, ?, ?, ?)
+        (pr_number, repository, title, description, author, base_branch, head_branch, pr_data, last_accessed_at)
+        VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)
       `, [
         prInfo.number,
         repository,
@@ -102,7 +106,8 @@ async function storePRData(db, prInfo, prData, diff, changedFiles, worktreePath,
         prData.author,
         prData.base_branch,
         prData.head_branch,
-        JSON.stringify(extendedPRData)
+        JSON.stringify(extendedPRData),
+        now
       ]);
       logger.info(`Created new PR metadata (ID: ${result.lastID})`);
     }
@@ -115,6 +120,9 @@ async function storePRData(db, prInfo, prData, diff, changedFiles, worktreePath,
       SELECT id FROM reviews WHERE pr_number = ? AND repository = ? COLLATE NOCASE
     `, [prInfo.number, repository]);
 
+    let isNewReview;
+    let reviewId;
+
     if (existingReview) {
       // Update existing review (preserves ID)
       await run(db, `
@@ -128,6 +136,8 @@ async function storePRData(db, prInfo, prData, diff, changedFiles, worktreePath,
         }),
         existingReview.id
       ]);
+      isNewReview = false;
+      reviewId = existingReview.id;
       logger.info(`Updated existing review (ID: ${existingReview.id})`);
     } else {
       // Insert new review
@@ -143,6 +153,8 @@ async function storePRData(db, prInfo, prData, diff, changedFiles, worktreePath,
           created_at: new Date().toISOString()
         })
       ]);
+      isNewReview = true;
+      reviewId = result.lastID;
       logger.info(`Created new review (ID: ${result.lastID})`);
     }
 
@@ -150,6 +162,8 @@ async function storePRData(db, prInfo, prData, diff, changedFiles, worktreePath,
     await run(db, 'COMMIT');
     logger.info(`Stored PR data for ${repository} #${prInfo.number}`);
 
+    return { isNewReview, reviewId };
+
   } catch (error) {
     // Rollback transaction on error
     await run(db, 'ROLLBACK');
@@ -465,7 +479,7 @@ async function setupPRReview({ db, owner, repo, prNumber, githubToken, config, o
   // Step: store - Persist PR data and register repository location
   // ------------------------------------------------------------------
   progress({ step: 'store', status: 'running', message: 'Storing pull request data...' });
-  await storePRData(db, prInfo, prData, diff, changedFiles, worktreePath);
+  const { isNewReview, reviewId } = await storePRData(db, prInfo, prData, diff, changedFiles, worktreePath);
 
   // Register the repository path for future sessions if it wasn't already known
   if (knownPath === null && repositoryPath) {
@@ -478,6 +492,15 @@ async function setupPRReview({ db, owner, repo, prNumber, githubToken, config, o
   }
   progress({ step: 'store', status: 'completed', message: 'Pull request data stored.' });
 
+  // Fire review.started hook for new reviews (non-blocking).
+  // The GET route fires review.loaded on page load; firing review.started
+  // here ensures the first-time event isn't lost because storePRData already
+  // created the review record before the GET route's getOrCreate runs.
+  if (isNewReview) {
+    fireReviewStartedHook({ reviewId, prNumber, owner, repo, prData, config })
+      .catch(err => { logger.warn(`Review hook failed: ${err.message}`); });
+  }
+
   // ------------------------------------------------------------------
   // Return the review URL and title for the caller
   // ------------------------------------------------------------------