Fix broken builds (#529)

lukasIO · web-flow · commit ba26f8c40ee5 · 2025-08-28T14:57:10.000+02:00
* update proto files and downgrade typescript

* fix server-sdk-build

* separate typescript from non breaking dev deps

* prettier
diff --git a/package.json b/package.json
@@ -38,7 +38,7 @@
     "eslint-plugin-tsdoc": "^0.4.0",
     "prettier": "^3.2.5",
     "turbo": "^2.0.0",
-    "typescript": "^5.4.5",
+    "typescript": "5.8.2",
     "vitest": "^3.0.0"
   },
   "packageManager": "pnpm@9.15.7"
diff --git a/packages/livekit-rtc/package.json b/packages/livekit-rtc/package.json
@@ -46,7 +46,7 @@
     }
   },
   "dependencies": {
-    "@bufbuild/protobuf": "^1.10.0",
+    "@bufbuild/protobuf": "^1.10.1",
     "@livekit/mutex": "^1.0.0",
     "@livekit/typed-emitter": "^3.0.0",
     "pino": "^9.0.0",
@@ -57,8 +57,8 @@
     "@types/node": "^22.13.10",
     "prettier": "^3.0.3",
     "tsup": "^8.3.5",
-    "typescript": "^5.2.2",
-    "@bufbuild/protoc-gen-es": "^1.10.0"
+    "typescript": "5.8.2",
+    "@bufbuild/protoc-gen-es": "^1.10.1"
   },
   "optionalDependencies": {
     "@livekit/rtc-node-darwin-arm64": "workspace:*",
diff --git a/packages/livekit-rtc/src/data_streams/stream_reader.ts b/packages/livekit-rtc/src/data_streams/stream_reader.ts
@@ -58,8 +58,8 @@ export class ByteStreamReader extends BaseStreamReader<ByteStreamInfo> {
             this.handleChunkReceived(value);
             return { done: false, value: value.content! };
           }
-        } catch (error) {
-          log.error('error processing stream update', error);
+        } catch (error: any) {
+          log.error('error processing stream update: %s', error);
           return { done: true, value: undefined };
         }
       },
@@ -135,8 +135,8 @@ export class TextStreamReader extends BaseStreamReader<TextStreamInfo> {
               value: decoder.decode(value.content!),
             };
           }
-        } catch (error) {
-          log.error('error processing stream update', error);
+        } catch (error: any) {
+          log.error('error processing stream update: %s', error);
           return { done: true, value: undefined };
         }
       },
diff --git a/packages/livekit-rtc/src/proto/audio_frame_pb.ts b/packages/livekit-rtc/src/proto/audio_frame_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file audio_frame.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/data_stream_pb.ts b/packages/livekit-rtc/src/proto/data_stream_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file data_stream.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/e2ee_pb.ts b/packages/livekit-rtc/src/proto/e2ee_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file e2ee.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/ffi_pb.ts b/packages/livekit-rtc/src/proto/ffi_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file ffi.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/handle_pb.ts b/packages/livekit-rtc/src/proto/handle_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file handle.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/participant_pb.ts b/packages/livekit-rtc/src/proto/participant_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file participant.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/room_pb.ts b/packages/livekit-rtc/src/proto/room_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file room.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/rpc_pb.ts b/packages/livekit-rtc/src/proto/rpc_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file rpc.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/stats_pb.ts b/packages/livekit-rtc/src/proto/stats_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file stats.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/track_pb.ts b/packages/livekit-rtc/src/proto/track_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file track.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/track_publication_pb.ts b/packages/livekit-rtc/src/proto/track_publication_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file track_publication.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/proto/video_frame_pb.ts b/packages/livekit-rtc/src/proto/video_frame_pb.ts
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-es v1.10.0 with parameter "target=ts,import_extension=.js"
+// @generated by protoc-gen-es v1.10.1 with parameter "target=ts,import_extension=.js"
 // @generated from file video_frame.proto (package livekit.proto, syntax proto2)
 /* eslint-disable */
 // @ts-nocheck
diff --git a/packages/livekit-rtc/src/room.ts b/packages/livekit-rtc/src/room.ts
@@ -574,7 +574,10 @@ export class Room extends (EventEmitter as new () => TypedEmitter<RoomCallbacks>
       const streamHandlerCallback = this.byteStreamHandlers.get(streamHeader.topic ?? '');
 
       if (!streamHandlerCallback) {
-        log.debug('ignoring incoming byte stream due to no handler for topic', streamHeader.topic);
+        log.debug(
+          'ignoring incoming byte stream due to no handler for topic: %s',
+          streamHeader.topic ?? 'undefined',
+        );
         return;
       }
       let streamController: ReadableStreamDefaultController<DataStream_Chunk>;
@@ -605,7 +608,10 @@ export class Room extends (EventEmitter as new () => TypedEmitter<RoomCallbacks>
       const streamHandlerCallback = this.textStreamHandlers.get(streamHeader.topic ?? '');
 
       if (!streamHandlerCallback) {
-        log.debug('ignoring incoming text stream due to no handler for topic', streamHeader.topic);
+        log.debug(
+          'ignoring incoming text stream due to no handler for topic: %s',
+          streamHeader.topic ?? 'undefined',
+        );
         return;
       }
       let streamController: ReadableStreamDefaultController<DataStream_Chunk>;
diff --git a/packages/livekit-server-sdk/package.json b/packages/livekit-server-sdk/package.json
@@ -42,7 +42,7 @@
     "test:edge": "vitest --environment edge-runtime run"
   },
   "dependencies": {
-    "@bufbuild/protobuf": "^1.7.2",
+    "@bufbuild/protobuf": "^1.10.1",
     "@livekit/protocol": "^1.39.0",
     "camelcase-keys": "^9.0.0",
     "jose": "^5.1.2"
@@ -56,7 +56,7 @@
     "prettier": "^3.0.0",
     "tsup": "^8.3.5",
     "typedoc": "^0.28.0",
-    "typescript": "5.9.x",
+    "typescript": "5.8.2",
     "vite": "^5.2.9",
     "vitest": "^3.0.0"
   },
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/renovate.json b/renovate.json

Original file line number	Diff line number	Diff line change
`@@ -58,8 +58,8 @@ export class ByteStreamReader extends BaseStreamReader<ByteStreamInfo> {`
`58`	`58`	`this.handleChunkReceived(value);`
`59`	`59`	`return { done: false, value: value.content! };`
`60`	`60`	`}`
`61`		`- } catch (error) {`
`62`		`- log.error('error processing stream update', error);`
	`61`	`+ } catch (error: any) {`
	`62`	`+ log.error('error processing stream update: %s', error);`
`63`	`63`	`return { done: true, value: undefined };`
`64`	`64`	`}`
`65`	`65`	`},`
`@@ -135,8 +135,8 @@ export class TextStreamReader extends BaseStreamReader<TextStreamInfo> {`
`135`	`135`	`value: decoder.decode(value.content!),`
`136`	`136`	`};`
`137`	`137`	`}`
`138`		`- } catch (error) {`
`139`		`- log.error('error processing stream update', error);`
	`138`	`+ } catch (error: any) {`
	`139`	`+ log.error('error processing stream update: %s', error);`
`140`	`140`	`return { done: true, value: undefined };`
`141`	`141`	`}`
`142`	`142`	`},`