toolroll 0.9.5 → 0.9.6

package/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 ## Unreleased
 
+## 0.9.6 — 2026-10-01
+
+- **Every task's status, clear at a glance.** One headline from a fixed set
+  (Queued, Planning, Needs you, Building, Ready for review, Complete, Failed,
+  Stopped) and one plain sentence, the same words on the Tasks list, the task
+  page, the result, Crew, the Telegram/Slack/Discord/Teams card and
+  `toolroll status`. Checks, the pull request, requirements and saved
+  evidence sit underneath as quiet rows. A pull request that couldn't open
+  is an amber note with one action, never a red chip beside Complete: red
+  now means only Failed. The exact technical reason stays under Details.
+
 ## 0.9.5 — 2026-10-01
 
 - **A demo you can feel in 90 seconds.** In `npx toolroll demo`, Chat has a

package/dist/assignment-presentation.d.ts

@@ -3,6 +3,7 @@
  * checks, approvals, ownership, or the exact completion identity. */
 import type { AssignmentSnapshot } from './assignment.js';
 import { type DisplayStatus, type WorkStatus } from './workspace-ui.js';
+import { type PullRequestFact, type StatusAction, type TaskStatus, type TaskStatusFacts } from './task-status.js';
 export type AssignmentWorkStatus = DisplayStatus & Partial<Pick<WorkStatus, 'views' | 'rank'>>;
 export type AssignmentAttention = {
     id: string;
@@ -14,12 +15,22 @@ export type AssignmentAttention = {
 export declare function historicalAssessmentReason(reason: string): boolean;
 /** Known storage-cap notices do not mean the retained bytes are damaged. */
 export declare function shortenedMaterialReason(detail: string): boolean;
-export declare function assignmentPresentationOf(assignment: AssignmentSnapshot, options?: {
+export type AssignmentStatusOptions = {
     workStatus?: AssignmentWorkStatus;
     diagnostics?: WorkStatus['diagnostics'];
     additionalAttention?: readonly string[];
-}): {
+    /** A live planner rather than a builder holds the work. */
+    planning?: boolean;
+    pullRequest?: PullRequestFact | null;
+    action?: StatusAction | null;
+    links?: TaskStatusFacts['links'];
+    evidence?: TaskStatusFacts['evidence'];
+};
+/** The one shared status (task-status.ts) for this assignment. */
+export declare function assignmentTaskStatusOf(assignment: AssignmentSnapshot, options?: AssignmentStatusOptions): TaskStatus;
+export declare function assignmentPresentationOf(assignment: AssignmentSnapshot, options?: AssignmentStatusOptions): {
     status: WorkStatus;
+    taskStatus: TaskStatus;
     primaryAction: import("./work-summary.js").WorkAction | null;
     attention: AssignmentAttention[];
     diagnostics: Pick<DisplayStatus, "detail" | "label" | "token" | "tone">[];

package/dist/assignment-presentation.js

@@ -1,9 +1,6 @@
 import { GOAL_ASSESSMENT_PENDING, manualReviewCriterionOf } from './proof.js';
 import { failedCheckExit } from './workspace-ui.js';
-const LABELS = {
-    working: 'Working', checking: 'Checking', 'needs-decision': 'Needs your decision',
-    'ready-to-check': 'Ready', complete: 'Complete', cancelled: 'Cancelled',
-};
+import { assignmentStatusFacts, taskStatusOf, workToneOf } from './task-status.js';
 /** Only the retired assessment requirement is history-only. Specific missing
  * material, defects and failed checks remain visible, including reviewer findings. */
 export function historicalAssessmentReason(reason) {
@@ -19,13 +16,29 @@ export function shortenedMaterialReason(detail) {
         detail === 'The changed-file list was cut short; the counts are complete.' ||
         /^Saved (?:terminal-diff|report) #\d+ is incomplete\.$/.test(detail);
 }
+/** The one shared status (task-status.ts) for this assignment. */
+export function assignmentTaskStatusOf(assignment, options = {}) {
+    const work = options.workStatus;
+    return taskStatusOf(assignmentStatusFacts(assignment, {
+        work: work === undefined ? null : { token: work.token, ...(work.views === undefined ? {} : { views: work.views }), detail: work.detail },
+        ...(options.planning === undefined ? {} : { planning: options.planning }),
+        ...(options.pullRequest === undefined ? {} : { pullRequest: options.pullRequest }),
+        ...(options.action === undefined ? {} : { action: options.action }),
+        ...(options.links === undefined ? {} : { links: options.links }),
+        ...(options.evidence === undefined ? {} : { evidence: options.evidence }),
+    }));
+}
 export function assignmentPresentationOf(assignment, options = {}) {
     const { state } = assignment, work = options.workStatus;
+    const taskStatus = assignmentTaskStatusOf(assignment, options);
+    // The headline and its sentence are the shared words; the token, views and
+    // rank stay the existing projection's (pages and tests key off them).
+    const shared = { label: taskStatus.headline, tone: workToneOf(taskStatus.headline), detail: taskStatus.sentence, action: null };
+    const needsYou = taskStatus.headline === 'Needs you';
     const status = state === 'working'
-        ? { ...(work ?? { label: LABELS[state], tone: 'muted' }), token: `assignment-${state}`,
-            detail: assignment.detail, action: null, views: work?.views?.filter(view => view !== 'completed') ?? ['all'], rank: work?.rank ?? 2 }
-        : { token: `assignment-${state}`, label: state === 'checking' ? work?.label ?? LABELS[state] : LABELS[state], detail: assignment.detail,
-            action: null, tone: state === 'needs-decision' ? 'attention' : state === 'checking' ? work?.tone ?? 'live' : state === 'ready-to-check' ? 'ready' : state === 'complete' ? 'done' : 'muted',
+        ? { ...shared, token: `assignment-${state}`,
+            views: [...new Set([...(work?.views?.filter(view => view !== 'completed') ?? ['all']), ...(needsYou ? ['needs-you'] : [])])], rank: needsYou ? 0 : work?.rank ?? 2 }
+        : { ...shared, token: `assignment-${state}`,
             views: state === 'ready-to-check' || state === 'needs-decision' ? ['all', 'needs-you'] : state === 'complete' ? ['all', 'completed']
                 : state === 'checking' && work?.views?.includes('running') ? ['all', 'running'] : ['all'],
             rank: state === 'needs-decision' || state === 'ready-to-check' ? 0 : state === 'checking' ? 1 : state === 'complete' ? 3 : 4 };
@@ -78,5 +91,5 @@ export function assignmentPresentationOf(assignment, options = {}) {
             return false;
         return true;
     });
-    return { status, primaryAction: assignment.primaryAction, attention: [...attention.values()], diagnostics };
+    return { status, taskStatus, primaryAction: assignment.primaryAction, attention: [...attention.values()], diagnostics };
 }

package/dist/assignment-ui.d.ts

@@ -2,7 +2,8 @@
  * existing projections and signed operating-mode terms, never this renderer. */
 import type { AssignmentSnapshot } from './assignment.js';
 import type { DisplayStatus, StatusTone, WorkStatus } from './workspace-ui.js';
-import { type AssignmentWorkStatus } from './assignment-presentation.js';
+import { type AssignmentStatusOptions, type AssignmentWorkStatus } from './assignment-presentation.js';
+import { type PullRequestFact, type TaskStatus } from './task-status.js';
 /** The existing verified receipt reader may discover damage after a verdict
  * was saved. Keep the limitation visible without inventing another work stage. */
 export declare function assignmentWithEvidence(assignment: AssignmentSnapshot, resultStatus: DisplayStatus | null, resultRunId?: number | null): AssignmentSnapshot;
@@ -19,38 +20,32 @@ export type AssignmentAttemptRow = {
 /** Every version of the assignment, oldest first, as the card lists them. */
 export declare function assignmentAttemptsOf(assignment: AssignmentSnapshot): AssignmentAttemptRow[];
 export declare function assignmentAttemptsHtml(assignment: AssignmentSnapshot): string;
-type AssignmentCardOptions = {
+type AssignmentCardOptions = Omit<AssignmentStatusOptions, 'links'> & {
     hideAction?: boolean;
     problem?: boolean;
-    diagnostics?: WorkStatus['diagnostics'];
-    workStatus?: AssignmentWorkStatus;
     resultHref?: string;
+    pullRequest?: PullRequestFact | null;
 };
 /** The status card's content: one projection for the server HTML and the
- * rebuilt task page, so the two never word the same state differently. */
+ * rebuilt task page, so the two never word the same state differently. The
+ * headline, sentence and detail rows are the shared task status; the exact
+ * recorded reasons stay one tap away under Details, never as red text. */
 export type AssignmentCard = {
     token: string;
     tone: StatusTone;
     label: string;
+    status: TaskStatus;
     action: {
         label: string;
         href: string;
         openResult: boolean;
     } | null;
-    /** Ready with passing checks: a verdict chip replaces the detail sentence. */
-    passed: {
-        by: string | null;
-    } | null;
-    detail: {
-        text: string;
-        problem: boolean;
-    };
-    problems: string[];
+    /** Exact technical reasons and diagnostics, shown on request. */
+    reasons: string[];
     diagnostics: {
         token: string;
         label: string;
         detail: string;
-        problem: boolean;
     }[];
     notices: {
         summary: string;

package/dist/assignment-ui.js

@@ -1,5 +1,7 @@
 import { assignmentPresentationOf, shortenedMaterialReason } from './assignment-presentation.js';
 import { plainReasonWords } from './proof.js';
+import { chatResultHref } from './chat-controls.js';
+import { statusDetailsHtml, statusWhyHtml } from './task-status.js';
 const escape = (value) => value.replace(/[&<>"']/g, char => ({ '&': '&amp;', '<': '&lt;', '>': '&gt;', '"': '&quot;', "'": '&#39;' })[char]);
 /** The existing verified receipt reader may discover damage after a verdict
  * was saved. Keep the limitation visible without inventing another work stage. */
@@ -63,25 +65,23 @@ function savedMaterialNotice(detail, assignment) {
         !assignment.receipt.artifacts.some(one => one.id === Number(earlier[1])) ? 'history' : null;
 }
 export function assignmentCardOf(assignment, options = {}) {
-    const presentation = assignmentPresentationOf(assignment, options);
-    const { status, diagnostics } = presentation;
+    const receipt = assignment.receipt;
+    const links = receipt === null ? {} : { result: options.resultHref ?? chatResultHref(receipt.taskId, receipt.runId), checks: chatResultHref(receipt.taskId, receipt.runId, 'checks'),
+        pullRequest: `/t/${encodeURIComponent(assignment.rootId)}#merge` };
+    const presentation = assignmentPresentationOf(assignment, { ...options, links });
+    const { status, diagnostics, taskStatus } = presentation;
     const href = assignment.primaryAction?.code === 'open-result' && options.resultHref !== undefined ? options.resultHref : assignmentActionHref(assignment);
     const attention = presentation.attention.map(one => one.detail);
     const notices = [...attention, ...diagnostics.map(one => one.detail)].filter(one => savedMaterialNotice(one, assignment) !== null);
     const partial = notices.some(one => savedMaterialNotice(one, assignment) === 'partial');
     const history = notices.some(one => savedMaterialNotice(one, assignment) === 'history');
-    const ready = assignment.state === "ready-to-check" || assignment.state === "complete";
-    const checkProblem = ready && assignment.receipt !== null && assignment.receipt.completionKind !== 'research-report' && assignment.receipt.checks.status !== 'passed';
+    const recorded = assignment.detail === taskStatus.sentence || taskStatus.headline === 'Ready for review' || taskStatus.headline === 'Complete' ? [] : [assignment.detail];
     return {
-        token: status.token, tone: status.tone, label: status.label,
+        token: status.token, tone: status.tone, label: status.label, status: taskStatus,
         action: options.hideAction || href === null ? null : { label: assignment.primaryAction.label, href, openResult: options.resultHref !== undefined },
-        // A good outcome at a glance; the same words stay in the result.
-        passed: ready && !checkProblem && assignment.receipt !== null && assignment.receipt.checks.status === 'passed'
-            ? { by: assignment.completion === null ? null : assignment.completion.actor.replace(/^(?:operator|coordinator|lead):/, '') } : null,
-        detail: { text: assignment.detail, problem: checkProblem || options.problem === true && !ready },
-        problems: attention.filter(one => savedMaterialNotice(one, assignment) === null).map(plainReasonWords),
+        reasons: [...new Set([...recorded, ...attention.filter(one => savedMaterialNotice(one, assignment) === null).map(plainReasonWords)])].filter(one => one !== taskStatus.sentence),
         diagnostics: diagnostics.filter(one => savedMaterialNotice(one.detail, assignment) === null)
-            .map(one => ({ token: one.token, label: one.label, detail: one.detail, problem: one.tone === 'problem' || one.tone === 'attention' })),
+            .map(one => ({ token: one.token, label: one.label, detail: one.detail })),
         notices: notices.length === 0 ? null : { summary: [partial ? 'Saved output is partial' : '', history ? 'Earlier material unavailable' : ''].filter(Boolean).join(' · '), lines: [...new Set(notices)] },
         attempts: assignmentAttemptsOf(assignment),
         lead: assignment.owner === null ? null : { label: assignment.owner.label, active: assignment.owner.active },
@@ -89,14 +89,13 @@ export function assignmentCardOf(assignment, options = {}) {
 }
 export function assignmentSummaryHtml(assignment, options = {}) {
     const card = assignmentCardOf(assignment, options);
-    return `<section class="${options.compact ? 'assignment-summary' : 'card assignment-summary'}" aria-label="assignment progress" data-assignment="${escape(assignment.rootId)}" data-work-status="${card.token}" data-tone="${card.tone}"${options.compact ? '' : ' data-task-status'}>` +
-        (options.compact ? `<span class="status-line" data-work-status="${card.token}" data-tone="${card.tone}"><i class="status-dot" aria-hidden="true"></i><span class="status-label">${card.label}</span></span>` : `<h2 class="assignment-state">${card.label}</h2>`) +
+    const tone = card.status.tone;
+    return `<section class="${options.compact ? 'assignment-summary' : 'card assignment-summary'}" aria-label="assignment progress" data-assignment="${escape(assignment.rootId)}" data-work-status="${card.token}" data-tone="${card.tone}" data-headline="${escape(card.status.headline)}" data-headline-tone="${tone}"${options.compact ? '' : ' data-task-status'}>` +
+        (options.compact ? `<span class="status-line" data-work-status="${card.token}" data-tone="${card.tone}"><i class="status-dot" aria-hidden="true"></i><span class="status-label">${card.label}</span></span>` : `<h2 class="assignment-state status-headline"><i aria-hidden="true"></i>${escape(card.label)}</h2>`) +
+        `<p class="meta assignment-detail status-sentence">${escape(card.status.sentence)}</p>` +
         (card.action === null ? '' : `<a class="${options.compact ? 'work-action' : 'button-link'}" href="${escape(card.action.href)}"${card.action.openResult ? ' data-open-result' : ''}${options.compact ? '' : ' data-primary-action'}>${escape(card.action.label)}${options.compact ? ' →' : ''}</a>`) +
-        (!options.compact && card.passed !== null
-            ? `<p class="meta assignment-detail assignment-verdict"><span class="verdict-chip verdict-chip--success">Checks passed</span>${card.passed.by === null ? '' : ` Completed by ${escape(card.passed.by)}`}</p>`
-            : `<p class="${card.detail.problem ? 'problem' : 'meta'} assignment-detail">${escape(card.detail.text)}</p>`) +
-        card.problems.map(one => `<p class="problem">${escape(one)}</p>`).join('') +
-        card.diagnostics.map(one => `<p class="${one.problem ? 'problem' : 'meta'}" data-work-diagnostic="${escape(one.token)}">${escape(one.label)} · ${escape(one.detail)}</p>`).join('') +
+        (options.compact ? '' : statusDetailsHtml(card.status)) +
+        statusWhyHtml(card.status, card.reasons, card.diagnostics) +
         (card.notices === null ? '' : `<details class="assignment-notices"><summary>${card.notices.summary}</summary>${card.notices.lines.map(one => `<p class="meta">${escape(one)}</p>`).join('')}</details>`) +
         assignmentAttemptsHtml(assignment) + `</section>`;
 }