mockpit 1.1.0 → 1.3.0

package/dist/server/feedbackBatch.d.ts

@@ -1,8 +1,12 @@
-import { type AskOption, type Comment, type CommentAnchor, type Mock, OTHER_ID, type Post, type Reply, type Store } from "./types.ts";
+import { type Decisions } from "./decisions.ts";
+import { type AskOption, type Comment, type CommentAnchor, type Mock, OTHER_ID, type PartComment, type Post, type Reaction, type Reply, type Store } from "./types.ts";
 export interface VariantRef {
     state: string | null;
     variant: string;
 }
+export interface InUseRef extends VariantRef {
+    version: number;
+}
 export interface FeedbackComment {
     seq: number;
     text: string;
@@ -13,21 +17,29 @@ export interface FeedbackComment {
     anchors?: Comment["anchors"];
     viewport?: number;
 }
+export interface ImageRef {
+    id: string;
+    url: string;
+}
 export interface OtherChoice {
     id: typeof OTHER_ID;
     label: string;
     other: true;
+    images?: ImageRef[];
 }
 export interface AnsweredAsk {
     ask: string;
     text: string;
     chosen: (AskOption | OtherChoice)[];
     note?: string;
+    noteImages?: ImageRef[];
 }
-export interface FeedbackReply extends Omit<Reply, "others" | "notes"> {
+export interface FeedbackReply extends Omit<Reply, "others" | "notes" | "otherImages" | "noteImages"> {
     seq: number;
     at: string;
     asks: AnsweredAsk[];
+    reactions?: string[];
+    decisions?: Decisions;
 }
 export interface FeedbackBatch {
     mockId: string | null;
@@ -36,8 +48,19 @@ export interface FeedbackBatch {
     title: string | null;
     reply: FeedbackReply | null;
     comments: FeedbackComment[];
-    accepted: VariantRef[];
+    accepted: InUseRef[];
     archived: VariantRef[];
+    dropped: VariantRef[];
+    handoff?: string;
+    resolved?: true;
+    dismissed?: {
+        ask: string;
+        text: string;
+    }[];
 }
-export declare function groupFeedback(comments: Comment[], mocks: Map<string, Mock>, posts: Map<string, Post[]>): FeedbackBatch[];
-export declare function buildFeedbackBatches(store: Store, comments: Comment[]): Promise<FeedbackBatch[]>;
+export declare function reactionLine(c: PartComment & {
+    reaction: Reaction;
+}): string;
+export declare function groupFeedback(comments: Comment[], mocks: Map<string, Mock>, posts: Map<string, Post[]>, base?: string): FeedbackBatch[];
+export declare function buildFeedbackBatches(store: Store, comments: Comment[], base: string): Promise<FeedbackBatch[]>;
+export declare function feedbackImages(batches: FeedbackBatch[]): ImageRef[];

package/dist/server/feedbackBatch.js

@@ -2,11 +2,36 @@
 // The `feedback` verb, the watch long-poll and the write piggyback all return this,
 // so an agent reads one grouped object instead of re-deriving which mock a flat
 // comment list belongs to. Runtime-agnostic (no node imports).
-import { builtinAsk, OTHER_ID, } from "./types.js";
+import { decisionsFor } from "./decisions.js";
+import { builtinAsk, inUseVersion, DEFAULT_VARIANT, OTHER_ID, } from "./types.js";
 const ref = (p) => ({ state: p.state, variant: p.variant });
-function answered(reply, mock, posts) {
+const REACTION_LEAD = {
+    up: "👍",
+    down: "👎",
+    more: "more like this:",
+};
+// `👍 part "hero" (variant B, state Mobile)`, `👎 variant B`, `more like this: the page`.
+export function reactionLine(c) {
+    // A mock published without variant names has only "default": naming it is noise.
+    const variant = c.variant && c.variant !== DEFAULT_VARIANT ? c.variant : null;
+    const where = [
+        c.part !== null && variant ? `variant ${variant}` : "",
+        c.state ? `state ${c.state}` : "",
+    ].filter(Boolean);
+    const target = c.part !== null
+        ? `part ${JSON.stringify(c.part)}`
+        : variant
+            ? `variant ${variant}`
+            : "the page";
+    const tail = where.length ? ` (${where.join(", ")})` : "";
+    const note = c.text ? ` “${c.text}”` : "";
+    return `${REACTION_LEAD[c.reaction]} ${target}${tail}${note}`;
+}
+const isReaction = (c) => !!c.reaction;
+function answered(reply, mock, posts, base) {
     const others = reply.others ?? {};
     const notes = reply.notes ?? {};
+    const refs = (ids) => (ids ?? []).map((id) => ({ id, url: `${base}/a/${encodeURIComponent(id)}` }));
     const askIds = new Set([
         ...Object.keys(reply.answers),
         ...Object.keys(others),
@@ -18,13 +43,21 @@ function answered(reply, mock, posts) {
         const answer = reply.answers[askId];
         const ids = answer === undefined ? [] : Array.isArray(answer) ? answer : [answer];
         const chosen = ids.map((id) => ask?.options.find((o) => o.id === id) ?? { id, label: id });
+        const otherImages = refs(reply.otherImages?.[askId]);
         if (others[askId] !== undefined)
-            chosen.push({ id: OTHER_ID, label: others[askId], other: true });
+            chosen.push({
+                id: OTHER_ID,
+                label: others[askId],
+                other: true,
+                ...(otherImages.length ? { images: otherImages } : {}),
+            });
+        const noteImages = refs(reply.noteImages?.[askId]);
         out.push({
             ask: askId,
             text: ask?.text ?? "",
             chosen,
             ...(notes[askId] !== undefined ? { note: notes[askId] } : {}),
+            ...(noteImages.length ? { noteImages } : {}),
         });
     }
     return out;
@@ -33,8 +66,8 @@ function answered(reply, mock, posts) {
 // one. A mock gets a fresh batch whenever a second reply arrives for it, so a
 // batch carries at most one reply and nothing is merged away. A comment whose
 // mock is unknown still gets a batch — feedback is never dropped for want of
-// context.
-export function groupFeedback(comments, mocks, posts) {
+// context. `base` (origin + base path) makes image URLs absolute.
+export function groupFeedback(comments, mocks, posts, base = "") {
     const batches = [];
     const open = new Map();
     for (const c of comments) {
@@ -42,7 +75,7 @@ export function groupFeedback(comments, mocks, posts) {
         const mock = c.mockId ? mocks.get(c.mockId) : undefined;
         const mockPosts = (c.mockId && posts.get(c.mockId)) || [];
         let batch = open.get(key);
-        if (!batch || (c.kind === "reply" && batch.reply)) {
+        if (!batch || (c.kind === "reply" && (batch.reply || batch.resolved))) {
             batch = {
                 mockId: c.mockId,
                 project: mock?.project ?? null,
@@ -52,20 +85,39 @@ export function groupFeedback(comments, mocks, posts) {
                 comments: [],
                 accepted: [],
                 archived: [],
+                dropped: [],
             };
             open.set(key, batch);
             batches.push(batch);
         }
+        if (c.kind === "reply" && c.payload?.resolved) {
+            batch.resolved = true;
+            batch.dismissed = c.payload.resolved.dismissed.map((id) => ({
+                ask: id,
+                text: (mock?.asks.find((a) => a.id === id) ?? (mock && builtinAsk(mock, mockPosts, id)))
+                    ?.text ?? "",
+            }));
+            continue;
+        }
         if (c.kind === "reply" && c.payload) {
-            const { others: _others, notes: _notes, ...payload } = c.payload;
+            const { others: _others, notes: _notes, otherImages: _otherImages, noteImages: _noteImages, ...payload } = c.payload;
+            const reactions = (payload.comments ?? []).filter(isReaction).map(reactionLine);
             batch.reply = {
                 ...payload,
+                comments: (payload.comments ?? []).filter((x) => !x.reaction),
                 seq: c.seq,
                 at: c.createdAt,
-                asks: answered(c.payload, mock, mockPosts),
+                asks: answered(c.payload, mock, mockPosts, base),
+                ...(reactions.length ? { reactions } : {}),
             };
-            batch.accepted = mockPosts.filter((p) => p.status === "accepted").map(ref);
+            const decisions = mock ? decisionsFor(mock, mockPosts, c.payload) : null;
+            if (decisions)
+                batch.reply.decisions = decisions;
+            batch.accepted = mockPosts
+                .filter((p) => p.status === "accepted")
+                .map((p) => ({ ...ref(p), version: inUseVersion(p) }));
             batch.archived = mockPosts.filter((p) => p.status === "archived").map(ref);
+            batch.dropped = mockPosts.filter((p) => p.status === "dropped").map(ref);
             continue;
         }
         const post = c.postId ? mockPosts.find((p) => p.id === c.postId) : undefined;
@@ -82,7 +134,7 @@ export function groupFeedback(comments, mocks, posts) {
     }
     return batches;
 }
-export async function buildFeedbackBatches(store, comments) {
+export async function buildFeedbackBatches(store, comments, base) {
     const mocks = new Map();
     const posts = new Map();
     for (const c of comments) {
@@ -94,5 +146,20 @@ export async function buildFeedbackBatches(store, comments) {
         mocks.set(c.mockId, mock);
         posts.set(c.mockId, await store.listPosts({ mockId: c.mockId }));
     }
-    return groupFeedback(comments, mocks, posts);
+    return groupFeedback(comments, mocks, posts, base);
+}
+// Every image a list of batches carries, in order — what the MCP tiers attach
+// as image content so the model sees what the user attached.
+export function feedbackImages(batches) {
+    const out = [];
+    for (const b of batches) {
+        for (const a of b.reply?.asks ?? []) {
+            for (const c of a.chosen)
+                if ("other" in c && c.images)
+                    out.push(...c.images);
+            if (a.noteImages)
+                out.push(...a.noteImages);
+        }
+    }
+    return out;
 }

package/dist/server/fills.d.ts

@@ -0,0 +1,41 @@
+import type { Knobs, Surface } from "./types.ts";
+export interface Fill {
+    name: string;
+    label?: string;
+}
+export interface SlotInfo {
+    part: string;
+    label?: string;
+    fills: Fill[];
+    default: string;
+    parent: {
+        part: string;
+        fill: string;
+    } | null;
+}
+interface Range {
+    part: string;
+    fill: string;
+    start: number;
+    end: number;
+    tagEnd: number;
+}
+export interface FillScan {
+    slots: SlotInfo[];
+    ranges: Range[];
+}
+export declare function scanFills(html: string): FillScan;
+export declare function slotKnobs(slots: SlotInfo[]): Knobs;
+export declare function slotCss(slots: SlotInfo[]): string;
+export declare function composeFills(html: string, chosen: Record<string, string>): string;
+export declare function slotsInSurfaces(surfaces: Surface[]): SlotInfo[];
+export type FillSplice = {
+    ok: true;
+    surfaces: Surface[];
+    applied: string[];
+} | {
+    ok: false;
+    error: string;
+};
+export declare function spliceFills(surfaces: Surface[], edits: Record<string, string>): FillSplice;
+export {};

package/dist/server/fills.js

@@ -0,0 +1,328 @@
+// Slots: a part with alternative fills, declared in the agent's own markup.
+//
+//   <section data-part="entry" data-fill="dots">…</section>
+//   <section data-part="entry" data-fill="skeleton">…</section>
+//
+// Elements sharing a `data-part` name with two or more distinct `data-fill`
+// values make one slot. The first fill in document order is the default unless
+// one carries `data-fill-default`. A fill may hold further slots: the scan
+// records, for each slot, the fill it sits in, so the viewer can ask about it
+// only while that fill is chosen.
+//
+// String-only and runtime-agnostic, like parts.ts: identity is read from the
+// markup, never from a DOM. The scan is a tolerant tag walker, not a parser:
+// it tracks open elements to know containment and each fill's extent, skips
+// comments and raw-text elements, and treats an unclosed element as ending at
+// the next close of an outer one.
+const VOID = new Set([
+    "area",
+    "base",
+    "br",
+    "col",
+    "embed",
+    "hr",
+    "img",
+    "input",
+    "link",
+    "meta",
+    "param",
+    "source",
+    "track",
+    "wbr",
+]);
+const RAW = new Set(["script", "style", "textarea", "title"]);
+const TOKEN = /<!--[\s\S]*?-->|<\/([A-Za-z][\w:-]*)\s*>|<([A-Za-z][\w:-]*)((?:\s[^<>]*?)?)(\/?)>/g;
+// Longest names first, and each must end at the name's end: `data-fill`
+// must not match the start of `data-fill-label`.
+const ATTR = /\s(data-part-label|data-part|data-fill-label|data-fill-default|data-fill)(?=[\s=/>]|$)(?:\s*=\s*(?:"([^"]*)"|'([^']*)'|([^\s"'=<>`]+)))?/g;
+// A slot's knob path is its part name, so the name must be a valid path segment.
+const PATH_SEGMENT = /^[A-Za-z_][\w-]{0,63}$/;
+const MAX_SLOTS = 100;
+const decode = (s) => s
+    .replace(/&quot;/g, '"')
+    .replace(/&#39;|&apos;/g, "'")
+    .replace(/&lt;/g, "<")
+    .replace(/&gt;/g, ">")
+    .replace(/&amp;/g, "&")
+    .trim()
+    .slice(0, 200);
+function attributes(raw) {
+    const out = {};
+    ATTR.lastIndex = 0;
+    let m;
+    while ((m = ATTR.exec(raw)))
+        out[m[1]] = decode(m[2] ?? m[3] ?? m[4] ?? "");
+    return out;
+}
+export function scanFills(html) {
+    const { slots, ranges } = scanAll(html);
+    const real = slots.filter((s) => s.fills.length >= 2);
+    const isSlot = new Set(real.map((s) => s.part));
+    return { slots: real, ranges: ranges.filter((r) => isSlot.has(r.part)) };
+}
+// Every part carrying `data-fill`, a lone fill included: a splice may add the
+// second fill that makes it a slot.
+function scanAll(html) {
+    const ranges = [];
+    // Per part, in document order: each fill element and the fill it sits in.
+    const seen = new Map();
+    const stack = [];
+    const close = (i, end) => {
+        for (let j = stack.length - 1; j >= i; j--) {
+            const f = stack[j].fill;
+            if (f)
+                f.end = end;
+        }
+        stack.length = i;
+    };
+    TOKEN.lastIndex = 0;
+    let m;
+    while ((m = TOKEN.exec(html))) {
+        if (m[0].startsWith("<!--"))
+            continue;
+        if (m[1]) {
+            const tag = m[1].toLowerCase();
+            for (let i = stack.length - 1; i >= 0; i--) {
+                if (stack[i].tag === tag) {
+                    close(i, TOKEN.lastIndex);
+                    break;
+                }
+            }
+            continue;
+        }
+        const tag = m[2].toLowerCase();
+        const attrs = attributes(m[3] ?? "");
+        let fill = null;
+        const part = attrs["data-part"];
+        const name = attrs["data-fill"];
+        if (part && name && PATH_SEGMENT.test(part)) {
+            fill = { part, fill: name, start: m.index, end: html.length, tagEnd: TOKEN.lastIndex };
+            ranges.push(fill);
+            let slot = seen.get(part);
+            if (!slot && seen.size < MAX_SLOTS) {
+                const outer = [...stack].reverse().find((o) => o.fill)?.fill ?? null;
+                slot = {
+                    fills: [],
+                    defaults: [],
+                    parent: outer ? { part: outer.part, fill: outer.fill } : null,
+                };
+                seen.set(part, slot);
+            }
+            if (slot) {
+                if (attrs["data-part-label"] && !slot.label)
+                    slot.label = attrs["data-part-label"];
+                if (!slot.fills.some((f) => f.name === name)) {
+                    slot.fills.push({
+                        name,
+                        ...(attrs["data-fill-label"] ? { label: attrs["data-fill-label"] } : {}),
+                    });
+                }
+                if ("data-fill-default" in attrs)
+                    slot.defaults.push(name);
+            }
+        }
+        if (VOID.has(tag) || m[4] === "/") {
+            if (fill)
+                fill.end = TOKEN.lastIndex;
+            continue;
+        }
+        if (RAW.has(tag)) {
+            // Skip to the matching close: markup inside script or style is text.
+            const endTag = new RegExp(`</${tag}\\s*>`, "gi");
+            endTag.lastIndex = TOKEN.lastIndex;
+            const e = endTag.exec(html);
+            TOKEN.lastIndex = e ? endTag.lastIndex : html.length;
+            if (fill)
+                fill.end = TOKEN.lastIndex;
+            continue;
+        }
+        stack.push({ tag, fill });
+    }
+    const slots = [];
+    for (const [part, s] of seen) {
+        slots.push({
+            part,
+            ...(s.label ? { label: s.label } : {}),
+            fills: s.fills,
+            default: s.defaults[0] ?? s.fills[0].name,
+            parent: s.parent,
+        });
+    }
+    return { slots, ranges };
+}
+// Each slot is a select knob at its part's path: the fills are its options and
+// the default fill is its value. A knob the agent declares at the same path
+// wins (to relabel the options), so these are merged under declared knobs.
+export function slotKnobs(slots) {
+    const out = {};
+    for (const s of slots) {
+        const config = {
+            type: "select",
+            options: s.fills.map((f) => (f.label ? { value: f.name, label: f.label } : f.name)),
+            value: s.default,
+        };
+        out[s.part] = config;
+    }
+    return out;
+}
+const cssString = (s) => `"${s.replace(/[\\"]/g, (c) => `\\${c}`).replace(/[\n\r<>]/g, (c) => `\\${c.charCodeAt(0).toString(16)} `)}"`;
+// Shows only the chosen fill of each slot, keyed on the `data-k-<part>`
+// attribute the knob runtime keeps on <html>. The chosen fill is never styled:
+// only the others are hidden, so a fill keeps whatever display the agent gave it.
+export function slotCss(slots) {
+    const rules = [];
+    for (const s of slots) {
+        const attr = `data-k-${s.part}`;
+        for (const f of s.fills) {
+            const el = `[data-part=${cssString(s.part)}][data-fill=${cssString(f.name)}]`;
+            const chosen = `[${attr}=${cssString(f.name)}]`;
+            rules.push(f.name === s.default
+                ? `html[${attr}]:not(${chosen}) ${el}{display:none!important}`
+                : `html:not(${chosen}) ${el}{display:none!important}`);
+        }
+    }
+    return rules.join("\n");
+}
+const FILL_ATTRS = /\s(data-fill-label|data-fill-default|data-fill)(?=[\s=/>]|$)(?:\s*=\s*(?:"[^"]*"|'[^']*'|[^\s"'=<>`]+))?/g;
+// The screen as chosen: every slot reduced to its chosen fill, the others cut
+// out with whatever they contained, and the fill attributes dropped from the
+// fills that stay. `chosen` maps a slot's part to a fill; a slot it does not
+// name keeps its default.
+export function composeFills(html, chosen) {
+    const { slots, ranges } = scanFills(html);
+    const pick = new Map(slots.map((s) => [s.part, chosen[s.part] ?? s.default]));
+    const cut = ranges.filter((r) => pick.get(r.part) !== r.fill);
+    const kept = ranges.filter((r) => pick.get(r.part) === r.fill);
+    const edits = [];
+    for (const r of cut) {
+        if (cut.some((o) => o !== r && o.start <= r.start && r.end <= o.end))
+            continue;
+        edits.push({ start: r.start, end: r.end, text: "" });
+    }
+    for (const r of kept) {
+        if (cut.some((o) => o.start <= r.start && r.end <= o.end))
+            continue;
+        edits.push({
+            start: r.start,
+            end: r.tagEnd,
+            text: html.slice(r.start, r.tagEnd).replace(FILL_ATTRS, ""),
+        });
+    }
+    edits.sort((a, b) => b.start - a.start);
+    let out = html;
+    for (const e of edits)
+        out = out.slice(0, e.start) + e.text + out.slice(e.end);
+    return out;
+}
+// A post's slots across its html surfaces (the first surface that declares a
+// part wins), cached per surfaces array: a stored post never mutates it.
+const bySurfaces = new WeakMap();
+export function slotsInSurfaces(surfaces) {
+    const hit = bySurfaces.get(surfaces);
+    if (hit)
+        return hit;
+    const out = [];
+    for (const sf of surfaces) {
+        if (sf.kind !== "html")
+            continue;
+        for (const slot of scanFills(sf.html).slots) {
+            if (!out.some((o) => o.part === slot.part))
+                out.push(slot);
+        }
+    }
+    bySurfaces.set(surfaces, out);
+    return out;
+}
+const attrValue = (s) => s.replace(/&/g, "&amp;").replace(/"/g, "&quot;").replace(/</g, "&lt;");
+const FIRST_TAG = /^(?:\s|<!--[\s\S]*?-->)*<([A-Za-z][\w:-]*)((?:\s[^<>]*?)?)(\/?)>/;
+// The replacement must be one element marked as that fill: the marks are added
+// when missing and a contradicting one is refused, so a splice cannot quietly
+// land in another slot.
+function markFill(html, part, fill) {
+    const m = FIRST_TAG.exec(html);
+    if (!m)
+        return { error: `fill "${part}#${fill}" must be html starting with its element` };
+    const attrs = attributes(m[2] ?? "");
+    if (attrs["data-part"] !== undefined && attrs["data-part"] !== part) {
+        return { error: `fill "${part}#${fill}" is marked data-part="${attrs["data-part"]}"` };
+    }
+    if (attrs["data-fill"] !== undefined && attrs["data-fill"] !== fill) {
+        return { error: `fill "${part}#${fill}" is marked data-fill="${attrs["data-fill"]}"` };
+    }
+    const add = (attrs["data-part"] === undefined ? ` data-part="${attrValue(part)}"` : "") +
+        (attrs["data-fill"] === undefined ? ` data-fill="${attrValue(fill)}"` : "");
+    const at = m.index + m[0].indexOf("<") + 1 + m[1].length;
+    return html.slice(0, at) + add + html.slice(at);
+}
+// Replace one fill of a slot, or add it after the slot's last fill, keyed
+// "part#fill". Like spliceParts, every target is located in the original
+// markup and overlapping edits are refused.
+export function spliceFills(surfaces, edits) {
+    const docs = surfaces.flatMap((s, i) => (s.kind === "html" ? [{ i, surface: s }] : []));
+    if (docs.length === 0) {
+        return { ok: false, error: "fills needs an html surface; this variant has none" };
+    }
+    const scans = docs.map((d) => scanAll(d.surface.html).ranges);
+    const plan = new Map();
+    const targets = Object.keys(edits);
+    for (const [n, target] of targets.entries()) {
+        const at = target.indexOf("#");
+        const part = at > 0 ? target.slice(0, at) : "";
+        const fill = at > 0 ? target.slice(at + 1).trim() : "";
+        if (!PATH_SEGMENT.test(part) || !fill) {
+            return {
+                ok: false,
+                error: `fills keys are "part#fill", e.g. "entry#skeleton" (got "${target}")`,
+            };
+        }
+        const where = scans.flatMap((rs, d) => (rs.some((r) => r.part === part) ? [d] : []));
+        if (where.length === 0) {
+            return {
+                ok: false,
+                error: `no fills of "${part}"; mark its alternatives data-part="${part}" data-fill="<name>" with html or parts first`,
+            };
+        }
+        if (where.length > 1) {
+            return {
+                ok: false,
+                error: `fills of "${part}" are in several html surfaces; revise with surfaces`,
+            };
+        }
+        const d = where[0];
+        const own = scans[d].filter((r) => r.part === part);
+        const same = own.filter((r) => r.fill === fill);
+        if (same.length > 1) {
+            return {
+                ok: false,
+                error: `fill "${target}" appears ${same.length} times; revise with html or parts`,
+            };
+        }
+        const text = markFill(edits[target], part, fill);
+        if (typeof text !== "string")
+            return { ok: false, error: text.error };
+        const edit = same.length
+            ? { start: same[0].start, end: same[0].end, text, n }
+            : (() => {
+                const end = Math.max(...own.map((r) => r.end));
+                return { start: end, end, text: `\n${text}`, n };
+            })();
+        const list = plan.get(d) ?? [];
+        const clash = list.find((e) => e.start < edit.end && edit.start < e.end);
+        if (clash) {
+            return { ok: false, error: `fills "${targets[clash.n]}" and "${target}" overlap; send one` };
+        }
+        list.push(edit);
+        plan.set(d, list);
+    }
+    const next = [...surfaces];
+    for (const [d, list] of plan) {
+        const { i, surface } = docs[d];
+        let html = surface.html;
+        // From the end, and same-place additions last-listed first, so they land in listed order.
+        for (const e of list.toSorted((a, b) => b.start - a.start || b.n - a.n)) {
+            html = html.slice(0, e.start) + e.text + html.slice(e.end);
+        }
+        next[i] = { ...surface, html };
+    }
+    return { ok: true, surfaces: next, applied: targets };
+}

package/dist/server/handoff.d.ts

@@ -0,0 +1,6 @@
+import type { Comment, Session } from "./types.ts";
+export declare const HANDOFF_GRACE_MS: number;
+export declare const HANDOFF_NOTE = "Sent to an earlier agent session in this project that has gone quiet; it is yours now. Read the mock before acting.";
+export declare const ownerActiveAt: (s: Session) => string;
+export declare function supersedes(taker: Session, owner: Session, listening: boolean): boolean;
+export declare function handoffReady(comment: Comment, owner: Session, taker: Session, now: number): boolean;

package/dist/server/handoff.js

@@ -0,0 +1,38 @@
+// Orphaned replies: a Send lands on the session that owns the mock, and when
+// that agent restarts into a new session (a new conversation, /clear, a crash)
+// nobody would ever read it. The rule below decides when a newer session may
+// take such a reply instead. Runtime-agnostic (no node imports).
+//
+// The danger is the other way round: two agents can work in one repo at once,
+// and an agent waiting on the user looks exactly like a dead one — both are
+// silent. So each clause narrows who counts as "the same agent, restarted":
+//
+// - Same project, same cwd, same agent name. A restart reopens the same harness
+//   in the same checkout; a different worktree or tool is a parallel worker.
+// - The owner was created first and has made no call since the taker appeared
+//   (agentActiveAt, which the user's own Sends don't bump). An owner that kept
+//   working after the taker started is running alongside it, not gone.
+// - The owner has no feedback long-poll open right now (`mockpit watch`).
+//
+// A reply also waits HANDOFF_GRACE_MS after the Send, so the owner — the agent
+// the user is most likely about to tell — gets first claim. The one exception:
+// a keyless owner (no harness conversation id) can never come back, so a reply
+// it got before the taker existed goes at once — the user answered, then
+// restarted. A keyed owner always gets the grace: its conversation may resume.
+export const HANDOFF_GRACE_MS = 2 * 60_000;
+export const HANDOFF_NOTE = "Sent to an earlier agent session in this project that has gone quiet; it is yours now. Read the mock before acting.";
+export const ownerActiveAt = (s) => s.agentActiveAt ?? s.lastActiveAt;
+export function supersedes(taker, owner, listening) {
+    return (owner.id !== taker.id &&
+        owner.project !== null &&
+        owner.project === taker.project &&
+        owner.cwd === taker.cwd &&
+        owner.agent === taker.agent &&
+        owner.createdAt < taker.createdAt &&
+        ownerActiveAt(owner) < taker.createdAt &&
+        !listening);
+}
+export function handoffReady(comment, owner, taker, now) {
+    return ((!owner.keyed && comment.createdAt < taker.createdAt) ||
+        now - Date.parse(comment.createdAt) >= HANDOFF_GRACE_MS);
+}

package/dist/server/index.js

@@ -1,5 +1,6 @@
 import { serve } from "@hono/node-server";
-import { readFile } from "node:fs/promises";
+import { createHash } from "node:crypto";
+import { readdir, readFile } from "node:fs/promises";
 import { homedir } from "node:os";
 import { basename, dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
@@ -25,6 +26,18 @@ const [viewerHtml, setupText, topicTexts, pkgJson] = await Promise.all([
     readFile(join(root, "package.json"), "utf8"),
 ]);
 const topics = Object.fromEntries(GUIDE_TOPICS.map((id, i) => [id, topicTexts[i]]));
+// What renders surface documents: this server's own files and the locked
+// library versions. Hashed whole rather than picked apart, so an edit in dev or
+// an upgrade both reach browsers holding an old document; a change elsewhere
+// in the server only costs one re-fetch per document.
+const serverDir = dirname(fileURLToPath(import.meta.url));
+const renderHash = createHash("sha256");
+for (const name of (await readdir(serverDir)).sort()) {
+    if (/\.(ts|js)$/.test(name))
+        renderHash.update(await readFile(join(serverDir, name)));
+}
+renderHash.update(await readFile(join(root, "package-lock.json")).catch(() => pkgJson));
+const renderRev = renderHash.digest("hex").slice(0, 10);
 const pr = process.env.MOCKPIT_PUBLIC_READ;
 const publicRead = pr === "session" || pr === "full" ? pr : undefined;
 // Storage: SQLite via node:sqlite, the same SqlStore the Cloudflare Durable
@@ -55,6 +68,7 @@ const app = createApp({
     // notice); set it to the empty string to disable the update check
     version: process.env.MOCKPIT_VERSION ?? JSON.parse(pkgJson).version,
     upgradeCommand: "npm install -g mockpit",
+    renderRev,
     // `run` scripts execute in QuickJS inside a worker thread per run.
     executor: createNodeExecutor(),
     log: logPath ? createFileLog(logPath) : undefined,