@osfactory/har 1.10.1 → 1.12.0

package/dist/index.js

@@ -4519,7 +4519,9 @@ var init_schema = __esm({
       version: external_exports.number().int().default(1),
       createdAt: external_exports.string(),
       updatedAt: external_exports.string(),
-      outcome: WorkUnitOutcomeSchema.optional()
+      outcome: WorkUnitOutcomeSchema.optional(),
+      /** Outcomes cleared by a later `launch --work-id` on the same unit (#352), oldest first. */
+      previousOutcomes: external_exports.array(WorkUnitOutcomeSchema).optional()
     }).passthrough();
     WorkAttemptRecordSchema = external_exports.object({
       version: external_exports.number().int().default(1),
@@ -37319,11 +37321,6 @@ function upsertWorkUnit(harnessRoot, input) {
   ensureWorkEvidenceIgnored(harnessRoot);
   const existing = findWorkUnit(harnessRoot, metadata.workUnitId);
   if (existing) {
-    if (existing.outcome) {
-      throw new Error(
-        `Work unit ${metadata.workUnitId} is already ${existing.outcome.decision}; reopening is not supported`
-      );
-    }
     for (const field of ["source", "sourceUrl", "title", "parentWorkUnitId"]) {
       if (metadata[field] !== void 0 && existing[field] !== void 0 && metadata[field] !== existing[field]) {
         throw new Error(
@@ -37349,7 +37346,10 @@ function upsertWorkUnit(harnessRoot, input) {
     version: 1,
     createdAt: existing?.createdAt ?? now,
     updatedAt: now,
-    outcome: existing?.outcome
+    // A new launch on a decided unit reopens it (#352): a teardown or complete
+    // closed one attempt, not the issue. The old outcome stays on the record.
+    outcome: void 0,
+    previousOutcomes: existing?.outcome ? [...existing.previousOutcomes ?? [], existing.outcome] : existing?.previousOutcomes
   });
   writeRecord2(
     hashedRecordPath(harnessRoot, WORK_UNITS_DIR, metadata.workUnitId),
@@ -40781,6 +40781,35 @@ var init_worktree_ownership = __esm({
   }
 });
 
+// src/runtime/slot-database.ts
+function planSlotDatabaseSteps(env3, processManager) {
+  if (processManager === "simulator") return [];
+  const steps = [];
+  if (env3.HARNESS_DB_MIGRATE_CMD) {
+    steps.push({
+      envVar: "HARNESS_DB_MIGRATE_CMD",
+      label: `Applying database schema: ${env3.HARNESS_DB_MIGRATE_CMD}`,
+      command: env3.HARNESS_DB_MIGRATE_CMD,
+      what: "migrate"
+    });
+  }
+  if (env3.HARNESS_DB_SEED_CMD && !infraEnabled(env3, "db")) {
+    steps.push({
+      envVar: "HARNESS_DB_SEED_CMD",
+      label: `Running seeds: ${env3.HARNESS_DB_SEED_CMD}`,
+      command: env3.HARNESS_DB_SEED_CMD,
+      what: "seed"
+    });
+  }
+  return steps;
+}
+var init_slot_database = __esm({
+  "src/runtime/slot-database.ts"() {
+    "use strict";
+    init_infra();
+  }
+});
+
 // src/runtime/hooks.ts
 function readHookHarnessEnv(harnessDir) {
   const envPath = path37.join(harnessDir, "harness.env");
@@ -41203,13 +41232,13 @@ async function launchSession(options) {
       ], { env: { ...process.env, ...env3 } });
       if (bootstrap.code !== 0) throw new Error(`minimal data bootstrap failed (${bootstrap.code})`);
     }
-    if (pm !== "simulator" && env3.HARNESS_DB_MIGRATE_CMD) {
-      log(`Applying database schema: ${env3.HARNESS_DB_MIGRATE_CMD}`);
-      const migrate = exec("bash", [
+    for (const step of planSlotDatabaseSteps(env3, pm)) {
+      log(step.label);
+      const result = exec("bash", [
         "-c",
-        `cd ${JSON.stringify(session.workDir)} && set -a && . ${JSON.stringify(session.envFile)} && set +a && eval "$HARNESS_DB_MIGRATE_CMD"`
+        `cd ${JSON.stringify(session.workDir)} && set -a && . ${JSON.stringify(session.envFile)} && set +a && eval "$${step.envVar}"`
       ], { env: { ...process.env, ...env3 } });
-      if (migrate.code !== 0) throw new Error(`database migrate command failed (${migrate.code})`);
+      if (result.code !== 0) throw new Error(`database ${step.what} command failed (${result.code})`);
     }
     let simName = "";
     let simUdid = "";
@@ -41386,6 +41415,7 @@ var init_launch = __esm({
     init_worktree_ownership();
     init_exec();
     init_infra();
+    init_slot_database();
     init_worktree();
     init_agent_env();
     init_provision();
@@ -86163,6 +86193,7 @@ Run \`har env doctor\` for the full report.
     const doctorSummary = summarizeDoctorReport(doctor);
     if (doctorSummary) launchBanner += `WARN: ${doctorSummary}
 `;
+    const launchNotices = [];
     if (isTelemetryEnabled() && process.env.NODE_ENV !== "test") {
       const ensured = await ensureTelemetryInfrastructure({ startIfNeeded: true });
       if (ensured.message) {
@@ -86202,6 +86233,7 @@ Usage from Cursor / Claude / Codex (@osfactory/otel-hook) will appear under Work
           blocked: true
         };
       }
+      const priorOutcome = findWorkUnit(harnessRoot, workUnitId)?.outcome;
       upsertWorkUnit(harnessRoot, {
         workUnitId,
         source: options.source,
@@ -86210,6 +86242,12 @@ Usage from Cursor / Claude / Codex (@osfactory/otel-hook) will appear under Work
         parentWorkUnitId: options.parentWorkUnitId,
         relatedLinks: options.relatedLinks
       });
+      if (priorOutcome) {
+        const reason = priorOutcome.reason ? `: ${priorOutcome.reason}` : "";
+        launchNotices.push(
+          `Reopened work unit ${workUnitId} (${priorOutcome.decision} ${priorOutcome.decidedAt}${reason})`
+        );
+      }
       createWorkAttempt(harnessRoot, {
         attemptId,
         workUnitId,
@@ -86276,11 +86314,16 @@ Usage from Cursor / Claude / Codex (@osfactory/otel-hook) will appear under Work
           });
         }
       }
+      if (launchNotices.length) {
+        launchBanner += launchNotices.map((notice) => `NOTE: ${notice}
+`).join("");
+      }
       if (launchBanner) {
         envResult.stderr = `${launchBanner}${envResult.stderr ?? ""}`;
       }
-      if (guard.readiness?.warnings?.length) {
-        envResult.warnings = guard.readiness.warnings;
+      const warnings = [...guard.readiness?.warnings ?? [], ...launchNotices];
+      if (warnings.length) {
+        envResult.warnings = warnings;
       }
     }
     return envResult;

package/dist/templates/har-boilerplate/harness.env

@@ -50,8 +50,13 @@ export HARNESS_ECOSYSTEM="auto"
 # list the ones this project needs here (e.g. "db redis mailpit").
 export HARNESS_INFRA_SERVICES="db"
 
-# Database setup commands (run once against template DB in setup-infra.sh).
-# Empty = not configured; setup-infra.sh skips the step.
+# Database setup commands. Both run from the work dir with the slot env sourced.
+# - MIGRATE runs once against the Postgres template DB (when "db" is in
+#   HARNESS_INFRA_SERVICES) and again per slot at launch, so schema drift after a
+#   code change surfaces as a launch error, not runtime 500s.
+# - SEED runs once against the Postgres template DB (slots clone it), or once per
+#   slot at launch for file-backed databases such as SQLite (#345).
+# Empty = not configured; the step is skipped.
 export HARNESS_DB_MIGRATE_CMD=""
 export HARNESS_DB_SEED_CMD=""
 

package/dist/templates/har-boilerplate-cli/harness.env

@@ -38,8 +38,13 @@ export HARNESS_ECOSYSTEM="auto"
 # compose file, then list the ones this project needs here (e.g. "db redis").
 export HARNESS_INFRA_SERVICES=""
 
-# Database setup commands (run once against template DB in setup-infra.sh).
-# Empty = not configured; setup-infra.sh skips the step.
+# Database setup commands. Both run from the work dir with the slot env sourced.
+# - MIGRATE runs once against the Postgres template DB (when "db" is in
+#   HARNESS_INFRA_SERVICES) and again per slot at launch, so schema drift after a
+#   code change surfaces as a launch error, not runtime 500s.
+# - SEED runs once against the Postgres template DB (slots clone it), or once per
+#   slot at launch for file-backed databases such as SQLite (#345).
+# Empty = not configured; the step is skipped.
 export HARNESS_DB_MIGRATE_CMD=""
 export HARNESS_DB_SEED_CMD=""
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@osfactory/har",
-  "version": "1.10.1",
+  "version": "1.12.0",
   "description": "HAR: open agent harness CLI and MCP server for coding agents, isolated worktrees, and software-factory workflows",
   "keywords": [
     "har",