repo-harness 0.24.2 → 0.24.3

package/README.md

@@ -585,8 +585,8 @@ repo-harness commit scripts or hooks unless that repo adopts the same policy.
 
 ## Current Release
 
-- npm package: `repo-harness@0.24.2`
-- Generated workflow stamp: `repo-harness@0.24.2+template@0.24.2`
+- npm package: `repo-harness@0.24.3`
+- Generated workflow stamp: `repo-harness@0.24.3+template@0.24.3`
 - GitHub repository: `Ancienttwo/repo-harness`
 - Release notes and history: [`docs/CHANGELOG.md`](docs/CHANGELOG.md)
 

package/assets/skill-version.json

@@ -1,6 +1,6 @@
 {
-  "version": "0.24.2",
-  "templateVersion": "0.24.2",
+  "version": "0.24.3",
+  "templateVersion": "0.24.3",
   "skillName": "repo-harness",
   "contractId": "tasks-first-harness-v1",
   "compatibility": {

package/dist/hook-entry.js

@@ -3220,6 +3220,8 @@ function ownerLiveness(pid, startTime) {
     if (error.code === "ESRCH")
       return "dead";
   }
+  if (startTime === null)
+    return "unverified";
   if (lastLiveOwner !== null && lastLiveOwner.pid === pid && lastLiveOwner.startTime === startTime && Date.now() - lastLiveOwner.checkedAt < LIVE_OWNER_RECHECK_MS)
     return "live";
   let observed;
@@ -9634,8 +9636,8 @@ import { readFileSync as readFileSync17 } from "fs";
 import { join as join27 } from "path";
 var PACKAGE_JSON_URL = new URL("../../../package.json", import.meta.url);
 function runningHarnessVersion() {
-  if ("0.24.2".length > 0) {
-    return "0.24.2";
+  if ("0.24.3".length > 0) {
+    return "0.24.3";
   }
   const manifest = JSON.parse(readFileSync17(PACKAGE_JSON_URL, "utf-8"));
   if (typeof manifest.version !== "string" || manifest.version.length === 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "repo-harness",
-  "version": "0.24.2",
+  "version": "0.24.3",
   "description": "Installs, migrates, audits, and repairs repo-local agentic development harnesses",
   "license": "MIT",
   "repository": {

package/src/effects/locking/exclusive-directory-lock.ts

@@ -90,15 +90,22 @@ export interface ExclusiveDirectoryLockHandle {
 export interface ExclusiveDirectoryLockOwner {
   readonly lockPath: string;
   readonly pid: number;
-  /** OS process start time of the owner; see readProcessStartTime. */
-  readonly startTime: string;
+  /**
+   * OS process start time of the owner; see readProcessStartTime. It is null
+   * when the record has none: records published before start times were
+   * recorded (#934), such as the lock a 0.24.1 parent holds during `update`.
+   * Without it, a running owner is never proven live, and the record is never
+   * reclaimed.
+   */
+  readonly startTime: string | null;
   readonly token: string;
 }
 
 /**
  * `dead` means no process now runs as the recorded owner: the PID is free, or
  * a process with a different start time has it. `unverified` means the PID is
- * in use but the process table could not be read.
+ * in use but the process table could not be read, or the record has no start
+ * time to compare.
  */
 export type ExclusiveDirectoryLockOwnerLiveness = 'live' | 'dead' | 'unverified';
 
@@ -142,12 +149,14 @@ function ownerTokenFromFileName(entry: string): { readonly token: string; readon
 
 let lastLiveOwner: { readonly pid: number; readonly startTime: string; readonly checkedAt: number } | null = null;
 
-function ownerLiveness(pid: number, startTime: string): ExclusiveDirectoryLockOwnerLiveness {
+function ownerLiveness(pid: number, startTime: string | null): ExclusiveDirectoryLockOwnerLiveness {
   try {
     process.kill(pid, 0);
   } catch (error) {
     if ((error as NodeJS.ErrnoException).code === 'ESRCH') return 'dead';
   }
+  // Without a start time, a live PID may be a reuse, so it is not proven live.
+  if (startTime === null) return 'unverified';
   // Waiters poll every 10 ms. Reuse of a recently verified owner's PID needs
   // that owner to exit first, so the table read runs at most once per interval.
   if (lastLiveOwner !== null
@@ -166,7 +175,7 @@ function ownerLiveness(pid: number, startTime: string): ExclusiveDirectoryLockOw
 }
 
 /** Only a proven dead owner is stale; an unverified owner keeps the lock. */
-function ownerIsAlive(pid: number, startTime: string): boolean {
+function ownerIsAlive(pid: number, startTime: string | null): boolean {
   return ownerLiveness(pid, startTime) !== 'dead';
 }
 
@@ -342,7 +351,7 @@ export function readExclusiveDirectoryLockOwner(
     return null;
   }
   const startTime = recordedStartTime(parsed);
-  if (parsed.pid !== owner.pid || parsed.token !== owner.token || startTime === null) return null;
+  if (parsed.pid !== owner.pid || parsed.token !== owner.token) return null;
   assertLockAncestors(location);
   const currentDirectory = lstatSync(location.lockPath);
   const currentOwner = lstatSync(ownerPath);