@phnx-labs/agents-cli 1.22.92 → 1.22.94

package/dist/lib/session/state.js

@@ -399,10 +399,10 @@ export function structuredQuestionFromAsk(args) {
     return { text, reason: 'question', options: options.length ? options : undefined };
 }
 /**
- * Canonical approve/deny choices for an interactive prompt that carries no
- * agent-supplied option list: Claude's plan-review and permission dialogs. Approve
- * is reliably option 1; deny/keep-planning maps to ESC, which cancels the prompt in
- * every variant (2- or 3-option) — safer than guessing a digit that could differ.
+ * Canonical approve/send-back choices for Claude's plan-review dialog, which
+ * carries no agent-supplied option list. Approve is reliably option 1; keep-planning
+ * maps to ESC, which cancels the prompt in every variant (2- or 3-option) — safer
+ * than guessing a digit that could differ.
  */
 function planReviewQuestion() {
     return {
@@ -414,15 +414,22 @@ function planReviewQuestion() {
         ],
     };
 }
-function permissionQuestion(preview) {
-    return {
-        text: preview ? `Permission — ${preview}` : 'Waiting on a permission prompt',
-        reason: 'permission',
-        options: [
-            { label: 'Approve', key: '1' },
-            { label: 'Deny', key: 'esc' },
-        ],
-    };
+/**
+ * The harness's own stamp on an event, as ms — or undefined when the event carries
+ * none. A stamp LATER than the file's last write cannot have come from the
+ * transcript (a parser that finds no stamp fills the field with its own parse
+ * time), so it is rejected rather than read as "just happened": the mtime is the
+ * physical upper bound on when any transcript line was written.
+ */
+function eventStampMs(e, mtimeMs) {
+    if (!e?.timestamp)
+        return undefined;
+    const ms = Date.parse(e.timestamp);
+    if (!Number.isFinite(ms))
+        return undefined;
+    if (mtimeMs != null && ms > mtimeMs)
+        return undefined;
+    return ms;
 }
 /** Does an assistant message read as a question directed at the user? */
 function looksLikeQuestion(text) {
@@ -461,7 +468,8 @@ function lastOf(events, pred) {
  */
 export function inferActivity(events, ctx = {}) {
     const windowMs = ctx.activeWindowMs ?? ACTIVE_WINDOW_MS;
-    const fresh = ctx.mtimeMs != null && Date.now() - ctx.mtimeMs < windowMs;
+    const nowMs = ctx.nowMs ?? Date.now();
+    const fresh = ctx.mtimeMs != null && nowMs - ctx.mtimeMs < windowMs;
     // A non-live process (pidAlive === false) can never be "working"; the strongest
     // it gets is "waiting on you" (a dangling question) or "idle".
     const canWork = ctx.pidAlive !== false && (ctx.pidAlive === true || fresh);
@@ -494,6 +502,7 @@ export function inferActivity(events, ctx = {}) {
         lastRole: lastMsg?.role,
         lastEventKind: last?.type,
         lastActivityMs: ctx.mtimeMs,
+        lastEventMs: eventStampMs(last, ctx.mtimeMs),
         preview: previewSource ? describeEvent(previewSource) : undefined,
         todos,
         tail: tail.length ? tail : undefined,
@@ -514,14 +523,17 @@ export function inferActivity(events, ctx = {}) {
         const question = structuredQuestionFromAsk(lastPlanOrAsk.args) ?? { text: 'Asked you a question', reason: 'question' };
         return { ...base, activity: 'waiting_input', awaitingReason: 'question', preview: question.text, question };
     }
-    // Pending tool call (tool_use with no following tool_result): mid-turn.
+    // Pending tool call (tool_use with no following tool_result): the call is in
+    // flight for as long as the process is alive. A command that has run for ten
+    // minutes and a permission dialog that has sat for ten minutes leave the SAME
+    // transcript — a tool_use with no result and a quiet file — so elapsed time is
+    // not evidence of a request, and this engine never labels one `permission`
+    // (PHNX-3999: the "is the user idle?" reminder and the two-minute mark were both
+    // being rendered with Approve/Deny). The harness's own `permission_prompt` hook
+    // event, carried by the feed block, is the only evidence a dialog is up; the
+    // attention reconciler confirms it against `lastEventMs`.
     if (last.type === 'tool_use') {
-        if (canWork && fresh)
-            return { ...base, activity: 'working' };
-        // Alive but the file hasn't moved — likely blocked on a permission prompt.
-        if (ctx.pidAlive)
-            return { ...base, activity: 'waiting_input', awaitingReason: 'permission', question: permissionQuestion(base.preview) };
-        return { ...base, activity: 'idle' };
+        return { ...base, activity: canWork ? 'working' : 'idle' };
     }
     // Thinking or a tool result just landed → agent is mid-turn if recently active.
     if (last.type === 'thinking' || last.type === 'tool_result' || last.type === 'error') {
@@ -537,12 +549,16 @@ export function inferActivity(events, ctx = {}) {
         // A prose question takes a free-text reply (no select-list), so no options/keys.
         // Unlike the structural plan/ask signals above, the prose heuristic DECAYS: a
         // question nobody answered within PROSE_QUESTION_FRESH_MS is a session that
-        // ended, not one that needs you (RUSH-1522). A prose "?" is only "waiting on
-        // you" while the file is DEMONSTRABLY fresh — with no mtime signal at all we
-        // cannot assert freshness, so the heuristic must not fire (an unknown-age
-        // prose question is a session that ended, closing RUSH-1522's null-mtime hole
-        // where a null mtime kept the question forever).
-        const questionFresh = ctx.mtimeMs != null && Date.now() - ctx.mtimeMs < PROSE_QUESTION_FRESH_MS;
+        // ended, not one that needs you (RUSH-1522). The question's age is the
+        // assistant message's own stamp when the harness wrote one — the exact moment
+        // it was asked — and the file's last write when it did not; the age is measured
+        // against `ctx.nowMs`, never a clock frozen at parse time, so the verdict
+        // expires on schedule even while the bytes sit still (PHNX-3999). With no age
+        // evidence at all the heuristic must not fire (an unknown-age prose question is
+        // a session that ended, closing RUSH-1522's null-mtime hole where a null mtime
+        // kept the question forever).
+        const askedAtMs = eventStampMs(last, ctx.mtimeMs) ?? ctx.mtimeMs;
+        const questionFresh = askedAtMs != null && nowMs - askedAtMs < PROSE_QUESTION_FRESH_MS;
         if (questionFresh && looksLikeQuestion(last.content ?? '')) {
             const text = oneLine(last.content ?? '');
             return { ...base, activity: 'waiting_input', awaitingReason: 'question', question: { text, reason: 'question' } };

package/dist/lib/types.d.ts

@@ -1303,6 +1303,13 @@ export interface BrowserProfileConfig {
     logDir?: string;
     /** Optional SSH host where logDir lives, e.g. "user@remote-host". */
     logHost?: string;
+    /** Stable native identity for an Arc Space profile; display names are never addresses. */
+    arc?: {
+        profileId: string;
+        profileName: string;
+        spaceId: string;
+        spaceTitle: string;
+    };
 }
 /** Options controlling which agents and resources are synced during `agents sync` / `agents use`. */
 export interface SyncOptions {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@phnx-labs/agents-cli",
-  "version": "1.22.92",
+  "version": "1.22.94",
   "description": "One CLI for all your AI coding agents - versions, config, cloud dispatch, sessions, and teams (now with first-class Grok Build CLI support)",
   "type": "module",
   "main": "dist/index.js",