ticlawk 0.1.17-dev.2 → 0.1.17-dev.20

package/scripts/publish-dev.sh

@@ -0,0 +1,77 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+ROOT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+cd "$ROOT_DIR"
+
+EXPECTED_VERSION=""
+DRY_RUN=false
+
+while [[ $# -gt 0 ]]; do
+  case "$1" in
+    --expected-version)
+      EXPECTED_VERSION="${2:-}"
+      if [[ -z "$EXPECTED_VERSION" ]]; then
+        echo "--expected-version requires a value" >&2
+        exit 2
+      fi
+      shift 2
+      ;;
+    --dry-run)
+      DRY_RUN=true
+      shift
+      ;;
+    *)
+      echo "unknown argument: $1" >&2
+      echo "usage: npm run publish:dev -- [--expected-version X.Y.Z-dev.N] [--dry-run]" >&2
+      exit 2
+      ;;
+  esac
+done
+
+VERSION="$(node -p "JSON.parse(require('fs').readFileSync('package.json', 'utf8')).version")"
+
+if [[ -n "$EXPECTED_VERSION" && "$EXPECTED_VERSION" != "$VERSION" ]]; then
+  echo "expected version ($EXPECTED_VERSION) does not match cli/package.json ($VERSION)" >&2
+  exit 1
+fi
+
+if [[ "$VERSION" != *-* ]]; then
+  echo "refusing to publish stable version $VERSION with the dev script" >&2
+  exit 1
+fi
+
+NPM_ARGS=(--registry https://registry.npmjs.org/)
+TMP_NPMRC=""
+if [[ -n "${NPM_TOKEN:-}" ]]; then
+  TMP_NPMRC="$(mktemp)"
+  chmod 0600 "$TMP_NPMRC"
+  printf '//registry.npmjs.org/:_authToken=%s\n' "$NPM_TOKEN" > "$TMP_NPMRC"
+  NPM_ARGS+=(--userconfig "$TMP_NPMRC")
+fi
+
+cleanup() {
+  if [[ -n "$TMP_NPMRC" ]]; then
+    rm -f "$TMP_NPMRC"
+  fi
+}
+trap cleanup EXIT
+
+if npm "${NPM_ARGS[@]}" view "ticlawk@$VERSION" version >/dev/null 2>&1; then
+  echo "ticlawk@$VERSION is already published" >&2
+  exit 1
+fi
+
+if ! npm "${NPM_ARGS[@]}" whoami >/dev/null; then
+  echo "npm auth failed; run npm login or set a valid NPM_TOKEN with publish access to ticlawk" >&2
+  exit 1
+fi
+
+npm run verify:public-package
+
+PUBLISH_ARGS=(publish --access public --tag dev)
+if [[ "$DRY_RUN" == "true" ]]; then
+  PUBLISH_ARGS+=(--dry-run)
+fi
+
+npm "${NPM_ARGS[@]}" "${PUBLISH_ARGS[@]}"

package/src/adapters/ticlawk/api.mjs

@@ -30,9 +30,8 @@ export function getConnectorWsUrl() {
   return DEFAULT_CONNECTOR_WS_URL;
 }
 
-// Non-terminal event writes are serialized per agent for stable logs.
-// Terminal turn events bypass this queue so telemetry cannot hold up the
-// delivery lifecycle.
+// Agent event writes must preserve per-agent order while still allowing
+// different agents to proceed concurrently.
 const agentEventQueues = new Map(); // agentId -> Promise
 
 export class TiclawkUpdateRequiredError extends Error {
@@ -152,13 +151,13 @@ export async function updateChannel(id, updates) {
 
 // ── Deliveries (replaces legacy message_jobs poll/ack) ──
 
-export async function claimPendingDeliveries(hostId, limit = 5, excludedChannels = []) {
+export async function claimPendingDeliveries(hostId, limit = 5, excludedAgentIds = []) {
   const { data } = await apiFetch('/api/agent/deliveries/claim-pending', {
     method: 'POST',
     body: JSON.stringify(withTiclawkVersion({
       runtime_host_id: hostId,
       limit,
-      excluded_channels: excludedChannels,
+      excluded_agent_ids: excludedAgentIds,
     })),
   });
   return data || [];
@@ -189,7 +188,6 @@ export async function sendAgentMessage({
   runtimeHostId,
   visibility,
   mediaAssetIds,
-  metadata,
 }) {
   const { data } = await apiFetch('/api/agent/messages/send', {
     method: 'POST',
@@ -202,7 +200,6 @@ export async function sendAgentMessage({
       runtime_host_id: runtimeHostId ?? null,
       visibility: visibility || null,
       media_asset_ids: Array.isArray(mediaAssetIds) && mediaAssetIds.length > 0 ? mediaAssetIds : undefined,
-      metadata: metadata ?? undefined,
     }),
   });
   return data || null;
@@ -225,7 +222,7 @@ export async function readAgentMessages({
   return data || [];
 }
 
-export async function createAgentTask({ actingAgentId, conversationId, text, title, assignAgentId }) {
+export async function createAgentTask({ actingAgentId, conversationId, text, title }) {
   return apiFetch('/api/agent/tasks/create', {
     method: 'POST',
     body: JSON.stringify({
@@ -233,7 +230,6 @@ export async function createAgentTask({ actingAgentId, conversationId, text, tit
       conversation_id: conversationId,
       text,
       title: title ?? null,
-      assign_agent_id: assignAgentId ?? null,
     }),
   });
 }
@@ -286,71 +282,6 @@ export async function updateAgentTask({ actingAgentId, taskId, status }) {
   });
 }
 
-export async function reportGoalTransition({
-  actingAgentId, conversationId, transitionId, outcome, detail, currentTaskId,
-}) {
-  return apiFetch('/api/agent/goal/report', {
-    method: 'POST',
-    body: JSON.stringify({
-      acting_as_agent_id: actingAgentId,
-      conversation_id: conversationId,
-      transition_id: transitionId,
-      outcome,
-      detail: detail ?? null,
-      current_task_id: currentTaskId ?? null,
-    }),
-  });
-}
-
-export async function noteGoalChanged({ actingAgentId, conversationId }) {
-  return apiFetch('/api/agent/goal/changed', {
-    method: 'POST',
-    body: JSON.stringify({
-      acting_as_agent_id: actingAgentId,
-      conversation_id: conversationId,
-    }),
-  });
-}
-
-export async function requestGoalApproval({
-  actingAgentId, conversationId, title, detail, ttlSeconds,
-}) {
-  return apiFetch('/api/agent/approval/request', {
-    method: 'POST',
-    body: JSON.stringify({
-      acting_as_agent_id: actingAgentId,
-      conversation_id: conversationId,
-      title,
-      detail: detail ?? null,
-      ttl_seconds: ttlSeconds ?? null,
-    }),
-  });
-}
-
-export async function listGoalApprovals({ actingAgentId, conversationId }) {
-  const params = new URLSearchParams();
-  params.set('acting_as_agent_id', actingAgentId);
-  params.set('conversation_id', conversationId);
-  const { data } = await apiFetch(`/api/agent/approval/list?${params}`);
-  return data || [];
-}
-
-export async function resolveGoalApproval({
-  actingAgentId, requestId, decision, originalText, confidence, sourceMessageId,
-}) {
-  return apiFetch('/api/agent/approval/resolve', {
-    method: 'POST',
-    body: JSON.stringify({
-      acting_as_agent_id: actingAgentId,
-      request_id: requestId,
-      decision,
-      original_text: originalText ?? null,
-      confidence: confidence ?? null,
-      source_message_id: sourceMessageId ?? null,
-    }),
-  });
-}
-
 export async function listAgentTasks({ actingAgentId, conversationId }) {
   const params = new URLSearchParams();
   params.set('acting_as_agent_id', actingAgentId);
@@ -553,294 +484,63 @@ export async function removeAgentGroupMember({
   );
 }
 
-// ── Workstreams (managed groups) ──
-
-export async function createWorkstream({
-  actingAgentId, name, description, charter, memberAgentIds,
-}) {
-  return apiFetch('/api/agent/workstreams', {
-    method: 'POST',
-    body: JSON.stringify({
-      acting_as_agent_id: actingAgentId,
-      name,
-      description: description ?? null,
-      charter: charter ?? null,
-      member_agent_ids: memberAgentIds || [],
-    }),
-  });
-}
-
-export async function deleteWorkstream({ actingAgentId, conversationId }) {
-  return apiFetch(
-    `/api/agent/workstreams/${encodeURIComponent(conversationId)}`,
-    {
-      method: 'DELETE',
-      body: JSON.stringify({ acting_as_agent_id: actingAgentId }),
-    },
-  );
-}
-
-export async function listWorkstreams({ actingAgentId }) {
-  const params = new URLSearchParams();
-  params.set('acting_as_agent_id', actingAgentId);
-  const { data } = await apiFetch(`/api/agent/workstreams?${params}`);
-  return data || [];
-}
-
-// ── Agents ──
-
-export async function listAgentSlots({ actingAgentId }) {
-  const params = new URLSearchParams();
-  params.set('acting_as_agent_id', actingAgentId);
-  const { data } = await apiFetch(`/api/agent/agents?${params}`);
-  return data || [];
-}
-
-export async function createAgentSlot({
-  actingAgentId, name, runtime, description, displayName, model,
-}) {
-  return apiFetch('/api/agent/agents', {
-    method: 'POST',
-    body: JSON.stringify({
-      acting_as_agent_id: actingAgentId,
-      name,
-      runtime,
-      description: description ?? null,
-      display_name: displayName ?? null,
-      model: model ?? null,
-    }),
-  });
-}
-
-export async function archiveAgentSlot({ actingAgentId, agentId }) {
-  return apiFetch(
-    `/api/agent/agents/${encodeURIComponent(agentId)}`,
-    {
-      method: 'DELETE',
-      body: JSON.stringify({ acting_as_agent_id: actingAgentId }),
-    },
-  );
-}
-
-// ── Services ──
-
-export async function createService({
-  actingAgentId, name, description, contractSchema, endpointConfig,
-}) {
-  return apiFetch('/api/agent/services', {
-    method: 'POST',
-    body: JSON.stringify({
-      acting_as_agent_id: actingAgentId,
-      name,
-      description: description ?? null,
-      contract_schema: contractSchema ?? null,
-      endpoint_config: endpointConfig,
-    }),
-  });
-}
-
-export async function updateService({ actingAgentId, serviceId, ...patch }) {
-  return apiFetch(
-    `/api/agent/services/${encodeURIComponent(serviceId)}`,
-    {
-      method: 'PATCH',
-      body: JSON.stringify({ acting_as_agent_id: actingAgentId, ...patch }),
-    },
-  );
-}
-
-export async function deleteService({ actingAgentId, serviceId }) {
-  return apiFetch(
-    `/api/agent/services/${encodeURIComponent(serviceId)}`,
-    {
-      method: 'DELETE',
-      body: JSON.stringify({ acting_as_agent_id: actingAgentId }),
-    },
-  );
-}
-
-export async function listServices({ actingAgentId }) {
-  const params = new URLSearchParams();
-  params.set('acting_as_agent_id', actingAgentId);
-  const { data } = await apiFetch(`/api/agent/services?${params}`);
-  return data || [];
-}
-
-export async function getServiceInfo({ actingAgentId, name }) {
-  const params = new URLSearchParams();
-  params.set('acting_as_agent_id', actingAgentId);
-  return apiFetch(
-    `/api/agent/services/${encodeURIComponent(name)}/info?${params}`,
-  );
-}
-
-export async function callService({ actingAgentId, name, input }) {
-  return apiFetch(
-    `/api/agent/services/${encodeURIComponent(name)}/call`,
-    {
-      method: 'POST',
-      body: JSON.stringify({ acting_as_agent_id: actingAgentId, input }),
-    },
-  );
-}
-
-// ── Briefings ──
-
-export async function getBriefing({actingAgentId, briefingId}) {
-  const params = new URLSearchParams();
-  params.set('acting_as_agent_id', actingAgentId);
-  return apiFetch(`/api/agent/briefings/${encodeURIComponent(briefingId)}?${params}`);
-}
-
-export async function publishBriefing({actingAgentId, bodyText, attachmentAssetId, currentConversationId, responseMode}) {
-  const body = { acting_as_agent_id: actingAgentId };
-  if (bodyText != null) body.body_text = bodyText;
-  if (attachmentAssetId != null) body.attachment_asset_id = attachmentAssetId;
-  if (currentConversationId != null) body.current_conversation_id = currentConversationId;
-  if (responseMode != null) body.response_mode = responseMode;
-  return apiFetch('/api/agent/briefings', {
-    method: 'POST',
-    body: JSON.stringify(body),
-  });
-}
-
-// ── Credentials (slot creation + daemon sync) ──
-
-export async function fetchCredentials() {
-  return apiFetch('/api/agent/credentials', { method: 'GET' });
-}
-
-export async function requestCredential({
-  actingAgentId, name, description, workstreamId,
-}) {
-  return apiFetch('/api/agent/credentials', {
-    method: 'POST',
-    body: JSON.stringify({
-      acting_as_agent_id: actingAgentId,
-      name,
-      description: description ?? null,
-      workstream_id: workstreamId ?? null,
-    }),
-  });
-}
-
-// ── Workstream dashboard ──
-
-export async function setWorkstreamDashboard({
-  actingAgentId, conversationId, dataJson, htmlTemplate,
-}) {
-  const body = { acting_as_agent_id: actingAgentId };
-  // Distinguish "omit" from "set to null" — only include keys the caller
-  // explicitly passed (including null clears the field).
-  if (dataJson !== undefined) body.data_json = dataJson;
-  if (htmlTemplate !== undefined) body.html_template = htmlTemplate;
-  return apiFetch(
-    `/api/agent/workstreams/${encodeURIComponent(conversationId)}/dashboard`,
-    { method: 'POST', body: JSON.stringify(body) },
-  );
-}
-
-export async function getWorkstreamDashboard({ actingAgentId, conversationId }) {
-  const params = new URLSearchParams();
-  params.set('acting_as_agent_id', actingAgentId);
-  return apiFetch(
-    `/api/agent/workstreams/${encodeURIComponent(conversationId)}/dashboard?${params}`,
-  );
-}
-
-// ── Workstream charter ──
-
-export async function getWorkstreamCharter({ actingAgentId, conversationId }) {
-  const params = new URLSearchParams();
-  params.set('acting_as_agent_id', actingAgentId);
-  return apiFetch(
-    `/api/agent/workstreams/${encodeURIComponent(conversationId)}/charter?${params}`,
-  );
-}
-
-export async function setWorkstreamCharter({ actingAgentId, conversationId, charter }) {
-  return apiFetch(
-    `/api/agent/workstreams/${encodeURIComponent(conversationId)}/charter`,
-    {
-      method: 'POST',
-      body: JSON.stringify({
-        acting_as_agent_id: actingAgentId,
-        charter: charter ?? null,
-      }),
-    },
-  );
-}
-
 // ── Channel event pipe ──
 
 export async function postEvent({ agent, agent_id, runtime_host_id, session_id, cwd, runtime_version, event, required = false }) {
   const agentId = agent_id || null;
   const queueKey = agentId || `${agent}:${session_id || ''}`;
+  const previous = agentEventQueues.get(queueKey) || Promise.resolve(null);
   const eventName = event?.worker_event_name || event?.hook_event_name || event?.event_name || 'unknown';
   const turnId = event?.turn_id || event?.reply_to_message_id || null;
   const seq = event?.event_seq ?? null;
   const deltaChars = typeof event?.delta === 'string' ? event.delta.length : null;
 
-  if (eventName === 'worker.message.delta') {
-    debugLog('events', 'delta.drop', {
-      agent,
-      agentId,
-      sessionId: shortId(session_id),
-      turnId: shortId(turnId),
-      deltaChars,
-    });
-    return null;
-  }
-
-  const postOnce = async () => {
-    const startedAt = Date.now();
-    try {
-      const result = await apiFetch('/api/events', {
-        method: 'POST',
-        body: JSON.stringify({ agent, agent_id: agentId, runtime_host_id, session_id, cwd, runtime_version, event }),
-        timeout: 5000,
-      });
-      if (required && result?.matched === false) {
-        throw new Error(`event was not matched (${eventName})`);
-      }
-      debugLog('events', 'post.ok', {
-        agent,
-        agentId,
-        sessionId: shortId(session_id),
-        runtimeVersion: runtime_version ?? null,
-        turnId: shortId(turnId),
-        eventName,
-        seq,
-        durationMs: Date.now() - startedAt,
-      });
-      return result;
-    } catch (err) {
-      debugError('events', 'post.failed', {
-        agent,
-        agentId,
-        sessionId: shortId(session_id),
-        runtimeVersion: runtime_version ?? null,
-        turnId: shortId(turnId),
-        eventName,
-        seq,
-        durationMs: Date.now() - startedAt,
-        error: err?.message || 'unknown error',
-      });
-      if (required) {
-        throw err;
-      }
-      return null;
-    }
-  };
-
-  if (eventName === 'worker.turn.complete' || eventName === 'worker.turn.error') {
-    return postOnce();
-  }
-
-  const previous = agentEventQueues.get(queueKey) || Promise.resolve(null);
   const queued = previous
     .catch(() => null)
-    .then(postOnce);
+    .then(async () => {
+      const startedAt = Date.now();
+      try {
+        const result = await apiFetch('/api/events', {
+          method: 'POST',
+          body: JSON.stringify({ agent, agent_id: agentId, runtime_host_id, session_id, cwd, runtime_version, event }),
+          timeout: 5000,
+        });
+        if (required && result?.matched === false) {
+          throw new Error(`event was not matched (${eventName})`);
+        }
+        debugLog('events', 'post.ok', {
+          agent,
+          agentId,
+          sessionId: shortId(session_id),
+          runtimeVersion: runtime_version ?? null,
+          turnId: shortId(turnId),
+          eventName,
+          seq,
+          deltaChars,
+          durationMs: Date.now() - startedAt,
+        });
+        return result;
+      } catch (err) {
+        debugError('events', 'post.failed', {
+          agent,
+          agentId,
+          sessionId: shortId(session_id),
+          runtimeVersion: runtime_version ?? null,
+          turnId: shortId(turnId),
+          eventName,
+          seq,
+          deltaChars,
+          durationMs: Date.now() - startedAt,
+          error: err?.message || 'unknown error',
+        });
+        if (required) {
+          throw err;
+        }
+        // Best-effort by default. Callers that need a hard failure set
+        // `required: true` (used for the final reply path).
+        return null;
+      }
+    });
 
   agentEventQueues.set(queueKey, queued);
   queued.finally(() => {
@@ -852,34 +552,6 @@ export async function postEvent({ agent, agent_id, runtime_host_id, session_id,
   return queued;
 }
 
-// ── Pair (no auth needed) ──
-
-export async function pair(payload) {
-  const url = `${getApiUrl()}/dispatch`;
-  const res = await fetch(url, {
-    method: 'POST',
-    headers: { 'Content-Type': 'application/json' },
-    body: JSON.stringify({ action: 'pair', ...payload }),
-    signal: AbortSignal.timeout(15000),
-  });
-  const body = await res.json().catch(() => ({}));
-  return {
-    statusCode: res.status,
-    ...body,
-  };
-}
-
-export async function pairPreview(payload) {
-  const url = `${getApiUrl()}/dispatch`;
-  const res = await fetch(url, {
-    method: 'POST',
-    headers: { 'Content-Type': 'application/json' },
-    body: JSON.stringify({ action: 'pair-preview', ...payload }),
-    signal: AbortSignal.timeout(15000),
-  });
-  return res.json();
-}
-
 async function pairingRequest(payload, timeout = 15000) {
   const url = `${getApiUrl()}/api/agent-pairings`;
   const res = await fetch(url, {

package/src/adapters/ticlawk/credentials.mjs

@@ -6,56 +6,14 @@
  * using the connector-specific env name.
  */
 
-import { AF_CONFIG_PATH, loadPersistentConfig, persistConfig, TICLAWK_CONNECTOR_API_KEY } from '../../core/config.mjs';
-
-export const TICLAWK_CREDENTIAL_NAMES = 'TICLAWK_CREDENTIAL_NAMES';
-
-function isRuntimeCredentialName(value) {
-  return /^[A-Z][A-Z0-9_]*$/.test(String(value || '').trim());
-}
-
-function parseCredentialNames(value) {
-  return String(value || '')
-    .split(',')
-    .map((name) => name.trim())
-    .filter(isRuntimeCredentialName);
-}
+import { AF_CONFIG_PATH, persistConfig, TICLAWK_CONNECTOR_API_KEY } from '../../core/config.mjs';
 
 export function persistApiCredential(apiKey) {
   if (!apiKey || !apiKey.startsWith('tk_')) return;
 
   persistConfig({
     [TICLAWK_CONNECTOR_API_KEY]: apiKey,
-    TICLAWK_SETUP_CODE: '',
   });
   process.env[TICLAWK_CONNECTOR_API_KEY] = apiKey;
-  delete process.env.TICLAWK_SETUP_CODE;
   console.log(`[connect] saved ${TICLAWK_CONNECTOR_API_KEY} to ${AF_CONFIG_PATH}`);
 }
-
-export function persistRuntimeCredentials(credentials = []) {
-  const current = loadPersistentConfig();
-  const previousNames = new Set(parseCredentialNames(current[TICLAWK_CREDENTIAL_NAMES]));
-  const nextNames = [];
-  const updates = {};
-
-  for (const credential of Array.isArray(credentials) ? credentials : []) {
-    const name = String(credential?.name || '').trim();
-    const value = typeof credential?.value === 'string' ? credential.value : '';
-    if (!isRuntimeCredentialName(name) || !value) continue;
-    updates[name] = value;
-    nextNames.push(name);
-  }
-
-  const nextNameSet = new Set(nextNames);
-  for (const name of previousNames) {
-    if (!nextNameSet.has(name)) updates[name] = '';
-  }
-  updates[TICLAWK_CREDENTIAL_NAMES] = nextNames.join(',');
-
-  persistConfig(updates);
-  return {
-    saved: nextNames.length,
-    removed: [...previousNames].filter((name) => !nextNameSet.has(name)).length,
-  };
-}