@kitman-sh/software-engineering 0.10.56 → 0.10.58

package/dist/bundle.js

@@ -21470,7 +21470,7 @@ var build_default = defineLoop({
       summary: `Review round ${last.round} passed with nothing blocking.`,
       review: `review:${last.round}`
     }).pipe(catchTag2("StepFailed", (error) => succeed7({ approved: false, reason: error.reason }))) : undefined;
-    const landed = input.land === true ? yield* call("land", shepherd_default, {
+    const landed = input.land !== false ? yield* call("land", shepherd_default, {
       changeRequestId: opened.id,
       ...input.taskId === undefined ? undefined : { taskId: input.taskId },
       ...input.branch === undefined ? undefined : { branch: input.branch }
@@ -22252,7 +22252,8 @@ An earlier Build of this failed. ${retry.steer}` };
       ...taskId === undefined ? undefined : { taskId },
       repoBindingId: head.repoBindingId,
       branch: head.branch,
-      revision: head.revision
+      revision: head.revision,
+      land: false
     }).pipe(catchTag2("ChildFailed", (failure) => succeed7(failure)));
     if (built instanceof ChildFailed)
       return settled(ticket, { status: "failed", repoBindingId: head.repoBindingId, reason: built.reason });
@@ -29298,4 +29299,4 @@ var runKit = (config, identity, packages) => {
   runMain2(program.pipe(flatMap3((code) => sync2(() => process.exit(code)))));
 };
 // .kitman-build-entry.ts
-runKit(kitman_config_default, { name: "@kitman-sh/software-engineering", version: "0.10.56" }, []);
+runKit(kitman_config_default, { name: "@kitman-sh/software-engineering", version: "0.10.58" }, []);

package/dist/declarations.json

@@ -3090,7 +3090,7 @@
   "carried": [
     {
       "name": "@kitman-sh/software-engineering",
-      "version": "0.10.56",
+      "version": "0.10.58",
       "setup": "2a237548a69b0cd53ec3be7f687cfa8c2128335b3a655d34e185f2abce6a3c37",
       "loops": [
         "build",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kitman-sh/software-engineering",
-  "version": "0.10.56",
+  "version": "0.10.58",
   "description": "Everything for building software: Loops, their Agents, the skills they use and the Ponytail plugin.",
   "type": "module",
   "license": "UNLICENSED",
@@ -17,12 +17,12 @@
     "tsconfig.json"
   ],
   "dependencies": {
-    "@kitman-sh/sdk": "0.10.56",
+    "@kitman-sh/sdk": "0.10.58",
     "effect": "4.1.2"
   },
   "kitman": {
     "bundle": "dist/bundle.js",
-    "digest": "bbbaf2e21dc9c84ed9f1522a0b7d94d450fccab21918b9b84cf53d4d604abf3a",
+    "digest": "ed3f43456848d81db262c65cd16fbbc8ae2683d7b6f44a0348c0de96507e0319",
     "protocolVersion": 5,
     "declarations": "dist/declarations.json",
     "setup": {

package/src/build.ts

@@ -57,7 +57,7 @@ export const BuildInput = Schema.Struct({
   maxRounds: Schema.optional(Schema.Int),
   /** The Task's open ChangeRequest an earlier Run left, which this Build fixes forward instead of rebuilding. */
   continuing: Schema.optional(Continuing),
-  /** Sees the ChangeRequest through to merge with Shepherd once it opens. */
+  /** Sees the ChangeRequest through to merge with Shepherd once it opens; false leaves it open for a person. */
   land: Schema.optional(Schema.Boolean),
   /**
    * An open ChangeRequest to fix forward on: the Run works its branch, never force-pushing, and reports in one comment
@@ -517,7 +517,7 @@ export default defineLoop({
           : undefined
 
       const landed =
-        input.land === true
+        input.land !== false
           ? yield* call('land', shepherd, {
               changeRequestId: opened.id,
               ...(input.taskId === undefined ? undefined : { taskId: input.taskId }),

package/src/plan.ts

@@ -285,7 +285,9 @@ const buildTicket = (
         ...(taskId === undefined ? undefined : { taskId }),
         repoBindingId: head.repoBindingId,
         branch: head.branch,
-        revision: head.revision
+        revision: head.revision,
+        // Plan merges each Build's ChangeRequest into its integration branch itself.
+        land: false
       }).pipe(Effect.catchTag('ChildFailed', (failure: ChildFailed) => Effect.succeed(failure)))
 
       if (built instanceof ChildFailed)