@oss-autopilot/core 1.6.1 → 1.6.3

package/dist/commands/startup.js

@@ -139,6 +139,31 @@ export function openInBrowser(url) {
         }
     });
 }
+/**
+ * Trigger a data refresh on a running dashboard server.
+ * Hits POST /api/refresh so the SPA picks up fresh data on its next poll.
+ * Non-fatal: errors are logged but don't propagate (#830).
+ */
+async function triggerDashboardRefresh(port) {
+    try {
+        const res = await fetch(`http://127.0.0.1:${port}/api/refresh`, {
+            method: 'POST',
+            headers: { Origin: `http://oss.localhost:${port}` },
+            signal: AbortSignal.timeout(5000),
+        });
+        if (!res.ok) {
+            const body = await res.text().catch(() => '');
+            console.error(`[STARTUP] Dashboard refresh returned ${res.status}: ${body}`);
+            return false;
+        }
+        await res.text().catch(() => { });
+        return true;
+    }
+    catch (error) {
+        console.error(`[STARTUP] Could not trigger dashboard refresh: ${errorMessage(error)}`);
+        return false;
+    }
+}
 /**
  * Run startup checks and return structured output.
  * Returns StartupOutput with one of three shapes:
@@ -184,14 +209,20 @@ export async function runStartup() {
     // 4. Launch interactive SPA dashboard
     // Skip opening on first run (0 PRs) — the welcome flow handles onboarding
     let dashboardUrl;
-    let dashboardOpened = false;
+    let dashboardStatus;
     if (daily.digest.summary.totalActivePRs > 0) {
         try {
             const spaResult = await launchDashboardServer();
             if (spaResult) {
                 dashboardUrl = spaResult.url;
-                openInBrowser(spaResult.url);
-                dashboardOpened = true;
+                if (spaResult.alreadyRunning) {
+                    const refreshed = await triggerDashboardRefresh(spaResult.port);
+                    dashboardStatus = refreshed ? 'refreshed' : 'running';
+                }
+                else {
+                    openInBrowser(spaResult.url);
+                    dashboardStatus = 'opened';
+                }
             }
             else {
                 console.error('[STARTUP] Dashboard SPA assets not found. Build with: cd packages/dashboard && pnpm run build');
@@ -201,10 +232,16 @@ export async function runStartup() {
             console.error('[STARTUP] SPA dashboard launch failed:', errorMessage(error));
         }
     }
-    // Append dashboard status to brief summary (only startup opens the browser, not daily)
-    if (dashboardOpened) {
+    // Append dashboard status to brief summary
+    if (dashboardStatus === 'opened') {
         daily.briefSummary += ' | Dashboard opened in browser';
     }
+    else if (dashboardStatus === 'refreshed') {
+        daily.briefSummary += ' | Dashboard refreshed';
+    }
+    else if (dashboardStatus === 'running') {
+        daily.briefSummary += ' | Dashboard running';
+    }
     // 5. Detect issue list
     const issueList = detectIssueList();
     return {

package/dist/core/review-analysis.d.ts

@@ -47,6 +47,12 @@ export declare function isAllSelfReplies(reviewId: number, reviewComments: Revie
  * synthetic placeholders (#199).
  */
 export declare function getInlineCommentBody(reviewId: number, reviewComments: ReviewComment[]): string | undefined;
+/**
+ * Check whether a review has any associated inline comments.
+ * Used to prevent the acknowledgment filter from discarding reviews
+ * that also posted inline comments (#829).
+ */
+export declare function reviewHasInlineComments(reviewId: number, reviewComments: ReviewComment[]): boolean;
 /**
  * Check if there are unresponded comments from maintainers.
  * Combines issue comments and review comments into a timeline,

package/dist/core/review-analysis.js

@@ -89,6 +89,14 @@ export function isAllSelfReplies(reviewId, reviewComments) {
 export function getInlineCommentBody(reviewId, reviewComments) {
     return reviewComments.find((c) => c.pull_request_review_id === reviewId && c.body?.trim())?.body?.trim();
 }
+/**
+ * Check whether a review has any associated inline comments.
+ * Used to prevent the acknowledgment filter from discarding reviews
+ * that also posted inline comments (#829).
+ */
+export function reviewHasInlineComments(reviewId, reviewComments) {
+    return reviewComments.some((c) => c.pull_request_review_id === reviewId);
+}
 /**
  * Check if there are unresponded comments from maintainers.
  * Combines issue comments and review comments into a timeline,
@@ -136,6 +144,7 @@ export function checkUnrespondedComments(comments, reviews, reviewComments, user
             body: resolvedBody,
             createdAt: review.submitted_at,
             isUser: author.toLowerCase() === usernameLower,
+            reviewId: review.id,
         });
     }
     // Sort by date
@@ -149,6 +158,7 @@ export function checkUnrespondedComments(comments, reviews, reviewComments, user
     }
     // Find maintainer comments after the user's last comment
     let lastMaintainerComment;
+    let lastMaintainerReviewId;
     for (const item of timeline) {
         if (item.isUser)
             continue; // Skip user's own comments
@@ -163,10 +173,14 @@ export function checkUnrespondedComments(comments, reviews, reviewComments, user
                 body: item.body.slice(0, 200) + (item.body.length > 200 ? '...' : ''),
                 createdAt: item.createdAt,
             };
+            lastMaintainerReviewId = item.reviewId;
         }
     }
-    // Filter out pure acknowledgment comments that don't require a response
-    if (lastMaintainerComment && isAcknowledgmentComment(lastMaintainerComment.body)) {
+    // Filter out pure acknowledgment comments that don't require a response.
+    // Skip the filter when the review has inline comments — those indicate
+    // feedback beyond the summary body that likely needs a response (#829).
+    const lastReviewHasInlineComments = lastMaintainerReviewId != null && reviewHasInlineComments(lastMaintainerReviewId, reviewComments);
+    if (lastMaintainerComment && isAcknowledgmentComment(lastMaintainerComment.body) && !lastReviewHasInlineComments) {
         lastMaintainerComment = undefined;
     }
     return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oss-autopilot/core",
-  "version": "1.6.1",
+  "version": "1.6.3",
   "description": "CLI and core library for managing open source contributions",
   "type": "module",
   "bin": {