omp-vcc 0.1.9 → 0.1.11

package/README.md

@@ -90,7 +90,7 @@ Additive VCC+shake is automatic. Eager chain: `chainShakeHint:true`. Explicit mo
 
 ```sh
 bunx tsc --noEmit
-bun test                  # 661 tests, 56 files, 2095 expects, 0 fail
+bun test                  # 800 tests, 61 files, 2396 expects, 0 fail
 bun test tests/e2e --timeout 120000  # 124 E2E
 bun run smoke             # 13 checks: 3 hooks + 6 cmds + 2 tools + dedup (+ pipeline)
 omp plugin link . && omp plugin doctor

package/extensions/main.ts

@@ -94,11 +94,9 @@ export default function (pi: ExtensionAPI): void {
 
       const q = p.query?.trim();
 
-      if (q && parseEntryRef(q)) {
-        const ref = parseEntryRef(q);
-        if (!ref) {
-          return { content: [{ type: "text", text: "Invalid entry ref query." }], details: undefined };
-        }
+      const entryRef = q ? parseEntryRef(q) : null;
+      if (entryRef) {
+        const ref = entryRef;
         if (lineageEntryIds) {
           const { rendered } = loadAllMessages(sessionFile, false, lineageEntryIds);
           const exists = rendered.some((m) => m.index === ref.index);
@@ -113,11 +111,9 @@ export default function (pi: ExtensionAPI): void {
         return { content: [{ type: "text", text }], details: undefined };
       }
 
-      if (q && parseDrillDown(q)) {
-        const parsed = parseDrillDown(q);
-        if (!parsed) {
-          return { content: [{ type: "text", text: "Invalid drill-down query." }], details: undefined };
-        }
+      const drill = q ? parseDrillDown(q) : null;
+      if (drill) {
+        const parsed = drill;
         if (lineageEntryIds) {
           const { rendered } = loadAllMessages(sessionFile, false, lineageEntryIds);
           const exists = rendered.some((m) => m.index === parsed.index);
@@ -248,11 +244,9 @@ export default function (pi: ExtensionAPI): void {
         }
       } catch (err: unknown) {
         const msg = err instanceof Error ? err.message : String(err);
-        if (msg === "Compaction cancelled" || msg === "Already compacted") {
-          try { c.ui.notify("Nothing to compact", "warning"); } catch {}
-        } else {
-          try { c.ui.notify(`Compaction failed: ${msg}`, "error"); } catch {}
-        }
+        const cancelled = msg === "Compaction cancelled" || msg === "Already compacted";
+        const note = cancelled ? "Nothing to compact" : `Compaction failed: ${msg}`;
+        try { c.ui.notify(note, cancelled ? "warning" : "error"); } catch {}
       }
     },
   });

package/extensions/vcc-core/core/drill-down.ts

@@ -204,13 +204,12 @@ export function parseDrillDown(query: string): {
   }
 
   if (suffix !== undefined) {
-    // Parse "offset" or "offset:limit"
+    // Parse "offset" or "offset:limit". The pattern above admits only
+    // numeric suffixes, so offset always parses here.
     const parts = suffix.split(":");
     const offset = parseInt(parts[0], 10);
     const limit = parts[1] !== undefined ? parseInt(parts[1], 10) : undefined;
-    if (!Number.isNaN(offset)) {
-      return { index, pathPattern, full: false, offset, limit };
-    }
+    return { index, pathPattern, full: false, offset, limit };
   }
 
   return {

package/extensions/vcc-core/core/migrate-stale.ts

@@ -84,7 +84,7 @@ export function migrateStalePluginEntries(home?: string): string {
             if (k.startsWith("@")) {
               const scopeDir = join(nm, k.split("/")[0]!);
               try {
-                if (fsSync.readdirSync(scopeDir).length === 0) fsSync.rmSync(scopeDir, { force: true });
+              if (fsSync.readdirSync(scopeDir).length === 0) fsSync.rmSync(scopeDir, { force: true, recursive: true });
               } catch {}
             }
           } catch {}
@@ -119,7 +119,7 @@ export function migrateStalePluginEntries(home?: string): string {
           if (k.startsWith("@")) {
             const scopeDir = join(nm, k.split("/")[0]!);
             try {
-              if (fsSync.readdirSync(scopeDir).length === 0) fsSync.rmSync(scopeDir, { force: true });
+              if (fsSync.readdirSync(scopeDir).length === 0) fsSync.rmSync(scopeDir, { force: true, recursive: true });
             } catch {}
           }
         } catch {}
@@ -145,7 +145,7 @@ export function migrateStalePluginEntries(home?: string): string {
         if (cand.startsWith("@")) {
           const scopeDir = join(nm, cand.split("/")[0]!);
           try {
-            if (fsSync.readdirSync(scopeDir).length === 0) fsSync.rmSync(scopeDir, { force: true });
+              if (fsSync.readdirSync(scopeDir).length === 0) fsSync.rmSync(scopeDir, { force: true, recursive: true });
           } catch {}
         }
       } catch {}

package/extensions/vcc-core/core/summarize.ts

@@ -126,11 +126,9 @@ const mergeFileLines = (prev: string, fresh: string): string => {
       return null;
     }
     const grouped = /^ \(\+(\d+) more( under (.+?))?\): (.*)$/.exec(tail);
-    if (grouped) {
-      const dir = grouped[3] ?? "";
-      return { prefix: dir && !dir.endsWith("/") ? `${dir}/` : dir, rest: grouped[4] };
-    }
-    return null;
+    if (!grouped) return null;
+    const dir = grouped[3] ?? "";
+    return { prefix: dir && !dir.endsWith("/") ? `${dir}/` : dir, rest: grouped[4] };
   };
   for (const text of [prev, fresh]) {
     let current: { cat: string; prefix: string; rest: string } | null = null;

package/extensions/vcc-core/core/token-estimate.ts

@@ -71,9 +71,7 @@ export const IMAGE_CONTENT_CHARS = 4800;
 const safeJsonStringify = (value: unknown): string => {
   try {
     return JSON.stringify(value ?? "") ?? "";
-  } catch {
-    return "";
-  }
+  } catch { return ""; }
 };
 
 /**

package/extensions/vcc-core/hook.ts

@@ -32,8 +32,8 @@ try {
 
 export { PI_VCC_COMPACT_INSTRUCTION } from "./core/compact-args";
 export const OMP_VCC_COMPACT_INSTRUCTION = "__omp_vcc__";
-// Accept both pi and omp sentinels for backwards compat
-const isVccSentinel = (s: string | undefined) => s === PI_VCC_COMPACT_INSTRUCTION || s === OMP_VCC_COMPACT_INSTRUCTION;
+// (Both pi/omp sentinels are matched inline at the explicit-mode bypass in
+// the session_before_compact handler below.)
 
 export interface CompactionStats {
   summarized: number;
@@ -162,20 +162,6 @@ export const triggerInvisibleContinue = (pi: ExtensionAPI): void => {
   );
 };
 
-const clearPendingAutoContinue = () => {
-  clearTimeout(pendingAutoContinueTimer as any);
-  pendingAutoContinueTimer = null;
-};
-
-const scheduleAutoContinue = (pi: any) => {
-  clearPendingAutoContinue();
-  pendingAutoContinueTimer = setTimeout(() => {
-    pendingAutoContinueTimer = null;
-    try {
-      triggerInvisibleContinue(pi);
-    } catch {}
-  }, 0);
-};
 
 export const getLastCompactionStats = (pi?: any) => {
   if (pi) {
@@ -1308,11 +1294,9 @@ export const registerPiVccCommand = (pi: any) => {
           }
         },
         onError: (err) => {
-          if (err.message === "Compaction cancelled" || err.message === "Already compacted") {
-            ctx.ui.notify("Nothing to compact", "warning");
-          } else {
-            ctx.ui.notify(`Compaction failed: ${err.message}`, "error");
-          }
+          const cancelled = err.message === "Compaction cancelled" || err.message === "Already compacted";
+          const msg = cancelled ? "Nothing to compact" : `Compaction failed: ${err.message}`;
+          ctx.ui.notify(msg, cancelled ? "warning" : "error");
         },
       });
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "omp-vcc",
-  "version": "0.1.9",
+  "version": "0.1.11",
   "type": "module",
   "description": "Algorithmic VCC compaction for omp - fast lossless no-LLM",
   "author": "Zhu Lin <zhulin@czl.my>",