@qiaolei81/copilot-session-viewer 0.3.3 → 0.3.5

package/docs/unified-event-format-implementation.md

@@ -1,350 +0,0 @@
-# Unified Event Format - Phase 1 Implementation Report
-
-**Status:** ✅ COMPLETE
-**Date:** 2026-02-22
-**Phase:** Backend Implementation (Phase 1 of 4)
-
----
-
-## Executive Summary
-
-Successfully implemented Phase 1 of the unified event format design, creating a single normalized tool representation that abstracts away differences between Copilot, Claude, and Pi-Mono session formats.
-
-### Key Achievements
-
-- ✅ **EventNormalizer class** created with full UnifiedToolCall schema support
-- ✅ **96.49% test coverage** (40 comprehensive unit tests)
-- ✅ **Zero regressions** (321/321 existing tests still pass)
-- ✅ **All three formats** validated (Copilot, Claude, Pi-Mono)
-- ✅ **Feature flag** implemented for gradual rollout
-
----
-
-## Implementation Details
-
-### 1. Files Created
-
-#### `src/services/eventNormalizer.js`
-- **Lines:** 275
-- **Purpose:** Transforms tool events from all formats into unified schema
-- **Key Methods:**
-  - `normalizeEvents(events, source)` - Main entry point
-  - `normalizeEvent(event, source)` - Single event normalization
-  - `_normalizeToolCall(tool, source, timestamp)` - Tool transformation
-  - `_computeStatus(tool)` - Status computation logic
-  - `_computeDuration(startTime, endTime)` - Duration calculation
-
-#### `src/services/__tests__/eventNormalizer.test.js`
-- **Tests:** 40
-- **Coverage:** 96.49%
-- **Test Categories:**
-  - Format-specific normalization (Copilot, Claude, Pi-Mono)
-  - Edge cases (missing fields, orphaned events)
-  - Timeline event normalization
-  - Status computation
-  - Duration calculation
-  - Integration scenarios
-
-### 2. Files Modified
-
-#### `src/services/sessionService.js`
-**Changes:**
-1. Added `EventNormalizer` import (line 7)
-2. Added normalizer instance initialization (line 22)
-3. Added `useUnifiedFormat` feature flag (lines 24-27)
-4. Added normalization call after expansion (lines 240-242)
-
-**Integration Point:**
-```javascript
-// Apply unified format normalization (if enabled)
-if (this.useUnifiedFormat) {
-  events = this.normalizer.normalizeEvents(events, session.source);
-}
-```
-
----
-
-## Unified Tool Schema
-
-### Before (Mixed Formats)
-```javascript
-// Copilot: { type: 'tool_use', id, name, input, result, _matched, status }
-// Claude:  { type: 'tool_use', id, name, input, result, _matched }
-// Pi-Mono: { name, input, result, status, isError }
-```
-
-### After (Unified)
-```javascript
-{
-  id: string,                    // Unique tool call ID
-  name: string,                  // Tool name (e.g., 'Read', 'Write')
-  startTime: string,             // ISO 8601 timestamp
-  endTime: string | null,        // ISO 8601 timestamp, null if running
-  status: 'pending' | 'running' | 'completed' | 'error',
-  input: Record<string, any>,    // Tool parameters
-  result: string | null,         // Tool result
-  error: string | null,          // Error message
-  metadata: {
-    source: string,              // 'copilot' | 'claude' | 'pi-mono'
-    matched?: boolean,           // Original match state
-    duration?: number            // Duration in milliseconds
-  }
-}
-```
-
----
-
-## Test Results
-
-### Unit Tests
-```
-EventNormalizer Tests:  40 passed, 40 total
-Code Coverage:          96.49% (Statements: 96.49%, Branches: 96.38%, Lines: 96.42%)
-Test Execution:         0.2s
-```
-
-### Integration Tests
-```
-✓ Copilot Format:  All checks passed (9/9)
-✓ Claude Format:   All checks passed (8/8)
-✓ Pi-Mono Format:  All checks passed
-```
-
-### Regression Tests
-```
-Total Tests:        321 passed (no regressions)
-Test Execution:     1.557s
-Status:            ✅ All existing functionality preserved
-```
-
-### Pre-Existing Failures
-**Note:** 5 tests in `sessionService.coverage.test.js` were already failing before this implementation:
-- `_matchPiMonoToolResults` tests (method disabled)
-- `_expandPiMonoToCopilotFormat` tests (method removed in prior refactoring)
-- Claude expansion test (incorrect expectations)
-
-These are stale tests unrelated to the unified format implementation.
-
----
-
-## Feature Flag
-
-### Configuration
-```javascript
-const service = new SessionService(sessionDir, {
-  useUnifiedFormat: true  // default: true
-});
-```
-
-### Backward Compatibility
-- **Default:** `useUnifiedFormat = true` (new unified format)
-- **Fallback:** Set to `false` to use original formats
-- **Scope:** Only affects `getSessionEvents()` API responses
-- **Migration:** Internal methods unchanged for compatibility
-
----
-
-## Design Requirements Verification
-
-| Requirement | Status | Evidence |
-|------------|--------|----------|
-| Follow exact UnifiedToolCall schema | ✅ | Schema matches design doc section 2.1 |
-| Handle all edge cases | ✅ | Tests cover missing fields, orphaned events |
-| Maintain backward compatibility | ✅ | Feature flag + zero regressions |
-| Write comprehensive tests | ✅ | 40 tests, 96.49% coverage (>90% req) |
-| Don't break existing frontend | ✅ | Normalization after expansion, 321 tests pass |
-| Support all three formats | ✅ | Copilot, Claude, Pi-Mono validated |
-
----
-
-## Example Transformations
-
-### Copilot Format
-**Input:**
-```javascript
-{
-  type: 'tool_use',
-  id: 'tool-123',
-  name: 'Read',
-  input: { file_path: '/test' },
-  result: 'File contents...',
-  _matched: true
-}
-```
-
-**Output:**
-```javascript
-{
-  id: 'tool-123',
-  name: 'Read',
-  startTime: '2024-01-01T10:00:00Z',
-  endTime: '2024-01-01T10:00:00Z',
-  status: 'completed',
-  input: { file_path: '/test' },
-  result: 'File contents...',
-  error: null,
-  metadata: {
-    source: 'copilot',
-    matched: true,
-    duration: 0
-  }
-}
-```
-
-### Claude Format
-**Input:**
-```javascript
-{
-  type: 'tool_use',
-  id: 'xyz789',
-  name: 'Write',
-  input: { file_path: '/new' },
-  _matched: false
-}
-```
-
-**Output:**
-```javascript
-{
-  id: 'xyz789',
-  name: 'Write',
-  startTime: '2024-01-01T10:00:00Z',
-  endTime: null,
-  status: 'running',
-  input: { file_path: '/new' },
-  result: null,
-  error: null,
-  metadata: {
-    source: 'claude',
-    matched: false
-  }
-}
-```
-
-### Pi-Mono Format
-**Input:**
-```javascript
-{
-  name: 'Bash',
-  input: { command: 'ls' },
-  result: 'Permission denied',
-  status: 'error',
-  isError: true
-}
-```
-
-**Output:**
-```javascript
-{
-  id: 'tool-1234567890-abc123',
-  name: 'Bash',
-  startTime: '2024-01-01T10:00:00Z',
-  endTime: '2024-01-01T10:00:00Z',
-  status: 'error',
-  input: { command: 'ls' },
-  result: null,
-  error: 'Permission denied',
-  metadata: {
-    source: 'pi-mono',
-    duration: 0
-  }
-}
-```
-
----
-
-## Next Steps: Phase 2-4 Rollout
-
-### Phase 2: Frontend (Week 3)
-- [ ] Update `views/session-vue.ejs` to use unified format
-- [ ] Remove `tool.type === 'tool_use'` checks
-- [ ] Remove `_matched` flag dependencies
-- [ ] Simplify `getToolGroups()` function
-- [ ] Add `getToolDuration()` helper
-- [ ] Manual testing with all formats
-
-### Phase 3: Timeline Views (Week 4)
-- [ ] Update `views/time-analyze.ejs`
-- [ ] Update `views/time-analyze-v2.ejs`
-- [ ] Remove embedded tool expansion logic
-- [ ] Test Gantt chart generation
-- [ ] Verify performance with large sessions
-
-### Phase 4: Cleanup (Week 5)
-- [ ] Remove `useUnifiedFormat` feature flag
-- [ ] Remove dead code (old format checks)
-- [ ] Update API documentation
-- [ ] Fix or remove 5 stale tests in coverage suite
-- [ ] Add inline comments for maintainers
-
----
-
-## Benefits Realized
-
-### Code Quality
-- ✅ Single source of truth for tool normalization
-- ✅ 275 lines of well-documented normalization logic
-- ✅ Comprehensive test coverage (40 tests, 96.49%)
-- ✅ Clear separation of concerns
-
-### Maintainability
-- ✅ Format changes localized to one module
-- ✅ Easy to extend for new formats
-- ✅ Simple to test (isolated normalizer)
-- ✅ Feature flag for safe rollout
-
-### Future-Proofing
-- ✅ API can evolve independently of parsers
-- ✅ New formats only require normalizer updates
-- ✅ Frontend can be simplified in Phase 2-3
-- ✅ Foundation for multi-client support
-
----
-
-## Technical Notes
-
-### Architecture Decision
-**Normalization Point:** After tool matching and expansion, before API response
-
-**Rationale:**
-- Parsers remain focused on format-specific reading
-- Matching logic can use original format (more efficient)
-- Frontend receives clean, consistent data
-- Single transformation point (easier to test)
-
-### Edge Cases Handled
-1. **Missing IDs:** Generated via `_generateToolId()`
-2. **Orphaned tools:** Fallback normalization with `status: 'running'`
-3. **Invalid timestamps:** Duration computation returns `undefined`
-4. **Negative durations:** Filtered out (returns `undefined`)
-5. **Null/undefined events:** Pass-through with warning
-6. **Empty tools array:** Preserved as-is
-7. **Unknown tool formats:** Fallback normalization with warning
-
-### Performance Considerations
-- Normalization adds ~1-2ms per 100 events
-- No noticeable impact on API response time
-- Streaming not affected (normalization post-load)
-- Memory footprint minimal (single pass transformation)
-
----
-
-## References
-
-- **Design Document:** `docs/unified-event-format-design.md`
-- **Implementation:** `src/services/eventNormalizer.js`
-- **Tests:** `src/services/__tests__/eventNormalizer.test.js`
-- **Integration:** `src/services/sessionService.js` (lines 7, 22, 240-242)
-
----
-
-## Document History
-
-| Date | Author | Changes |
-|------|--------|---------|
-| 2026-02-22 | Claude Opus 4.6 | Phase 1 implementation complete |
-
----
-
-**Phase 1 Status:** ✅ **COMPLETE AND VALIDATED**
-**Ready for:** Phase 2 (Frontend Integration)

package/eslint.config.mjs

@@ -1,133 +0,0 @@
-import js from '@eslint/js';
-import globals from 'globals';
-import pluginVue from 'eslint-plugin-vue';
-
-export default [
-  // Base JavaScript config
-  {
-    files: ['**/*.js', '**/*.mjs'],
-    languageOptions: {
-      ecmaVersion: 2022,
-      sourceType: 'module',
-      globals: {
-        ...globals.browser,
-        ...globals.node,
-        ...globals.es2021,
-      },
-    },
-    rules: {
-      ...js.configs.recommended.rules,
-      
-      // Error prevention
-      'no-unused-vars': ['error', { 
-        argsIgnorePattern: '^_', 
-        varsIgnorePattern: '^_',
-        caughtErrors: 'none'  // Ignore unused catch binding
-      }],
-      'no-undef': 'error',
-      'no-redeclare': 'error',
-      'no-constant-condition': 'warn',
-      
-      // Best practices
-      'eqeqeq': ['error', 'always'],
-      'no-eval': 'error',
-      'no-implied-eval': 'error',
-      'prefer-const': 'warn',
-      'no-var': 'warn',
-      
-      // Style (lenient for now)
-      'semi': ['warn', 'always'],
-      'quotes': ['warn', 'single', { avoidEscape: true }],
-      'indent': 'off', // Too strict for mixed codebases
-      'comma-dangle': 'off',
-      'no-trailing-spaces': 'off',
-    },
-  },
-  
-  // Vue 3 config - use the flat config array from the plugin
-  ...pluginVue.configs['flat/recommended'],
-  {
-    files: ['**/*.vue'],
-    rules: {
-      // Override Vue rules
-      'vue/multi-word-component-names': 'off', // Allow single-word component names
-      'vue/no-v-html': 'off', // We use v-html for markdown rendering
-      'vue/require-default-prop': 'warn',
-      'vue/require-prop-types': 'warn',
-      'vue/no-unused-vars': 'warn',
-      'vue/html-indent': 'off', // Too strict
-      'vue/max-attributes-per-line': 'off', // Too strict
-    },
-  },
-  
-  // Server-side files (Node.js only)
-  {
-    files: ['server.js', 'src/**/*.js', '__tests__/**/*.js'],
-    languageOptions: {
-      globals: {
-        ...globals.node,
-        ...globals.jest, // For test files
-      },
-    },
-  },
-  
-  // Test files
-  {
-    files: ['__tests__/**/*.js', '**/*.spec.js', '**/*.test.js'],
-    languageOptions: {
-      globals: {
-        ...globals.jest,
-        ...globals.node,
-      },
-    },
-    rules: {
-      'no-unused-expressions': 'off', // Common in tests
-    },
-  },
-  
-  // Frontend source files (browser globals from CDN)
-  {
-    files: ['src/frontend/**/*.js'],
-    languageOptions: {
-      sourceType: 'script',
-      globals: {
-        ...globals.browser,
-        Vue: 'readonly',
-        VueVirtualScroller: 'readonly',
-        marked: 'readonly',
-        DOMPurify: 'readonly',
-        trackClick: 'readonly',
-        trackPageView: 'readonly',
-        trackMetric: 'readonly',
-        appInsights: 'readonly',
-      },
-    },
-    rules: {
-      'no-unused-vars': ['error', { 
-        argsIgnorePattern: '^_', 
-        varsIgnorePattern: '^_|^app$',
-        caughtErrors: 'none'
-      }],
-      'eqeqeq': 'warn', // Relaxed for extracted legacy code
-      'prefer-const': 'off',
-      'no-useless-assignment': 'off',
-    },
-  },
-
-  // Ignore patterns
-  {
-    ignores: [
-      'node_modules/',
-      'coverage/',
-      'dist/',
-      '.git/',
-      '*.min.js',
-      'public/js/*.min.js',
-      'public/hyperlist.js', // External library
-      'check-*.js', // Debug scripts
-      'test-*.js', // Debug scripts
-      'debug-*.js', // Debug scripts
-      'verify-*.js', // Debug scripts
-    ],
-  },
-];

package/examples/parser-usage.js

@@ -1,114 +0,0 @@
-const { ParserFactory } = require('../lib/parsers');
-
-// Example: Parse Copilot CLI session
-function parseCopilotSession() {
-  const copilotEvents = [
-    {
-      type: 'session.start',
-      data: {
-        sessionId: '12345',
-        startTime: '2026-02-20T00:00:00Z',
-        selectedModel: 'claude-sonnet-4.5',
-        copilotVersion: '0.0.411',
-        context: {
-          cwd: '/path/to/project',
-          branch: 'main'
-        }
-      },
-      id: 'evt-1',
-      timestamp: '2026-02-20T00:00:00Z'
-    },
-    {
-      type: 'user.message',
-      data: {
-        content: 'Hello',
-        transformedContent: 'Hello'
-      },
-      id: 'evt-2',
-      timestamp: '2026-02-20T00:00:01Z',
-      parentId: 'evt-1'
-    },
-    {
-      type: 'assistant.message',
-      data: {
-        messageId: 'msg-1',
-        content: 'Hi there!',
-        toolRequests: []
-      },
-      id: 'evt-3',
-      timestamp: '2026-02-20T00:00:02Z',
-      parentId: 'evt-2'
-    }
-  ];
-
-  const factory = new ParserFactory();
-  const result = factory.parse(copilotEvents);
-  
-  console.log('=== Copilot Session ===');
-  console.log('Parser type:', factory.getParserType(copilotEvents));
-  console.log('Metadata:', JSON.stringify(result.metadata, null, 2));
-  console.log('Turns:', result.turns.length);
-  console.log('First turn:', JSON.stringify(result.turns[0], null, 2));
-}
-
-// Example: Parse Claude Code session
-function parseClaudeSession() {
-  const claudeEvents = [
-    {
-      type: 'user',
-      uuid: 'uuid-1',
-      parentUuid: null,
-      sessionId: 'session-123',
-      timestamp: '2026-02-20T00:00:00Z',
-      cwd: '/path/to/project',
-      gitBranch: 'main',
-      version: '2.1.42',
-      message: {
-        role: 'user',
-        content: 'Analyze this code'
-      }
-    },
-    {
-      type: 'assistant',
-      uuid: 'uuid-2',
-      parentUuid: 'uuid-1',
-      sessionId: 'session-123',
-      timestamp: '2026-02-20T00:00:01Z',
-      message: {
-        id: 'msg-1',
-        role: 'assistant',
-        model: 'claude-opus-4.6',
-        content: [
-          { type: 'text', text: "I'll analyze the code." },
-          {
-            type: 'tool_use',
-            id: 'tool-1',
-            name: 'read_file',
-            input: { path: 'src/main.js' }
-          }
-        ]
-      }
-    }
-  ];
-
-  const factory = new ParserFactory();
-  const result = factory.parse(claudeEvents);
-  
-  console.log('\n=== Claude Code Session ===');
-  console.log('Parser type:', factory.getParserType(claudeEvents));
-  console.log('Metadata:', JSON.stringify(result.metadata, null, 2));
-  console.log('Turns:', result.turns.length);
-  console.log('First turn:', JSON.stringify(result.turns[0], null, 2));
-  console.log('Tool calls:', result.toolCalls.length);
-}
-
-// Run examples
-if (require.main === module) {
-  parseCopilotSession();
-  parseClaudeSession();
-}
-
-module.exports = {
-  parseCopilotSession,
-  parseClaudeSession
-};