Refactor package manager implementations and add comprehensive tests

- Removed unnecessary constructor parameters and simplified isConfigured methods in Chocolatey, Gentoo, Homebrew, Nix, RPM, Scoop, and Winget package managers.
- Updated generateManifest methods to return promises consistently across package managers.
- Enhanced error handling and logging in RPM package manager.
- Added a new CrossForkPRSubmissionParams interface for better PR submission handling.
- Implemented unit tests for all package managers, ensuring correct manifest generation and submission logic.
- Created a Vitest configuration file for running tests.

release:patch

Author: ralyodio

.claude/settings.local.json

@@ -202,7 +202,8 @@
       "Bash(set +a)",
       "Bash(gh workflow run:*)",
       "Bash(gh workflow:*)",
-      "Bash(pnpm -F @pairux/web typecheck:*)"
+      "Bash(pnpm -F @pairux/web typecheck:*)",
+      "Bash(./.husky/pre-commit)"
     ],
     "deny": [
       "Bash(npm *)",

.github/workflows/submit-packages.yml

@@ -95,7 +95,7 @@ jobs:
           # If "all" or contains chocolatey, we need to exclude chocolatey from linux job
           if [ "$INPUT_PMS" = "all" ]; then
             # Run all except chocolatey on Linux
-            PMS="homebrew,scoop,winget,aur,apt,rpm"
+            PMS="homebrew,scoop,winget,aur,apt,rpm,gentoo,nix"
           else
             # Remove chocolatey from the list for Linux job
             PMS=$(echo "$INPUT_PMS" | sed 's/chocolatey,//g' | sed 's/,chocolatey//g' | sed 's/^chocolatey$//g')

apps/desktop/src/main/capture/sources.ts

@@ -31,7 +31,9 @@ export async function getCaptureSources(types: ('screen' | 'window')[]): Promise
       } satisfies CaptureSource;
     });
   } catch (error) {
+    // On Wayland and some Linux systems, desktopCapturer may not work
+    // Return empty array instead of throwing to allow fallback to getDisplayMedia
     console.error('[Capture] Failed to get capture sources:', error);
-    throw error;
+    return [];
   }
 }

apps/desktop/src/main/tray.test.ts

@@ -25,6 +25,7 @@ vi.mock('electron', () => {
     nativeImage: {
       createFromPath: vi.fn().mockReturnValue({}),
       createFromBuffer: vi.fn().mockReturnValue({}),
+      createEmpty: vi.fn().mockReturnValue({}),
     },
     app: {
       isPackaged: false,

apps/desktop/src/main/tray.ts

@@ -44,36 +44,34 @@ function createTrayIcon(status: TrayStatus): Electron.NativeImage {
   };
 
   const iconPath = join(resourcePath, iconMap[status]);
+  console.log('[Tray] Looking for icon at:', iconPath, 'exists:', existsSync(iconPath));
 
   if (existsSync(iconPath)) {
-    return nativeImage.createFromPath(iconPath);
+    const icon = nativeImage.createFromPath(iconPath);
+    if (!icon.isEmpty()) {
+      return icon;
+    }
+    console.warn('[Tray] Icon loaded but is empty:', iconPath);
   }
 
-  // Fallback: create simple colored circle icons programmatically
-  const size = process.platform === 'darwin' ? 22 : 16;
-  const colors: Record<TrayStatus, string> = {
-    idle: '#6b7280', // Gray
-    active: '#22c55e', // Green
-    paused: '#f59e0b', // Amber
-    error: '#ef4444', // Red
-  };
+  // Fallback: use the main app icon
+  const mainIconPath = app.isPackaged
+    ? join(process.resourcesPath, 'icon.png')
+    : join(__dirname, '../../resources/icon.png');
+
+  console.log('[Tray] Trying fallback icon at:', mainIconPath, 'exists:', existsSync(mainIconPath));
 
-  const color = colors[status];
-
-  // Create a simple SVG circle
-  const cx = String(size / 2);
-  const cy = String(size / 2);
-  const outerR = String(size / 2 - 1);
-  const innerR = String(size / 4);
-  const svg = `
-    <svg width="${String(size)}" height="${String(size)}" xmlns="http://www.w3.org/2000/svg">
-      <circle cx="${cx}" cy="${cy}" r="${outerR}" fill="${color}"/>
-      <circle cx="${cx}" cy="${cy}" r="${innerR}" fill="white"/>
-    </svg>
-  `;
-
-  const buffer = Buffer.from(svg);
-  return nativeImage.createFromBuffer(buffer);
+  if (existsSync(mainIconPath)) {
+    const icon = nativeImage.createFromPath(mainIconPath);
+    if (!icon.isEmpty()) {
+      // Resize for tray
+      const size = process.platform === 'darwin' ? 22 : 16;
+      return icon.resize({ width: size, height: size });
+    }
+  }
+
+  console.warn('[Tray] No icon found, using empty image');
+  return nativeImage.createEmpty();
 }
 
 /**
@@ -194,10 +192,23 @@ export function initializeTray(trayCallbacks: TrayCallbacks): Tray {
   const icon = createTrayIcon('idle');
   tray = new Tray(icon);
 
-  tray.setContextMenu(buildContextMenu());
+  const contextMenu = buildContextMenu();
+  tray.setContextMenu(contextMenu);
   updateTooltip();
 
-  // Double-click to show window (Windows/Linux)
+  // On Linux, single-click should show the context menu
+  // On Windows/macOS, single-click behavior varies by platform
+  tray.on('click', () => {
+    if (process.platform === 'linux') {
+      // On Linux, explicitly pop up the context menu on click
+      tray?.popUpContextMenu(contextMenu);
+    } else {
+      // On Windows/macOS, single-click shows window
+      callbacks?.onShowWindow();
+    }
+  });
+
+  // Double-click to show window (all platforms)
   tray.on('double-click', () => {
     callbacks?.onShowWindow();
   });

apps/desktop/src/renderer/components/capture/CapturePreview.tsx

@@ -333,14 +333,16 @@ export function CapturePreview({
                       setRecordingQuality(e.target.value as RecordingQuality);
                     }}
                     className="rounded bg-background px-2 py-1 text-xs"
+                    title="Recording quality - affects file size and bitrate"
                   >
-                    <option value="720p">720p</option>
-                    <option value="1080p">1080p</option>
-                    <option value="4k">4K</option>
+                    <option value="720p">720p (2.5 Mbps)</option>
+                    <option value="1080p">1080p (5 Mbps)</option>
+                    <option value="4k">4K (15 Mbps)</option>
                   </select>
                   <button
                     onClick={() => void handleStartRecording()}
                     className="flex items-center gap-1.5 rounded-md bg-red-600 px-3 py-1.5 text-sm font-medium text-white transition-colors hover:bg-red-700"
+                    title="Start recording to local file"
                   >
                     <Circle className="h-3 w-3 fill-current" />
                     Record

apps/desktop/src/renderer/components/layout/TitleBar.tsx

@@ -23,8 +23,7 @@ export function TitleBar() {
       } pr-4`}
     >
       <div className="no-drag flex items-center gap-2">
-        <img src="/logo.svg" alt="PairUX" className="h-4 w-4" />
-        <span className="text-sm font-medium">PairUX</span>
+        <img src="/favicon.svg" alt="PairUX" className="h-5 w-auto" />
       </div>
 
       <div className="flex-1" />

apps/web/src/components/Logo.tsx

@@ -1,45 +1,34 @@
-import { Monitor } from 'lucide-react';
+import Image from 'next/image';
 import Link from 'next/link';
 import { cn } from '@/lib/utils';
 
 interface LogoProps {
   size?: 'sm' | 'md' | 'lg';
-  showText?: boolean;
   className?: string;
 }
 
 const sizes = {
-  sm: {
-    container: 'h-8 w-8',
-    icon: 'h-4 w-4',
-    text: 'text-lg',
-  },
-  md: {
-    container: 'h-9 w-9',
-    icon: 'h-5 w-5',
-    text: 'text-xl',
-  },
-  lg: {
-    container: 'h-10 w-10',
-    icon: 'h-6 w-6',
-    text: 'text-2xl',
-  },
+  sm: { height: 24 },
+  md: { height: 32 },
+  lg: { height: 40 },
 };
 
-export function Logo({ size = 'md', showText = true, className }: LogoProps) {
+export function Logo({ size = 'md', className }: LogoProps) {
   const sizeConfig = sizes[size];
+  // Logo aspect ratio is approximately 2.83:1 (512.75 / 181.44)
+  const width = Math.round(sizeConfig.height * 2.83);
 
   return (
-    <Link href="/" className={cn('flex items-center gap-2', className)}>
-      <div
-        className={cn(
-          'bg-primary-600 flex items-center justify-center rounded-lg',
-          sizeConfig.container
-        )}
-      >
-        <Monitor className={cn('text-white', sizeConfig.icon)} />
-      </div>
-      {showText && <span className={cn('font-bold text-gray-900', sizeConfig.text)}>PairUX</span>}
+    <Link href="/" className={cn('flex items-center', className)}>
+      <Image
+        src="/logo.svg"
+        alt="PairUX"
+        width={width}
+        height={sizeConfig.height}
+        className="h-auto"
+        style={{ height: sizeConfig.height }}
+        priority
+      />
     </Link>
   );
 }

apps/web/src/components/footer.tsx

@@ -47,7 +47,7 @@ const footerLinks = {
       href: 'https://community.chocolatey.org/packages/pairux',
       external: true,
     },
-    { name: 'AUR (Arch)', href: 'https://aur.archlinux.org/packages/pairux', external: true },
+    { name: 'AUR (Arch)', href: 'https://aur.archlinux.org/packages/pairux-bin', external: true },
     { name: 'APT (Debian)', href: 'https://github.com/profullstack/pairux-apt', external: true },
     { name: 'RPM (Fedora)', href: 'https://github.com/profullstack/pairux-rpm', external: true },
     { name: 'Gentoo', href: 'https://github.com/profullstack/gentoo-pairux', external: true },
@@ -87,9 +87,8 @@ export function Footer() {
         <div className="grid grid-cols-2 gap-8 lg:grid-cols-5">
           {/* Brand */}
           <div className="col-span-2 lg:col-span-1">
-            <Link href="/" className="flex items-center gap-2">
-              <Image src="/logo.svg" alt="PairUX" width={36} height={36} className="h-9 w-9" />
-              <span className="text-xl font-bold text-gray-900">PairUX</span>
+            <Link href="/" className="flex items-center">
+              <Image src="/logo.svg" alt="PairUX" width={113} height={40} className="h-10 w-auto" />
             </Link>
             <p className="mt-4 text-sm text-gray-600">
               Collaborative screen sharing with simultaneous remote control. Like Screenhero, but

apps/web/src/lib/supabase/server.ts

@@ -4,6 +4,18 @@ import { createClient as createSupabaseClient } from '@supabase/supabase-js';
 import { cookies, headers } from 'next/headers';
 import type { Database } from '@pairux/shared-types';
 
+/**
+ * Extract Bearer token from Authorization header if present.
+ */
+export async function getBearerToken(): Promise<string | null> {
+  const headerStore = await headers();
+  const authHeader = headerStore.get('authorization');
+  if (authHeader?.startsWith('Bearer ')) {
+    return authHeader.slice(7);
+  }
+  return null;
+}
+
 /**
  * Create a Supabase client for server-side use.
  * Supports both cookie-based auth (web browser) and Bearer token auth (desktop app).
@@ -17,23 +29,25 @@ export async function createClient() {
   }
 
   // Check for Bearer token in Authorization header (desktop app)
-  const headerStore = await headers();
-  const authHeader = headerStore.get('authorization');
+  const token = await getBearerToken();
 
-  if (authHeader?.startsWith('Bearer ')) {
-    const token = authHeader.slice(7);
-    // Desktop app: use token-based auth
-    return createSupabaseClient<Database>(url, key, {
-      global: {
-        headers: {
-          Authorization: `Bearer ${token}`,
-        },
-      },
+  if (token) {
+    // Desktop app: create client and set the session with the access token
+    const supabase = createSupabaseClient<Database>(url, key, {
       auth: {
         persistSession: false,
         autoRefreshToken: false,
       },
     });
+
+    // Set the auth session using the access token
+    // This allows getUser() to work correctly
+    await supabase.auth.setSession({
+      access_token: token,
+      refresh_token: '', // We don't have refresh token in header, but it's required
+    });
+
+    return supabase;
   }
 
   // Web browser: use cookie-based auth