toolroll 0.5.0

package/dist/chat-inbox.d.ts

@@ -0,0 +1,57 @@
+import { type ChatApp } from "./flow-triggers.js";
+import { type ChatBinding } from "./chat-delivery-state.js";
+import type { FlowCardSource, FlowTriggerRow, Store } from "./store.js";
+/** "flow 12" or "flow off", after any mention of the bot; Telegram's "/flow 12" or "/flow@bot 12" too. */
+export declare const FLOW_WORDS: RegExp;
+/** A channel's own id: Teams puts the thread's message id on the end of a channel conversation. */
+export declare const channelOf: (app: ChatApp, conversation: string) => string;
+/** The flow a channel feeds, if it feeds one. */
+export declare function watchedChannel(store: Store, app: ChatApp, installation: string, conversation: string): FlowTriggerRow | null;
+/** What a receiver needs to know about a channel message: whether the channel feeds a flow, and whether these are the words that connect one. */
+export declare function channelInbox(store: Store, app: ChatApp, installation: string, conversation: string, text: string): {
+    watched: boolean;
+    command: boolean;
+};
+/** "flow 12" / "flow off" from a paired approver in a channel: connect it to the flow, or stop it feeding one. */
+export declare function connectChannel(store: Store, input: {
+    app: ChatApp;
+    installation: string;
+    conversation: string;
+    binding: Pick<ChatBinding, "id" | "approver">;
+    text: string;
+    repos: readonly string[];
+    followsConversation: boolean;
+}, now: Date): string;
+/**
+ * A message in a channel that feeds a flow: a new card (and what to say in
+ * its thread), a reply that joins a card's discussion (nothing to say), or
+ * why it was left out.
+ */
+export declare function takeChannelMessage(store: Store, trigger: FlowTriggerRow, message: {
+    app: ChatApp;
+    conversation: string;
+    ts: string;
+    thread: string;
+    text: string;
+    who: string;
+}, now: Date): {
+    said: string | null;
+    link?: {
+        label: string;
+        path: string;
+    };
+};
+/**
+ * Answer in the thread a card came from (an Update zone): one message,
+ * through the chat of whoever connected the channel. Idempotent per card
+ * visit, so a retried step never answers twice.
+ */
+export declare function replyInChannel(store: Store, source: NonNullable<FlowCardSource["chat"]>, visit: {
+    card: number;
+    entry: number;
+}, text: string, now: Date): {
+    ok: true;
+} | {
+    ok: false;
+    said: string;
+};

package/dist/chat-inbox.js

@@ -0,0 +1,123 @@
+/**
+ * Chat channels as inboxes (v89): a Slack, Discord or Teams channel feeds a
+ * flow.
+ *
+ * - Connecting happens in the channel itself: a paired approver sends
+ *   "flow 12" (or "flow off"), which proves the channel is one they are in
+ *   and that the flow is one they can change. One channel feeds one flow; a
+ *   channel that follows a team conversation is left as it is.
+ * - From then on every new message there — from anyone the app lets the bot
+ *   see — is a card in that flow, and the bot says so in the message's
+ *   thread. A reply in that thread joins the card's discussion.
+ * - The card remembers where it came from, so an Update zone can answer in
+ *   that same thread, through the chat of the person who connected it.
+ *
+ * Nothing here gives a message any authority: a card's work waits for the
+ * usual approvals, and text that looks like a key never becomes a card.
+ */
+import { flowCardHref, flowDefinitionOf } from "./flow-engine.js";
+import { addTriggerCard, CHAT_APP_NAMES, triggerConfigOf } from "./flow-triggers.js";
+import { ChatState, chatHash } from "./chat-delivery-state.js";
+/** "flow 12" or "flow off", after any mention of the bot; Telegram's "/flow 12" or "/flow@bot 12" too. */
+export const FLOW_WORDS = /^(?:<@[^>]+>\s*)?\/?flow(?:@[A-Za-z0-9_]{1,64})?\s+(off|[1-9][0-9]{0,9})\s*$/i;
+const TITLE_CHARS = 120;
+/** A channel's own id: Teams puts the thread's message id on the end of a channel conversation. */
+export const channelOf = (app, conversation) => app === "teams" ? conversation.replace(/;messageid=[0-9]+$/, "") : conversation;
+/** The flow a channel feeds, if it feeds one. */
+export function watchedChannel(store, app, installation, conversation) {
+    return store.chatTriggerFor(app, installation, channelOf(app, conversation));
+}
+/** What a receiver needs to know about a channel message: whether the channel feeds a flow, and whether these are the words that connect one. */
+export function channelInbox(store, app, installation, conversation, text) {
+    return { watched: watchedChannel(store, app, installation, conversation) !== null, command: FLOW_WORDS.test(text.trim()) };
+}
+/** "flow 12" / "flow off" from a paired approver in a channel: connect it to the flow, or stop it feeding one. */
+export function connectChannel(store, input, now) {
+    const words = FLOW_WORDS.exec(input.text.trim());
+    if (words === null)
+        return "Send “flow” and a flow's number, like “flow 12”, or “flow off”.";
+    const chat = channelOf(input.app, input.conversation);
+    const current = store.chatTriggerFor(input.app, input.installation, chat);
+    const name = CHAT_APP_NAMES[input.app];
+    if (words[1].toLowerCase() === "off") {
+        if (current === null)
+            return "This channel doesn't feed a flow.";
+        const flow = store.getFlow(current.flow);
+        if (flow !== null && !input.repos.includes(flow.repo))
+            return "Only someone who can change that flow can stop this channel feeding it.";
+        store.updateFlowTrigger(current.id, { state: "removed", lastAt: now.toISOString(), lastOutcome: `Disconnected in ${name} by ${input.binding.approver}.` }, now);
+        return `This channel no longer feeds ${flow?.name ?? "its flow"}.`;
+    }
+    if (input.followsConversation)
+        return "This channel follows a team conversation. Send “team off” first to make it feed a flow instead.";
+    const flow = store.getFlow(Number(words[1]));
+    if (flow === null || flow.state !== "active" || !input.repos.includes(flow.repo) || flowDefinitionOf(flow) === null)
+        return `There's no flow ${words[1]} you can change. The number is in the flow's address: /flows/12.`;
+    if (current !== null && current.flow === flow.id)
+        return `This channel already feeds ${flow.name}.`;
+    store.transact(() => {
+        if (current !== null)
+            store.updateFlowTrigger(current.id, { state: "removed", lastAt: now.toISOString(), lastOutcome: `Moved to ${flow.name} by ${input.binding.approver}.` }, now);
+        const id = store.addFlowTrigger({ flow: flow.id, kind: "chat", hookHash: null, cursor: null, nextAt: null, by: input.binding.approver,
+            configJson: JSON.stringify({ kind: "chat", app: input.app, installation: input.installation, chat, binding: input.binding.id, zone: null }) }, now);
+        store.updateFlowTrigger(id, { lastAt: now.toISOString(), lastOutcome: `Connected in ${name} by ${input.binding.approver}.` }, now);
+    });
+    return `This channel now feeds ${flow.name}: each new message here becomes a card, and replies in its thread join the card's discussion. Send “flow off” to stop.`;
+}
+/** A message's words as a card: the first line is the title, all of it the details when there's more. */
+function cardWords(text) {
+    const clean = text.replace(/<@[A-Z0-9]+>/g, "").replace(/\r\n?/g, "\n").trim();
+    const first = clean.split("\n")[0].trim();
+    const title = first.length > TITLE_CHARS ? `${first.slice(0, TITLE_CHARS - 1).trimEnd()}…` : first;
+    return { title: title || "Message", description: clean === title ? null : clean.slice(0, 4000) };
+}
+/**
+ * A message in a channel that feeds a flow: a new card (and what to say in
+ * its thread), a reply that joins a card's discussion (nothing to say), or
+ * why it was left out.
+ */
+export function takeChannelMessage(store, trigger, message, now) {
+    const config = triggerConfigOf(trigger);
+    if (config?.kind !== "chat")
+        return { said: null };
+    const name = CHAT_APP_NAMES[message.app];
+    if (message.text.trim() === "")
+        return { said: null };
+    if (message.thread !== message.ts) {
+        // A reply in the thread of a message that became a card: it joins that card's discussion.
+        const card = store.flowTriggerCard(trigger.id, `msg:${message.thread}`);
+        if (card !== null && store.getFlowCard(card)?.state === "active")
+            store.addFlowComment({ card, author: `${message.who} (${name})`.slice(0, 64), body: message.text.slice(0, 2000), mentions: [] }, now);
+        return { said: null };
+    }
+    const words = cardWords(message.text);
+    const source = { kind: "chat", label: `${name} message`, url: null,
+        chat: { app: message.app, installation: config.installation, chat: config.chat, conversation: message.conversation, thread: message.ts, binding: config.binding } };
+    const made = addTriggerCard(store, trigger, { key: `msg:${message.ts}`, title: words.title, description: words.description, source }, name, now);
+    const flow = store.getFlow(trigger.flow);
+    if (made.made === "added") {
+        store.updateFlowTrigger(trigger.id, { lastAt: now.toISOString(), lastOutcome: "Added a card." }, now);
+        return { said: `Added to ${flow?.name ?? "the flow"} as a card. Replies here join its discussion.`, ...(flow === null ? {} : { link: { label: "Open the card", path: flowCardHref(flow.id, made.card) } }) };
+    }
+    return { said: made.made === "skipped" && made.note !== null ? `Not added: ${made.note.replace(/^[^:]+: /, "")}.` : null };
+}
+/**
+ * Answer in the thread a card came from (an Update zone): one message,
+ * through the chat of whoever connected the channel. Idempotent per card
+ * visit, so a retried step never answers twice.
+ */
+export function replyInChannel(store, source, visit, text, now) {
+    if (source.app !== "slack" && source.app !== "discord" && source.app !== "teams")
+        return { ok: false, said: `Answering in a ${CHAT_APP_NAMES[source.app] ?? source.app} group isn't supported yet, so nothing was posted. The card's link is in the group.` };
+    const state = new ChatState(store, source.app);
+    const grant = state.bindingById(source.binding);
+    if (grant === null || !state.live(grant))
+        return { ok: false, said: `The person who connected that ${CHAT_APP_NAMES[source.app]} channel is no longer paired, so nothing was posted.` };
+    const id = chatHash(`${source.app}:flow-reply:${visit.card}:${visit.entry}`);
+    store.transact(() => {
+        if (state.enqueue({ id, installation: grant.installation, binding: grant.id, kind: "message", channel: source.conversation, member: grant.member,
+            ts: source.thread, thread: source.thread, payload: "{}", created: now.toISOString() }))
+            state.plan(id, [{ text: text.slice(0, 3500), channel: source.conversation }], now);
+    });
+    return { ok: true };
+}

package/dist/chat-polish.d.ts

@@ -0,0 +1,16 @@
+import type { ChatSnapshot } from './store.js';
+/** Presentation only. These are recorded completions, not a claim that
+ * work happened since the user last visited, or that every result passed. */
+export declare function completedWorkHtml(snapshot: ChatSnapshot, projectLabels: readonly string[], assignments?: Readonly<Record<string, {
+    state: string;
+}>>): string;
+/** Keep activity truthful: the turn's recorded step count is not a
+ * percentage or an invented tool phase. Billing detail stays available. */
+export declare function chatWorkingHtml(input: {
+    details: string;
+    stopForm?: string;
+    keyed?: boolean;
+}): string;
+/** Recorded activity is inspectable, not a row of performance badges. */
+export declare function chatActivityDetailsHtml(activity: string | null): string;
+export declare const CHAT_POLISH_CSS = "\n  /* Quiet workspace surfaces: color signals state, not an AI aesthetic. */\n  body { background: var(--background); }\n  .chat-workspace .chat-overview, .chat-workspace .chat-plan, .chat-workspace .result-panel,\n  .chat-workspace .decide-card { background: var(--card); box-shadow: none; }\n  .chat-workspace .result-panel { border-radius: .75rem; }\n  .chat-workspace .chat-empty::before { content: none; }\n  .chat-workspace .thread .msg.mate::before { background: var(--muted); box-shadow: none; border: 0; }\n  .chat-workspace .thread .msg.op { box-shadow: none; border: 0; }\n  .chat-workspace .completion-receipt { background: var(--card); box-shadow: none; }\n  .chat-workspace .completion-receipt::after { content: none; }\n  .chat-workspace .thread { min-height: 0; }\n  .chat-workspace .chat-turn-history { margin: 1rem 0; border: 0; border-top: 1px solid var(--border); border-radius: 0; background: transparent; box-shadow: none; padding: 0; }\n  .chat-turn-history > summary { display: flex; align-items: center; gap: .75rem; min-height: 44px; cursor: pointer; font-size: .875rem; }\n  .chat-turn-history > summary .meta { margin-left: auto; }\n  .chat-turn-history .row { display: flex; flex-wrap: wrap; gap: .4rem; }\n  .chat-turn-history .row .right { float: none; margin-left: auto; }\n  .chat-workspace .task-live-summary { margin-bottom: .75rem; padding-bottom: .25rem; }\n  .chat-workspace .task-journey > details, .chat-workspace .result-feedback-history,\n  .chat-workspace .chat-limits, .chat-workspace .task-chat-agents {\n    border: 0; border-radius: 0; background: transparent; box-shadow: none; padding: 0; margin: .25rem 0;\n  }\n  .chat-workspace .task-chat-agents > summary { min-height: 44px; cursor: pointer; font-size: .8125rem; color: var(--muted-foreground); }\n  .chat-workspace .task-chat-agents p { margin: 0 0 .5rem; }\n  .chat-workspace .decide-options .decide-option { border: 0; border-top: 1px solid var(--border); border-radius: 0; background: transparent; padding: .75rem 0; }\n  .chat-workspace .chat-plan { padding: 1rem; margin-top: .5rem; }\n  .chat-workspace .result-knowledge { margin-bottom: .5rem; }\n  .chat-workspace .result-panel [data-result-view] > details { margin-block: 0; padding-block: 0; }\n  .chat-workspace .result-panel [data-result-view] > details > summary { min-height: 44px; display: flex; align-items: center; }\n  .chat-workspace .result-feedback-history > summary { min-height: 44px; }\n  .chat-activity-details { border: 0; background: transparent; box-shadow: none; border-radius: 0; padding: 0; margin: 0; }\n  .chat-activity-details > summary { min-height: 44px; display: flex; align-items: center; gap: .4rem; cursor: pointer; color: var(--muted-foreground); font-size: .75rem; }\n  .chat-activity-details > summary::after { content: '\u2304'; }\n  .chat-activity-details[open] > summary::after { transform: rotate(180deg); }\n  .chat-activity-details .chat-activity { gap: .25rem .75rem; }\n  .chat-activity-details .chat-activity span { border: 0; padding: 0; border-radius: 0; font: inherit; font-size: .75rem; }\n  .chat-workspace .chat-message-foot { justify-content: flex-start; align-items: baseline; margin-top: .25rem; }\n  .chat-activity-row { border: 0; background: transparent; box-shadow: none; padding: .5rem 0; min-height: 44px; gap: .65rem; }\n  .chat-activity-row details { flex: 1; min-width: 0; border: 0; background: transparent; padding: 0; margin: 0; box-shadow: none; }\n  .chat-activity-row summary { min-height: 44px; display: flex; align-items: center; cursor: pointer; width: fit-content; font-size: .875rem; font-weight: 550; }\n  .chat-activity-row summary::after { content: '\u2304'; margin-left: .65rem; color: var(--muted-foreground); }\n  .chat-activity-row details[open] summary::after { transform: rotate(180deg); }\n  .chat-activity-row .meta { padding-bottom: .5rem; overflow-wrap: anywhere; }\n  .activity-dot { width: 6px; height: 6px; border-radius: 50%; background: currentColor; flex: none; }\n  .task-live-summary { border-bottom: 1px solid var(--border); margin-bottom: 1rem; padding-bottom: .5rem; }\n  .task-live-summary .task-journey, .task-live-summary .task-control { border: 0; border-radius: 0; box-shadow: none; background: transparent; margin: 0; padding: .65rem 0; }\n  .task-live-summary .task-journey h2 { font-size: 1rem; }\n  .task-live-summary .agents-strip { margin: .4rem 0; }\n  .chat-decisions .chat-section-head { margin-bottom: .5rem; }\n  .chat-decisions h2 { font-size: .875rem; font-weight: 550; color: var(--muted-foreground); }\n  .decision-context summary { cursor: pointer; min-height: 44px; display: flex; align-items: center; font-size: .8125rem; color: var(--muted-foreground); }\n  .decision-context { border: 0; padding: 0; margin: 0; background: transparent; box-shadow: none; }\n  .decide-options { display: grid; gap: .5rem; }\n  .decide-options .decide-option { margin: 0; padding: .65rem .75rem; border: 1px solid var(--border); border-radius: .75rem; background: var(--background); }\n  .decide-options .decide-option .meta { display: block; margin-top: .3rem; overflow-wrap: anywhere; }\n  .decide-options .decide-option button { min-height: 44px; max-width: 100%; white-space: normal; text-align: left; }\n  .chat-completed { margin-top: 1rem; }\n  .chat-completed h3 { font-size: .8125rem; font-weight: 550; margin: 0 0 .25rem; color: var(--muted-foreground); }\n  .chat-completed-action { flex: none; font-size: .75rem; white-space: nowrap; color: var(--muted-foreground); }\n  .chat-overview .chat-overview-stat { box-shadow: none; border: 0; background: transparent; border-radius: 0; }\n  .chat-overview .chat-overview-stat + .chat-overview-stat { border-left: 1px solid var(--border); }\n  .result-panel .result-request { border-top: 1px solid var(--border); padding-top: 1rem; }\n  .result-panel .result-head h2 { color: var(--foreground); }\n  .result-panel .result-request h3 { color: var(--foreground); font-size: .875rem; text-transform: none; letter-spacing: 0; }\n  .result-panel .result-feedback-actions { display: flex; justify-content: flex-end; align-items: center; flex-wrap: wrap; gap: .5rem; }\n  .result-panel .result-feedback-actions > [data-request-changes] { order: 2; background: var(--foreground); color: var(--background); border-color: var(--foreground); box-shadow: none; }\n  .result-panel .result-feedback-actions > button { width: auto; min-height: 44px; white-space: nowrap; }\n  .result-panel .result-feedback-actions > .quiet { background: transparent; border-color: transparent; box-shadow: none; }\n  .result-panel .result-feedback-actions > button:disabled { opacity: .45; cursor: default; }\n  .result-panel .result-feedback-link { display: inline-flex; align-items: center; min-height: 44px; font-size: .8125rem; text-underline-offset: 3px; }\n  .result-panel .result-pin > summary, .result-panel .result-notes > summary, .result-panel .result-details > summary { min-height: 44px; }\n  .result-panel .diff-comment-form textarea { min-height: 80px; margin: 0; font-size: 1rem; }\n  .result-panel .result-attention { background: transparent; border: 0; border-radius: 0; box-shadow: none; padding: 0; margin: .75rem 0; color: var(--foreground); }\n  .result-panel .result-attention ul { margin: 0; padding-left: 1.15rem; }\n  .result-panel .result-attention ul:has(> li:only-child) { list-style: none; padding-left: 0; }\n  .result-panel .result-knowledge { border: 0; border-top: 1px solid var(--border); background: transparent; box-shadow: none; border-radius: 0; padding: .25rem 0; }\n  .result-panel .revision-from-comments { border: 0; background: transparent; box-shadow: none; padding: .5rem 0; }\n  .result-panel .result-pin summary { font-size: .8125rem; min-height: 44px; display: flex; align-items: center; cursor: pointer; }\n  .result-panel .diff-comment-limit { font-size: .75rem; }\n  .chat-main .chat-plan { box-shadow: none; }\n  .proposal-review-notes { margin: .5rem 0; padding-left: 1.15rem; overflow-wrap: anywhere; }\n  .proposal-review-notes li + li { margin-top: .5rem; }\n  .proposal-actions .button-link, .proposal-actions button { min-height: 44px; white-space: nowrap; }\n  .chat-stop-confirm > summary { min-height: 44px; display: flex; align-items: center; cursor: pointer; }\n  .task-chat-title-line h1, .task-main-title, .resume-target, .resume-ceremony, .proposal-task_action, .chat-stop-confirm { overflow-wrap: anywhere; }\n  .chat-main .chat-task-back { display: none; }\n  .knowledge-library .knowledge-intro { padding: .5rem 0 1.25rem; margin-bottom: 1.5rem; border-bottom: 1px solid var(--border); }\n  .knowledge-library .knowledge-instructions { line-height: 1.65; margin-bottom: .5rem; }\n  .knowledge-library .knowledge-editor summary { width: fit-content; font-size: .875rem; font-weight: 550; }\n  .knowledge-library .knowledge-editor, .knowledge-library > details { border: 0; border-bottom: 1px solid var(--border); border-radius: 0; box-shadow: none; background: transparent; padding: 0; }\n  .knowledge-library .knowledge-editor { border-bottom: 0; }\n  .knowledge-library .card { border: 0; border-bottom: 1px solid var(--border); border-radius: 0; padding: 0 0 .5rem; margin: 0; box-shadow: none; background: transparent; }\n  .knowledge-library .card[open] { padding-bottom: 1rem; }\n  .knowledge-library .card > summary { padding: .85rem 0; }\n  @media (prefers-reduced-motion: no-preference) {\n    .chat-activity-row .activity-dot { animation: quiet-activity 1.8s ease-in-out infinite; }\n    .chat-completed .chat-overview-item { transition: background-color 140ms ease; }\n    @keyframes quiet-activity { 50% { opacity: .35; } }\n  }\n  @media (max-width: 600px) {\n    .chat-workspace .result-panel { background: transparent; border-radius: 0; }\n    .chat-completed .chat-overview-item { align-items: flex-start; flex-wrap: wrap; gap: .35rem; }\n    .chat-completed .chat-overview-copy { flex-basis: 65%; }\n    .task-live-summary .task-journey { gap: .5rem; }\n  }\n";

package/dist/chat-polish.js

@@ -0,0 +1,135 @@
+const escape = (value) => value.replace(/[&<>"']/g, char => ({ '&': '&amp;', '<': '&lt;', '>': '&gt;', '"': '&quot;', "'": '&#39;' }[char]));
+/** Presentation only. These are recorded completions, not a claim that
+ * work happened since the user last visited, or that every result passed. */
+export function completedWorkHtml(snapshot, projectLabels, assignments) {
+    const completed = snapshot.tasks.filter(task => assignments === undefined ? task.state === 'done'
+        && task.proofVerdict !== 'short' && task.proofVerdict !== 'refuted'
+        && !task.historyProblem && !task.otherActive?.length
+        && task.dispatch?.condition !== 'waiting' : assignments[task.id]?.state === 'complete').slice(0, 3);
+    if (!completed.length)
+        return '';
+    return `<section class="chat-completed" aria-label="Completed work"><h3>Completed work</h3>` + completed.map(task => {
+        return `<a class="chat-overview-item" href="/t/${encodeURIComponent(task.rootId ?? task.id)}"><span class="chat-overview-copy"><strong>${escape(task.title)}</strong><span>${escape(projectLabels[task.repoIndex] ?? `Project ${task.repoIndex + 1}`)}</span></span><span class="chat-completed-action">Open task <span aria-hidden="true">→</span></span></a>`;
+    }).join('') + '</section>';
+}
+/** Keep activity truthful: the turn's recorded step count is not a
+ * percentage or an invented tool phase. Billing detail stays available. */
+export function chatWorkingHtml(input) {
+    return `<div class="chat-thinking chat-activity-row" id="latest" role="status"${input.keyed ? ' data-key="pending"' : ''}>` +
+        `<span class="activity-dot" aria-hidden="true"></span><details><summary>Working<span class="activity-ellipsis" aria-hidden="true">…</span></summary><p class="meta">${escape(input.details)}</p></details>` +
+        (input.stopForm ?? '') + '</div>';
+}
+/** Recorded activity is inspectable, not a row of performance badges. */
+export function chatActivityDetailsHtml(activity) {
+    if (!activity)
+        return '';
+    return `<details class="chat-activity-details"><summary>Activity</summary><div class="chat-activity" aria-label="work performed">${activity.split(' · ').map(one => `<span>${escape(one)}</span>`).join('')}</div></details>`;
+}
+// Local primitives adapted from the researched interaction patterns. No
+// imported kit, animation runtime, simulated progress, or changed authority.
+export const CHAT_POLISH_CSS = `
+  /* Quiet workspace surfaces: color signals state, not an AI aesthetic. */
+  body { background: var(--background); }
+  .chat-workspace .chat-overview, .chat-workspace .chat-plan, .chat-workspace .result-panel,
+  .chat-workspace .decide-card { background: var(--card); box-shadow: none; }
+  .chat-workspace .result-panel { border-radius: .75rem; }
+  .chat-workspace .chat-empty::before { content: none; }
+  .chat-workspace .thread .msg.mate::before { background: var(--muted); box-shadow: none; border: 0; }
+  .chat-workspace .thread .msg.op { box-shadow: none; border: 0; }
+  .chat-workspace .completion-receipt { background: var(--card); box-shadow: none; }
+  .chat-workspace .completion-receipt::after { content: none; }
+  .chat-workspace .thread { min-height: 0; }
+  .chat-workspace .chat-turn-history { margin: 1rem 0; border: 0; border-top: 1px solid var(--border); border-radius: 0; background: transparent; box-shadow: none; padding: 0; }
+  .chat-turn-history > summary { display: flex; align-items: center; gap: .75rem; min-height: 44px; cursor: pointer; font-size: .875rem; }
+  .chat-turn-history > summary .meta { margin-left: auto; }
+  .chat-turn-history .row { display: flex; flex-wrap: wrap; gap: .4rem; }
+  .chat-turn-history .row .right { float: none; margin-left: auto; }
+  .chat-workspace .task-live-summary { margin-bottom: .75rem; padding-bottom: .25rem; }
+  .chat-workspace .task-journey > details, .chat-workspace .result-feedback-history,
+  .chat-workspace .chat-limits, .chat-workspace .task-chat-agents {
+    border: 0; border-radius: 0; background: transparent; box-shadow: none; padding: 0; margin: .25rem 0;
+  }
+  .chat-workspace .task-chat-agents > summary { min-height: 44px; cursor: pointer; font-size: .8125rem; color: var(--muted-foreground); }
+  .chat-workspace .task-chat-agents p { margin: 0 0 .5rem; }
+  .chat-workspace .decide-options .decide-option { border: 0; border-top: 1px solid var(--border); border-radius: 0; background: transparent; padding: .75rem 0; }
+  .chat-workspace .chat-plan { padding: 1rem; margin-top: .5rem; }
+  .chat-workspace .result-knowledge { margin-bottom: .5rem; }
+  .chat-workspace .result-panel [data-result-view] > details { margin-block: 0; padding-block: 0; }
+  .chat-workspace .result-panel [data-result-view] > details > summary { min-height: 44px; display: flex; align-items: center; }
+  .chat-workspace .result-feedback-history > summary { min-height: 44px; }
+  .chat-activity-details { border: 0; background: transparent; box-shadow: none; border-radius: 0; padding: 0; margin: 0; }
+  .chat-activity-details > summary { min-height: 44px; display: flex; align-items: center; gap: .4rem; cursor: pointer; color: var(--muted-foreground); font-size: .75rem; }
+  .chat-activity-details > summary::after { content: '⌄'; }
+  .chat-activity-details[open] > summary::after { transform: rotate(180deg); }
+  .chat-activity-details .chat-activity { gap: .25rem .75rem; }
+  .chat-activity-details .chat-activity span { border: 0; padding: 0; border-radius: 0; font: inherit; font-size: .75rem; }
+  .chat-workspace .chat-message-foot { justify-content: flex-start; align-items: baseline; margin-top: .25rem; }
+  .chat-activity-row { border: 0; background: transparent; box-shadow: none; padding: .5rem 0; min-height: 44px; gap: .65rem; }
+  .chat-activity-row details { flex: 1; min-width: 0; border: 0; background: transparent; padding: 0; margin: 0; box-shadow: none; }
+  .chat-activity-row summary { min-height: 44px; display: flex; align-items: center; cursor: pointer; width: fit-content; font-size: .875rem; font-weight: 550; }
+  .chat-activity-row summary::after { content: '⌄'; margin-left: .65rem; color: var(--muted-foreground); }
+  .chat-activity-row details[open] summary::after { transform: rotate(180deg); }
+  .chat-activity-row .meta { padding-bottom: .5rem; overflow-wrap: anywhere; }
+  .activity-dot { width: 6px; height: 6px; border-radius: 50%; background: currentColor; flex: none; }
+  .task-live-summary { border-bottom: 1px solid var(--border); margin-bottom: 1rem; padding-bottom: .5rem; }
+  .task-live-summary .task-journey, .task-live-summary .task-control { border: 0; border-radius: 0; box-shadow: none; background: transparent; margin: 0; padding: .65rem 0; }
+  .task-live-summary .task-journey h2 { font-size: 1rem; }
+  .task-live-summary .agents-strip { margin: .4rem 0; }
+  .chat-decisions .chat-section-head { margin-bottom: .5rem; }
+  .chat-decisions h2 { font-size: .875rem; font-weight: 550; color: var(--muted-foreground); }
+  .decision-context summary { cursor: pointer; min-height: 44px; display: flex; align-items: center; font-size: .8125rem; color: var(--muted-foreground); }
+  .decision-context { border: 0; padding: 0; margin: 0; background: transparent; box-shadow: none; }
+  .decide-options { display: grid; gap: .5rem; }
+  .decide-options .decide-option { margin: 0; padding: .65rem .75rem; border: 1px solid var(--border); border-radius: .75rem; background: var(--background); }
+  .decide-options .decide-option .meta { display: block; margin-top: .3rem; overflow-wrap: anywhere; }
+  .decide-options .decide-option button { min-height: 44px; max-width: 100%; white-space: normal; text-align: left; }
+  .chat-completed { margin-top: 1rem; }
+  .chat-completed h3 { font-size: .8125rem; font-weight: 550; margin: 0 0 .25rem; color: var(--muted-foreground); }
+  .chat-completed-action { flex: none; font-size: .75rem; white-space: nowrap; color: var(--muted-foreground); }
+  .chat-overview .chat-overview-stat { box-shadow: none; border: 0; background: transparent; border-radius: 0; }
+  .chat-overview .chat-overview-stat + .chat-overview-stat { border-left: 1px solid var(--border); }
+  .result-panel .result-request { border-top: 1px solid var(--border); padding-top: 1rem; }
+  .result-panel .result-head h2 { color: var(--foreground); }
+  .result-panel .result-request h3 { color: var(--foreground); font-size: .875rem; text-transform: none; letter-spacing: 0; }
+  .result-panel .result-feedback-actions { display: flex; justify-content: flex-end; align-items: center; flex-wrap: wrap; gap: .5rem; }
+  .result-panel .result-feedback-actions > [data-request-changes] { order: 2; background: var(--foreground); color: var(--background); border-color: var(--foreground); box-shadow: none; }
+  .result-panel .result-feedback-actions > button { width: auto; min-height: 44px; white-space: nowrap; }
+  .result-panel .result-feedback-actions > .quiet { background: transparent; border-color: transparent; box-shadow: none; }
+  .result-panel .result-feedback-actions > button:disabled { opacity: .45; cursor: default; }
+  .result-panel .result-feedback-link { display: inline-flex; align-items: center; min-height: 44px; font-size: .8125rem; text-underline-offset: 3px; }
+  .result-panel .result-pin > summary, .result-panel .result-notes > summary, .result-panel .result-details > summary { min-height: 44px; }
+  .result-panel .diff-comment-form textarea { min-height: 80px; margin: 0; font-size: 1rem; }
+  .result-panel .result-attention { background: transparent; border: 0; border-radius: 0; box-shadow: none; padding: 0; margin: .75rem 0; color: var(--foreground); }
+  .result-panel .result-attention ul { margin: 0; padding-left: 1.15rem; }
+  .result-panel .result-attention ul:has(> li:only-child) { list-style: none; padding-left: 0; }
+  .result-panel .result-knowledge { border: 0; border-top: 1px solid var(--border); background: transparent; box-shadow: none; border-radius: 0; padding: .25rem 0; }
+  .result-panel .revision-from-comments { border: 0; background: transparent; box-shadow: none; padding: .5rem 0; }
+  .result-panel .result-pin summary { font-size: .8125rem; min-height: 44px; display: flex; align-items: center; cursor: pointer; }
+  .result-panel .diff-comment-limit { font-size: .75rem; }
+  .chat-main .chat-plan { box-shadow: none; }
+  .proposal-review-notes { margin: .5rem 0; padding-left: 1.15rem; overflow-wrap: anywhere; }
+  .proposal-review-notes li + li { margin-top: .5rem; }
+  .proposal-actions .button-link, .proposal-actions button { min-height: 44px; white-space: nowrap; }
+  .chat-stop-confirm > summary { min-height: 44px; display: flex; align-items: center; cursor: pointer; }
+  .task-chat-title-line h1, .task-main-title, .resume-target, .resume-ceremony, .proposal-task_action, .chat-stop-confirm { overflow-wrap: anywhere; }
+  .chat-main .chat-task-back { display: none; }
+  .knowledge-library .knowledge-intro { padding: .5rem 0 1.25rem; margin-bottom: 1.5rem; border-bottom: 1px solid var(--border); }
+  .knowledge-library .knowledge-instructions { line-height: 1.65; margin-bottom: .5rem; }
+  .knowledge-library .knowledge-editor summary { width: fit-content; font-size: .875rem; font-weight: 550; }
+  .knowledge-library .knowledge-editor, .knowledge-library > details { border: 0; border-bottom: 1px solid var(--border); border-radius: 0; box-shadow: none; background: transparent; padding: 0; }
+  .knowledge-library .knowledge-editor { border-bottom: 0; }
+  .knowledge-library .card { border: 0; border-bottom: 1px solid var(--border); border-radius: 0; padding: 0 0 .5rem; margin: 0; box-shadow: none; background: transparent; }
+  .knowledge-library .card[open] { padding-bottom: 1rem; }
+  .knowledge-library .card > summary { padding: .85rem 0; }
+  @media (prefers-reduced-motion: no-preference) {
+    .chat-activity-row .activity-dot { animation: quiet-activity 1.8s ease-in-out infinite; }
+    .chat-completed .chat-overview-item { transition: background-color 140ms ease; }
+    @keyframes quiet-activity { 50% { opacity: .35; } }
+  }
+  @media (max-width: 600px) {
+    .chat-workspace .result-panel { background: transparent; border-radius: 0; }
+    .chat-completed .chat-overview-item { align-items: flex-start; flex-wrap: wrap; gap: .35rem; }
+    .chat-completed .chat-overview-copy { flex-basis: 65%; }
+    .task-live-summary .task-journey { gap: .5rem; }
+  }
+`;

package/dist/chat-review.d.ts

@@ -0,0 +1,46 @@
+import type { Store } from "./store.js";
+import type { VerifiedApprover } from "./principal.js";
+export type ReviewSnapshot = {
+    task: string;
+    root: string;
+    execution: string;
+    run: number;
+    artifact: number;
+    sha: string;
+    source: string;
+    title: string;
+    notes: {
+        id: number;
+        note: string;
+        path: string | null;
+        line: number | null;
+    }[];
+};
+type Problem = {
+    ok: false;
+    message: string;
+};
+/** Read precisely the result named, never silently substitute a newer version. */
+export declare function readChatResult(store: Store, who: VerifiedApprover, evidenceRoot: string | undefined, task: string, run?: number): {
+    ok: true;
+    snapshot: ReviewSnapshot;
+} | Problem;
+export type ReviewRequest = {
+    snapshot: ReviewSnapshot;
+    operation: "note" | "revise";
+    note: string | null;
+    path: string | null;
+    line: number | null;
+    notes: number[];
+};
+export declare function reviewInputProblem(note: string | null, path: string | null, line: number | null): string | null;
+/** Chat and the result form write the same comments and call the same revision service.
+ * A failed combined note+revision rolls back the note too; no half-completed action.
+ */
+export declare function applyChatReview(store: Store, who: VerifiedApprover, evidenceRoot: string, request: ReviewRequest, now: Date, allowMode: boolean): {
+    ok: true;
+    taskId: string;
+    run: number;
+    said: string;
+} | Problem;
+export {};

package/dist/chat-review.js

@@ -0,0 +1,98 @@
+import { readVerifiedArtifact } from "./evidence.js";
+import { hasForbiddenControls, validateNote } from "./decision.js";
+import { isRevisionFeedback, revisionSourceOf } from "./result-review.js";
+import { createResultRevision } from "./result-actions.js";
+/** Read precisely the result named, never silently substitute a newer version. */
+export function readChatResult(store, who, evidenceRoot, task, run) {
+    const ref = store.lookupRef(task);
+    if (ref?.repo == null || !who.repos.includes(ref.repo))
+        return { ok: false, message: "That task is not in your projects." };
+    const family = store.taskFamilyOf(task, who.repos, false);
+    if (family === null || family.problem !== null)
+        return { ok: false, message: "This task's revision history needs repair." };
+    const found = run === undefined
+        ? store.runsFor(ref.id).find(one => (one.role === "builder" || one.role === "repair") && one.outcome !== null)
+        : store.getRun(run);
+    if (found == null || found.taskRef !== ref.id || found.outcome === null)
+        return { ok: false, message: "There is no finished result for that version yet." };
+    const artifact = store.artifactsFor(found.id).find(one => one.kind === "terminal-diff");
+    if (artifact === undefined || evidenceRoot === undefined)
+        return { ok: false, message: "The saved changes are unavailable. Feedback has not been added." };
+    const proven = readVerifiedArtifact(evidenceRoot, artifact);
+    if (!proven.ok)
+        return { ok: false, message: "The saved changes could not be verified. Restore the result before adding feedback." };
+    return { ok: true, snapshot: {
+            task, root: family.root.id, execution: family.current.id, run: found.id,
+            artifact: artifact.id, sha: artifact.sha256, source: revisionSourceOf(store.getScope(task)?.digest ?? null),
+            title: store.getTask(task)?.title ?? task,
+            notes: store.liveDiffComments(found.id).filter(isRevisionFeedback).map(one => ({ id: one.id, note: one.note, path: one.path, line: one.line })),
+        } };
+}
+export function reviewInputProblem(note, path, line) {
+    if (note !== null) {
+        const valid = validateNote(note);
+        if (!valid.ok)
+            return valid.problem;
+    }
+    if (path !== null && (path.length > 300 || path.trim() === "" || hasForbiddenControls(path)))
+        return "Choose a valid file path.";
+    if (line !== null && (path === null || !Number.isInteger(line) || line < 1 || line > 1_000_000))
+        return "Choose a file and a valid line number.";
+    if (note === null && (path !== null || line !== null))
+        return "Write a note before attaching it to a file.";
+    return null;
+}
+/** Chat and the result form write the same comments and call the same revision service.
+ * A failed combined note+revision rolls back the note too; no half-completed action.
+ */
+export function applyChatReview(store, who, evidenceRoot, request, now, allowMode) {
+    try {
+        return store.transact(() => store.savepoint(() => {
+            const fail = (message) => { throw new ReviewRefusal(message); };
+            const { snapshot: saw, operation, note, path, line, notes } = request;
+            const problem = reviewInputProblem(note, path, line);
+            if (problem !== null)
+                return fail(problem);
+            const current = readChatResult(store, who, evidenceRoot, saw.task, saw.run);
+            if (!current.ok)
+                return fail(current.message);
+            const live = current.snapshot;
+            if (live.execution !== saw.execution || live.source !== saw.source || live.artifact !== saw.artifact || live.sha !== saw.sha)
+                return fail("This result changed. Review it again before confirming.");
+            if (operation === "revise" && saw.execution !== saw.task)
+                return fail("A newer revision is current. Review that version before requesting changes.");
+            if (!Array.isArray(notes) || new Set(notes).size !== notes.length || notes.some(id => !saw.notes.some(n => n.id === id) || !live.notes.some(n => n.id === id)))
+                return fail("Some feedback changed or was already used. Review it again.");
+            if (operation === "note" && (note === null || notes.length > 0))
+                return fail("A saved note needs feedback text and cannot consume other notes.");
+            const batch = [...notes].sort((a, b) => a - b);
+            if (note !== null) {
+                const id = store.addDiffComment({ artifactId: live.artifact, runId: live.run, path, line, note, author: who.name }, now);
+                if (id === null)
+                    return fail("The note could not be saved.");
+                batch.push(id);
+            }
+            if (operation === "note")
+                return { ok: true, taskId: live.root, run: live.run, said: "Feedback saved. No work started." };
+            if (batch.length === 0)
+                return fail("Add feedback or select saved notes before requesting changes.");
+            const result = createResultRevision(store, evidenceRoot, {
+                run: live.run, batch: batch.join(","), source: live.source, actor: who.name, repos: who.repos, allowMode,
+            }, now);
+            if (!result.ok)
+                return fail(result.message);
+            const scope = store.getScope(result.id);
+            const approved = scope?.approvedDigest != null && scope.approvedDigest === scope.digest;
+            const planning = store.lookupRef(result.id)?.plan === "requested";
+            return { ok: true, taskId: result.id, run: live.run,
+                said: approved ? "Revision created under your automatic approval settings." : planning ? "Revision created. Updating the plan with your notes; you'll approve it next." : "Revision created. Review and approve it to start." };
+        }));
+    }
+    catch (error) {
+        if (error instanceof ReviewRefusal)
+            return { ok: false, message: error.message };
+        throw error;
+    }
+}
+class ReviewRefusal extends Error {
+}

package/dist/chat-rooms.d.ts

@@ -0,0 +1,123 @@
+/**
+ * Team conversations inside any chat transport: a group or channel follows
+ * one team conversation, a private chat may select one, and messages a
+ * paired teammate sends there are saved through the same team operation
+ * the browser and CLI use. The transport supplies identity (who sent it),
+ * where rooms are recorded, and how to reply; nothing here runs a model.
+ */
+import type { Store } from "./store.js";
+import { TeamLeads } from "./team-leads.js";
+import { type PhoneLink } from "./chat-channel.js";
+export type RoomCommand = {
+    kind: "list";
+} | {
+    kind: "select";
+    index: number;
+} | {
+    kind: "off";
+};
+/** `/team` lists, `/team <n>` chooses, `/team off` (or `private`) leaves.
+ * The bare word works too, because Slack and Discord keep unregistered
+ * slash text for themselves. Any other text is not ours. */
+export declare function roomCommand(text: string): RoomCommand | null;
+export declare const PAIR_FIRST = "Pair your account first: open Settings in Toolroll, choose this chat service, and send the code here in a private chat.";
+/** A room is a grant from one exact pairing (`binding`): its delivery and its
+ * sends are rechecked against that person's current membership. */
+export type Room = {
+    id: number;
+    chatId: string;
+    kind: "group" | "private";
+    conversation: string;
+    boundBy: string;
+    binding: number;
+    cursor: number;
+};
+/** Where a transport records which chat follows which conversation. */
+export type RoomBackend = {
+    room(chatId: string): Room | null;
+    bind(args: {
+        chatId: string;
+        kind: "group" | "private";
+        conversation: string;
+        by: string;
+        binding: number;
+    }): {
+        ok: true;
+    } | {
+        ok: false;
+        reason: "group-taken";
+    };
+    /** The pairing a room was granted from, if it is still live. */
+    grant(binding: number): {
+        approver: string;
+        generation: number;
+        chatId: string;
+    } | null;
+    unbind(chatId: string, by: string): boolean;
+};
+export type RoomCounts = {
+    ignored: number;
+    chatQueued?: number;
+    chatRefused?: number;
+    statusReplies?: number;
+};
+export type RoomInbound = {
+    store: Store;
+    /** The transport's request-id prefix, e.g. "telegram", "slack". */
+    channel: string;
+    backend: RoomBackend;
+    chatId: string;
+    isGroup: boolean;
+    /** The paired sender, already proved by the transport; null when unpaired. */
+    sender: {
+        approver: string;
+        generation: number;
+        binding: number;
+    } | null;
+    /** A transport-unique key for this inbound message (update id, event id). */
+    updateKey: string;
+    text: string;
+    /** The enrolled ceiling; null when unreadable. */
+    projects: readonly string[] | null;
+    origin: string | null;
+    now: Date;
+    report: RoomCounts;
+    /** Queue one reply into the chat the message came from. */
+    say: (text: string, link?: PhoneLink) => void;
+};
+export declare function teamRequestId(channel: string, chatId: string, updateKey: string): string;
+export declare function conversationRow(store: Store, id: string): {
+    conversation: string;
+    title: string;
+    thread: number;
+    lead: string;
+    leadName: string;
+    visibility: string;
+} | null;
+/**
+ * Handle one inbound message that belongs to the team layer. Returns true
+ * when it was consumed here (a `/team` command in any chat, any message in
+ * a followed room, or a private message while a conversation is selected);
+ * false hands the message to the transport's personal path untouched.
+ */
+export declare function applyRoomInbound(input: RoomInbound): boolean;
+/** Whether the pairing a room was granted from still holds: live, the same
+ * person, still a manager of a team conversation for a group (a viewer for a
+ * private selection). */
+export declare function roomGrantAllowed(store: Store, domain: TeamLeads, backend: RoomBackend, room: Room): boolean;
+export declare function teamDomain(store: Store, projects: readonly string[] | null): TeamLeads;
+export type RoomMessage = {
+    id: number;
+    role: "assistant" | "operator";
+    text: string;
+    turn: number | null;
+    author: string | null;
+    requestId: string | null;
+    status: string | null;
+};
+/** The conversation messages a room has not received yet, oldest first. */
+export declare function roomMessagesAfter(store: Store, thread: number, cursor: number, limit?: number): RoomMessage[];
+/** What a room shows for one message: the lead's reply as is, a teammate's message with its author, this room's own messages and cancelled ones not at all. */
+export declare function roomMessageText(message: RoomMessage, channel: string, chatId: string): string | null;
+/** Whose binding a room card rides: the person the lead was answering, else any paired participant. */
+export declare function roomCardApprover(store: Store, conversation: string, turn: number, paired: readonly string[]): string | null;