.woodpecker/ci.yml

@@ -12,6 +12,7 @@ steps:
     image: node:24-slim
     depends_on: []
     commands:
+      - npm ci
       - npm run test
     when:
       - event: pull_request

server/api/live/webrtc/create-transport.post.js

@@ -26,7 +26,7 @@ export default defineEventHandler(async (event) => {
   const url = getRequestURL(event)
   const requestHost = url.hostname
   const router = await getRouter(sessionId)
-  const { transport, params } = await createTransport(router, Boolean(isProducer), requestHost)
+  const { transport, params } = await createTransport(router, requestHost)
 
   if (isProducer) {
     updateLiveSession(sessionId, {

server/utils/liveSessions.js

@@ -20,7 +20,7 @@ export const createSession = (userId, label = '') => {
   return session
 }
 
-export const getLiveSession = (id) => sessions.get(id)
+export const getLiveSession = id => sessions.get(id)
 
 export const getActiveSessionByUserId = (userId) => {
   const now = Date.now()
@@ -41,7 +41,7 @@ export const updateLiveSession = (id, updates) => {
   session.updatedAt = now
 }
 
-export const deleteLiveSession = (id) => sessions.delete(id)
+export const deleteLiveSession = id => sessions.delete(id)
 
 export const clearSessions = () => sessions.clear()
 

server/utils/mediasoup.js

@@ -63,7 +63,7 @@ const resolveAnnouncedIp = (requestHost) => {
   return getAnnouncedIpFromInterfaces()
 }
 
-export const createTransport = async (router, _isProducer = false, requestHost = undefined) => {
+export const createTransport = async (router, requestHost = undefined) => {
   const announcedIp = resolveAnnouncedIp(requestHost)
   const listenIps = announcedIp
     ? [{ ip: '0.0.0.0', announcedIp }, { ip: '127.0.0.1' }]
@@ -94,7 +94,7 @@ export const createTransport = async (router, _isProducer = false, requestHost =
   }
 }
 
-export const getTransport = (transportId) => transports.get(transportId)
+export const getTransport = transportId => transports.get(transportId)
 
 export const createProducer = async (transport, track) => {
   const producer = await transport.produce({ track })
@@ -103,7 +103,7 @@ export const createProducer = async (transport, track) => {
   return producer
 }
 
-export const getProducer = (producerId) => producers.get(producerId)
+export const getProducer = producerId => producers.get(producerId)
 
 export const getTransports = () => transports
 

server/utils/webrtcSignaling.js

@@ -32,7 +32,7 @@ export async function handleWebSocketMessage(userId, sessionId, type, data) {
       }
       case 'create-transport': {
         const router = await getRouter(sessionId)
-        const { transport, params } = await createTransport(router, true)
+        const { transport, params } = await createTransport(router)
         updateLiveSession(sessionId, { transportId: transport.id, routerId: router.id })
         return { type: 'transport-created', data: params }
       }

test/unit/mediasoup.spec.js

@@ -25,7 +25,7 @@ describe('Mediasoup', () => {
 
   it('should create a transport', async () => {
     const router = await getRouter(sessionId)
-    const { transport, params } = await createTransport(router, true)
+    const { transport, params } = await createTransport(router)
     expect(transport).toBeDefined()
     expect(params.id).toBe(transport.id)
     expect(params.iceParameters).toBeDefined()
@@ -35,7 +35,7 @@ describe('Mediasoup', () => {
 
   it('should create a transport with requestHost IPv4 and return valid params', async () => {
     const router = await getRouter(sessionId)
-    const { transport, params } = await createTransport(router, true, '192.168.2.100')
+    const { transport, params } = await createTransport(router, '192.168.2.100')
     expect(transport).toBeDefined()
     expect(params.id).toBe(transport.id)
     expect(params.iceParameters).toBeDefined()