@ai-hero/sandcastle 0.4.6 → 0.4.8

package/dist/templates/parallel-planner/merge-prompt.md

@@ -15,7 +15,11 @@ After all branches are merged, make a single commit summarizing the merge.
 
 # CLOSE ISSUES
 
-For each branch that was merged, close its issue. Here are all the issues:
+For each branch that was merged, close its issue using the following command:
+
+`{{CLOSE_TASK_COMMAND}}`
+
+Here are all the issues:
 
 {{ISSUES}}
 

package/dist/templates/parallel-planner-with-review/implement-prompt.md

@@ -6,7 +6,7 @@ Pull in the issue using `{{VIEW_TASK_COMMAND}}`. If it has a parent PRD, pull th
 
 Only work on the issue specified.
 
-Work on branch {{BRANCH}}. Make commits, run tests, and close the issue when done.
+Work on branch {{BRANCH}}. Make commits, run tests, and close the issue when done using `{{CLOSE_TASK_COMMAND}}`.
 
 # CONTEXT
 
@@ -51,7 +51,7 @@ Keep it concise.
 
 # THE ISSUE
 
-If the task is not complete, leave a comment on the GitHub issue with what was done.
+If the task is not complete, leave a comment on the issue with what was done.
 
 Do not close the issue - this will be done later.
 

package/dist/templates/parallel-planner-with-review/main.mts

@@ -41,10 +41,7 @@ const hooks = {
 // Copy node_modules from the host into the worktree before each sandbox
 // starts. Avoids a full npm install from scratch; the hook above handles
 // platform-specific binaries and any packages added since the last copy.
-const copyToWorkspace = ["node_modules"];
-
-// Cap the number of concurrent sandboxes to avoid resource exhaustion.
-const MAX_CONCURRENCY = 4;
+const copyToWorktree = ["node_modules"];
 
 // ---------------------------------------------------------------------------
 // Main loop
@@ -64,9 +61,7 @@ for (let iteration = 1; iteration <= MAX_ITERATIONS; iteration++) {
   // -------------------------------------------------------------------------
   const plan = await sandcastle.run({
     hooks,
-    copyToWorkspace,
     sandbox: docker(),
-    branchStrategy: { type: "merge-to-head" },
     name: "planner",
     // One iteration is enough: the planner just needs to read and reason,
     // not write code.
@@ -109,75 +104,48 @@ for (let iteration = 1; iteration <= MAX_ITERATIONS; iteration++) {
   // and reviewer share the same sandbox instance per branch. The implementer
   // runs first; if it produces commits, the reviewer runs in the same sandbox.
   //
-  // A semaphore limits concurrency to MAX_CONCURRENCY sandboxes at once.
   // Promise.allSettled means one failing pipeline doesn't cancel the others.
   // -------------------------------------------------------------------------
 
-  // Simple semaphore for concurrency limiting
-  let running = 0;
-  const waiting: Array<() => void> = [];
-  const acquire = (): Promise<void> => {
-    if (running < MAX_CONCURRENCY) {
-      running++;
-      return Promise.resolve();
-    }
-    return new Promise<void>((resolve) => {
-      waiting.push(() => {
-        running++;
-        resolve();
-      });
-    });
-  };
-  const release = () => {
-    running--;
-    const next = waiting.shift();
-    if (next) next();
-  };
-
   const settled = await Promise.allSettled(
     issues.map(async (issue) => {
-      await acquire();
+      const sandbox = await sandcastle.createSandbox({
+        branch: issue.branch,
+        sandbox: docker(),
+        hooks,
+        copyToWorktree,
+      });
+
       try {
-        const sandbox = await sandcastle.createSandbox({
-          branch: issue.branch,
-          sandbox: docker(),
-          hooks,
-          copyToWorkspace,
+        // Run the implementer
+        const implement = await sandbox.run({
+          name: "implementer",
+          maxIterations: 100,
+          agent: sandcastle.claudeCode("claude-sonnet-4-6"),
+          promptFile: "./.sandcastle/implement-prompt.md",
+          promptArgs: {
+            TASK_ID: issue.id,
+            ISSUE_TITLE: issue.title,
+            BRANCH: issue.branch,
+          },
         });
 
-        try {
-          // Run the implementer
-          const implement = await sandbox.run({
-            name: "implementer",
-            maxIterations: 100,
+        // Only review if the implementer produced commits
+        if (implement.commits.length > 0) {
+          await sandbox.run({
+            name: "reviewer",
+            maxIterations: 1,
             agent: sandcastle.claudeCode("claude-sonnet-4-6"),
-            promptFile: "./.sandcastle/implement-prompt.md",
+            promptFile: "./.sandcastle/review-prompt.md",
             promptArgs: {
-              TASK_ID: issue.id,
-              ISSUE_TITLE: issue.title,
               BRANCH: issue.branch,
             },
           });
-
-          // Only review if the implementer produced commits
-          if (implement.commits.length > 0) {
-            await sandbox.run({
-              name: "reviewer",
-              maxIterations: 1,
-              agent: sandcastle.claudeCode("claude-sonnet-4-6"),
-              promptFile: "./.sandcastle/review-prompt.md",
-              promptArgs: {
-                BRANCH: issue.branch,
-              },
-            });
-          }
-
-          return implement;
-        } finally {
-          await sandbox.close();
         }
+
+        return implement;
       } finally {
-        release();
+        await sandbox.close();
       }
     }),
   );
@@ -228,9 +196,7 @@ for (let iteration = 1; iteration <= MAX_ITERATIONS; iteration++) {
   // -------------------------------------------------------------------------
   await sandcastle.run({
     hooks,
-    copyToWorkspace,
     sandbox: docker(),
-    branchStrategy: { type: "merge-to-head" },
     name: "merger",
     maxIterations: 1,
     agent: sandcastle.claudeCode("claude-sonnet-4-6"),

package/dist/templates/parallel-planner-with-review/merge-prompt.md

@@ -15,7 +15,11 @@ After all branches are merged, make a single commit summarizing the merge.
 
 # CLOSE ISSUES
 
-For each branch that was merged, close its issue. Here are all the issues:
+For each branch that was merged, close its issue using the following command:
+
+`{{CLOSE_TASK_COMMAND}}`
+
+Here are all the issues:
 
 {{ISSUES}}
 

package/dist/templates/sequential-reviewer/main.mts

@@ -36,7 +36,7 @@ const hooks = {
 // Copy node_modules from the host into the worktree before each sandbox
 // starts. Avoids a full npm install from scratch; the hook above handles
 // platform-specific binaries and any packages added since the last copy.
-const copyToWorkspace = ["node_modules"];
+const copyToWorktree = ["node_modules"];
 
 // ---------------------------------------------------------------------------
 // Main loop
@@ -57,7 +57,7 @@ for (let iteration = 1; iteration <= MAX_ITERATIONS; iteration++) {
   // -------------------------------------------------------------------------
   const implement = await sandcastle.run({
     hooks,
-    copyToWorkspace,
+    copyToWorktree,
     sandbox: docker(),
     branchStrategy: { type: "merge-to-head" },
     name: "implementer",
@@ -86,7 +86,7 @@ for (let iteration = 1; iteration <= MAX_ITERATIONS; iteration++) {
   // -------------------------------------------------------------------------
   await sandcastle.run({
     hooks,
-    copyToWorkspace,
+    copyToWorktree,
     sandbox: docker(),
     branchStrategy: { type: "branch", branch },
     name: "reviewer",

package/dist/templates/simple-loop/main.mts

@@ -28,7 +28,7 @@ await run({
 
   // Branch strategy — merge-to-head creates a temporary branch for the agent
   // to work on, then merges the result back to HEAD when the run completes.
-  // This is required when using copyToWorkspace, since head mode bind-mounts
+  // This is required when using copyToWorktree, since head mode bind-mounts
   // the host directory directly (no worktree to copy into).
   branchStrategy: { type: "merge-to-head" },
 
@@ -36,7 +36,7 @@ await run({
   // starts. This avoids a full npm install from scratch on every iteration.
   // The onSandboxReady hook still runs npm install as a safety net to handle
   // platform-specific binaries and any packages added since the last copy.
-  copyToWorkspace: ["node_modules"],
+  copyToWorktree: ["node_modules"],
 
   // Lifecycle hooks — commands that run inside the sandbox at specific points.
   hooks: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ai-hero/sandcastle",
-  "version": "0.4.6",
+  "version": "0.4.8",
   "description": "CLI for orchestrating AI agents in isolated sandbox environments",
   "type": "module",
   "main": "./dist/index.js",

package/dist/CopyToWorkspace.d.ts

@@ -1,8 +0,0 @@
-import { Effect } from "effect";
-/**
- * Copy files and directories from the host repo root to the worktree root,
- * using `cp -R --reflink=auto` for copy-on-write when the filesystem supports it.
- * Missing paths are silently skipped.
- */
-export declare const copyToWorkspace: (paths: string[], hostRepoDir: string, worktreePath: string) => Effect.Effect<void, never, never>;
-//# sourceMappingURL=CopyToWorkspace.d.ts.map

package/dist/CopyToWorkspace.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"CopyToWorkspace.d.ts","sourceRoot":"","sources":["../src/CopyToWorkspace.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAEhC;;;;GAIG;AACH,eAAO,MAAM,eAAe,mGAyBxB,CAAC"}

package/dist/CopyToWorkspace.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"CopyToWorkspace.js","sourceRoot":"","sources":["../src/CopyToWorkspace.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAC9C,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACrC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAEhC;;;;GAIG;AACH,MAAM,CAAC,MAAM,eAAe,GAAG,CAC7B,KAAe,EACf,WAAmB,EACnB,YAAoB,EACQ,EAAE,CAC9B,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC;IAClB,KAAK,MAAM,YAAY,IAAI,KAAK,EAAE,CAAC;QACjC,MAAM,GAAG,GAAG,IAAI,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC;QAC5C,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YACrB,SAAS;QACX,CAAC;QACD,MAAM,IAAI,GAAG,IAAI,CAAC,YAAY,EAAE,YAAY,CAAC,CAAC;QAC9C,KAAK,CAAC,CAAC,MAAM,CAAC,KAAK,CAAc,CAAC,MAAM,EAAE,EAAE;YAC1C,QAAQ,CAAC,IAAI,EAAE,CAAC,IAAI,EAAE,gBAAgB,EAAE,GAAG,EAAE,IAAI,CAAC,EAAE,CAAC,KAAK,EAAE,EAAE;gBAC5D,IAAI,KAAK,EAAE,CAAC;oBACV,0DAA0D;oBAC1D,QAAQ,CAAC,IAAI,EAAE,CAAC,IAAI,EAAE,GAAG,EAAE,IAAI,CAAC,EAAE,GAAG,EAAE;wBACrC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC;oBACpC,CAAC,CAAC,CAAC;gBACL,CAAC;qBAAM,CAAC;oBACN,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC;gBACpC,CAAC;YACH,CAAC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;IACL,CAAC;AACH,CAAC,CAAC,CAAC"}

package/dist/templates/blank/.env.example

@@ -1,5 +0,0 @@
-# Anthropic API key
-# If you want to use your Claude subscription instead of an API key, see https://github.com/mattpocock/sandcastle/issues/191
-ANTHROPIC_API_KEY=
-# GitHub personal access token
-GH_TOKEN=

package/dist/templates/parallel-planner/.env.example

@@ -1,5 +0,0 @@
-# Anthropic API key
-# If you want to use your Claude subscription instead of an API key, see https://github.com/mattpocock/sandcastle/issues/191
-ANTHROPIC_API_KEY=
-# GitHub personal access token
-GH_TOKEN=

package/dist/templates/parallel-planner-with-review/.env.example

@@ -1,5 +0,0 @@
-# Anthropic API key
-# If you want to use your Claude subscription instead of an API key, see https://github.com/mattpocock/sandcastle/issues/191
-ANTHROPIC_API_KEY=
-# GitHub personal access token
-GH_TOKEN=

package/dist/templates/sequential-reviewer/.env.example

@@ -1,5 +0,0 @@
-# Anthropic API key
-# If you want to use your Claude subscription instead of an API key, see https://github.com/mattpocock/sandcastle/issues/191
-ANTHROPIC_API_KEY=
-# GitHub personal access token
-GH_TOKEN=

package/dist/templates/simple-loop/.env.example

@@ -1,5 +0,0 @@
-# Anthropic API key
-# If you want to use your Claude subscription instead of an API key, see https://github.com/mattpocock/sandcastle/issues/191
-ANTHROPIC_API_KEY=
-# GitHub personal access token
-GH_TOKEN=