@awak-app/simy-cli 0.7.16 → 0.7.17

package/src/bounded-local-task-subtask-pool.js

@@ -58,6 +58,7 @@ export function createBoundedLocalTaskSubtaskPool({
   prepareWorkspace = prepareLocalTaskWorkspace,
   leaseSeconds = DEFAULT_LEASE_SECONDS,
   heartbeatIntervalMs = DEFAULT_HEARTBEAT_INTERVAL_MS,
+  now = Date.now,
   setIntervalImpl = setInterval,
   clearIntervalImpl = clearInterval,
   onError = () => {},
@@ -111,10 +112,12 @@ export function createBoundedLocalTaskSubtaskPool({
     let completionConfirmed = false;
     let completionState = null;
     let providerStarted = false;
+    let knownLeaseExpiry = Number.NaN;
     const startedAt = Date.now();
     try {
       normalized = normalizeDurableLocalTaskSubtaskClaim(claim, {
         parentAuthority: parent,
+        nowMs: now(),
         leaseSeconds,
       });
       if (normalized.disposition !== "execute") {
@@ -129,6 +132,12 @@ export function createBoundedLocalTaskSubtaskPool({
       }
       batchId ??= normalized.batch.batch_id;
 
+      knownLeaseExpiry = normalized.attempt.leaseExpiresAtMs;
+      const checkLease = async () => {
+        if (!controller.signal.aborted && !completionConfirmed && now() >= knownLeaseExpiry) await cancel("lease_lost");
+        return !controller.signal.aborted;
+      };
+
       const execution = boundedLocalTaskSubtaskExecution(normalized, parent);
       const runtime = await materializeSubtaskRuntime({
         normalized,
@@ -140,7 +149,7 @@ export function createBoundedLocalTaskSubtaskPool({
         downloadSource: sourceDownloader,
         signal: controller.signal,
       });
-      if (controller.signal.aborted) return;
+      if (!await checkLease()) return;
 
       const fence = durableLocalTaskSubtaskFence(normalized, parent);
       let boundaryConfirmed = false;
@@ -161,7 +170,7 @@ export function createBoundedLocalTaskSubtaskPool({
         }).catch(onError);
         return;
       }
-      if (boundaryConfirmed !== true || controller.signal.aborted) {
+      if (boundaryConfirmed !== true || !await checkLease()) {
         await api.abandon_local_task_subtask_attempt_v2({
           ...fence,
           p_reason_code: controller.signal.aborted
@@ -174,6 +183,8 @@ export function createBoundedLocalTaskSubtaskPool({
 
       const heartbeat = async () => {
         if (controller.signal.aborted || completionConfirmed) return;
+        const renewalStartedAt = now();
+        if (!await checkLease()) return;
         try {
           const response =
             await api.heartbeat_local_task_subtask_attempt_v2({
@@ -190,6 +201,7 @@ export function createBoundedLocalTaskSubtaskPool({
           ) {
             return;
           }
+          if (!await checkLease()) return;
           if (
             response?.ok !== true ||
             response.cancel_requested === true ||
@@ -198,6 +210,8 @@ export function createBoundedLocalTaskSubtaskPool({
                 normalized.parent.cancel_generation)
           ) {
             await cancel("server_cancelled");
+          } else {
+            knownLeaseExpiry = renewalStartedAt + leaseSeconds * 1_000;
           }
         } catch (error) {
           if (completionConfirmed && isKnownSubtaskFenceError(error)) return;
@@ -214,9 +228,11 @@ export function createBoundedLocalTaskSubtaskPool({
         }
       };
       heartbeatTimer = setIntervalImpl(() => {
-        if (heartbeatInFlight) return;
-        heartbeatInFlight = heartbeat().finally(() => {
-          heartbeatInFlight = null;
+        void checkLease().then((valid) => {
+          if (!valid || heartbeatInFlight || completionConfirmed) return;
+          heartbeatInFlight = heartbeat().finally(() => {
+            heartbeatInFlight = null;
+          });
         });
       }, heartbeatIntervalMs);
 
@@ -245,6 +261,13 @@ export function createBoundedLocalTaskSubtaskPool({
           ),
         );
       }
+      // Lease loss after the paid boundary cannot establish a cancelled result.
+      // Preserve a known result, but leave interrupted work for server resolution.
+      if (providerStarted && controller.signal.reason === "lease_lost" &&
+          rawResult?.status === "cancelled") {
+        rawResult = { ...rawResult, status: "unknown",
+          failure_code: "local_task_subtask_provider_result_unknown" };
+      }
       const result = normalizeDurableLocalTaskSubtaskResult(
         normalizeRunResult(rawResult, startedAt),
         { normalizedClaim: normalized },
@@ -305,12 +328,15 @@ export function createBoundedLocalTaskSubtaskPool({
       }
     } finally {
       if (heartbeatTimer) clearIntervalImpl(heartbeatTimer);
-      if (heartbeatInFlight) await heartbeatInFlight;
+      // Pending renewal has its own fenced completion handler; it must not
+      // hold a finished/cancelled slot open after its timer is cleared.
     }
   };
 
   const scheduler = async () => {
     if (!initialized) {
+      // Initialization only acknowledges the canonical plan. Backend keeps
+      // existing attempts/reservations intact; claim reconciles old generations.
       const initializedBatch =
         await api.initialize_local_task_subtask_batch_v2({
           p_turn_id: parent.turnId,

package/src/bounded-local-task-subtasks-contract.js

@@ -1043,7 +1043,7 @@ function uploadedAttachmentRefs(value, name) {
   return refs;
 }
 
-function normalizeParentUploadedRef(item, name) {
+export function normalizeParentUploadedRef(item, name = "uploaded source") {
   record(item, name);
   const allowed = new Set([
     "schema_version",

package/src/cli-contract.js

@@ -13,7 +13,7 @@ import {
 } from "./durable-local-task-steps-contract.js";
 import { buildExecutionCapabilityContract } from "./execution-capability-contract.js";
 
-export const CLI_VERSION = "0.7.16";
+export const CLI_VERSION = "0.7.17";
 export const CLI_API_CONTRACT_VERSION = 10;
 
 export function narrativeImageCapabilityMetadata() {

package/src/codex-plugins/simy/.codex-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "simy",
   "version": "0.1.0",
-  "description": "SIMY quality and PR review. Delivery uses the explicitly requested CLI Agentic Loop.",
+  "description": "SIMY quality and PR review, with automatic checkpoint tracking through enabled provider hooks.",
   "author": {
     "name": "Specialist Doctors Inc.",
     "url": "https://github.com/SpecialistDoctors-Inc"
@@ -17,7 +17,7 @@
   "interface": {
     "displayName": "SIMY",
     "shortDescription": "Quality and PR review.",
-    "longDescription": "Run explicitly requested quality checks and PR reviews. Work directly in Codex by default. When the user explicitly requests SIMY Delivery Loop, use the CLI Agentic Loop; the local hook-based Delivery skill is retired.",
+    "longDescription": "Run explicitly requested quality checks and PR reviews. Enabled hooks automatically track eligible conversations from user prompts through checkpoints. Stops, permission waits and opt-outs are preserved. The local Delivery skill remains retired; independently managed work uses the CLI Agentic Loop.",
     "developerName": "Specialist Doctors Inc.",
     "category": "Developer Tools",
     "capabilities": [

package/src/codex-plugins/simy/hooks/README.md

@@ -18,9 +18,14 @@ scans ports or reads stored session tokens.
 
 ## Delivery Loop hooks
 
-`delivery-loop.mjs` is an independent opt-in workflow, shared by the Codex and
-Claude distributions. SessionStart supplies the exact session identity. It does
-not start a loop. After an explicit `simy delivery start`, Stop checks that
+`delivery-loop.mjs` is shared by the Codex and Claude distributions. The CLI
+prepares missing plugin sources automatically and maintains enabled plugin caches.
+SessionStart supplies identity without starting work. A nonempty UserPromptSubmit
+starts standard Delivery Loop by default for eligible connected conversations,
+including when SessionStart was missed. No per-project enable command is needed.
+`simy delivery auto off --provider codex|claude` saves a project override; updates
+preserve it. Existing loops, explicit stops, session opt-outs and revoked provider
+consent remain authoritative. No hook grants provider trust. Stop checks that
 session's checkpoint and returns bounded next-step instructions. Approval/input
 blocks, interruption, completion, failures and missing progress stop automatic continuation.
 Provider errors/exits are surfaced to SIMY's desktop; a dead provider is not