@awak-app/simy-cli 0.5.67 → 0.5.69

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@awak-app/simy-cli",
-  "version": "0.5.67",
+  "version": "0.5.69",
   "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.67";
+export const CLI_VERSION = "0.5.69";
 export const CLI_API_CONTRACT_VERSION = 10;
 
 export function withCliContract(

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 ""; }
+}