@saadnvd1/agent-os 0.3.580 → 0.3.582

package/components/Chat/EventRow.tsx

@@ -1,6 +1,5 @@
 "use client";
 
-import type { MouseEvent } from "react";
 import {
   Bot,
   CalendarClock,
@@ -15,6 +14,7 @@ import { eventParts, originOf } from "@/lib/chat/origin";
 import { hrefWithSession } from "@/lib/session-url";
 import { cn } from "@/lib/utils";
 import { UserMessage } from "./Messages";
+import { openSessionLink as openSession } from "./session-link";
 import { useRowState } from "./rowState";
 
 type UserItem = Extract<ChatItem, { kind: "user" }>;
@@ -34,14 +34,6 @@ const SHORT = 32;
 export const clock = (at: number) =>
   new Date(at).toLocaleTimeString([], { hour: "numeric", minute: "2-digit" });
 
-// Opens the session through its address, the way a shared link does.
-function openSession(e: MouseEvent<HTMLAnchorElement>, id: string) {
-  if (e.metaKey || e.ctrlKey || e.shiftKey || e.button !== 0) return;
-  e.preventDefault();
-  window.history.pushState(null, "", hrefWithSession(location.href, id));
-  window.dispatchEvent(new PopStateEvent("popstate"));
-}
-
 function Source({ origin }: { origin: ChatOrigin }) {
   const name = (
     <span className="text-foreground/80 block truncate font-medium">

package/components/Chat/Messages.tsx

@@ -12,27 +12,49 @@ import type { ChatItem } from "@/lib/chat/events";
 import { cn } from "@/lib/utils";
 import { leadingCommand } from "@/lib/chat/commands";
 import { compactedLine } from "@/lib/chat/stale";
-import { splitInlineCode, splitUserText } from "@/lib/chat/user-text";
+import { splitProse, splitUserText } from "@/lib/chat/user-text";
+import { inAppSessionId } from "@/lib/session-url";
 import { CodeBlock } from "./Code";
 import { CopyButton } from "./CopyButton";
 import { ImageThumb } from "./ImageThumb";
 import { Markdown } from "./Markdown";
+import { openSessionLink } from "./session-link";
 import { UndoButton } from "./Undo";
 import { useRowState } from "./rowState";
 
 type Of<K extends ChatItem["kind"]> = Extract<ChatItem, { kind: K }>;
 
+function UserLink({ href }: { href: string }) {
+  const id =
+    typeof window === "undefined"
+      ? null
+      : inAppSessionId(href, window.location.origin);
+  return (
+    <a
+      href={href}
+      target={id ? undefined : "_blank"}
+      rel={id ? undefined : "noopener noreferrer"}
+      onClick={id ? (e) => openSessionLink(e, id) : undefined}
+      className="font-medium [overflow-wrap:anywhere] underline underline-offset-2"
+    >
+      {href}
+    </a>
+  );
+}
+
 function Prose({ text }: { text: string }) {
   return (
     <>
-      {splitInlineCode(text).map((p, i) =>
-        p.code ? (
+      {splitProse(text).map((p, i) =>
+        p.kind === "code" ? (
           <code
             key={i}
             className="bg-foreground/[0.06] rounded px-1 py-0.5 font-mono text-[0.9em]"
           >
             {p.text}
           </code>
+        ) : p.kind === "link" ? (
+          <UserLink key={i} href={p.text} />
         ) : (
           <span key={i}>{p.text}</span>
         )

package/components/Chat/session-link.ts

@@ -0,0 +1,10 @@
+import type { MouseEvent } from "react";
+import { hrefWithSession } from "@/lib/session-url";
+
+// Opens the session through its address, the way a shared link does.
+export function openSessionLink(e: MouseEvent<HTMLAnchorElement>, id: string) {
+  if (e.metaKey || e.ctrlKey || e.shiftKey || e.button !== 0) return;
+  e.preventDefault();
+  window.history.pushState(null, "", hrefWithSession(location.href, id));
+  window.dispatchEvent(new PopStateEvent("popstate"));
+}

package/lib/chat/user-text.ts

@@ -62,3 +62,55 @@ export function splitInlineCode(
   if (last < text.length) out.push({ code: false, text: text.slice(last) });
   return out;
 }
+
+export type ProsePart =
+  | { kind: "text"; text: string }
+  | { kind: "code"; text: string }
+  | { kind: "link"; text: string };
+
+const URL_RE = /https?:\/\/[^\s<>"'`]+/gi;
+const TRAILING = /[.,;:!?]$/;
+
+// Punctuation that ends a sentence, or a closing paren with no opener in the
+// URL, belongs to the text around it.
+function trimUrl(url: string): string {
+  for (;;) {
+    if (TRAILING.test(url)) url = url.slice(0, -1);
+    else if (url.endsWith(")") && count(url, ")") > count(url, "("))
+      url = url.slice(0, -1);
+    else return url;
+  }
+}
+
+function count(s: string, ch: string): number {
+  return s.split(ch).length - 1;
+}
+
+// http(s) URLs in plain text, split out so they can be links.
+export function splitLinks(text: string): { link: boolean; text: string }[] {
+  const out: { link: boolean; text: string }[] = [];
+  let last = 0;
+  for (const m of text.matchAll(URL_RE)) {
+    const url = trimUrl(m[0]);
+    if (!/^https?:\/\/[^/?#]+/i.test(url)) continue;
+    if (m.index > last)
+      out.push({ link: false, text: text.slice(last, m.index) });
+    out.push({ link: true, text: url });
+    last = m.index + url.length;
+  }
+  if (last < text.length) out.push({ link: false, text: text.slice(last) });
+  return out;
+}
+
+// Prose split into code spans, links and the text between. Links are only
+// looked for outside code spans.
+export function splitProse(text: string): ProsePart[] {
+  return splitInlineCode(text).flatMap((p): ProsePart[] =>
+    p.code
+      ? [{ kind: "code", text: p.text }]
+      : splitLinks(p.text).map((l) => ({
+          kind: l.link ? "link" : "text",
+          text: l.text,
+        }))
+  );
+}

package/lib/session-url.ts

@@ -26,6 +26,18 @@ export function hrefWithSession(href: string, id: string | null): string {
   return url.pathname + url.search + url.hash;
 }
 
+/** The session a link to this same instance opens, or null for any other link. */
+export function inAppSessionId(href: string, origin: string): string | null {
+  let url: URL;
+  try {
+    url = new URL(href);
+  } catch {
+    return null;
+  }
+  if (url.origin !== origin || url.pathname !== "/") return null;
+  return sessionIdFromHref(href);
+}
+
 export function sessionLink(origin: string, id: string): string {
   return `${origin}/?${SESSION_PARAM}=${encodeURIComponent(id)}`;
 }

package/lib/tasks/queue.ts

@@ -189,7 +189,10 @@ const workspaceOf = (projectId: string) =>
 // gives its slot back until it resumes. A quick change
 // runs no agent, so it holds none. Without a
 // workspace, the project is the scope.
-function runningIn(scope: { workspaceId: string | null; projectId: string }) {
+function runningIn(
+  scope: { workspaceId: string | null; projectId: string },
+  forSlots = false
+) {
   const where = scope.workspaceId
     ? `p.workspace_id = @ws`
     : `p.id = @project AND p.workspace_id IS NULL`;
@@ -199,6 +202,7 @@ function runningIn(scope: { workspaceId: string | null; projectId: string }) {
         `SELECT s.id FROM sessions s JOIN projects p ON p.id = s.project_id
          WHERE ${where} AND s.archived_at IS NULL AND s.quick_of IS NULL
            AND (s.pause_state IS NULL OR s.pause_state != 'paused')
+           ${forSlots ? `AND NOT (COALESCE(s.plan_only, 0) = 1 AND COALESCE(s.plan_phase, '') != 'implementing')` : ""}
            AND s.task_status IN (${IN_FLIGHT.map((s) => `'${s}'`).join(", ")})`
       )
       .all({ ws: scope.workspaceId, project: scope.projectId }) as {
@@ -214,7 +218,8 @@ function startingIn(workspaceId: string): number {
       db
         .prepare(
           `SELECT COUNT(*) AS n FROM task_queue q JOIN projects p ON p.id = q.project_id
-         WHERE q.status = 'starting' AND p.workspace_id = ?`
+         WHERE q.status = 'starting' AND p.workspace_id = ?
+           AND q.plan_only IS NULL`
         )
         .get(workspaceId) as { n: number }
     ).n
@@ -315,7 +320,8 @@ const waitingIn = (workspaceId: string) =>
       .prepare(
         `SELECT COUNT(*) AS n FROM task_queue q JOIN projects p ON p.id = q.project_id
          WHERE q.status = 'queued' AND p.workspace_id = ?
-           AND q.after_task IS NULL AND q.after_any IS NULL`
+           AND q.after_task IS NULL AND q.after_any IS NULL
+           AND q.plan_only IS NULL`
       )
       .get(workspaceId) as { n: number }
   ).n;
@@ -344,9 +350,10 @@ export function queueIfNeeded(req: QueueRequest): QueuedTaskView | null {
   } else if (after)
     afterTask = resolveAfter(after, req.originWorkspaceId ?? workspaceId).id;
   const full =
+    !req.planOnly &&
     limit !== null &&
     !!workspaceId &&
-    (runningIn({ workspaceId, projectId: project.id }).length +
+    (runningIn({ workspaceId, projectId: project.id }, true).length +
       startingIn(workspaceId) >=
       limit ||
       waitingIn(workspaceId) > 0);
@@ -360,6 +367,7 @@ export function queueIfNeeded(req: QueueRequest): QueuedTaskView | null {
   const loaded =
     !waits &&
     !full &&
+    !req.planOnly &&
     !!req.originWorkspaceId &&
     loadHeldStart(project.id, req.prompt, setting);
   if (!waits && !full && !loaded) return null;
@@ -523,9 +531,10 @@ export const queuedView = (id: string) =>
 
 // Why an automatic start has to wait, or null. A start now by a person
 // isn't held.
-function autoHold(workspaceId: string | null): Hold | null {
+function autoHold(workspaceId: string | null, planOnly = false): Hold | null {
   if (workspaceId && isPaused(workspaceId))
     return { code: "orch_paused", text: "Waiting: the orchestrator is paused" };
+  if (planOnly) return null;
   const window = brakesEnabled() ? windowRefusal(readUsage()) : null;
   return window ? { code: "window", text: `Waiting: ${window}` } : null;
 }
@@ -609,7 +618,7 @@ async function startRow(
     return "failed";
   }
   if (!opts.force) {
-    const held = autoHold(workspaceId);
+    const held = autoHold(workspaceId, !!r.plan_only);
     if (held) {
       if (r.note !== held.text)
         patch(r.id, { note: held.text, wait_code: held.code });
@@ -621,7 +630,8 @@ async function startRow(
     if (
       workspaceId &&
       limit !== null &&
-      runningIn({ workspaceId, projectId: r.project_id }).length +
+      !r.plan_only &&
+      runningIn({ workspaceId, projectId: r.project_id }, true).length +
         startingIn(workspaceId) >=
         limit
     )
@@ -643,7 +653,7 @@ async function startRow(
   }
   if (!claimed) return "held";
   const placed = opts.placement ?? null;
-  if (!opts.force)
+  if (!opts.force && !r.plan_only)
     recordAutoStart(placed?.hostId ?? LOCAL, !!opts.heavy, Date.now());
   // A linked machine's start never sees this machine's held messages: they
   // go with the brief it is sent.
@@ -747,7 +757,7 @@ function groupFor(
   const scope = { workspaceId, projectId };
   return {
     inFlight:
-      runningIn(scope).length +
+      runningIn(scope, true).length +
       (workspaceId ? (reserved.get(workspaceId) ?? 0) : 0) +
       startingRows,
     limit: limitOf(workspaceId) ?? Number.POSITIVE_INFINITY,
@@ -814,8 +824,9 @@ function candidateOf(
       heavy: heavyNeeds(needs),
       needsUnmet: !remote && !meetsNeeds(needs, machineCapabilities()),
       remote,
+      planOnly: !!r.plan_only,
       before,
-      auto: autoHold(r.workspace_id),
+      auto: autoHold(r.workspace_id, !!r.plan_only),
       any,
     },
   };

package/lib/tasks/schedule.ts

@@ -82,6 +82,9 @@ export interface Candidate {
   needsUnmet: boolean;
   // Starts on another machine: this one's load doesn't matter to it.
   remote: boolean;
+  // A plan-only task before approval: it reads code and writes one file, so
+  // load, spacing and the slot limit don't apply. Pause and `after` still do.
+  planOnly?: boolean;
   // `after X` hold, from the caller (it needs the database).
   before: Hold | null;
   // Orchestrator Pause and the usage window, which hold after `after`.
@@ -127,11 +130,12 @@ const seconds = (ms: number) => Math.max(1, Math.ceil(ms / 1000));
 
 // The machine holds, in order. Null: it may start here now.
 export function machineHold(
-  c: Pick<Candidate, "heavy" | "needsUnmet" | "remote" | "priority">,
+  c: Pick<Candidate, "heavy" | "needsUnmet" | "remote" | "priority"> &
+    Partial<Pick<Candidate, "planOnly">>,
   m: MachineView,
   now: number
 ): Hold | null {
-  if (c.remote) return null;
+  if (c.remote || c.planOnly) return null;
   if (c.needsUnmet) return { code: "needs", text: `Waiting: needs a Mac` };
   if (!m.enabled) return null;
   if (m.reading)
@@ -226,7 +230,7 @@ export function planStarts(
         priority: c.priority,
       });
       if (hold) return void plan.explain.set(c.id, view(hold));
-      if (inFlight >= group.limit)
+      if (!c.planOnly && inFlight >= group.limit)
         return void plan.explain.set(
           c.id,
           view({
@@ -240,8 +244,8 @@ export function planStarts(
         if (free) used.add(free);
         plan.consume.set(c.id, free);
       }
-      inFlight += 1;
-      if (!c.remote) {
+      if (!c.planOnly) inFlight += 1;
+      if (!c.remote && !c.planOnly) {
         m.lastAutoStartAt = now;
         if (c.heavy) {
           m.lastHeavyStartAt = now;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saadnvd1/agent-os",
-  "version": "0.3.580",
+  "version": "0.3.582",
   "description": "Self-hosted web UI for managing Claude Code sessions",
   "bin": {
     "agent-os": "scripts/agent-os"