opencode-ship 1.0.0 → 1.1.0

package/dist/cli.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-// opencode-ship CLI v1.0.0
+// opencode-ship CLI v1.1.0
 var __defProp = Object.defineProperty;
 var __getOwnPropNames = Object.getOwnPropertyNames;
 var __esm = (fn, res) => function __init() {
@@ -516,8 +516,8 @@ var init_root_config = __esm({
 });
 
 // src/profile.js
-var PROFILES = Object.freeze(["core", "engineering"]);
-var DEFAULT_PROFILE = "core";
+var PROFILES = Object.freeze(["engineering"]);
+var DEFAULT_PROFILE = "engineering";
 function isValidProfile(name) {
   return typeof name === "string" && PROFILES.includes(name);
 }
@@ -526,19 +526,27 @@ function normalizeProfile(name) {
   if (!isValidProfile(name)) return null;
   return name;
 }
+function isLegacyCoreProfile(name) {
+  return name === "core";
+}
 function resolveProfile({ cli = null, config = null, lock = null } = {}) {
   if (cli !== null && cli !== void 0) {
     const v = normalizeProfile(cli);
     if (v === null) {
-      throw new Error(`unknown CLI profile '${cli}' (expected one of: ${PROFILES.join(", ")})`);
+      throw new Error(
+        `unknown CLI profile '${cli}' (only 'engineering' is supported in 1.1.0; the 'core' profile was removed)`
+      );
     }
     return { profile: v, source: "cli" };
   }
   if (config && typeof config === "object" && config.profile !== void 0 && config.profile !== null) {
     const v = normalizeProfile(config.profile);
     if (v === null) {
+      if (isLegacyCoreProfile(config.profile)) {
+        return { profile: DEFAULT_PROFILE, source: "default" };
+      }
       throw new Error(
-        `unknown ship.config.json profile '${config.profile}' (expected one of: ${PROFILES.join(", ")})`
+        `unknown ship.config.json profile '${config.profile}' (only 'engineering' is supported in 1.1.0)`
       );
     }
     return { profile: v, source: "config" };
@@ -546,8 +554,11 @@ function resolveProfile({ cli = null, config = null, lock = null } = {}) {
   if (lock && typeof lock === "object" && lock.manager && lock.manager.profile !== void 0) {
     const v = normalizeProfile(lock.manager.profile);
     if (v === null) {
+      if (isLegacyCoreProfile(lock.manager.profile)) {
+        return { profile: DEFAULT_PROFILE, source: "default" };
+      }
       throw new Error(
-        `unknown lock manager.profile '${lock.manager.profile}' (expected one of: ${PROFILES.join(", ")})`
+        `unknown lock manager.profile '${lock.manager.profile}' (only 'engineering' is supported in 1.1.0)`
       );
     }
     return { profile: v, source: "lock" };
@@ -559,7 +570,7 @@ function resolveProfile({ cli = null, config = null, lock = null } = {}) {
 var USAGE = `opencode-ship <command> [options]
 
 Commands:
-  init        Install or update managed files in this project.
+  init        Install managed files in this project. One-liner: pnpm dlx opencode-ship@latest init
   diff        Show what would change without writing.
   update      Apply pending updates after recovering the journal.
   doctor      Validate environment, lock, and references.
@@ -569,16 +580,19 @@ Commands:
 
 Options:
   --root <path>               Project root (defaults to cwd).
-  --profile <name>            Override active profile: ${PROFILES.join(", ")}.
+  --profile engineering       Override active profile (engineering only).
   --force-config              Rewrite the user config from detection (init only).
   --force-root-config         Create opencode.json when absent (init only).
   --strict-doctor             Fail init when doctor reports unhealthy checks.
   --replace-managed           Replace locally-modified managed files (update only).
   --purge-config              Remove ship.config.json when uninstalling.
-  --planner-model <id>        Engineering model id for the strong planner.
-  --builder-model <id>        Engineering model id for the cheap builder.
-  --final-reviewer-model <id> Engineering model id for the Standards + Spec reviewer.
+  --planner-model <id>        Strong planner model id (init only, optional).
+  --builder-model <id>        Cheap builder model id (init only, optional).
+  --final-reviewer-model <id> Final Standards + Spec reviewer model id (init only, optional).
   --json                      Emit a JSON envelope instead of human output.
+
+After init succeeds, restart OpenCode and run /setup-ship-workflow to
+fill in the workflow.models fields and the per-repo docs.
 `;
 var MODEL_ID_RE = /^[a-zA-Z0-9_.-]+\/[a-zA-Z0-9_.-]+$/;
 function parseFlags(argv) {
@@ -609,6 +623,11 @@ function parseFlags(argv) {
       if (value === void 0) {
         return { error: "--profile requires a value" };
       }
+      if (value === "core") {
+        return {
+          error: "the 'core' profile was removed in opencode-ship 1.1.0; only 'engineering' is supported. Run /setup-ship-workflow to migrate."
+        };
+      }
       if (!isValidProfile(value)) {
         return { error: `unknown profile '${value}' (expected one of: ${PROFILES.join(", ")})` };
       }
@@ -654,6 +673,11 @@ function helpText() {
   return USAGE;
 }
 
+// src/installer/commands/init.js
+import { promisify as promisify2 } from "node:util";
+import { writeFile as writeFile7, mkdir as mkdirAsync2 } from "node:fs/promises";
+import { dirname as dirname9, resolve as resolvePath } from "node:path";
+
 // src/installer/executor.js
 import { existsSync as existsSync13 } from "node:fs";
 import { mkdir as mkdir5, readFile as readFile8, rename as rename5, unlink as unlink3, writeFile as writeFile5 } from "node:fs/promises";
@@ -689,7 +713,7 @@ function resolvePackageRoot(startUrl) {
 import { readFileSync as readFileSync2, existsSync as existsSync2 } from "node:fs";
 import { dirname as dirname2, resolve as resolve2 } from "node:path";
 import { fileURLToPath as fileURLToPath2 } from "node:url";
-var PACKAGE_VERSION = "1.0.0";
+var PACKAGE_VERSION = "1.1.0";
 var TEMPLATE_SET = `v${PACKAGE_VERSION}`;
 
 // src/installer/catalog.js
@@ -743,7 +767,7 @@ var CATALOG = [
     path: ".opencode/plugins/opencode-ship.js",
     source: resolve3(packageRoot, "dist/plugin.js"),
     mode: 420,
-    profiles: ["core", "engineering"]
+    profiles: ["engineering"]
   },
   {
     id: "agent:delivery-reviewer",
@@ -751,7 +775,7 @@ var CATALOG = [
     path: ".opencode/agents/delivery-reviewer.md",
     source: resolve3(packageRoot, "assets/agents/delivery-reviewer.md"),
     mode: 420,
-    profiles: ["core", "engineering"]
+    profiles: ["engineering"]
   },
   {
     id: "agent:delivery-verifier",
@@ -759,7 +783,7 @@ var CATALOG = [
     path: ".opencode/agents/delivery-verifier.md",
     source: resolve3(packageRoot, "assets/agents/delivery-verifier.md"),
     mode: 420,
-    profiles: ["core", "engineering"]
+    profiles: ["engineering"]
   },
   ...ENGINEERING_AGENTS.map((name) => ({
     id: `agent:${name}`,
@@ -783,7 +807,7 @@ var CATALOG = [
     path: ".opencode/skills/delivery-workflow/SKILL.md",
     source: resolve3(packageRoot, "assets/skills/delivery-workflow/SKILL.md"),
     mode: 420,
-    profiles: ["core", "engineering"]
+    profiles: ["engineering"]
   },
   {
     id: "skill:planning-research-checkpoint",
@@ -791,7 +815,7 @@ var CATALOG = [
     path: ".opencode/skills/planning-research-checkpoint/SKILL.md",
     source: resolve3(packageRoot, "assets/skills/planning-research-checkpoint/SKILL.md"),
     mode: 420,
-    profiles: ["core", "engineering"]
+    profiles: ["engineering"]
   },
   ...MATT_SKILLS.map((name) => ({
     id: `skill:matt:${name}`,
@@ -808,7 +832,31 @@ var CATALOG = [
     source: resolve3(packageRoot, `assets/skills/${name}/SKILL.md`),
     mode: 420,
     profiles: ["engineering"]
-  }))
+  })),
+  {
+    id: "skill:setup-ship-workflow",
+    kind: "skill",
+    path: ".opencode/skills/setup-ship-workflow/SKILL.md",
+    source: resolve3(packageRoot, "assets/skills/setup-engineering-workflow/SKILL.md"),
+    mode: 420,
+    profiles: ["engineering"]
+  },
+  {
+    id: "skill:skill-discovery",
+    kind: "skill",
+    path: ".opencode/skills/skill-discovery/SKILL.md",
+    source: resolve3(packageRoot, "assets/skills/skill-discovery/SKILL.md"),
+    mode: 420,
+    profiles: ["engineering"]
+  },
+  {
+    id: "command:setup-ship-workflow",
+    kind: "support",
+    path: ".opencode/commands/setup-ship-workflow.md",
+    source: resolve3(packageRoot, "assets/commands/setup-ship-workflow.md"),
+    mode: 420,
+    profiles: ["engineering"]
+  }
 ];
 function filterCatalogByProfile(catalog, profile) {
   const effective = profile === void 0 || profile === null ? DEFAULT_PROFILE : profile;
@@ -913,8 +961,8 @@ var ship_config_schema_default = {
     schemaVersion: { enum: [1, 2] },
     profile: {
       type: "string",
-      enum: ["core", "engineering"],
-      description: "Active profile (precedence layer 2: ship.config > lock > core default)."
+      enum: ["engineering"],
+      description: "Active profile. Engineering is the only supported profile in 1.1.0."
     },
     owner: {
       type: "string",
@@ -1043,13 +1091,13 @@ var ship_config_schema_default = {
     },
     workflow: {
       type: "object",
-      description: "Required when profile is engineering. Ignored when profile is core.",
+      description: "Workflow configuration. Models are optional at write time; the setup-ship-workflow skill fills them in. Once all three are present, ship-deliver can start.",
       additionalProperties: false,
       properties: {
         models: {
           type: "object",
           additionalProperties: false,
-          required: ["planner", "builder", "finalReviewer"],
+          description: "Optional model roles. All three roles must be present before ship-deliver can run.",
           properties: {
             planner: {
               type: "string",
@@ -1078,31 +1126,7 @@ var ship_config_schema_default = {
         }
       }
     }
-  },
-  allOf: [
-    {
-      if: {
-        required: ["profile"],
-        properties: { profile: { const: "engineering" } }
-      },
-      then: {
-        required: ["workflow"],
-        properties: {
-          workflow: {
-            required: ["models", "approval"],
-            properties: {
-              models: {
-                required: ["planner", "builder", "finalReviewer"]
-              },
-              approval: {
-                required: ["mirrorToIssue", "maxFailedRounds"]
-              }
-            }
-          }
-        }
-      }
-    }
-  ]
+  }
 };
 
 // src/installer/validation.js
@@ -1241,7 +1265,8 @@ function renderDefaultConfig(detection, overrides = {}) {
   const safeVerification = Array.isArray(detection?.verificationPlan) && detection.verificationPlan.length ? detection.verificationPlan.map((step) => ({ id: step.id, argv: step.argv })) : [{ id: "typecheck", argv: ["npm", "run", "typecheck"] }];
   const repo = detection?.repository ?? overrides.repository ?? "owner/repo";
   return {
-    schemaVersion: 1,
+    schemaVersion: 2,
+    profile: "engineering",
     project: {
       remote: detection?.remote ?? "origin",
       repository: repo,
@@ -1270,9 +1295,19 @@ function renderDefaultConfig(detection, overrides = {}) {
       ready: { requires: ["review", "local-verification", "remote-ci"], stopAfterReady: true },
       merge: { strategy: "squash", policy: "explicit-user-request-only", requireFreshGates: true },
       cleanup: { when: "next-task", requireUnpublishedGuard: true }
+    },
+    workflow: {
+      models: {},
+      approval: { mirrorToIssue: true, maxFailedRounds: 3 }
     }
   };
 }
+function hasCompletedModels(configValue) {
+  const models = configValue?.workflow?.models;
+  if (!models || typeof models !== "object") return false;
+  const idRe = /^[A-Za-z0-9_.-]+\/[A-Za-z0-9_.-]+$/;
+  return typeof models.planner === "string" && idRe.test(models.planner) && typeof models.builder === "string" && idRe.test(models.builder) && typeof models.finalReviewer === "string" && idRe.test(models.finalReviewer);
+}
 
 // src/installer/planner.js
 init_json_pointer();
@@ -2801,6 +2836,10 @@ function legacyToShipConfig(legacy, detection = null) {
       ready: legacy.ready ?? { requires: ["review", "local-verification", "remote-ci"], stopAfterReady: true },
       merge: legacy.merge ?? { strategy: "squash", policy: "explicit-user-request-only", requireFreshGates: true },
       cleanup: legacy.cleanup && typeof legacy.cleanup === "object" && "when" in legacy.cleanup ? legacy.cleanup : { when: "next-task", requireUnpublishedGuard: true }
+    },
+    workflow: {
+      models: {},
+      approval: { mirrorToIssue: true, maxFailedRounds: 3 }
     }
   };
 }
@@ -2845,14 +2884,7 @@ async function previewInstall({ rootPath, profile = null, replaceManaged, forceC
     });
     if (candidate.kind === "create" || candidate.kind === "update") {
       const configValue2 = candidate.configValue;
-      if (!configValue2.workflow || !configValue2.workflow.models) {
-        return { ok: false, error: { kind: "engineering-models-required", message: "engineering profile requires workflow.models.{planner,builder,finalReviewer}" } };
-      }
-      const { planner, builder, finalReviewer } = configValue2.workflow.models;
-      if (!planner || !builder || !finalReviewer) {
-        return { ok: false, error: { kind: "engineering-models-required", message: "all three workflow.models.* are required" } };
-      }
-      if (!configValue2.workflow.approval || !configValue2.workflow.approval.mirrorToIssue || configValue2.workflow.approval.maxFailedRounds !== 3) {
+      if (!configValue2.workflow || !configValue2.workflow.approval) {
         return { ok: false, error: { kind: "engineering-approval-required", message: "engineering profile requires workflow.approval.{mirrorToIssue:true, maxFailedRounds:3}" } };
       }
     }
@@ -2866,7 +2898,8 @@ async function previewInstall({ rootPath, profile = null, replaceManaged, forceC
     detection,
     lock,
     forceOverwrite: Boolean(forceConfig),
-    migrationSeed: migrationReport?.proposedConfigSeed ?? null
+    migrationSeed: migrationReport?.proposedConfigSeed ?? null,
+    models
   });
   const filePlan = await planFileInstall({ repoRoot, lock, allowUnowned: Boolean(replaceManaged), catalog: activeCatalog });
   const staleFilePlan = await planStaleFileRemoval({ repoRoot, lock, staleCatalog });
@@ -2878,6 +2911,7 @@ async function previewInstall({ rootPath, profile = null, replaceManaged, forceC
   });
   const planMode = resolved.profile === "engineering" ? { id: "/agent/plan/permission", block: planModePermissions().build, scope: "engineering" } : null;
   const rootPlan = await planRootConfigApply({ repoRoot, lock, forceRepair: Boolean(forceRootConfig), planMode });
+  const setupPending = resolved.profile === "engineering" && !lock?.manager?.setupComplete && !hasCompletedModels(configValue?.workflow?.models ?? {}) && !models?.planner;
   const plan = [...filePlan ?? [], ...staleFilePlan, ...migrationPlan, configPlan, rootPlan];
   const conflicts = plan.filter((p) => p && p.kind === "conflict");
   const summary = summarise(plan);
@@ -2892,7 +2926,8 @@ async function previewInstall({ rootPath, profile = null, replaceManaged, forceC
     plan,
     conflicts,
     summary,
-    migrationReport
+    migrationReport,
+    setupPending
   };
 }
 async function previewUninstall({ rootPath }) {
@@ -2927,7 +2962,7 @@ function summarise(plan) {
   }
   return counts;
 }
-async function assembleLock({ repoRoot, plan, lock, configPlan, rootPlan, profile = null }) {
+async function assembleLock({ repoRoot, plan, lock, configPlan, rootPlan, profile = null, models = null }) {
   const files = [];
   const remain = lock?.files?.filter((f) => !plan.some((op) => op?.relPath === f.path)) ?? [];
   for (const op of plan) {
@@ -2955,18 +2990,18 @@ async function assembleLock({ repoRoot, plan, lock, configPlan, rootPlan, profil
   const rootPointers = rootPlan?.pointerRecords ?? lock?.manager?.rootDocuments?.[0]?.pointers ?? [];
   const hasRootPlan = Boolean(rootPlan?.target || rootPlan?.pointerRecords && rootPlan.pointerRecords.length > 0);
   const hasRootDocuments = rootPlan?.pointerRecords && rootPlan.pointerRecords.length > 0 || lock?.manager?.rootDocuments && lock.manager.rootDocuments.length > 0;
+  const configModels = configPlan?.configValue?.workflow?.models ?? lock?.manager?.config?.models ?? {};
+  const completedModels = hasCompletedModels({ workflow: { models: configModels } });
   return {
     contractVersion: CURRENT_LOCK_SCHEMA,
     manager: {
       schemaVersion: CURRENT_LOCK_SCHEMA,
       name: "opencode-ship",
-      version: "1.0.0",
+      version: "1.1.0",
       templateSet: TEMPLATE_SET_ID,
-      // Newly written locks always carry the resolved profile so
-      // future CLI invocations without --profile still resolve to
-      // the same choice through the lock-precedence layer.
       profile: profile ?? lock?.manager?.profile,
       appliedAt: (/* @__PURE__ */ new Date()).toISOString(),
+      setupComplete: completedModels,
       config: {
         path: ".opencode/ship.config.json",
         sha256: configSha ?? lock?.manager?.config?.sha256 ?? "",
@@ -3383,7 +3418,35 @@ async function runDoctor({ rootPath, profile, json, writeOutput = true }) {
   return { issues, exitCode, plan, checks, profile: resolved };
 }
 
+// src/installer/setup-pending.js
+import { existsSync as existsSync15, readFileSync as readFileSync6, unlinkSync, writeFile as writeFile6, mkdir as mkdirAsync } from "node:fs";
+import { promisify } from "node:util";
+import { resolve as resolve14, dirname as dirname8 } from "node:path";
+var writeFileAsync = promisify(writeFile6);
+var mkdirAsyncAsync = promisify(mkdirAsync);
+var REL_PATH = ".opencode/ship.setup-pending.json";
+function setupPendingPath(repoRoot) {
+  return resolve14(repoRoot, REL_PATH);
+}
+async function writeSetupPending(repoRoot, payload) {
+  const path = setupPendingPath(repoRoot);
+  await mkdirAsyncAsync(dirname8(path), { recursive: true });
+  await writeFileAsync(path, JSON.stringify(payload, null, 2) + "\n", "utf8");
+}
+function clearSetupPending(repoRoot) {
+  const path = setupPendingPath(repoRoot);
+  if (!existsSync15(path)) return false;
+  try {
+    unlinkSync(path);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
 // src/installer/commands/init.js
+var writeFileAsync2 = promisify2(writeFile7);
+var mkdirAsyncAsync2 = promisify2(mkdirAsync2);
 async function runInit(options) {
   try {
     validateCatalog();
@@ -3393,21 +3456,23 @@ async function runInit(options) {
     }
     throw e;
   }
-  const preview = await previewInstall({
-    rootPath: options.rootPath ?? null,
-    profile: options.profile ?? null,
-    replaceManaged: false,
-    forceConfig: Boolean(options.forceConfig),
-    forceRootConfig: Boolean(options.forceRootConfig),
-    models: options.models ?? null
-  });
+  let preview;
+  try {
+    preview = await previewInstall({
+      rootPath: options.rootPath ?? null,
+      profile: options.profile ?? null,
+      replaceManaged: false,
+      forceConfig: Boolean(options.forceConfig),
+      forceRootConfig: Boolean(options.forceRootConfig),
+      models: options.models ?? null
+    });
+  } catch (e) {
+    return emitFailure(2, e?.message ?? "invalid input", options.json, "init");
+  }
   if (!preview.ok) {
     if (preview.error?.kind === "unsupported-lock-schema") {
       return emitFailure(5, `unsupported lock schema: ${(preview.error.issues ?? []).join("; ")}`, options.json, "init");
     }
-    if (preview.error?.kind === "engineering-models-required") {
-      return emitFailure(2, preview.error.message, options.json, "init");
-    }
     if (preview.error?.kind === "engineering-approval-required") {
       return emitFailure(2, preview.error.message, options.json, "init");
     }
@@ -3438,6 +3503,14 @@ async function runInit(options) {
       exitCode = 1;
     }
   }
+  const setupPending = Boolean(preview.setupPending);
+  if (setupPending && preview.repoRoot) {
+    await writeSetupPending(preview.repoRoot, {
+      profile: preview.profile?.profile ?? "engineering",
+      reason: "workflow.models is empty; run /setup-ship-workflow to fill in model roles",
+      createdAt: (/* @__PURE__ */ new Date()).toISOString()
+    });
+  }
   if (options.json) {
     const envelope = {
       reportVersion: 1,
@@ -3449,19 +3522,56 @@ async function runInit(options) {
       diagnostics: committed.diagnostics ?? [],
       doctor: doctor.issues ?? [],
       doctorChecks: doctor.checks ?? [],
+      setupPending,
       exitCode
     };
     Object.assign(envelope, committed.extra ?? {}, { doctor: doctor.issues ?? [] });
     process.stdout.write(JSON.stringify(envelope, null, 2) + "\n");
-  } else if (exitCode !== 0) {
-    process.stdout.write(`opencode-ship: doctor reported ${doctor.issues.length} unhealthy check(s)
-`);
   } else {
-    process.stdout.write(`opencode-ship: installed; doctor OK
-`);
+    printHumanResult({
+      prefix: "opencode-ship",
+      exitCode,
+      doctorIssues: doctor.issues,
+      setupPending
+    });
   }
   process.exitCode = exitCode;
-  return { ok: exitCode === 0, exitCode };
+  return { ok: exitCode === 0, exitCode, setupPending };
+}
+function printHumanResult({ prefix, exitCode, doctorIssues, setupPending }) {
+  const lines = [];
+  if (exitCode === 0) {
+    lines.push(`${prefix}: installed; doctor OK`);
+  } else {
+    lines.push(`${prefix}: installed with warnings`);
+  }
+  if (Array.isArray(doctorIssues) && doctorIssues.length > 0) {
+    lines.push("");
+    lines.push("Doctor reported:");
+    for (const issue of doctorIssues) lines.push(`  - ${issue}`);
+  }
+  if (setupPending) {
+    lines.push("");
+    lines.push("NEXT:");
+    lines.push("  1. Restart OpenCode in this repo (if you haven't already).");
+    lines.push("  2. In chat, run: /setup-ship-workflow");
+    lines.push("     (or type: continue ship setup)");
+    lines.push("  3. The skill will ask for:");
+    lines.push("       - issue tracker (GitHub / GitLab / local / other)");
+    lines.push("       - triage labels (defaults are fine)");
+    lines.push("       - domain docs (single-context default)");
+    lines.push("       - AI model roles (planner / builder / finalReviewer)");
+    lines.push("  4. After setup, try: Ship issue <number>");
+    lines.push("");
+    lines.push("The controller will refuse to dispatch until setup is complete.");
+  } else {
+    lines.push("");
+    lines.push("NEXT:");
+    lines.push("  1. Restart OpenCode in this repo.");
+    lines.push("  2. Run: opencode-ship doctor && to confirm everything is clean.");
+    lines.push("  3. Run: Ship issue <number>     (or /setup-ship-workflow to customise first)");
+  }
+  process.stdout.write(lines.join("\n") + "\n");
 }
 function emitFailure(code, message, json, command) {
   if (json) {
@@ -3606,11 +3716,15 @@ async function runUpdate(options) {
     }
     throw e;
   }
+  const hasExplicitModels = options.models && (options.models.planner || options.models.builder || options.models.finalReviewer);
   const preview = await previewInstall({
     rootPath: options.rootPath,
     profile: options.profile ?? null,
     replaceManaged: options.replaceManaged,
-    forceConfig: options.forceConfig,
+    // If the user passes model flags, the run is explicitly
+    // populating workflow.models. We must rewrite the config in
+    // that case even when the existing config is otherwise valid.
+    forceConfig: Boolean(options.forceConfig || hasExplicitModels),
     forceRootConfig: options.forceRootConfig,
     models: options.models ?? null
   });
@@ -3627,6 +3741,9 @@ async function runUpdate(options) {
     return emitFailure2(3, "modified managed files; rerun with --replace-managed", options.json, "update");
   }
   const committed = await commitInstall(preview, { json: options.json, command: "update" });
+  if (committed.extra?.exitCode === 0 && preview.repoRoot && !preview.setupPending) {
+    clearSetupPending(preview.repoRoot);
+  }
   if (options.json) {
     process.stdout.write(JSON.stringify({
       reportVersion: 1,
@@ -3637,6 +3754,7 @@ async function runUpdate(options) {
       summary: committed.summary ?? {},
       diagnostics: committed.diagnostics ?? [],
       exitCode: committed.extra?.exitCode ?? 0,
+      setupPending: Boolean(preview.setupPending),
       ...committed.extra ?? {}
     }, null, 2) + "\n");
   } else if (committed.extra?.exitCode === 0) {

package/dist/core.js

@@ -1,4 +1,4 @@
-// opencode-ship/core v1.0.0
+// opencode-ship/core v1.1.0
 
 // src/adapter.js
 import { readFile, writeFile, mkdir, rename } from "node:fs/promises";