@haystackeditor/cli 0.15.6 → 0.15.7

package/dist/commands/setup.js

@@ -56,6 +56,10 @@ function buildAnswerMap(options) {
             answers.upgrade_entire = true;
         if (!('install_hooks' in answers))
             answers.install_hooks = true;
+        // Don't auto-launch a browser in an unattended --yes run; the feed URL is
+        // still printed and returned in the result for the caller to open.
+        if (!('open_feed' in answers))
+            answers.open_feed = false;
     }
     return answers;
 }
@@ -1120,11 +1124,23 @@ async function runSetupFlow(options) {
     else {
         await stepInstallEntire(selectedRepos, token, bootstrapPRs);
     }
+    // Close the loop: point the user to their Haystack feed (the inbox/card stack
+    // where their PRs surface). Interactive users get offered a browser open; an
+    // agent gets the URL in the `result` event to surface/open as it sees fit.
+    const feedUrl = `${HAYSTACK_API}/inbox`;
+    console.log(chalk.bold('\n  Your Haystack feed:'));
+    console.log(`  ${chalk.cyan(feedUrl)}\n`);
+    if (ui.interactive) {
+        const openFeed = await ui.confirm({ id: 'open_feed', message: 'Open your Haystack feed now?', default: true });
+        if (openFeed)
+            tryOpenBrowser(feedUrl);
+    }
     // Final structured outcome for non-interactive/agent callers.
     ui.result({
         status: 'complete',
         repos: selectedRepos,
         pullRequests: [...bootstrapPRs.entries()].map(([repo, pr]) => ({ repo, url: pr.prUrl })),
+        feedUrl,
     });
 }
 // =============================================================================

package/dist/commands/triage.js

@@ -68,20 +68,54 @@ function parsePRIdentifier(identifier) {
 // ============================================================================
 // Output formatting
 // ============================================================================
+/**
+ * Decide whether a finding is one the auto-fixer is already handling (and thus
+ * shouldn't be re-fixed). The fixer re-derives — often paraphrases — its manifest
+ * summaries rather than copying them verbatim, so exact string matching misses
+ * them. This mirrors the normalized + fuzzy matching the backend uses in
+ * agent/cloudflare/src/auto-fix-post-synthesis.ts (normalizeText / categoriesCompatible).
+ */
+function normalizeText(value) {
+    return value.toLowerCase().replace(/\s+/g, ' ').trim();
+}
+/** Categories match if equal, or if one's tokens are a subset of the other's. */
+function categoriesCompatible(left, right) {
+    const a = normalizeText(left);
+    const b = normalizeText(right);
+    if (a === b)
+        return true;
+    const aTokens = a.split(/[^a-z0-9]+/).filter(Boolean);
+    const bTokens = b.split(/[^a-z0-9]+/).filter(Boolean);
+    const aSet = new Set(aTokens);
+    const bSet = new Set(bTokens);
+    return ((bTokens.length > 0 && bTokens.every((t) => aSet.has(t))) ||
+        (aTokens.length > 0 && aTokens.every((t) => bSet.has(t))));
+}
+/** True if the finding matches any item the auto-fixer is handling. */
+function isAutoFixing(finding, manifest) {
+    const items = manifest?.fixedItems ?? [];
+    if (items.length === 0)
+        return false;
+    const summaryKey = normalizeText(finding.summary);
+    if (summaryKey.length === 0)
+        return false;
+    return items.some((item) => normalizeText(item.summary) === summaryKey &&
+        categoriesCompatible(finding.category, item.category));
+}
 function printRichOutput(pr, synthesis, manifest) {
     const reviewUrl = `https://haystackeditor.com/review/${pr.owner}/${pr.repo}/${pr.prNumber}`;
     console.log(`\n  ${chalk.bold(`Haystack Triage: ${pr.owner}/${pr.repo}#${pr.prNumber}`)}\n`);
     // Auto-fixer status — what's being fixed vs what it skipped
     if (manifest && (manifest.fixedItems.length > 0 || manifest.leftForReview.length > 0)) {
         if (manifest.fixedItems.length > 0) {
-            console.log(chalk.bold('  Auto-fixer is handling:\n'));
+            console.log(chalk.bold('  Auto-fixer is handling these (leave them alone):\n'));
             for (const item of manifest.fixedItems) {
                 console.log(`    - [${item.category}] ${item.summary}`);
             }
-            console.log('');
+            console.log(`  ${chalk.dim("Don't fix these. The auto-fixer is already on them and will push its own commit.")}\n`);
         }
         if (manifest.leftForReview.length > 0) {
-            console.log(chalk.bold('  Auto-fixer skipped:\n'));
+            console.log(chalk.bold('  Auto-fixer skipped these (they need you):\n'));
             for (const item of manifest.leftForReview) {
                 console.log(`    - [${item.category}] ${item.summary}`);
                 console.log(`      ${chalk.dim(item.reason)}`);
@@ -89,10 +123,13 @@ function printRichOutput(pr, synthesis, manifest) {
             console.log('');
         }
     }
-    // Findings — the actionable output for agents
-    const findings = synthesis.synthesisDisplay || [];
+    // Findings — the actionable output for agents. Exclude anything the auto-fixer
+    // is already handling so the agent doesn't duplicate (and fight) its work.
+    const allFindings = synthesis.synthesisDisplay || [];
+    const findings = allFindings.filter((f) => !isAutoFixing(f, manifest));
+    const fixingCount = manifest?.fixedItems?.length ?? 0;
     if (findings.length > 0) {
-        console.log(chalk.bold('  Findings:\n'));
+        console.log(chalk.bold('  Findings to address:\n'));
         for (let i = 0; i < findings.length; i++) {
             const finding = findings[i];
             const sourceTag = finding.source ? ` (${finding.source})` : '';
@@ -112,7 +149,11 @@ function printRichOutput(pr, synthesis, manifest) {
             console.log('');
         }
     }
-    else if (!manifest || manifest.fixedItems.length === 0) {
+    else if (fixingCount > 0 && allFindings.length > 0) {
+        // Every finding is being auto-fixed — nothing left for the agent.
+        console.log(chalk.dim('  Nothing to address. Every finding is being auto-fixed (see above).\n'));
+    }
+    else {
         console.log(chalk.dim('  No findings.\n'));
     }
     console.log(`  ${chalk.dim(reviewUrl)}\n`);
@@ -123,6 +164,7 @@ function printJsonOutput(pr, synthesis, manifest) {
         repo: pr.repo,
         prNumber: pr.prNumber,
         rating: synthesis.haystackRating,
+        // Issues the auto-fixer is already on — leave these alone, do not re-fix.
         autoFixerHandling: manifest?.fixedItems ?? [],
         autoFixerSkipped: manifest?.leftForReview ?? [],
         findings: (synthesis.synthesisDisplay || []).map((f) => ({
@@ -131,6 +173,8 @@ function printJsonOutput(pr, synthesis, manifest) {
             detail: f.detail,
             agentFixPrompt: f.agentFixPrompt || null,
             source: f.source || null,
+            // True when the auto-fixer is handling this finding — don't fix it yourself.
+            autoFixing: isAutoFixing(f, manifest),
         })),
     };
     console.log(JSON.stringify(output, null, 2));
@@ -159,16 +203,19 @@ function printHookOutput(pr, synthesis, manifest, status) {
         console.log(`[Haystack] ${label}: Analysis in progress...`);
         return;
     }
-    const findings = synthesis.synthesisDisplay || [];
     const stars = ratingStars(synthesis.haystackRating);
     const fixingCount = manifest?.fixedItems?.length ?? 0;
-    if (findings.length === 0 && fixingCount === 0) {
+    // Only count findings the auto-fixer is NOT handling \u2014 those are what need the user.
+    const actionable = (synthesis.synthesisDisplay || []).filter((f) => !isAutoFixing(f, manifest));
+    if (actionable.length === 0 && fixingCount === 0) {
         console.log(`[Haystack] \u2705 ${label} ${stars}: Good to merge`);
     }
+    else if (actionable.length === 0) {
+        // Everything is being auto-fixed \u2014 nothing for the user to do.
+        console.log(`[Haystack] \u2705 ${label} ${stars}: ${fixingCount} being auto-fixed, nothing for you to do`);
+    }
     else {
-        const parts = [];
-        if (findings.length > 0)
-            parts.push(`${findings.length} finding(s)`);
+        const parts = [`${actionable.length} finding(s)`];
         if (fixingCount > 0)
             parts.push(`${fixingCount} being auto-fixed`);
         console.log(`[Haystack] \u26A0\uFE0F  ${label} ${stars}: Needs your input (${parts.join(', ')})`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@haystackeditor/cli",
-  "version": "0.15.6",
+  "version": "0.15.7",
   "description": "Set up Haystack for your project — automated PR review, triage, and merge queue",
   "type": "module",
   "bin": {