[bug] Update reference from obsolete to active

Update TODOs to link from the obsolete https://fxbug.dev/42177573 to the
active https://fxbug.dev/296283299 for optional capability discovery.

While modifying these, change the formatting to use the
`TODO(https://fxbug.dev/XXX): DESCRIPTION` format.

Bug: 296283299
Change-Id: Id51743bad4a8892afc41a1606af9b02dbe522b03
Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/1190174
Reviewed-by: Sen Jiang <[email protected]>
Commit-Queue: Antonio Linhart <[email protected]>
Reviewed-by: Bruno Dal Bo <[email protected]>
API-Review: Bruno Dal Bo <[email protected]>
Reviewed-by: Colin Nelson <[email protected]>
Reviewed-by: Claire Gonyeo <[email protected]>

Author: antoniolinhart

sdk/fidl/fuchsia.net.dhcp/client.fidl

@@ -86,11 +86,11 @@ closed protocol ClientProvider {
 
     /// No-op method that allows checking for presence.
     ///
-    /// It's not currently possible for a client with an optionally-provided
-    /// protocol to check whether there's someone on the other end without
-    /// making a FIDL call (https://fxbug.dev/42177573). This method provides a
-    /// workaround by giving a client a method that it can call to check for
-    /// liveness.
+    /// TODO(https://fxbug.dev/296283299): It's not currently possible for a
+    /// client with an optionally-provided protocol to check whether there's
+    /// someone on the other end without making a FIDL call . This method
+    /// provides a workaround by giving a client a method that it can call to
+    /// check for liveness.
     ///
     /// TODO(https://fxbug.dev/42076541): Remove this once the DHCP out-of-stack
     /// client is always being used.

src/developer/remote-control/src/main.rs

@@ -93,8 +93,8 @@ async fn exec_server(config: &Config) -> Result<(), Error> {
     let weak_router = Arc::downgrade(&router);
     std::mem::drop(router);
     let usb_fut = async move {
-        // TODO: Change this info! to error! Once we can return normally if USB support is disabled
-        // (https://fxbug.dev/42177573)
+        // TODO(https://fxbug.dev/296283299): Change this info! to error! Once
+        // we can return normally if USB support is disabled
         if let Err(e) = usb::run_usb_links(weak_router.clone()).await {
             info!("USB scanner failed with error {e:?}");
         }

src/sys/tools/component/src/lib.rs

@@ -51,10 +51,11 @@ pub async fn exec() -> Result<()> {
             resolve_cmd(args.query, lifecycle_controller, realm_query, writer).await
         }
         ComponentSubcommand::Explore(args) => {
-            // TODO(https://fxbug.dev/42177573): Verify that the optional Launcher protocol is available
-            // before connecting.
+            // TODO(https://fxbug.dev/296283299): Verify that the optional Launcher protocol is
+            // available before connecting.
             let dash_launcher = connect_to_protocol::<fdash::LauncherMarker>()?;
-            // TODO(https://fxbug.dev/42077838): Use Stdout::raw instead, when a command is not provided
+            // TODO(https://fxbug.dev/42077838): Use Stdout::raw instead, when a command is
+            // not provided.
             let stdout = Stdout::buffered();
 
             #[allow(clippy::large_futures)]

src/sys/tools/package/src/lib.rs

@@ -14,7 +14,7 @@ pub async fn exec() -> anyhow::Result<()> {
 
     match args.subcommand {
         crate::args::PackageSubcommand::Explore(args) => {
-            // TODO(https://fxbug.dev/42177573): Verify that the optional Launcher protocol is
+            // TODO(https://fxbug.dev/296283299): Verify that the optional Launcher protocol is
             // available before connecting.
             let dash_launcher = connect_to_protocol::<fdash::LauncherMarker>()?;
             // TODO(https://fxbug.dev/42077838): Use Stdout::raw when a command is not provided.