rvlib-shipkit 2.4.0 → 2.5.0

package/src/home/process-rows.ts

@@ -146,6 +146,8 @@ export const CORE_ABOUT = {
 export type CoreOpts = {
     /** the daemon's own version, asked of the daemon; null when it does not answer */
     daemonVersion?: string | null
+    /** the daemon runs older code than its tree, and why; null when it does not */
+    daemonHeld?: string | null
     /** where each listens, when pm does not say (an older pm, or pm down) */
     ports?: CorePorts | null
     /** one of shipkit's own not answering the page, and why: its row says
@@ -174,9 +176,9 @@ export function coreGroup(pm: PmAnswer | null, o: CoreOpts = {}): ProcessGroup {
         const c = pm?.children.find((x) => x.id === id)
         const port = ports?.[id] ?? null
         const version = id === 'daemon' ? (o.daemonVersion ?? null) : null
-        if (c == null) rows.push(coreRow(id, { status: 'unknown', pid: null, since: null, note: null, tail: [], version, port, restart: false }))
+        if (c == null) rows.push(coreRow(id, { status: 'unknown', pid: null, since: null, note: id === 'daemon' ? (o.daemonHeld ?? null) : null, tail: [], version, port, restart: false }))
         else {
-            const note = [c.note, c.restarts > 0 ? `${c.restarts} restart(s)` : null].filter((x) => x != null).join(', ')
+            const note = [id === 'daemon' ? (o.daemonHeld ?? null) : null, c.note, c.restarts > 0 ? `${c.restarts} restart(s)` : null].filter((x) => x != null).join(', ')
             rows.push(coreRow(id, { status: c.status, pid: c.pid, since: c.since, note: note === '' ? null : note, tail: c.tail, version, port, restart: true }))
         }
     }

package/src/home/run-daemon.ts

@@ -115,7 +115,7 @@ export async function runDaemon(root: string, io: HomeIo, o: { port?: number } =
             // the rows every gate filed land here, one commit a day at most
             also: (now) => { for (const n of landMotherhome({ now })) io.printErr(n) },
         })
-        const code = reloadOnOwnCode({ role: 'daemon', entry: fileURLToPath(import.meta.url), busy: () => hubs.busy(), before: () => { sweeps.stop(); home.stop() }, io })
+        const code = reloadOnOwnCode({ role: 'daemon', entry: fileURLToPath(import.meta.url), busy: () => hubs.busy(), before: () => { sweeps.stop(); home.stop() }, keep: true, io })
         io.print(`  ${house.repos.length} repos, state in ${dir}, reloads on ${code.files} lib files`)
         for (const sig of ['SIGINT', 'SIGTERM'] as const)
             process.on(sig, () => {

package/src/review/model.ts

@@ -186,6 +186,25 @@ export const asOptions = (i: Item): boolean => isChoice(i) || (i.views.length >=
 /** the most stars a reaction gives */
 export const STARS = 5
 
+/** where a card is, for a fresh agent session in any repo: the repo as `~/dev/<repo>`, the card's url where it is open
+ *  now, and its phone url when the repo has a public host */
+export interface CardWhere {
+    root: string
+    open: string
+    phone?: string | null
+}
+
+/** one paste that hands a card to any agent session: what it is, where, the command that shows it whole, its links */
+export function pasteBlock(i: Item, state: string, w: CardWhere): string {
+    return [
+        `review card "${i.title}" (${state}), id ${i.id}`,
+        `repo: ${w.root}`,
+        `see it whole: cd ${w.root} && sk review show ${i.id}`,
+        `open: ${w.open}`,
+        ...(w.phone ? [`phone: ${w.phone}`] : []),
+    ].join('\n')
+}
+
 /** a view's name as the page and the CLI show it: its label, or « view n » */
 export const viewName = (v: ReviewView | undefined, i: number): string => v?.label || `view ${i + 1}`
 

package/src/review/paste.ts

@@ -0,0 +1,30 @@
+// where a card lives, for the paste that hands it to another agent session (model.ts pasteBlock): the repo's path as
+// `~/dev/<repo>`, the house page's url, the phone's url through the repo's public host. Node side: the page reads it
+// from /api/review/spec
+import { homedir } from 'node:os'
+import { homePort } from '../home/home-paths.ts'
+import { loadRepoConfig } from '../repo-config.ts'
+import { defaultHubPort } from '../serve/hub-port.ts'
+
+export interface PasteHosts {
+    /** the repo's path, `~` for the home folder */
+    root: string
+    /** the house page of the repo, `#page:review:<id>` goes after it */
+    open: string
+    /** the repo's hub through its public host, null when it has none */
+    phone: string | null
+}
+
+export async function pasteHosts(root: string): Promise<PasteHosts> {
+    const c = await loadRepoConfig(root)
+    const home = homedir()
+    const host = c.serve?.publicHost
+    return {
+        root: root.startsWith(`${home}/`) ? `~${root.slice(home.length)}` : root,
+        open: `http://localhost:${homePort()}/r/${encodeURIComponent(c.name)}/`,
+        phone: host ? `http://${host}:${c.serve?.port ?? defaultHubPort(c.name)}/` : null,
+    }
+}
+
+/** a card's url under a hub base */
+export const cardUrl = (base: string, id: string): string => `${base}#page:review:${id}`

package/src/review/routes.ts

@@ -3,11 +3,12 @@
 // relative to the repo hub's base. null: not a path of these
 import { existsSync, mkdirSync, mkdtempSync, renameSync, rmSync, statSync, writeFileSync } from 'node:fs'
 import { tmpdir } from 'node:os'
-import { extname, join } from 'node:path'
+import { dirname, extname, join } from 'node:path'
 import { isAnimFormat, isAnimLabels, isAnimMode, pairAnimation } from './anim.ts'
 import { startAuto } from './auto.ts'
 import { bad, KEEP, NO_STORE, ranged, TYPES } from './http.ts'
 import type { Rect, Subject } from './model.ts'
+import { pasteHosts } from './paste.ts'
 import { openReview, partFile, viewOfItem } from './project.ts'
 import { roleOf } from './role.ts'
 import { addMessage, imageFile, markRead, raiseItem, rateSubject, readAllCached, readItem, specOf, storeDir, storeStamp, threadImage, writeVerdict } from './store.ts'
@@ -110,8 +111,14 @@ async function reviewRoute(req: Request, path: string, dir: string, unit: Review
     }
     if (path === '/api/review/watchers' && req.method === 'GET') return Response.json(liveWatchers(dir), { headers: NO_STORE })
     // the unit's words, the dispatcher's holder: what the page draws the list's filters, labels and board with
-    if (path === '/api/review/spec' && req.method === 'GET')
-        return Response.json({ ...specData(specOf(dir)), role: roleOf(dir), sources: (unit?.sources ?? []).map((s) => s.id), auto: (unit?.auto ?? []).map((r) => ({ id: r.id, every: r.every })) }, { headers: NO_STORE })
+    if (path === '/api/review/spec' && req.method === 'GET') {
+        // where a card lives, for the paste that hands it to another agent session; a config that does not load leaves it out
+        const where = await pasteHosts(dirname(dirname(dir))).catch(() => null)
+        return Response.json(
+            { ...specData(specOf(dir)), role: roleOf(dir), sources: (unit?.sources ?? []).map((s) => s.id), auto: (unit?.auto ?? []).map((r) => ({ id: r.id, every: r.every })), where },
+            { headers: NO_STORE },
+        )
+    }
     if ((path === '/api/review/raise' || path === '/api/review/rate') && req.method === 'POST') {
         let img: ReturnType<typeof dataUrlFile> = null
         try {

package/src/scaffold/files/agent-index.md

@@ -18,8 +18,8 @@ Read the matching page before touching a surface:
 
 | read when touching… | page |
 |---|---|
-| any source file | `agent/coding.md` |
-| recent changes and why | `agent/changelog.md` |{{CLI_ROW}}
+| any source file | → agent/coding.md |
+| recent changes and why | → agent/changelog.md |{{CLI_ROW}}
 
 Grow this table: a subsystem, a service, a runbook each earn their own page,
 cross-linked from here.

package/src/scaffold/project-scaffold.ts

@@ -236,7 +236,7 @@ export function installedLib(root: string): { version?: string; exports?: Record
 /** the agent index lines that load the essentials page and list the full one */
 const cliIndexVars = (name: string, cli: boolean): { CLI_INCLUDE: string; CLI_ROW: string } => ({
     CLI_INCLUDE: cli ? `\n@${name}-essentials.md` : '',
-    CLI_ROW: cli ? `\n| every command of the project CLI | \`${cliPages(name).all}\` |` : '',
+    CLI_ROW: cli ? `\n| every command of the project CLI | → ${cliPages(name).all} |` : '',
 })
 
 /** an ADOPTED repo asking for the project CLI (`sk start <name> --cli`): the
@@ -257,7 +257,7 @@ export function adoptCli(root: string, opts: { name: string; print?: (line: stri
     addScripts(root, written, p, opts.name, false)
     const index = join(root, 'agent/index.md')
     const include = `@${opts.name}-essentials.md`
-    if (existsSync(index) && !readFileSync(index, 'utf8').split('\n').includes(include)) p(`next: in agent/index.md, load the everyday commands with the line ${include} and list \`${cliPages(opts.name).all}\` in its table`)
+    if (existsSync(index) && !readFileSync(index, 'utf8').split('\n').includes(include)) p(`next: in agent/index.md, load the everyday commands with the line ${include} and list → ${cliPages(opts.name).all} in its table`)
     const deps = { ...(pkg as { dependencies?: object }).dependencies, ...(pkg as { devDependencies?: object }).devDependencies }
     p(`ok: project CLI written: ${written.join(', ')} — run \`bun run cli\``)
     if (!('rvlib-shipkit' in deps)) p('next: bun add -d rvlib-shipkit   (the CLI imports rvlib-shipkit/cli-kit)')

package/src/serve/hub-app/activity-st.ts

@@ -10,6 +10,11 @@ import { clampCap, DEFAULT_CAP, DEFAULT_MINUTES, idleRow, WINDOW_MINUTES, type A
 export { ageShort, CAP_MAX, CAP_MIN, clampCap, DEFAULT_CAP, dirtWords, filesWords, upstreamWords, windowWords } from '../../home/activity-shape.ts'
 import type { HomeSt } from './home-st.ts'
 import { ownTrouble, reach } from './reach.ts'
+import type { PushResult } from '../../fleet/push-fleet.ts'
+
+/** the repos holding commits no remote has: ahead of their upstream, or on a branch with none */
+export const unpushedCount = (rows: Pick<ActivityRow, 'ahead' | 'upstream' | 'branch'>[]): number =>
+    rows.filter((r) => (r.ahead ?? 0) > 0 || (r.upstream == null && r.branch != null)).length
 
 export const ACTIVITY_POLL_MS = 60_000
 const MINUTES_KEY = 'home:activity:minutes'
@@ -38,6 +43,10 @@ function keep(key: string, n: number): void {
 export class ActivitySt {
     data: ActivityAnswer | null = null
     error = ''
+    /** the push button is running */
+    pushing = false
+    /** what the last push said, one line a repo that needed it */
+    pushed: PushResult[] | null = null
     minutes: number = keptMinutes()
     /** the most commits a repo shows before a click shows the rest */
     cap: number = clampCap(kept(CAP_KEY) ?? DEFAULT_CAP)
@@ -118,6 +127,28 @@ export class ActivitySt {
         this.st.select(name)
     }
 
+    /** repos holding commits no remote has */
+    get unpushed(): number {
+        return unpushedCount(this.data?.rows ?? [])
+    }
+
+    /** push every house repo that needs it, through the daemon (fleet/push-fleet.ts), then read git again */
+    async pushAll(): Promise<void> {
+        if (this.pushing) return
+        runInAction(() => { this.pushing = true; this.pushed = null })
+        try {
+            const r = await reach('/api/home/push-all', { method: 'POST', signal: AbortSignal.timeout(180_000) })
+            if (!r.ok) throw new Error(`${r.status} ${await r.text()}`)
+            const res = (await r.json()) as PushResult[]
+            runInAction(() => { this.pushed = res.filter((x) => x.action !== 'none'); this.error = '' })
+        } catch (e) {
+            runInAction(() => { this.error = ownTrouble(e, this.st.machine, (m) => `push every repo: ${m}`) })
+        } finally {
+            runInAction(() => { this.pushing = false })
+            void this.poll(true)
+        }
+    }
+
     /** the refresh button: a new git pass now, whatever the countdown says */
     refresh(): void {
         if (!this.loading) void this.poll(true)

package/src/serve/hub-app/activity-view.tsx

@@ -189,6 +189,12 @@ export const ActivityView = observer(({ app }: { app: HomeSt }) => {
                     <span className="acdim">commits</span>
                 </label>
                 <Next app={app} />
+                {st.unpushed > 0 || st.pushing ? (
+                    <button type="button" className="acpill acpush" disabled={st.pushing} onClick={() => void st.pushAll()}
+                        title="git push in every house repo with unpushed commits; never a force, a repo that fell behind waits for a pull (sk fleet push does the same)">
+                        {st.pushing ? 'pushing…' : `↑ push ${st.unpushed} unpushed`}
+                    </button>
+                ) : null}
                 {d == null ? <span className="acdim">reading every repo…</span> : (
                     <>
                         <span className="acpill">{busy.length} busy</span>
@@ -198,6 +204,9 @@ export const ActivityView = observer(({ app }: { app: HomeSt }) => {
                 )}
             </div>
             {st.error !== '' ? <div className="hnote">{st.error}</div> : null}
+            {st.pushed == null || st.pushed.length === 0 ? null : (
+                <div className="acpushed">{st.pushed.map((p) => <div key={p.name} className={p.ok ? 'ok' : p.action === 'skip' ? 'skip' : 'fail'}>{p.line}</div>)}</div>
+            )}
             {d != null && busy.length === 0 ? <div className="acdim">nothing committed or uncommitted in the last {windowWords(st.minutes)}</div> : null}
             {busy.length + idle.length === 0 ? null : (
                 <table className={`actable${st.stale ? ' stale' : ''}`}>

package/src/serve/hub-app/activity.css

@@ -72,3 +72,9 @@ button.acpill:disabled{cursor:default}
 .acfork .hico{width:13px;height:13px}
 .acgrp:hover .acfork{opacity:1}
 .acfork:hover{color:var(--accent)}
+.acpush{color:var(--ink);border-color:var(--accent)}
+/* what the last push said: a line a repo, pushed in the ok colour, skipped dim, failed red */
+.acpushed{margin:0 0 var(--s3);font:12px/1.5 ui-monospace,Menlo,monospace}
+.acpushed .ok{color:var(--ok)}
+.acpushed .skip{color:var(--dim)}
+.acpushed .fail{color:var(--red)}

package/src/serve/hub-app/home-strip.tsx

@@ -256,7 +256,7 @@ const downCard = (m: MachineSt): Card[] => {
 
 /** what is wrong with the machine itself, one card each: the machine not
  *  answering, a crashed child of pm, a crashed global process (its switch
- *  starts it again), a failed poll. While pm is silent its last answer says
+ *  starts it again), a failed poll, the last good code running. While pm is silent its last answer says
  *  nothing here */
 const machineCards = (m: MachineSt): Card[] => {
     const pm = m.pmDown !== '' ? null : m.pm
@@ -265,6 +265,8 @@ const machineCards = (m: MachineSt): Card[] => {
         ...(pm?.children ?? []).filter((c) => c.status === 'crashed' || c.status === 'down').map((c): Card => ({ key: `child:${c.id}`, tone: 'red', head: `${c.id} ${STATUS_WORD[c.status]}`, body: c.note ?? c.tail.at(-1) ?? '' })),
         ...(pm?.resources ?? []).filter((r) => r.status === 'crashed').map((r): Card => ({ key: `res:${r.id}`, tone: 'red', head: 'crashed', body: <ResourceChip m={m} r={r} /> })),
         ...(m.error !== '' ? [{ key: 'poll', tone: 'red' as const, body: m.error }] : []),
+        // the tree fails to load and the machine runs its last good code (core/last-good.ts)
+        ...(m.data?.held != null ? [{ key: 'held', tone: 'warn' as const, head: 'last good code', body: m.data.held }] : []),
     ]
 }
 

package/src/serve/hub-app/machine-st.ts

@@ -110,7 +110,7 @@ export class MachineSt {
      *  `shipkit global-processes` prints) */
     get processGroups(): ProcessGroup[] {
         const silent = { ...(this.down === '' ? {} : { [this.downWho]: this.down }), ...(this.pmDown === '' ? {} : { pm: this.pmDown }) }
-        return processGroups(this.pm, this.st.projects.list.map((p) => p.name), { daemonVersion: this.data?.version ?? null, silent })
+        return processGroups(this.pm, this.st.projects.list.map((p) => p.name), { daemonVersion: this.data?.version ?? null, daemonHeld: this.data?.held ?? null, silent })
     }
 
     resource(id: string): ResourceView | null {

package/src/serve/hub-app/review/review-answer.tsx

@@ -2,7 +2,7 @@
 // goes to the agent, the verdicts (the phone's in a bar under the thumb), then the thread's conversation and his reply
 // box. When the page reads in one column the header sits over the views instead (review-views.tsx)
 import { observer } from 'mobx-react-lite'
-import { useState } from 'react'
+import { Icon, ICON } from '../icons.tsx'
 import { laneOf, placeText, STATUS_LABEL, stateOf, verdictOf, viewName, withdrawnOf, type ReviewItem, type ReviewStatus } from '../../../review/model.ts'
 import { renderMd } from '../md.ts'
 import { AutoTag, blurOnEscape, Empty, LaneTag, Listeners, REACTIONS, Rework, secs, short, SIDE_LABEL, ThumbOf, toneOf, VERDICTS } from './kit.tsx'
@@ -15,12 +15,9 @@ export const ReviewForm = observer(({ rv }: { rv: ReviewSt }) => {
     const r = x.item
     const draft = rv.draft
     const sent = verdictOf(x)
-    // in one column the header and the verdicts sit over the views
-    const headAbove = rv.headInViews
-    const verdictsAbove = rv.stacked && !rv.phone.on
     return (
         <div className="rv-form">
-            {headAbove ? null : <CardHead rv={rv} x={x} verdicts={false} />}
+            {rv.headInViews ? null : <CardHead rv={rv} x={x} />}
             <KindView app={rv} x={x} />
             <AnimPanel app={rv} x={x} />
             {rv.newer.map((y) => <Linked key={y.item.id} rv={rv} label="replaced by" id={y.item.id} item={y} />)}
@@ -30,14 +27,6 @@ export const ReviewForm = observer(({ rv }: { rv: ReviewSt }) => {
             {rv.history.length > 0 ? <div className="rv-divider">earlier versions</div> : null}
             {rv.history.map((y, i) => <Linked key={y.item.id} rv={rv} label={`v${rv.history.length - i}`} id={y.item.id} item={y} />)}
             <Outgoing rv={rv} />
-            <label className="rv-label">Comment</label>
-            <textarea className="rv-comment" rows={3} placeholder="the overall word" value={draft.comment} onKeyDown={blurOnEscape} onChange={(e) => rv.setComment(e.currentTarget.value)} />
-            {verdictsAbove ? null : <div className="rv-listenrow"><span className="rv-dim">who reads your answer</span><Listeners rv={rv} cards={[x]} /></div>}
-            {!rv.phone.on && !verdictsAbove && !r.wip && r.origin.by === 'agent' ? (
-                <div className="rv-verdicts">
-                    {VERDICTS.map((v) => <VerdictButton key={v.status} v={v} sent={sent?.status} busy={rv.busy} send={() => void rv.submit(v.status)} />)}
-                </div>
-            ) : null}
             <Messages rv={rv} />
             <div className="rv-formfoot">
                 <button className="ghost" onClick={() => rv.nextPending()}>next waiting (N)</button>
@@ -77,10 +66,12 @@ function stateWords(x: ReviewItem): string {
 }
 
 /** the card's header, for every card: the title, one tight line (the state as the list's pill, the lane, when, the
- *  commit, the id to copy), then what the agent says. No subject row: the list's row names it, a kind's facts say the
- *  rest; no id row: the url carries it. With `verdicts`, the four verdicts under it (the page in one column) */
-export const CardHead = observer(({ rv, x, verdicts }: { rv: ReviewSt; x: ReviewItem; verdicts: boolean }) => {
+ *  commit, the button that copies the card for an agent), then what the agent says. No subject row: the list's row names it, a kind's facts say the
+ *  rest; no id row: the url carries it. Under it the four verdicts (on the phone they ride the bar under the thumb)
+ *  and his overall comment, on every card and in every layout */
+export const CardHead = observer(({ rv, x }: { rv: ReviewSt; x: ReviewItem }) => {
     const r = x.item
+    const draft = rv.draft
     const status = rv.statusOf(x)
     const sent = verdictOf(x)
     const lane = laneOf(x)
@@ -93,41 +84,29 @@ export const CardHead = observer(({ rv, x, verdicts }: { rv: ReviewSt; x: Review
                 {r.origin.by === 'auto' ? <AutoTag rule={r.origin.rule} /> : null}
                 <span>{stateWords(x)}</span>
                 {r.commit ? <span className="rv-mono" title={`commit ${r.commit}`}>@{r.commit.slice(0, 9)}</span> : null}
-                <CopyId id={r.id} />
+                <button className="rv-copyid ghost" title={`copy this card for an agent (C): its title, its repo, sk review show ${r.id}, its link`} aria-label="copy this card for an agent" onClick={() => void rv.copyCard()}>
+                    <Icon d={ICON.copy} />
+                </button>
             </div>
             <Rework rv={rv} x={x} />
             {r.views.length > 0 && r.what ? <div className="md rv-what" dangerouslySetInnerHTML={{ __html: renderMd(r.what) }} /> : null}
             {r.keeps ? <div className="rv-keeps"><b>keeps:</b> {r.keeps}</div> : null}
             {r.why ? <div className="rv-pre rv-why">another subject: {r.why}</div> : null}
-            {verdicts && !r.wip && r.origin.by === 'agent' ? (
+            {!rv.phone.on && !r.wip && r.origin.by === 'agent' ? (
                 <div className="rv-headverdicts">
-                    {VERDICTS.map((v) => <VerdictButton key={v.status} v={v} sent={sent?.status} busy={rv.busy} send={() => void rv.submit(v.status)} short />)}
+                    {VERDICTS.map((v) => <VerdictButton key={v.status} v={v} sent={sent?.status} busy={rv.busy} send={() => void rv.submit(v.status)} />)}
                     <Listeners rv={rv} cards={[x]} />
                 </div>
             ) : null}
+            <textarea className="rv-comment" rows={2} placeholder="your overall word (a verdict sends it)" value={draft.comment} onKeyDown={blurOnEscape} onChange={(e) => rv.setComment(e.currentTarget.value)} />
         </div>
     )
 })
 
-/** the id, copied on a click: an agent names a card by it */
-const CopyId = ({ id }: { id: string }) => {
-    const [done, setDone] = useState(false)
-    return (
-        <button className="rv-copyid ghost" title={`${id}: click to copy`} onClick={() => {
-            void navigator.clipboard?.writeText(id).then(() => {
-                setDone(true)
-                setTimeout(() => setDone(false), 1200)
-            })
-        }}>
-            {done ? '✓ copied' : '⧉ id'}
-        </button>
-    )
-}
-
 /** one verdict: plain until one is sent, then the chosen one filled and the others dimmed */
-const VerdictButton = ({ v, sent, busy, send, short: brief }: { v: (typeof VERDICTS)[number]; sent: ReviewStatus | undefined; busy: boolean; send: () => void; short?: boolean }) => (
-    <button className={`rv-verdict ${toneOf(v.status)}${sent === v.status ? ' chosen' : ''}${sent && sent !== v.status ? ' off' : ''}`} disabled={busy} onClick={send} title={brief ? `${v.label} (${v.key})` : undefined}>
-        <span className="rv-vglyph">{sent === v.status ? '✓' : v.glyph}</span> {brief ? v.short : v.label} ({v.key}){sent === v.status && !brief ? ' · chosen' : ''}
+const VerdictButton = ({ v, sent, busy, send }: { v: (typeof VERDICTS)[number]; sent: ReviewStatus | undefined; busy: boolean; send: () => void }) => (
+    <button className={`rv-verdict ${toneOf(v.status)}${sent === v.status ? ' chosen' : ''}${sent && sent !== v.status ? ' off' : ''}`} disabled={busy} onClick={send} title={`${v.label} (${v.key})`}>
+        <span className="rv-vglyph">{sent === v.status ? '✓' : v.glyph}</span> {v.short} ({v.key})
     </button>
 )
 

package/src/serve/hub-app/review/review-page.tsx

@@ -8,7 +8,7 @@ import { Group, Panel, Separator } from 'react-resizable-panels'
 import type { RepoSt } from '../repo-st.ts'
 import { ReviewForm, VerdictBar } from './review-answer.tsx'
 import { ReviewList } from './review-list.tsx'
-import type { PhonePane } from './review-st.ts'
+import type { PhonePane, ReviewSt } from './review-st.ts'
 import { ReviewImage } from './review-views.tsx'
 
 const PANE_LABEL: Record<PhonePane, string> = { list: 'cards', card: 'card', answer: 'answer' }
@@ -51,9 +51,12 @@ export const ReviewPage = observer(({ app }: { app: RepoSt }) => {
                     {rv.phone.pane === 'list' ? <ReviewList rv={rv} /> : rv.phone.pane === 'card' ? <ReviewImage rv={rv} /> : <ReviewForm rv={rv} />}
                 </div>
                 {rv.phone.pane === 'list' ? null : <VerdictBar rv={rv} />}
+                <Toast rv={rv} />
             </div>
         )
     return (
+        <>
+        <Toast rv={rv} />
         <Group className="rv-cols" orientation="horizontal" elementRef={el}>
             <Panel id="review-list" defaultSize={rv.stacked ? '32%' : '20%'} minSize={160} className="rv-pane" style={PANEL}>
                 <ReviewList rv={rv} />
@@ -79,6 +82,7 @@ export const ReviewPage = observer(({ app }: { app: RepoSt }) => {
                 </>
             )}
         </Group>
+        </>
     )
 })
 
@@ -92,5 +96,8 @@ export const PhoneTop = observer(({ app }: { app: RepoSt }) => (
     </div>
 ))
 
+/** what the last copy put on the clipboard, a moment over the page */
+const Toast = observer(({ rv }: { rv: ReviewSt }) => (rv.toast ? <div className="rv-toast" role="status">{rv.toast}</div> : null))
+
 /** a panel never scrolls itself: the column inside it does (hub-app.tsx, PANEL) */
 export const PANEL = { overflow: 'hidden' } as const

package/src/serve/hub-app/review/review-st.ts

@@ -10,6 +10,7 @@ import {
     isPair,
     laneOf,
     moveOf,
+    pasteBlock,
     reworkOf,
     STARS,
     STATUS_LABEL,
@@ -31,6 +32,7 @@ import { answererOf, chainOf, childrenOf, conversationOf, followedIds, isActive,
 import { reach } from '../reach.ts'
 import type { RepoSt } from '../repo-st.ts'
 import type { Role } from '../../../review/role.ts'
+import type { PasteHosts } from '../../../review/paste.ts'
 import { ClipSt } from './clip-st.ts'
 
 interface Draft {
@@ -66,7 +68,7 @@ export const drawnSide = (mode: CompareMode, drawn: Side): Side => (mode === 'mo
 
 /** below this width the page is the phone's: one column at a time, the verdicts in a bar under the thumb */
 /** a repo's review unit as the page reads it: the words of its kinds and topics, its dispatcher and who holds the role */
-export type RepoSpec = Spec & { role: Role | null; sources: string[]; auto: { id: string; every: string }[] }
+export type RepoSpec = Spec & { role: Role | null; sources: string[]; auto: { id: string; every: string }[]; where?: PasteHosts | null }
 
 export const PHONE_PX = 768
 /** below this width of the review page itself (the project list and the sidebar already took theirs), `auto` stacks
@@ -144,6 +146,26 @@ async function postJson<T>(url: string, body: unknown): Promise<T> {
     return JSON.parse(text) as T
 }
 
+/** a text to the clipboard: the clipboard api, else (a page over plain http, the phone through a public host) the
+ *  old copy command on a hidden field */
+async function copyText(text: string): Promise<boolean> {
+    try {
+        await navigator.clipboard.writeText(text)
+        return true
+    } catch {
+        const el = document.createElement('textarea')
+        el.value = text
+        el.setAttribute('readonly', '')
+        el.style.position = 'fixed'
+        el.style.opacity = '0'
+        document.body.appendChild(el)
+        el.select()
+        const ok = document.execCommand('copy')
+        el.remove()
+        return ok
+    }
+}
+
 /** the pair's compare mode, the curtain's line and the onion's opacity, shared by every pair of the page */
 export class CompareSt {
     mode: CompareMode = 'side'
@@ -234,6 +256,9 @@ export class ReviewSt {
     unfolded: Record<string, boolean> = {}
     busy = false
     error: string | null = null
+    /** what the last copy put on the clipboard, in words, shown a moment */
+    toast: string | null = null
+    private toastTimer: ReturnType<typeof setTimeout> | null = null
     /** the card against its answer, his pick, kept with the prefs */
     place: CardPlace = 'auto'
     /** the page is narrower than STACK_PX */
@@ -294,7 +319,7 @@ export class ReviewSt {
         } catch {
             // a broken pref is the default
         }
-        makeAutoObservable<this, 'st' | 'clips' | 'clipsCard' | 'reading' | 'timer' | 'loadList' | 'loadWatchers' | 'prefix'>(this, {
+        makeAutoObservable<this, 'st' | 'clips' | 'clipsCard' | 'reading' | 'timer' | 'loadList' | 'loadWatchers' | 'prefix' | 'toastTimer'>(this, {
             st: false,
             clips: false,
             clipsCard: false,
@@ -303,6 +328,7 @@ export class ReviewSt {
             loadList: false,
             loadWatchers: false,
             prefix: false,
+            toastTimer: false,
             compare: false,
             phone: false,
         })
@@ -356,6 +382,27 @@ export class ReviewSt {
         }
     }
 
+    /** the open card as one paste a fresh agent session in any repo acts on: what it is, its repo, the command that shows
+     *  it whole, its link here and on the phone (C) */
+    async copyCard(): Promise<void> {
+        const x = this.open
+        if (!x) return
+        const id = x.item.id
+        const w = this.spec?.where
+        const here = typeof location === 'undefined' ? (w?.open ?? '') : `${location.origin}${location.pathname}${location.search}`
+        const text = pasteBlock(x.item, STATUS_LABEL[this.statusOf(x)], {
+            root: w?.root ?? '<the repo>',
+            open: `${here}#page:review:${id}`,
+            phone: w?.phone ? `${w.phone}#page:review:${id}` : null,
+        })
+        const ok = await copyText(text)
+        runInAction(() => {
+            this.toast = ok ? `copied for an agent: title, repo, sk review show, ${w?.phone ? 'links' : 'link'}` : 'the browser refused the copy'
+            if (this.toastTimer !== null) clearTimeout(this.toastTimer)
+            this.toastTimer = setTimeout(() => runInAction(() => (this.toast = null)), 2600)
+        })
+    }
+
     setBoard(on: boolean): void {
         this.board = on
     }
@@ -871,7 +918,7 @@ export class ReviewSt {
     }
 
     /** the page's keys, as the atelier takes its own: true when it used the key. V approves, F ok with a fix, R rejects,
-     *  A archives, N the next waiting card, M the rings, Delete the picked rectangle; on a clip Space plays, ← → a frame
+     *  A archives, N the next waiting card, C copies the card for an agent, M the rings, Delete the picked rectangle; on a clip Space plays, ← → a frame
      *  (a sound: two seconds); on option rows 1 to 9 play that row, ⇧1 to ⇧5 score the row playing, P picks it. A digit is
      *  read from its physical key (`code`), so ⇧ and an AZERTY keyboard read the same */
     key(e: Pick<KeyboardEvent, 'key' | 'shiftKey'> & { code?: string }): boolean {
@@ -894,6 +941,7 @@ export class ReviewSt {
         else if (k === 'r') void this.submit('rejected')
         else if (k === 'a') void this.submit('archived')
         else if (k === 'n') this.nextPending()
+        else if (k === 'c') void this.copyCard()
         else if (k === 'm') this.toggleMarks()
         else if ((e.key === 'Delete' || e.key === 'Backspace') && this.rect !== null) this.removeRect(this.rect)
         else return false

package/src/serve/hub-app/review/review-views.tsx

@@ -22,7 +22,7 @@ export const ReviewImage = observer(({ rv }: { rv: ReviewSt }) => {
     const rows = rv.rows
     return (
         <div className="rv-views">
-            {rv.headInViews ? <CardHead rv={rv} x={x} verdicts={rv.stacked && !rv.phone.on} /> : null}
+            {rv.headInViews ? <CardHead rv={rv} x={x} /> : null}
             <div className="rv-viewsbar">
                 {views.length > 1 && !rows ? <JumpList rv={rv} /> : <span />}
                 <span className="rv-viewsctl">

package/src/serve/hub-app/review.css

@@ -203,14 +203,16 @@
 .rv-head{display:flex;flex-direction:column;gap:var(--s1)}
 .rv-meta{display:flex;flex-wrap:wrap;gap:var(--s1) var(--s2);align-items:center;font-size:12px;color:var(--dim);line-height:18px}
 .rv-meta .rv-badge{font-size:10px;line-height:18px;max-width:none}
-.rv-copyid{font-size:11.5px;padding:0 var(--s1);line-height:18px;color:var(--dim)}
+.rv-copyid{display:inline-flex;align-items:center;padding:var(--s0);color:var(--dim)}
 .rv-copyid:hover{color:var(--ink)}
+.rv-toast{position:fixed;left:50%;bottom:var(--s4);transform:translateX(-50%);z-index:40;max-width:calc(100% - 2 * var(--s4));padding:var(--s1) var(--s3);border-radius:6px;border:1px solid var(--line);background:var(--panel);color:var(--ink);font-size:12.5px;box-shadow:0 6px 20px var(--shadow)}
 .rv-what{font-size:13.5px;line-height:1.45;color:var(--ink)}
 .rv-what ul{margin:var(--s0) 0;padding-left:var(--s4)}
 .rv-what p{margin:var(--s0) 0}
 .rv-why{color:var(--dim);font-size:12px}
 .rv-headverdicts{display:flex;flex-wrap:wrap;gap:var(--s1);align-items:center;margin-top:var(--s1)}
 .rv-headverdicts .rv-verdict{padding:0 var(--s2);line-height:24px;font-size:12.5px}
+.rv-head .rv-comment{margin-top:var(--s1);width:100%}
 .rv-keeps{color:var(--ok);font-size:13px}
 .rv-facts{display:flex;flex-direction:column}
 .rv-fact{display:flex;gap:var(--s2);align-items:center;min-height:22px;font-size:12.5px}
@@ -227,9 +229,6 @@
 .rv-outbadge.rect{--tone:var(--warn)}
 .rv-outbadge.rect.on{--tone:var(--blue)}
 .rv-x{flex:none;padding:0 var(--s1);border:0}
-.rv-label{font-size:12px;font-weight:600;margin-top:var(--s1)}
-.rv-listenrow{display:flex;gap:var(--s2);align-items:center;font-size:11px}
-.rv-verdicts{display:flex;flex-direction:column;gap:var(--s1)}
 .rv-verdict{text-align:left;padding:var(--s1) var(--s3);font-size:13px;border-color:color-mix(in srgb,var(--tone) 60%,var(--line))}
 .rv-verdict .rv-vglyph{color:var(--tone);font-weight:800}
 .rv-verdict.chosen{background:var(--tone);color:var(--bg);border-color:var(--tone)}
@@ -252,6 +251,7 @@
 .rv-verdictbar .rv-verdict{flex:1;text-align:center;font-size:14px;padding:var(--s2) var(--s1);color:var(--tone)}
 .rv-verdictbar .rv-verdict.chosen{color:var(--bg)}
 .rv-phone .rv-views{gap:var(--s3)}
+.rv-phone .rv-toast{top:var(--s4);bottom:auto}
 /* a row on a phone: ▶, the label and the time on one line, the strip and the acts under them */
 .rv-phone .rv-optrow{flex-wrap:wrap;row-gap:var(--s1)}
 .rv-phone .rv-optlabel{flex:1;max-width:none}