ticlawk 0.1.16-dev.1 → 0.1.16-dev.3

package/src/runtimes/openclaw/index.mjs

@@ -2,7 +2,27 @@ import { normalizeOutboundMedia } from '../../core/media/outbound.mjs';
 import { reportSubprocessFailure, sendAdapterMessage, recordActivity, terminalRuntimeFailure } from '../../core/runtime-support.mjs';
 import { buildStandingPrompt } from '../_shared/standing-prompt.mjs';
 import { GATEWAY_HOST, GATEWAY_PORT } from './identity.mjs';
-import { buildOpenClawSessionKey, normalizeOpenClawAgentId, resolveOpenClawWorkspace } from './target.mjs';
+
+// Cheap availability probe used by the harness picker. We can't use
+// isGatewayReady() here because the daemon only opens the gateway WS
+// after at least one openclaw binding exists (registerOpenClawChannel),
+// so an empty install would always look "unavailable" and you'd never
+// be able to pick it for the first time. Probing the gateway's plain
+// HTTP /health avoids that chicken-and-egg.
+const OPENCLAW_HEALTH_TIMEOUT_MS = 1500;
+async function probeOpenClawGatewayHealth() {
+  try {
+    const res = await fetch(`http://${GATEWAY_HOST}:${GATEWAY_PORT}/health`, {
+      signal: AbortSignal.timeout(OPENCLAW_HEALTH_TIMEOUT_MS),
+    });
+    if (!res.ok) return false;
+    const body = await res.json().catch(() => null);
+    return body?.ok === true;
+  } catch {
+    return false;
+  }
+}
+import { buildOpenClawSessionKey, normalizeOpenClawAgentId } from './target.mjs';
 import { askGateway, isGatewayReady, registerOpenClawChannel } from './gateway.mjs';
 
 // Tracks which (agentId, sessionKey) pairs already saw the standing
@@ -87,18 +107,24 @@ export const openClawRuntime = {
     return isGatewayReady();
   },
 
+  // Health probe for the harness picker. openclaw is gateway-based —
+  // "available" means the local openclaw gateway is up and responds to
+  // an unauthenticated /health request. Doesn't depend on the daemon
+  // having an open WS, so the picker can offer openclaw on a fresh
+  // install where no binding exists yet.
+  async health() {
+    return {
+      available: await probeOpenClawGatewayHealth(),
+      path: null,
+      version: null,
+    };
+  },
+
   async resolveBinding(payload) {
     if (!payload?.agentId) {
       throw new Error('agentId is required for openclaw binding');
     }
     const agentId = normalizeOpenClawAgentId(payload.agentId);
-    const workspace = String(
-      payload.workdir
-      || payload.projectDir
-      || payload.cwd
-      || resolveOpenClawWorkspace(agentId)
-      || '',
-    ).trim();
     const sessionKey = payload.sessionKey
       ? String(payload.sessionKey).trim()
       : buildOpenClawSessionKey(agentId);
@@ -110,11 +136,6 @@ export const openClawRuntime = {
         sessionKey,
         gatewayHost: GATEWAY_HOST,
         gatewayPort: GATEWAY_PORT,
-        ...(workspace ? {
-          workdir: workspace,
-          projectDir: workspace,
-          cwd: workspace,
-        } : {}),
       },
     };
   },

package/src/runtimes/openclaw/target.mjs

@@ -5,42 +5,12 @@
  * `agent:<id>:main` session key is derived internally when dispatching.
  */
 
-import { existsSync, readFileSync } from 'node:fs';
-import { join } from 'node:path';
-
-let cachedOpenClawConfig = null;
-
-function loadOpenClawConfig() {
-  if (cachedOpenClawConfig !== null) return cachedOpenClawConfig;
-  const home = process.env.HOME || '';
-  const configPath = home ? join(home, '.openclaw', 'openclaw.json') : '';
-  if (!configPath || !existsSync(configPath)) {
-    cachedOpenClawConfig = {};
-    return cachedOpenClawConfig;
-  }
-  try {
-    cachedOpenClawConfig = JSON.parse(readFileSync(configPath, 'utf8')) || {};
-  } catch {
-    cachedOpenClawConfig = {};
-  }
-  return cachedOpenClawConfig;
-}
-
 export function normalizeOpenClawAgentId(value) {
   const trimmed = String(value || '').trim().toLowerCase();
   if (!trimmed) return 'main';
   return trimmed.replace(/[^a-z0-9_-]+/g, '-').replace(/^-+|-+$/g, '') || 'main';
 }
 
-export function resolveOpenClawWorkspace(agentId) {
-  const normalizedId = normalizeOpenClawAgentId(agentId);
-  const config = loadOpenClawConfig();
-  const agents = Array.isArray(config?.agents?.list) ? config.agents.list : [];
-  const matched = agents.find((agent) => normalizeOpenClawAgentId(agent?.id || agent?.name || '') === normalizedId);
-  const workspace = String(matched?.workspace || '').trim();
-  return workspace || '';
-}
-
 export function buildOpenClawSessionKey(agentId = 'main') {
   return `agent:${normalizeOpenClawAgentId(agentId)}:main`;
 }

package/src/runtimes/opencode/index.mjs

@@ -11,6 +11,7 @@ import { existsSync } from 'node:fs';
 import { basename } from 'node:path';
 import { buildAgentRuntimeEnv } from '../../core/runtime-env.mjs';
 import { buildStandingPrompt } from '../_shared/standing-prompt.mjs';
+import { ensureAgentHome } from '../../core/agent-home.mjs';
 import {
   createOpenCodeSession,
   getOpenCodeRuntimeHealth,
@@ -97,8 +98,6 @@ export const openCodeRuntime = {
         displayName: payload.name || session.title || basename(requestedCwd) || 'opencode',
         runtimeMeta: {
           sessionId: session.sessionId,
-          workdir: requestedCwd,
-          cwd: requestedCwd,
           runtimePath: opencodePath,
           opencodePath,
           opencodeVersion,
@@ -107,20 +106,11 @@ export const openCodeRuntime = {
       };
     }
 
-    if (!requestedCwd) {
-      throw new Error('cwd or sessionId is required for opencode binding');
-    }
-    if (!existsSync(requestedCwd)) {
-      throw new Error(`opencode cwd not found locally: ${requestedCwd}`);
-    }
-
     return {
       runtime: this.name,
-      displayName: payload.name || basename(requestedCwd) || 'opencode',
+      displayName: payload.name || (requestedCwd ? basename(requestedCwd) : 'opencode'),
       runtimeMeta: {
         sessionId: null,
-        workdir: requestedCwd,
-        cwd: requestedCwd,
         runtimePath: opencodePath,
         opencodePath,
         opencodeVersion,
@@ -139,16 +129,9 @@ export const openCodeRuntime = {
     const adapter = ctx.adapter;
     const meta = binding.runtimeMeta || {};
     const runtimeOpenCodePath = meta.opencodePath || meta.runtimePath || null;
-
-    if (!meta.cwd || !existsSync(meta.cwd)) {
-      await sendAdapterMessage(adapter, binding, {
-        type: 'assistant',
-        text: `⚠️ opencode cwd not found: ${meta.cwd || '(missing)'}`,
-        media: [],
-        replyToMessageId: inbound.messageId || null,
-      });
-      return true;
-    }
+    const agentHome = ensureAgentHome(binding.id, {
+      displayName: binding.display_name || binding.name || null,
+    });
 
     // For image inbound, resolve the attached media to local file paths
     // and forward them to opencode via `--file` (mirrors how Codex uses
@@ -195,7 +178,7 @@ export const openCodeRuntime = {
           binding,
           agent: this.name,
           sessionId: sessionId || meta.sessionId || binding.id,
-          cwd: cwd || meta.cwd,
+          cwd: cwd || agentHome,
           replyToMessageId: inbound.messageId || null,
           event: {
             hook_event_name: 'worker.message.delta',
@@ -217,7 +200,7 @@ export const openCodeRuntime = {
       });
       const standingPrompt = buildStandingPrompt({ agentId: binding.id });
       const result = shouldStreamRuntime(this.name, this)
-        ? await this.runTurnStream({ sessionId: shouldRotate ? null : meta.sessionId, cwd: meta.cwd, opencodePath, agentEnv }, message, {
+        ? await this.runTurnStream({ sessionId: shouldRotate ? null : meta.sessionId, cwd: agentHome, opencodePath, agentEnv }, message, {
             standingPrompt,
             files,
             onEvent: async (event) => {
@@ -227,7 +210,7 @@ export const openCodeRuntime = {
                   binding,
                   agent: this.name,
                   sessionId: event.sessionId || meta.sessionId || binding.id,
-                  cwd: meta.cwd,
+                  cwd: agentHome,
                   replyToMessageId: inbound.messageId || null,
                   event: {
                     hook_event_name: 'worker.turn.start',
@@ -238,18 +221,17 @@ export const openCodeRuntime = {
               } else if (event?.type === 'message.delta' && event.text) {
                 deltaAggregator.push(event.text, {
                   sessionId: event.sessionId || meta.sessionId || binding.id,
-                  cwd: meta.cwd,
+                  cwd: agentHome,
                 });
               }
             },
           })
-        : await this.runTurn({ sessionId: shouldRotate ? null : meta.sessionId, cwd: meta.cwd, opencodePath, agentEnv }, message, { files, standingPrompt });
+        : await this.runTurn({ sessionId: shouldRotate ? null : meta.sessionId, cwd: agentHome, opencodePath, agentEnv }, message, { files, standingPrompt });
 
       await deltaAggregator.flush();
 
       const nextBinding = await updateBindingRuntimeMeta(ctx, binding, {
         sessionId: result?.sessionId || meta.sessionId,
-        cwd: result?.cwd || meta.cwd,
         runtimePath: opencodePath,
         opencodePath,
         opencodeVersion,
@@ -265,7 +247,7 @@ export const openCodeRuntime = {
         binding: nextBinding,
         agent: this.name,
         sessionId: result?.sessionId || meta.sessionId || binding.id,
-        cwd: result?.cwd || meta.cwd,
+        cwd: result?.cwd || agentHome,
         replyToMessageId: inbound.messageId || null,
         event: {
           hook_event_name: 'Stop',
@@ -281,7 +263,7 @@ export const openCodeRuntime = {
         binding,
         agent: this.name,
         sessionId: meta.sessionId || binding.id,
-        cwd: meta.cwd,
+        cwd: agentHome,
         replyToMessageId: inbound.messageId || null,
         event: {
           hook_event_name: 'worker.turn.error',
@@ -313,7 +295,7 @@ export const openCodeRuntime = {
       binding,
       agent: this.name,
       sessionId: meta.sessionId || binding.id,
-      cwd: meta.cwd || '',
+      cwd: ensureAgentHome(binding.id) || '',
       event: {
         hook_event_name: 'Stop',
         worker_event_name: 'worker.turn.complete',

package/src/runtimes/pi/index.mjs

@@ -4,10 +4,10 @@
  * Wraps the pi CLI RPC mode and exposes the ticlawk runtime contract.
  */
 
-import { existsSync } from 'node:fs';
 import { basename } from 'node:path';
 import { buildAgentRuntimeEnv } from '../../core/runtime-env.mjs';
 import { buildStandingPrompt } from '../_shared/standing-prompt.mjs';
+import { ensureAgentHome } from '../../core/agent-home.mjs';
 import {
   buildPiImagesFromInbound,
   discoverPiSessions,
@@ -80,8 +80,6 @@ export const piRuntime = {
         displayName: payload.name || basename(session.cwd || requestedCwd) || 'pi',
         runtimeMeta: {
           sessionId: session.sessionId,
-          workdir: session.cwd || requestedCwd,
-          cwd: session.cwd || requestedCwd,
           path: session.path || null,
           runtimePath: piPath,
           piPath,
@@ -91,20 +89,11 @@ export const piRuntime = {
       };
     }
 
-    if (!requestedCwd) {
-      throw new Error('cwd or sessionId is required for pi binding');
-    }
-    if (!existsSync(requestedCwd)) {
-      throw new Error(`pi cwd not found locally: ${requestedCwd}`);
-    }
-
     return {
       runtime: this.name,
-      displayName: payload.name || basename(requestedCwd) || 'pi',
+      displayName: payload.name || (requestedCwd ? basename(requestedCwd) : 'pi'),
       runtimeMeta: {
         sessionId: null,
-        workdir: requestedCwd,
-        cwd: requestedCwd,
         path: null,
         runtimePath: piPath,
         piPath,
@@ -123,16 +112,9 @@ export const piRuntime = {
     if (!binding) return false;
     const adapter = ctx.adapter;
     const meta = binding.runtimeMeta || {};
-
-    if (!meta.cwd || !existsSync(meta.cwd)) {
-      await sendAdapterMessage(adapter, binding, {
-        type: 'assistant',
-        text: `⚠️ pi cwd not found: ${meta.cwd || '(missing)'}`,
-        media: [],
-        replyToMessageId: inbound.messageId || null,
-      });
-      return true;
-    }
+    const agentHome = ensureAgentHome(binding.id, {
+      displayName: binding.display_name || binding.name || null,
+    });
 
     let images = [];
     let message = inbound.text || '';
@@ -167,7 +149,7 @@ export const piRuntime = {
           binding,
           agent: this.name,
           sessionId: sessionId || meta.sessionId || binding.id,
-          cwd: cwd || meta.cwd,
+          cwd: cwd || agentHome,
           replyToMessageId: inbound.messageId || null,
           event: {
             hook_event_name: 'worker.message.delta',
@@ -189,7 +171,7 @@ export const piRuntime = {
       });
       const standingPrompt = buildStandingPrompt({ agentId: binding.id });
       const result = shouldStreamRuntime(this.name, this)
-        ? await this.runTurnStream({ sessionId: shouldRotate ? null : meta.sessionId, cwd: meta.cwd, piPath: runtimePiPath, agentEnv }, message, {
+        ? await this.runTurnStream({ sessionId: shouldRotate ? null : meta.sessionId, cwd: agentHome, piPath: runtimePiPath, agentEnv }, message, {
             standingPrompt,
             images,
             onEvent: async (event) => {
@@ -199,7 +181,7 @@ export const piRuntime = {
                   binding,
                   agent: this.name,
                   sessionId: event.sessionId || meta.sessionId || binding.id,
-                  cwd: meta.cwd,
+                  cwd: agentHome,
                   replyToMessageId: inbound.messageId || null,
                   event: {
                     hook_event_name: 'worker.turn.start',
@@ -210,17 +192,16 @@ export const piRuntime = {
               } else if (event?.type === 'message.delta' && event.text) {
                 deltaAggregator.push(event.text, {
                   sessionId: event.sessionId || meta.sessionId || binding.id,
-                  cwd: meta.cwd,
+                  cwd: agentHome,
                 });
               }
             },
           })
-        : await this.runTurn({ sessionId: shouldRotate ? null : meta.sessionId, cwd: meta.cwd, piPath: runtimePiPath, agentEnv }, message, { images, standingPrompt });
+        : await this.runTurn({ sessionId: shouldRotate ? null : meta.sessionId, cwd: agentHome, piPath: runtimePiPath, agentEnv }, message, { images, standingPrompt });
 
       await deltaAggregator.flush();
       const nextBinding = await updateBindingRuntimeMeta(ctx, binding, {
         sessionId: result?.sessionId || meta.sessionId,
-        cwd: result?.cwd || meta.cwd,
         path: result?.path || meta.path || null,
         runtimePath: runtimePiPath,
         piPath: runtimePiPath,
@@ -234,7 +215,7 @@ export const piRuntime = {
         binding: nextBinding,
         agent: this.name,
         sessionId: result?.sessionId || meta.sessionId || binding.id,
-        cwd: result?.cwd || meta.cwd,
+        cwd: result?.cwd || agentHome,
         replyToMessageId: inbound.messageId || null,
         event: {
           hook_event_name: 'Stop',
@@ -250,7 +231,7 @@ export const piRuntime = {
         binding,
         agent: this.name,
         sessionId: meta.sessionId || binding.id,
-        cwd: meta.cwd,
+        cwd: agentHome,
         replyToMessageId: inbound.messageId || null,
         event: {
           hook_event_name: 'worker.turn.error',
@@ -282,7 +263,7 @@ export const piRuntime = {
       binding,
       agent: this.name,
       sessionId: meta.sessionId || binding.id,
-      cwd: meta.cwd || '',
+      cwd: ensureAgentHome(binding.id) || '',
       event: {
         hook_event_name: 'Stop',
         worker_event_name: 'worker.turn.complete',

package/ticlawk.mjs

@@ -21,6 +21,7 @@ import {
   persistConfig,
 } from './src/core/config.mjs';
 import { startLocalHttpServer } from './src/core/http.mjs';
+import { startReminderTicker } from './src/core/reminder-ticker.mjs';
 import { installProcessDiagnostics } from './src/core/diagnostics.mjs';
 import * as logger from './src/core/logger.mjs';
 import { Bus } from './src/core/bus.mjs';
@@ -30,6 +31,28 @@ import { buildRuntimeContext, normalizeServiceType } from './src/core/runtime-re
 import { belongsToRuntimeHost, getBindingRuntimeHostId, getHostId } from './src/core/host-id.mjs';
 import { readPkgVersion } from './src/core/update.mjs';
 import { buildImageMessageFromInbound } from './src/core/media/inbound.mjs';
+import { existsSync as _fsExists, unlinkSync as _fsUnlink } from 'node:fs';
+import { join as _pathJoin } from 'node:path';
+import { getActiveProfile } from './src/core/profiles.mjs';
+
+// Pre-profile builds wrote bindings to ~/.ticlawk/bindings.json. The
+// profile flow now owns binding persistence under
+// ~/.ticlawk/profiles/<adapter>/<userId>/bindings.json. The root file
+// hasn't had a writer for a while but lingers as stale state. Prune
+// once per daemon start so it doesn't confuse `jq` audits later.
+function pruneLegacyRootBindings() {
+  if (!getActiveProfile()) return;
+  const rootPath = _pathJoin(AF_HOME, 'bindings.json');
+  if (!_fsExists(rootPath)) return;
+  try {
+    _fsUnlink(rootPath);
+    logger.debugLog?.('startup', 'legacy-root-bindings.pruned', { path: rootPath });
+  } catch (err) {
+    logger.debugError?.('startup', 'legacy-root-bindings.prune-failed', {
+      path: rootPath, error: err?.message || String(err),
+    });
+  }
+}
 
 // Re-export the config-owned paths so local tooling can inspect the
 // ticlawk home/config/log locations without reaching into src/.
@@ -80,7 +103,7 @@ function createUpsertBindingWithSync(runtimes, adapter) {
   };
 }
 
-function createCacheBinding(runtimes, getAdapter) {
+function createPersistBinding(runtimes, getAdapter) {
   return async (binding) => {
     const nextBinding = await upsertBinding(binding);
     if (!belongsToRuntimeHost(nextBinding)) {
@@ -101,13 +124,13 @@ function createCacheBinding(runtimes, getAdapter) {
   };
 }
 
-function createBaseRuntimeCtx(runtimes, cacheBinding, upsertBindingWithSync) {
+function createBaseRuntimeCtx(runtimes, persistBinding, upsertBindingWithSync) {
   return {
     runtimes,
     getBinding,
     listBindings,
     deleteBinding,
-    cacheBinding,
+    persistBinding,
     upsertBinding: upsertBindingWithSync,
     buildImageMessageFromInbound,
     logger,
@@ -221,11 +244,12 @@ export async function startTiclawk() {
   if (started) return;
   started = true;
   installProcessDiagnostics();
+  pruneLegacyRootBindings();
 
   const { runtimeList, runtimes } = await buildRuntimeContext();
   const resolveRuntimeBinding = createResolveRuntimeBinding(runtimes);
   let adapter;
-  const cacheBinding = createCacheBinding(runtimes, () => adapter);
+  const persistBinding = createPersistBinding(runtimes, () => adapter);
   let baseRuntimeCtx;
   let syncBinding = async (binding) => {
     if (!adapter) {
@@ -233,13 +257,13 @@ export async function startTiclawk() {
     }
     return upsertBinding(binding);
   };
-  baseRuntimeCtx = createBaseRuntimeCtx(runtimes, cacheBinding, (binding) => syncBinding(binding));
+  baseRuntimeCtx = createBaseRuntimeCtx(runtimes, persistBinding, (binding) => syncBinding(binding));
   adapter = createAdapter(
     'ticlawk',
     createAdapterContext(baseRuntimeCtx, resolveRuntimeBinding)
   );
   syncBinding = createUpsertBindingWithSync(runtimes, adapter);
-  baseRuntimeCtx = createBaseRuntimeCtx(runtimes, cacheBinding, (binding) => syncBinding(binding));
+  baseRuntimeCtx = createBaseRuntimeCtx(runtimes, persistBinding, (binding) => syncBinding(binding));
 
   printBanner(adapter);
   if (typeof adapter.refreshBindings === 'function') {
@@ -252,6 +276,7 @@ export async function startTiclawk() {
     adapter,
     ctx: { listBindings, getBinding },
   });
+  startReminderTicker();
   await recoverAllRuntimes(runtimeList, adapter);
   await reconcileBindingsAfterRestart(runtimes, adapter);
   await adapter.start();