@awak-app/simy-cli 0.7.35 → 0.7.37

package/src/session-supervision/owner-transport.js

@@ -12,10 +12,10 @@ export const ownerSocketPath = (root, platform = process.platform) => platform =
 const socketPath = ownerSocketPath;
 // The socket and nonce are accessible only to the local OS account. Application
 // account/provider checks still run in the owner's ordinary command handler.
-export async function createOwnerTransport({ root, nonce, allowedActions, dispatch, allowControllerArchive = false }) {
+export async function createOwnerTransport({ root, nonce, allowedActions, dispatch, allowControllerArchive = false, allowDesktopClient = false }) {
   const writers = new WeakMap();
-  const peers = new Set(), subscribers = new Set(), requests = new Map();
-  let latest;
+  const peers = new Set(), subscribers = new Set(), desktopSubscribers = new Set(), requests = new Map();
+  let latest, latestDesktop;
   const send = (peer, frame) => {
     if (peer.destroyed) return;
     // A valid full-history status can already occupy up to 64 MiB of the
@@ -28,10 +28,10 @@ export async function createOwnerTransport({ root, nonce, allowedActions, dispat
     peer.setEncoding('utf8');
     const writer = createDesktopFrameWriter(peer);
     writers.set(peer, writer);
-    let authenticated = false, controllerArchive = false, buffer = '';
+    let authenticated = false, controllerArchive = false, desktopClient = false, buffer = '';
     const timeout = setTimeout(() => peer.destroy(), 5000);
     peer.on('error', () => {});
-    peer.on('close', () => { writer.close(); clearTimeout(timeout); peers.delete(peer); subscribers.delete(peer); for (const [id, item] of requests) if (item.peer === peer) requests.delete(id); });
+    peer.on('close', () => { writer.close(); clearTimeout(timeout); peers.delete(peer); subscribers.delete(peer); desktopSubscribers.delete(peer); for (const [id, item] of requests) if (item.peer === peer) requests.delete(id); });
     peer.on('data', chunk => {
       buffer += chunk.toString();
       if (buffer.length > 16384) { peer.destroy(); return; }
@@ -42,16 +42,18 @@ export async function createOwnerTransport({ root, nonce, allowedActions, dispat
         if (!command || typeof command !== 'object' || Array.isArray(command)) { peer.destroy(); return; }
         if (!authenticated) {
           if (command.nonce !== nonce) { peer.destroy(); return; }
-          authenticated = true; controllerArchive = allowControllerArchive && command.client === 'archive-controller'; clearTimeout(timeout); peers.add(peer);
+          authenticated = true; desktopClient = allowDesktopClient && command.client === 'desktop'; controllerArchive = allowControllerArchive && command.client === 'archive-controller'; clearTimeout(timeout); peers.add(peer);
+          if (command.desktopProtocol === 1 && desktopClient) send(peer, { type: 'desktop_owner_ready', version: 1 });
           // One-shot controls need their result, not multi-megabyte broadcasts
           // that can fill the write queue before the result is sent.
           if (command.subscribeStatus !== false) {
-            subscribers.add(peer);
-            if (latest) send(peer, latest);
+            (desktopClient ? desktopSubscribers : subscribers).add(peer);
+            const snapshot = desktopClient ? latestDesktop : latest;
+            if (snapshot) send(peer, snapshot);
           }
           continue;
         }
-        if (!allowedActions.has(command.action) || typeof command.id !== 'string' || command.id.length > 128 || command.hostId && command.hostId !== 'local' && !(controllerArchive && ['session_archive', 'session_restore', 'session_provider_setup'].includes(command.action))) {
+        if ((!desktopClient && !allowedActions.has(command.action)) || typeof command.id !== 'string' || command.id.length > 128 || command.hostId && command.hostId !== 'local' && !desktopClient && !(controllerArchive && ['session_archive', 'session_restore', 'session_provider_setup'].includes(command.action))) {
           send(peer, { type: 'result', id: command.id, ok: false, code: 'unsupported_remote_action' }); continue;
         }
         if (requests.size >= 256) { peer.destroy(); return; }
@@ -73,15 +75,23 @@ export async function createOwnerTransport({ root, nonce, allowedActions, dispat
         if (item) { requests.delete(frame.id); send(item.peer, { ...frame, id: item.id }); }
       } else if (frame.type === 'status' || frame.type === 'notify' && (!frame.hostId || frame.hostId === 'local')) {
         for (const peer of subscribers) send(peer, frame);
+        if (frame.type === 'notify') for (const peer of desktopSubscribers) send(peer, frame);
       }
     },
+    // Local native viewers need the same complete, scoped projection as the
+    // original window. Remote observers retain their local-only catalog.
+    publishDesktop(frame) {
+      if (frame.type !== 'status') return;
+      latestDesktop = frame;
+      for (const peer of desktopSubscribers) send(peer, frame);
+    },
     async close() { for (const peer of peers) peer.destroy(); await new Promise(resolve => server.close(resolve)); },
   };
 }
 
 // Returns false only when no compatible owner endpoint exists. Never starts a
 // second owner, kills a process, or converts a disconnect into an execution retry.
-export async function relayToOwner({ root, webOrigin, input = process.stdin, output = process.stdout }) {
+export async function relayToOwner({ root, webOrigin, input = process.stdin, output = process.stdout, desktopClient = false }) {
   let owner;
   try { owner = JSON.parse(await readFile(join(root, 'desktop-app.lock', 'owner.json'), 'utf8')); }
   catch (error) { if (error.code === 'ENOENT') return false; throw error; }
@@ -97,7 +107,11 @@ export async function relayToOwner({ root, webOrigin, input = process.stdin, out
     if (webOrigin && (!owner.webOrigin || normalizeWebOrigin(owner.webOrigin) !== normalizeWebOrigin(webOrigin)))
       throw Object.assign(new Error('The existing SIMY owner belongs to another Web address. Keep its tasks and reconnect using its saved environment.'), { code: 'remote_owner_origin_mismatch' });
   } catch (error) { peer.destroy(); throw error; }
-  peer.write(JSON.stringify({ nonce: owner.nonce }) + '\n');
+  peer.write(JSON.stringify({ nonce: owner.nonce, ...(desktopClient ? { client: 'desktop', desktopProtocol: 1 } : {}) }) + '\n');
+  // Old owners ignore unknown hello fields and expose only the restricted SSH
+  // projection. Never attach a native window or forward its controls as though
+  // that were a complete desktop endpoint. Keep the original owner untouched.
+  if (desktopClient) await confirmDesktopOwner(peer);
   peer.pipe(output, { end: false });
   const lines = createInterface({ input, crlfDelay: Infinity });
   lines.on('line', line => { if (line.length <= 16384 && !peer.destroyed) peer.write(line + '\n'); });
@@ -107,6 +121,34 @@ export async function relayToOwner({ root, webOrigin, input = process.stdin, out
   return true;
 }
 
+function confirmDesktopOwner(peer) {
+  return new Promise((resolve, reject) => {
+    const incompatible = () => Object.assign(new Error('The running SIMY owner does not support desktop window handoff.'), { code: 'desktop_owner_update_required' });
+    let buffer = Buffer.alloc(0);
+    const finish = error => {
+      clearTimeout(timer);
+      peer.removeListener('data', data); peer.removeListener('close', closed); peer.removeListener('error', failed);
+      if (error) { peer.destroy(); reject(error); } else resolve();
+    };
+    const closed = () => finish(incompatible());
+    const failed = error => finish(error);
+    const data = chunk => {
+      const newline = chunk.indexOf(10);
+      const line = newline < 0 ? chunk : chunk.subarray(0, newline);
+      if (buffer.length + line.length > 16384) { finish(incompatible()); return; }
+      buffer = Buffer.concat([buffer, line]);
+      if (newline < 0) return;
+      let frame; try { frame = JSON.parse(buffer.toString('utf8')); } catch { finish(incompatible()); return; }
+      if (frame?.type !== 'desktop_owner_ready' || frame.version !== 1) { finish(incompatible()); return; }
+      peer.pause();
+      if (newline + 1 < chunk.length) peer.unshift(chunk.subarray(newline + 1));
+      finish();
+    };
+    const timer = setTimeout(closed, 5000);
+    peer.on('data', data); peer.once('close', closed); peer.once('error', failed);
+  });
+}
+
 // A terminal command attaches to the existing execution owner. It never starts
 // another supervisor or retries a mutation after a lost response.
 export async function requestOwnerArchive({ root, command, signal, timeoutMs = 55_000 }) {

package/src/session-supervision/remote-errors.js

@@ -6,7 +6,7 @@ const messages = {
   claude_desktop_accessibility_required: 'Enable macOS Accessibility for the app running SIMY on the conversation host. Use simy sessions provider-setup --provider claude --host ALIAS --session UUID to open that host’s setting.',
   claude_desktop_ui_unavailable: 'Open the exact Claude Desktop Code conversation on the selected host and retry.',
   claude_desktop_ui_changed: 'The Claude Desktop window changed or is covered. Retry when the selected conversation is visible.',
-  claude_desktop_running: 'Stop this conversation and turn off automatic continuation before organizing it.',
+  claude_desktop_running: 'Stop this conversation and turn off Delivery Loop before organizing it.',
   claude_desktop_unsaved_input: 'Claude Desktop contains unsent input. Preserve it on the conversation host before retrying.',
   claude_desktop_identity_unverified: 'Claude Desktop conversation identity could not be verified. Refresh the exact conversation.',
   claude_desktop_identity_changed: 'Claude Desktop account or conversation changed. Refresh before retrying.',
@@ -24,7 +24,7 @@ const messages = {
   session_model_unavailable: 'Select a model in the original remote conversation, then refresh.',
   model_metadata_unavailable: 'Select a model in the original remote conversation, then refresh.',
   session_workspace_changed: 'The remote workspace changed. Refresh and select the conversation again.',
-  session_archived: 'Restore the original remote conversation before enabling Auto-continue.',
+  session_archived: 'Restore the original remote conversation before turning on Delivery Loop.',
   existing_loop: 'Use this remote conversation’s existing task controls.',
   task_stopped: 'This remote task was stopped. Review the original conversation and explicitly request resumption.',
   task_settled: 'The remote task has finished. Refresh and select its explicit follow-up controls.',

package/src/session-supervision/remote-hosts.js

@@ -194,10 +194,18 @@ export function createRemoteHostManager({ root, store, spawnImpl = spawn, onChan
             if (status.controllerEventsVersion === 1) void pullEvents(connection).catch(() => {});
             else for (const event of status.controllerEvents || []) void receiveNotification(connection, event)?.catch(() => {});
             const expectedScope = preferenceScope;
-            void preferences.change(expectedScope, state => {
+            connection.statusPersistence = preferences.change(expectedScope, state => {
               for (const row of status.sessionSupervision.sessions) {
                 const key = JSON.stringify([connection.id, row.provider, row.session]);
                 const saved = state.conversations[key] ||= {};
+                // A lost command response is not proof of a failed save. Only
+                // this newly received, authorized host frame may resolve it;
+                // cached/offline projections and a requested value cannot.
+                const auto = saved.policy?.session_set_auto_continue;
+                if (['unknown', 'confirming'].includes(auto?.state) &&
+                    typeof row.autoContinue === 'boolean' && row.autoContinue === auto.command.enabled) {
+                  auto.state = 'applied'; delete auto.code;
+                }
                 if (['ready','partial'].includes(row.statistics?.state)) saved.statistics = { ...row.statistics, observedAt: row.statistics.observedAt || now() };
                 if (!saved.importedRemoteDisplay) {
                   saved.pinned ??= row.listPinned === true;
@@ -209,7 +217,7 @@ export function createRemoteHostManager({ root, store, spawnImpl = spawn, onChan
               state.hosts ||= {};
               state.hosts[connection.id] = { sessionSupervision: status.sessionSupervision, deliveryHooks: status.deliveryHooks,
                 controllerCompletions: [...new Map([...(state.hosts[connection.id]?.controllerCompletions || []), ...(status.controllerCompletions || [])].map(item => [JSON.stringify([item.provider,item.provider_session_id,item.task_id]),item])).values()], tasks: status.tasks, sqmTasks: status.sqmTasks };
-            }).catch(() => {});
+            }).then(changed).catch(() => {});
           }
           changed();
         } else if (frame.type === 'notify' && connection.ready && local && typeof frame.id === 'string' && ['finished','failed','paused','needs_decision'].includes(frame.kind)) {
@@ -508,6 +516,7 @@ export function createRemoteHostManager({ root, store, spawnImpl = spawn, onChan
     connection.flushing = true;
     const expectedScope = preferenceScope;
     try {
+      await connection.statusPersistence;
       const state = await preferences.load(expectedScope);
       const pending = Object.entries(state.conversations).flatMap(([key, row]) => {
         if (JSON.parse(key)[0] !== connection.id) return [];
@@ -536,7 +545,9 @@ export function createRemoteHostManager({ root, store, spawnImpl = spawn, onChan
         await preferences.change(expectedScope, data => {
           const current = data.conversations[key]?.policy?.[item.command.action];
           if (current?.command.id === item.command.id) {
-            current.state = result.nextState || (result.ok ? 'applied' : result.code === 'remote_operation_unknown' ? 'unknown' : 'failed');
+            current.state = result.nextState || (result.ok
+              ? item.command.action === 'session_set_auto_continue' ? 'confirming' : 'applied'
+              : result.code === 'remote_operation_unknown' ? 'unknown' : 'failed');
             current.code = result.code;
           }
         });

package/src/session-supervision/work-state.js

@@ -99,14 +99,14 @@ export function sessionWorkState(candidate, run, { now = Date.now(), provider }
   // Saved consent is not execution: name the real dispatch blocker rather than
   // presenting a fenced conversation as waiting for its AI.
   if (candidate.autoContinue === true && candidate.dispatchReadiness?.state === 'blocked' && !['completed', 'stopped'].includes(run?.state))
-    return result('blocked', '自動継続を送信できません', candidate.autoContinueReason || '自動継続はオンですが、作業を送信できません。',
+    return result('blocked', 'Delivery Loop の作業を送信できません', candidate.autoContinueReason || 'Delivery Loop はオンですが、作業を送信できません。',
       candidate.dispatchReadiness.code === 'update_recovery_required' ? 'update_reconnect' : candidate.dispatchReadiness.code === 'writer_reconnect_required' ? 'reconnect_original' : 'open_original', true);
   if (run?.state === 'verification_pending' &&
       (onlyManualAcceptancePending(run) || candidate.autoContinueStatus === 'needs_user'))
     return result('unfinished', '残りの作業あり', '実際の結果の確認が残っています。', 'open_original', true);
   if (candidate.autoContinue === true && ['starting', 'recovering'].includes(candidate.autoContinueStatus))
-    return result(candidate.autoContinueStatus, candidate.autoContinueStatus === 'starting' ? '自動継続を準備中' : '自動復旧中',
-      candidate.autoContinueReason || '自動継続はオンで保存されています。');
+    return result(candidate.autoContinueStatus, candidate.autoContinueStatus === 'starting' ? 'Delivery Loop を準備中' : '自動復旧中',
+      candidate.autoContinueReason || 'Delivery Loop はオンで保存されています。');
   if ((run?.autoRecovery || run?.managed?.autoRecovery) && ['recovery_required', 'auto_resuming'].includes(run.state)) return result('checking', 'AIが状況確認中', '自動復旧の結果を待っています。');
   if (candidate.autoContinue === true && candidate.autoContinueStatus === 'supervising' && run?.state === 'verification_pending')
     return result('checking', '結果を確認中', '作業の結果を自動で確認しています。');

package/src/sqm/agent-skill.js

@@ -8,7 +8,7 @@ import { hookRuntimeCommand } from "../hook-runtime.js";
 import { inspectClaudePlugin, installClaudePlugin } from "./claude-plugin.js";
 
 export const SQM_AGENT_SKILL_ID = "simy-sqm-pr-check";
-export const SQM_AGENT_SKILL_VERSION = "3.4.0";
+export const SQM_AGENT_SKILL_VERSION = "3.5.0";
 export const SIMY_CODEX_PLUGIN_ID = "simy";
 export const SIMY_CODEX_PLUGIN_VERSION = "0.1.0";
 
@@ -22,10 +22,10 @@ const LEGACY_SKILL_DIGEST = "c9ab46e855280196dbc93fd36ba849418e0c4e500dd6e4b7b03
 
 const SKILL_CONTENT = `---
 name: ${SQM_AGENT_SKILL_ID}
-description: Run SIMY Software Quality Monitor checks for PR work when the user asks @simy for a quality check, invokes this SQM skill, or asks to use SQM. Conversation continuation uses the separate continue entry. Ordinary PR creation or git push alone does not activate this skill.
+description: Apply a SIMY SQM check as a gate before creating or proposing a pull request, only after the user opted in with @simy, this SQM skill, or a request to use SQM for the task. An immediate SQM check of the current work uses the plugin sqm-loop entry; Delivery Loop uses the separate continue entry. Ordinary PR creation or git push alone does not activate this skill.
 ---
 
-# SIMY SQM PR check
+# SIMY SQM check before a PR
 
 Apply this workflow only when the user has explicitly requested quality checks with \`@simy\`, invoked this SQM skill, or requested SQM for the current task. A request to start or continue the original task uses SIMY's \`continue\` entry; selecting that entry does not request SQM. A quoted mention, repository organization, installed plugin, or ordinary request to create a PR is not opt-in. Honor a later request to stop using SQM.
 

package/src/sqm/claude-plugin.js

@@ -97,7 +97,7 @@ export async function claudePluginFiles() {
   for (const [name, contents] of [...files].sort(([a], [b]) => a.localeCompare(b))) hash.update(name).update("\0").update(contents);
   const manifest = {
     name: "simy", version: `${sourceManifest.version.split("+")[0]}+claude.${hash.digest("hex").slice(0, 20)}`,
-    description: "SIMY Auto Continue through the original conversation controller, quality checks, and PR review for Claude Code.",
+    description: "SIMY Delivery Loop through the original conversation controller, SQM checks, and PR review for Claude Code.",
     author: sourceManifest.author, repository: sourceManifest.repository,
     license: sourceManifest.license, keywords: sourceManifest.keywords,
   };