Files
PaperClipAI/ui/src/components/task-chat/TweakPanel.tsx
T
scotttongandClaude Fable 5 c185e64b77 feat(ui): chat-style task view behind an experimental flag (#10606)
## Thinking Path

> - Paperclip is the open source app people use to manage AI agents for
work.
> - Operators spend most of their time on the issue detail page. They
talk to the assigned agent there through comments.
> - The current page reads as a ticket form. The thread sits below
properties, the composer sits mid-page, and live agent activity renders
as dense transcript logs.
> - Talking to an agent is a conversation. A chat-first layout matches
that mental model better than a ticket form.
> - A layout change this large must not disrupt current users. It needs
a safe opt-in path and full parity with the existing thread features.
> - This pull request adds a chat-style task view behind a new
"Chat-Style Tasks" experiment toggle. The flag is off by default and the
existing page is unchanged when it is off.
> - The benefit is a focused, readable conversation with the agent: live
tool activity folds into compact summaries, the composer stays at the
bottom, and properties, plan, and artifacts move into header tabs.

## Linked Issues or Issue Description

Refs #49 (chat with agents is a much-wanted feature).

Related PRs found in the dedup search:
- #4489 — an earlier, closed attempt to promote the conversation to the
primary surface on issue detail. This PR is a fresh, flag-gated take on
the same goal.
- #8837 — an open PR that proposes a two-column task layout. It
restructures the same page but keeps the ticket paradigm; this PR is
orthogonal because it is opt-in and chat-first.

**Subsystem affected**

UI (issue detail page).

**Problem or motivation**

The issue detail page presents agent conversations as a ticket:
properties first, thread below, composer in the middle of the page, and
raw transcript noise during live runs. Users who mainly converse with
their agents must scroll past chrome to follow the conversation, and
live activity is hard to read.

**Proposed solution**

An opt-in chat-style view of the issue detail page, gated by a new
"Chat-Style Tasks" experiment toggle in Settings → Experimental. With
the flag on, the thread fills the center pane, the composer docks to the
bottom of the viewport, Properties / Plan / Artifacts become header
tabs, live turns show a status pill with the current tool action and
elapsed time, and settled turns collapse to a "Worked · N tools" summary
that expands into per-tool rows. With the flag off, nothing changes.

**Alternatives considered**

Restyling the existing layout in place (rejected: too disruptive without
an opt-out), and a separate chat page beside the issue page (rejected:
splits the task's single source of truth). A per-request lab page
(`/task-chat-lab`, dev-only) was kept for design iteration instead.

**Roadmap alignment**

ROADMAP.md "CEO Chat" wants lighter conversations that still resolve to
real work objects. This PR keeps the core task-and-comments model — it
only changes presentation, opt-in — so it does not duplicate that
planned work.

## What Changed

- New `enableTaskChatRedesign` instance setting, exposed as a
"Chat-Style Tasks" experiment card in Settings → Experimental (shared
feature catalog, validators, server instance-settings service, and UI
settings page).
- New `ui/src/components/task-chat/` component family: chat thread with
turn grouping, agent reply bubbles, live status pill, collapsible turn
summaries with per-tool rows, plan tab with a sticky CTA action bar,
inline interaction cards, per-request mode chips, and a bottom-docked
composer.
- A shared tool taxonomy (`tool-taxonomy.ts`) maps tool names to verbs
and icons; the status pill, tool rows, and the classic transcript view
all use it.
- A transcript adapter converts stored run logs into chat turns; it
dedupes tool-call updates by `toolUseId` so tool counts match the
expanded rows, and it keeps a tool row's first real name when later
generic updates arrive.
- Composer: posts on Cmd/Ctrl+Enter, supports image paste with
object-URL thumbnail previews (revoked on clear/unmount), and uploads
through the issue attachments route.
- `IssueDetail.tsx`: with the flag on, pane tabs move to the header bar,
the header is not sticky, and the chat fills the center; with the flag
off, the previous layout renders unchanged.
- Motion tokens for the new animations live in `ui/src/index.css` with a
`motion-tokens.ts` catalog and a test that keeps the two in sync (the
catalog now also covers the shared enter/exit/swap tokens that the
decision/quicklook block declares).
- A dev-only `/task-chat-lab` page with fixtures and a tweak panel for
motion tuning.

## Verification

- `pnpm typecheck` — clean across the workspace.
- `pnpm check:token-gates` — 3/3 CLEAN.
- `cd ui && pnpm vitest run` — 3,344 of 3,345 tests pass locally. The
one failure is `IssueProperties.test.tsx` monitor-row time formatting,
which is timezone-sensitive: it also fails on unmodified `origin/master`
in a non-UTC timezone and passes with `TZ=UTC`. It is not related to
this change.
- `cd server && pnpm vitest run
src/__tests__/instance-settings-service.test.ts` — 21/21 pass (covers
the new setting).
- Manual: start the dev server, open Settings → Experimental, enable
"Chat-Style Tasks", and open any issue. The thread fills the page, the
composer docks to the bottom, and Properties / Plan / Artifacts appear
as header tabs. Assign an agent and comment to watch a live run: the
status pill shows the current tool action with elapsed time, and the
finished turn folds into a "Worked · N tools" summary. Disable the
toggle and confirm the classic page is unchanged.
- Visual snapshot baselines are intentionally not updated: per
`doc/design/DECISION-SHEET.md`, "Per-change snapshot verification
demoted to dormant (Jul 13 2026)".

## Risks

- The flag-off path goes through the same `IssueDetail.tsx` file, so a
regression there would affect current users. Mitigation: the classic
markup renders through the same components as before behind explicit
flag conditionals, and the full UI suite passes.
- The transcript adapter interprets stored run-log formats, including
legacy entries without `toolUseId`. Malformed logs degrade to generic
tool rows rather than crashing.
- The new view changes no server behavior other than one additive
instance setting; it is additive and default-off. Overall risk with the
flag off is low.

## Model Used

- Claude (Anthropic), model id `claude-fable-5` (Claude Fable 5),
extended thinking enabled, agentic tool use (file editing, shell, test
execution) via Claude Code / Claude Agent SDK.

## Checklist

- [x] I have included a thinking path that traces from project context
to this change
- [x] I have specified the model used (with version and capability
details)
- [x] I have checked ROADMAP.md and confirmed this PR does not duplicate
planned core work
- [x] I have searched GitHub for duplicate or related PRs and linked
them above
- [x] I have either (a) linked existing issues with `Fixes: #` / `Closes
#` / `Refs #` OR (b) described the issue in-PR following the relevant
issue template
- [x] I have not referenced internal/instance-local Paperclip issues or
links (only public GitHub `#NNN` / `github.com/paperclipai/paperclip`
URLs)
- [x] My branch name describes the change (e.g. `docs/...`, `fix/...`)
and contains no internal Paperclip ticket id or instance-derived details
- [x] I have run tests locally and they pass
- [x] I have added or updated tests where applicable
- [x] I have updated relevant documentation to reflect my changes
- [x] I have considered and documented any risks above
- [ ] All Paperclip CI gates are green
- [ ] Greptile is 5/5 with no open P2s, recommendations, or follow-ups
- [x] I will address all Greptile and reviewer comments before
requesting merge

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
2026-08-01 02:26:47 -07:00

242 lines
9.1 KiB
TypeScript

import { useCallback, useEffect, useMemo, useRef, useState } from "react";
import { cn } from "@/lib/utils";
import { Copy, GripHorizontal, Minus, Plus, RotateCcw } from "lucide-react";
import {
EASING_PRESETS,
MOTION_TOKENS,
MOTION_TOKEN_EXPORT_FALLBACK,
MOTION_TOKEN_GROUPS,
parseCssTimeMs,
type MotionTokenDef,
} from "./motion-tokens";
const POS_KEY = "tc-tweak-pos";
const OVERRIDES_KEY = "tc-tweak-overrides";
function readComputed(name: string): string {
if (typeof document === "undefined") return "";
return getComputedStyle(document.documentElement).getPropertyValue(name).trim();
}
function loadSession<T>(key: string, fallback: T): T {
try {
const raw = sessionStorage.getItem(key);
return raw ? (JSON.parse(raw) as T) : fallback;
} catch {
return fallback;
}
}
function saveSession(key: string, value: unknown) {
try {
sessionStorage.setItem(key, JSON.stringify(value));
} catch {
/* session storage unavailable — panel still works, just not persisted */
}
}
/**
* Hand-rolled motion tweak panel (dev/flag only). Floating, draggable,
* minimizable overlay that lists every motion token, writes changes live to the
* CSS custom properties (no reload), exports a paste-ready :root block, and
* persists position + overrides for the session only. No third-party GUI lib —
* one file, one dev guard, trivial to strip out later.
*/
export function TweakPanel() {
const [pos, setPos] = useState<{ x: number; y: number }>(() =>
loadSession(POS_KEY, { x: 24, y: 24 }),
);
const [minimized, setMinimized] = useState(false);
const [values, setValues] = useState<Record<string, string>>({});
const [overrides, setOverrides] = useState<Record<string, string>>(() =>
loadSession(OVERRIDES_KEY, {}),
);
const [exportText, setExportText] = useState<string | null>(null);
const dragRef = useRef<{ dx: number; dy: number } | null>(null);
// Initialize display values from computed styles, then re-apply any saved
// session overrides so a reload-free session keeps its tuning.
useEffect(() => {
const initial: Record<string, string> = {};
for (const t of MOTION_TOKENS) initial[t.name] = readComputed(t.name);
for (const [name, value] of Object.entries(overrides)) {
document.documentElement.style.setProperty(name, value);
initial[name] = value;
}
setValues(initial);
// eslint-disable-next-line react-hooks/exhaustive-deps
}, []);
const setToken = useCallback((name: string, value: string) => {
document.documentElement.style.setProperty(name, value);
setValues((v) => ({ ...v, [name]: value }));
setOverrides((o) => {
const next = { ...o, [name]: value };
saveSession(OVERRIDES_KEY, next);
return next;
});
}, []);
const resetAll = useCallback(() => {
for (const name of Object.keys(overrides)) {
document.documentElement.style.removeProperty(name);
}
setOverrides({});
saveSession(OVERRIDES_KEY, {});
const refreshed: Record<string, string> = {};
for (const t of MOTION_TOKENS) refreshed[t.name] = readComputed(t.name);
setValues(refreshed);
}, [overrides]);
const onHeaderPointerDown = useCallback(
(e: React.PointerEvent) => {
(e.target as HTMLElement).setPointerCapture(e.pointerId);
dragRef.current = { dx: e.clientX - pos.x, dy: e.clientY - pos.y };
},
[pos],
);
const onHeaderPointerMove = useCallback((e: React.PointerEvent) => {
if (!dragRef.current) return;
const next = { x: e.clientX - dragRef.current.dx, y: e.clientY - dragRef.current.dy };
setPos(next);
}, []);
const onHeaderPointerUp = useCallback(() => {
if (dragRef.current) {
dragRef.current = null;
saveSession(POS_KEY, pos);
}
}, [pos]);
const buildExport = useCallback(() => {
const lines = MOTION_TOKENS.map((t) => {
const val = values[t.name] || readComputed(t.name);
return ` ${t.name}: ${val};`;
});
setExportText(`:root {\n${lines.join("\n")}\n}\n`);
}, [values]);
const grouped = useMemo(() => {
const map: Record<string, MotionTokenDef[]> = {};
for (const g of MOTION_TOKEN_GROUPS) map[g] = [];
for (const t of MOTION_TOKENS) map[t.group].push(t);
return map;
}, []);
return (
<div
className="fixed z-50 w-72 select-none rounded-lg border border-border bg-card text-card-foreground shadow-lg"
style={{ left: pos.x, top: pos.y }}
data-testid="task-chat-tweak-panel"
>
<div
className="flex cursor-grab items-center gap-2 rounded-t-lg border-b border-border bg-muted/50 px-2 py-1.5 text-xs font-medium active:cursor-grabbing"
onPointerDown={onHeaderPointerDown}
onPointerMove={onHeaderPointerMove}
onPointerUp={onHeaderPointerUp}
>
<GripHorizontal className="h-3.5 w-3.5 text-muted-foreground" />
<span>Motion tweak panel</span>
<div className="ml-auto flex items-center gap-1">
<button type="button" title="Reset all" onClick={resetAll} className="rounded p-0.5 hover:bg-accent">
<RotateCcw className="h-3.5 w-3.5" />
</button>
<button
type="button"
title={minimized ? "Expand" : "Minimize"}
onClick={() => setMinimized((m) => !m)}
className="rounded p-0.5 hover:bg-accent"
>
{minimized ? <Plus className="h-3.5 w-3.5" /> : <Minus className="h-3.5 w-3.5" />}
</button>
</div>
</div>
{minimized ? null : (
<div className="max-h-(--sz-tweak-panel-max) overflow-y-auto p-2">
{MOTION_TOKEN_GROUPS.map((group) => (
<div key={group} className="mb-3">
<p className="mb-1 text-(length:--text-nano) font-semibold uppercase tracking-wide text-muted-foreground">{group}</p>
<div className="flex flex-col gap-2">
{grouped[group].map((t) => (
<div key={t.name} data-token={t.name} className="flex flex-col gap-1">
<div className="flex items-center justify-between text-(length:--text-micro)">
<span className="truncate font-mono text-muted-foreground">{t.name.replace("--motion-", "")}</span>
<span className="ml-2 shrink-0 tabular-nums">
{t.kind === "time" ? `${Math.round(parseCssTimeMs(values[t.name] ?? "0"))}ms` : ""}
</span>
</div>
{t.kind === "time" ? (
<input
type="range"
aria-label={t.name}
min={t.min ?? 0}
max={t.max ?? 1000}
step={t.step ?? 10}
value={Math.round(parseCssTimeMs(values[t.name] ?? "0"))}
onChange={(e) => setToken(t.name, `${e.target.value}ms`)}
className="h-1.5 w-full"
/>
) : (
<select
aria-label={t.name}
value={values[t.name] ?? ""}
onChange={(e) => setToken(t.name, e.target.value)}
className="w-full rounded border border-border bg-background px-1 py-0.5 text-(length:--text-micro)"
>
{EASING_PRESETS.every((p) => p.value !== (values[t.name] ?? "")) && values[t.name] ? (
<option value={values[t.name]}>{values[t.name]}</option>
) : null}
{EASING_PRESETS.map((p) => (
<option key={p.value} value={p.value}>
{p.label}
</option>
))}
</select>
)}
</div>
))}
</div>
</div>
))}
<div className="mt-1 flex gap-2">
<button
type="button"
onClick={buildExport}
className="flex flex-1 items-center justify-center gap-1 rounded border border-border px-2 py-1 text-xs hover:bg-accent"
>
<Copy className="h-3.5 w-3.5" />
Copy as @theme
</button>
</div>
{exportText ? (
<textarea
readOnly
data-testid="task-chat-tweak-export"
className="mt-2 h-32 w-full resize-none rounded border border-border bg-muted/40 p-2 font-mono text-(length:--text-nano)"
value={exportText}
onFocus={(e) => {
e.currentTarget.select();
void navigator.clipboard?.writeText(exportText).catch(() => {});
}}
/>
) : null}
</div>
)}
</div>
);
}
/**
* Build the :root export string from arbitrary token values. Exposed for the
* finish-line test that asserts export produces valid CSS. Kept in sync with
* the panel's own export.
*/
export function buildTweakExport(values: Record<string, string>): string {
const lines = MOTION_TOKENS.map((t) => ` ${t.name}: ${values[t.name] ?? MOTION_TOKEN_EXPORT_FALLBACK};`);
return `:root {\n${lines.join("\n")}\n}\n`;
}