gh-here 3.0.2 → 3.1.0

package/public/js/draft-manager.js

@@ -1,36 +0,0 @@
-/**
- * Draft management for file editing
- */
-
-import { STORAGE_KEYS } from './constants.js';
-
-export const DraftManager = {
-  saveDraft(filePath, content) {
-    localStorage.setItem(`${STORAGE_KEYS.DRAFT_PREFIX}${filePath}`, content);
-  },
-
-  loadDraft(filePath) {
-    return localStorage.getItem(`${STORAGE_KEYS.DRAFT_PREFIX}${filePath}`);
-  },
-
-  clearDraft(filePath) {
-    localStorage.removeItem(`${STORAGE_KEYS.DRAFT_PREFIX}${filePath}`);
-  },
-
-  hasDraftChanges(filePath, originalContent) {
-    const draft = this.loadDraft(filePath);
-    return draft !== null && draft !== originalContent;
-  },
-
-  getAllDrafts() {
-    const drafts = {};
-    for (let i = 0; i < localStorage.length; i++) {
-      const key = localStorage.key(i);
-      if (key.startsWith(STORAGE_KEYS.DRAFT_PREFIX)) {
-        const filePath = key.replace(STORAGE_KEYS.DRAFT_PREFIX, '');
-        drafts[filePath] = localStorage.getItem(key);
-      }
-    }
-    return drafts;
-  }
-};

package/public/js/editor-manager.js

@@ -1,159 +0,0 @@
-/**
- * Monaco Editor management
- */
-
-import { CONFIG, EDITOR_OPTIONS } from './constants.js';
-import { getLanguageFromExtension } from './utils.js';
-import { DraftManager } from './draft-manager.js';
-import { showDraftDialog } from './modal-manager.js';
-import { showNotification } from './notification.js';
-
-export class EditorManager {
-  constructor(theme) {
-    this.fileEditor = null;
-    this.newFileEditor = null;
-    this.theme = theme;
-    this.ready = false;
-    this.init();
-  }
-
-  init() {
-    if (typeof require === 'undefined') {
-      return;
-    }
-
-    require.config({ paths: { vs: CONFIG.MONACO_CDN } });
-
-    require(['vs/editor/editor.main'], () => {
-      self.MonacoEnvironment = {
-        getWorker: () => undefined
-      };
-
-      const monacoTheme = this.theme === 'dark' ? 'vs-dark' : 'vs';
-      monaco.editor.setTheme(monacoTheme);
-
-      this.initializeNewFileEditor();
-      this.ready = true;
-      window.monacoReady = true;
-    });
-  }
-
-  initializeNewFileEditor() {
-    const container = document.getElementById('new-file-content');
-    if (!container) {
-      return;
-    }
-
-    const monacoTheme = this.theme === 'dark' ? 'vs-dark' : 'vs';
-    this.newFileEditor = monaco.editor.create(container, {
-      ...EDITOR_OPTIONS,
-      value: '',
-      language: 'plaintext',
-      theme: monacoTheme
-    });
-  }
-
-  async createFileEditor(container, filePath, originalContent) {
-    if (!this.ready) {
-      await this.waitForReady();
-    }
-
-    const filename = filePath.split('/').pop() || 'file.txt';
-    const language = getLanguageFromExtension(filename);
-    const availableLanguages = monaco.languages.getLanguages().map(lang => lang.id);
-    const validLanguage = availableLanguages.includes(language) ? language : 'plaintext';
-    const monacoTheme = this.theme === 'dark' ? 'vs-dark' : 'vs';
-
-    let contentToLoad = originalContent;
-
-    if (DraftManager.hasDraftChanges(filePath, originalContent)) {
-      const draftChoice = await showDraftDialog(filePath);
-      if (draftChoice === 'load') {
-        contentToLoad = DraftManager.loadDraft(filePath);
-      } else if (draftChoice === 'discard') {
-        DraftManager.clearDraft(filePath);
-      }
-    } else {
-      const draft = DraftManager.loadDraft(filePath);
-      if (draft && draft === originalContent) {
-        DraftManager.clearDraft(filePath);
-      }
-    }
-
-    if (!this.fileEditor) {
-      this.fileEditor = monaco.editor.create(container, {
-        ...EDITOR_OPTIONS,
-        value: contentToLoad,
-        language: validLanguage,
-        theme: monacoTheme
-      });
-
-      this.fileEditor.onDidChangeModelContent(() => {
-        DraftManager.saveDraft(filePath, this.fileEditor.getValue());
-      });
-    } else {
-      this.fileEditor.setValue(contentToLoad);
-      const model = this.fileEditor.getModel();
-      if (model) {
-        monaco.editor.setModelLanguage(model, validLanguage);
-      }
-    }
-
-    setTimeout(() => this.fileEditor.layout(), 50);
-
-    return this.fileEditor;
-  }
-
-  waitForReady() {
-    return new Promise(resolve => {
-      const check = () => {
-        if (this.ready) {
-          resolve();
-        } else {
-          setTimeout(check, 100);
-        }
-      };
-      check();
-    });
-  }
-
-  updateLanguage(filename) {
-    if (this.newFileEditor) {
-      const language = getLanguageFromExtension(filename);
-      const model = this.newFileEditor.getModel();
-      if (model) {
-        monaco.editor.setModelLanguage(model, language);
-      }
-    }
-  }
-
-  toggleWordWrap(editor) {
-    if (!editor) {
-      return false;
-    }
-
-    const currentWrap = editor.getOption(monaco.editor.EditorOption.wordWrap);
-    const newWrap = currentWrap === 'off' ? 'on' : 'off';
-    editor.updateOptions({ wordWrap: newWrap });
-    return newWrap === 'on';
-  }
-
-  getValue(editorType = 'file') {
-    const editor = editorType === 'file' ? this.fileEditor : this.newFileEditor;
-    return editor ? editor.getValue() : '';
-  }
-
-  focus(editorType = 'file') {
-    const editor = editorType === 'file' ? this.fileEditor : this.newFileEditor;
-    if (editor) {
-      editor.focus();
-    }
-  }
-
-  layout(editorType = 'file') {
-    const editor = editorType === 'file' ? this.fileEditor : this.newFileEditor;
-    if (editor) {
-      editor.layout();
-    }
-  }
-}

package/test.js

@@ -1,138 +0,0 @@
-/**
- * Lightweight smoke tests for gh-here
- * Run with: node test.js
- */
-
-const { chromium } = require('playwright');
-const { spawn } = require('child_process');
-
-const TEST_PORT = 5556; // Use different port to avoid conflicts
-const BASE_URL = `http://localhost:${TEST_PORT}`;
-
-async function sleep(ms) {
-  return new Promise(resolve => setTimeout(resolve, ms));
-}
-
-async function runTests() {
-  console.log('Starting gh-here server...');
-
-  // Start server
-  const server = spawn('node', ['bin/gh-here.js', `--port=${TEST_PORT}`], {
-    cwd: __dirname,
-    stdio: 'pipe'
-  });
-
-  // Wait for server to start
-  await sleep(2000);
-
-  let browser;
-  let failures = 0;
-
-  try {
-    console.log('Launching browser...');
-    browser = await chromium.launch({ headless: true });
-    const context = await browser.newContext();
-    const page = await context.newPage();
-
-    // Test 1: Root page loads
-    console.log('\n✓ Test 1: Root page loads');
-    await page.goto(BASE_URL);
-    const title = await page.title();
-    if (!title.includes('gh-here')) {
-      console.error('  ✗ FAILED: Page title incorrect');
-      failures++;
-    }
-
-    // Test 2: File tree exists (may be hidden on root)
-    console.log('✓ Test 2: File tree element exists');
-    const fileTreeExists = await page.$('#file-tree');
-    if (!fileTreeExists) {
-      console.error('  ✗ FAILED: File tree element not found');
-      failures++;
-    }
-
-    // Test 3: Navigate to a file and check line numbers display correctly
-    console.log('✓ Test 3: File view with line numbers');
-    await page.goto(`${BASE_URL}/?path=lib/renderers.js`);
-    await page.waitForSelector('.line-container', { timeout: 5000 });
-
-    // Check that line numbers are in a vertical column (not nested)
-    const lineContainers = await page.$$('.line-container');
-    if (lineContainers.length < 10) {
-      console.error('  ✗ FAILED: Not enough line containers found');
-      failures++;
-    }
-
-    // Check line 1 and line 2 have sequential numbers
-    const line1 = await page.$('.line-container[data-line="1"]');
-    const line2 = await page.$('.line-container[data-line="2"]');
-    if (!line1 || !line2) {
-      console.error('  ✗ FAILED: Line containers missing data-line attributes');
-      failures++;
-    }
-
-    // Verify line numbers are not nested (check display property)
-    const line1Display = await page.$eval('.line-container[data-line="1"]',
-      el => window.getComputedStyle(el).display
-    );
-    if (line1Display !== 'block') {
-      console.error(`  ✗ FAILED: Line containers should have display:block, got ${line1Display}`);
-      failures++;
-    }
-
-    // Test 4: Check if modified files show diff button
-    console.log('✓ Test 4: Modified files show diff button');
-    await page.goto(BASE_URL);
-    const diffButtons = await page.$$('.diff-btn');
-    // Should have at least one diff button if there are modified files
-    console.log(`  Found ${diffButtons.length} diff buttons`);
-
-    // Test 5: Gitignore toggle exists and is clickable
-    console.log('✓ Test 5: Gitignore toggle button');
-    const gitignoreToggle = await page.$('#gitignore-toggle');
-    if (!gitignoreToggle) {
-      console.error('  ✗ FAILED: Gitignore toggle button not found');
-      failures++;
-    }
-
-    // Test 6: Theme toggle works
-    console.log('✓ Test 6: Theme toggle');
-    const themeToggle = await page.$('#theme-toggle');
-    if (!themeToggle) {
-      console.error('  ✗ FAILED: Theme toggle button not found');
-      failures++;
-    }
-
-    // Test 7: Search functionality
-    console.log('✓ Test 7: Search input exists');
-    const searchInput = await page.$('#file-search, #root-file-search');
-    if (!searchInput) {
-      console.error('  ✗ FAILED: Search input not found');
-      failures++;
-    }
-
-    console.log('\n' + '='.repeat(50));
-    if (failures === 0) {
-      console.log('✓ All tests passed!');
-    } else {
-      console.log(`✗ ${failures} test(s) failed`);
-    }
-    console.log('='.repeat(50) + '\n');
-
-  } catch (error) {
-    console.error('\n✗ Test failed with error:', error.message);
-    failures++;
-  } finally {
-    if (browser) {
-      await browser.close();
-    }
-
-    // Kill server
-    server.kill();
-
-    // Exit with appropriate code
-    process.exit(failures > 0 ? 1 : 0);
-  }
-}
-
-runTests();

package/tests/draftManager.test.js

@@ -1,241 +0,0 @@
-// Tests for DraftManager utility
-// Run with: node tests/draftManager.test.js
-
-// Mock localStorage for testing
-class MockLocalStorage {
-  constructor() {
-    this.store = {};
-  }
-
-  getItem(key) {
-    return this.store.hasOwnProperty(key) ? this.store[key] : null;
-  }
-
-  setItem(key, value) {
-    this.store[key] = String(value);
-  }
-
-  removeItem(key) {
-    delete this.store[key];
-  }
-
-  clear() {
-    this.store = {};
-  }
-
-  key(index) {
-    const keys = Object.keys(this.store);
-    return keys[index] || null;
-  }
-
-  get length() {
-    return Object.keys(this.store).length;
-  }
-}
-
-// Create testable version of DraftManager with injected storage
-function createDraftManager(storage) {
-  return {
-    STORAGE_PREFIX: 'gh-here-draft-',
-    
-    saveDraft(filePath, content) {
-      storage.setItem(`${this.STORAGE_PREFIX}${filePath}`, content);
-    },
-    
-    loadDraft(filePath) {
-      return storage.getItem(`${this.STORAGE_PREFIX}${filePath}`);
-    },
-    
-    clearDraft(filePath) {
-      storage.removeItem(`${this.STORAGE_PREFIX}${filePath}`);
-    },
-    
-    hasDraftChanges(filePath, originalContent) {
-      const draft = this.loadDraft(filePath);
-      return draft !== null && draft !== originalContent;
-    },
-    
-    getAllDrafts() {
-      const drafts = {};
-      for (let i = 0; i < storage.length; i++) {
-        const key = storage.key(i);
-        if (key && key.startsWith(this.STORAGE_PREFIX)) {
-          const filePath = key.replace(this.STORAGE_PREFIX, '');
-          drafts[filePath] = storage.getItem(key);
-        }
-      }
-      return drafts;
-    }
-  };
-}
-
-// Simple test framework
-function test(name, fn) {
-  try {
-    fn();
-    console.log(`✅ ${name}`);
-  } catch (error) {
-    console.log(`❌ ${name}: ${error.message}`);
-  }
-}
-
-function assertEqual(actual, expected, message = '') {
-  if (actual !== expected) {
-    throw new Error(`Expected "${expected}" but got "${actual}". ${message}`);
-  }
-}
-
-function assertNull(actual, message = '') {
-  if (actual !== null) {
-    throw new Error(`Expected null but got "${actual}". ${message}`);
-  }
-}
-
-function assertTrue(actual, message = '') {
-  if (!actual) {
-    throw new Error(`Expected truthy but got "${actual}". ${message}`);
-  }
-}
-
-function assertFalse(actual, message = '') {
-  if (actual) {
-    throw new Error(`Expected falsy but got "${actual}". ${message}`);
-  }
-}
-
-// Test cases
-console.log('🧪 Testing Draft Manager\n');
-
-test('saves and loads drafts correctly', () => {
-  const storage = new MockLocalStorage();
-  const draftManager = createDraftManager(storage);
-  
-  const filePath = '/src/app.js';
-  const content = 'console.log("hello world");';
-  
-  // Initially no draft
-  assertNull(draftManager.loadDraft(filePath));
-  
-  // Save draft
-  draftManager.saveDraft(filePath, content);
-  
-  // Load draft
-  assertEqual(draftManager.loadDraft(filePath), content);
-});
-
-test('clears drafts correctly', () => {
-  const storage = new MockLocalStorage();
-  const draftManager = createDraftManager(storage);
-  
-  const filePath = '/src/app.js';
-  const content = 'console.log("test");';
-  
-  // Save and verify
-  draftManager.saveDraft(filePath, content);
-  assertEqual(draftManager.loadDraft(filePath), content);
-  
-  // Clear and verify
-  draftManager.clearDraft(filePath);
-  assertNull(draftManager.loadDraft(filePath));
-});
-
-test('detects draft changes correctly', () => {
-  const storage = new MockLocalStorage();
-  const draftManager = createDraftManager(storage);
-  
-  const filePath = '/src/app.js';
-  const originalContent = 'const x = 1;';
-  const draftContent = 'const x = 2;';
-  
-  // No draft initially
-  assertFalse(draftManager.hasDraftChanges(filePath, originalContent));
-  
-  // Save draft with different content
-  draftManager.saveDraft(filePath, draftContent);
-  assertTrue(draftManager.hasDraftChanges(filePath, originalContent));
-  
-  // Save draft with same content
-  draftManager.saveDraft(filePath, originalContent);
-  assertFalse(draftManager.hasDraftChanges(filePath, originalContent));
-});
-
-test('gets all drafts correctly', () => {
-  const storage = new MockLocalStorage();
-  const draftManager = createDraftManager(storage);
-  
-  // Add some non-draft items to storage
-  storage.setItem('other-key', 'other-value');
-  
-  // Add drafts
-  draftManager.saveDraft('/src/app.js', 'app content');
-  draftManager.saveDraft('/src/utils.js', 'utils content');
-  draftManager.saveDraft('/README.md', 'readme content');
-  
-  const allDrafts = draftManager.getAllDrafts();
-  
-  // Should only return draft items
-  assertEqual(Object.keys(allDrafts).length, 3);
-  assertEqual(allDrafts['/src/app.js'], 'app content');
-  assertEqual(allDrafts['/src/utils.js'], 'utils content');
-  assertEqual(allDrafts['/README.md'], 'readme content');
-  
-  // Should not include non-draft items
-  assertEqual(allDrafts['other-key'], undefined);
-});
-
-test('handles multiple drafts independently', () => {
-  const storage = new MockLocalStorage();
-  const draftManager = createDraftManager(storage);
-  
-  const file1 = '/src/app.js';
-  const file2 = '/src/utils.js';
-  const content1 = 'app content';
-  const content2 = 'utils content';
-  
-  // Save drafts for different files
-  draftManager.saveDraft(file1, content1);
-  draftManager.saveDraft(file2, content2);
-  
-  // Each file has its own draft
-  assertEqual(draftManager.loadDraft(file1), content1);
-  assertEqual(draftManager.loadDraft(file2), content2);
-  
-  // Clear one draft, other remains
-  draftManager.clearDraft(file1);
-  assertNull(draftManager.loadDraft(file1));
-  assertEqual(draftManager.loadDraft(file2), content2);
-});
-
-test('uses correct storage prefix', () => {
-  const storage = new MockLocalStorage();
-  const draftManager = createDraftManager(storage);
-  
-  const filePath = '/src/app.js';
-  const content = 'test content';
-  
-  draftManager.saveDraft(filePath, content);
-  
-  // Check that correct key is used in storage
-  const expectedKey = 'gh-here-draft-/src/app.js';
-  assertEqual(storage.getItem(expectedKey), content);
-  
-  // Verify storage length
-  assertEqual(storage.length, 1);
-});
-
-test('handles empty content correctly', () => {
-  const storage = new MockLocalStorage();
-  const draftManager = createDraftManager(storage);
-  
-  const filePath = '/empty.js';
-  const emptyContent = '';
-  
-  draftManager.saveDraft(filePath, emptyContent);
-  assertEqual(draftManager.loadDraft(filePath), ''); // localStorage stores empty string as empty string
-  
-  // Empty content should be detected as different from non-empty
-  assertTrue(draftManager.hasDraftChanges(filePath, 'non-empty'), 'Empty draft should be different from non-empty original');
-  assertFalse(draftManager.hasDraftChanges(filePath, ''));
-});
-
-console.log('\n🎉 Draft Manager tests complete!');

package/tests/fileTypeDetection.test.js

@@ -1,111 +0,0 @@
-// Simple unit tests for file type detection
-// Run with: node tests/fileTypeDetection.test.js
-
-// Import the functions from our file-utils module
-const { isImageFile, isBinaryFile, isTextFile } = require('../lib/file-utils');
-
-function test(description, testFn) {
-  try {
-    testFn();
-    console.log(`✅ ${description}`);
-  } catch (error) {
-    console.log(`❌ ${description}`);
-    console.log(`   Error: ${error.message}`);
-  }
-}
-
-function assert(condition, message) {
-  if (!condition) {
-    throw new Error(message || 'Assertion failed');
-  }
-}
-
-// Test isImageFile function
-test('should detect PNG files as images', () => {
-  assert(isImageFile('photo.png'), 'PNG should be detected as image');
-  assert(isImageFile('PNG'), 'Extension-only PNG should work');
-});
-
-test('should detect common image formats', () => {
-  const imageFiles = ['test.jpg', 'test.jpeg', 'test.gif', 'test.svg', 'test.webp', 'test.bmp', 'test.tiff', 'test.ico'];
-  imageFiles.forEach(file => {
-    assert(isImageFile(file), `${file} should be detected as image`);
-  });
-});
-
-test('should not detect text files as images', () => {
-  const textFiles = ['readme.md', 'script.js', 'style.css', 'config.json'];
-  textFiles.forEach(file => {
-    assert(!isImageFile(file), `${file} should not be detected as image`);
-  });
-});
-
-// Test isBinaryFile function
-test('should detect images as binary files', () => {
-  assert(isBinaryFile('photo.png'), 'Images should be binary');
-  assert(isBinaryFile('animation.gif'), 'GIFs should be binary');
-});
-
-test('should detect archives as binary files', () => {
-  const archives = ['file.zip', 'backup.tar', 'compressed.gz', 'archive.rar', 'package.7z'];
-  archives.forEach(file => {
-    assert(isBinaryFile(file), `${file} should be detected as binary`);
-  });
-});
-
-test('should detect executables as binary files', () => {
-  const executables = ['program.exe', 'binary.bin', 'application.app'];
-  executables.forEach(file => {
-    assert(isBinaryFile(file), `${file} should be detected as binary`);
-  });
-});
-
-test('should detect documents as binary files', () => {
-  const docs = ['document.pdf', 'spreadsheet.xlsx', 'presentation.pptx'];
-  docs.forEach(file => {
-    assert(isBinaryFile(file), `${file} should be detected as binary`);
-  });
-});
-
-test('should not detect text files as binary', () => {
-  const textFiles = ['readme.md', 'script.js', 'style.css', 'data.json', 'config.yml', 'index.html'];
-  textFiles.forEach(file => {
-    assert(!isBinaryFile(file), `${file} should not be detected as binary`);
-  });
-});
-
-// Test isTextFile function
-test('should detect common text files', () => {
-  const textFiles = ['readme.md', 'script.js', 'style.css', 'data.json', 'config.yml', 'index.html', 'app.py', 'main.go'];
-  textFiles.forEach(file => {
-    assert(isTextFile(file), `${file} should be detected as text`);
-  });
-});
-
-test('should not detect binary files as text', () => {
-  const binaryFiles = ['photo.png', 'archive.zip', 'program.exe', 'document.pdf'];
-  binaryFiles.forEach(file => {
-    assert(!isTextFile(file), `${file} should not be detected as text`);
-  });
-});
-
-// Test edge cases
-test('should handle files without extensions', () => {
-  assert(isTextFile('README'), 'Files without extensions should default to text');
-  assert(isTextFile('Makefile'), 'Common text files without extensions should be text');
-});
-
-test('should handle extension-only input', () => {
-  assert(isImageFile('png'), 'Should handle bare extensions');
-  assert(isBinaryFile('exe'), 'Should handle bare extensions for binary');
-  assert(isTextFile('js'), 'Should handle bare extensions for text');
-});
-
-test('should be case insensitive', () => {
-  assert(isImageFile('PHOTO.PNG'), 'Should handle uppercase extensions');
-  assert(isBinaryFile('ARCHIVE.ZIP'), 'Should handle uppercase extensions');
-  assert(isTextFile('SCRIPT.JS'), 'Should handle uppercase extensions');
-});
-
-// Run all tests
-console.log('Running file type detection tests...\n');