Merge pull request #6 from vernonthedev/develop

refactor: migrated project structure and added tests

Author: vernonthedev

.babelrc.json

@@ -0,0 +1,12 @@
+{
+  "presets": [
+    [
+      "@babel/preset-env",
+      {
+        "targets": {
+          "node": "current"
+        }
+      }
+    ]
+  ]
+}

content.js assets/content.js.backupcontent.js renamed to assets/content.js.backup

@@ -43,6 +43,40 @@ class GitHubGitLabTheme {
     this.setupNavigationListener();
   }
 
+isRepositoryPage() {
+    const pathname = window.location.pathname;
+    
+    // Check for user profile repositories page
+    if (pathname.match(/^\/[^\/]+$/)) {
+      const pageHeader = document.querySelector('h1');
+      if (pageHeader && pageHeader.textContent.includes('Repositories')) {
+        return true;
+      }
+    }
+    
+    // Check for user repositories tab
+    if (pathname.includes('?tab=repositories')) {
+      return true;
+    }
+    
+    // Check for organization repositories page
+    if (pathname.includes('/orgs/') && 
+        (pathname.includes('/repositories') || 
+         document.querySelector('[data-test-selector="org-repositories-list"]'))) {
+      return true;
+    }
+    
+    // Check for repository list containers
+    const repoContainers = [
+      '#user-repositories-list',
+      '#org-repositories-list',
+      '[data-testid="repository-list-container"]',
+      'div[data-test-selector="org-repositories-list"]'
+    ];
+    
+    return repoContainers.some(selector => document.querySelector(selector));
+  }
+
   setupNavigationListener() {
     // Listen for URL changes (GitHub is an SPA)
     let currentUrl = window.location.href;
@@ -55,7 +89,7 @@ class GitHubGitLabTheme {
           this.run();
         }, 500);
       }
-    }).observe(document, { subtree: true, childList: true });
+    }).observe(document.body, { subtree: true, childList: true });
   }
 
   applyDarkTheme() {
@@ -182,10 +216,25 @@ class GitHubGitLabTheme {
     await this.saveSetting('customGroups', Array.from(this.customGroups));
   }
 
-  run() {
+run() {
+    // Only run on repository-related pages
+    if (!this.isRepositoryPage()) {
+      console.log('[GitLab Theme] Not a repository page, skipping processing');
+      return;
+    }
+    
     // Apply dark theme first
     this.applyDarkTheme();
 
+    // Check if we're on an organization page for debugging
+    const isOrgPage = window.location.pathname.includes('/orgs/') || 
+                      window.location.pathname.match(/^\/[^\/]+$/) && 
+                      document.querySelector('[data-test-selector="org-header"]');
+    
+    if (isOrgPage) {
+      console.log('[GitLab Theme] Organization page detected');
+    }
+    
     // Setup observer for dynamic content
     this.setupMutationObserver();
 
@@ -194,7 +243,7 @@ class GitHubGitLabTheme {
     this.processRepositories();
   }
 
-  setupMutationObserver() {
+setupMutationObserver() {
     if (this.observer) this.observer.disconnect();
 
     this.observer = new MutationObserver((mutations) => {
@@ -211,36 +260,56 @@ class GitHubGitLabTheme {
       if (shouldProcess) {
         if (this.debounceTimer) clearTimeout(this.debounceTimer);
         this.debounceTimer = setTimeout(() => {
-          this.addGroupControls();
-          this.processRepositories();
+          // Only process if we're still on a repository page
+          if (this.isRepositoryPage()) {
+            this.addGroupControls();
+            this.processRepositories();
+          }
         }, 300);
       }
     });
 
-    this.observer.observe(document.body, {
-      childList: true,
-      subtree: true
+    // Only observe specific containers, not the entire body
+    const targetSelectors = [
+      '#user-repositories-list',
+      '#org-repositories-list',
+      '[data-testid="repository-list-container"]',
+      'div[data-test-selector="org-repositories-list"]',
+      'main[role="main"]'
+    ];
+
+    targetSelectors.forEach(selector => {
+      const element = document.querySelector(selector);
+      if (element) {
+        this.observer.observe(element, {
+          childList: true,
+          subtree: true
+        });
+      }
     });
   }
 
-  findRepositoryContainers() {
+findRepositoryContainers() {
     const selectors = [
       '#user-repositories-list',
       '#org-repositories-list', 
       '[data-testid="repository-list-container"]',
-      '.repo-list',
       '[data-filterable-for="your-repos-filter"]',
+      '[data-filterable-for="org-repos-filter"]',
       '.js-repo-list',
       'ul[data-test-selector="profile-repository-list"]',
-      'div[aria-label="Repositories"]'
+      'div[aria-label="Repositories"]',
+      // Org page specific selectors
+      'div[data-test-selector="org-repositories-list"]',
+      '[data-test-selector="org-repo-list"]'
     ];
 
     const containers = [];
     selectors.forEach(selector => {
       try {
         const elements = document.querySelectorAll(selector);
         elements.forEach(el => {
-          if (el && !this.processedContainers.has(el)) {
+          if (el && !this.processedContainers.has(el) && this.isValidRepositoryContainer(el)) {
             containers.push(el);
             this.processedContainers.add(el);
           }
@@ -250,9 +319,16 @@ class GitHubGitLabTheme {
       }
     });
 
+    console.log(`[GitLab Theme] Found ${containers.length} repository containers`);
     return containers;
   }
 
+  isValidRepositoryContainer(container) {
+    // Check if container has repository items
+    const repoItems = this.findRepositoryItems(container);
+    return repoItems.length > 0;
+  }
+
   findRepositoryItems(container) {
     const itemSelectors = [
       '[itemprop="owns"]',
@@ -264,15 +340,21 @@ class GitHubGitLabTheme {
       '.fork',
       '.archived',
       'li[itemprop="owns"]',
-      'div[data-testid="repository-item"]'
+      'div[data-testid="repository-item"]',
+      // Org page specific selectors
+      'li[data-test-selector="repository-list-item"]',
+      'div[data-test-selector="repository-list-item"]',
+      '.Box-row', // GitHub uses Box-row for repo items on org pages
+      'li[itemprop="codeRepository"]',
+      'div[itemprop="codeRepository"]'
     ];
 
     let items = [];
     itemSelectors.forEach(selector => {
       try {
         const found = container.querySelectorAll(selector);
         found.forEach(item => {
-          const nameElement = item.querySelector('h3 a, a[itemprop="name codeRepository"], .wb-break-all a, [data-testid="repository-name"]');
+          const nameElement = item.querySelector('h3 a, a[itemprop="name codeRepository"], .wb-break-all a, [data-testid="repository-name"], [itemprop="name"], .Link--primary');
           if (nameElement && !items.includes(item)) {
             items.push(item);
           }
@@ -285,7 +367,7 @@ class GitHubGitLabTheme {
     return items;
   }
 
-  processRepositories() {
+processRepositories() {
     if (this.isProcessing) return;
     this.isProcessing = true;
 
@@ -299,12 +381,15 @@ class GitHubGitLabTheme {
         const items = this.findRepositoryItems(container);
 
         if (items.length > 0) {
-          if (this.groupingEnabled) {
-            this.createGroupCards(container, items);
-          } else {
-            this.displayAllRepos(container, items);
+          // Only process if container hasn't been processed already
+          if (!container.dataset.gitlabProcessed) {
+            if (this.groupingEnabled) {
+              this.createGroupCards(container, items);
+            } else {
+              this.displayAllRepos(container, items);
+            }
+            container.dataset.gitlabProcessed = 'true';
           }
-          container.dataset.gitlabProcessed = 'true';
         }
       });
 
@@ -315,46 +400,56 @@ class GitHubGitLabTheme {
     }
   }
 
-  createGroupCards(container, items) {
+createGroupCards(container, items) {
     const groups = this.extractGroups(items);
 
     if (groups.size <= 1) {
       this.displayAllRepos(container, items);
       return;
     }
 
-    // Clear existing content
-    container.innerHTML = '';
-    container.classList.add('gitlab-grouped-repositories');
-
-    const fragment = document.createDocumentFragment();
+    // Store original content and clear container safely
+    const originalContent = container.innerHTML;
+    const originalClasses = container.className;
 
-    // Create group cards section
-    const groupCardsSection = this.createGroupCardsSection(groups, container);
-    fragment.appendChild(groupCardsSection);
+    try {
+      container.innerHTML = '';
+      container.classList.add('gitlab-grouped-repositories');
 
-    // Create hidden repo containers for each group
-    const repoContainersSection = this.createRepoContainers(groups, container);
-    fragment.appendChild(repoContainersSection);
+      const fragment = document.createDocumentFragment();
+      
+      // Create group cards section
+      const groupCardsSection = this.createGroupCardsSection(groups, container);
+      fragment.appendChild(groupCardsSection);
 
-    container.appendChild(fragment);
+      // Create hidden repo containers for each group
+      const repoContainersSection = this.createRepoContainers(groups, container);
+      fragment.appendChild(repoContainersSection);
 
-    // Show the first group by default
-    const firstGroup = Array.from(groups.keys())[0];
-    console.log(`[GitLab Theme] Auto-showing first group: ${firstGroup}`);
-    
-    // Wait a bit for DOM to settle, then simulate first group click
-    setTimeout(() => {
-      console.log(`[GitLab Theme] Attempting to show first group...`);
-      const firstCard = container.querySelector(`.gitlab-group-card[data-group-id="${firstGroup}"]`);
-      if (firstCard) {
-        console.log(`[GitLab Theme] Found first group card, simulating click`);
-        firstCard.click();
-      } else {
-        console.error(`[GitLab Theme] Could not find first group card: ${firstGroup}`);
-        console.log(`[GitLab Theme] Available cards:`, container.querySelectorAll('.gitlab-group-card'));
-      }
-    }, 200);
+      container.appendChild(fragment);
+
+      // Show the first group by default
+      const firstGroup = Array.from(groups.keys())[0];
+      console.log(`[GitLab Theme] Auto-showing first group: ${firstGroup}`);
+      
+      // Wait a bit for DOM to settle, then simulate first group click
+      setTimeout(() => {
+        console.log(`[GitLab Theme] Attempting to show first group...`);
+        const firstCard = container.querySelector(`.gitlab-group-card[data-group-id="${firstGroup}"]`);
+        if (firstCard) {
+          console.log(`[GitLab Theme] Found first group card, simulating click`);
+          firstCard.click();
+        } else {
+          console.error(`[GitLab Theme] Could not find first group card: ${firstGroup}`);
+          console.log(`[GitLab Theme] Available cards:`, container.querySelectorAll('.gitlab-group-card'));
+        }
+      }, 200);
+    } catch (error) {
+      console.error('[GitLab Theme] Error creating group cards, reverting to original content:', error);
+      container.innerHTML = originalContent;
+      container.className = originalClasses;
+      this.displayAllRepos(container, items);
+    }
   }
 
   extractGroups(items) {
@@ -568,7 +663,12 @@ class GitHubGitLabTheme {
     reposSection.scrollIntoView({ behavior: 'smooth', block: 'start' });
   }
 
-  displayAllRepos(container, items) {
+displayAllRepos(container, items) {
+    // Only modify if we haven't already processed this container
+    if (container.dataset.gitlabProcessed === 'true') {
+      return;
+    }
+
     const fragment = document.createDocumentFragment();
 
     items.forEach(item => {
@@ -593,12 +693,15 @@ class GitHubGitLabTheme {
       'h3 a',
       'a[itemprop="name codeRepository"]',
       '.wb-break-all a',
-      '[data-testid="repository-name"]'
+      '[data-testid="repository-name"]',
+      '[itemprop="name"]',
+      '.Link--primary',
+      'a[href*="/"][title]'
     ];
 
     for (const selector of nameSelectors) {
       const element = item.querySelector(selector);
-      if (element) {
+      if (element && element.textContent.trim()) {
         return element.textContent.trim();
       }
     }
@@ -635,7 +738,10 @@ class GitHubGitLabTheme {
     const repoSections = [
       '#user-repositories-list',
       '#org-repositories-list',
-      '[data-testid="repository-list-container"]'
+      '[data-testid="repository-list-container"]',
+      'div[data-test-selector="org-repositories-list"]',
+      '.org-repos',
+      '#org-repositories'
     ];
 
     for (const selector of repoSections) {