cleanup comments

Author: stephen-derosa

bridge/README.md

@@ -57,7 +57,6 @@ bridge.unregisterRpcMethod("greet");
 //    Controller side: send commands to the publisher
 controller_bridge.requestRemoteTrackMute("robot-1", "mic");    // mute audio track "mic"
 controller_bridge.requestRemoteTrackUnmute("robot-1", "mic");  // unmute it
-controller_bridge.requestTrackRelease("robot-1", "cam"); // unpublish video track "cam"
 
 // 7. Cleanup is automatic (RAII), or explicit:
 mic.reset();       // unpublishes the audio track
@@ -158,7 +157,6 @@ bridge.connect(url, token, options);
 | `unregisterRpcMethod(method_name)` | Unregister a previously registered RPC handler. |
 | `requestRemoteTrackMute(identity, track_name)` | Ask a remote participant to mute a track by name. Throws `livekit::RpcError` on failure. |
 | `requestRemoteTrackUnmute(identity, track_name)` | Ask a remote participant to unmute a track by name. Throws `livekit::RpcError` on failure. |
-| `requestTrackRelease(identity, track_name)` | Ask a remote participant to release (unpublish) a track by name. Throws `livekit::RpcError` on failure. |
 
 ### `BridgeAudioTrack`
 

bridge/include/livekit_bridge/livekit_bridge.h

@@ -402,7 +402,7 @@ class LiveKitBridge {
                                const std::shared_ptr<livekit::Track> &track,
                                VideoFrameCallback cb);
 
-  /// Execute a track action (mute/unmute/release) by track name.
+  /// Execute a track action (mute/unmute) by track name.
   /// Used as the TrackActionFn callback for RpcController.
   /// Throws livekit::RpcError if the track is not found.
   /// @pre Caller does NOT hold mutex_ (acquires it internally).

bridge/tests/integration/test_bridge_rpc_roundtrip.cpp

@@ -234,42 +234,6 @@ TEST_F(BridgeRemoteTrackControlTest, RemoteMuteVideoTrack) {
   video_track->release();
 }
 
-// ---------------------------------------------------------------------------
-// Test 6: Remote release of a data track.
-//
-// Data tracks have no mute/unmute, only release.
-// ---------------------------------------------------------------------------
-TEST_F(BridgeRemoteTrackControlTest, RemoteReleaseDataTrack) {
-  skipIfNotConfigured();
-
-  std::cout << "\n=== Bridge Remote Release Data Track Test ===" << std::endl;
-
-  LiveKitBridge publisher;
-  LiveKitBridge controller;
-
-  ASSERT_TRUE(connectPair(controller, publisher));
-
-  const std::string publisher_identity = "rpc-receiver";
-
-  auto data_track = publisher.createDataTrack("sensor-data");
-  ASSERT_NE(data_track, nullptr);
-  ASSERT_TRUE(data_track->isPublished());
-
-  std::this_thread::sleep_for(2s);
-
-  std::cout << "Requesting release on data track..." << std::endl;
-  EXPECT_NO_THROW(
-      controller.requestTrackRelease(publisher_identity, "sensor-data"));
-
-  std::this_thread::sleep_for(500ms);
-
-  EXPECT_TRUE(data_track->isReleased());
-
-  std::vector<std::uint8_t> payload{0x01, 0x02};
-  bool pushed = data_track->pushFrame(payload);
-  EXPECT_FALSE(pushed) << "pushFrame should fail after remote release";
-}
-
 // ---------------------------------------------------------------------------
 // Test 7: Remote mute on a nonexistent track returns an error.
 // ---------------------------------------------------------------------------