@khorsheed/dsh-file-preview 0.1.1 → 0.2.0

package/lib/types/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/index.ts"],"names":[],"mappings":"AAAA;;GAEG;AAMH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,qBAAqB,CAAA;AAClD,OAAO,KAAK,EAAE,KAAK,EAAiB,MAAM,wBAAwB,CAAA;AAGlE,OAAO,EAAU,mBAAmB,EAAE,MAAM,kCAAkC,CAAA;AAC9E,OAAO,CAAC,MAAM,0BAA0B,CAAA;AAIxC,OAAO,KAAK,EAAE,iBAAiB,EAAoB,eAAe,EAAE,eAAe,EAAE,iBAAiB,EAAuB,kBAAkB,EAAE,MAAM,YAAY,CAAA;AAEnK,OAAO,QAAQ,qBAAqB,CAAC;IACnC,UAAU,OAAO;QACf,WAAW,EAAE,kBAAkB,CAAA;KAChC;CACF;AA4HD;;;;;;;GAOG;AACH,qBAAa,kBAAmB,SAAQ,mBAAmB;IACzD,MAAM,CAAC,MAAM,WAAS;IAEtB,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,iBAAiB,CAAC,CAKjC;IAEF,gDAAgD;IAChD,MAAM,CAAC,QAAQ,CAAC,WAAW,yBAAwB;IAEnD,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAA6B;IACtD;iFAC6E;IAC7E,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAS;IACpC,yEAAyE;IACzE,OAAO,CAAC,QAAQ,CAAC,MAAM,CAA2B;IAClD,wEAAwE;IACxE,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAsD;IACnF,4EAA4E;IAC5E,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAgC;IAC1D,wEAAwE;IACxE,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAkE;IAE5F;;;;;;OAMG;IACH,YACE,GAAG,EAAE,OAAO,EACZ,MAAM,GAAE,iBAAsB,EAC9B,IAAI,GAAE;QAAE,YAAY,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;KAAO,EAoCnF;IAED,OAAO,KAAK,EAAE,GAEb;IAED;;;;;;OAMG;YACW,UAAU;IAmExB;;;;;;OAMG;IAEH,IAAI,CAAC,KAAK,EAAE,KAAK,GAAG,eAAe,CAgBlC;IAED;;;;;;;;;;;OAWG;IAEH,SAAS,CAAC,KAAK,EAAE,KAAK,GAAG,kBAAkB,CAiC1C;IAED;;;;;;;;OAQG;IAEG,IAAI,CAAC,KAAK,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,eAAe,CAAC,CA6DpF;IAED;;;;;;;;;;;;OAYG;IAEG,MAAM,CAAC,KAAK,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,iBAAiB,CAAC,CAgCxF;CACF;eAEc,kBAAkB"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/index.ts"],"names":[],"mappings":"AAAA;;GAEG;AAMH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,qBAAqB,CAAA;AAClD,OAAO,KAAK,EAAE,KAAK,EAAiB,MAAM,wBAAwB,CAAA;AAGlE,OAAO,EAAU,mBAAmB,EAAE,MAAM,kCAAkC,CAAA;AAC9E,OAAO,CAAC,MAAM,0BAA0B,CAAA;AAIxC,OAAO,KAAK,EAAE,iBAAiB,EAAoB,eAAe,EAAE,eAAe,EAAE,iBAAiB,EAAuB,kBAAkB,EAAE,MAAM,YAAY,CAAA;AAEnK,OAAO,QAAQ,qBAAqB,CAAC;IACnC,UAAU,OAAO;QACf,WAAW,EAAE,kBAAkB,CAAA;KAChC;CACF;AAiJD;;;;;;;;GAQG;AACH,qBAAa,kBAAmB,SAAQ,mBAAmB;IACzD,MAAM,CAAC,MAAM,WAAS;IAEtB,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,iBAAiB,CAAC,CAKjC;IAEF,gDAAgD;IAChD,MAAM,CAAC,QAAQ,CAAC,WAAW,yBAAwB;IAEnD,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAA6B;IACtD;iFAC6E;IAC7E,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAS;IACpC,yEAAyE;IACzE,OAAO,CAAC,QAAQ,CAAC,MAAM,CAA2B;IAClD,wEAAwE;IACxE,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAsD;IACnF,4EAA4E;IAC5E,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAgC;IAC1D,wEAAwE;IACxE,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAkE;IAE5F;;;;;;OAMG;IACH,YACE,GAAG,EAAE,OAAO,EACZ,MAAM,GAAE,iBAAsB,EAC9B,IAAI,GAAE;QAAE,YAAY,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;KAAO,EAoCnF;IAED,OAAO,KAAK,EAAE,GAEb;IAED;;;;;;OAMG;YACW,UAAU;IAmExB;;;;;;;;;;OAUG;YACW,cAAc;IAiB5B;;;;;;;;OAQG;IAEG,IAAI,CAAC,KAAK,EAAE,KAAK,GAAG,OAAO,CAAC,eAAe,CAAC,CAmBjD;IAED;;;;;;;;;;;;;;OAcG;IAEG,SAAS,CAAC,KAAK,EAAE,KAAK,GAAG,OAAO,CAAC,kBAAkB,CAAC,CAyCzD;IAED;;;;;;;;OAQG;IAEG,IAAI,CAAC,KAAK,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,eAAe,CAAC,CA6DpF;IAED;;;;;;;;;;;;OAYG;IAEG,MAAM,CAAC,KAAK,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,iBAAiB,CAAC,CAgCxF;CACF;eAEc,kBAAkB"}

package/lib/types/index.js

@@ -67,7 +67,8 @@ function registerArtifactSkill(ctx) {
         return;
     }
     try {
-        const skillFile = join(dirname(fileURLToPath(import.meta.url)), '..', 'skills', '3d-artifact', 'SKILL.md');
+        const skillDir = join(dirname(fileURLToPath(import.meta.url)), '..', 'skills', '3d-artifact');
+        const skillFile = join(skillDir, 'SKILL.md');
         const raw = readFileSync(skillFile, 'utf8');
         const match = /^---\n([\s\S]*?)\n---\n([\s\S]*)$/.exec(raw);
         const name = /^name: (.+)$/m.exec(match?.[1] ?? '')?.[1]?.trim();
@@ -77,7 +78,16 @@ function registerArtifactSkill(ctx) {
             ctx.logger.warn('file-preview: shipped SKILL.md is malformed — the 3d-artifact skill is not registered');
             return;
         }
-        ctx.effect(() => skills.register({ name, description, content, source: 'runtime' }));
+        ctx.effect(() => skills.register({
+            name,
+            description,
+            content,
+            source: 'runtime',
+            provider: 'file-preview',
+            // Content-only (only SKILL.md in the bundle): no `resourceBase`, so the
+            // catalog renders a virtual single-SKILL.md node. Add `resourceBase`
+            // only when the bundle later gains scripts/assets beside SKILL.md.
+        }));
     }
     catch (error) {
         ctx.logger.warn(`file-preview: shipped SKILL.md unreadable (${String(error)}) — the 3d-artifact skill is not registered`);
@@ -145,8 +155,9 @@ function readErrorMessage(error) {
 }
 /**
  * The `filePreview` Remote service: session-scoped read-only file inspection
- * for web surfaces. `list` is a pure fold over the session log (no filesystem
- * access); `read` resolves the path against the session cwd and serves current
+ * for web surfaces. `list` is a fold over the session log, then filtered to the
+ * paths that still exist on disk (resolved against the session cwd and stat-ed
+ * per call); `read` resolves the path against the session cwd and serves current
  * text content through `ctx.fs` (or a browser URL for images), capped by
  * config. Image bytes ride a dedicated host route so the browser loads them
  * natively without bloating the RPC channel.
@@ -310,31 +321,65 @@ let FilePreviewService = (() => {
             });
             res.end(Buffer.from(bytes));
         }
+        /** Resolve the session's recorded display paths against its cwd and keep only
+         *  those that currently exist as a regular file. The products surfaces fold
+         *  the session log (which is history, not disk state): a path the log wrote
+         *  may have been cleaned up since (a temp script the turn deleted), and such
+         *  a path is not a product to list. Existence is checked fresh on every call
+         *  (never cached), so a deletion that lands after a fold is visible on the
+         *  next fetch. Unresolvable/unstat-able paths are treated as absent.
+         * @param agent - owning live agent; its session cwd anchors relative paths.
+         * @param paths - the display paths to probe.
+         * @returns the subset that resolves to an existing regular file.
+         */
+        async retainExisting(agent, paths) {
+            if (paths.length === 0)
+                return new Set();
+            const cwd = agent.session.header.cwd;
+            const existing = new Set();
+            for (const path of paths) {
+                if (existing.has(path))
+                    continue;
+                try {
+                    const target = await this.fs.resolve(path, cwd === undefined ? {} : { cwd });
+                    const info = await this.fs.stat(target);
+                    if (info?.type === 'file')
+                        existing.add(path);
+                }
+                catch {
+                    // Unresolvable or stat failed: the path is not present to list.
+                }
+            }
+            return existing;
+        }
         /**
          * List the files one session wrote or edited: the log fold plus any
          * bash-written files the collector verified (the S2 seam), merged so a
-         * captured path the fold already knows keeps its log-derived entry.
+         * captured path the fold already knows keeps its log-derived entry. Only
+         * paths that currently exist on disk are returned — a file the session
+         * wrote and later deleted is history, not a product.
          * @param agent - owning live agent; its session log is the data source.
-         * @returns first-seen files capped by `maxFiles`, with the log watermark.
+         * @returns existing first-seen files capped by `maxFiles`, with the log watermark.
          */
-        list(agent) {
-            const folded = foldFilePreview(agent.session.events, this.resolved.maxFiles);
+        async list(agent) {
+            const folded = foldFilePreview(agent.session.snapshotEvents(), this.resolved.maxFiles);
             const captured = this.collector?.captured(agent.session.id);
-            if (captured === undefined || captured.size === 0)
-                return folded;
-            const seen = new Set(folded.entries.map(entry => entry.path));
-            const extra = [];
-            for (const [path, write] of captured) {
-                if (seen.has(path) || folded.entries.length + extra.length >= this.resolved.maxFiles)
-                    continue;
-                extra.push({ path, op: 'write', seq: write.seq, turn: write.turn, step: write.step, diffs: [] });
+            let entries = folded.entries;
+            if (captured !== undefined && captured.size > 0) {
+                const seen = new Set(entries.map(entry => entry.path));
+                const extra = [];
+                for (const [path, write] of captured) {
+                    if (seen.has(path) || entries.length + extra.length >= this.resolved.maxFiles)
+                        continue;
+                    extra.push({ path, op: 'write', seq: write.seq, turn: write.turn, step: write.step, diffs: [] });
+                }
+                entries = [...entries, ...extra];
             }
-            if (extra.length === 0)
-                return folded;
+            const existing = await this.retainExisting(agent, entries.map(entry => entry.path));
             return {
-                entries: [...folded.entries, ...extra],
+                entries: entries.filter(entry => existing.has(entry.path)),
                 asOfSeq: folded.asOfSeq,
-                truncated: folded.truncated || folded.entries.length + extra.length >= this.resolved.maxFiles,
+                truncated: folded.truncated || entries.length >= this.resolved.maxFiles,
             };
         }
         /**
@@ -345,13 +390,16 @@ let FilePreviewService = (() => {
          * result diff meta) and merging the collector's bash-written captures by
          * their own turn. The fold is cached per session and invalidated by the log
          * watermark, so repeated card fetches do not refold. A path touched in two
-         * turns appears in BOTH turn groups — per-turn attribution is exact.
+         * turns appears in BOTH turn groups — per-turn attribution is exact. Only
+         * paths that currently exist on disk are returned (a deleted temp script is
+         * history, not a product); existence is probed fresh on every call, so the
+         * cached fold is filtered per request rather than cached.
          * @param agent - owning live agent; its session log is the data source.
-         * @returns per-turn file groups plus the scanned watermark.
+         * @returns existing per-turn file groups plus the scanned watermark.
          */
-        turnFiles(agent) {
+        async turnFiles(agent) {
             const session = agent.session;
-            const events = session.events;
+            const events = session.snapshotEvents();
             const asOfSeq = events.length === 0 ? -1 : events[events.length - 1].seq;
             const cached = this.turnCache.get(session.id);
             let byTurn;
@@ -386,7 +434,15 @@ let FilePreviewService = (() => {
                     continue;
                 turns.push({ turn, files: files.sort((a, b) => a.seq - b.seq) });
             }
-            return { turns, asOfSeq };
+            // Filter the assembled groups to paths that still exist on disk (fresh per
+            // call — the fold cache is by log watermark, existence moves independently).
+            const existing = await this.retainExisting(agent, turns.flatMap(group => group.files.map(file => file.path)));
+            return {
+                turns: turns
+                    .map(group => ({ turn: group.turn, files: group.files.filter(file => existing.has(file.path)) }))
+                    .filter(group => group.files.length > 0),
+                asOfSeq,
+            };
         }
         /**
          * Read the current content of one recorded file, resolved against the

package/package.json

@@ -7,7 +7,7 @@
     "cordis",
     "file-preview"
   ],
-  "version": "0.1.1",
+  "version": "0.2.0",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/types/index.d.ts",
@@ -59,24 +59,25 @@
   },
   "devDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
-    "@deepseek-ai/dsh-agent": "^0.1.1-rc.1",
-    "@deepseek-ai/dsh-fs": "^0.1.1-rc.1",
-    "@deepseek-ai/dsh-invariants": "^0.1.1-rc.1",
-    "@deepseek-ai/dsh-llm": "^0.1.1-rc.1",
-    "@deepseek-ai/dsh-loader-smoke": "^0.1.1-rc.1",
-    "@deepseek-ai/dsh-native-command": "^0.1.1-rc.1",
-    "@deepseek-ai/dsh-session": "^0.1.1-rc.1",
-    "@deepseek-ai/dsh-skill": "^0.1.1-rc.1",
-    "@deepseek-ai/dsh-tools": "^0.1.1-rc.1",
-    "@deepseek-ai/dsh-typert-protocol": "^0.1.1-rc.1"
+    "@deepseek-ai/dsh-agent": "^0.1.2-rc.1",
+    "@deepseek-ai/dsh-fs": "^0.1.2-rc.1",
+    "@deepseek-ai/dsh-invariants": "^0.1.2-rc.1",
+    "@deepseek-ai/dsh-llm": "^0.1.2-rc.1",
+    "@deepseek-ai/dsh-loader-smoke": "^0.1.2-rc.1",
+    "@deepseek-ai/dsh-native-command": "^0.1.2-rc.1",
+    "@deepseek-ai/dsh-session": "^0.1.2-rc.1",
+    "@deepseek-ai/dsh-skill": "^0.1.2-rc.1",
+    "@deepseek-ai/dsh-tools": "^0.1.2-rc.1",
+    "@deepseek-ai/dsh-typert-protocol": "^0.1.2-rc.1",
+    "@deepseek-ai/dsh-util-values": "^0.1.2-rc.1"
   },
   "dsh": {
     "bundle": {
       "patch": "./cordis.patch.yml"
     },
     "compat": {
-      "minHost": "0.1.0-rc.6",
-      "verifiedHost": "0.1.1-rc.2"
+      "minHost": "0.1.2-rc.1",
+      "verifiedHost": "0.1.2-rc.1"
     }
   }
 }