@blogic-cz/agent-tools 0.15.8 → 0.15.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blogic-cz/agent-tools",
-  "version": "0.15.8",
+  "version": "0.15.11",
   "description": "CLI tools for AI coding agent workflows — GitHub, database, Kubernetes, Azure DevOps, logs, sessions, and audit",
   "keywords": [
     "agent",

package/src/session-tool/claude-code.ts

@@ -1,11 +1,12 @@
 import { Effect } from "effect";
 import { readdir } from "node:fs/promises";
+import { basename, join } from "node:path";
 
 import type { MessageSummary } from "./types";
 
 import { SessionReadError, SessionStorageNotFoundError, type SessionError } from "./errors";
 
-export const encodeProjectPath = (dir: string): string => dir.replaceAll("/", "-");
+export const encodeProjectPath = (dir: string): string => dir.replaceAll(/[/\\:]/g, "-");
 
 export type ContentBlock =
   | { type: "text"; text: string }
@@ -172,7 +173,7 @@ export const getClaudeCodeSessions = (
           continue;
         }
 
-        const projectPath = `${basePath}/${entry.name}`;
+        const projectPath = join(basePath, entry.name);
         let files: string[];
         try {
           // eslint-disable-next-line eslint/no-await-in-loop -- directory scan must finish before filtering file list
@@ -186,7 +187,7 @@ export const getClaudeCodeSessions = (
             continue;
           }
 
-          sessionFiles.push(`${projectPath}/${fileName}`);
+          sessionFiles.push(join(projectPath, fileName));
         }
       }
 
@@ -200,8 +201,8 @@ export const getClaudeCodeSessions = (
   });
 
 const getFileNameWithoutExtension = (filePath: string): string => {
-  const fileName = filePath.split("/").pop();
-  if (fileName === undefined) {
+  const fileName = basename(filePath);
+  if (fileName === "") {
     return "";
   }
 

package/src/session-tool/pi.ts

@@ -2,6 +2,7 @@ import { Effect } from "effect";
 // node:fs/promises, not Bun.file/Bun.Glob: vitest runs under Node, where the Bun globals do not
 // exist, and every function below is reached by the suite. See AGENTS.md.
 import { open, readFile, readdir, stat } from "node:fs/promises";
+import { join } from "node:path";
 
 import type { MessageSummary, SessionSummary } from "./types";
 
@@ -117,10 +118,10 @@ const walkSessionFiles = async (basePath: string): Promise<string[]> => {
     directories
       .filter((entry) => entry.isDirectory())
       .map(async (directory) => {
-        const directoryPath = `${basePath}/${directory.name}`;
+        const directoryPath = join(basePath, directory.name);
         return (await readdir(directoryPath, { withFileTypes: true }))
           .filter((entry) => entry.isFile() && entry.name.endsWith(".jsonl"))
-          .map((entry) => `${directoryPath}/${entry.name}`);
+          .map((entry) => join(directoryPath, entry.name));
       }),
   );
   return files.flat();

package/src/session-tool/service.ts

@@ -1,5 +1,6 @@
 import { Context, Effect, Layer } from "effect";
 import { readdir } from "node:fs/promises";
+import { basename } from "node:path";
 
 import type { MessageSummary, SessionInfo, SessionSource, SessionSummary } from "./types";
 
@@ -52,7 +53,7 @@ const UUID_SESSION_ID_REGEX =
   /^[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}$/u;
 
 const getSessionIdFromClaudeFile = (filePath: string): string => {
-  const fileName = filePath.split("/").pop() ?? "";
+  const fileName = basename(filePath);
   return fileName.endsWith(".jsonl") ? fileName.slice(0, -".jsonl".length) : fileName;
 };
 
@@ -333,7 +334,7 @@ export class SessionService extends Context.Service<
 
                     summaries.push({
                       sessionID: parsed.sessionID ?? sessionId,
-                      id: parsed.id ?? filePath.split("/").pop()?.replace(".json", "") ?? "",
+                      id: parsed.id ?? basename(filePath).replace(".json", ""),
                       title: parsed.summary.title,
                       body: parsed.summary.body ?? "",
                       created: parsed.time?.created ?? 0,

package/src/shared/prerequisites/runtime.ts

@@ -557,10 +557,23 @@ const acquireVpn = <E>(
         while (true) {
           const snapshot = store.snapshot();
           if (snapshot.lifecycle === "UNKNOWN") {
+            const contended = snapshot.activeLeases > 0;
+            if (!contended) {
+              const observed = yield* runStatusBefore(driver, connectDeadline, runCommand);
+              if (
+                observed !== undefined &&
+                (store.reconcileUnknown(snapshot, observed, Date.now()) ||
+                  store.snapshot().lifecycle !== "UNKNOWN")
+              ) {
+                continue;
+              }
+            }
             return yield* fail(
               new PrerequisiteRunError({
                 message: `VPN prerequisite "${key}" has unknown ownership: ${snapshot.evidence ?? "no evidence"}`,
-                hint: "Quiesce all agent-tools processes and remove its VPN runtime state before retrying.",
+                hint: contended
+                  ? "Another agent-tools process holds an active lease on this VPN. Retry once it has finished."
+                  : missingVpnToolHint(driver),
               }),
             );
           }

package/src/shared/prerequisites/store.ts

@@ -534,6 +534,28 @@ export class VpnStore {
     return false;
   }
 
+  /** Recovers terminal UNKNOWN from an observed status; fenced so it can never take ownership from a holder. */
+  reconcileUnknown(snapshot: VpnStateSnapshot, connected: boolean, now: number): boolean {
+    return this.immediate(
+      () =>
+        this.db
+          .query(
+            `UPDATE vpn_state SET lifecycle=?, managed_epoch_id=NULL, idle_deadline=NULL,
+               revision=revision+1, evidence=?, updated_at=?
+             WHERE singleton=1 AND lifecycle='UNKNOWN' AND operation_id IS NULL AND revision=?
+               AND NOT EXISTS (SELECT 1 FROM leases WHERE status='ACTIVE')`,
+          )
+          .run(
+            connected ? "EXTERNAL" : "DOWN",
+            connected
+              ? "Recovered from unknown ownership: VPN observed connected outside agent-tools."
+              : "Recovered from unknown ownership: VPN observed disconnected.",
+            now,
+            snapshot.revision,
+          ).changes === 1,
+    );
+  }
+
   private commitOperation(
     guard: OperationGuard,
     from: VpnLifecycle,