@saadnvd1/agent-os 0.3.766 → 0.3.768

package/lib/tasks/contract/plan.ts

@@ -25,7 +25,7 @@ import { getProject } from "../../projects";
 import { planFor, splitOf } from "../../plans/features";
 import { moveTask } from "../move-flow";
 import * as client from "./client";
-import { meetsNeeds, needsOf } from "../placement";
+import { meetsNeeds, needsOf, taskPinnedToMac } from "../placement";
 import { remotePlanPath } from "./plan-path";
 import { DEFAULT_PRIORITY, isPriority } from "../schedule-types";
 import { localHolds } from "../hold-checks";
@@ -55,6 +55,9 @@ function checkedSplit(s: Session, text: string | null) {
   }
 }
 
+const pinned = (s: Session, project: { working_directory: string }) =>
+  taskPinnedToMac(s.id, expandHome(project.working_directory));
+
 /**
  * Where a written plan is built: "there" (approved on the machine that
  * planned it) or "home" (moved here first). Pieces are checked against this
@@ -70,6 +73,9 @@ export function implementTarget(
   // The project lives there: nothing here to build it in.
   if (!project || isRemoteHost(project.host_id)) return "there";
   if (split) return "home";
+  // An explicit Mac pin builds on the Mac, wherever the plan ran.
+  if (pinned(s, project))
+    return meetsNeeds(["mac"], machineCapabilities()) ? "home" : "there";
   const { needs } = needsOf({
     projectId: project.id,
     projectPath: expandHome(project.working_directory),
@@ -229,6 +235,7 @@ export async function handoffHome(
     projectPath: expandHome(project.working_directory),
     prompt: s.task_prompt ?? "",
   });
+  if (pinned(s, project) && !needs.includes("mac")) needs.push("mac");
   const holds = localHolds(
     {
       needs,

package/lib/tasks/placement.ts

@@ -79,6 +79,28 @@ export function inferNeeds(projectPath: string): Need[] {
   return [];
 }
 
+// Whether the starter asked for a Mac outright: the request's `machine`
+// (stored on the queue row, which shares the task's id), a recipe's, or the
+// project's own machine setting. A need read from the brief or the files
+// doesn't count: that one a plan can ignore.
+export function pinnedToMac(
+  stored: string | null,
+  projectPath: string
+): boolean {
+  const asked = parseNeeds(stored);
+  if (asked) return asked.includes("mac");
+  return !!loadProjectConfig(projectPath).config.machine?.needs?.includes(
+    "mac"
+  );
+}
+
+export function taskPinnedToMac(taskId: string, projectPath: string): boolean {
+  const row = db
+    .prepare(`SELECT machine_needs FROM task_queue WHERE id = ?`)
+    .get(taskId) as { machine_needs: string | null } | undefined;
+  return pinnedToMac(row?.machine_needs ?? null, projectPath);
+}
+
 export function needsOf(input: {
   projectId: string;
   projectPath: string;

package/lib/tasks/queue-placement.ts

@@ -13,6 +13,7 @@ import { usesGitCrypt } from "../worktree-add";
 import { list as hostTasks } from "./contract/client";
 import {
   asPlanPhase,
+  pinnedToMac,
   place,
   unplaceable,
   wantsPlacing,
@@ -90,11 +91,22 @@ async function peerOptions(
   return out;
 }
 
-// A start that plans first goes by its plan phase: it needs no Mac or
-// simulator, and a plan-only one that no machine can take starts here
+// A start that plans first goes by its plan phase: it needs no simulator, and a plan-only one that no machine can take starts here
 // as ever (it never waits on load).
 const plansFirst = (r: QueueRow) => !!(r.plan || r.plan_only);
 
+// Planning needs no simulator or Xcode, and a need read from the files or the
+// brief doesn't hold it. Only an explicit Mac pin does: the work is Mac-only
+// (often because the other machine can't reach what it touches).
+const planNeeds = (r: QueueRow, needs: Need[]): Need[] => {
+  const project = getProject(r.project_id);
+  return project &&
+    needs.includes("mac") &&
+    pinnedToMac(r.machine_needs, expandHome(project.working_directory))
+    ? ["mac"]
+    : [];
+};
+
 // Marks each start the home machine's load rules out as remote and returns
 // where it goes. "stale": the line moved while the machines were asked, so
 // this look is over, and the one a change during it asks for sees the line
@@ -143,7 +155,7 @@ export async function placeCandidates(
       asked(c),
       home,
       options,
-      plan ? [] : (needsOf.get(c.id) ?? []),
+      plan ? planNeeds(r, needsOf.get(c.id) ?? []) : (needsOf.get(c.id) ?? []),
       now
     );
     if (!spot) continue;

package/package.json

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