prettier

Author: neo773

apps/loom-importer-extension/src/background.ts

@@ -17,7 +17,6 @@ import { getServerBaseUrl, CapUrls } from "./utils/urls";
 
 const baseUrl = getServerBaseUrl();
 
-// Check for authentication token
 async function checkAuthToken(): Promise<string | null> {
   try {
     const cookie = await chrome.cookies.get({
@@ -41,13 +40,11 @@ async function checkAuthToken(): Promise<string | null> {
   }
 }
 
-// Verify token freshness and update if needed
 async function verifyTokenFreshness(): Promise<void> {
   try {
     const data = await chrome.storage.local.get("authData");
     const authData = data.authData as AuthResponse | undefined;
 
-    // If no token exists or token is older than 1 minute, refresh it
     if (!authData?.timestamp || Date.now() - authData.timestamp > 60000) {
       await checkAuthToken();
     }
@@ -56,7 +53,6 @@ async function verifyTokenFreshness(): Promise<void> {
   }
 }
 
-// Redirect to login if needed
 async function redirectToLogin(): Promise<void> {
   const tabs = await chrome.tabs.query({ active: true, currentWindow: true });
   const currentTab = tabs[0];
@@ -65,7 +61,6 @@ async function redirectToLogin(): Promise<void> {
   }
 }
 
-// Listen for cookie changes
 chrome.cookies.onChanged.addListener(async (changeInfo: CookieChangeInfo) => {
   if (
     changeInfo.cookie.domain.includes(baseUrl) &&
@@ -83,13 +78,9 @@ chrome.cookies.onChanged.addListener(async (changeInfo: CookieChangeInfo) => {
   }
 });
 
-// Check token freshness periodically
-setInterval(verifyTokenFreshness, 30000); // Check every 30 seconds
-
-// Forward checklist messages to popup
+setInterval(verifyTokenFreshness, 30000);
 function forwardToPopup(message: ImportMessage): void {
   try {
-    // chrome.runtime.sendMessage returns void in MV3, not a Promise
     chrome.runtime.sendMessage(message, (response) => {
       if (chrome.runtime.lastError) {
         console.warn(
@@ -106,17 +97,14 @@ function forwardToPopup(message: ImportMessage): void {
   }
 }
 
-// Store message listeners to avoid duplication
 const messageListeners = new Map();
 
-// Handle messages from content scripts and popup
 chrome.runtime.onMessage.addListener(
   (
     request: { action?: string; type?: string; [key: string]: any },
     sender: chrome.runtime.MessageSender,
     sendResponse: (response: any) => void
   ) => {
-    // Handle auth status requests from popup
     if (request.action === "getAuthStatus") {
       verifyTokenFreshness()
         .then(() => checkAuthToken())
@@ -130,26 +118,21 @@ chrome.runtime.onMessage.addListener(
           console.error("Error getting auth status:", error);
           sendResponse({ token: null, error: String(error) });
         });
-      return true; // Required for async response
+      return true;
     }
 
-    // Handle checklist messages from content scripts
     if (request.type && request.type.startsWith("CAP_")) {
-      // Log all Cap messages for debugging
       console.log(`Received Cap message: ${request.type}`, request);
 
-      // Store selected videos in local storage
       if (request.type === "CAP_LOOM_VIDEOS_SELECTED" && request.videos) {
         chrome.storage.local.set({ selectedVideos: request.videos });
 
-        // Create a copy of the message to avoid mutation issues
         const capMessage: ImportMessage = {
           type: request.type,
           videos: [...request.videos],
         };
         forwardToPopup(capMessage);
       } else if (request.type) {
-        // Forward all other Cap messages to the popup, ensuring type is defined
         const capMessage: ImportMessage = {
           type: request.type,
           ...request,
@@ -165,18 +148,13 @@ chrome.runtime.onMessage.addListener(
   }
 );
 
-// Allow external extensions to communicate with this extension
 chrome.runtime.onMessageExternal.addListener(
   (
     request: { type: string; [key: string]: any },
     sender: chrome.runtime.MessageSender,
     sendResponse: (response: any) => void
   ) => {
-    // Verify the sender is trusted if needed
-    // if (sender.id !== TRUSTED_EXTENSION_ID) return false;
-
     if (request.type && request.type.startsWith("CAP_")) {
-      // Make a copy of the message with a guaranteed type property
       const capMessage: ImportMessage = {
         ...request,
       };

apps/loom-importer-extension/src/content_scripts/App.tsx

@@ -3,12 +3,10 @@ import "./App.css";
 import { ImportProvider, useImport } from "../context/ImportContext";
 import ImportProgress from "../components/ImportProgress";
 
-// The inner App component that uses context
 const AppContent: React.FC = () => {
   const { importState, processVideos } = useImport();
   const { currentPage } = importState;
 
-  // Don't render anything if we're not on a relevant page
   if (currentPage === "other") return null;
 
   return (
@@ -23,7 +21,6 @@ const AppContent: React.FC = () => {
   );
 };
 
-// The main App component that provides context
 function App() {
   return (
     <ImportProvider>

apps/loom-importer-extension/src/context/ImportContext.tsx

@@ -3,8 +3,7 @@ import { LoomExportData } from "../types/loom";
 import * as LoomScraper from "../services/loomScraper";
 import { ImportStep, useImportStore } from "../store/importStore";
 
-export { ImportStep }; // Re-export for backward compatibility
-
+export { ImportStep };
 export interface ImportState {
   currentStep: ImportStep;
   error: string | null;
@@ -22,44 +21,36 @@ interface ImportContextType {
   resetImport: () => void;
 }
 
-// Create context with a default undefined value
 const ImportContext = createContext<ImportContextType | undefined>(undefined);
 
 export const ImportProvider: React.FC<{ children: ReactNode }> = ({
   children,
 }) => {
-  // Use the Zustand store
   const store = useImportStore();
 
-  // Initialize page detection
   useEffect(() => {
     store.initializePageDetection();
   }, []);
 
-  // Load existing data
   useEffect(() => {
     store.loadExistingData();
   }, [store.currentPage]);
 
-  // Setup member scraping
   useEffect(() => {
     const cleanup = store.setupMemberScraping();
     return cleanup;
   }, [store.currentPage, store.currentStep, store.data]);
 
-  // Setup workspace detection
   useEffect(() => {
     const cleanup = store.setupWorkspaceDetection();
     return cleanup;
   }, [store.currentPage, store.currentStep]);
 
-  // Setup video selection
   useEffect(() => {
     const cleanup = store.setupVideoSelection();
     return cleanup;
   }, [store.currentPage, store.currentStep]);
 
-  // Create an object that matches the original context shape
   const importContextValue: ImportContextType = {
     importState: {
       currentStep: store.currentStep,
@@ -80,7 +71,6 @@ export const ImportProvider: React.FC<{ children: ReactNode }> = ({
   );
 };
 
-// Hook to use the import context
 export const useImport = (): ImportContextType => {
   const context = useContext(ImportContext);
   if (context === undefined) {

apps/loom-importer-extension/src/hooks/useAuth.ts

@@ -20,7 +20,6 @@ export function useAuth() {
     user: null,
   });
 
-  // Memoize the API instance to prevent it from causing re-renders
   const api = useMemo(() => new CapApi(), []);
 
   const fetchUserData = useCallback(async () => {
@@ -103,7 +102,6 @@ export function useAuth() {
       console.error("Error in useAuth:", error);
     }
 
-    // Cleanup function to prevent state updates if the component unmounts
     return () => {
       isMounted = false;
     };

apps/loom-importer-extension/src/hooks/useWorkspaces.ts

@@ -12,7 +12,6 @@ export function useWorkspaces(isAuthenticated: boolean) {
   const [isError, setIsError] = useState(false);
   const [status, setStatus] = useState("");
 
-  // Memoize the API instance to prevent it from causing re-renders
   const api = useMemo(() => new CapApi(), []);
 
   useEffect(() => {
@@ -35,7 +34,6 @@ export function useWorkspaces(isAuthenticated: boolean) {
 
     fetchWorkspaces();
 
-    // Cleanup function to prevent state updates if the component unmounts
     return () => {
       isMounted = false;
     };

apps/loom-importer-extension/src/popup/popup.tsx

@@ -57,7 +57,6 @@ const PopupContent = () => {
   };
 
   const handleEmailSelected = (email: string) => {
-    // Call setSelectedUserEmail directly
     setSelectedUserEmail(email);
   };
 
@@ -75,7 +74,6 @@ const PopupContent = () => {
     }
   };
 
-  // Get the checklist items using the centralized utility
   const checklistItems = getChecklistItemsForStep(importState.currentStep);
 
   return (

apps/loom-importer-extension/src/services/loomScraper.ts

@@ -2,7 +2,6 @@ import { LoomExportData, Video, WorkspaceMember } from "../types/loom";
 import { waitForElement } from "../utils/dom";
 import JSConfetti from "js-confetti";
 
-// Page types
 export type LoomPage = "members" | "workspace" | "other";
 
 /**
@@ -39,13 +38,11 @@ export const loadExistingData = (): Promise<LoomExportData | null> => {
  * Scrapes workspace members from the Loom members page
  */
 export const scrapeWorkspaceMembers = async (): Promise<WorkspaceMember[]> => {
-  // Wait for the table to load
   const tableElement = await waitForElement('div[role="table"]');
   if (!tableElement) {
     throw new Error("Table element not found");
   }
 
-  // Give it a bit more time to ensure data is fully loaded
   await new Promise((resolve) => setTimeout(resolve, 1000));
 
   const members: WorkspaceMember[] = [];
@@ -119,20 +116,17 @@ export const saveMembersToStorage = async (
 export const setupVideoSelection = async (
   onSelectionChange: (hasSelectedVideos: boolean) => void
 ): Promise<() => void> => {
-  // Wait for video elements
   const videoElement = await waitForElement("article[data-videoid]");
   if (!videoElement) {
     throw new Error("No videos found on page");
   }
 
-  // Use Loom's native checkboxes
   const articles = document.querySelectorAll<HTMLElement>(
     "article[data-videoid]"
   );
 
   const listeners: { element: HTMLElement; listener: EventListener }[] = [];
 
-  // Add event listeners to all checkboxes
   articles.forEach((article) => {
     const videoId = article.getAttribute("data-videoid");
     if (!videoId) return;
@@ -143,20 +137,17 @@ export const setupVideoSelection = async (
     if (!checkbox) return;
 
     const listener = () => {
-      // Check if any videos are selected
       const anyVideoSelected = Array.from(
         document.querySelectorAll<HTMLInputElement>('input[id^="bulk-action-"]')
       ).some((cb) => cb.checked);
 
-      // Call the callback with the result
       onSelectionChange(anyVideoSelected);
     };
 
     checkbox.addEventListener("change", listener);
     listeners.push({ element: checkbox, listener });
   });
 
-  // Return a cleanup function to remove all event listeners
   return () => {
     listeners.forEach(({ element, listener }) => {
       element.removeEventListener("change", listener);
@@ -215,7 +206,6 @@ export const processVideos = (
   workspaceMembers: WorkspaceMember[]
 ): Video[] => {
   return rawVideos.map((video) => {
-    // Find the workspace member that matches the owner name
     const owner = workspaceMembers.find((member) =>
       member.name.includes(video.ownerName)
     ) || { name: video.ownerName, email: "" };
@@ -242,7 +232,6 @@ export const saveProcessedVideos = async (
 
   return new Promise((resolve) => {
     chrome.storage.local.set({ loomImportData: updatedData }, () => {
-      // Notify extension that import is complete
       chrome.runtime.sendMessage({
         type: "CAP_LOOM_IMPORT_COMPLETE",
         data: updatedData,
@@ -263,16 +252,13 @@ export const completeVideoImport = async (
   currentData: LoomExportData
 ): Promise<LoomExportData> => {
   try {
-    // Get selected videos
     const rawVideos = getSelectedVideos();
 
-    // Process videos
     const processedVideos = processVideos(
       rawVideos,
       currentData.workspaceMembers
     );
 
-    // Save videos
     return await saveProcessedVideos(currentData, processedVideos);
   } catch (error) {
     console.error("Failed to process videos:", error);