@qiaolei81/copilot-session-viewer 0.2.5 → 0.2.6

package/lib/parsers/index.js

@@ -2,7 +2,7 @@ const BaseSessionParser = require('./base-parser');
 const CopilotSessionParser = require('./copilot-parser');
 const ClaudeSessionParser = require('./claude-parser');
 const PiMonoParser = require('./pi-mono-parser');
-const VsCodeParser = require('./vscode-parser');
+// const VsCodeParser = require('./vscode-parser'); // TODO: VSCode parser disabled
 const ParserFactory = require('./parser-factory');
 
 module.exports = {
@@ -10,6 +10,6 @@ module.exports = {
   CopilotSessionParser,
   ClaudeSessionParser,
   PiMonoParser,
-  VsCodeParser,
+  // VsCodeParser, // TODO: VSCode parser disabled
   ParserFactory
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qiaolei81/copilot-session-viewer",
-  "version": "0.2.5",
+  "version": "0.2.6",
   "description": "Web UI for viewing GitHub Copilot CLI session logs",
   "author": "Lei Qiao <qiaolei81@gmail.com>",
   "license": "MIT",

package/src/app.js

@@ -19,6 +19,9 @@ const UploadController = require('./controllers/uploadController');
 function createApp(options = {}) {
   const app = express();
 
+  // Disable Express's automatic ETag generation (prevents 304 on live/active session files)
+  app.set('etag', false);
+
   // Create controller instances (with optional dependency injection)
   const sessionController = new SessionController(options.sessionService);
   const insightController = new InsightController(options.insightService, options.sessionService);

package/src/controllers/sessionController.js

@@ -162,25 +162,12 @@ class SessionController {
         }
       }
 
-      // Get session metadata for ETag generation
+      // Get session (needed for findById, no caching)
       const session = await this.sessionService.sessionRepository.findById(sessionId);
       if (!session) {
         return res.status(404).json({ error: 'Session not found' });
       }
 
-      // Generate ETag from session ID + timestamp + pagination params (if used)
-      const crypto = require('crypto');
-      const etagBase = isPaginationRequested 
-        ? `${sessionId}-${session.updatedAt || session.createdAt}-${limit}-${offset}`
-        : `${sessionId}-${session.updatedAt || session.createdAt}`;
-      const etag = crypto.createHash('md5').update(etagBase).digest('hex');
-
-      // Check If-None-Match header (client cache)
-      const clientEtag = req.headers['if-none-match'];
-      if (clientEtag === etag) {
-        return res.status(304).end(); // Not Modified - use cached version
-      }
-
       // Load events (with or without pagination)
       if (isPaginationRequested) {
         result = await this.sessionService.getSessionEvents(sessionId, { limit, offset });
@@ -190,10 +177,9 @@ class SessionController {
         result = events; // Direct array
       }
 
-      // Set caching headers
+      // No caching for events - session files are live/active
       res.set({
-        'ETag': etag,
-        'Cache-Control': 'private, max-age=0, no-cache', // Disable cache during development
+        'Cache-Control': 'no-store',
         'Vary': 'Accept-Encoding'
       });
 

package/src/services/sessionRepository.js

@@ -39,11 +39,12 @@ class SessionRepository {
           dir: process.env.PI_MONO_SESSION_DIR || 
                path.join(os.homedir(), '.pi', 'agent', 'sessions')
         },
-        {
-          type: 'vscode',
-          dir: process.env.VSCODE_WORKSPACE_STORAGE_DIR ||
-               path.join(os.homedir(), 'Library', 'Application Support', 'Code', 'User', 'workspaceStorage')
-        }
+        // TODO: VSCode parser disabled
+        // {
+        //   type: 'vscode',
+        //   dir: process.env.VSCODE_WORKSPACE_STORAGE_DIR ||
+        //        path.join(os.homedir(), 'Library', 'Application Support', 'Code', 'User', 'workspaceStorage')
+        // }
       ];
     }
     
@@ -110,12 +111,10 @@ class SessionRepository {
           if (stats.isDirectory()) {
             return this._scanPiMonoDir(fullPath, entry);
           }
-        } else if (source.type === 'vscode') {
-          // VSCode: workspaceStorage/<hash>/chatSessions/<uuid>.json
-          // Each top-level entry is a workspace hash directory
-          if (stats.isDirectory()) {
-            return this._scanVsCodeWorkspaceDir(fullPath);
-          }
+        // } else if (source.type === 'vscode') { // TODO: VSCode disabled
+        //   if (stats.isDirectory()) {
+        //     return this._scanVsCodeWorkspaceDir(fullPath);
+        //   }
         }
         return null;
       });
@@ -275,8 +274,8 @@ class SessionRepository {
         session = await this._findClaudeSession(sessionId, source.dir);
       } else if (source.type === 'pi-mono') {
         session = await this._findPiMonoSession(sessionId, source.dir);
-      } else if (source.type === 'vscode') {
-        session = await this._findVsCodeSession(sessionId, source.dir);
+      // } else if (source.type === 'vscode') { // TODO: VSCode disabled
+      //   session = await this._findVsCodeSession(sessionId, source.dir);
       }
 
       if (session) return session;

package/src/services/sessionService.js

@@ -151,27 +151,24 @@ class SessionService {
         console.error('Error searching Pi-Mono sessions:', err);
         return [];
       }
-    } else if (session.source === 'vscode') {
-      // VSCode format: read JSON file directly, convert to event array via VsCodeParser
-      const { VsCodeParser } = require('../../lib/parsers');
-      const vscodeParser = new VsCodeParser();
-      try {
-        const raw = await fs.promises.readFile(session.filePath, 'utf-8');
-        let sessionJson;
-        if (session.filePath.endsWith('.jsonl')) {
-          sessionJson = this.sessionRepository._parseVsCodeJsonl(raw);
-        } else {
-          sessionJson = JSON.parse(raw);
-        }
-        if (!sessionJson) return [];
-        const parsed = vscodeParser.parseVsCode(sessionJson);
-        // Convert tool.invocation events → assistant.message with data.tools
-        // so frontend can render them using the same tool-list component
-        return this._expandVsCodeEvents(parsed.allEvents);
-      } catch (err) {
-        console.error('Error reading VSCode session:', err);
-        return [];
-      }
+    // } else if (session.source === 'vscode') { // TODO: VSCode disabled
+    //   const { VsCodeParser } = require('../../lib/parsers');
+    //   const vscodeParser = new VsCodeParser();
+    //   try {
+    //     const raw = await fs.promises.readFile(session.filePath, 'utf-8');
+    //     let sessionJson;
+    //     if (session.filePath.endsWith('.jsonl')) {
+    //       sessionJson = this.sessionRepository._parseVsCodeJsonl(raw);
+    //     } else {
+    //       sessionJson = JSON.parse(raw);
+    //     }
+    //     if (!sessionJson) return [];
+    //     const parsed = vscodeParser.parseVsCode(sessionJson);
+    //     return this._expandVsCodeEvents(parsed.allEvents);
+    //   } catch (err) {
+    //     console.error('Error reading VSCode session:', err);
+    //     return [];
+    //   }
     }
 
 

package/views/session-vue.ejs

@@ -1864,6 +1864,15 @@
             }
             
             console.log('[Navigation] Events loaded:', loadedEvents.value.length);
+
+            // Update 'Updated' time from last event timestamp (more accurate than file mtime)
+            if (loadedEvents.value.length > 0) {
+              const lastEvent = loadedEvents.value[loadedEvents.value.length - 1];
+              const lastTime = lastEvent.timestamp || lastEvent.time || lastEvent.data?.timestamp;
+              if (lastTime) {
+                metadata.value.updated = new Date(lastTime);
+              }
+            }
             
             // Check for URL query parameters and jump to event AFTER events are loaded
             const urlParams = new URLSearchParams(window.location.search);