@awak-app/simy-cli 0.5.66 → 0.5.68

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@awak-app/simy-cli",
-  "version": "0.5.66",
+  "version": "0.5.68",
   "description": "Local SIMY executor for non-coding Local Tasks and Agentic Loop runs.",
   "type": "module",
   "bin": {

package/src/agent.js

@@ -878,6 +878,10 @@ export async function startAgent({
         return (dependencies.runSqmLocalReview || ((value, selected, options) => runStructuredProviderReview(selected, value, options)))(request, provider, { beforeDispatch });
       },
       onChange: notifyDesktopStatusChange,
+      getWorkspaceUrl: () => {
+        const slug = desktopAccess.snapshot().organizationSlug;
+        return slug ? new URL(`/org/${encodeURIComponent(slug)}/quality-monitor/`, apiOrigin).toString() : null;
+      },
       getHistoryScope: () => sqmHistoryScope(apiOrigin, session),
       recordHistory: (scope, item) => sqmHistory.record(scope, item),
       onHistoryError: () => { if (!quiet) console.error("Could not save the local SQM analysis history."); },

package/src/cli-contract.js

@@ -13,7 +13,7 @@ import {
 } from "./durable-local-task-steps-contract.js";
 import { buildExecutionCapabilityContract } from "./execution-capability-contract.js";
 
-export const CLI_VERSION = "0.5.66";
+export const CLI_VERSION = "0.5.68";
 export const CLI_API_CONTRACT_VERSION = 10;
 
 export function withCliContract(

package/src/session-supervision/claude-sessions.js

@@ -3,6 +3,7 @@ import { lstat, open, readdir, realpath } from 'node:fs/promises';
 import { homedir } from 'node:os';
 import { isAbsolute, join, relative, resolve, sep } from 'node:path';
 import { boundConversation, CONVERSATION_LIMITS, textContent } from './conversation-context.js';
+import { sessionTitle } from './session-titles.js';
 
 const UUID = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
 const MAX_PROJECTS = 256;
@@ -54,21 +55,41 @@ function metadataFromPrefix(bytes, id) {
   const text = bytes.toString('utf8');
   const complete = text.slice(0, text.lastIndexOf('\n') + 1);
   if (!complete) return null;
-  let cwd = '', title = '', found = false;
+  let cwd = '', title = '', customTitle = '', found = false;
   for (const line of complete.split('\n').slice(0, MAX_LINES)) {
     if (!line) continue;
     let item;
     try { item = JSON.parse(line); } catch { return null; }
     if (item.sessionId !== id) return null;
+    if (item.type === 'custom-title' && item.isSidechain !== true) { customTitle = sessionTitle(item.customTitle); continue; }
     if (item.type !== 'user' && item.type !== 'assistant') continue;
     if (item.isSidechain === true) return null;
     if (typeof item.cwd !== 'string' || !isAbsolute(item.cwd)) continue;
     if (!cwd) cwd = resolve(item.cwd);
     if (cwd !== resolve(item.cwd)) return null;
-    if (!title && typeof item.slug === 'string') title = item.slug.replace(/[\x00-\x1f\x7f]/g, '').slice(0, 120);
+    if (!title) title = sessionTitle(item.slug);
     found = true;
   }
-  return found ? { cwd, title } : null;
+  return found ? { cwd, title: customTitle || title } : null;
+}
+
+// /rename appends a custom-title envelope, often beyond the bounded prefix.
+// Inspect only those envelopes; conversation text is never used as a title.
+async function latestCustomTitle(handle, size, id) {
+  const offset = Math.max(0, size - 64 * 1024);
+  const buffer = Buffer.alloc(Math.min(size, 64 * 1024));
+  const { bytesRead } = await handle.read(buffer, 0, buffer.length, offset);
+  let text = buffer.subarray(0, bytesRead).toString('utf8');
+  if (offset) text = text.slice(text.indexOf('\n') + 1);
+  text = text.slice(0, text.lastIndexOf('\n') + 1);
+  let title = '';
+  for (const line of text.split('\n')) {
+    try {
+      const item = JSON.parse(line);
+      if (item.type === 'custom-title' && item.sessionId === id && item.isSidechain !== true) title = sessionTitle(item.customTitle);
+    } catch { /* Ignore partial or unrelated tail entries. */ }
+  }
+  return title;
 }
 
 async function inspectTranscript(root, projectName, fileName, observedAt, includeContext = false, cacheMetadata = false) {
@@ -104,6 +125,7 @@ async function inspectTranscript(root, projectName, fileName, observedAt, includ
     const buffer = Buffer.allocUnsafe(Math.min(opened.size, MAX_PREFIX_BYTES));
     const { bytesRead } = await handle.read(buffer, 0, buffer.length, 0);
     const metadata = metadataFromPrefix(buffer.subarray(0, bytesRead), id);
+    if (metadata) metadata.title = await latestCustomTitle(handle, opened.size, id) || metadata.title;
     if (cacheMetadata) {
       const [after, pathAfter] = await Promise.all([handle.stat({ bigint: true }), lstat(path, { bigint: true })]);
       if (fingerprint(after) === key && fingerprint(pathAfter) === key && pathAfter.isFile() && !pathAfter.isSymbolicLink())

package/src/session-supervision/index.js

@@ -4,6 +4,7 @@ import { TERMINAL as TERMINAL_RUNS, controlDelivery, listRecords, readRecord, pr
 import { listCodexSessions, openCodexAppServer, readCodexSession, CodexAppServerError } from './codex-app-server.js';
 import { listClaudeSessions, readClaudeSession } from './claude-sessions.js';
 import { originalSessionTarget } from './original-session-target.js';
+import { readCodexTitles, sessionTitle } from './session-titles.js';
 
 function autoContinueState(record, isRunning, supported) {
   if (record?.autoContinue !== true) return { autoContinueStatus: 'off', autoContinueReason: null };
@@ -55,7 +56,7 @@ async function liveClaudeHook(record, metadata, inspectProcess) {
 }
 
 export async function discoverSessions({ root, openClient = openCodexAppServer, listClaude = listClaudeSessions,
-  inspectProcess = processIdentity, now = Date.now(), signal } = {}) {
+  inspectProcess = processIdentity, now = Date.now(), signal, listCodexTitles = readCodexTitles } = {}) {
   const assertActive = () => { if (signal?.aborted) throw new CodexAppServerError('cancelled', 'Conversation discovery was cancelled.'); };
   assertActive();
   const records = await listRecords(root);
@@ -81,6 +82,8 @@ export async function discoverSessions({ root, openClient = openCodexAppServer,
   // Historical sessions must not fan out hundreds of process-inspection
   // subprocesses. Preserve input ordering while limiting live metadata probes.
   const items = [...byKey.values()], projected = new Array(items.length);
+  const codexTitles = items.some(item => item.provider === 'codex' && !sessionTitle(item.title)) ? await listCodexTitles() : new Map();
+  assertActive();
   let nextItem = 0;
   const project = async item => {
     const record = recordsByKey.get(`${item.provider}:${item.session}`);
@@ -97,7 +100,8 @@ export async function discoverSessions({ root, openClient = openCodexAppServer,
         else if (isRunning === undefined && !record.runtime.isRunning) isRunning = false;
       } catch { /* Unverified process leaves runtime status unknown. */ }
     }
-    return { provider: item.provider, session: item.session, cwd: item.cwd, title: item.title, source: item.source,
+    return { provider: item.provider, session: item.session, cwd: item.cwd,
+      title: sessionTitle(item.title) || (item.provider === 'codex' ? codexTitles.get(item.session) || '' : ''), source: item.source,
       ...(typeof isRunning === 'boolean' ? { isRunning } : {}),
       ...(item.providerGoal !== undefined ? { providerGoal: item.providerGoal } : {}),
       ...(item.goalReadState ? { goalReadState: item.goalReadState } : {}),

package/src/session-supervision/session-titles.js

@@ -0,0 +1,48 @@
+import { constants } from 'node:fs';
+import { lstat, open } from 'node:fs/promises';
+import { homedir } from 'node:os';
+import { join } from 'node:path';
+
+const UUID = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
+const MAX_INDEX_BYTES = 1024 * 1024;
+let cached;
+const fingerprint = info => [info.dev, info.ino, info.size, info.mtimeNs, info.ctimeNs].join(':');
+
+export function sessionTitle(value) {
+  return typeof value === 'string' ? value.replace(/[\x00-\x1f\x7f]/g, ' ').replace(/\s+/g, ' ').trim().slice(0, 200) : '';
+}
+
+// Read title metadata only, including when an older CLI cannot list threads.
+// This never reads prompts or changes the provider's continuation capabilities.
+export async function readCodexTitles({ codexHome = process.env.CODEX_HOME || join(homedir(), '.codex') } = {}) {
+  const path = join(codexHome, 'session_index.jsonl');
+  let handle;
+  try {
+    const before = await lstat(path, { bigint: true });
+    if (!before.isFile() || before.isSymbolicLink()) return new Map();
+    const key = fingerprint(before);
+    if (cached?.path === path && cached.key === key) return new Map(cached.titles);
+    handle = await open(path, constants.O_RDONLY | (constants.O_NOFOLLOW || 0));
+    if (fingerprint(await handle.stat({ bigint: true })) !== key) return new Map();
+    const size = Number(before.size), offset = Math.max(0, size - MAX_INDEX_BYTES);
+    const buffer = Buffer.alloc(Math.min(size, MAX_INDEX_BYTES));
+    const { bytesRead } = await handle.read(buffer, 0, buffer.length, offset);
+    let text = buffer.subarray(0, bytesRead).toString('utf8');
+    if (offset) text = text.slice(text.indexOf('\n') + 1);
+    text = text.slice(0, text.lastIndexOf('\n') + 1);
+    const titles = new Map();
+    for (const line of text.split('\n')) {
+      try {
+        const item = JSON.parse(line);
+        if (!UUID.test(item.id || '')) continue;
+        const title = sessionTitle(item.thread_name);
+        // Appended renames win, including an explicitly cleared title.
+        if (typeof item.thread_name === 'string') titles.set(item.id, title);
+      } catch { /* A broken entry does not hide other sessions. */ }
+    }
+    if (fingerprint(await handle.stat({ bigint: true })) !== key || fingerprint(await lstat(path, { bigint: true })) !== key) return new Map();
+    cached = { path, key, titles };
+    return new Map(titles);
+  } catch { return new Map(); }
+  finally { await handle?.close(); }
+}

package/src/sqm/history-store.js

@@ -60,7 +60,7 @@ export function createSqmHistoryStore({ root = defaultSessionRoot(), now = () =>
       const item = (await this.items(scope)).find(item => item.key === key);
       if (!item) return null;
       return { key, instruction: item.instruction || `SQM local review · ${item.provider || "local"}`,
-        result: item.result || "", error: item.state === "interrupted" ? "This client stopped before the local run finished. Check the web workspace for its current retry status." : item.error || "", links: [] };
+        result: item.result || "", error: item.state === "interrupted" ? "This client stopped before the local run finished. Check the web workspace for its current retry status." : item.error || "", links: item.webUrl ? [{ label: "Analysis details", url: item.webUrl }] : [] };
     },
   };
 }
@@ -69,7 +69,7 @@ function metadata(entry) {
   if (!entry || !["analysis", "check"].includes(entry.kind) || typeof entry.key !== "string" || !entry.key.startsWith(entry.kind === "analysis" ? "sqm:" : "sqm-check:")) return null;
   if (!["running", "completed", "failed", "unavailable", "interrupted", "verification_pending"].includes(entry.state) || !Number.isFinite(Date.parse(entry.startedAt))) return null;
   const item = {};
-  for (const name of ["key", "id", "kind", "label", "state", "provider", "phase", "repository", "startedAt", "finishedAt", "instruction", "result", "error"]) {
+  for (const name of ["key", "id", "kind", "label", "state", "provider", "phase", "repository", "startedAt", "finishedAt", "instruction", "result", "error", "errorCode", "webUrl"]) {
     if (typeof entry[name] === "string") item[name] = entry[name].slice(0, ["instruction", "result", "error"].includes(name) ? 4096 : 512);
   }
   return item;

package/src/sqm/review-worker.js

@@ -18,11 +18,12 @@ const DEFAULT_EXECUTION_TIMEOUT_MS = 240_000;
 export const SQM_CODEX_MODEL = "gpt-6.1-sol";
 
 export class SqmLocalReviewError extends Error {
-  constructor(message, { status = null, code = "sqm_local_review_error" } = {}) {
+  constructor(message, { status = null, code = "sqm_local_review_error", stage = null, jobId = null, failureRecorded = false } = {}) {
     super(message);
     this.name = "SqmLocalReviewError";
     this.status = status;
     this.code = code;
+    this.stage = stage; this.jobId = jobId; this.failureRecorded = failureRecorded;
   }
 }
 
@@ -69,6 +70,7 @@ export function createSqmLocalReviewApi({
         throw new SqmLocalReviewError(webApiErrorMessage(payload, response.status), {
           status: response.status,
           code: machineCode(payload?.code || payload?.detail?.code),
+          stage: payload?.detail?.stage, jobId: payload?.detail?.job_id, failureRecorded: payload?.detail?.failure_recorded === true,
         });
       }
       return payload;
@@ -87,6 +89,7 @@ export function createSqmLocalReviewWorker({
   onError = () => {},
   onChange = () => {},
   getHistoryScope = () => null,
+  getWorkspaceUrl = () => null,
   recordHistory = async () => {},
   onHistoryError = () => {},
 } = {}) {
@@ -101,12 +104,13 @@ export function createSqmLocalReviewWorker({
   const persist = async (scope, item) => {
     try { await recordHistory(scope, item); } catch { onHistoryError(); /* History failure must not fail a leased review. */ }
   };
-  const updateTask = async (id, state, scope) => {
+  const updateTask = async (id, state, scope, failure = null, failureReported = null) => {
     const item = history.find(item => item.id === id);
     if (item) {
       item.state = state; item.finishedAt = new Date().toISOString();
       item.result = state === "completed" ? "SQM review submitted." : "";
-      item.error = state === "failed" ? "Local review failed. Check the SQM workspace for retry status." : "";
+      item.error = state === "failed" ? formatFailure(failure, failureReported) : "";
+      if (failure) { item.phase = failure.stage; item.errorCode = failure.code; }
       await persist(scope, item);
     }
     onChange();
@@ -120,13 +124,12 @@ export function createSqmLocalReviewWorker({
     lease_token: claim.lease_token,
   });
 
-  const processClaim = async (claim, provider) => {
-    const historyScope = getHistoryScope();
+  const processClaim = async (claim, provider, historyScope, workspaceUrl) => {
     activeJobId = claim.job_id;
     const priorIndex = history.findIndex(item => item.id === claim.job_id);
     if (priorIndex >= 0) history.splice(priorIndex, 1);
     history.unshift({ id: claim.job_id, key: `sqm:${claim.job_id}`, label: `SQM · ${String(claim.job_id).slice(0, 8)}`,
-      kind: "analysis", state: "running", provider, startedAt: new Date().toISOString() });
+      kind: "analysis", state: "running", provider, startedAt: new Date().toISOString(), webUrl: analysisTaskUrl(workspaceUrl, claim.job_id) });
     historyScopes.set(history[0], JSON.stringify(historyScope));
     history.splice(50);
     await persist(historyScope, history[0]);
@@ -137,8 +140,10 @@ export function createSqmLocalReviewWorker({
     };
     heartbeatTimer = setInterval(heartbeat, heartbeatIntervalMs);
     heartbeatTimer.unref?.();
+    let stage = "analysis";
     try {
       const analysis = await runReview(claim.request, provider);
+      stage = "advance";
       const advanced = await api.request({
         action: "advance",
         ...leaseBody(claim, provider),
@@ -146,7 +151,9 @@ export function createSqmLocalReviewWorker({
         executor_model: analysis.model,
       });
       if (advanced.completed) { await updateTask(claim.job_id, "completed", historyScope); return advanced; }
+      stage = "adjudication";
       const adjudication = await runReview(advanced.request, provider);
+      stage = "complete";
       const completed = await api.request({
         action: "complete",
         ...leaseBody(claim, provider),
@@ -157,7 +164,16 @@ export function createSqmLocalReviewWorker({
       await updateTask(claim.job_id, "completed", historyScope);
       return completed;
     } catch (error) {
-      await updateTask(claim.job_id, "failed", historyScope);
+      const failure = reviewFailure(error, stage, provider);
+      let failureReported = error.failureRecorded === true;
+      if (!failureReported) {
+        try {
+          const report = await api.request({ action: "fail", ...leaseBody(claim, provider),
+            error_stage: failure.stage, error_code: failure.code, ...(failure.status ? { error_status: failure.status } : {}) });
+          failureReported = report?.recorded === true;
+        } catch { /* Keep the original failure even when reporting fails. */ }
+      }
+      await updateTask(claim.job_id, "failed", historyScope, failure, failureReported);
       throw error;
     } finally {
       if (heartbeatTimer) clearInterval(heartbeatTimer);
@@ -173,7 +189,7 @@ export function createSqmLocalReviewWorker({
     desktopTasks() { return { activeCount: activeJobId ? 1 : 0, items: visibleHistory().map(item => ({ ...item })) }; },
     desktopTaskDetail(key) {
       const item = visibleHistory().find(item => item.key === key);
-      return item ? { key, instruction: `SQM local review · ${item.provider}`, result: item.state === "completed" ? "SQM review submitted." : "", error: item.state === "failed" ? "Local review failed. Check the SQM workspace for retry status." : "", links: [] } : null;
+      return item ? { key, instruction: `SQM local review · ${item.provider}`, result: item.state === "completed" ? "SQM review submitted." : "", error: item.error || "", links: item.webUrl ? [{ label: "Analysis details", url: item.webUrl }] : [] } : null;
     },
     resetHistory() { if (!pollPromise) { history.splice(0); issue = null; } },
     poll() {
@@ -183,6 +199,8 @@ export function createSqmLocalReviewWorker({
       assertProvider(provider);
       const releaseWork = beginWork?.();
       if (!releaseWork) return Promise.resolve(null);
+      const historyScope = getHistoryScope();
+      const workspaceUrl = getWorkspaceUrl();
       pollPromise = (async () => {
         const claim = await api.request({
           action: "claim",
@@ -192,9 +210,18 @@ export function createSqmLocalReviewWorker({
         });
         if (issue) { issue = null; onChange(); }
         if (!claim?.claimed) return null;
-        return processClaim(claim, provider);
+        return processClaim(claim, provider, historyScope, workspaceUrl);
       })()
-        .catch((error) => {
+        .catch(async (error) => {
+          if (error.jobId && /^[a-f0-9-]{36}$/i.test(error.jobId) && !history.some(item => item.id === error.jobId)) {
+            const scope = historyScope;
+            const failure = reviewFailure(error, "claim", provider);
+            const item = { id: error.jobId, key: `sqm:${error.jobId}`, label: `SQM · ${error.jobId.slice(0, 8)}`,
+              kind: "analysis", state: "failed", provider, startedAt: new Date().toISOString(), finishedAt: new Date().toISOString(),
+              phase: failure.stage, errorCode: failure.code, error: formatFailure(failure, error.failureRecorded === true), webUrl: analysisTaskUrl(workspaceUrl, error.jobId) };
+            history.unshift(item); historyScopes.set(item, JSON.stringify(scope)); history.splice(50);
+            await persist(scope, item);
+          }
           issue = error.status === 403 ? "forbidden" : error.status === 401 ? "signed_out" : "unavailable";
           onError(error);
           return null;
@@ -453,3 +480,38 @@ function assertProvider(provider) {
     throw new TypeError("Unsupported SQM local review provider");
   }
 }
+
+function reviewFailure(error, stage, provider) {
+  let code = machineCode(error?.code);
+  const message = errorMessage(error);
+  if (code === "sqm_local_review_error" || code === `${provider}_local_review_failed`) {
+    if (/timed out|timeout/i.test(message)) code = "analysis_timeout";
+    else if (providerSignInRequired(message)) code = "provider_sign_in_required";
+    else if (stage === "analysis" || stage === "adjudication") code = `${provider}_local_review_failed`;
+  }
+  return { stage: /^[a-z_]{1,40}$/.test(error?.stage || "") ? error.stage : stage, code,
+    status: Number.isInteger(error?.status) && error.status >= 400 && error.status <= 599 ? error.status : null };
+}
+
+function formatFailure(failure, failureReported = null) {
+  if (!failure) return "SQM analysis failed. No failure diagnostics were recorded.";
+  const hint = failure.code === "provider_sign_in_required" ? "Sign in to the local AI app again."
+    : failure.code === "analysis_timeout" ? "The local AI analysis exceeded its time limit."
+    : failure.code === "sqm_source_loading_failed" ? "SIMY could not load evidence for this analysis."
+    : failure.status === 403 || failure.code === "desktop_access_required" ? "Check SQM and Developer access for the selected organization."
+    : failure.status === 401 ? "Reconnect your SIMY session."
+    : failure.code.endsWith("_local_unavailable") ? "Install or update the selected local AI app."
+    : failure.code.endsWith("_local_review_failed") ? "The local AI app did not complete a valid structured review."
+    : "Open analysis details to inspect the task state.";
+  return `${failure.stage} · ${failure.code}${failure.status ? ` · HTTP ${failure.status}` : ""}\n${hint}${failureReported === false ? "\nThe failure could not be reported to SIMY Web. The web task may still show as analyzing until its lease expires." : ""}`;
+}
+
+function analysisTaskUrl(workspaceUrl, id) {
+  if (!workspaceUrl || !/^[a-f0-9-]{36}$/i.test(id)) return "";
+  try {
+    const url = new URL(workspaceUrl);
+    if (!/^https?:$/.test(url.protocol)) return "";
+    url.searchParams.set("qualityTab", "analysis"); url.searchParams.set("qualityAnalysis", id);
+    return url.toString();
+  } catch { return ""; }
+}