engineering-memory 1.11.21 → 1.11.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "engineering-memory",
-  "version": "1.11.21",
+  "version": "1.11.22",
   "description": "Installs the Engineering Memory skill and its local MCP bridge. Sign in after installing; your organization and project are resolved from your account.",
   "license": "UNLICENSED",
   "type": "module",

package/runtime/build.json

@@ -1,3 +1,3 @@
 {
-  "gitHead": "9df251d4cf095e0f9ed2e7da4e480bbf60142c81"
+  "gitHead": "bd6497867c15105530a624348536ab34f1604d07"
 }

package/runtime/dist/src/runtime/bridge-service.js

@@ -689,6 +689,18 @@ export class BridgeService {
                 (repository.projectId && repository.projectId !== projectId)) {
                 throw refuse('The live task belongs to a different project than the one this repository is bound to.', 'project.resolve');
             }
+            if (pointer?.closedAt &&
+                pointer.worktreeGeneration &&
+                this.dependencies.worktreePool &&
+                !(await this.dependencies.worktreePool.find(projectId, taskSlug, repository.repoRoot)))
+                return asJsonValue({
+                    taskClosed: true,
+                    folderReleased: true,
+                    branch: pointer.branch ?? null,
+                    ...pendingQuestionnairesFields(pendingQuestionnaires),
+                    repository: publicRepository(repository),
+                    nextAction: 'This task is closed and its folder has been released, so there is nothing to resume or release. Its branch keeps the committed work; start new work with task.branch.',
+                });
             const adoptable = pointer &&
                 !pointer.worktreeGeneration &&
                 normalizeTaskMode(pointer.mode) !== 'read_only' &&
@@ -2730,6 +2742,17 @@ export class BridgeService {
                 return asJsonValue({ operation, complete: true, removed: entry.repoRoot });
             }
             const entry = await pool.forPath(projectId, repository.repoRoot);
+            if ((operation === 'release' || operation === 'cancel_delivery') &&
+                !(await pool.find(projectId, input.externalTaskId, repository.repoRoot)) &&
+                ((entry?.phase === 'released' && entry.externalTaskId === input.externalTaskId) ||
+                    (await this.dependencies.activeContexts.loadForSlug(repository.repoFingerprint, input.externalTaskId))?.worktreeGeneration === input.generation))
+                return asJsonValue({
+                    operation,
+                    complete: true,
+                    repoRoot: repository.repoRoot,
+                    alreadyReleased: true,
+                    nextAction: "This task's folder was already released; there is nothing left to release.",
+                });
             if (!entry ||
                 entry.externalTaskId !== input.externalTaskId ||
                 entry.generation !== input.generation)

package/runtime/dist/src/utilities/local-mutex.js

@@ -3,7 +3,10 @@ import { performance } from 'node:perf_hooks';
 import { sha256 } from './hash.js';
 import { BridgeRecoveryError } from '../runtime/recovery-error.js';
 export async function localMutex(identity, work, timeoutMs = 60000) {
-    const port = 20000 + (parseInt(sha256(identity).slice(0, 8), 16) % 30000);
+    const hash = sha256(identity);
+    const address = process.platform === 'win32'
+        ? { path: String.raw `\\.\pipe\engineering-memory-` + hash }
+        : { host: '127.0.0.1', port: 20000 + (parseInt(hash.slice(0, 8), 16) % 30000) };
     const deadline = performance.now() + timeoutMs;
     let server;
     for (;;) {
@@ -11,7 +14,7 @@ export async function localMutex(identity, work, timeoutMs = 60000) {
         try {
             await new Promise((resolve, reject) => {
                 server.once('error', reject);
-                server.listen({ host: '127.0.0.1', port, exclusive: true }, () => {
+                server.listen({ ...address, exclusive: true }, () => {
                     server.removeListener('error', reject);
                     resolve();
                 });

package/skill/references/lifecycle.md

@@ -40,7 +40,7 @@ Renew `task.heartbeat` using the exact returned task, path and ownership generat
 
 Use `task.pause` when work stops or is handed to another client. It preserves task state and files, invalidates the old owner, and allows safe clean directories to be reused. It is not `task.abandon`. Resume reacquires the original directory when possible or uses the retained task branch in another safe directory. `worktree.reconcile` recovers interrupted allocation and validated local ownership records against Git. Never edit the registry, force checkout, stash/reset/clean, remove branches, kill a client or delete a worktree to bypass a refusal.
 
-`task.close` retains a managed checkout while changes or delivery remain pending. After delivery, release with `deliveryOutcome:'delivered'` and the current generation. When the user decides not to deliver, call `worktree.release` with `deliveryOutcome:'cancelled'`; the product asks what happens to the folder and saves any changes to a ref. Never run `git restore` or `git clean` yourself to free a worktree, and never report a delivery that did not happen. Dirty/staged/untracked files, pending outbox/intents and unfinished Git operations prevent release. An old release cannot affect a reused directory. User-owned existing directories are never added to automatic cleanup. When the user wants to work in the main folder, they answer that in `task.branch`'s start form; when an open task holds the folder, that option names the task that would move out. A closed or abandoned task stops holding the folder by itself; a task still running in another client keeps it until it is paused there. Never edit or delete the reservation ref by hand.
+`task.close` retains a managed checkout while changes or delivery remain pending. After delivery, release with `deliveryOutcome:'delivered'` and the current generation. Once the task's commits are on a remote branch the pool settles the folder by itself; `worktree.release` then answers `alreadyReleased`, `session.resume` of the closed task answers `folderReleased`, and nothing further is needed. When the user decides not to deliver, call `worktree.release` with `deliveryOutcome:'cancelled'`; the product asks what happens to the folder and saves any changes to a ref. Never run `git restore` or `git clean` yourself to free a worktree, and never report a delivery that did not happen. Dirty/staged/untracked files, pending outbox/intents and unfinished Git operations prevent release. An old release cannot affect a reused directory. User-owned existing directories are never added to automatic cleanup. When the user wants to work in the main folder, they answer that in `task.branch`'s start form; when an open task holds the folder, that option names the task that would move out. A closed or abandoned task stops holding the folder by itself; a task still running in another client keeps it until it is paused there. Never edit or delete the reservation ref by hand.
 
 Read-only tasks allocate no worktree. If the user authorizes writing, settle `task.branch` for that same externalTaskId, use the allocated repoRoot, then call `context.prepare_change` with `transitionToWrite`. Its pinned source must still match; choosing a different source requires a new task, never rewriting the original task's source identity. Existing lease, discipline, source-memory, verify and commit gates remain mandatory.