@snappedly-tools/shipyard 0.9.2 → 0.11.0-staging.36280043078

package/dist/templates/sequential-reviewer/handoff.sh

@@ -103,6 +103,7 @@ for ((i=1; i<${#scope_ids[@]}; i++)); do
     echo "Could not mark ticket #${scope_ids[i]} complete; retry issue completion when GitHub is available" >&2
     exit 75
   fi
+  remove_issue_label "${scope_ids[i]}" ready-for-human
   remove_issue_label "${scope_ids[i]}" shipyard:blocked
   remove_issue_label "${scope_ids[i]}" shipyard:pending
 done
@@ -128,7 +129,11 @@ if ! gh label create "$status_label" --repo "$repo" --color "$status_color" --de
   echo "Warning: could not create $status_label" >&2
   status_synced=false
 fi
-if ! gh issue edit "$issue" --repo "$repo" --add-label "$status_label"; then
+if gh issue edit "$issue" --repo "$repo" --add-label "$status_label"; then
+  if [[ "$status_label" == shipyard:complete ]]; then
+    remove_issue_label "$issue" ready-for-human
+  fi
+else
   if [[ "$branch" == "shipyard/issue-$issue" ]]; then
     echo "Could not mark ticket #$issue complete; retry issue completion when GitHub is available" >&2
     exit 75
@@ -170,6 +175,6 @@ if [[ "$status_synced" == true ]]; then
     fi
   done
 else
-  echo "Warning: status labels need reconciliation; retaining ticket activation" >&2
+  echo "Warning: status labels need reconciliation; add shipyard to the ticket to retry" >&2
 fi
 printf '%s\n' "$url"

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

@@ -22,24 +22,27 @@ const roleModels = {
   routine: readRoleModel("routine"),
   strong: readRoleModel("strong"),
 };
-const CODEX_REASONING_EFFORTS = shipyard.CODEX_REASONING_EFFORTS;
-type CodexReasoningEffort = shipyard.CodexReasoningEffort;
-const readCodexReasoningEffort = (
+const REASONING_EFFORTS = shipyard.REASONING_EFFORTS;
+type ReasoningEffort = shipyard.ReasoningEffort;
+const readRoleReasoningEffort = (
   role: ModelRole,
-): CodexReasoningEffort | undefined => {
-  if (!CODEX_PROVIDER) return undefined;
-  const envName = `SHIPYARD_CODEX_${role.toUpperCase()}_REASONING_EFFORT`;
-  const effort = process.env[envName]?.trim();
+): ReasoningEffort | undefined => {
+  const envName = `SHIPYARD_${role.toUpperCase()}_REASONING_EFFORT`;
+  const legacyName = `SHIPYARD_CODEX_${role.toUpperCase()}_REASONING_EFFORT`;
+  const sharedEffort = process.env[envName]?.trim();
+  const effort =
+    sharedEffort ||
+    (CODEX_PROVIDER ? process.env[legacyName]?.trim() : undefined);
   if (!effort) return undefined;
-  if (!(CODEX_REASONING_EFFORTS as readonly string[]).includes(effort))
+  if (!(REASONING_EFFORTS as readonly string[]).includes(effort))
     throw new Error(
-      `${envName} must be one of ${CODEX_REASONING_EFFORTS.join(", ")}; received "${effort}"`,
+      `${sharedEffort ? envName : legacyName} must be one of ${REASONING_EFFORTS.join(", ")}; received "${effort}"`,
     );
-  return effort as CodexReasoningEffort;
+  return effort as ReasoningEffort;
 };
 const roleEfforts = {
-  routine: readCodexReasoningEffort("routine"),
-  strong: readCodexReasoningEffort("strong"),
+  routine: readRoleReasoningEffort("routine"),
+  strong: readRoleReasoningEffort("strong"),
 };
 const readCodexRoleModel = (role: ModelRole, defaultModel: AgentModel) => {
   if (!CODEX_PROVIDER || typeof defaultModel === "string") return defaultModel;
@@ -136,6 +139,7 @@ for (let iteration = 0; iteration < MAX_ITERATIONS; iteration++) {
     title: string;
     branch: string;
     kind: "standalone" | "spec";
+    triageReady?: boolean;
     body?: string;
     tickets?: Array<{
       id: string;
@@ -143,6 +147,7 @@ for (let iteration = 0; iteration < MAX_ITERATIONS; iteration++) {
       body: string;
       state: string;
       blockedBy: Array<{ id: string; title: string; state: string }>;
+      triageReady: boolean;
     }>;
     completedTicketIds?: string[];
     outstandingTicketIds?: string[];
@@ -176,24 +181,33 @@ for (let iteration = 0; iteration < MAX_ITERATIONS; iteration++) {
         repository,
         "--add-label",
         "shipyard:pending",
+        "--remove-label",
+        "shipyard",
       ]);
     const sandbox = await shipyard.createSandbox({
       branch: issue.branch,
       sandbox: sandboxProvider,
       hooks,
+      copyToWorktree: [".shipyard/setup.sh"],
     });
     let evidence: string;
     try {
       for (const ticketId of issue.kind === "spec"
         ? (issue.tickets ?? []).map((ticket) => ticket.id)
         : [issue.id]) {
-        await sandbox.run({
-          name: `triage #${ticketId}`,
-          agent: roleAgent("routine", shipyard.CODEX_MODELS.routine),
-          maxIterations: 1,
-          promptFile: "./.shipyard/triage-prompt.md",
-          promptArgs: { TASK_ID: ticketId },
-        });
+        const ready =
+          issue.kind === "spec"
+            ? issue.tickets?.find((ticket) => ticket.id === ticketId)
+                ?.triageReady
+            : issue.triageReady;
+        if (!ready)
+          await sandbox.run({
+            name: `triage #${ticketId}`,
+            agent: roleAgent("routine", shipyard.CODEX_MODELS.routine),
+            maxIterations: 1,
+            promptFile: "./.shipyard/triage-prompt.md",
+            promptArgs: { TASK_ID: ticketId, BASE_BRANCH: targetBranch },
+          });
         verifyTriage(ticketId);
       }
       const implement = await sandbox.run({
@@ -237,7 +251,7 @@ for (let iteration = 0; iteration < MAX_ITERATIONS; iteration++) {
           `Issue #${issue.id} has unresolved review findings: ${review.stdout.trim().slice(-1200)}`,
         );
       }
-      evidence = `${implementationEvidence}\n\n${reviewEvidence}`;
+      evidence = `${implementationEvidence}\n\nReview: APPROVED\n${reviewEvidence}`;
     } finally {
       await closeClean(sandbox);
     }
@@ -245,6 +259,7 @@ for (let iteration = 0; iteration < MAX_ITERATIONS; iteration++) {
     const publication = await shipyard.createSandbox({
       branch: issue.branch,
       sandbox: sandboxProvider,
+      copyToWorktree: [".shipyard/handoff.sh"],
     });
     try {
       publicationUncertain = true;

package/dist/templates/sequential-reviewer/select-issues.mjs

@@ -124,6 +124,17 @@ const dependencies = (id) => {
 };
 const planningSpec = (body) =>
   /work item type:\*?\*?\s*planning spec/i.test(body ?? "");
+const triageStates = [
+  "needs-triage",
+  "needs-info",
+  "ready-for-agent",
+  "ready-for-human",
+  "wontfix",
+];
+const triageReady = (labels) =>
+  triageStates
+    .filter((state) => labels.some((label) => label.name === state))
+    .join() === "ready-for-agent";
 const textParent = (body, issueId) => {
   const match = /^##?\s*Parent\b[^\n]*\n/im.exec(body ?? "");
   if (!match) return undefined;
@@ -476,9 +487,10 @@ for (const candidate of activated) {
         ),
     );
     selectedTicketIds = new Set(selectableTickets.map((ticket) => ticket.id));
-    const tickets = selectableTickets.map(({ labels: _labels, ...ticket }) => ({
+    const tickets = selectableTickets.map(({ labels, ...ticket }) => ({
       ...ticket,
       blockedBy: dependencies(ticket.id),
+      triageReady: triageReady(labels),
     }));
     const completedTicketIds = linkedTickets
       .filter((ticket) =>
@@ -756,7 +768,13 @@ for (const candidate of activated) {
             completedTicketIds,
             outstandingTicketIds,
           }
-        : { id, title: candidate.title, branch, kind: "standalone" },
+        : {
+            id,
+            title: candidate.title,
+            branch,
+            kind: "standalone",
+            triageReady: triageReady(candidate.labels ?? []),
+          },
     );
   } catch (error) {
     if (!(error instanceof SelectionError)) throw error;

package/dist/templates/sequential-reviewer/setup.sh

@@ -18,7 +18,14 @@ if [[ -f package.json ]]; then
     else manager=npm; fi
   fi
   case "$manager" in
-    npm) npm install ;;
+    npm)
+      if [[ -f package-lock.json ]]; then
+        if ! npm ci; then
+          echo "npm ci failed; installing from package.json without changing package-lock.json" >&2
+          npm install --no-package-lock
+        fi
+      else npm install --no-package-lock
+      fi ;;
     pnpm) corepack pnpm install ;;
     yarn) corepack yarn install ;;
     bun)

package/dist/templates/sequential-reviewer/triage-prompt.md

@@ -1,5 +1,7 @@
 Triage GitHub issue #{{TASK_ID}} before Shipyard implementation.
 
-Follow `/triage` and the repository's `docs/agents/triage.md` label mapping. Read the full issue and comments, verify its claim against the codebase, then apply the appropriate category and state label and post the brief or triage notes required by the skill. The maintainer has authorized Shipyard to apply an evidence-based recommendation during this unattended intake. If a decision requires maintainer judgment, use `needs-triage` and record the question. Do not implement the issue or start a PR in this phase.
+Follow `/triage` and the repository's triage label mapping. Read the full issue and comments. Verify the reported behavior against `origin/{{BASE_BRANCH}}`, the integration baseline. The checked-out task branch may contain unfinished changes from an earlier attempt. Do not close the issue as already complete based on the task branch or comments about that candidate.
 
-Report the applied state and evidence. Shipyard will independently re-read GitHub labels and implement only an open issue carrying both `shipyard` and the sole triage state `ready-for-agent`.
+Apply the appropriate category and state label and post the brief or triage notes required by the skill. The maintainer has authorized Shipyard to apply an evidence-based recommendation during this unattended intake. Derive acceptance criteria from the stated outcome. For example, equal spacing between named header items is a testable visual goal without a pixel target. Choose a representative supported viewport if none is specified. An inaccessible attachment does not require `needs-info` when the text establishes the current and desired behavior; record the verification limit in the brief. Do not treat equal CSS gap values alone as proof that the rendered gaps are equal. Apply `needs-info` and ask one concrete question only when missing details change the intended result or prevent meaningful verification. Use `needs-triage` only when the available facts leave a product or policy decision for the maintainer, not for routine layout or implementation choices. Preserve the `shipyard` or `shipyard:pending` label. Do not implement the issue or start a PR in this phase.
+
+Report the applied state and evidence. Shipyard will independently re-read GitHub labels and implement only an open issue carrying `shipyard` or `shipyard:pending` and the sole triage state `ready-for-agent`.

package/dist/templates/sequential-reviewer/verify-triage.sh

@@ -11,9 +11,18 @@ gh issue view "$issue" --repo "$repo" --json state,labels |
     const issue = JSON.parse(require("node:fs").readFileSync(0, "utf8"));
     const labels = new Set(issue.labels.map((label) => label.name));
     const states = ["needs-triage", "needs-info", "ready-for-agent", "ready-for-human", "wontfix"];
-    if (issue.state !== "OPEN" || !labels.has("shipyard") ||
-        states.filter((state) => labels.has(state)).join() !== "ready-for-agent") {
-      console.error("Issue cannot be implemented: requires an open issue with shipyard and only ready-for-agent as its triage state");
+    if (issue.state !== "OPEN" ||
+        !(labels.has("shipyard") || labels.has("shipyard:pending"))) {
+      console.error("Issue cannot be implemented: requires an open issue with shipyard or shipyard:pending");
+      process.exit(1);
+    }
+    const triageStates = states.filter((state) => labels.has(state));
+    if (triageStates.join() === "needs-info") {
+      console.error("Issue needs information from the reporter before implementation");
+      process.exit(1);
+    }
+    if (triageStates.join() !== "ready-for-agent") {
+      console.error(`Issue cannot be implemented: triage state is ${triageStates.join() || "missing"}; requires only ready-for-agent`);
       process.exit(1);
     }
   '

package/dist/templates/shared/block-scope.sh

@@ -108,6 +108,10 @@ Work was not handed off. To retry, resolve the problem, remove shipyard:blocked,
 fi
 
 if [[ "$branch" == "shipyard/spec-$root" ]]; then
+  if ! remove_issue_status "$root" ready-for-human; then
+    echo "Warning: could not clear spec triage status" >&2
+    status_synced=false
+  fi
   if ! remove_issue_status "$root" shipyard:outstanding-tasks; then
     echo "Warning: could not clear spec status" >&2
     status_synced=false
@@ -122,9 +126,11 @@ else
 fi
 if [[ "$branch" == "shipyard/spec-$root" ]]; then
   for affected_id in "${affected_ids[@]:1}"; do
+    remove_issue_status "$affected_id" ready-for-human || status=75
     remove_issue_status "$affected_id" shipyard:pending || status=75
   done
 else
+  remove_issue_status "$root" ready-for-human || status=75
   remove_issue_status "$root" shipyard:pending || status=75
 fi
 

package/dist/templates/shared/handoff.sh

@@ -103,6 +103,7 @@ for ((i=1; i<${#scope_ids[@]}; i++)); do
     echo "Could not mark ticket #${scope_ids[i]} complete; retry issue completion when GitHub is available" >&2
     exit 75
   fi
+  remove_issue_label "${scope_ids[i]}" ready-for-human
   remove_issue_label "${scope_ids[i]}" shipyard:blocked
   remove_issue_label "${scope_ids[i]}" shipyard:pending
 done
@@ -128,7 +129,11 @@ if ! gh label create "$status_label" --repo "$repo" --color "$status_color" --de
   echo "Warning: could not create $status_label" >&2
   status_synced=false
 fi
-if ! gh issue edit "$issue" --repo "$repo" --add-label "$status_label"; then
+if gh issue edit "$issue" --repo "$repo" --add-label "$status_label"; then
+  if [[ "$status_label" == shipyard:complete ]]; then
+    remove_issue_label "$issue" ready-for-human
+  fi
+else
   if [[ "$branch" == "shipyard/issue-$issue" ]]; then
     echo "Could not mark ticket #$issue complete; retry issue completion when GitHub is available" >&2
     exit 75
@@ -170,6 +175,6 @@ if [[ "$status_synced" == true ]]; then
     fi
   done
 else
-  echo "Warning: status labels need reconciliation; retaining ticket activation" >&2
+  echo "Warning: status labels need reconciliation; add shipyard to the ticket to retry" >&2
 fi
 printf '%s\n' "$url"

package/dist/templates/shared/select-issues.mjs

@@ -124,6 +124,17 @@ const dependencies = (id) => {
 };
 const planningSpec = (body) =>
   /work item type:\*?\*?\s*planning spec/i.test(body ?? "");
+const triageStates = [
+  "needs-triage",
+  "needs-info",
+  "ready-for-agent",
+  "ready-for-human",
+  "wontfix",
+];
+const triageReady = (labels) =>
+  triageStates
+    .filter((state) => labels.some((label) => label.name === state))
+    .join() === "ready-for-agent";
 const textParent = (body, issueId) => {
   const match = /^##?\s*Parent\b[^\n]*\n/im.exec(body ?? "");
   if (!match) return undefined;
@@ -476,9 +487,10 @@ for (const candidate of activated) {
         ),
     );
     selectedTicketIds = new Set(selectableTickets.map((ticket) => ticket.id));
-    const tickets = selectableTickets.map(({ labels: _labels, ...ticket }) => ({
+    const tickets = selectableTickets.map(({ labels, ...ticket }) => ({
       ...ticket,
       blockedBy: dependencies(ticket.id),
+      triageReady: triageReady(labels),
     }));
     const completedTicketIds = linkedTickets
       .filter((ticket) =>
@@ -756,7 +768,13 @@ for (const candidate of activated) {
             completedTicketIds,
             outstandingTicketIds,
           }
-        : { id, title: candidate.title, branch, kind: "standalone" },
+        : {
+            id,
+            title: candidate.title,
+            branch,
+            kind: "standalone",
+            triageReady: triageReady(candidate.labels ?? []),
+          },
     );
   } catch (error) {
     if (!(error instanceof SelectionError)) throw error;

package/dist/templates/shared/setup.sh

@@ -18,7 +18,14 @@ if [[ -f package.json ]]; then
     else manager=npm; fi
   fi
   case "$manager" in
-    npm) npm install ;;
+    npm)
+      if [[ -f package-lock.json ]]; then
+        if ! npm ci; then
+          echo "npm ci failed; installing from package.json without changing package-lock.json" >&2
+          npm install --no-package-lock
+        fi
+      else npm install --no-package-lock
+      fi ;;
     pnpm) corepack pnpm install ;;
     yarn) corepack yarn install ;;
     bun)

package/dist/templates/shared/triage-prompt.md

@@ -1,5 +1,7 @@
 Triage GitHub issue #{{TASK_ID}} before Shipyard implementation.
 
-Follow `/triage` and the repository's `docs/agents/triage.md` label mapping. Read the full issue and comments, verify its claim against the codebase, then apply the appropriate category and state label and post the brief or triage notes required by the skill. The maintainer has authorized Shipyard to apply an evidence-based recommendation during this unattended intake. If a decision requires maintainer judgment, use `needs-triage` and record the question. Do not implement the issue or start a PR in this phase.
+Follow `/triage` and the repository's triage label mapping. Read the full issue and comments. Verify the reported behavior against `origin/{{BASE_BRANCH}}`, the integration baseline. The checked-out task branch may contain unfinished changes from an earlier attempt. Do not close the issue as already complete based on the task branch or comments about that candidate.
 
-Report the applied state and evidence. Shipyard will independently re-read GitHub labels and implement only an open issue carrying both `shipyard` and the sole triage state `ready-for-agent`.
+Apply the appropriate category and state label and post the brief or triage notes required by the skill. The maintainer has authorized Shipyard to apply an evidence-based recommendation during this unattended intake. Derive acceptance criteria from the stated outcome. For example, equal spacing between named header items is a testable visual goal without a pixel target. Choose a representative supported viewport if none is specified. An inaccessible attachment does not require `needs-info` when the text establishes the current and desired behavior; record the verification limit in the brief. Do not treat equal CSS gap values alone as proof that the rendered gaps are equal. Apply `needs-info` and ask one concrete question only when missing details change the intended result or prevent meaningful verification. Use `needs-triage` only when the available facts leave a product or policy decision for the maintainer, not for routine layout or implementation choices. Preserve the `shipyard` or `shipyard:pending` label. Do not implement the issue or start a PR in this phase.
+
+Report the applied state and evidence. Shipyard will independently re-read GitHub labels and implement only an open issue carrying `shipyard` or `shipyard:pending` and the sole triage state `ready-for-agent`.

package/dist/templates/shared/verify-triage.sh

@@ -11,9 +11,18 @@ gh issue view "$issue" --repo "$repo" --json state,labels |
     const issue = JSON.parse(require("node:fs").readFileSync(0, "utf8"));
     const labels = new Set(issue.labels.map((label) => label.name));
     const states = ["needs-triage", "needs-info", "ready-for-agent", "ready-for-human", "wontfix"];
-    if (issue.state !== "OPEN" || !labels.has("shipyard") ||
-        states.filter((state) => labels.has(state)).join() !== "ready-for-agent") {
-      console.error("Issue cannot be implemented: requires an open issue with shipyard and only ready-for-agent as its triage state");
+    if (issue.state !== "OPEN" ||
+        !(labels.has("shipyard") || labels.has("shipyard:pending"))) {
+      console.error("Issue cannot be implemented: requires an open issue with shipyard or shipyard:pending");
+      process.exit(1);
+    }
+    const triageStates = states.filter((state) => labels.has(state));
+    if (triageStates.join() === "needs-info") {
+      console.error("Issue needs information from the reporter before implementation");
+      process.exit(1);
+    }
+    if (triageStates.join() !== "ready-for-agent") {
+      console.error(`Issue cannot be implemented: triage state is ${triageStates.join() || "missing"}; requires only ready-for-agent`);
       process.exit(1);
     }
   '

package/dist/templates/simple-loop/block-scope.sh

@@ -108,6 +108,10 @@ Work was not handed off. To retry, resolve the problem, remove shipyard:blocked,
 fi
 
 if [[ "$branch" == "shipyard/spec-$root" ]]; then
+  if ! remove_issue_status "$root" ready-for-human; then
+    echo "Warning: could not clear spec triage status" >&2
+    status_synced=false
+  fi
   if ! remove_issue_status "$root" shipyard:outstanding-tasks; then
     echo "Warning: could not clear spec status" >&2
     status_synced=false
@@ -122,9 +126,11 @@ else
 fi
 if [[ "$branch" == "shipyard/spec-$root" ]]; then
   for affected_id in "${affected_ids[@]:1}"; do
+    remove_issue_status "$affected_id" ready-for-human || status=75
     remove_issue_status "$affected_id" shipyard:pending || status=75
   done
 else
+  remove_issue_status "$root" ready-for-human || status=75
   remove_issue_status "$root" shipyard:pending || status=75
 fi
 

package/dist/templates/simple-loop/handoff.sh

@@ -103,6 +103,7 @@ for ((i=1; i<${#scope_ids[@]}; i++)); do
     echo "Could not mark ticket #${scope_ids[i]} complete; retry issue completion when GitHub is available" >&2
     exit 75
   fi
+  remove_issue_label "${scope_ids[i]}" ready-for-human
   remove_issue_label "${scope_ids[i]}" shipyard:blocked
   remove_issue_label "${scope_ids[i]}" shipyard:pending
 done
@@ -128,7 +129,11 @@ if ! gh label create "$status_label" --repo "$repo" --color "$status_color" --de
   echo "Warning: could not create $status_label" >&2
   status_synced=false
 fi
-if ! gh issue edit "$issue" --repo "$repo" --add-label "$status_label"; then
+if gh issue edit "$issue" --repo "$repo" --add-label "$status_label"; then
+  if [[ "$status_label" == shipyard:complete ]]; then
+    remove_issue_label "$issue" ready-for-human
+  fi
+else
   if [[ "$branch" == "shipyard/issue-$issue" ]]; then
     echo "Could not mark ticket #$issue complete; retry issue completion when GitHub is available" >&2
     exit 75
@@ -170,6 +175,6 @@ if [[ "$status_synced" == true ]]; then
     fi
   done
 else
-  echo "Warning: status labels need reconciliation; retaining ticket activation" >&2
+  echo "Warning: status labels need reconciliation; add shipyard to the ticket to retry" >&2
 fi
 printf '%s\n' "$url"

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

@@ -22,24 +22,27 @@ const roleModels = {
   routine: readRoleModel("routine"),
   strong: readRoleModel("strong"),
 };
-const CODEX_REASONING_EFFORTS = shipyard.CODEX_REASONING_EFFORTS;
-type CodexReasoningEffort = shipyard.CodexReasoningEffort;
-const readCodexReasoningEffort = (
+const REASONING_EFFORTS = shipyard.REASONING_EFFORTS;
+type ReasoningEffort = shipyard.ReasoningEffort;
+const readRoleReasoningEffort = (
   role: ModelRole,
-): CodexReasoningEffort | undefined => {
-  if (!CODEX_PROVIDER) return undefined;
-  const envName = `SHIPYARD_CODEX_${role.toUpperCase()}_REASONING_EFFORT`;
-  const effort = process.env[envName]?.trim();
+): ReasoningEffort | undefined => {
+  const envName = `SHIPYARD_${role.toUpperCase()}_REASONING_EFFORT`;
+  const legacyName = `SHIPYARD_CODEX_${role.toUpperCase()}_REASONING_EFFORT`;
+  const sharedEffort = process.env[envName]?.trim();
+  const effort =
+    sharedEffort ||
+    (CODEX_PROVIDER ? process.env[legacyName]?.trim() : undefined);
   if (!effort) return undefined;
-  if (!(CODEX_REASONING_EFFORTS as readonly string[]).includes(effort))
+  if (!(REASONING_EFFORTS as readonly string[]).includes(effort))
     throw new Error(
-      `${envName} must be one of ${CODEX_REASONING_EFFORTS.join(", ")}; received "${effort}"`,
+      `${sharedEffort ? envName : legacyName} must be one of ${REASONING_EFFORTS.join(", ")}; received "${effort}"`,
     );
-  return effort as CodexReasoningEffort;
+  return effort as ReasoningEffort;
 };
 const roleEfforts = {
-  routine: readCodexReasoningEffort("routine"),
-  strong: readCodexReasoningEffort("strong"),
+  routine: readRoleReasoningEffort("routine"),
+  strong: readRoleReasoningEffort("strong"),
 };
 const readCodexRoleModel = (role: ModelRole, defaultModel: AgentModel) => {
   if (!CODEX_PROVIDER || typeof defaultModel === "string") return defaultModel;
@@ -133,6 +136,7 @@ for (let iteration = 0; iteration < 3; iteration++) {
     title: string;
     branch: string;
     kind: "standalone" | "spec";
+    triageReady?: boolean;
     body?: string;
     tickets?: Array<{
       id: string;
@@ -140,6 +144,7 @@ for (let iteration = 0; iteration < 3; iteration++) {
       body: string;
       state: string;
       blockedBy: Array<{ id: string; title: string; state: string }>;
+      triageReady: boolean;
     }>;
     completedTicketIds?: string[];
     outstandingTicketIds?: string[];
@@ -173,24 +178,33 @@ for (let iteration = 0; iteration < 3; iteration++) {
         repository,
         "--add-label",
         "shipyard:pending",
+        "--remove-label",
+        "shipyard",
       ]);
     const sandbox = await shipyard.createSandbox({
       branch: issue.branch,
       sandbox: sandboxProvider,
       hooks,
+      copyToWorktree: [".shipyard/setup.sh"],
     });
     let evidence: string;
     try {
       for (const ticketId of issue.kind === "spec"
         ? (issue.tickets ?? []).map((ticket) => ticket.id)
         : [issue.id]) {
-        await sandbox.run({
-          name: `triage #${ticketId}`,
-          agent: roleAgent("routine", shipyard.CODEX_MODELS.routine),
-          maxIterations: 1,
-          promptFile: "./.shipyard/triage-prompt.md",
-          promptArgs: { TASK_ID: ticketId },
-        });
+        const ready =
+          issue.kind === "spec"
+            ? issue.tickets?.find((ticket) => ticket.id === ticketId)
+                ?.triageReady
+            : issue.triageReady;
+        if (!ready)
+          await sandbox.run({
+            name: `triage #${ticketId}`,
+            agent: roleAgent("routine", shipyard.CODEX_MODELS.routine),
+            maxIterations: 1,
+            promptFile: "./.shipyard/triage-prompt.md",
+            promptArgs: { TASK_ID: ticketId, BASE_BRANCH: targetBranch },
+          });
         verifyTriage(ticketId);
       }
       const result = await sandbox.run({
@@ -225,6 +239,7 @@ for (let iteration = 0; iteration < 3; iteration++) {
     const publication = await shipyard.createSandbox({
       branch: issue.branch,
       sandbox: sandboxProvider,
+      copyToWorktree: [".shipyard/handoff.sh"],
     });
     try {
       publicationUncertain = true;

package/dist/templates/simple-loop/select-issues.mjs

@@ -124,6 +124,17 @@ const dependencies = (id) => {
 };
 const planningSpec = (body) =>
   /work item type:\*?\*?\s*planning spec/i.test(body ?? "");
+const triageStates = [
+  "needs-triage",
+  "needs-info",
+  "ready-for-agent",
+  "ready-for-human",
+  "wontfix",
+];
+const triageReady = (labels) =>
+  triageStates
+    .filter((state) => labels.some((label) => label.name === state))
+    .join() === "ready-for-agent";
 const textParent = (body, issueId) => {
   const match = /^##?\s*Parent\b[^\n]*\n/im.exec(body ?? "");
   if (!match) return undefined;
@@ -476,9 +487,10 @@ for (const candidate of activated) {
         ),
     );
     selectedTicketIds = new Set(selectableTickets.map((ticket) => ticket.id));
-    const tickets = selectableTickets.map(({ labels: _labels, ...ticket }) => ({
+    const tickets = selectableTickets.map(({ labels, ...ticket }) => ({
       ...ticket,
       blockedBy: dependencies(ticket.id),
+      triageReady: triageReady(labels),
     }));
     const completedTicketIds = linkedTickets
       .filter((ticket) =>
@@ -756,7 +768,13 @@ for (const candidate of activated) {
             completedTicketIds,
             outstandingTicketIds,
           }
-        : { id, title: candidate.title, branch, kind: "standalone" },
+        : {
+            id,
+            title: candidate.title,
+            branch,
+            kind: "standalone",
+            triageReady: triageReady(candidate.labels ?? []),
+          },
     );
   } catch (error) {
     if (!(error instanceof SelectionError)) throw error;

package/dist/templates/simple-loop/setup.sh

@@ -18,7 +18,14 @@ if [[ -f package.json ]]; then
     else manager=npm; fi
   fi
   case "$manager" in
-    npm) npm install ;;
+    npm)
+      if [[ -f package-lock.json ]]; then
+        if ! npm ci; then
+          echo "npm ci failed; installing from package.json without changing package-lock.json" >&2
+          npm install --no-package-lock
+        fi
+      else npm install --no-package-lock
+      fi ;;
     pnpm) corepack pnpm install ;;
     yarn) corepack yarn install ;;
     bun)

package/dist/templates/simple-loop/triage-prompt.md

@@ -1,5 +1,7 @@
 Triage GitHub issue #{{TASK_ID}} before Shipyard implementation.
 
-Follow `/triage` and the repository's `docs/agents/triage.md` label mapping. Read the full issue and comments, verify its claim against the codebase, then apply the appropriate category and state label and post the brief or triage notes required by the skill. The maintainer has authorized Shipyard to apply an evidence-based recommendation during this unattended intake. If a decision requires maintainer judgment, use `needs-triage` and record the question. Do not implement the issue or start a PR in this phase.
+Follow `/triage` and the repository's triage label mapping. Read the full issue and comments. Verify the reported behavior against `origin/{{BASE_BRANCH}}`, the integration baseline. The checked-out task branch may contain unfinished changes from an earlier attempt. Do not close the issue as already complete based on the task branch or comments about that candidate.
 
-Report the applied state and evidence. Shipyard will independently re-read GitHub labels and implement only an open issue carrying both `shipyard` and the sole triage state `ready-for-agent`.
+Apply the appropriate category and state label and post the brief or triage notes required by the skill. The maintainer has authorized Shipyard to apply an evidence-based recommendation during this unattended intake. Derive acceptance criteria from the stated outcome. For example, equal spacing between named header items is a testable visual goal without a pixel target. Choose a representative supported viewport if none is specified. An inaccessible attachment does not require `needs-info` when the text establishes the current and desired behavior; record the verification limit in the brief. Do not treat equal CSS gap values alone as proof that the rendered gaps are equal. Apply `needs-info` and ask one concrete question only when missing details change the intended result or prevent meaningful verification. Use `needs-triage` only when the available facts leave a product or policy decision for the maintainer, not for routine layout or implementation choices. Preserve the `shipyard` or `shipyard:pending` label. Do not implement the issue or start a PR in this phase.
+
+Report the applied state and evidence. Shipyard will independently re-read GitHub labels and implement only an open issue carrying `shipyard` or `shipyard:pending` and the sole triage state `ready-for-agent`.