@sideboard-ai/core 0.1.168 → 0.1.170

package/dist/{chunk-NYMFNUT5.js → chunk-GQHVICNT.js}

@@ -33,7 +33,7 @@ import {
   summarizeTurnStderr,
   toolDescription,
   turnFailChatText
-} from "./chunk-3A54S244.js";
+} from "./chunk-UDEWGTP6.js";
 import {
   extractPresentedPlan,
   readPlanFile,
@@ -45,9 +45,7 @@ import {
 import {
   SLACK_REPLY_FORMATTING,
   addWorkspace,
-  assertOrchestratorCapableAgent,
   attachmentsFromWorktreePaths,
-  coerceOrchestratorAgent,
   coordinatorSystemPrompt,
   coordinatorTurnReminder,
   createGlobalChat,
@@ -58,7 +56,6 @@ import {
   isCloudCoordinatorThread,
   isGlobalRepoPath,
   isGlobalThread,
-  isOrchestratorCapableAgent,
   isOrchestratorThread,
   isSlackCoordinatorThread,
   orchestratorSessionPoisonedByBuiltins,
@@ -67,7 +64,7 @@ import {
   stageAbsolutePathsAsAttachments,
   stageBuffersAsAttachments,
   syncWorkspacesFromThreads
-} from "./chunk-RS756MKF.js";
+} from "./chunk-P2VCMIZU.js";
 import {
   addPrStackLayer,
   allocateTeamName,
@@ -112,10 +109,12 @@ import {
   resolveRepoRoot,
   takenSlugsFromThread,
   threadDisplayLabel,
+  ticketSlugForBranch,
   withRepoGitLock,
   worktreeDisplayLabelForGroup,
-  worktreeNameFromPath
-} from "./chunk-TW5A2HFY.js";
+  worktreeNameFromPath,
+  worktreeSlugForTicket
+} from "./chunk-Y7DJA3UW.js";
 import {
   ATTACHMENTS_DIR,
   DETACHED_JOBS_DIR,
@@ -146,12 +145,15 @@ import {
   run
 } from "./chunk-FNQNJOHY.js";
 import {
+  assertOrchestratorCapableAgent,
   childEnvWithAppSettings,
+  coerceOrchestratorAgent,
   formatViewerContextDirective,
   formatViewerContextReminder,
   getIssueSource,
   isAbleTimeConnected,
   isLinearConnected,
+  isOrchestratorCapableAgent,
   isSecureFileEncrypted,
   loadAppSettings,
   orchestrationQuotaFallbackAgent,
@@ -159,13 +161,13 @@ import {
   readSecureJson,
   resolveEffectiveIssueSource,
   resolveNewThreadOptions,
-  resolveThreadDefaults,
+  resolveOrchestratorDefaults,
   resolveVaultKey,
   resolveViewerProfileForRepo,
   stripNestedElectronEnv,
   updateAdvancedSettings,
   writeSecureJson
-} from "./chunk-LTXS26K5.js";
+} from "./chunk-YMF5WCIA.js";
 import {
   writePrivateFile
 } from "./chunk-NSTQ6QKD.js";
@@ -399,7 +401,7 @@ async function continueSourceThread(threadId, prompt) {
       await continueOnReply(threadId, prompt);
       return;
     }
-    const { getOrchestrator: getOrchestrator2 } = await import("./orchestrator-XG3QCMOS.js");
+    const { getOrchestrator: getOrchestrator2 } = await import("./orchestrator-LE6L6MJC.js");
     await getOrchestrator2().send(threadId, prompt);
   } catch {
   }
@@ -742,9 +744,9 @@ async function spawnAgentTurn(thread, input, onEvent) {
       `Cannot spawn ${thread.agent}: thread ${thread.id} has no worktreePath`
     );
   }
-  const { isGlobalThread: isGlobalThread2 } = await import("./global-workspace-JNNFVSP7.js");
+  const { isGlobalThread: isGlobalThread2 } = await import("./global-workspace-4SI3CRRT.js");
   if (isGlobalThread2(thread)) {
-    const { ensureGlobalCoordinatorCwd: ensureGlobalCoordinatorCwd2 } = await import("./coordinator-prompt-Y372QRFE.js");
+    const { ensureGlobalCoordinatorCwd: ensureGlobalCoordinatorCwd2 } = await import("./coordinator-prompt-RRUU2OYO.js");
     ensureGlobalCoordinatorCwd2(
       isOrchestratorThread(thread) ? { orchestratorThreadId: thread.id } : void 0
     );
@@ -955,6 +957,51 @@ function agentGitPrompt(action, opts) {
       return "Merge PR.";
   }
 }
+var RESOLVE_CONFLICTS_RE = /^Merge the remote branch(?: \(([^)]+)\))? into your branch and resolve conflicts\. Then, commit and push your changes\.$/;
+function gitActionExpansion(action, base) {
+  switch (action) {
+    case "commit-push":
+      return "Commit any uncommitted work with a purpose-stating message, then push to origin (`git push -u origin HEAD`; this updates an existing PR when one is linked). Do not start a checks loop unless a goal was given.";
+    case "create-draft":
+      return "Commit, push, then create a draft PR with `gh pr create --draft -R <origin-owner/name>` (title/body from the change purpose). Update the existing PR instead if one is linked.";
+    case "create-web":
+      return "Commit, push, then `gh pr create --web -R <origin-owner/name>`.";
+    case "resolve-conflicts":
+      return `Fetch the PR base${base ? ` (\`${base}\`)` : ""}, merge it into this branch, resolve conflicts carefully, then commit and push until the PR is mergeable.`;
+    case "merge":
+      return "This phrase is the explicit ask to merge this thread's open pull request on GitHub. If `gh stack view` shows a stack, use `gh stack merge`; otherwise `gh pr merge` (respect repo defaults / squash vs merge). Do not force-push main/master or merge locally into the main checkout.";
+  }
+}
+function expandCanonicalGitRequest(prompt) {
+  const trimmed = prompt.trim();
+  let action = null;
+  let base = null;
+  for (const candidate of AGENT_GIT_ACTIONS) {
+    if (candidate === "resolve-conflicts") continue;
+    if (agentGitPrompt(candidate) === trimmed) {
+      action = candidate;
+      break;
+    }
+  }
+  if (!action) {
+    const m = RESOLVE_CONFLICTS_RE.exec(trimmed);
+    if (m) {
+      action = "resolve-conflicts";
+      base = m[1] ?? null;
+    }
+  }
+  let expansion = action ? gitActionExpansion(action, base) : null;
+  if (!expansion && trimmed === "Address review comments.") {
+    expansion = "Read the PR review feedback (`gh pr view --comments`, `gh api` review comments), make the requested changes, commit, and push.";
+  } else if (!expansion && FIX_CI_RE.test(trimmed)) {
+    expansion = "Investigate that failing check (`gh pr checks`, `gh run view --log-failed`), fix it, commit, and push. Loop only if a goal to keep going until it passes was given.";
+  }
+  if (!expansion) return prompt;
+  return `${trimmed}
+
+(Sideboard git request \u2014 do not ask for clarification: ${expansion})`;
+}
+var FIX_CI_RE = /^Fix CI: .+\.$/;
 
 // src/git/pr-merge-archive.ts
 function normalizePrState(state) {
@@ -1196,47 +1243,84 @@ function formatToolPart(t, detail) {
 }
 function formatMessagesAsTranscript(messages, opts) {
   const tools = opts?.tools ?? "full";
-  const blocks = [];
-  for (const m of messages) {
-    if (m.role === "summary") {
-      blocks.push(`## Prior summary
-${m.text}`);
-      continue;
-    }
-    if (m.role === "user") {
-      blocks.push(`### User
+  const thinking = opts?.thinking ?? true;
+  return messages.map((m) => formatMessageBlock(m, { tools, thinking })).join("\n\n");
+}
+function formatMessageBlock(m, opts) {
+  if (m.role === "summary") return `## Prior summary
+${m.text}`;
+  if (m.role === "user") return `### User
+${m.text}`;
+  const bits = [];
+  if (m.text?.trim()) {
+    bits.push(`### Agent
 ${m.text}`);
-      continue;
-    }
-    const bits = [];
-    if (m.text?.trim()) {
-      bits.push(`### Agent
-${m.text}`);
-    } else {
-      bits.push("### Agent");
-    }
-    const thinking = (m.parts ?? []).filter((p) => p.type === "thinking");
-    for (const th of thinking) {
+  } else {
+    bits.push("### Agent");
+  }
+  if (opts.thinking) {
+    for (const th of m.parts ?? []) {
       if (th.type !== "thinking" || !th.text.trim()) continue;
       bits.push("Thinking:");
       bits.push(th.text);
     }
-    const toolParts = (m.parts ?? []).filter(
-      (p) => p.type === "tool"
-    );
-    if (tools !== "none" && toolParts.length > 0) {
-      if (tools === "summary") bits.push("Tools:");
-      for (const t of toolParts) {
-        bits.push(formatToolPart(t, tools));
-      }
+  }
+  const toolParts = (m.parts ?? []).filter(
+    (p) => p.type === "tool"
+  );
+  if (opts.tools !== "none" && toolParts.length > 0) {
+    if (opts.tools === "summary") bits.push("Tools:");
+    for (const t of toolParts) {
+      bits.push(formatToolPart(t, opts.tools));
     }
-    blocks.push(bits.join("\n"));
   }
-  return blocks.join("\n\n");
+  return bits.join("\n");
 }
+var SEED_MAX_CHARS = 6e4;
+var SEED_FULL_TOOL_MESSAGES = 6;
 function buildSessionSeed(messages, opts) {
   if (messages.length === 0) return null;
-  const body = formatMessagesAsTranscript(messages, { tools: opts?.tools ?? "full" });
+  const tools = opts?.tools ?? "full";
+  const fullCount = opts?.fullToolMessages ?? SEED_FULL_TOOL_MESSAGES;
+  const maxChars = opts?.maxChars ?? SEED_MAX_CHARS;
+  const olderTools = tools === "none" ? "none" : "summary";
+  const fullFrom = Math.max(0, messages.length - fullCount);
+  const blocks = messages.map(
+    (m, i) => formatMessageBlock(m, {
+      tools: i >= fullFrom ? tools : olderTools,
+      thinking: false
+    })
+  );
+  const keep = new Array(blocks.length).fill(false);
+  let used = 0;
+  for (let i = 0; i < blocks.length; i++) {
+    if (messages[i].role === "summary") {
+      keep[i] = true;
+      used += blocks[i].length + 2;
+    }
+  }
+  for (let i = blocks.length - 1; i >= 0; i--) {
+    if (keep[i]) continue;
+    const next = used + blocks[i].length + 2;
+    if (next > maxChars && used > 0) break;
+    keep[i] = true;
+    used = next;
+  }
+  const dropped = keep.filter((k) => !k).length;
+  const kept = [];
+  let markerPlaced = false;
+  for (let i = 0; i < blocks.length; i++) {
+    if (!keep[i]) continue;
+    if (dropped > 0 && !markerPlaced && messages[i].role !== "summary") {
+      kept.push(`_(${dropped} older message${dropped === 1 ? "" : "s"} omitted for length)_`);
+      markerPlaced = true;
+    }
+    kept.push(blocks[i]);
+  }
+  if (dropped > 0 && !markerPlaced) {
+    kept.push(`_(${dropped} older message${dropped === 1 ? "" : "s"} omitted for length)_`);
+  }
+  const body = kept.join("\n\n");
   if (!body.trim()) return null;
   return [
     "Sideboard conversation context (restored after compaction or a new session):",
@@ -1290,7 +1374,7 @@ function findLastBrightsyContextSummary(messages) {
 function buildBrightsySessionSeed(messages) {
   const match = findLastBrightsyContextSummary(messages);
   const tail = match ? messages.slice(match.index + 1) : messages;
-  const body = formatMessagesAsTranscript(tail, { tools: "none" });
+  const body = formatMessagesAsTranscript(tail, { tools: "none", thinking: false });
   if (!match && !body.trim()) return null;
   const blocks = [
     "Sideboard conversation context (restored after compaction or a new session):",
@@ -3359,7 +3443,7 @@ async function createThread(input, _onSetupLine) {
   });
   await requireAgent(resolved.agent);
   if (input.cowboy) {
-    const { cowboyModeEnabled } = await import("./app-settings-BRLZQA7N.js");
+    const { cowboyModeEnabled } = await import("./app-settings-LF3TS7XS.js");
     if (!cowboyModeEnabled()) {
       throw new Error(
         "Cowboy mode is off. Enable it in Settings \u2192 Advanced."
@@ -3417,7 +3501,7 @@ async function createThread(input, _onSetupLine) {
   let sourceIsFork = false;
   if (!input.cowboy && sourceType === "branch" && (!sourceRef || sourceRef === "HEAD" || sourceRef === "default")) {
     if (isAbleTimeConnected() && getIssueSource() === "abletime") {
-      const { ensureAbleTimeTask, issueAttachmentForAbleTimeTask } = await import("./abletime-BMS53RIZ.js");
+      const { ensureAbleTimeTask, issueAttachmentForAbleTimeTask } = await import("./abletime-ESVTHTFH.js");
       const task = await ensureAbleTimeTask({
         title: title?.trim() || "Untitled work",
         description: input.prompt?.trim() || void 0
@@ -3481,10 +3565,11 @@ async function createThread(input, _onSetupLine) {
     throw new Error("Use adoptThread() for adopt sources");
   }
   const team = allocateTeamSlug(repoPath);
+  const slug = sourceType === "ticket" ? worktreeSlugForTicket(persistedSourceRef, team.slug) : team.slug;
   const { branchName, worktreePath } = await createThreadWorktree({
     repoPath,
     sourceRef,
-    slug: team.slug
+    slug
   });
   copyConfiguredFiles(repoPath, worktreePath);
   const explicitTitle = title?.trim();
@@ -3924,7 +4009,7 @@ async function adoptThread(input) {
     messages: input.messages ?? []
   });
   writeThread(thread);
-  const { ensureWorkspace: ensureWorkspace2 } = await import("./workspaces-BWZSDSGG.js");
+  const { ensureWorkspace: ensureWorkspace2 } = await import("./workspaces-NOC2MWQV.js");
   await ensureWorkspace2(repoPath);
   const { ensureWorktreeSideboardIgnored: ensureWorktreeSideboardIgnored2 } = await import("./worktree-exclude-N5X5I422.js");
   await ensureWorktreeSideboardIgnored2(input.worktreePath);
@@ -5596,7 +5681,7 @@ function expandComposerPrompt(worktreePath, prompt, opts) {
 }
 
 // src/agents/instructions.ts
-import { existsSync as existsSync16, readFileSync as readFileSync12, statSync as statSync4 } from "fs";
+import { existsSync as existsSync16 } from "fs";
 import { join as join15 } from "path";
 
 // src/skills/detached-job-path.ts
@@ -5642,12 +5727,17 @@ function normPath(p) {
 function formatRenameBranchDirective(thread, opts) {
   if (!isPlaceholderBranch(thread.branchName, thread.worktreePath)) return null;
   const dir = worktreeNameFromPath(thread.worktreePath);
+  const ticket = thread.sourceType === "ticket" ? ticketSlugForBranch(thread.sourceRef ?? "") : null;
+  const example = ticket ? `\`feat/${ticket}-dark-mode\` or \`fix/${ticket}-panel-width\`` : "`fix/panel-width` or `feat/dark-mode`";
   const lines = [
     "Branch naming (do this early in the turn):",
-    `- Current branch \`${thread.branchName}\` is a temporary placeholder. The worktree folder \`${dir}\` is a stable soccer-team nickname \u2014 do not rename or leave that directory.`,
-    "- Rename the git branch to a short kebab-case name that describes this task (what you are changing), e.g. `fix/panel-width` or `feat/dark-mode`:",
+    `- Current branch \`${thread.branchName}\` is a temporary placeholder. The worktree folder \`${dir}\` is a stable nickname \u2014 do not rename or leave that directory.`,
+    `- Rename the git branch to a short kebab-case name that describes this task (what you are changing), e.g. ${example}:`,
     "  `git branch -m <new-name>`",
     "- Prefer Conventional Commits style prefixes when they fit (`fix/`, `feat/`, `chore/`, `docs/`).",
+    ...ticket ? [
+      `- Keep ticket \`${ticket}\` in the new branch name so the issue stays findable.`
+    ] : [],
     "- Never push this placeholder to main/master."
   ];
   const custom = opts?.customPrompt?.trim();
@@ -5681,67 +5771,38 @@ function formatWorktreeDirective(thread, opts) {
   lines.push(
     "- Always use this worktree's `origin` remote (`git remote get-url origin` from this cwd). Never push to or open PRs against `upstream` (template remotes)."
   );
-  lines.push(
-    "- Push with `git push -u origin HEAD` (or the current branch name). Do not `git push upstream`."
-  );
-  lines.push("");
   lines.push(formatGitAuthModeDirective(opts?.gitAuthMode ?? "auto"));
   lines.push(
-    "- Derive the PR title and body from what the changes actually do and why \u2014 inspect the diff/commits and the user request. Do not use the soccer-team worktree nickname or placeholder branch as the PR title."
+    "- PR title/body and commit messages state what the change does and why (inspect the diff/commits and the user request) \u2014 never the worktree nickname or placeholder branch. Concise imperative title (Conventional Commits when it fits: feat:/fix:/chore:/docs:); body = intent, key changes, test notes."
   );
   lines.push(
-    "- Prefer a concise imperative title (Conventional Commits style when it fits: feat:/fix:/chore:/docs:). Body should summarize intent, key changes, and test notes."
-  );
-  lines.push(
-    "- Commit with messages that state the purpose of the change (same standard as the PR). Stay on this thread branch. Never push directly to main/master or merge locally into the main checkout. Do not merge the PR unless this turn is a Merge PR request or the user explicitly asked. When merging, use GitHub from this worktree (`gh pr merge` / `gh stack merge`)."
+    "- Stay on this thread branch. Never push to main/master or merge locally into the main checkout. Merge the PR only when this turn is a Merge PR request or the user explicitly asked \u2014 via GitHub from this worktree (`gh pr merge` / `gh stack merge`)."
   );
   if (thread.prUrl) {
     lines.push(
       `- A PR already exists (${thread.prUrl}). Update it (push + edit title/body if the purpose drifted) instead of opening a duplicate.`
     );
-  } else if (opts?.githubSlug) {
-    lines.push(
-      `- Prefer a draft PR first: \`gh pr create --draft -R ${opts.githubSlug}\` (or update via \`gh pr edit -R ${opts.githubSlug}\`) once the change set is coherent. Always pass \`-R ${opts.githubSlug}\` (this worktree's origin). Bare \`gh pr create\` may target upstream instead of origin on dual-remote checkouts. Mark ready for review only when asked. Title/body must reflect the change purpose, not the worktree name. If GitHub rejects the GraphQL body as too long, retry with a short \`--body-file\` (limit 65,536 characters) \u2014 do not paste a changelog or diff into \`--body\`.`
-    );
   } else {
+    const slug = opts?.githubSlug ?? "<origin-owner/name>";
+    const resolve = opts?.githubSlug ? "" : " Resolve `<origin-owner/name>` with `git remote get-url origin` in this worktree.";
     lines.push(
-      "- Prefer a draft PR first: `gh pr create --draft -R <origin-owner/name>` (or update via `gh pr edit -R \u2026`) once the change set is coherent. Resolve `<origin-owner/name>` with `git remote get-url origin` in this worktree \u2014 never from `upstream`. Mark ready for review only when asked. Title/body must reflect the change purpose, not the worktree name. If GitHub rejects the GraphQL body as too long, retry with a short `--body-file` (limit 65,536 characters) \u2014 do not paste a changelog or diff into `--body`."
+      `- Prefer a draft PR first: \`gh pr create --draft -R ${slug}\` (update via \`gh pr edit -R ${slug}\`). Always pass \`-R\` \u2014 bare \`gh pr create\` may target upstream on dual-remote checkouts.${resolve} Mark ready for review only when asked. If GitHub rejects the body as too long, retry with a short \`--body-file\` \u2014 do not paste a changelog or diff into \`--body\`.`
     );
   }
-  lines.push("");
   lines.push(
-    "Short git requests from the Sideboard UI are complete instructions \u2014 expand them using the rules above without asking for clarification:"
-  );
-  lines.push(
-    '- "Commit and push." \u2192 commit any uncommitted work with a purpose-stating message, then push to origin (updates an existing PR if one is linked). Do not start a checks loop unless they gave a goal.'
-  );
-  lines.push(
-    '- "Commit, push, and open a draft PR." \u2192 commit, push, then create a draft PR with `gh pr create --draft -R \u2026` (title/body from the change purpose).'
-  );
-  lines.push(
-    '- "Commit, push, and open a PR in the browser." \u2192 commit, push, then `gh pr create --web -R \u2026`.'
-  );
-  lines.push(
-    '- "Fix CI: <name>." \u2192 investigate that failing check, fix it, commit, and push. Loop only if they gave a goal to keep going until it passes.'
-  );
-  lines.push(
-    '- "Merge the remote branch (<base>) into your branch and resolve conflicts. Then, commit and push your changes." \u2192 fetch the PR base, merge it into this branch, resolve conflicts carefully, commit, and push until the PR is mergeable.'
-  );
-  lines.push(
-    '- "Address review comments." \u2192 read PR review feedback, make the requested changes, commit, and push.'
-  );
-  lines.push(
-    '- "Merge PR." \u2192 merge this thread\'s open pull request on GitHub (this phrase is the explicit ask). If `gh stack view` shows a stack, use `gh stack merge`; otherwise `gh pr merge` (respect repo defaults / squash vs merge). Do not force-push main/master or merge locally into the main checkout.'
+    '- Sideboard git buttons send short phrases ("Commit and push.", "Merge PR.", \u2026) with their meaning attached \u2014 act on them without asking for clarification. If the user gives a goal (Greptile 5/5, CI green, until checks pass), you get the watch-fix-push playbook with that request.'
   );
   lines.push("");
-  lines.push(formatPrGateDirective());
-  lines.push("");
-  lines.push(formatProcessGuideDirective());
+  lines.push(formatProcessGuideDirective({ worktreePath: thread.worktreePath }));
   return lines.join("\n");
 }
 function formatWorktreeReminder() {
   return "Sideboard worktree: stay in this cwd for all file and git work. Push and open PRs against origin, never upstream. Do not edit the main repo checkout. If a goal is given (Greptile 5/5, CI green), watch-fix-push until it lands \u2014 do not watch after every push.";
 }
+var PR_GOAL_RE = /greptile|\bci\b.*\b(green|pass)|checks?\s+(pass|green)|until\s+.*\b(pass|green|merge|clean)|\bfix ci\b|watch[- ]fix[- ]push/i;
+function mentionsPrGoal(prompt) {
+  return PR_GOAL_RE.test(prompt);
+}
 var GITHUB_TICKET_REF = /^(?:#?\d+|gh-\d+)$/i;
 var KEYED_TICKET_REF = /^(?:[A-Z][A-Z0-9]{0,9}-\d+|[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12})$/i;
 function issueTicketFromThread(thread, preferredSource = "github") {
@@ -5810,29 +5871,27 @@ function formatPrGateDirective() {
     "- Greptile 5/5: wait for the greptile check; read the latest `N/5` from the PR body or Greptile\u2019s most recently *updated* issue comment; also fetch unresolved review comments / the \u201CPrompt to fix all with AI\u201D block. Done = **5/5 and zero unresolved Greptile comments**. If Greptile is idle, comment `@greptile review`. Cap about 5 iterations, then report leftovers. Do not merge unless asked."
   ].join("\n");
 }
-function formatProcessGuideDirective() {
+function formatProcessGuideDirective(opts) {
+  const worktree = opts?.worktreePath?.trim();
+  const hasGraphSkill = !!worktree && existsSync16(join15(worktree, ".claude/skills/graph-engineering/SKILL.md"));
+  const hasReviewSkill = !!worktree && existsSync16(join15(worktree, ".claude/skills/review/SKILL.md"));
   return [
     "Process guides (recurring work only):",
-    "- Long jobs (pack, test, deploy, anything that may run more than ~30s): Sideboard always provides `/long-running`. Detach and wait; `stop_job` if hanging or wrong. Do not ask the human to poll.",
-    "- If `.claude/skills/graph-engineering/SKILL.md` exists, follow it (`/graph-engineering`) for migrations, ports, batch fixes, and other fan-out. Judge first; state on disk; grow the rulebook; do not patch around it.",
-    "- If this same shape of work will happen again, write `.claude/skills/<kebab-name>/SKILL.md` in this worktree (Claude Code project skill). Sideboard `/name`, Claude Code, and `attach` all load that path. Do not leave the method only in chat.",
-    "- Merge-readiness notes: if `.claude/skills/review/SKILL.md` already exists, edit it. Otherwise write them to `.context/review.md` (copied from `.sideboard/review.md` when that file exists). Do not create a review skill.",
-    "- Do not write new skills under `.sideboard/skills/` (that folder only \u2014 other `.sideboard/` files such as settings.toml are fine). Point Codex/OpenCode at the Claude skill from `AGENTS.md`. Optional: symlink `.cursor/skills/<name>` to the Claude skill.",
+    hasGraphSkill ? "- Migrations, ports, batch fixes, and other fan-out: follow `.claude/skills/graph-engineering/SKILL.md` (`/graph-engineering`). Judge first; state on disk; grow the rulebook; do not patch around it." : null,
+    "- If this same shape of work will happen again, write `.claude/skills/<kebab-name>/SKILL.md` in this worktree (Claude Code project skill; Sideboard `/name`, Claude Code, and `attach` all load it). Not under `.sideboard/skills/`. Point Codex/OpenCode at it from `AGENTS.md`.",
+    hasReviewSkill ? "- Merge-readiness notes go in `.claude/skills/review/SKILL.md` (edit it; do not create another review skill)." : "- Merge-readiness notes go in `.context/review.md` (copied from `.sideboard/review.md` when that file exists). Do not create a review skill.",
     "- Skip a guide for a one-off. If a matching skill exists, follow it. Same miss twice \u2192 edit the skill, do not patch around it."
-  ].join("\n");
+  ].filter(Boolean).join("\n");
 }
 function formatLongRunningDirective(opts) {
   const invoke = formatDetachedJobInvoke(opts?.scriptPath);
   return [
-    "Long-running jobs (mandatory when a command may run more than ~30s):",
-    "A Sideboard worktree turn SIGTERMs the agent shell (and its process group) when the user sends another message or the turn is interrupted. `block_until_ms: 0` is not enough. Do not ask the human to poll.",
-    `Helper (same tool as \`scripts/detached-job.cjs\` when that file exists in the worktree): \`${invoke}\``,
-    `- Start once: \`${invoke} start <id> -- <command> [args...]\` (cwd = this worktree). If JSON says already-running, do not start again.`,
-    "- Immediately `present_artifact` `type=log` with `artifact_id=<id>` and `status=running` \u2014 the side column is the live view.",
-    `- Loop Sideboard MCP \`wait_for_job\` with the same id (returns in ~45s). stillRunning \u2192 present the same id with \`content=delta\` only \u2192 wait_for_job again. Shell fallback: \`${invoke} wait <id>\`. Do not resend the full log or HTML.`,
-    `- If it is hanging, producing no useful output, or doing the wrong thing: MCP \`stop_job\` (same id, optional reason) or \`${invoke} stop <id>\`. Do not loop forever. Do not stop a pack/test/deploy that is clearly making progress.`,
-    "- Do not end the turn with \u201CI\u2019ll let you know when it\u2019s done.\u201D Stay in the loop until stillRunning is false (or you stopped it).",
-    "- ok \u2192 finish the task. failed / stopped \u2192 read the log, fix or narrow the command, start once.",
+    "Long-running jobs (mandatory when a command may run more than ~30s \u2014 pack, test, deploy, `gh pr checks --watch`):",
+    "A Sideboard worktree turn SIGTERMs the agent shell (and its process group) when the user sends another message or the turn is interrupted; `block_until_ms: 0` is not enough. Detach instead, and never ask the human to poll.",
+    `- Start once: \`${invoke} start <id> -- <command> [args...]\` (cwd = this worktree; same tool as \`scripts/detached-job.cjs\` when the worktree has it). If JSON says already-running, do not start again.`,
+    "- Immediately `present_artifact` `type=log` with `artifact_id=<id>` and `status=running`, then loop Sideboard MCP `wait_for_job` (returns in ~45s). stillRunning \u2192 present the same id with `content=delta` only \u2192 wait again. Shell fallback: the same helper with `wait <id>`.",
+    "- Hanging, no useful output, or the wrong thing \u2192 `stop_job` (or the helper with `stop <id>`). Do not stop a pack/test/deploy that is clearly making progress.",
+    "- Stay in the loop until stillRunning is false (or you stopped it). ok \u2192 finish the task. failed / stopped \u2192 read the log, fix or narrow the command, start once.",
     "State: `.context/.sideboard/detached-jobs/<id>/` (local scratch). Full guide: `/long-running` (always available)."
   ].join("\n");
 }
@@ -5842,27 +5901,11 @@ function formatLongRunningReminder(opts) {
 }
 function formatArtifactDirective() {
   return [
-    "Sideboard side column (desktop UI):",
-    "claude.ai\u2019s \u201CArtifact\u201D tool does NOT exist in Claude Code. That is expected.",
-    "Do not unprompted-duplicate a payload. Chat markdown (including tables) already renders in the transcript \u2014 do not also call present_schema with those same rows just to display them. If the user asks for an editable / interactive table, call present_schema even if markdown already showed the data. Do not also call present_artifact for a document you already fenced in chat.",
-    "Documents (HTML/SVG/markdown):",
-    "1) Emit a fenced code block tagged `html` (preferred), `svg`, or `markdown` with the FULL document \u2014 Sideboard opens a side column. Example:",
-    "```html",
-    "<!DOCTYPE html><html><head><title>Demo</title></head><body><h1>Hi</h1></body></html>",
-    "```",
-    "2) Or call Sideboard MCP `present_artifact` with title, type (html|svg|markdown|react|log), and content \u2014 not both a fence and this tool for the same body.",
-    "   type=log is append-only: same `artifact_id`, `content` = new lines only (plus optional status/phase). Do not resend the full log or wrap it in HTML.",
-    "CMS / JSON Schema forms & tables (Brightsy or any schema+schemaUi source):",
-    "3) Call Sideboard MCP `present_schema` when the user needs to filter, edit, publish, or persist rows \u2014 including after you already showed a markdown table, if they then ask for an editable table. If they only need to read the data, a markdown table is enough. When you do call it, pass title, mode (table|form), and either:",
-    "   - datasource=brightsy + resource_id (record type UUID) after fetching types via Brightsy MCP, or",
-    "   - datasource=inline + resource: { id, title, schema, schemaUi } and optional records/record.",
-    "Files / media browser (CMS file manager column):",
-    "4) Call Sideboard MCP `present_files` with optional title, path, and datasource (brightsy|memory).",
-    "   Opens the Files column for browse/upload/pick. Do NOT say a file manager UI is missing.",
-    "Multiple-choice questions:",
-    "5) Call Sideboard MCP `ask_user` only when work is blocked on choosing among a few concrete options (approach forks, which API, auth vs cookies). First write a short chat message that explains the decision and what each option means (tradeoffs, when to pick it). Include a description on every option. After calling, stop and wait for their next message with answers. If you are asking a real multiple-choice, use ask_user rather than chat bullets so Sideboard shows the composer picker.",
-    "Do not call ask_user for greetings, check-ins, \u201Chello\u201D, open-ended how-can-I-help, or to invent a menu of possible next tasks \u2014 reply in chat. If one option is the obvious default, proceed without asking.",
-    "Never say artifacts, CMS UI, or the Files column are unavailable. present_schema is for interactive list/edit/publish (use it when they ask to edit, even if chat already had a markdown table); present_files for storage UI; html fences for standalone pages; ask_user only for those blocked predefined-option questions."
+    "Sideboard side column (desktop UI) \u2014 Sideboard MCP tools present_artifact, present_schema, present_files, ask_user (see their descriptions). claude.ai\u2019s \u201CArtifact\u201D tool does not exist here; never say artifacts, the CMS UI, or the Files column are unavailable.",
+    "- Standalone documents: a fenced block tagged `html` (preferred), `svg`, or `markdown` containing the FULL document opens the side column by itself. Use that or present_artifact \u2014 never both for the same body.",
+    "- present_artifact type=log appends: same artifact_id, content = new lines only (plus status/phase). Do not resend the full log or wrap it in HTML.",
+    "- Data: a markdown table is enough to read. Call present_schema only when the user needs to filter/edit/publish/persist rows \u2014 including when they ask for an editable table after you already showed markdown. Never re-present rows you already wrote just to display them.",
+    "- ask_user only when work is blocked on a few concrete options (approach fork, which API, auth vs cookies): first a short chat message explaining the decision and each option, then the call (description on every option), then stop and wait. Not for greetings, check-ins, or an invented menu of next tasks \u2014 reply in chat. If one option is the obvious default, proceed."
   ].join("\n");
 }
 function formatUiReminder() {
@@ -6004,7 +6047,7 @@ async function syncThreadBranchFromGit(threadId) {
 
 // src/store/schedules.ts
 import { randomUUID as randomUUID4 } from "crypto";
-import { existsSync as existsSync17, readFileSync as readFileSync13 } from "fs";
+import { existsSync as existsSync17, readFileSync as readFileSync12 } from "fs";
 import { join as join16 } from "path";
 import { Cron } from "croner";
 var DURATION_RE = /^(\d+)(s|m|h|d)$/i;
@@ -6117,7 +6160,7 @@ function readAll() {
   const path = schedulesPath();
   if (!existsSync17(path)) return [];
   try {
-    const parsed = JSON.parse(readFileSync13(path, "utf8"));
+    const parsed = JSON.parse(readFileSync12(path, "utf8"));
     if (!Array.isArray(parsed)) return [];
     return parsed.map((row) => {
       try {
@@ -6240,7 +6283,7 @@ function formatScheduledPrompt(name, prompt) {
 ${prompt}`;
 }
 async function defaultDeps() {
-  const { getOrchestrator: getOrchestrator2, startOrchestration: startOrchestration2 } = await import("./orchestrator-XG3QCMOS.js");
+  const { getOrchestrator: getOrchestrator2, startOrchestration: startOrchestration2 } = await import("./orchestrator-LE6L6MJC.js");
   const orch = getOrchestrator2();
   return {
     findThread: (id) => findThreadByRef(id),
@@ -6250,7 +6293,7 @@ async function defaultDeps() {
 }
 function pickAgent(schedule) {
   if (schedule.agent) return schedule.agent;
-  return coerceOrchestratorAgent(resolveThreadDefaults().agent);
+  return coerceOrchestratorAgent(resolveOrchestratorDefaults().agent);
 }
 async function fireSchedule(id) {
   const schedule = getSchedule(id);
@@ -6502,7 +6545,7 @@ var Orchestrator = class {
           orphans: orphans.map((o) => ({ path: o.path, repoPath: o.repoPath }))
         });
       }
-      const { autoCleanupOrphansEnabled } = await import("./app-settings-BRLZQA7N.js");
+      const { autoCleanupOrphansEnabled } = await import("./app-settings-LF3TS7XS.js");
       if (autoCleanupOrphansEnabled() && shouldRunWorktreeCleanup() && orphans.length > 0) {
         await cleanupOrphanWorktrees({ repoPaths });
       }
@@ -6781,7 +6824,7 @@ var Orchestrator = class {
     }
     await setup;
     if (skipSetup) return;
-    const { autoRunAfterSetupEnabled } = await import("./app-settings-BRLZQA7N.js");
+    const { autoRunAfterSetupEnabled } = await import("./app-settings-LF3TS7XS.js");
     if (autoRunAfterSetupEnabled()) {
       try {
         await this.startDev(threadId);
@@ -7064,17 +7107,21 @@ var Orchestrator = class {
     const slackReplyContext = formatSlackRepliesForTurn(
       pendingSlackExternalReplies(thread.messages)
     );
-    const agentPrompt = [
-      thread.planMode ? PLAN_MODE_INSTRUCTION : null,
-      orchestrationReminder,
+    const turnReminders = [
       worktreeReminder,
       optionalServicesReminder,
       issueToolsReminder,
       viewerContextReminder,
       artifactReminder,
-      longRunningReminder,
+      longRunningReminder
+    ].filter(Boolean).join("\n\n") || void 0;
+    const prGateDirective = thread.agent !== "brightsy" && !isOrchestratorThread(thread) && mentionsPrGoal(expandedPrompt) ? formatPrGateDirective() : null;
+    const agentPrompt = [
+      thread.planMode ? PLAN_MODE_INSTRUCTION : null,
+      orchestrationReminder,
+      prGateDirective,
       slackReplyContext,
-      expandedPrompt
+      expandCanonicalGitRequest(expandedPrompt)
     ].filter(Boolean).join("\n\n");
     if (thread.attachments.length > 0) {
       updateThread(threadId, { attachments: [] });
@@ -7092,7 +7139,7 @@ var Orchestrator = class {
     }
     const isBrightsy = fresh.agent === "brightsy";
     const isOrchestration = isOrchestratorThread(fresh);
-    const { autoRenameBranchEnabled, getGithubGitAuthMode } = await import("./app-settings-BRLZQA7N.js");
+    const { autoRenameBranchEnabled, getGithubGitAuthMode } = await import("./app-settings-LF3TS7XS.js");
     const gitAuthMode = getGithubGitAuthMode();
     const worktreeDirective = isBrightsy || isOrchestration ? null : formatWorktreeDirective(fresh, {
       githubSlug: await resolveGithubRepoSlug(fresh.worktreePath).catch(
@@ -7164,7 +7211,7 @@ var Orchestrator = class {
       const stderrTail = [];
       const handle = await spawnAgentTurn(
         fresh,
-        { cachedPrefix, prompt: agentPrompt },
+        { cachedPrefix, prompt: agentPrompt, systemPrompt: turnReminders },
         (event) => {
           this.emit({ type: "turn_output", threadId, event });
           noteTurnLiveEvent(threadId, event);
@@ -7273,7 +7320,7 @@ var Orchestrator = class {
         ].filter(Boolean).join("\n\n---\n\n");
         const retryHandle = await spawnAgentTurn(
           retryThread,
-          { cachedPrefix: retryPrefix, prompt: agentPrompt },
+          { cachedPrefix: retryPrefix, prompt: agentPrompt, systemPrompt: turnReminders },
           (event) => {
             this.emit({ type: "turn_output", threadId, event });
             if (event.type === "session_id") {
@@ -8004,7 +8051,7 @@ var Orchestrator = class {
         updateThread(thread.id, { title: meta.title });
       }
     }
-    const { autoArchiveOnMergeEnabled } = await import("./app-settings-BRLZQA7N.js");
+    const { autoArchiveOnMergeEnabled } = await import("./app-settings-LF3TS7XS.js");
     const latest = this.requireThread(thread.id);
     if (!shouldAutoArchiveOnPrMerge({
       previousPrState: prevState || null,
@@ -8321,7 +8368,7 @@ var Orchestrator = class {
     this.emit({ type: "status_changed", threadId: archived.id, status: "archived" });
     if (thread.repoPath && !isGlobalRepoPath(thread.repoPath)) {
       try {
-        const { ensureWorkspace: ensureWorkspace2 } = await import("./workspaces-BWZSDSGG.js");
+        const { ensureWorkspace: ensureWorkspace2 } = await import("./workspaces-NOC2MWQV.js");
         await ensureWorkspace2(thread.repoPath);
       } catch {
       }
@@ -8348,7 +8395,7 @@ var Orchestrator = class {
           await runArchiveScript(thread.repoPath, thread.worktreePath);
         } catch {
         }
-        const { deleteBranchOnPurgeEnabled } = await import("./app-settings-BRLZQA7N.js");
+        const { deleteBranchOnPurgeEnabled } = await import("./app-settings-LF3TS7XS.js");
         const deleteBranch = opts?.deleteBranch ?? deleteBranchOnPurgeEnabled();
         await removeWorktree(thread.repoPath, thread.worktreePath, {
           deleteBranch: deleteBranch ? thread.branchName : void 0
@@ -8375,7 +8422,7 @@ var Orchestrator = class {
           `Cowboy checkout missing: ${thread.worktreePath}. Re-add the project folder, then restore.`
         );
       }
-      const { createThreadWorktree: createThreadWorktree2 } = await import("./worktree-7X7XNSYL.js");
+      const { createThreadWorktree: createThreadWorktree2 } = await import("./worktree-GXO2VDQF.js");
       const slug = thread.worktreePath.split("/").pop();
       const dest = thread.worktreePath;
       await withRepoGitLock(thread.repoPath, async () => {
@@ -8485,7 +8532,7 @@ async function startOrchestration(opts) {
     sourceRef: "default",
     ...createOpts
   }).catch(async () => {
-    const { resolveDefaultBranch: resolveDefaultBranch2, resolveRepoRoot: resolveRepoRoot2 } = await import("./worktree-7X7XNSYL.js");
+    const { resolveDefaultBranch: resolveDefaultBranch2, resolveRepoRoot: resolveRepoRoot2 } = await import("./worktree-GXO2VDQF.js");
     const repo = await resolveRepoRoot2(repoPath);
     const def = await resolveDefaultBranch2(repo);
     return orch.createThread({