@shepai/cli 1.90.1-pr267.467b85c → 1.90.1-pr269.a59b088

package/dist/src/presentation/web/app/actions/get-feature-phase-timings.d.ts

@@ -11,7 +11,6 @@ export interface RejectionFeedbackData {
     iteration: number;
     message: string;
     phase?: string;
-    attachments?: string[];
 }
 type GetPhaseTimingsResult = {
     timings: PhaseTimingData[];

package/dist/src/presentation/web/app/actions/get-feature-phase-timings.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"get-feature-phase-timings.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/get-feature-phase-timings.ts"],"names":[],"mappings":"AAMA,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,MAAM,CAAC;IACd,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,qBAAqB;IACpC,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,MAAM,EAAE,CAAC;CACxB;AAED,KAAK,qBAAqB,GACtB;IAAE,OAAO,EAAE,eAAe,EAAE,CAAC;IAAC,iBAAiB,EAAE,qBAAqB,EAAE,CAAA;CAAE,GAC1E;IAAE,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAEtB,wBAAsB,sBAAsB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,qBAAqB,CAAC,CA2B9F"}
+{"version":3,"file":"get-feature-phase-timings.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/get-feature-phase-timings.ts"],"names":[],"mappings":"AAMA,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,MAAM,CAAC;IACd,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,qBAAqB;IACpC,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,KAAK,qBAAqB,GACtB;IAAE,OAAO,EAAE,eAAe,EAAE,CAAC;IAAC,iBAAiB,EAAE,qBAAqB,EAAE,CAAA;CAAE,GAC1E;IAAE,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAEtB,wBAAsB,sBAAsB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,qBAAqB,CAAC,CA2B9F"}

package/dist/src/presentation/web/app/actions/get-feature-phase-timings.js

@@ -42,9 +42,6 @@ async function readRejectionFeedback(featureId) {
             iteration: Number(entry.iteration ?? 1),
             message: String(entry.message ?? ''),
             phase: entry.phase ? String(entry.phase) : undefined,
-            attachments: Array.isArray(entry.attachments)
-                ? entry.attachments.map(String)
-                : undefined,
         }));
     }
     catch {

package/dist/src/presentation/web/app/api/attachments/preview/route.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"route.d.ts","sourceRoot":"","sources":["../../../../../../../../src/presentation/web/app/api/attachments/preview/route.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAC/C,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AA8D3C,wBAAsB,GAAG,CAAC,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,YAAY,CAAC,CA0BrE"}
+{"version":3,"file":"route.d.ts","sourceRoot":"","sources":["../../../../../../../../src/presentation/web/app/api/attachments/preview/route.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAC/C,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAsB3C,wBAAsB,GAAG,CAAC,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,YAAY,CAAC,CA6BrE"}

package/dist/src/presentation/web/app/api/attachments/preview/route.js

@@ -1,6 +1,6 @@
 import { NextResponse } from 'next/server';
-import { readFile, stat, readdir } from 'fs/promises';
-import { basename, extname, join, resolve } from 'path';
+import { readFile, stat } from 'fs/promises';
+import { extname, resolve } from 'path';
 import { getShepHomeDir } from '../../../../../../../packages/core/src/infrastructure/services/filesystem/shep-directory.service.js';
 const MIME_MAP = {
     '.png': 'image/png',
@@ -18,44 +18,6 @@ const MIME_MAP = {
     '.yaml': 'text/yaml',
     '.yml': 'text/yaml',
 };
-/**
- * When the exact path doesn't exist (e.g. pending-<id> was renamed to <slug>),
- * search all sibling attachment directories for the same filename.
- */
-async function findAttachmentFallback(requestedPath, attachmentsRoot) {
-    const filename = basename(requestedPath);
-    try {
-        const dirs = await readdir(attachmentsRoot, { withFileTypes: true });
-        for (const entry of dirs) {
-            if (!entry.isDirectory())
-                continue;
-            const candidate = join(attachmentsRoot, entry.name, filename);
-            try {
-                await stat(candidate);
-                return candidate;
-            }
-            catch {
-                // not in this directory
-            }
-        }
-    }
-    catch {
-        // attachments root doesn't exist
-    }
-    return null;
-}
-function serveFile(filePath, mimeTypeHint) {
-    return readFile(filePath).then((buffer) => {
-        const ext = extname(filePath).toLowerCase();
-        const contentType = MIME_MAP[ext] ?? mimeTypeHint ?? 'application/octet-stream';
-        return new NextResponse(buffer, {
-            headers: {
-                'Content-Type': contentType,
-                'Cache-Control': 'private, max-age=3600',
-            },
-        });
-    });
-}
 export async function GET(request) {
     const path = request.nextUrl.searchParams.get('path');
     const mimeTypeHint = request.nextUrl.searchParams.get('mimeType');
@@ -69,15 +31,17 @@ export async function GET(request) {
     }
     try {
         await stat(path);
-        return await serveFile(path, mimeTypeHint);
+        const buffer = await readFile(path);
+        const ext = extname(path).toLowerCase();
+        const contentType = MIME_MAP[ext] ?? mimeTypeHint ?? 'application/octet-stream';
+        return new NextResponse(buffer, {
+            headers: {
+                'Content-Type': contentType,
+                'Cache-Control': 'private, max-age=3600',
+            },
+        });
     }
     catch {
-        // File not found at exact path — try fallback search
-        // This handles the case where pending-<sessionId>/ was renamed to <featureSlug>/
-        const fallbackPath = await findAttachmentFallback(path, attachmentsRoot);
-        if (fallbackPath) {
-            return await serveFile(fallbackPath, mimeTypeHint);
-        }
         return NextResponse.json({ error: 'File not found' }, { status: 404 });
     }
 }

package/dist/src/presentation/web/components/common/feature-drawer-tabs/activity-tab.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"activity-tab.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-drawer-tabs/activity-tab.tsx"],"names":[],"mappings":"AAGA,OAAO,KAAK,EACV,eAAe,EACf,qBAAqB,EACtB,MAAM,yCAAyC,CAAC;AAGjD,MAAM,WAAW,gBAAgB;IAC/B,OAAO,EAAE,eAAe,EAAE,GAAG,IAAI,CAAC;IAClC,OAAO,EAAE,OAAO,CAAC;IACjB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,iBAAiB,CAAC,EAAE,qBAAqB,EAAE,CAAC;CAC7C;AA4ED,wBAAgB,WAAW,CAAC,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,iBAAiB,EAAE,EAAE,gBAAgB,2CA8D3F"}
+{"version":3,"file":"activity-tab.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-drawer-tabs/activity-tab.tsx"],"names":[],"mappings":"AAGA,OAAO,KAAK,EACV,eAAe,EACf,qBAAqB,EACtB,MAAM,yCAAyC,CAAC;AAEjD,MAAM,WAAW,gBAAgB;IAC/B,OAAO,EAAE,eAAe,EAAE,GAAG,IAAI,CAAC;IAClC,OAAO,EAAE,OAAO,CAAC;IACjB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,iBAAiB,CAAC,EAAE,qBAAqB,EAAE,CAAC;CAC7C;AAsED,wBAAgB,WAAW,CAAC,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,iBAAiB,EAAE,EAAE,gBAAgB,2CA8D3F"}

package/dist/src/presentation/web/components/common/feature-drawer-tabs/activity-tab.js

@@ -1,7 +1,6 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { Loader2, AlertCircle, Clock } from 'lucide-react';
-import { InlineAttachments } from '../../common/inline-attachments/index.js';
 const NODE_TO_PHASE = {
     analyze: 'Analyzing',
     requirements: 'Requirements',
@@ -39,7 +38,7 @@ function groupTimingsByRun(timings) {
     return groups;
 }
 /**
- * Map each run:rejected timing entry to its rejection feedback message and attachments.
+ * Map each run:rejected timing entry to its rejection feedback message.
  * Matches by sequential order: 1st rejected event → 1st feedback entry, etc.
  */
 function buildRejectionMessageMap(timings, feedback) {
@@ -50,8 +49,7 @@ function buildRejectionMessageMap(timings, feedback) {
     for (const t of timings) {
         if (t.phase === 'run:rejected' && rejectionIdx < feedback.length) {
             const key = `${t.agentRunId}-${t.phase}-${t.startedAt}`;
-            const entry = feedback[rejectionIdx];
-            map.set(key, { message: entry.message, attachments: entry.attachments });
+            map.set(key, feedback[rejectionIdx].message);
             rejectionIdx++;
         }
     }
@@ -81,18 +79,17 @@ function RunGroup({ group, groupIdx, multiRun, maxDurationMs, rejectionMessages,
     return (_jsxs("div", { className: "flex flex-col gap-1.5", children: [multiRun ? (_jsxs("div", { className: "text-muted-foreground text-xs font-medium", children: ["Run #", groupIdx + 1] })) : null, group.timings.map((t) => {
                 if (isLifecycleEvent(t.phase)) {
                     const key = `${t.agentRunId}-${t.phase}-${t.startedAt}`;
-                    const rejection = t.phase === 'run:rejected' ? rejectionMessages.get(key) : undefined;
-                    return (_jsx(LifecycleEventRow, { timing: t, message: rejection?.message, attachments: rejection?.attachments }, key));
+                    const message = t.phase === 'run:rejected' ? rejectionMessages.get(key) : undefined;
+                    return _jsx(LifecycleEventRow, { timing: t, message: message }, key);
                 }
                 return (_jsx(NodeTimingRow, { timing: t, maxDurationMs: maxDurationMs }, `${t.agentRunId}-${t.phase}-${t.startedAt}`));
             })] }));
 }
-function LifecycleEventRow({ timing, message, attachments, }) {
+function LifecycleEventRow({ timing, message }) {
     const event = LIFECYCLE_EVENTS[timing.phase];
     if (!event)
         return null;
-    const hasAttachments = attachments && attachments.length > 0;
-    return (_jsxs("div", { className: `text-xs ${event.colorClass}`, children: [_jsx("span", { children: event.label }), message ? (_jsxs("span", { "data-testid": "rejection-feedback-text", className: "text-muted-foreground ml-2 font-normal italic", children: ["\u2014 ", message] })) : null, hasAttachments ? (_jsx("div", { "data-testid": "rejection-feedback-attachments", className: "mt-1.5 ml-2", children: _jsx(InlineAttachments, { text: "", attachmentPaths: attachments }) })) : null] }));
+    return (_jsxs("div", { className: `text-xs ${event.colorClass}`, children: [_jsx("span", { children: event.label }), message ? (_jsxs("span", { "data-testid": "rejection-feedback-text", className: "text-muted-foreground ml-2 font-normal italic", children: ["\u2014 ", message] })) : null] }));
 }
 function NodeTimingRow({ timing, maxDurationMs, }) {
     const label = timing.phase.includes(':')

package/dist/src/presentation/web/components/common/feature-drawer-tabs/overview-tab.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"overview-tab.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-drawer-tabs/overview-tab.tsx"],"names":[],"mappings":"AAWA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AAMxE,MAAM,WAAW,gBAAgB;IAC/B,IAAI,EAAE,eAAe,CAAC;CACvB;AAED,wBAAgB,WAAW,CAAC,EAAE,IAAI,EAAE,EAAE,gBAAgB,2CAwCrD"}
+{"version":3,"file":"overview-tab.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-drawer-tabs/overview-tab.tsx"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AAMxE,MAAM,WAAW,gBAAgB;IAC/B,IAAI,EAAE,eAAe,CAAC;CACvB;AAED,wBAAgB,WAAW,CAAC,EAAE,IAAI,EAAE,EAAE,gBAAgB,2CAwCrD"}

package/dist/src/presentation/web/components/common/feature-drawer-tabs/overview-tab.js

@@ -1,7 +1,6 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { ExternalLink, GitBranch, GitCommitHorizontal, Zap } from 'lucide-react';
-import { InlineAttachments } from '../../common/inline-attachments/index.js';
 import { PrStatus } from '../../../../../../packages/core/src/domain/generated/output.js';
 import { cn } from '../../../lib/utils.js';
 import { Badge } from '../../ui/badge.js';
@@ -51,7 +50,7 @@ function FeatureInfo({ data }) {
         Boolean(data.createdAt);
     if (!hasInfo)
         return null;
-    return (_jsxs(_Fragment, { children: [_jsx(Separator, {}), _jsxs("div", { "data-testid": "feature-drawer-info", className: "flex flex-col gap-3 p-4", children: [data.branch ? (_jsxs("div", { className: "flex flex-col gap-0.5", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: "Branch" }), _jsxs("span", { className: "flex items-center gap-1.5 text-sm", children: [_jsx(GitBranch, { className: "text-muted-foreground h-3.5 w-3.5 shrink-0" }), _jsx("code", { className: "bg-muted rounded px-1 py-0.5 font-mono text-xs", children: data.branch }), data.baseBranch ? (_jsxs("span", { className: "text-muted-foreground text-xs", children: ["from ", data.baseBranch] })) : null] })] })) : null, data.oneLiner ? (_jsxs("div", { className: "flex flex-col gap-0.5", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: "One-Liner" }), _jsx("span", { className: "text-sm leading-relaxed", children: data.oneLiner })] })) : null, data.userQuery ? (_jsxs("div", { className: "flex flex-col gap-0.5", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: "User Query" }), _jsx(InlineAttachments, { text: data.userQuery })] })) : null, data.summary ? (_jsxs("div", { className: "flex flex-col gap-0.5", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: "Summary" }), _jsx("span", { className: "text-sm leading-relaxed", children: data.summary })] })) : null, data.createdAt ? (_jsx(DetailRow, { label: "Created", value: formatRelativeTime(data.createdAt) })) : null] })] }));
+    return (_jsxs(_Fragment, { children: [_jsx(Separator, {}), _jsxs("div", { "data-testid": "feature-drawer-info", className: "flex flex-col gap-3 p-4", children: [data.branch ? (_jsxs("div", { className: "flex flex-col gap-0.5", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: "Branch" }), _jsxs("span", { className: "flex items-center gap-1.5 text-sm", children: [_jsx(GitBranch, { className: "text-muted-foreground h-3.5 w-3.5 shrink-0" }), _jsx("code", { className: "bg-muted rounded px-1 py-0.5 font-mono text-xs", children: data.branch }), data.baseBranch ? (_jsxs("span", { className: "text-muted-foreground text-xs", children: ["from ", data.baseBranch] })) : null] })] })) : null, data.oneLiner ? (_jsxs("div", { className: "flex flex-col gap-0.5", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: "One-Liner" }), _jsx("span", { className: "text-sm leading-relaxed", children: data.oneLiner })] })) : null, data.userQuery ? (_jsxs("div", { className: "flex flex-col gap-0.5", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: "User Query" }), _jsx("span", { className: "text-sm leading-relaxed", children: data.userQuery })] })) : null, data.summary ? (_jsxs("div", { className: "flex flex-col gap-0.5", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: "Summary" }), _jsx("span", { className: "text-sm leading-relaxed", children: data.summary })] })) : null, data.createdAt ? (_jsx(DetailRow, { label: "Created", value: formatRelativeTime(data.createdAt) })) : null] })] }));
 }
 // ── PR Info section ──────────────────────────────────────────────────
 const prStatusStyles = {