@saadnvd1/agent-os 0.3.839 → 0.3.848

package/scripts/update-run.test.ts

@@ -78,20 +78,27 @@ describe("scripts/update-run.mjs", () => {
 
 // A stand-in for the restarted server: its build comes from a file the fake
 // redeploy writes, and its startedAt can be made to change (a crash loop).
-async function fakeServer(opts: { restarts?: boolean } = {}) {
+async function fakeServer(
+  opts: { restarts?: boolean; switchAt?: number } = {}
+) {
   let n = 0;
+  let calls = 0;
   const server = http.createServer((req, res) => {
-    const build = fs.readFileSync(path.join(dir, "build"), "utf8").trim();
+    let build = fs.readFileSync(path.join(dir, "build"), "utf8").trim();
     res.setHeader("content-type", "application/json");
-    if (req.url === "/api/updates/health")
+    if (req.url === "/api/updates/health") {
+      // Something else (a deploy job) restarts it into another build.
+      const other = opts.switchAt !== undefined && calls++ >= opts.switchAt;
+      if (other) build = "other";
       return res.end(
         JSON.stringify({
           ok: true,
           build,
-          startedAt: opts.restarts ? `s${n++}` : "s",
+          startedAt: opts.restarts ? `s${n++}` : other ? "s-other" : "s",
           migrations: { known: 3, applied: 3 },
         })
       );
+    }
     res.end("{}");
   });
   await new Promise<void>((r) => server.listen(0, "127.0.0.1", r));
@@ -170,6 +177,57 @@ describe("scripts/update-run.mjs checks the new server", () => {
     }
   });
 
+  it("a restart into a different build is left alone: no rollback, nothing marked", async () => {
+    fs.writeFileSync(path.join(dir, "next-build"), "new");
+    fs.writeFileSync(path.join(dir, "rollback-works"), "");
+    const srv = await fakeServer({ switchAt: 2 });
+    try {
+      const s = await go(REDEPLOY, "x", verifying(srv.port, "new"));
+      expect(s).toMatchObject({ phase: "failed", outcome: "interrupted" });
+      expect(s.error).toContain("something else restarted it");
+      expect(rolled()).toBeNull();
+    } finally {
+      await srv.close();
+    }
+  });
+
+  it("a rollback records the build it went back to as autodeploy's deployed marker", async () => {
+    fs.writeFileSync(path.join(dir, "next-build"), "stale");
+    fs.writeFileSync(path.join(dir, "rollback-works"), "");
+    const state = path.join(dir, "autodeploy");
+    fs.mkdirSync(state);
+    fs.writeFileSync(path.join(state, "deployed"), "f5054a0\n");
+    const srv = await fakeServer();
+    try {
+      const s = await go(REDEPLOY, "x", {
+        ...verifying(srv.port, "new"),
+        AGENTOS_AUTODEPLOY_DIR: state,
+      });
+      expect(s.outcome).toBe("rolled_back");
+      expect(fs.readFileSync(path.join(state, "deployed"), "utf8").trim()).toBe(
+        "a"
+      );
+    } finally {
+      await srv.close();
+    }
+  });
+
+  it("a rollback on a machine without autodeploy creates no marker", async () => {
+    fs.writeFileSync(path.join(dir, "next-build"), "stale");
+    fs.writeFileSync(path.join(dir, "rollback-works"), "");
+    const state = path.join(dir, "autodeploy");
+    const srv = await fakeServer();
+    try {
+      await go(REDEPLOY, "x", {
+        ...verifying(srv.port, "new"),
+        AGENTOS_AUTODEPLOY_DIR: state,
+      });
+      expect(fs.existsSync(state)).toBe(false);
+    } finally {
+      await srv.close();
+    }
+  });
+
   it("when the rollback doesn't come up either, it ends broken", async () => {
     fs.writeFileSync(path.join(dir, "next-build"), "stale");
     const srv = await fakeServer();

package/components/Settings/NoApprovalsSection.tsx

@@ -1,103 +0,0 @@
-"use client";
-
-import { useState } from "react";
-import { Button } from "@/components/ui/button";
-import { SettingToggle } from "@/components/Settings/SettingToggle";
-import { useApprovalsModeQuery, useSetApprovalsMode } from "@/data/approvals";
-
-// What it switches off, said plainly before it can be switched on.
-export const NO_APPROVALS_WARNING = [
-  "Every ask is approved the moment it is raised, hard lines included: public posts and messages, money, irreversible or destructive changes, credentials.",
-  "A PR stuck failing its gates, with no CI, or too big to review still merges.",
-  "Brakes stop holding new starts. Plan-only tasks still wait for approve_plan.",
-  "Every agent AgentOS starts runs with its permission prompts off, and prompts already waiting are allowed.",
-];
-
-export function NoApprovalsSection() {
-  const { data, isPending, isError, error } = useApprovalsModeQuery();
-  const set = useSetApprovalsMode();
-  const [confirming, setConfirming] = useState(false);
-
-  if (isPending)
-    return <div className="bg-muted/40 h-16 animate-pulse rounded-lg" />;
-  if (isError)
-    return <p className="text-destructive text-sm">{error.message}</p>;
-
-  return (
-    <div className="space-y-3">
-      <SettingToggle
-        title="No approvals"
-        detail="Shuts off every approval AgentOS would otherwise wait on, and every agent's permission prompts. Off by default."
-        checked={data.on}
-        onChange={(on) => {
-          if (on) setConfirming(true);
-          else set.mutate(false);
-        }}
-      />
-      {confirming && !data.on && (
-        <div
-          role="alertdialog"
-          aria-label="Turn on No approvals"
-          className="rounded-lg border border-red-500/20 bg-red-500/10 p-3"
-        >
-          <p className="mb-2 text-sm font-medium text-red-400">
-            Turn on No approvals?
-          </p>
-          <ul className="text-muted-foreground mb-3 list-disc space-y-1 pl-5 text-xs">
-            {NO_APPROVALS_WARNING.map((line) => (
-              <li key={line}>{line}</li>
-            ))}
-            <li>
-              Terminals already running in an agent other than Claude keep
-              asking until they are restarted. Each approval made for you is
-              logged below.
-            </li>
-          </ul>
-          <div className="flex gap-2">
-            <Button
-              size="sm"
-              variant="destructive"
-              disabled={set.isPending}
-              onClick={() =>
-                set.mutate(true, { onSuccess: () => setConfirming(false) })
-              }
-            >
-              {set.isPending ? "Turning on..." : "Turn on No approvals"}
-            </Button>
-            <Button
-              size="sm"
-              variant="ghost"
-              disabled={set.isPending}
-              onClick={() => setConfirming(false)}
-            >
-              Cancel
-            </Button>
-          </div>
-        </div>
-      )}
-      {set.error && (
-        <p className="text-destructive text-xs">{set.error.message}</p>
-      )}
-      <div>
-        <p className="mb-1 text-sm font-medium">Approved for you</p>
-        {data.audit.length === 0 ? (
-          <p className="text-muted-foreground text-xs">Nothing yet.</p>
-        ) : (
-          <ul className="space-y-1">
-            {data.audit.map((row) => (
-              <li key={row.id} className="text-xs">
-                <span className="text-muted-foreground">
-                  {row.at} · {row.source} ·{" "}
-                </span>
-                {row.subject}
-                {row.detail ? (
-                  <span className="text-muted-foreground"> ({row.detail})</span>
-                ) : null}
-              </li>
-            ))}
-          </ul>
-        )}
-      </div>
-    </div>
-  );
-}

package/components/Settings/SettingToggle.tsx

@@ -1,36 +0,0 @@
-"use client";
-
-import { Switch } from "@/components/ui/switch";
-
-// One labelled switch on a Settings page; a locked one says so.
-export function SettingToggle({
-  title,
-  detail,
-  checked,
-  locked,
-  onChange,
-}: {
-  title: string;
-  detail: string;
-  checked: boolean;
-  locked?: boolean;
-  onChange: (on: boolean) => void;
-}) {
-  return (
-    <label className="bg-muted/40 flex items-start gap-3 rounded-lg px-3 py-2.5">
-      <div className="min-w-0 flex-1">
-        <p className="text-sm font-medium">{title}</p>
-        <p className="text-muted-foreground text-xs">
-          {detail}
-          {locked ? " Can't be changed from here." : ""}
-        </p>
-      </div>
-      <Switch
-        checked={checked}
-        disabled={locked}
-        onCheckedChange={onChange}
-        className="mt-0.5"
-      />
-    </label>
-  );
-}

package/components/Settings/SetupSection.tsx

@@ -1,46 +0,0 @@
-"use client";
-
-import { Button } from "@/components/ui/button";
-import { HealthList } from "@/components/Welcome/HealthList";
-import { AgentList } from "@/components/Welcome/AgentList";
-import { ReachRow } from "@/components/Reach/ReachRow";
-import { OwnerNameRow } from "./OwnerNameRow";
-import { settingsUiActions } from "@/stores/settingsUi";
-import { welcomeUiActions } from "@/stores/welcomeUi";
-
-// What this machine has for running AgentOS and agents, and the first-run setup again.
-export function SetupSection() {
-  return (
-    <div className="flex flex-col gap-4">
-      <OwnerNameRow />
-      <div className="bg-muted/40 flex items-center gap-3 rounded-lg px-3 py-2.5">
-        <div className="min-w-0 flex-1">
-          <p className="text-sm font-medium">First-run setup</p>
-          <p className="text-muted-foreground text-xs">
-            Agents, your first project, and a first session.
-          </p>
-        </div>
-        <Button
-          size="sm"
-          variant="outline"
-          className="min-h-11 sm:min-h-8"
-          onClick={() => {
-            settingsUiActions.close();
-            welcomeUiActions.open("agents");
-          }}
-        >
-          Run setup again
-        </Button>
-      </div>
-      <div className="flex flex-col gap-2">
-        <h3 className="text-sm font-medium">Health</h3>
-        <HealthList />
-        <ReachRow />
-      </div>
-      <div className="flex flex-col gap-2">
-        <h3 className="text-sm font-medium">Agents on this machine</h3>
-        <AgentList />
-      </div>
-    </div>
-  );
-}