@superjackfruitlabs/salaam 0.0.4 → 0.0.5

package/dist/src/service.js

@@ -1,3 +1,6 @@
+import { readFileSync } from 'node:fs';
+import { DangerClassifier, loadDangerClassifierOptions } from './danger-classifier.js';
+import { WebSearch, loadWebSearchConfig } from './web-search.js';
 import { createServer } from 'node:http';
 import { loadPersona } from './persona.js';
 import { mkdirSync, chmodSync, chownSync, existsSync, unlinkSync, realpathSync, statSync, } from 'node:fs';
@@ -5,6 +8,7 @@ import { join, resolve } from 'node:path';
 import { providerRoute } from './provider-config.js';
 import { TaskContext } from './task-context.js';
 import { ContextAdmission } from './context-admission.js';
+import { ProjectInstructions, userInstructionsPath } from './project-instructions.js';
 import { SLASH_COMMANDS } from './tui-commands.js';
 import { TaskWork } from './task-work.js';
 import { TaskBudget } from './task-budget.js';
@@ -20,6 +24,7 @@ import { OperatorInbox } from './operator-inbox.js';
 import { RecurringWork } from './recurring-work.js';
 import { JobWork } from './job-work.js';
 import { Store } from './store.js';
+import { pickerSummary } from './session-pick.js';
 import { acquireStateLock } from './state-lock.js';
 import { Artifacts } from './artifacts.js';
 import { Supervisor } from './supervisor.js';
@@ -53,11 +58,15 @@ import { McpAdmission } from './mcp-admission.js';
 import { McpWork } from './mcp-work.js';
 import { ToolkitWork } from './toolkit-work.js';
 import { permissionPolicy, configurePermissions } from './autonomy.js';
-import { chooseLocalSandbox } from './command-sandbox.js';
+import { expandPrompt, storeImages } from './mentions.js';
+import { ApprovalRules } from './approval-rules.js';
+import { chooseLocalSandbox, resolveReadPaths } from './command-sandbox.js';
 import { LocalCoding } from './local-coding.js';
+import { Hooks } from './hooks.js';
 import { localDiff } from './local-diff.js';
 import { reviewUndo, applyUndo, reconcileUndo } from './local-undo.js';
 import { forkPiSession } from './pi-sessions.js';
+import { TurnCheckpoints } from './turn-rewind.js';
 import { StationWork } from './station-work.js';
 import { readToolkitConfig } from './toolkit-config.js';
 import { probeToolsUser, serviceToolIdentity } from './tool-identity.js';
@@ -115,7 +124,25 @@ export class SalaamService {
     hostReceipt;
     operatorAccess;
     agentpodWorkProgression;
+    /** Phase 2b: whether interactive sessions here may choose full access (no OS sandbox). */
+    allowFullAccessSessions;
+    allowYoloSessions;
+    reviewers = new Map();
+    reviewerDefault = 'off';
+    reviewerPin;
+    /** The reviewer a session uses: the host's pin, else its own choice, else the config default. */
+    reviewerChoice(session) {
+        if (this.reviewerPin)
+            return this.reviewerPin;
+        const row = this.store.db
+            .prepare('SELECT choice FROM danger_classifier_choice WHERE session_id=?')
+            .get(session);
+        const choice = row?.choice ?? this.reviewerDefault;
+        return choice === 'off' || this.reviewers.has(choice) ? choice : 'off';
+    }
     constructor(options) {
+        this.allowFullAccessSessions = options.allowFullAccessSessions !== false;
+        this.allowYoloSessions = options.allowYoloSessions !== false;
         this.agentpodWorkProgression = options.agentpodWorkProgression === true;
         if (this.agentpodWorkProgression && (!options.stationConfig || !options.agentpodConfig))
             throw Error('Strict work requires reviewed AgentPod protected station configuration');
@@ -157,7 +184,8 @@ export class SalaamService {
         this.socket = join(this.stateDir, 'operator.sock');
         this.bridgeSocket = join(this.stateDir, 'bridge.sock');
         if (Buffer.byteLength(this.bridgeSocket) > 100)
-            throw new Error('State path is too long for a Unix socket; choose a shorter --state path');
+            throw new Error(`State path is too long for a Unix socket (${Buffer.byteLength(this.bridgeSocket)} bytes; the limit is 100): ` +
+                'use the default (salaam code, no --state), or a shorter --state or SALAAM_STATE_DIR such as /tmp/salaam-state.');
         this.release = acquireStateLock(this.stateDir);
         try {
             this.store = new Store(join(this.stateDir, 'journal.sqlite'));
@@ -170,6 +198,10 @@ export class SalaamService {
                 this.artifacts = new Artifacts(join(this.stateDir, 'artifacts'), this.store);
                 this.tasks = new TaskContext(this.store, this.artifacts);
                 this.tasks.admitted = new ContextAdmission(this.store);
+                this.tasks.instructions = new ProjectInstructions(this.store, this.artifacts, options.userInstructions === undefined
+                    ? userInstructionsPath()
+                    : options.userInstructions);
+                this.tasks.checkpoints = new TurnCheckpoints(this.store, this.stateDir, this.toolIdentity.mode !== 'tools-user');
                 this.taskWork = new TaskWork(this.store, this.tasks);
                 this.taskBudget = new TaskBudget(this.store, this.taskWork);
                 if (options.admitModelRequest) {
@@ -252,10 +284,43 @@ export class SalaamService {
                 const localSandbox = {
                     ...chooseLocalSandbox(options.localSandbox ?? 'workspace-write'),
                     allowUnsandboxed: options.unsandboxedCommands === 'allow',
+                    ...(options.localSandboxReadPaths?.length
+                        ? { readPaths: resolveReadPaths(options.localSandboxReadPaths) }
+                        : {}),
                     ...(options.commandNetwork ? { network: { socket: options.commandNetwork.socket } } : {}),
                 };
                 this.localCoding = new LocalCoding(this.store, this.managed, (id, epoch, attempt) => this.supervisor.assertActive(id, epoch, attempt), this.toolIdentity, localSandbox);
+                this.localCoding.rules = new ApprovalRules(this.store, options.permissionsFile);
+                this.localCoding.interactive = (id) => this.supervisor.isInteractiveCoding(id);
+                this.localCoding.allowOutsideSandbox = this.allowFullAccessSessions;
                 this.supervisor.localCoding = this.localCoding;
+                if (options.dangerClassifierConfig) {
+                    const loaded = loadDangerClassifierOptions(JSON.parse(readFileSync(options.dangerClassifierConfig, 'utf8')));
+                    for (const o of loaded.options)
+                        this.reviewers.set(o.name, new DangerClassifier(o));
+                    this.reviewerDefault = loaded.default;
+                }
+                if (options.dangerClassifierPin !== undefined) {
+                    if (options.dangerClassifierPin !== 'off' &&
+                        !this.reviewers.has(options.dangerClassifierPin))
+                        throw Error(`The pinned command reviewer ${options.dangerClassifierPin} is not configured`);
+                    this.reviewerPin = options.dangerClassifierPin;
+                }
+                if (this.reviewers.size) {
+                    this.store.db.exec('CREATE TABLE IF NOT EXISTS danger_classifier_choice(session_id TEXT PRIMARY KEY, choice TEXT NOT NULL)');
+                    this.localCoding.classifier = {
+                        review: (s, command, cwd) => {
+                            const reviewer = this.reviewers.get(this.reviewerChoice(s));
+                            return reviewer ? reviewer.review(s, command, cwd) : Promise.resolve(null);
+                        },
+                    };
+                }
+                const local = this.localCoding;
+                this.supervisor.hooks = new Hooks(this.store, (session, command, input, timeout) => local.runHook(session, command, input, timeout), options.hooksFile);
+                const webConfig = options.webSearchConfig
+                    ? loadWebSearchConfig(options.webSearchConfig)
+                    : { providers: [] };
+                this.supervisor.web = new WebSearch(webConfig);
                 this.agents = new AgentProfiles(this.store, this.supervisor.route, async (route, session) => {
                     const prepared = await this.supervisor.prepareProviderRoute(route);
                     const task = this.taskWork.forSession(session);
@@ -1298,7 +1363,10 @@ export class SalaamService {
                 return;
             }
             if (url.pathname === '/v1/sessions' && req.method === 'GET') {
-                this.send(res, 200, this.store.sessions().map((x) => this.supervisor.inspect(x.id)));
+                this.send(res, 200, this.store.sessions().map((x) => {
+                    const session = this.supervisor.inspect(x.id);
+                    return { ...session, ...pickerSummary(session, this.store) };
+                }));
                 return;
             }
             if (url.pathname === '/v1/sessions' && req.method === 'POST') {
@@ -1389,13 +1457,52 @@ export class SalaamService {
             }
             if (parts[0] === 'v1' && parts[1] === 'sessions' && parts[2]) {
                 const id = parts[2];
+                // Desk-session approval rules (phase 2a): list, add (from a reviewed operation or written out),
+                // and remove. Adding or removing needs the session's control lease, like the permission policy.
+                if (parts[3] === 'approval-rules' && (parts.length === 4 || parts.length === 6)) {
+                    const rules = this.localCoding.rules;
+                    if (parts.length === 4 && req.method === 'GET') {
+                        this.send(res, 200, {
+                            applies: rules.applies(id),
+                            rules: rules.list(id),
+                            problems: rules.problems(id),
+                        });
+                        return;
+                    }
+                    const body = object(await readJson(req));
+                    this.store.authorizeControl(id, {
+                        client_id: string(body.client_id, 'client_id', 128),
+                        token: string(body.token, 'token', 128),
+                    });
+                    if (parts.length === 4 && req.method === 'POST') {
+                        const scope = string(body.scope, 'scope');
+                        if (scope !== 'session' && scope !== 'repo')
+                            throw new ServiceError('invalid_field', 'A rule scope is session or repo', 400);
+                        const request = string(body.request_id, 'request_id', 128);
+                        this.send(res, 200, body.operation_id !== undefined
+                            ? rules.addFromOperation(id, string(body.operation_id, 'operation_id', 128), scope, request)
+                            : rules.add(id, scope, body.rule, request));
+                        return;
+                    }
+                    if (parts.length === 6 && parts[5] === 'remove' && req.method === 'POST') {
+                        rules.remove(id, string(parts[4], 'rule', 128));
+                        this.send(res, 200, { removed: parts[4] });
+                        return;
+                    }
+                }
                 if (parts.length === 4 && parts[3] === 'permissions') {
                     if (req.method === 'GET') {
                         this.send(res, 200, {
                             policy: permissionPolicy(this.store, id),
                             configurable: this.store.localWorkspace(id),
                             coding: this.store.localCoding(id),
-                            command_auto_approval: false,
+                            // Desk coding sessions: approval rules and auto mode can approve commands (Phase 2a/2b).
+                            command_auto_approval: this.store.localCoding(id),
+                            full_access_allowed: this.allowFullAccessSessions,
+                            yolo_allowed: this.allowYoloSessions && !!this.localCoding.rules?.applies(id),
+                            ...(this.store.localCoding(id)
+                                ? { command_sandbox: this.localCoding.commandSandbox(id) }
+                                : {}),
                         });
                         return;
                     }
@@ -1404,16 +1511,128 @@ export class SalaamService {
                         this.send(res, 200, configurePermissions(this.store, id, string(body.request_id, 'request_id', 128), integer(body.expected_revision, 'expected_revision'), body.policy, {
                             client_id: string(body.client_id, 'client_id', 128),
                             token: string(body.token, 'token', 128),
+                        }, {
+                            allowFullAccess: this.allowFullAccessSessions,
+                            allowYolo: this.allowYoloSessions,
+                            deskSession: !!this.localCoding.rules?.applies(id),
                         }));
                         return;
                     }
                 }
+                if (parts.length === 4 && parts[3] === 'danger-classifier') {
+                    const view = () => {
+                        const choice = this.reviewers.size ? this.reviewerChoice(id) : 'off', chosen = this.reviewers.get(choice)?.config;
+                        return {
+                            configured: this.reviewers.size > 0,
+                            enabled: !!chosen,
+                            ...(chosen
+                                ? {
+                                    provider: chosen.provider,
+                                    model: chosen.model,
+                                    on_unavailable: chosen.on_unavailable,
+                                }
+                                : {}),
+                            choice,
+                            pinned: this.reviewerPin !== undefined,
+                            options: [...this.reviewers.values()].map(({ config: o }) => ({
+                                name: o.name,
+                                provider: o.provider,
+                                model: o.model,
+                            })),
+                        };
+                    };
+                    if (req.method === 'GET') {
+                        this.send(res, 200, view());
+                        return;
+                    }
+                    if (req.method === 'POST') {
+                        const body = object(await readJson(req));
+                        this.store.authorizeControl(id, {
+                            client_id: string(body.client_id, 'client_id', 128),
+                            token: string(body.token, 'token', 128),
+                        });
+                        // `enabled` is the earlier on/off form: on means the config's default.
+                        const choice = typeof body.enabled === 'boolean'
+                            ? body.enabled
+                                ? this.reviewerDefault === 'off'
+                                    ? [...this.reviewers.keys()][0]
+                                    : this.reviewerDefault
+                                : 'off'
+                            : string(body.choice, 'choice', 64);
+                        if (!this.reviewers.size && choice !== 'off')
+                            throw new ServiceError('classifier_unavailable', 'No command reviewer is configured on this host (--danger-classifier-config)');
+                        if (this.reviewerPin !== undefined)
+                            throw new ServiceError('classifier_pinned', `This host pinned the command reviewer (${this.reviewerPin}); sessions cannot change it`, 403);
+                        if (choice !== 'off' && !this.reviewers.has(choice ?? ''))
+                            throw new ServiceError('invalid_field', `Unknown reviewer ${choice}`);
+                        if (this.reviewers.size)
+                            this.store.transaction(() => {
+                                this.store.db
+                                    .prepare('INSERT INTO danger_classifier_choice(session_id,choice) VALUES(?,?) ON CONFLICT(session_id) DO UPDATE SET choice=excluded.choice')
+                                    .run(id, choice);
+                                this.store.event(id, 'permission.classifier-set', { choice });
+                            });
+                        this.send(res, 200, view());
+                        return;
+                    }
+                }
+                // Desk-session hooks (Phase 4b): list, trust the reviewed repository file, or revoke it.
+                if (parts[3] === 'hooks' && this.supervisor.hooks) {
+                    if (!this.supervisor.isInteractiveCoding(id))
+                        throw new ServiceError('hooks_denied', 'Hooks run only in your own desk sessions', 403);
+                    const hooks = this.supervisor.hooks;
+                    if (parts.length === 4 && req.method === 'GET') {
+                        this.send(res, 200, { sources: hooks.sources(id) });
+                        return;
+                    }
+                    if (parts.length === 5 && req.method === 'POST') {
+                        const body = object(await readJson(req));
+                        this.store.authorizeControl(id, {
+                            client_id: string(body.client_id, 'client_id', 128),
+                            token: string(body.token, 'token', 128),
+                        });
+                        if (parts[4] === 'trust') {
+                            this.send(res, 200, hooks.trust(id, string(body.digest, 'digest', 128)));
+                            return;
+                        }
+                        if (parts[4] === 'revoke') {
+                            hooks.revoke(id);
+                            this.send(res, 200, { revoked: true });
+                            return;
+                        }
+                    }
+                }
+                if (parts[3] === 'local-tasks' && this.localCoding) {
+                    if (!this.supervisor.isInteractiveCoding(id))
+                        throw new ServiceError('background_denied', 'Background commands are not admitted', 403);
+                    if (parts.length === 4 && req.method === 'GET') {
+                        this.send(res, 200, { tasks: this.localCoding.tasks(id) });
+                        return;
+                    }
+                    if (parts.length === 5 && req.method === 'GET') {
+                        this.send(res, 200, this.localCoding.taskOutput(id, parts[4]));
+                        return;
+                    }
+                    if (parts.length === 6 && parts[5] === 'stop' && req.method === 'POST') {
+                        this.send(res, 200, this.localCoding.stopTask(id, parts[4]));
+                        return;
+                    }
+                }
                 if (parts.length === 4 && parts[3] === 'local-diff' && req.method === 'GET') {
                     if (!this.store.localWorkspace(id))
                         throw new ServiceError('local_denied', 'Local workspace required');
                     this.send(res, 200, { text: await localDiff(this.store.session(id).cwd) });
                     return;
                 }
+                // An edit request's exact files, before and after, so an editor can show it as a diff.
+                if (parts.length === 4 && parts[3] === 'local-edits' && req.method === 'GET') {
+                    if (!this.store.localCoding(id))
+                        throw new ServiceError('local_denied', 'Local coding session required');
+                    this.send(res, 200, this.localCoding.edits(id, url.searchParams.has('operation_id')
+                        ? { operation: string(url.searchParams.get('operation_id'), 'operation_id', 128) }
+                        : { toolCall: string(url.searchParams.get('tool_call'), 'tool_call', 512) }));
+                    return;
+                }
                 if (parts.length === 4 && parts[3] === 'steer' && req.method === 'POST') {
                     const body = object(await readJson(req));
                     this.send(res, 200, await this.supervisor.steer(id, string(body.request_id, 'request_id', 128), string(body.text, 'text', 65536), {
@@ -1450,6 +1669,83 @@ export class SalaamService {
                         return;
                     }
                 }
+                if ((parts.length === 4 || parts.length === 5) &&
+                    (parts[3] === 'checkpoints' || parts[3] === 'rewind' || parts[3] === 'review')) {
+                    const checkpoints = this.tasks.checkpoints;
+                    // Operator ruling 2026-10-31: background commands may keep writing; say so, do not block.
+                    const running = (value) => value && this.localCoding
+                        ? {
+                            ...value,
+                            background_running: this.localCoding
+                                .tasks(id)
+                                .filter((t) => t.state === 'running').length,
+                        }
+                        : value;
+                    const controller = (body) => ({
+                        client_id: string(body.client_id, 'client_id', 128),
+                        token: string(body.token, 'token', 128),
+                    });
+                    const idle = () => {
+                        this.supervisor.assertIdle(id);
+                        if (this.store.db
+                            .prepare("SELECT 1 FROM attempts WHERE session_id=? AND state IN ('accepted','dispatching','running')")
+                            .get(id))
+                            throw new ServiceError('session_busy', 'Wait for the current turn to finish');
+                    };
+                    if (parts[3] === 'checkpoints' && parts.length === 4 && req.method === 'GET') {
+                        this.send(res, 200, { checkpoints: checkpoints.list(id) });
+                        return;
+                    }
+                    if (parts[3] === 'rewind' && parts.length === 4 && req.method === 'GET') {
+                        this.send(res, 200, running(checkpoints.reviewRewind(id, string(url.searchParams.get('attempt'), 'attempt', 128), string(url.searchParams.get('mode'), 'mode', 16))));
+                        return;
+                    }
+                    if (parts[3] === 'rewind' && parts.length === 4 && req.method === 'POST') {
+                        if (this.agentpod)
+                            throw new ServiceError('admission_required', 'Managed station sessions cannot rewind');
+                        const body = object(await readJson(req));
+                        const request = string(body.request_id, 'request_id', 128);
+                        this.send(res, 200, checkpoints.rewind(id, request, string(body.attempt_id, 'attempt_id', 128), string(body.mode, 'mode', 16), string(body.review_digest, 'review_digest', 128), controller(body), idle, (leaf, summary) => this.store.forkLocal(id, request + ':fork', this.store.session(id).revision, controller(body), (source, target) => {
+                            const dir = join(this.stateDir, 'workers', target, 'sessions');
+                            mkdirSync(dir, { recursive: true, mode: 0o700 });
+                            return forkPiSession(source.pi_session_file, source.cwd, dir, target, {
+                                leaf,
+                                summary,
+                            });
+                        }, { leaf })));
+                        return;
+                    }
+                    if (parts[3] === 'review' && parts.length === 4 && req.method === 'GET') {
+                        this.send(res, 200, { review: running(checkpoints.review(id)) });
+                        return;
+                    }
+                    if (parts[3] === 'review' && parts[4] === 'hunks' && req.method === 'GET') {
+                        const q = (name) => string(url.searchParams.get(name), name, 1024);
+                        this.send(res, 200, {
+                            hunks: checkpoints.hunks(id, q('path'), q('base'), q('current')),
+                        });
+                        return;
+                    }
+                    if (parts[3] === 'review' && parts.length === 4 && req.method === 'POST') {
+                        const body = object(await readJson(req));
+                        const hunk = body.hunk === undefined
+                            ? undefined
+                            : {
+                                text: string(object(body.hunk).text, 'hunk.text', 256 * 1024),
+                                patch: string(object(body.hunk).patch, 'hunk.patch', 256 * 1024),
+                            };
+                        this.send(res, 200, {
+                            review: running(checkpoints.act(id, {
+                                action: string(body.action, 'action', 16),
+                                path: string(body.path, 'path', 1024),
+                                ...(hunk ? { hunk } : {}),
+                                base: string(body.base, 'base', 128),
+                                current: string(body.current, 'current', 128),
+                            }, controller(body), idle)),
+                        });
+                        return;
+                    }
+                }
                 if (parts.length === 4 && parts[3] === 'fork' && req.method === 'POST') {
                     if (this.agentpod)
                         throw new ServiceError('admission_required', 'Managed station sessions cannot be forked locally');
@@ -1845,6 +2141,29 @@ export class SalaamService {
                     this.send(res, 200, this.store.control(id, string(body.request_id, 'request_id', 128), string(body.client_id, 'client_id', 128), action, integer(body.expected_epoch, 'expected_epoch'), body.token === undefined ? undefined : string(body.token, 'token', 128)));
                     return;
                 }
+                if (parts[3] === 'route' && req.method === 'POST') {
+                    const body = object(await readJson(req));
+                    this.store.authorizeControl(id, {
+                        client_id: string(body.client_id, 'client_id', 128),
+                        token: string(body.token, 'token', 128),
+                    });
+                    if (!this.tasks.instructions.applies(id))
+                        throw new ServiceError('route_switch_unavailable', 'Only interactive sessions can switch model mid-session', 409);
+                    this.supervisor.assertIdle(id);
+                    const input = object(body.route);
+                    const requested = providerRoute(string(input.provider, 'provider'), input.model === undefined ? undefined : string(input.model, 'model'), undefined, undefined, undefined, input.reasoning);
+                    const prepared = await this.supervisor.prepareProviderRoute(requested).catch(() => {
+                        throw new ServiceError('provider_auth_required', `Configure ${requested.provider} with sal auth login ${requested.provider} or sal auth set ${requested.provider}, then retry.`, 400);
+                    });
+                    if (prepared.output_cap !== (this.store.localOutputCap(id) ?? true))
+                        throw new ServiceError('provider_output_cap_changed', 'That route enforces output caps differently; start a new session for it', 409);
+                    this.supervisor.assertIdle(id);
+                    this.store.switchRoute(id, prepared.route);
+                    await this.supervisor.stopSession(id);
+                    void this.supervisor.start(id).catch(() => { });
+                    this.send(res, 200, this.supervisor.inspect(id));
+                    return;
+                }
                 if (parts[3] === 'resume' && req.method === 'POST') {
                     const body = object(await readJson(req));
                     this.store.authorizeControl(id, {
@@ -1858,6 +2177,13 @@ export class SalaamService {
                         throw new ServiceError('provider_route_changed', 'Restart the service with this session’s original provider route, or create a new session');
                     if (session.state !== 'ready' && session.state !== 'starting')
                         void this.supervisor.start(id).catch(() => { });
+                    // Show the project instructions an interactive session will use before its first turn.
+                    try {
+                        this.tasks.instructions?.refresh(id);
+                    }
+                    catch {
+                        /* reported on the next turn */
+                    }
                     this.send(res, 202, this.supervisor.inspect(id));
                     return;
                 }
@@ -1873,12 +2199,23 @@ export class SalaamService {
                 }
                 if (parts[3] === 'submissions' && req.method === 'POST') {
                     const body = object(await readJson(req));
-                    const submission = this.store.accept(id, string(body.request_id, 'request_id', 128), integer(body.expected_revision, 'expected_revision'), string(body.text, 'text', 256 * 1024), {
+                    const local = this.store.localCoding(id);
+                    let text = string(body.text, 'text', 256 * 1024);
+                    let images = [];
+                    // Desk prompts attach their @mentions (Phase 3b); board work never reads the tree here.
+                    if (local) {
+                        if (body.confirm_sensitive !== undefined && typeof body.confirm_sensitive !== 'boolean')
+                            throw new ServiceError('invalid_field', 'Invalid confirm_sensitive flag', 400);
+                        ({ text, images } = expandPrompt(this.store.session(id).cwd, text, body.confirm_sensitive === true));
+                    }
+                    const submission = this.store.accept(id, string(body.request_id, 'request_id', 128), integer(body.expected_revision, 'expected_revision'), text, {
                         client_id: string(body.client_id, 'client_id', 128),
                         token: string(body.token, 'token', 128),
                     }, body.continues_attempt_id === undefined
                         ? null
                         : string(body.continues_attempt_id, 'continues_attempt_id', 128));
+                    // Same tick as the acceptance, so dispatch cannot claim the submission without them.
+                    storeImages(this.store.db, submission.id, images);
                     this.send(res, 202, submission);
                     return;
                 }

package/dist/src/session-pick.js

@@ -0,0 +1,77 @@
+import { typedText } from './shell-passthrough.js';
+import { statSync } from 'node:fs';
+import { OFFLINE_ROUTE } from './provider-config.js';
+export function parseCodeStart(flags) {
+    const chosen = ['continue', 'new', 'resume'].filter((f) => flags.includes('--' + f));
+    if (chosen.length > 1)
+        throw new Error('Choose one of --continue, --new or --resume.');
+    return chosen[0] ?? 'continue';
+}
+const folder = (path) => (path.length > 1 ? path.replace(/\/+$/, '') : path);
+/** The most recently created session in this folder; sessions come oldest first. */
+export function latestSessionFor(sessions, cwd) {
+    const want = folder(cwd);
+    return [...sessions].reverse().find((s) => folder(s.cwd) === want)?.id;
+}
+/** Whether a new `salaam code` session should use Anthropic: only when nothing was chosen, the
+ * running service defaults to offline (as a TUI-started one does) and a key is stored. */
+export function deskUsesAnthropic(chosen, serviceProvider, hasAnthropicKey) {
+    return (!chosen &&
+        (serviceProvider ?? OFFLINE_ROUTE.provider) === OFFLINE_ROUTE.provider &&
+        hasAnthropicKey);
+}
+/** `salaam -c` and `salaam --continue` are short for `salaam code --continue`. */
+export function continueArgs(args) {
+    if (args[0] !== '-c' && args[0] !== '--continue')
+        return;
+    return ['code', '--continue', ...args.slice(1)];
+}
+const activeAt = (s) => s.last_active_at ? Date.parse(s.last_active_at) : -Infinity;
+/** This folder's sessions first, then every other one; newest first within each. */
+export function pickerOrder(sessions, cwd) {
+    const want = folder(cwd);
+    const newest = (a, b) => activeAt(b) - activeAt(a);
+    const here = sessions.filter((s) => folder(s.cwd) === want).sort(newest);
+    return [...here, ...sessions.filter((s) => folder(s.cwd) !== want).sort(newest)];
+}
+export function relativeTime(at, now = Date.now()) {
+    if (!at)
+        return '';
+    const s = Math.max(0, (now - Date.parse(at)) / 1000);
+    if (s < 60)
+        return 'just now';
+    if (s < 3600)
+        return `${Math.floor(s / 60)} min ago`;
+    if (s < 86400)
+        return `${Math.floor(s / 3600)} h ago`;
+    if (s < 2 * 86400)
+        return 'yesterday';
+    return `${Math.floor(s / 86400)} days ago`;
+}
+/** One picker row: what you last asked (one short line), when, and where. */
+export function sessionDescription(s, now = Date.now()) {
+    const prompt = s.last_prompt?.replace(/\s+/g, ' ').trim();
+    const quoted = prompt ? `“${prompt.length > 60 ? prompt.slice(0, 59) + '…' : prompt}”` : '';
+    return [quoted || s.state, relativeTime(s.last_active_at, now), s.cwd]
+        .filter(Boolean)
+        .join(' · ');
+}
+/** What the picker shows for a desk session: its newest prompt (at most 200 characters) and when
+ * its transcript last changed. Board and station sessions get nothing extra. */
+export function pickerSummary(session, store) {
+    if (!session.capabilities.includes('local-coding-reviewed'))
+        return {};
+    const prompt = store.lastPrompt(session.id);
+    let at;
+    try {
+        if (session.pi_session_file)
+            at = statSync(session.pi_session_file).mtime.toISOString();
+    }
+    catch {
+        /* no transcript yet */
+    }
+    return {
+        ...(prompt ? { last_prompt: typedText(prompt).slice(0, 200) } : {}),
+        ...(at ? { last_active_at: at } : {}),
+    };
+}

package/dist/src/shell-passthrough.js

@@ -0,0 +1,39 @@
+/**
+ * `!command` in a desk session (trial polish #12, 2026-10-31). The terminal never runs the command
+ * itself: it asks the model to run exactly that text through salaam_local. The service reads the
+ * same fixed prompt back from the turn's submission and binds the turn to the typed command, so a
+ * model that runs anything else is refused, whatever the session's mode.
+ */
+const LEAD = 'I want to run this shell command myself. Run exactly this command with salaam_local';
+export function shellPassthrough(text) {
+    const t = text.trim();
+    if (!t.startsWith('!') || t.startsWith('!!'))
+        return undefined;
+    const command = t.slice(1).trim();
+    if (command === '' || command.includes('\n') || command.includes('```'))
+        return undefined;
+    return [
+        LEAD,
+        '(action "command"), unchanged: no edits, no extra flags, no other commands. Then show me its',
+        'output and exit status, and do nothing else unless I ask.',
+        '',
+        '```sh',
+        command,
+        '```',
+    ].join('\n');
+}
+/** The command a turn's prompt binds it to, when the prompt is exactly a `!command` request. */
+export function boundShellCommand(prompt) {
+    if (!prompt.startsWith(LEAD + '\n'))
+        return undefined;
+    const lines = prompt.split('\n');
+    if (lines.length !== 7 || lines[4] !== '```sh' || lines[6] !== '```')
+        return undefined;
+    const command = lines[5];
+    return shellPassthrough('!' + command) === prompt ? command : undefined;
+}
+/** What the person typed, for display: `! COMMAND` for a `!command` prompt, else the text. */
+export function typedText(prompt) {
+    const command = boundShellCommand(prompt);
+    return command === undefined ? prompt : '! ' + command;
+}