@nodii/dev 0.6.38 → 0.6.40

package/dist/core.js

@@ -4,7 +4,7 @@ import { chmod, mkdir, readFile, rm, writeFile } from "node:fs/promises";
 import { homedir, platform } from "node:os";
 import { dirname, join } from "node:path";
 
-export const VERSION = "0.6.38";
+export const VERSION = "0.6.40";
 export const configPath = () =>
   process.env.NODII_DEV_CONFIG_PATH ??
   join(homedir(), ".config/nodii-dev/config.json");

package/dist/freeze.js

@@ -109,6 +109,7 @@ export const GATHER_LOOKUP_IDS = ["figma-guidelines"];
 /** Build the stored contract; Delta Gather keeps lane-authored sections. */
 export function freezeFromGather({
   gather,
+  ask,
   boundaries,
   issue,
   laneNotes,
@@ -131,7 +132,8 @@ export function freezeFromGather({
     );
   return {
     objective: {
-      ask: gather.objective,
+      // Dev's Lane contract names the ask; without one, Gather's objective.
+      ask: ask ?? gather.objective,
       issue: issue === undefined ? (current?.objective?.issue ?? null) : issue,
       laneNotes:
         laneNotes === undefined

package/dist/lanes.js

@@ -98,7 +98,23 @@ const agentFollowUp = (ref, shortlistId, flags) => [
     .filter((flag) => typeof flags[flag] === "string")
     .map((flag) => `--${flag} ${shellQuote(flags[flag])}`),
 ].join(" ");
+// Dev's Lane contract (hub-58): objective, issue body, the coordinator's lane
+// notes, done check and boundaries, printed before anything else.
+const contractLines = (data) => (data.laneContract ? ["Lane contract", data.laneContract, ""] : []);
+async function laneContract(id, request, options) {
+  try {
+    return (await request("GET", `/api/v1/lanes/${encode(id)}/contract`, undefined, options)).data?.text ?? null;
+  } catch (error) {
+    // A Dev from before the Lane contract route: say so, never hide it.
+    if (error?.code === "HTTP_404" || error?.code === "NOT_FOUND") return "(this Dev does not serve the Lane contract yet; read the lane with `nodii-dev lane show`)";
+    throw error;
+  }
+}
+// Gather never ends on a bare "finished" (hub-58): the next step is the task.
+const nextStepText = (ref, doneCheck) =>
+  `Gather is finished. Next: Task. Work the objective until ${doneCheck} is met, then run \`nodii-dev lane land ${ref}\` (or the lane's finish command). Don't stop before that.`;
 const agentShortlistText = (ref, data, flags) => [
+  ...contractLines(data),
   `Trigger shortlist ${data.shortlistId} · ${JSON.stringify(data.objective)}`,
   "You are Gather's picker (picker: agent). Nothing is recorded yet.",
   ...data.triggers.map((trigger) => `  ${trigger.code}\t${trigger.title ?? ""}: ${trigger.sentence ?? ""}${trigger.forcedBy?.length ? `  [forced by ${trigger.forcedBy.join(", ")}]` : ""}`),
@@ -387,8 +403,10 @@ export async function laneCommand(action, args, flags, { request = devRequest, c
         : await request("POST", "/api/v1/rules/shortlist", shortlistBody, { signal });
       const shortlistId = shortlist.data?.shortlistId;
       const triggers = shortlist.data?.triggers ?? [];
-      if (byAgent && !byAgent.shortlist)
-        return { payload: { data: { recorded: false, picker: "agent", shortlistId, objective, triggers } }, human: (p) => agentShortlistText(ref, p.data, flags) };
+      if (byAgent && !byAgent.shortlist) {
+        const contract = await laneContract(id, request, { signal });
+        return { payload: { data: { recorded: false, picker: "agent", shortlistId, objective, triggers, laneContract: contract } }, human: (p) => agentShortlistText(ref, p.data, flags) };
+      }
       // Only the local model pick takes the per-lane pick lock; an agent pick
       // records a choice already made (AGENT-HARNESS-19 + GATHER-NA).
       const pickInput = {
@@ -409,7 +427,8 @@ export async function laneCommand(action, args, flags, { request = devRequest, c
           // No picker on this Mac: the agent picks from the same shortlist (hub-58).
           const fallback = agentFallbackReason(error);
           if (!fallback) throw error;
-          return { payload: { data: { recorded: false, picker: "agent", fallback, shortlistId, objective, triggers } }, human: (p) => `picker: agent (${p.data.fallback})\n${agentShortlistText(ref, p.data, flags)}` };
+          const contract = await laneContract(id, request, { signal });
+          return { payload: { data: { recorded: false, picker: "agent", fallback, shortlistId, objective, triggers, laneContract: contract } }, human: (p) => `picker: agent (${p.data.fallback})\n${agentShortlistText(ref, p.data, flags)}` };
         }
       }
       const payload = await request("POST", `/api/v1/lanes/${encode(id)}/gather`, {
@@ -421,6 +440,7 @@ export async function laneCommand(action, args, flags, { request = devRequest, c
         ...(sectionLimit !== undefined ? { sectionLimitPerDocument: sectionLimit } : {}),
         shortlistId,
         picker: byAgent ? "agent" : picked.model ?? PICKER_MODEL,
+        doneCheck,
       }, { signal });
       const figma = figmaScope.map((file) => ({
         fileKey: file.fileKey,
@@ -434,7 +454,19 @@ export async function laneCommand(action, args, flags, { request = devRequest, c
         lane.launch?.narrowingRemainder && `Narrowing remainder: ${lane.launch.narrowingRemainder}`,
       ].filter(Boolean);
       const freezeRequest = (method, path, body, options = {}) => request(method, path, body, { ...options, signal });
-      await storeFreeze(id, {
+      // Dev's Lane contract is what Gather printed; the Freeze stores it word for word.
+      const contract = payload.data?.contract;
+      const contractRepos = contract?.boundaries?.repos;
+      const fromContract = Boolean(contractRepos?.length && contractRepos.every((repository) => repository.repo && repository.branch && repository.worktree));
+      await storeFreeze(id, fromContract ? {
+        gather: payload.data,
+        ask: contract.objective.ask,
+        boundaries: contract.boundaries,
+        issue: contract.objective.issue,
+        laneNotes: contract.objective.laneNotes,
+        doneCheck: contract.objective.doneCheck ?? doneCheck,
+        lookups: lookup ? [lookup] : [],
+      } : {
         gather: payload.data,
         boundaries: { repos: repositories, ...(declaredFigma ? { figma } : {}) },
         issue,
@@ -453,7 +485,8 @@ export async function laneCommand(action, args, flags, { request = devRequest, c
       // The pick's model and latency ride along in --json (morning check records them).
       const result = { ...payload, picker: { model: picked.model ?? null, latency: picked.latency ?? null, ...(picked.timing ? { timing: picked.timing, stats: picked.stats } : {}) } };
       const timingLine = picked.timing ? `\n${pickTimingLine(picked.timing, picked.stats)}` : "";
-      return { payload: result, human: (p) => `${gatherText(p, objective, reasons)}${scopeNotice}${timingLine}` };
+      const doneText = contract?.objective?.doneCheck ?? doneCheck;
+      return { payload: result, human: (p) => `${gatherText(p, objective, reasons)}${scopeNotice}${timingLine}\n\n${nextStepText(ref, doneText)}` };
     } catch (error) {
       if (signal.aborted) throw new CliError("GATHER_TIMEOUT", `Gather's Dev requests exceeded their ${Math.round(deadline.ms / 1000)}-second deadline (the local pick runs on its own budget); nothing was retried.`, 1);
       throw error;

package/dist/runtime-launcher.js

@@ -106,10 +106,15 @@ export function buildRecoveryInvocation(data, { env = process.env, home = homedi
 
 function buildInvocation(options, { env = process.env, home = homedir() } = {}) {
   const { runId, laneKey, runtime, model, cwd, addDirs, profileHome, devConfigPath } = options;
+  // A headless lane has nobody to approve a tool call, so it runs in the
+  // operator-approved headless lane mode: no permission prompts, under the
+  // lane's own profile, with the harness safety hooks as the guard (they
+  // still run in this mode). Without it every Bash call, `nodii-dev lane
+  // gather` included, is refused "requires approval" (AGENT-HARNESS-43).
   const args = runtime === "codex"
-    ? ["exec", "--json", "--cd", cwd, "--model", model, ...addDirs.flatMap((path) => ["--add-dir", path]), "-"]
+    ? ["exec", "--json", "--dangerously-bypass-approvals-and-sandbox", "--cd", cwd, "--model", model, ...addDirs.flatMap((path) => ["--add-dir", path]), "-"]
     // Claude Code refuses --print with stream-json unless --verbose is set.
-    : ["--print", "--output-format", "stream-json", "--verbose", "--model", model, ...addDirs.flatMap((path) => ["--add-dir", path])];
+    : ["--print", "--output-format", "stream-json", "--verbose", "--dangerously-skip-permissions", "--model", model, ...addDirs.flatMap((path) => ["--add-dir", path])];
   return {
     command: runtime,
     args,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nodii/dev",
-  "version": "0.6.38",
+  "version": "0.6.40",
   "description": "Agent CLI for Nodii Dev, reached through the platform edge",
   "type": "module",
   "bin": {