toolroll 0.9.84 → 0.9.85

package/dist/server/render-chat.js

@@ -1,3 +1,4 @@
+import { parseFeedbackImage } from "../result-review.js";
 import { assignmentSummaryHtml } from "../assignment-ui.js";
 import {} from "../assignment.js";
 import {} from "../browser-workspace.js";
@@ -610,8 +611,8 @@ export function proposalCardParts(view, inert, decision, returnTo = null) {
         const snapshot = payload["snapshot"];
         const ids = Array.isArray(payload["notes"]) ? payload["notes"] : [];
         const selected = snapshot?.notes.filter(one => ids.includes(one.id)) ?? [];
-        const notes = [...selected, ...(text("note") === "" ? [] : [{ note: text("note"), path: text("path") || null, line: typeof payload["line"] === "number" ? payload["line"] : null }])];
-        what = html `<h3>${text("taskTitle") || task}</h3><ul class="proposal-review-notes">${notes.map(one => html `<li>${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : html `:${one.line}`}</span> · `}${one.note}</li>`)}</ul>${text("operation") === "revise" ? html `<p class="meta">Updates this task. Your approval settings apply.</p>` : html `<p class="meta">No work starts.</p>`}`;
+        const notes = [...selected, ...(text("note") === "" ? [] : [{ note: text("note"), path: text("path") || null, line: typeof payload["line"] === "number" ? payload["line"] : null, image: parseFeedbackImage(payload["image"]) || null }])];
+        what = html `<h3>${text("taskTitle") || task}</h3><ul class="proposal-review-notes">${notes.map(one => html `<li>${one.image == null ? "" : html `<a href="/r/${one.image.run}/evidence/${one.image.artifact}">Screenshot ${one.image.artifact} · result #${one.image.run}</a><br>`}${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : html `:${one.line}`}</span> · `}${one.note}</li>`)}</ul>${text("operation") === "revise" ? html `<p class="meta">Updates this task. Your approval settings apply.</p>` : html `<p class="meta">No work starts.</p>`}`;
     }
     else if (view.kind === "next") {
         what = html `<h3>Move <a href="${taskHref(task)}">${task}</a> to the front</h3>${facts(["current position", html `${String(payload["position"] ?? "?")} of ${String(payload["of"] ?? "?")}`], ["project", mono(repoId)])}`;

package/dist/server/render-pages.d.ts

@@ -3,7 +3,7 @@ import { type TaskOrigin } from "../task-source.js";
 import { type Html } from "../html.js";
 import { type AssignmentSnapshot } from "../assignment.js";
 import { type BoardCard } from "../board.js";
-import { type BrowserLimits } from "../browser-workspace.js";
+import { type BrowserLimits, type BrowserProjectsView } from "../browser-workspace.js";
 import { type CheckLevel } from "../check-levels.js";
 import { type DispatchDiagnosis } from "../dispatch.js";
 import { type FirstRunStep } from "../first-run.js";
@@ -451,7 +451,7 @@ export declare function projectsPage(chrome: Chrome, recent: {
     path: string;
     name: string;
     lastOpenedAt: string;
-}[], candidates: string[], open: string | null, problem: string | null, _unscopedMode: boolean, browsable?: boolean, onboard?: OnboardCardState | null, peeks?: Record<string, ProjectPeek | null>, returnTo?: string, pullRequestsOn?: (path: string) => boolean, checkLevelOf?: (path: string) => CheckLevel): Screen;
+}[], candidates: string[], open: string | null, problem: string | null, _unscopedMode: boolean, browsable?: boolean, onboard?: OnboardCardState | null, peeks?: Record<string, ProjectPeek | null>, returnTo?: string, pullRequestsOn?: (path: string) => boolean, checkLevelOf?: (path: string) => CheckLevel, activity?: BrowserProjectsView["activity"]): Screen;
 /** The no-script "move to the front" sentinel: the form cannot name the
  * front of a partition, so the handler resolves it (slice 1b, fix 1). */
 export declare const QUEUE_FRONT = "__TOP__";

package/dist/server/render-pages.js

@@ -1,3 +1,4 @@
+import { activityMetrics, activityScope, activityLimit } from '../project-activity.js';
 /** Everyday console pages: home, inbox, board, work, projects, fleet and the queue. */
 import { originNote } from "../task-source.js";
 import { html, htmlString, joinHtml, postForm, replaceMarkup, reauth } from "../html.js";
@@ -921,7 +922,10 @@ export function githubReposPage(chrome, data) {
         html `<p class="row" style="margin-top:.6rem"><a class="badge" href="/projects">← back to projects</a></p>`,
     ], "\n"), { chrome });
 }
-export function projectsPage(chrome, recent, candidates, open, problem, _unscopedMode, browsable = false, onboard = null, peeks = {}, returnTo = "/", pullRequestsOn, checkLevelOf) {
+export function projectsPage(chrome, recent, candidates, open, problem, _unscopedMode, browsable = false, onboard = null, peeks = {}, returnTo = "/", pullRequestsOn, checkLevelOf, activity) {
+    const activityHtml = activity === undefined ? null : html `<details><summary>Activity · ${activity.project}</summary>${activity.report === null
+        ? html `<p role="alert">Activity is unavailable.</p><a href="/projects">Reload</a>`
+        : html `<p>${activityScope(activity.report)}</p>${activity.report.truncated ? html `<p>${activityLimit(activity.report)}</p>` : null}${activityMetrics(activity.report).map(metric => html `<details><summary>${metric.label} · ${metric.value}</summary><p>${metric.detail}</p>${metric.rows.length === 0 ? html `<p>No tasks in this group.</p>` : html `<ul>${metric.rows.map(row => html `<li><a href="${row.href}">${row.title}</a> · Recorded checks: ${row.checks}</li>`)}</ul>`}</details>`)}`}</details>`;
     // The onboarding card (repo onboarding, findings 1-39): preview first,
     // then a password-confirmed clone into a configured root. Disabled
     // states explain themselves in words (finding 28/39).
@@ -1047,11 +1051,12 @@ export function projectsPage(chrome, recent, candidates, open, problem, _unscope
         recentItems.length === 0 && candidateItems.length === 0
             ? html `<div class="card"><p><strong>Nothing to open yet.</strong></p><p class="meta">Add one below — opening it registers it here for next time.</p></div>`
             : null,
+        activityHtml,
         recentItems.length > 0 ? html `<h2>Recent</h2>${cards(recentItems)}` : null,
         candidateItems.length > 0 ? html `<h2>Available</h2>${cards(candidateItems)}` : null,
         addCard,
     ], "\n"), { chrome, workspace: { view: {
-                kind: "projects", choosing, problem, returnTo,
+                kind: "projects", choosing, problem, returnTo, ...(activity === undefined ? {} : { activity }),
                 recent: recent.map(one => rowOf(one.path, one.name, one.lastOpenedAt)),
                 available: candidates.map(path => rowOf(path, projectName(path), null)),
                 add: { browse: browsable ? "/projects/browse" : null, github: onboard === null ? null : "/projects/github", html: htmlString(addForms) },

package/dist/server/render-results.d.ts

@@ -614,6 +614,8 @@ export type ResultDetail = {
     fileAnchors: ReadonlyMap<string, string>;
     comments: DiffComment[];
     pastComments?: DiffComment[];
+    /** Live saved notes whose screenshot would be refused now (result-actions.ts feedbackImageProblem). */
+    unusableImageNotes?: readonly number[];
     reviewerFindings: DiffComment[];
     /** The build's one automatic review, when its project had review on. */
     automaticReview?: BuildReviewView | null;

package/dist/server/render-results.js

@@ -1068,7 +1068,7 @@ export function proofBundleView(store, run, artifacts, root) {
         catch {
             problem = "the file could not be read";
         }
-        return [{ path, caption: captionFor(path), artifactId: artifact.id, problem }];
+        return [{ path, caption: captionFor(path), artifactId: artifact.id, sha256: artifact.sha256, problem }];
     });
     const stored = new Set(screenshots.map(one => one.path));
     const uncapturedScreenshots = (proof?.screenshots ?? []).map(one => one.path).filter(path => !stored.has(path));
@@ -1598,6 +1598,8 @@ export function resultPanelParts(detail, o) {
     const physicalReasons = receipt.reasons.filter(reason => !assessmentReasons.has(reason));
     const runId = run.id;
     const shots = proof?.screenshots ?? [];
+    const picture = (shot) => html `<figure class="result-image"><a class="receipt-shot" href="/r/${runId}/evidence/${shot.artifactId}"><img src="/r/${runId}/evidence/${shot.artifactId}" alt="${shot.caption}"><span>${shot.caption}</span></a>${detail.canAnnotate && o.csrf !== "" && shot.sha256 !== undefined
+        ? html `<button type="button" class="quiet" data-comment-image="${JSON.stringify({ run: runId, artifact: shot.artifactId, sha256: shot.sha256 })}" data-image-caption="${shot.caption}">Comment</button>` : ""}</figure>`;
     const shown = shots.filter(one => one.problem === null);
     const stat = terminal?.stat ?? null;
     const statOk = stat !== null && !("problem" in stat);
@@ -1694,7 +1696,7 @@ export function resultPanelParts(detail, o) {
     }
     else if (lead === "screenshots") {
         summaryParts.push(html `<div class="receipt-visuals result-visuals" aria-label="validated screenshots">${shown
-            .map(shot => html `<a class="receipt-shot" href="/r/${runId}/evidence/${shot.artifactId}"><img src="/r/${runId}/evidence/${shot.artifactId}" alt="${shot.caption}"><span>${shot.caption}</span></a>`)}</div>`);
+            .map(picture)}</div>`);
     }
     const unavailableShots = shots.filter(one => one.problem !== null);
     if (lead === "changes") {
@@ -1848,7 +1850,7 @@ export function resultPanelParts(detail, o) {
             ? directAssessment && !screenshotRequired ? html `` : html `<p class="meta" data-cockpit-source="screenshots">No screenshots${screenshotRequired ? " were captured, although one was required" : " were needed"}.</p>`
             : lead === "screenshots"
                 ? html `<p class="meta" data-cockpit-source="screenshots">${shown.length} validated screenshot${shown.length === 1 ? "" : "s"} shown in Summary${unavailableShots.length > 0 ? `; ${unavailableShots.length} unavailable` : ""}.</p>`
-                : html `<details class="cockpit-proof-group" data-cockpit-source="screenshots"><summary>Screenshots · ${shown.length}${unavailableShots.length > 0 ? ` (${unavailableShots.length} unavailable)` : ""}</summary><div class="receipt-visuals" aria-label="validated screenshots">${shown.map(shot => html `<a class="receipt-shot" href="/r/${runId}/evidence/${shot.artifactId}"><img src="/r/${runId}/evidence/${shot.artifactId}" alt="${shot.caption}"><span>${shot.caption}</span></a>`)}</div></details>`);
+                : html `<details class="cockpit-proof-group" data-cockpit-source="screenshots"><summary>Screenshots · ${shown.length}${unavailableShots.length > 0 ? ` (${unavailableShots.length} unavailable)` : ""}</summary><div class="receipt-visuals" aria-label="validated screenshots">${shown.map(picture)}</div></details>`);
         checkParts.push(receipt.caveats.length === 0
             ? directAssessment ? html `` : html `<p class="meta" data-cockpit-source="caveats">No caveats were reported.</p>`
             : html `<details class="cockpit-proof-group" data-cockpit-source="caveats"><summary>Agent caveats · ${receipt.caveats.length}</summary><ul>${receipt.caveats.map(one => html `<li>${one}</li>`)}</ul></details>`);
@@ -1863,19 +1865,29 @@ export function resultPanelParts(detail, o) {
         checkParts.push(o.extraChecks);
     // ---- request changes: both feedback styles, one sealed road ------------
     const requestParts = [html `<h3 class="so-sr-only">What should change?</h3>`];
+    const imageNote = (one) => {
+        if (one.image == null)
+            return html ``;
+        const shot = shots.find(shot => shot.artifactId === one.image.artifact && shot.sha256 === one.image.sha256 && one.image.run === runId && shot.problem === null);
+        // A live saved note whose screenshot no longer verifies can be kept without it, so it never blocks a revision.
+        const drop = o.csrf === "" || !(detail.unusableImageNotes ?? []).includes(one.id) ? html ``
+            : postForm(`/r/${runId}/comment`, html `<input type="hidden" name="intent" value="drop-image"><input type="hidden" name="comment" value="${String(one.id)}"><button type="submit" class="quiet">Keep note without screenshot</button>`, { attrs: { class: "feedback-image-drop" }, returnTo: o.returnTo, hidden: { tab: o.tab } });
+        return html `<figure class="feedback-image">${shot === undefined ? html `<figcaption>Screenshot ${one.image.artifact} is unavailable.</figcaption>`
+            : html `<a href="/r/${runId}/evidence/${shot.artifactId}"><img src="/r/${runId}/evidence/${shot.artifactId}" alt="${shot.caption}"></a><figcaption>${shot.caption} · Screenshot ${shot.artifactId}</figcaption>`}${drop}</figure>`;
+    };
     const pathWords = (path, line) => {
         const shownPath = `${path}${line === null ? "" : `:${line}`}`;
         const href = detail.editor === null ? null : editorFileHref(detail.editor.worktree, path, line);
         return href === null ? html `<span class="mono">${shownPath}</span> ` : html `<a class="mono" href="${href}">${shownPath}</a> `;
     };
     if (detail.comments.length > 0) {
-        requestParts.push(html `<p class="meta">Saved for later · ${detail.comments.length}</p><div class="diff-comments" data-result-notes="${detail.comments.length}">${detail.comments.map(one => html `<div class="diff-comment"><span class="diff-comment-pin" aria-hidden="true"></span><p>${one.path === null ? "" : pathWords(one.path, one.line)}${one.note}</p><span class="meta">${one.author} · ${whenTime(one.createdAt)}</span></div>`)}</div>`);
+        requestParts.push(html `<p class="meta">Saved for later · ${detail.comments.length}</p><div class="diff-comments" data-result-notes="${detail.comments.length}">${detail.comments.map(one => html `<div class="diff-comment">${imageNote(one)}<span class="diff-comment-pin" aria-hidden="true"></span><p>${one.path === null ? "" : pathWords(one.path, one.line)}${one.note}</p><span class="meta">${one.author} · ${whenTime(one.createdAt)}</span></div>`)}</div>`);
         if (o.csrf !== "" && !detail.canAnnotate) {
             requestParts.push(postForm(`/r/${runId}/revise`, html `${revisionSealFields(detail.comments, detail.sourceDigest)}<div><strong>${detail.comments.length} note${detail.comments.length === 1 ? "" : "s"} ready</strong></div><button type="submit">Request changes</button>`, { attrs: { class: "card revision-from-comments" }, returnTo: o.returnTo }));
         }
     }
     if ((detail.pastComments?.length ?? 0) > 0) {
-        requestParts.push(html `<details class="result-feedback-history"><summary>Earlier feedback</summary><div class="diff-comments" data-past-feedback>${detail.pastComments.map(one => html `<div class="diff-comment"><p>${one.path === null ? "" : pathWords(one.path, one.line)}${one.note}</p><span class="meta">${one.author} · ${whenTime(one.createdAt)}</span></div>`)}</div></details>`);
+        requestParts.push(html `<details class="result-feedback-history"><summary>Earlier feedback</summary><div class="diff-comments" data-past-feedback>${detail.pastComments.map(one => html `<div class="diff-comment">${imageNote(one)}<p>${one.path === null ? "" : pathWords(one.path, one.line)}${one.note}</p><span class="meta">${one.author} · ${whenTime(one.createdAt)}</span></div>`)}</div></details>`);
     }
     for (const revision of detail.revisions) {
         // The child's standing is the shared status projection's own words
@@ -1892,6 +1904,7 @@ export function resultPanelParts(detail, o) {
         requestParts.push(html `<details class="result-request-open result-request-form"${detail.comments.length > 0 ? " open" : ""}><summary>What should change?</summary>${postForm(`/r/${runId}/comment`, joinHtml([
             revisionSealFields(detail.comments, detail.sourceDigest),
             html `<input type="hidden" data-recorded-requests value="${[...detail.comments, ...(detail.pastComments ?? [])].filter(one => one.sourceKey?.startsWith(`review:${o.user}:`)).map(one => one.sourceKey.slice(`review:${o.user}:`.length)).join(",")}">`,
+            html `<input type="hidden" name="image" value=""><figure class="feedback-image" data-feedback-image hidden><img alt="Selected screenshot" hidden><figcaption></figcaption><button type="button" class="quiet" data-remove-image>Remove image</button></figure>`,
             html `<textarea name="note" rows="2" maxlength="${LIMITS.note}" placeholder="Describe the change…" aria-label="review comment" aria-describedby="comment-note-limit"${o.noted ? " autofocus" : ""}></textarea>`,
             html `<span class="meta diff-comment-limit" id="comment-note-limit">up to ${LIMITS.note} characters</span>`,
             html `<details class="result-pin"><summary>Attach to a file or line</summary>`,
@@ -1980,7 +1993,7 @@ export function personCheckItems(proof, patchText, shots, runId) {
         return {
             id: row.id, statement: row.statement, words: personCheckWords(row.statement), note: notes.length === 0 ? null : notes.join(" "),
             excerpts: files.filter(one => one.lines.length > 0).slice(0, 3),
-            shots: pictures.slice(0, 4).map(shot => ({ src: `/r/${runId}/evidence/${shot.artifactId}`, href: `/r/${runId}/evidence/${shot.artifactId}`, caption: shot.caption })),
+            shots: pictures.slice(0, 4).map(shot => ({ src: `/r/${runId}/evidence/${shot.artifactId}`, href: `/r/${runId}/evidence/${shot.artifactId}`, caption: shot.caption, ...(shot.sha256 === undefined ? {} : { image: { run: runId, artifact: shot.artifactId, sha256: shot.sha256 } }) })),
         };
     });
 }
@@ -2215,7 +2228,7 @@ export function runPage(chrome, run, taskId, artifacts, terminal = null, notes =
             noted,
             requestToken: result.requestToken,
             hrefFor: one => (one === "summary" ? `/r/${run.id}` : `/r/${run.id}?tab=${one}`),
-            returnTo: `/r/${run.id}`,
+            returnTo: `/r/${run.id}?record=1`,
             back: null,
         });
     return screen(`build #${run.id}`, joinHtml([

package/dist/server/render-tasks.d.ts

@@ -398,6 +398,7 @@ export type RevisionView = {
         line: number | null;
         note: string;
         author: string;
+        image?: import("../result-review.js").FeedbackImage | null;
     }[];
     /** The lineage and ACTUAL terms (contract handoff task 2), read from
      * the child's own rows — the same projection chat and the approval

package/dist/server/render-tasks.js

@@ -50,7 +50,7 @@ import { reviewHref, taskChatHref, taskHref } from "./http.js";
 import { coordinatorProposalsSection } from "./render-chat.js";
 import { completionReceiptCard, criterionMatrixHtml, evidenceWords, homePhaseWords, incidentWords, needsCheck, pullRequestCardHtml, reasonWords, receiptStatusOf, repairChainHtml, reportItemsHtml, reviewRetryPanel, runNoun, runOutcomeBadge, semanticCoverageHtml, verificationExplanation, verificationRecovered } from "./render-results.js";
 import {} from "./session.js";
-import { linkWords, originNote } from "../task-source.js";
+import { linkWords, originNote, requestName, requestWords } from "../task-source.js";
 export function contractChangeHtml(change) {
     const tag = html `<span class="contract-change-kind contract-change-${change.kind}">${change.kind}</span>`;
     switch (change.field) {
@@ -203,7 +203,8 @@ const fragmentString = (fragment) => fragment === "" ? "" : htmlString(fragment)
 function originWords(origin) {
     const flow = origin.flow === null ? null : origin.flow === "hidden" ? "from a flow" : origin.flow === "removed" ? "from a flow (removed)" : `from flow “${origin.flow.name}”`;
     const source = origin.source === null ? null : `${origin.source.label} (${origin.source.proved})`;
-    return [flow, source].filter(one => one !== null).join(" · ");
+    const request = origin.request == null ? null : `from ${requestName(origin.request)}`;
+    return [flow, source, request].filter(one => one !== null).join(" · ");
 }
 /** v126: a task's origin as a fact: the flow, linked for those who may open it, and the outside event, linked to where
  * it lives, with that link's host always shown. */
@@ -219,6 +220,12 @@ function originParts(origin) {
         if (origin.source.url !== null)
             parts.push(" · ", { label: linkWords(origin.source.url), href: origin.source.url });
     }
+    if (origin.request != null) {
+        if (parts.length > 0)
+            parts.push(" · ");
+        const where = requestWords(origin.request);
+        parts.push(`${requestName(origin.request)}${where === "" ? "" : `: ${where}`}`);
+    }
     return parts;
 }
 export function approvalSheetHtml(input) {
@@ -294,7 +301,7 @@ export function approvalSheetHtml(input) {
     const revisionDetail = input.revision === null ? "" : detail("Earlier build", joinHtml([
         html `<div class="revision-card" data-revision-feedback>`,
         html `<p><a href="${input.revisionSourceHref}"${input.surface === "chat" ? html ` data-revision-source` : ""}>${input.surface === "chat" ? html `Original result: build #${input.revision.sourceRun} →` : html `build #${input.revision.sourceRun}`}</a></p>`,
-        input.revision.comments.length === 0 ? "" : html `<ul>${input.revision.comments.map(one => html `<li>${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : `:${one.line}`}</span> · `}${one.note} <span class="meta">— ${one.author}</span></li>`)}</ul>`,
+        input.revision.comments.length === 0 ? "" : html `<ul>${input.revision.comments.map(one => html `<li>${revisionImageLink(one.image)}${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : `:${one.line}`}</span> · `}${one.note} <span class="meta">— ${one.author}</span></li>`)}</ul>`,
         revisionLineageHtml(input.revision.lineage),
         html `</div>`,
     ]));
@@ -1172,6 +1179,9 @@ export function executionPlanHtml(document, compact = false) {
         html `</div>`,
     ]);
 }
+function revisionImageLink(image) {
+    return image == null ? html `` : html `<a href="/r/${image.run}/evidence/${image.artifact}">Screenshot ${image.artifact} · result #${image.run}</a> · `;
+}
 /**
  * A revision's lineage and ACTUAL terms, in the same words on the task
  * page, the approval card, and chat (contract handoff task 2): which
@@ -1594,7 +1604,7 @@ export function taskBodyParts(data) {
             : joinHtml([
                 html `<div class="revision-card" data-revision-feedback>`,
                 html `<p><strong>${data.revision.kind === "ci-repair" ? "CI repair" : data.revision.kind === "criterion-repair" ? "Criterion repair" : "Revision feedback"}</strong> <span class="meta">from <a href="/r/${data.revision.sourceRun}">build #${data.revision.sourceRun}</a></span></p>`,
-                ...data.revision.comments.map(one => html `<p class="row"><span class="meta">${one.author}</span> ${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : `:${one.line}`}</span> `}${one.note}</p>`),
+                ...data.revision.comments.map(one => html `<p class="row"><span class="meta">${one.author}</span> ${revisionImageLink(one.image)}${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : `:${one.line}`}</span> `}${one.note}</p>`),
                 revisionLineageHtml(data.revision.lineage),
                 html `</div>`,
             ], "\n");
@@ -2170,7 +2180,7 @@ export function taskBodyParts(data) {
         const feedback = data.revision;
         thread.push({ key: "revision", at: task.createdAt, kind: "reply", who: "person", author: feedback.comments[0]?.author ?? "You",
             title: feedback.kind === "ci-repair" ? "Asked for a CI repair" : feedback.kind === "criterion-repair" ? "Asked for a repair" : "Asked for changes",
-            text: feedback.comments.map(one => `${one.path === null ? "" : `${one.path}${one.line === null ? "" : `:${one.line}`} — `}${one.note.trim()}`).filter(one => one !== "").join("\n") || null,
+            text: feedback.comments.map(one => `${one.image ? `Screenshot ${one.image.artifact} · result #${one.image.run} — ` : ""}${one.path === null ? "" : `${one.path}${one.line === null ? "" : `:${one.line}`} — `}${one.note.trim()}`).filter(one => one !== "").join("\n") || null,
             link: { label: `From build #${feedback.sourceRun}`, href: `/r/${feedback.sourceRun}` }, html: "", more: null });
     }
     for (const decision of data.decisions) {

package/dist/server/route-table.js

@@ -336,6 +336,7 @@ for (const one of ["policy", "approval", "request-limits", "sessions", "sign-in"
     post(one === "sessions" ? "people" : "settings", `settings.${one.replaceAll("/", "-")}-send`, `/settings/${one}`, { ...(["policy", "approval", "request-limits", "sessions", "sign-in", "updates", "updates/seen", "updates/cancel", "retention", "storage", "storage/clean", "storage/discard", "pull-requests", "checks", "backups", "data", "monitoring", "tools/connect", "tools/change"].includes(one) ? { scope: "step-up" } : {}), ...SETTINGS_ACT_ROLES[one] });
 }
 post("settings", "settings.flows-on", "/settings/flows/on", { limited: "collection", ...approver("An approver switches starter flows on.", "/settings/flows") });
+post("settings", "settings.integrations-youiyoux", "/settings/integrations/youiyoux", { limited: "collection", ...approver("An approver turns youiyoux requests on or off.", "/settings/integrations") });
 const MODELS = approver("Sign in as an approver to change models.", "/settings/models");
 for (const one of ["check", "watch", "update"])
     post("settings", `settings.models-${one}`, `/settings/models/${one}`, MODELS);

package/dist/server/settings.js

@@ -17,7 +17,7 @@ import { grantTool } from "../subagent-tools.js";
 import { learningHtml } from "../workspace-ui.js";
 import { adapterPolicy } from "./route-policy.js";
 import { createHash } from "node:crypto";
-import { dirname, join } from "node:path";
+import { basename, dirname, join } from "node:path";
 import { INSTALLATION_SCOPE } from "../agentconfig.js";
 import { rulesSummary } from "../approval-rules-ui.js";
 import { installMethod } from "../install-method.js";
@@ -64,6 +64,7 @@ import { startersHtml } from "../flow-starters-ui.js";
 import { starterOf, startersFor, switchOnStarter } from "../flow-starters.js";
 import { disconnectGoogle, googleConnected, googleConsent, readGoogleMail, saveGoogleClient } from "../google-mail.js";
 import { integrationsHtml } from "../integrations-ui.js";
+import { setYouiyouxIntake } from "../youiyoux-intake.js";
 import { clearProviderKey, keyStatus, plausibleKey, PROVIDER_KEY_ENV, readAuthMode, readProviderKey, saveProviderKey, setAuthMode, SUBSCRIPTION_CAPABLE, verdictWords, verifyProviderKey } from "../keys.js";
 import { decisionsHtml, framedSection, knowledgeHtml, MEMORY_INTRO, memorySearchHtml, proposalsHtml } from "../knowledge-ui.js";
 import { aboutYouOf, checkAboutYou, saveAboutYou } from "../lead-about.js";
@@ -1386,6 +1387,21 @@ ${repositoryContextHtml(chosen, query, result, who.role === 'approver' ? csrf :
         const switched = switchOnStarter(store, starter, repo, who.name, clock(), options.configDir ?? null);
         return switched.ok ? back("said", switched.said) : back("problem", switched.said);
     }
+    // Settings → Integrations → youiyoux → Turn off / Turn on: whether a project's change requests become tasks.
+    async function integrationsYouiyoux(ctx) {
+        const { who, response, posted } = ctx;
+        const body = readForm(posted, CONSOLE_FORMS.integrationsYouiyoux);
+        const repo = body.get("repo") ?? "";
+        const back = sayTo(response, "/settings/integrations");
+        const known = [...new Set([...(admissionList() ?? []), ...managedRepos(), ...store.knownRepos()])];
+        if (store.isDemo())
+            return refuse(response, who, 403, "An approver turns youiyoux requests on or off.", "/settings/integrations");
+        if (!known.includes(repo) || !visible(repo))
+            return refuse(response, who, 404, "No such project.", "/settings/integrations");
+        const on = body.get("on") === "1";
+        setYouiyouxIntake(store, repo, on, who.name, clock());
+        return back("said", on ? `youiyoux requests in ${basename(repo)} become tasks again.` : `youiyoux requests in ${basename(repo)} are off.`);
+    }
     // Settings → Integrations → Send test: one harmless read-only check of one integration, now.
     async function integrationsTest(ctx) {
         const { who, response, posted } = ctx;
@@ -2642,6 +2658,7 @@ ${modeWords(terms)
         "settings.backups-send": backupsSave,
         "settings.data-send": dataExport,
         "settings.integrations-test-send": integrationsTest,
+        "settings.integrations-youiyoux": integrationsYouiyoux,
         "settings.monitoring-send": monitoringSave,
         "settings.tools-connect-send": toolsConnect,
         "settings.tools-change-send": toolsChange,

package/dist/server/task-views.js

@@ -1,3 +1,5 @@
+import { feedbackImageProblem } from "../result-actions.js";
+import { parseFeedbackImage } from "../result-review.js";
 import { releaseViewOf } from "../release-view.js";
 import { releaseCiRunningOf } from "../release-ci-hold.js";
 import { releaseObligationsOf } from "../release-recheck.js";
@@ -229,6 +231,7 @@ export function createTaskViews(runtime) {
                     line: one.line === null || one.line === undefined ? null : Number(one.line),
                     note: String(one.note ?? ""),
                     author: String(one.author ?? "?"),
+                    image: parseFeedbackImage(one.image) || null,
                 })),
             };
         }
@@ -1119,6 +1122,9 @@ export function createTaskViews(runtime) {
             outsideTouches: files.filter(one => one.outsideTouches).map(one => one.path),
             fileAnchors,
             comments: runtime.store.liveDiffComments(run.id).filter(isRevisionFeedback),
+            // The same check that refuses a saved note's screenshot at revision time decides when it can be removed.
+            unusableImageNotes: runtime.store.liveDiffComments(run.id).filter(one => isRevisionFeedback(one) && one.image != null
+                && feedbackImageProblem(runtime.store, runtime.evidenceRoot, run.id, one.image) !== null).map(one => one.id),
             pastComments: runtime.store.allDiffComments(run.id).filter(one => one.reviewerRun === null && (one.consumedBy !== null || one.supersededBy !== null)),
             reviewerFindings: runtime.store.allDiffComments(run.id).filter(one => one.reviewerRun !== null),
             automaticReview: buildReviewOf(runtime.store, run.id),

package/dist/server/tasks.js

@@ -1,4 +1,5 @@
-import { releaseAuthorityOfTask, revokeRelease, runLiveCommands } from "../store.js";
+import { parseFeedbackImage, sameFeedbackImage } from "../result-review.js";
+import { dropFeedbackImage, releaseAuthorityOfTask, revokeRelease, runLiveCommands } from "../store.js";
 import { steerCapabilityOf } from "../steer-capabilities.js";
 import { canReveal, localCheckoutOf, revealInFinder } from "../checkout-proof.js";
 /** A form's way back to a flow card ("flow:<flow>:<card>"), read strictly: numbers only, never a path. */
@@ -48,7 +49,7 @@ import { fileTaskProposal } from "../proposal.js";
 import { isProviderId, PROVIDER_IDS, validateSpec } from "../provider.js";
 import { completeAndOpenPullRequest, mergeAsPerson } from '../pull-request-flow.js';
 import { isQualityMode } from "../quality.js";
-import { createResultRevision, requestResultChanges } from "../result-actions.js";
+import { createResultRevision, requestResultChanges, feedbackImageProblem } from "../result-actions.js";
 import { structuredHandoffView, terminalDiffView } from "../result-evidence-readers.js";
 import { fileAddTestsTask, requestFollowUpChecks } from '../result-follow-ups.js';
 import { commentSourceKey, isRevisionFeedback, parseResultTab, resultReturnTarget } from "../result-review.js";
@@ -1402,15 +1403,40 @@ export function createTasksHandlers(runtime) {
         const viewParam = returnTo.startsWith("/t/") ? "section" : "tab";
         const back = selectedTab === "summary" ? returnTo
             : `${returnTo.replace(/&section=(?:summary|changes|checks)$/, "")}${returnTo.includes("?") ? "&" : "?"}${viewParam}=${selectedTab}`;
+        if (body.get("intent") === "drop-image") {
+            // A saved note whose screenshot is gone or changed would block every revision that seals it. Its words stay:
+            // the note is superseded by an exact imageless copy, and the original remains in history.
+            // Who: a signed-in browser approver admitted to this run's project (checked above) — the same authority that
+            // seals any saved note into a revision. What: only a screenshot that is unusable right now, re-proved inside
+            // dropFeedbackImage's transaction, and recorded in the ledger under this person.
+            if (who.via !== "cookie" || who.role !== "approver")
+                return refuse(response, who, 403, "Only a signed-in approver can change saved feedback.", back);
+            const comment = Number(body.get("comment") ?? "");
+            if (!Number.isSafeInteger(comment) || comment < 1)
+                return refuse(response, who, 400, "Choose a saved note.", back);
+            const dropped = dropFeedbackImage(store, id, comment, who.name, one => feedbackImageProblem(store, evidenceRoot, id, one), now);
+            if (!dropped.ok && dropped.reason === "not-authorized")
+                return refuse(response, who, 403, "Only a signed-in approver on this project can change saved feedback.", back);
+            if (!dropped.ok)
+                return refuse(response, who, 409, dropped.reason === "image-usable"
+                    ? "That screenshot still verifies, so the note keeps it. Reload the result." : "That saved note changed or was already used. Reload the result.", back);
+            return redirect(response, `${back}#request-changes`);
+        }
+        const image = parseFeedbackImage(body.get("image"));
+        if (image === false)
+            return refuse(response, who, 400, "Choose a screenshot from this result again. Your draft is unchanged.", back);
         if (body.get("intent") === "revise") {
             const result = requestResultChanges(store, evidenceRoot, {
                 run: id, batch: body.get("batch"), source: body.get("source"), actor: who.name,
                 repos: admissionList(), includeUnplaced: visible(null), allowMode: who.via === "cookie",
-                note: body.get("note") ?? "", path: body.get("path") ?? "", line: body.get("line") ?? "", request: body.get("request"),
+                note: body.get("note") ?? "", path: body.get("path") ?? "", line: body.get("line") ?? "", request: body.get("request"), image,
             }, now);
             if (!result.ok) {
                 const request = body.get("request") ?? "";
-                const retryBack = /^[a-f0-9]{32}$/.test(request) ? `${back}${back.includes("?") ? "&" : "?"}conflict=${request}#request-changes` : back;
+                // The queue's link can stop resolving when a newer attempt finishes. The exact
+                // run record still hosts this account/run's recoverable feedback draft.
+                const retryBase = image === null ? back : `/r/${id}?record=1`;
+                const retryBack = /^[a-f0-9]{32}$/.test(request) ? `${retryBase}${retryBase.includes("?") ? "&" : "?"}conflict=${request}#request-changes` : retryBase;
                 return refuse(response, who, result.status, result.message, retryBack);
             }
             return redirect(response, revisionDestination(result.id, returnTo));
@@ -1445,7 +1471,17 @@ export function createTasksHandlers(runtime) {
         const request = body.get("request");
         const sourceKey = commentSourceKey(who.name, request);
         const path = rawPath === "" ? null : rawPath;
-        const inserted = store.addDiffComment({ artifactId: terminal.id, runId: id, path, line, note: note.note, author: who.name, ...(sourceKey === undefined ? {} : { sourceKey }) }, now);
+        // Freshness, custody and the insert in one write transaction: a newer attempt finishing in between can't leave a
+        // stale screenshot attached (check and write together).
+        const saved = store.transact(() => {
+            const imageProblem = feedbackImageProblem(store, evidenceRoot, id, image);
+            if (imageProblem !== null)
+                return { problem: imageProblem };
+            return { inserted: store.addDiffComment({ artifactId: terminal.id, runId: id, path, line, note: note.note, author: who.name, image, ...(sourceKey === undefined ? {} : { sourceKey }) }, now) };
+        });
+        if ("problem" in saved)
+            return refuse(response, who, 409, saved.problem, `/r/${id}?record=1#request-changes`);
+        const inserted = saved.inserted;
         if (inserted === null && sourceKey !== undefined) {
             // The token was used before (repair 2026-09-14, finding 1). A
             // request identity is IMMUTABLE: it names exactly one note on
@@ -1459,7 +1495,7 @@ export function createTasksHandlers(runtime) {
             if (earlier === null || earlier.run !== id) {
                 return refuse(response, who, 409, `this note's request identity was already used on another result (build #${earlier === null ? "?" : earlier.run}) — go back; your words are kept, and the form will carry a new identity`, conflictBack);
             }
-            if (earlier.path !== path || earlier.line !== line || earlier.note !== note.note) {
+            if (earlier.path !== path || earlier.line !== line || earlier.note !== note.note || !sameFeedbackImage(earlier.image, image)) {
                 return refuse(response, who, 409, `this request identity already recorded a different note on this result (${earlier.path === null ? "no file" : `${earlier.path}${earlier.line === null ? "" : `:${earlier.line}`}`}: "${oneLineOf(earlier.note, 80)}") — go back; your edited words are kept as a new note, and the form will carry a new identity`, conflictBack);
             }
         }

package/dist/store.d.ts

@@ -1,3 +1,4 @@
+import { type FeedbackImage } from "./result-review.js";
 import { type WritesMode } from "./writes-mode.js";
 import { type CheckAccess } from "./check-access.js";
 import { type ProjectSandbox } from "./project-sandbox.js";
@@ -139,6 +140,18 @@ export declare const V141_ADDED_COLUMNS: Readonly<Record<string, Readonly<Record
     definition: string;
     saved: string;
 }>>>>;
+/** v142 (youiyoux-intake.ts): youiyoux change requests as tasks. `youiyoux_intake` is one row per project the intake
+ * has looked at: a person's on/off choice (`enabled` NULL: automatic, on while the project uses youiyoux) and the last
+ * pass's answer (whether the project uses youiyoux, how many request files couldn't be read, and the problem, if any).
+ * `youiyoux_request` is one row per request taken, keyed by project and request id: the task it became (NULL: left
+ * out, `note` says why), never filed again, the status last written beside it, with the console address its links
+ * named ('' for none), and where the person pointed (route, component, file:line), cleaned, for the task page. An
+ * options request (`kind`) also keeps the options file its agent writes (`proposal`, p-<its hex>), set when filed and
+ * never read from the request again: whether it went to the project's main checkout from which result (`proposal_state`,
+ * `proposal_run`, and why not in `proposal_note`; `taken`: a file was already there, so it was left and is never read
+ * for a decision), the digest of what was copied apart from what a decision changes (`proposal_digest`), and what a
+ * person decided on it in youiyoux's hub (`decided`, `decided_note`: what Toolroll did about it). New tables only. */
+export declare const V142_SCHEMA = "\nCREATE TABLE IF NOT EXISTS youiyoux_intake (\n  repo        TEXT PRIMARY KEY,\n  enabled     INTEGER CHECK (enabled IS NULL OR enabled IN (0, 1)),\n  uses        INTEGER NOT NULL DEFAULT 0 CHECK (uses IN (0, 1)),\n  changed_by  TEXT,\n  changed_at  TEXT,\n  checked_at  TEXT,\n  invalid     INTEGER NOT NULL DEFAULT 0 CHECK (invalid >= 0),\n  problem     TEXT\n);\nCREATE TABLE IF NOT EXISTS youiyoux_request (\n  repo          TEXT NOT NULL,\n  request       TEXT NOT NULL CHECK (length(request) = 12 AND request GLOB 'r-[0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f]'),\n  task_id       TEXT,\n  note          TEXT,\n  filed_at      TEXT NOT NULL,\n  status        TEXT CHECK (status IS NULL OR status IN ('queued','needs-you','building','ready','done','failed','stopped')),\n  detail        TEXT,\n  status_at     TEXT,\n  written_at    TEXT,\n  written_base  TEXT,\n  write_problem TEXT,\n  route         TEXT,\n  component     TEXT,\n  source_at     TEXT,\n  kind          TEXT NOT NULL DEFAULT 'change' CHECK (kind IN ('change','options')),\n  proposal      TEXT CHECK (proposal IS NULL OR (length(proposal) = 12 AND proposal GLOB 'p-[0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f]')),\n  proposal_state TEXT CHECK (proposal_state IS NULL OR proposal_state IN ('copied','taken','missing','unsafe','retry')),\n  proposal_digest TEXT CHECK (proposal_digest IS NULL OR (length(proposal_digest) = 64 AND proposal_digest NOT GLOB '*[^0-9a-f]*')),\n  proposal_run  INTEGER,\n  proposal_note TEXT,\n  proposal_at   TEXT,\n  decided       TEXT CHECK (decided IS NULL OR decided IN ('chosen','dismissed')),\n  decided_at    TEXT,\n  decided_note  TEXT,\n  PRIMARY KEY (repo, request),\n  UNIQUE (task_id),\n  CHECK ((kind = 'options') = (proposal IS NOT NULL)),\n  CHECK (proposal_state IS NULL OR kind = 'options'),\n  CHECK (decided IS NULL OR coalesce(proposal_state = 'copied' AND proposal_digest IS NOT NULL, 0))\n);\n";
 /** A project's "Show commands in the live view" (live-detail.ts): absent means off. */
 export declare const LIVE_DETAIL_SCHEMA = "\nCREATE TABLE IF NOT EXISTS live_detail_setting (\n  repo       TEXT PRIMARY KEY,\n  commands   INTEGER NOT NULL CHECK (commands IN (0,1)),\n  updated_by TEXT NOT NULL,\n  updated_at TEXT NOT NULL\n);\n-- Every run a steering note reached, kept as it happens (live-notes.ts): a\n-- note's own attached/mailed fields move to the next attempt, this does not.\nCREATE TABLE IF NOT EXISTS steer_note_run (\n  note INTEGER NOT NULL REFERENCES task_steer(id) ON DELETE CASCADE,\n  run  INTEGER NOT NULL REFERENCES run(id) ON DELETE CASCADE,\n  how  TEXT NOT NULL CHECK (how IN ('attached','mailed','read')),\n  at   TEXT NOT NULL,\n  PRIMARY KEY (note, run, how)\n);\nCREATE INDEX IF NOT EXISTS steer_note_run_by_run ON steer_note_run (run, note);\n";
 export declare const CHAT_APPROVAL_SCHEMA = "\nCREATE TABLE IF NOT EXISTS chat_approval_setting (\n  approver             TEXT NOT NULL,\n  scope                TEXT NOT NULL,\n  enabled              INTEGER NOT NULL CHECK (enabled IN (0,1)),\n  full_access          INTEGER NOT NULL DEFAULT 0 CHECK (full_access IN (0,1)),\n  attempt_cap_microusd INTEGER CHECK (attempt_cap_microusd IS NULL OR attempt_cap_microusd >= 0),\n  generation           INTEGER NOT NULL,\n  digest               TEXT NOT NULL,\n  updated_by           TEXT NOT NULL,\n  updated_at           TEXT NOT NULL,\n  PRIMARY KEY (approver, scope)\n);\n";
@@ -219,7 +232,7 @@ export type SealedLedgerEntry = LedgerEntry & {
         hash: string;
     } | null;
 };
-export declare const SCHEMA_VERSION = 141;
+export declare const SCHEMA_VERSION = 143;
 /** v118: the tables merged into `credential`, in the order they move (a person's API tokens keep their ids first). */
 export declare const V118_MERGED_TABLES: Readonly<Record<string, CredentialKind>>;
 /**
@@ -376,6 +389,11 @@ export declare const V140_ADDED_COLUMNS: Readonly<Record<string, Readonly<Record
     definition: string;
     saved: string;
 }>>>>;
+/** v143: optional screenshot context on result feedback; existing feedback stays unchanged (saved rows read NULL). */
+export declare const V143_ADDED_COLUMNS: Readonly<Record<string, Readonly<Record<string, {
+    definition: string;
+    saved: string;
+}>>>>;
 /**
  * v135: one approval per release (docs/plans/release-once.md). `release_authority` is the single source of truth for a
  * release a person approved: one row per generation, changed only by compare-and-swap on `rev`, with a lease whose
@@ -2406,8 +2424,9 @@ export declare class Store {
     sealScopeApproval(taskId: string, by: string, now: Date, mutation?: Mutation, basis?: {
         kind: "mode" | "chat" | "grant" | "schedule" | "auto-check" | "auto-green";
         modeDigest: string;
-        /** A chat seal of a deploy gate: the release card it came from (chat-management.md §0a.5). */
-        card?: "release";
+        /** A chat seal of a deploy gate: the release card it came from (chat-management.md §0a.5). A chat seal of a task
+         * filed from outside text: `terms`, and the digest of every word the card showed (chat-terms.ts). */
+        card?: "release" | "terms";
         cardDigest?: string;
     }, approverKind?: ApproverKind): boolean;
     /** Whether this task's scope carries a LIVE seal: approved, and the
@@ -3207,6 +3226,7 @@ export declare class Store {
         author: string;
         /** Ingested comments carry their origin id — the idempotency key (M8.17). */
         sourceKey?: string;
+        image?: FeedbackImage | null;
     }, now: Date): number | null;
     /** Live (unsuperseded, unconsumed) comments on a run's terminal diff. */
     liveDiffComments(runId: number): DiffComment[];
@@ -3405,6 +3425,15 @@ export declare class Store {
         entry: number;
         sourceJson: string;
     } | null;
+    /** v142: where a task filed from a youiyoux change or options request came from, as kept when it was filed
+     * (youiyoux-intake.ts; a revision reads its first version's): the request, its kind, and where the person pointed. Null for every other task. */
+    youiyouxOriginOf(taskId: string): {
+        request: string;
+        kind: "change" | "options";
+        route: string | null;
+        component: string | null;
+        at: string | null;
+    } | null;
     /**
      * Seal a revision in ONE transaction (Codex M5-M8 audit, C-3; contract
      * handoff task 2): the SOURCE BINDING is proven here, against live rows,
@@ -7376,6 +7405,7 @@ export type DiffComment = {
     id: number;
     artifact: number;
     artifactSha: string;
+    image?: FeedbackImage | null;
     run: number;
     path: string | null;
     line: number | null;
@@ -8021,3 +8051,16 @@ export declare function sealStoreMoneyMethods(prototype: object): void;
  * when the prototype's value is the very reference recorded at the seal. `prototype` is Store's own unless a test
  * says otherwise; either way nothing but the recorded references is ever installed. */
 export declare function lockStoreMoneyMethods(store: object, prototype?: object): void;
+/** Takes a saved note's screenshot off without losing the note or its history (v143). Fail closed, in one write
+ * transaction: the note must be live and unconsumed on this run, its screenshot must be genuinely unusable right now
+ * (`unusable` is result-actions.ts feedbackImageProblem, re-read inside the transaction, never the page's say-so), and
+ * the change is recorded in the action ledger under the person who made it. The note is superseded by an exact
+ * imageless copy; the original stays in history. The caller has already proved the person may request changes on
+ * this run (the same authority that seals any saved note into a revision); that standing is re-proved here too. */
+export declare function dropFeedbackImage(store: Store, runId: number, commentId: number, actor: string, unusable: (image: FeedbackImage) => string | null, now: Date): {
+    ok: true;
+    id: number;
+} | {
+    ok: false;
+    reason: "not-found" | "image-usable" | "not-authorized";
+};