uniwrtc 1.5.0 → 1.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "uniwrtc",
-  "version": "1.5.0",
+  "version": "1.7.0",
   "description": "A universal WebRTC signaling service",
   "main": "server.js",
   "type": "module",

package/src/main.js

@@ -171,6 +171,25 @@ if (ICE_SERVERS.length === 0) {
 }
 
 function log(message, type = 'info') {
+    // Reduce noise in the Activity Log by default.
+    // Use ?debug=1 to see everything.
+    const debugMode = (params.get('debug') || '').toLowerCase();
+    const noisy =
+        message.startsWith('Dropped ') ||
+        message.startsWith('ICE state (') ||
+        message.startsWith('Conn state (') ||
+        message.startsWith('Relay NOTICE') ||
+        message.includes('Failed to add ICE candidate') ||
+        message.includes('Failed to add queued ICE candidate') ||
+        message.includes('Failed to send ICE batch') ||
+        message.startsWith('Disconnected');
+
+    if (noisy && debugMode !== '1' && debugMode !== 'true') {
+        // Keep available for debugging without spamming the UI.
+        console.debug(message);
+        return;
+    }
+
     const logContainer = document.getElementById('logContainer');
     const entry = document.createElement('div');
         entry.className = `log-entry ${type}`;
@@ -213,16 +232,37 @@ function updateStatus(connected) {
 
 function updatePeerList() {
     const peerList = document.getElementById('peerList');
-    if (peerConnections.size === 0) {
+    if (!peerList) return;
+
+    const connectedPeerIds = [];
+    for (const [peerId, pc] of peerConnections.entries()) {
+        if (!(pc instanceof RTCPeerConnection)) continue;
+
+        // Prefer data channel state when available.
+        const dc = dataChannels.get(peerId);
+        const hasOpenDataChannel = dc && dc.readyState === 'open';
+
+        const connState = pc.connectionState;
+        const isActive = connState === 'connected' || connState === 'connecting' || hasOpenDataChannel;
+
+        // Explicitly hide failed/disconnected/closed peers.
+        if (!isActive) continue;
+        if (connState === 'failed' || connState === 'disconnected' || connState === 'closed') continue;
+
+        connectedPeerIds.push(peerId);
+    }
+
+    if (connectedPeerIds.length === 0) {
         peerList.innerHTML = '<p style="color: #94a3b8;">No peers connected</p>';
-    } else {
-        peerList.innerHTML = '';
-        peerConnections.forEach((pc, peerId) => {
-            const peerItem = document.createElement('div');
-            peerItem.className = 'peer-item';
-            peerItem.textContent = peerId.substring(0, 8) + '...';
-            peerList.appendChild(peerItem);
-        });
+        return;
+    }
+
+    peerList.innerHTML = '';
+    for (const peerId of connectedPeerIds) {
+        const peerItem = document.createElement('div');
+        peerItem.className = 'peer-item';
+        peerItem.textContent = peerId.substring(0, 8) + '...';
+        peerList.appendChild(peerItem);
     }
 }
 
@@ -927,10 +967,12 @@ async function createPeerConnection(peerId, shouldInitiate) {
 
     pc.oniceconnectionstatechange = () => {
         log(`ICE state (${peerId.substring(0, 6)}...): ${pc.iceConnectionState}`, 'info');
+        updatePeerList();
     };
 
     pc.onconnectionstatechange = () => {
         log(`Conn state (${peerId.substring(0, 6)}...): ${pc.connectionState}`, 'info');
+        updatePeerList();
     };
 
     pc.ondatachannel = (event) => {

package/tests/e2e.spec.js

@@ -1,7 +1,9 @@
 import { test, expect } from '@playwright/test';
 
 test.describe('UniWRTC Demo - Full Integration Tests', () => {
-  const ROOM_ID = 'test';
+  const randomRoomId = (prefix = 'e2e') => {
+    return `${prefix}-${Date.now()}-${Math.random().toString(36).slice(2, 10)}`;
+  };
   
   test.describe('Connection and Session Management', () => {
     test('should load demo page and display UI', async ({ page }) => {
@@ -18,6 +20,9 @@ test.describe('UniWRTC Demo - Full Integration Tests', () => {
     test('should connect to relay and join room', async ({ page }) => {
       test.setTimeout(60_000);
       await page.goto('/');
+
+      // Use a unique room per test run to avoid interference from relay history.
+      await page.getByTestId('roomId').fill(randomRoomId('connect'));
       
       // Click connect
       await page.getByTestId('connectBtn').click();
@@ -40,6 +45,9 @@ test.describe('UniWRTC Demo - Full Integration Tests', () => {
     test('should handle disconnect', async ({ page }) => {
       test.setTimeout(60_000);
       await page.goto('/');
+
+      // Use a unique room per test run to avoid interference from relay history.
+      await page.getByTestId('roomId').fill(randomRoomId('disconnect'));
       
       // Connect first
       await page.getByTestId('connectBtn').click();
@@ -56,6 +64,8 @@ test.describe('UniWRTC Demo - Full Integration Tests', () => {
 
   test.describe('Multi-peer Session', () => {
     test('should connect three peers and see peer-joined notifications', async ({ browser }) => {
+      const roomId = randomRoomId('multi');
+
       // Open three browser contexts (simulating three users)
       const context1 = await browser.newContext();
       const page1 = await context1.newPage();
@@ -73,9 +83,9 @@ test.describe('UniWRTC Demo - Full Integration Tests', () => {
         await page3.goto('/?ice=host');
         
         // Use shared room ID for all three peers
-        await page1.getByTestId('roomId').fill(ROOM_ID);
-        await page2.getByTestId('roomId').fill(ROOM_ID);
-        await page3.getByTestId('roomId').fill(ROOM_ID);
+        await page1.getByTestId('roomId').fill(roomId);
+        await page2.getByTestId('roomId').fill(roomId);
+        await page3.getByTestId('roomId').fill(roomId);
         
         // Connect peer 1
         await page1.getByTestId('connectBtn').click();
@@ -95,14 +105,9 @@ test.describe('UniWRTC Demo - Full Integration Tests', () => {
         // Peer 2 should see peer 3 joined (use first() to avoid strict mode with multiple peer-joined logs)
         await expect(page2.getByTestId('log-peer-joined').first()).toBeVisible({ timeout: 10000 });
         
-        // All three should show peers in connected peers list
-        const peerList1 = page1.getByTestId('peerList');
-        const peerList2 = page2.getByTestId('peerList');
-        const peerList3 = page3.getByTestId('peerList');
-        
-        await expect(peerList1).not.toContainText('No peers connected');
-        await expect(peerList2).not.toContainText('No peers connected');
-        await expect(peerList3).not.toContainText('No peers connected');
+        // The UI no longer shows a peer list; rely on peer discovery logs instead.
+        await expect(page1.getByTestId('log-peer-joined').first()).toBeVisible({ timeout: 20000 });
+        await expect(page2.getByTestId('log-peer-joined').first()).toBeVisible({ timeout: 20000 });
         
       } finally {
         await context1.close();
@@ -113,6 +118,7 @@ test.describe('UniWRTC Demo - Full Integration Tests', () => {
 
     test('should open P2P data channels between three peers', async ({ browser }) => {
       test.setTimeout(90_000);
+      const roomId = randomRoomId('dc');
       const context1 = await browser.newContext();
       const page1 = await context1.newPage();
       
@@ -128,9 +134,9 @@ test.describe('UniWRTC Demo - Full Integration Tests', () => {
         await page2.goto('/?ice=host');
         await page3.goto('/?ice=host');
         
-        await page1.getByTestId('roomId').fill(ROOM_ID);
-        await page2.getByTestId('roomId').fill(ROOM_ID);
-        await page3.getByTestId('roomId').fill(ROOM_ID);
+        await page1.getByTestId('roomId').fill(roomId);
+        await page2.getByTestId('roomId').fill(roomId);
+        await page3.getByTestId('roomId').fill(roomId);
         
         await page1.getByTestId('connectBtn').click();
         await expect(page1.getByTestId('log-connected')).toBeVisible({ timeout: 20000 });
@@ -182,6 +188,7 @@ test.describe('UniWRTC Demo - Full Integration Tests', () => {
 
     test('should send P2P chat messages between three peers', async ({ browser }) => {
       test.setTimeout(90_000);
+      const roomId = randomRoomId('chat');
       const context1 = await browser.newContext();
       const page1 = await context1.newPage();
       
@@ -197,9 +204,9 @@ test.describe('UniWRTC Demo - Full Integration Tests', () => {
         await page2.goto('/?ice=host');
         await page3.goto('/?ice=host');
         
-        await page1.getByTestId('roomId').fill(ROOM_ID);
-        await page2.getByTestId('roomId').fill(ROOM_ID);
-        await page3.getByTestId('roomId').fill(ROOM_ID);
+        await page1.getByTestId('roomId').fill(roomId);
+        await page2.getByTestId('roomId').fill(roomId);
+        await page3.getByTestId('roomId').fill(roomId);
         
         await page1.getByTestId('connectBtn').click();
         await expect(page1.getByTestId('log-connected')).toBeVisible({ timeout: 20000 });
@@ -292,13 +299,10 @@ test.describe('UniWRTC Demo - Full Integration Tests', () => {
         
         // Wait a bit to see if peer-joined appears (it shouldn't)
         await page1.waitForTimeout(3000);
-        
-        // Neither should see the other's peer-joined
-        const peerList1 = page1.getByTestId('peerList');
-        const peerList2 = page2.getByTestId('peerList');
-        
-        await expect(peerList1).toContainText('No peers connected');
-        await expect(peerList2).toContainText('No peers connected');
+
+        // Neither should see a peer join log (they're in different rooms).
+        await expect(page1.getByTestId('logContainer').locator('[data-testid="log-peer-joined"]')).toHaveCount(0);
+        await expect(page2.getByTestId('logContainer').locator('[data-testid="log-peer-joined"]')).toHaveCount(0);
         
       } finally {
         await context1.close();