@shepai/cli 1.90.1 → 1.91.0-pr257.0ec533a

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

@@ -1,6 +1,7 @@
 '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';
@@ -50,7 +51,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("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] })] }));
+    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] })] }));
 }
 // ── PR Info section ──────────────────────────────────────────────────
 const prStatusStyles = {

package/dist/src/presentation/web/components/common/inline-attachments/index.d.ts

@@ -0,0 +1,3 @@
+export { InlineAttachments, parseAttachmentRefs } from './inline-attachments.js';
+export type { InlineAttachmentsProps } from './inline-attachments.js';
+//# sourceMappingURL=index.d.ts.map

package/dist/src/presentation/web/components/common/inline-attachments/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/inline-attachments/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AAC9E,YAAY,EAAE,sBAAsB,EAAE,MAAM,sBAAsB,CAAC"}

package/dist/src/presentation/web/components/common/inline-attachments/index.js

@@ -0,0 +1 @@
+export { InlineAttachments, parseAttachmentRefs } from './inline-attachments.js';

package/dist/src/presentation/web/components/common/inline-attachments/inline-attachments.d.ts

@@ -0,0 +1,20 @@
+export interface InlineAttachmentsProps {
+    /** Text that may contain `@/path/to/file` attachment references. */
+    text: string;
+    /** Additional attachment paths to render (e.g. from rejection feedback). */
+    attachmentPaths?: string[];
+}
+interface TextSegment {
+    type: 'text';
+    value: string;
+}
+interface AttachmentSegment {
+    type: 'attachment';
+    path: string;
+}
+type Segment = TextSegment | AttachmentSegment;
+/** Parse text into segments of plain text and attachment references. */
+export declare function parseAttachmentRefs(text: string): Segment[];
+export declare function InlineAttachments({ text, attachmentPaths }: InlineAttachmentsProps): import("react/jsx-runtime").JSX.Element;
+export {};
+//# sourceMappingURL=inline-attachments.d.ts.map

package/dist/src/presentation/web/components/common/inline-attachments/inline-attachments.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"inline-attachments.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/inline-attachments/inline-attachments.tsx"],"names":[],"mappings":"AA6BA,MAAM,WAAW,sBAAsB;IACrC,oEAAoE;IACpE,IAAI,EAAE,MAAM,CAAC;IACb,4EAA4E;IAC5E,eAAe,CAAC,EAAE,MAAM,EAAE,CAAC;CAC5B;AAED,UAAU,WAAW;IACnB,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;CACf;AAED,UAAU,iBAAiB;IACzB,IAAI,EAAE,YAAY,CAAC;IACnB,IAAI,EAAE,MAAM,CAAC;CACd;AAED,KAAK,OAAO,GAAG,WAAW,GAAG,iBAAiB,CAAC;AAE/C,wEAAwE;AACxE,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,EAAE,CAwB3D;AAED,wBAAgB,iBAAiB,CAAC,EAAE,IAAI,EAAE,eAAe,EAAE,EAAE,sBAAsB,2CA4BlF"}

package/dist/src/presentation/web/components/common/inline-attachments/inline-attachments.js

@@ -0,0 +1,72 @@
+/* eslint-disable @next/next/no-img-element -- Local file preview requires raw <img>, not next/image */
+'use client';
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { useState } from 'react';
+import { Dialog, DialogTrigger, DialogContent, DialogTitle } from '../../ui/dialog.js';
+import { VisuallyHidden } from 'radix-ui';
+import { DownloadIcon, ImageOff } from 'lucide-react';
+const IMAGE_EXTS = new Set(['.png', '.jpg', '.jpeg', '.gif', '.svg', '.webp', '.ico', '.bmp']);
+function isImagePath(path) {
+    const dot = path.lastIndexOf('.');
+    return dot >= 0 && IMAGE_EXTS.has(path.slice(dot).toLowerCase());
+}
+function previewUrl(path) {
+    const ext = path.lastIndexOf('.') >= 0 ? path.slice(path.lastIndexOf('.')) : '';
+    const mimeGuess = isImagePath(path) ? `image/${ext.slice(1).replace('jpg', 'jpeg')}` : '';
+    const params = new URLSearchParams({ path, ...(mimeGuess && { mimeType: mimeGuess }) });
+    return `/api/attachments/preview?${params.toString()}`;
+}
+function getFilename(path) {
+    return path.split('/').pop() ?? path;
+}
+/** Regex matching `@/absolute/path` attachment references in text. */
+const ATTACHMENT_REF_RE = /(?:^|\s)@(\/[^\s]+)/g;
+/** Parse text into segments of plain text and attachment references. */
+export function parseAttachmentRefs(text) {
+    const segments = [];
+    let lastIndex = 0;
+    for (const match of text.matchAll(ATTACHMENT_REF_RE)) {
+        const fullMatchStart = match.index;
+        // The path capture starts after the optional leading whitespace
+        const leadingWs = match[0].length - match[0].trimStart().length;
+        const refStart = fullMatchStart + leadingWs;
+        // Add text before this reference (including leading whitespace)
+        if (refStart > lastIndex) {
+            segments.push({ type: 'text', value: text.slice(lastIndex, refStart) });
+        }
+        segments.push({ type: 'attachment', path: match[1] });
+        lastIndex = fullMatchStart + match[0].length;
+    }
+    if (lastIndex < text.length) {
+        segments.push({ type: 'text', value: text.slice(lastIndex) });
+    }
+    return segments;
+}
+export function InlineAttachments({ text, attachmentPaths }) {
+    const segments = parseAttachmentRefs(text);
+    const hasInlineAttachments = segments.some((s) => s.type === 'attachment');
+    const extraPaths = attachmentPaths?.filter(Boolean) ?? [];
+    if (!hasInlineAttachments && extraPaths.length === 0) {
+        return _jsx("span", { className: "text-sm leading-relaxed", children: text });
+    }
+    return (_jsxs("div", { className: "flex flex-col gap-2", children: [segments.map((segment) => {
+                if (segment.type === 'text') {
+                    const trimmed = segment.value.trim();
+                    if (!trimmed)
+                        return null;
+                    return (_jsx("span", { className: "text-sm leading-relaxed", children: trimmed }, `text-${trimmed.slice(0, 40)}`));
+                }
+                return _jsx(AttachmentPreview, { path: segment.path }, `att-${segment.path}`);
+            }), extraPaths.map((path) => (_jsx(AttachmentPreview, { path: path }, `extra-${path}`)))] }));
+}
+function AttachmentPreview({ path }) {
+    const filename = getFilename(path);
+    const [loadError, setLoadError] = useState(false);
+    if (isImagePath(path)) {
+        if (loadError) {
+            return (_jsxs("div", { "data-testid": "inline-attachment-image-error", className: "text-muted-foreground flex items-center gap-2 rounded-md border px-3 py-2 text-sm", children: [_jsx(ImageOff, { className: "h-4 w-4 shrink-0" }), _jsx("span", { className: "truncate", children: filename })] }));
+        }
+        return (_jsxs(Dialog, { children: [_jsx(DialogTrigger, { asChild: true, children: _jsx("img", { src: previewUrl(path), alt: filename, "data-testid": "inline-attachment-image", onError: () => setLoadError(true), className: "max-h-48 max-w-full cursor-pointer rounded-md border object-contain transition-opacity hover:opacity-80" }) }), _jsxs(DialogContent, { className: "max-w-3xl gap-0 overflow-hidden border-0 p-0 [&>button:last-child]:!cursor-pointer [&>button:last-child]:!rounded-full [&>button:last-child]:!bg-black/70 [&>button:last-child]:!p-1.5 [&>button:last-child]:!text-white [&>button:last-child]:!opacity-100 [&>button:last-child]:!shadow-lg [&>button:last-child]:!backdrop-blur-md [&>button:last-child]:hover:!bg-black/90", children: [_jsx(VisuallyHidden.Root, { children: _jsxs(DialogTitle, { children: ["Preview: ", filename] }) }), _jsx("div", { className: "relative bg-black/90", children: _jsx("img", { src: previewUrl(path), alt: filename, className: "h-auto max-h-[70vh] w-full object-contain" }) }), _jsxs("div", { className: "bg-background flex items-center gap-3 px-4 py-3", children: [_jsx("div", { className: "flex min-w-0 flex-1 flex-col", children: _jsx("span", { className: "truncate text-sm font-medium", children: filename }) }), _jsx("a", { href: previewUrl(path), download: filename, className: "text-muted-foreground hover:text-foreground shrink-0 cursor-pointer rounded p-1.5 transition-colors", "aria-label": `Download ${filename}`, children: _jsx(DownloadIcon, { className: "h-4 w-4" }) })] })] })] }));
+    }
+    return (_jsx("a", { href: previewUrl(path), download: filename, "data-testid": "inline-attachment-file", className: "text-primary text-sm underline underline-offset-2", children: filename }));
+}

package/dist/src/presentation/web/components/common/inline-attachments/inline-attachments.stories.d.ts

@@ -0,0 +1,16 @@
+import type { Meta, StoryObj } from '@storybook/react';
+import { InlineAttachments } from './inline-attachments.js';
+declare const meta: Meta<typeof InlineAttachments>;
+export default meta;
+type Story = StoryObj<typeof InlineAttachments>;
+/** Plain text with no attachment references. */
+export declare const PlainText: Story;
+/** Text with an image attachment reference. */
+export declare const WithImageAttachment: Story;
+/** Text with multiple attachment references. */
+export declare const MultipleAttachments: Story;
+/** Text with a non-image file attachment. */
+export declare const FileAttachment: Story;
+/** Extra attachment paths passed via prop (e.g. rejection feedback). */
+export declare const ExtraAttachmentPaths: Story;
+//# sourceMappingURL=inline-attachments.stories.d.ts.map

package/dist/src/presentation/web/components/common/inline-attachments/inline-attachments.stories.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"inline-attachments.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/inline-attachments/inline-attachments.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AACvD,OAAO,EAAE,iBAAiB,EAAE,MAAM,sBAAsB,CAAC;AAEzD,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,OAAO,iBAAiB,CAYxC,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,iBAAiB,CAAC,CAAC;AAEhD,gDAAgD;AAChD,eAAO,MAAM,SAAS,EAAE,KAIvB,CAAC;AAEF,+CAA+C;AAC/C,eAAO,MAAM,mBAAmB,EAAE,KAIjC,CAAC;AAEF,gDAAgD;AAChD,eAAO,MAAM,mBAAmB,EAAE,KAIjC,CAAC;AAEF,6CAA6C;AAC7C,eAAO,MAAM,cAAc,EAAE,KAI5B,CAAC;AAEF,wEAAwE;AACxE,eAAO,MAAM,oBAAoB,EAAE,KAQlC,CAAC"}

package/dist/src/presentation/web/components/common/inline-attachments/inline-attachments.stories.js

@@ -0,0 +1,46 @@
+import { jsx as _jsx } from "react/jsx-runtime";
+import { InlineAttachments } from './inline-attachments.js';
+const meta = {
+    title: 'Common/InlineAttachments',
+    component: InlineAttachments,
+    tags: ['autodocs'],
+    parameters: { layout: 'padded' },
+    decorators: [
+        (Story) => (_jsx("div", { style: { maxWidth: '400px', border: '1px solid var(--color-border)', padding: '16px' }, children: _jsx(Story, {}) })),
+    ],
+};
+export default meta;
+/** Plain text with no attachment references. */
+export const PlainText = {
+    args: {
+        text: 'Fix the login bug where users cannot sign in with Google OAuth.',
+    },
+};
+/** Text with an image attachment reference. */
+export const WithImageAttachment = {
+    args: {
+        text: 'The button is misaligned, see screenshot:\n@/home/user/.shep/attachments/pending-abc/screenshot.png',
+    },
+};
+/** Text with multiple attachment references. */
+export const MultipleAttachments = {
+    args: {
+        text: 'Before: @/home/user/.shep/attachments/pending-abc/before.png\nAfter: @/home/user/.shep/attachments/pending-abc/after.png',
+    },
+};
+/** Text with a non-image file attachment. */
+export const FileAttachment = {
+    args: {
+        text: 'See the log file:\n@/home/user/.shep/attachments/pending-abc/error.log',
+    },
+};
+/** Extra attachment paths passed via prop (e.g. rejection feedback). */
+export const ExtraAttachmentPaths = {
+    args: {
+        text: 'Please fix the layout issues',
+        attachmentPaths: [
+            '/home/user/.shep/attachments/pending-abc/screenshot1.png',
+            '/home/user/.shep/attachments/pending-abc/screenshot2.png',
+        ],
+    },
+};

package/dist/src/presentation/web/hooks/use-selected-feature-id.js

@@ -6,6 +6,6 @@ import { usePathname } from 'next/navigation';
  */
 export function useSelectedFeatureId() {
     const pathname = usePathname();
-    const match = pathname.match(/^\/feature\/(.+)$/);
+    const match = pathname.match(/^\/feature\/([^/]+)/);
     return match?.[1] ?? null;
 }