@shepai/cli 1.150.1-pr470.1a21b64 → 1.150.1-pr472.b429b42

package/dist/src/presentation/web/app/actions/get-merge-review-data.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"get-merge-review-data.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/get-merge-review-data.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EACV,eAAe,EAEhB,MAAM,sDAAsD,CAAC;AAK9D,KAAK,wBAAwB,GAAG,eAAe,GAAG;IAAE,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAqCpE,wBAAsB,kBAAkB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,wBAAwB,CAAC,CAmG7F"}
+{"version":3,"file":"get-merge-review-data.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/get-merge-review-data.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EACV,eAAe,EAEhB,MAAM,sDAAsD,CAAC;AAK9D,KAAK,wBAAwB,GAAG,eAAe,GAAG;IAAE,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAqCpE,wBAAsB,kBAAkB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,wBAAwB,CAAC,CA2G7F"}

package/dist/src/presentation/web/app/actions/get-merge-review-data.js

@@ -92,7 +92,16 @@ export async function getMergeReviewData(featureId) {
                 const manifestPath = join(evidenceDir, 'manifest.json');
                 if (existsSync(manifestPath)) {
                     const raw = JSON.parse(readFileSync(manifestPath, 'utf-8'));
-                    evidence = normalizeEvidencePaths(raw, evidenceDir);
+                    const normalized = normalizeEvidencePaths(raw, evidenceDir);
+                    // Deduplicate: same type + relativePath means the same evidence entry
+                    const seen = new Set();
+                    evidence = normalized.filter((e) => {
+                        const key = `${e.type}:${e.relativePath}`;
+                        if (seen.has(key))
+                            return false;
+                        seen.add(key);
+                        return true;
+                    });
                 }
             }
             catch {

package/dist/src/presentation/web/components/common/merge-review/merge-review.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"merge-review.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/merge-review/merge-review.tsx"],"names":[],"mappings":"AAuBA,OAAO,KAAK,EAAE,gBAAgB,EAAuB,MAAM,uBAAuB,CAAC;AAmNnF,wBAAgB,WAAW,CAAC,EAC1B,IAAI,EACJ,QAAgB,EAChB,SAAS,EACT,QAAQ,EACR,YAAoB,EACpB,WAAmB,EACnB,SAAS,EACT,iBAAiB,GAClB,EAAE,gBAAgB,2CA+JlB"}
+{"version":3,"file":"merge-review.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/merge-review/merge-review.tsx"],"names":[],"mappings":"AAsBA,OAAO,KAAK,EAAE,gBAAgB,EAAuB,MAAM,uBAAuB,CAAC;AAuJnF,wBAAgB,WAAW,CAAC,EAC1B,IAAI,EACJ,QAAgB,EAChB,SAAS,EACT,QAAQ,EACR,YAAoB,EACpB,WAAmB,EACnB,SAAS,EACT,iBAAiB,GAClB,EAAE,gBAAgB,2CA+JlB"}

package/dist/src/presentation/web/components/common/merge-review/merge-review.js

@@ -1,11 +1,10 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { useState, useMemo } from 'react';
+import { useState } from 'react';
 import { ExternalLink, AlertTriangle, FileDiff, GitCommitHorizontal, GitBranch, ArrowRight, Camera, FileText, MonitorPlay, Terminal, Download, ChevronDown, ChevronRight, } from 'lucide-react';
 import { Badge } from '../../ui/badge.js';
 import { CiStatusBadge } from '../../common/ci-status-badge/index.js';
 import { DrawerActionBar } from '../../common/drawer-action-bar/index.js';
-import { EvidenceLightbox } from '../../common/evidence-lightbox/index.js';
 import { DiffView } from './diff-view.js';
 const EVIDENCE_ICONS = {
     Screenshot: Camera,
@@ -55,33 +54,8 @@ function EvidenceTextPreview({ url }) {
     }
     return (_jsx("pre", { className: "bg-muted/50 max-h-60 overflow-auto rounded-md p-3 font-mono text-[11px] leading-relaxed", children: content }));
 }
-function isImageEvidence(evidence) {
-    return (evidence.type === 'Screenshot' || IMAGE_EXTENSIONS.has(getExtension(evidence.relativePath)));
-}
 function EvidenceList({ evidence }) {
-    const [lightboxOpen, setLightboxOpen] = useState(false);
-    const [selectedIndex, setSelectedIndex] = useState(0);
-    const { imageEvidence, nonImageEvidence } = useMemo(() => {
-        const imgs = [];
-        const nonImgs = [];
-        for (const e of evidence) {
-            if (isImageEvidence(e)) {
-                imgs.push(e);
-            }
-            else {
-                nonImgs.push(e);
-            }
-        }
-        return { imageEvidence: imgs, nonImageEvidence: nonImgs };
-    }, [evidence]);
-    function handleThumbnailClick(index) {
-        setSelectedIndex(index);
-        setLightboxOpen(true);
-    }
-    return (_jsxs("div", { className: "border-border rounded-lg border", children: [_jsxs("div", { className: "px-4 py-3", children: [_jsxs("div", { className: "mb-3 flex items-center gap-2", children: [_jsx(Camera, { className: "text-muted-foreground h-4 w-4" }), _jsx("span", { className: "text-foreground text-xs font-semibold", children: "Evidence" }), _jsx(Badge, { variant: "secondary", className: "text-[10px]", children: evidence.length })] }), imageEvidence.length > 0 ? (
-                    /* eslint-disable @next/next/no-img-element -- Local evidence files require raw <img>, not next/image */
-                    _jsx("div", { className: "mb-3 grid grid-cols-3 gap-2", children: imageEvidence.map((e, i) => (_jsx("button", { type: "button", className: "cursor-pointer overflow-hidden rounded-md border", onClick: () => handleThumbnailClick(i), children: _jsx("img", { src: buildEvidenceUrl(e.relativePath), alt: e.description, className: "aspect-square w-full object-cover", loading: "lazy" }) }, `thumb-${e.relativePath}`))) })) : /* eslint-enable @next/next/no-img-element */
-                        null, nonImageEvidence.length > 0 ? (_jsx("ul", { className: "space-y-2", children: nonImageEvidence.map((e) => (_jsx(EvidenceItem, { evidence: e }, `${e.type}-${e.relativePath}`))) })) : null] }), _jsx(EvidenceLightbox, { images: imageEvidence, open: lightboxOpen, onOpenChange: setLightboxOpen, selectedIndex: selectedIndex, onSelectedIndexChange: setSelectedIndex })] }));
+    return (_jsx("div", { className: "border-border rounded-lg border", children: _jsxs("div", { className: "px-4 py-3", children: [_jsxs("div", { className: "mb-3 flex items-center gap-2", children: [_jsx(Camera, { className: "text-muted-foreground h-4 w-4" }), _jsx("span", { className: "text-foreground text-xs font-semibold", children: "Evidence" }), _jsx(Badge, { variant: "secondary", className: "text-[10px]", children: evidence.length })] }), _jsx("ul", { className: "space-y-2", children: evidence.map((e) => (_jsx(EvidenceItem, { evidence: e }, `${e.type}-${e.relativePath}`))) })] }) }));
 }
 export function MergeReview({ data, readOnly = false, onApprove, onReject, isProcessing = false, isRejecting = false, chatInput, onChatInputChange, }) {
     const { pr, diffSummary, fileDiffs, branch, warning, evidence, hideCiStatus } = data;