@afokapu/atdd-flow 0.3.84 → 0.3.86

package/conventions/atdd-flow.collaboration/atdd-flow.collaboration.convention.yaml

@@ -33,6 +33,10 @@ content:
     receipts inbound communication from any seat, but sends substantive direction, results, and
     wakes only to main@project. Only main@project executes that project repository merge; the
     operator may authorize, relay, or inspect but never merges.
+    Manage every project main, coordinator, and driver seat only through atdd-flow
+    canonical commands (spawn, bind, pi runtime launch, task lifecycle, handover); start,
+    stop, rebind, or succeed a seat by hand only under explicit owner direction and always
+    complete the supported bind/reconciliation immediately after.
     Post decisions and open questions as messages when they happen.
   projection: >-
     Keep this full policy only in this convention. Managed AGENTS.md and CLAUDE.md blocks

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@afokapu/atdd-flow",
-  "version": "0.3.84",
+  "version": "0.3.86",
   "description": "Filesystem-first Desk registry for projects, seats, tasks, threads, and handoffs.",
   "type": "module",
   "bin": {

package/src/collaboration-policy.ts

@@ -55,7 +55,17 @@ function lineCount(text: string) {
   return text ? text.replace(/\r?\n$/, "").split(/\r?\n/).length : 0;
 }
 
-/** Idempotently projects the canonical pointer while preserving user content and refusing oversized instruction files. */
+/** The managed block plus one separator line is the most a projection may add to any file. */
+const maxManagedDeltaLines = lineCount(block()) + 1;
+
+/** Fails closed when a projection would grow a file by more than the managed block; owner content is never counted. */
+function assertBoundedDelta(name: string, source: string, next: string) {
+  if (lineCount(next) - lineCount(source) > maxManagedDeltaLines) {
+    throw new Error(`${name} projection would add more than ${maxManagedDeltaLines} managed lines; refusing projection without changing user content.`);
+  }
+}
+
+/** Idempotently projects the canonical pointer while preserving every owner byte outside the managed markers. */
 export async function projectCollaborationPolicy(root: string) {
   const projections = await Promise.all(["AGENTS.md", "CLAUDE.md"].map(async (name) => {
     const file = join(root, name);
@@ -66,11 +76,7 @@ export async function projectCollaborationPolicy(root: string) {
     }
     return { name, file, source, next: projectCollaborationPolicyText(source) };
   }));
-  for (const { name, source, next } of projections) {
-    if (lineCount(source) > 20 || lineCount(next) > 20) {
-      throw new Error(`${name} exceeds the 20-line cap; refusing projection without changing user content.`);
-    }
-  }
+  for (const { name, source, next } of projections) assertBoundedDelta(name, source, next);
   await Promise.all(projections.map(async ({ file, source, next }) => {
     if (next !== source) await writeFile(file, next, "utf8");
   }));

package/src/seats.ts

@@ -255,7 +255,7 @@ async function taskCoordinatorBase(root: string, config: Project, task: Task, re
     const head = await runOutput(["git", "-C", coordinator.worktree, "rev-parse", "HEAD"]);
     if (head !== branchHead) throw new Error(`Coordinator ${coordinator.address} head is stale relative to ${branch}.`);
   } else {
-    try { await assertBranchReady(mode, coordinator.worktree, branch); }
+    try { await assertBranchReady(mode, coordinator.worktree, branch, coordinator.role === "main"); }
     catch (error) { throw new Error(`Coordinator ${coordinator.address} is not ready: ${(error as Error).message}`); }
   }
   return { coordinator, branch, commit: branchHead };
@@ -331,7 +331,7 @@ async function spawnTaskDriver(root: string, config: Project, role: Role, name:
     const currentBranch = await branchCommit(mode, base.coordinator.worktree, coordinatorBase.branch);
     if (currentBranch !== coordinatorBase.commit) throw new Error(`Coordinator ${base.coordinator.address} advanced during driver provisioning.`);
     if (!baseArgument) {
-      const current = await assertBranchReady(mode, base.coordinator.worktree, coordinatorBase.branch);
+      const current = await assertBranchReady(mode, base.coordinator.worktree, coordinatorBase.branch, base.coordinator.role === "main");
       if (current !== coordinatorBase.commit) throw new Error(`Coordinator ${base.coordinator.address} advanced during driver provisioning.`);
     }
     await atomicYaml(paths(root).seatFile(address), record);

package/src/vcs.ts

@@ -103,7 +103,11 @@ export async function isAncestor(cwd: string, ancestor: string, descendant: stri
   catch { return false; }
 }
 
-export async function assertBranchReady(mode: VcsMode, cwd: string, branch: string) {
+/**
+ * A branch is ready when its checkout is clean and at the branch tip. `allowDirtyParent` additionally accepts a Jujutsu working copy
+ * whose parent is exactly the branch tip with uncommitted changes (a migrated dirty primary); conflicts and any other lineage stay refused.
+ */
+export async function assertBranchReady(mode: VcsMode, cwd: string, branch: string, allowDirtyParent = false) {
   if (mode === "git") {
     if (await runOutput(["git", "-C", cwd, "status", "--porcelain"])) throw new Error(`Worktree ${cwd} is dirty.`);
     const checkedOut = await runOutput(["git", "-C", cwd, "branch", "--show-current"]);
@@ -120,7 +124,7 @@ export async function assertBranchReady(mode: VcsMode, cwd: string, branch: stri
       runOutput(["jj", "log", "-r", "@-", "--no-graph", "-T", "commit_id"], cwd),
       runOutput(["jj", "diff", "-r", "@", "--name-only"], cwd),
     ]);
-    if (parent === expected && !changed) return expected;
+    if (parent === expected && (!changed || allowDirtyParent)) return expected;
   }
   throw new Error(`Worktree ${cwd} is stale relative to ${branch}.`);
 }